Monday, 2024-06-17

stromgrenAs you might remember I asked last week about downgrading from the master branch to 2024.1 and as SvenKieske said the downgrade did not magically work, so I ended up doing a complete re-deploy.06:25
stromgrenAnyway, I need this patch (https://review.opendev.org/c/openstack/kolla-ansible/+/907166) for update External Ceph. What would be the best way of doing this in your opinion, to avoid as many issues as possible in the future?06:26
mnasiadkagreatgatsby: since Caracal or Bobcat we're doing rolling upgrades for RabbitMQ - before that, we shut down all of them at the same time - so if it's before that change - nothing will help :)07:16
mnasiadkastromgren: just create your own fork and backport this downstream?07:16
stromgrenmnasiadka: Thanks!07:21
opendevreviewTadas Sutkaitis proposed openstack/kolla master: fix: build from git source after cadf plugin introduction  https://review.opendev.org/c/openstack/kolla/+/92207708:39
SvenKieskemhm, does building kolla images work for you on stable/2023.1 branch? I get a strange error "AttributeError: cython_sources" always. last successful CI run was on 5.6.202408:49
SvenKieskesome trouble with wheel building somehow I guess08:50
SvenKieskeeven before wheel building, just installing requirements08:50
mnasiadkais that for all images SvenKieske ?09:14
SvenKieskeit's failing befora kolla-build is even run :) at least locally, just wanted to rule out local error09:17
SvenKieskeI have a feeling it is a local error, but pip error messages are not helpful sometimes :)09:18
SvenKiesketesting master now09:19
SvenKieskelol? "venv: failed with kolla-build is not allowed, use allowlist_externals to allow it"09:21
SvenKieskemhm, is this docs page still up to date? https://docs.openstack.org/kolla/latest/contributor/running-in-development.html#the-recommended-way-to-run-in-development09:22
SvenKieskeI was running "tox -e venv --kolla-build -b debian kolla-toolbox", is that discouraged maybe? it seems it's not really supported by our tox.ini?09:22
SvenKieskepatched it for now to move on to the actual problem09:24
SvenKieskeso I don't hit this dependency/wheel building error on master at least, so might be some dependency issue or something. I'll test kolla CI I guess09:26
mnasiadkaI'm pretty sure we don't support that since a long time09:27
SvenKieske:D okay, then I'll just propose a patch to drop the docs for that09:28
SvenKieskedo you just start a custom venv for testing kolla patches or how do you do it?09:29
fricklerthe issue is that tox doesn't actually install kolla into .tox/venv, only reqs09:30
fricklerif I manually run pip install, the tox call works09:30
SvenKieskethat seems to be about right, I had a suspicion that this might be the culprit. Would still be handy to have a tox env for handling all this stuff09:31
SvenKieskewill do it manually for now, just wanted to debug the "build multiple rmq versions" change, originally09:32
SvenKieskealright, works now09:35
opendevreviewRafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default  https://review.opendev.org/c/openstack/kolla-ansible/+/90495909:37
SvenKieskeweird that it fails in CI though: https://review.opendev.org/c/openstack/kolla/+/91897409:38
opendevreviewSven Kieske proposed openstack/kolla master: [docs] remove not working dev setup  https://review.opendev.org/c/openstack/kolla/+/92207909:43
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis  https://review.opendev.org/c/openstack/kolla-ansible/+/90918809:50
opendevreviewBartosz Bezak proposed openstack/kayobe master: CI: remove SLURP jobs for D cycle  https://review.opendev.org/c/openstack/kayobe/+/92208010:22
opendevreviewBartosz Bezak proposed openstack/kayobe master: remove migrate_rabbitmq_queues in D cycle  https://review.opendev.org/c/openstack/kayobe/+/92208110:24
opendevreviewBartosz Bezak proposed openstack/kayobe master: remove migrate_rabbitmq_queues in D cycle  https://review.opendev.org/c/openstack/kayobe/+/92208110:24
opendevreviewBartosz Bezak proposed openstack/kayobe master: remove migrate_rabbitmq_queues in D cycle  https://review.opendev.org/c/openstack/kayobe/+/92208111:03
opendevreviewBartosz Bezak proposed openstack/kayobe master: remove migrate_rabbitmq_queues in D cycle  https://review.opendev.org/c/openstack/kayobe/+/92208111:04
kevko\o11:06
SvenKieskeo/11:21
opendevreviewMichal Arbet proposed openstack/kolla master: Add mechanism for patching files in containers  https://review.opendev.org/c/openstack/kolla/+/82929511:22
opendevreviewMichal Arbet proposed openstack/kolla master: Add macro for patch files to all images  https://review.opendev.org/c/openstack/kolla/+/82929611:22
opendevreviewTadas Sutkaitis proposed openstack/kolla master: fix: build from git source after cadf plugin introduction  https://review.opendev.org/c/openstack/kolla/+/92207712:22
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Move mariadb deployment before loadbalancer  https://review.opendev.org/c/openstack/kolla-ansible/+/92209212:27
greatgatsbymnasiadka: thanks for the reply regarding the transient queues.  We're just trying to gracefully reboot a controller for OS upgrades, no kolla-ansible ansible upgrade/deploy involved at all.  It seems some services don't gracefully handle those queues not being available, which is what I think this ticket is trying to address: https://review.opendev.org/c/openstack/oslo.messaging/+/92150612:27
kevkogreatgatsby: yep :) that's our patch : 12:49
kevkogreatgatsby: :) 12:49
opendevreviewRafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default  https://review.opendev.org/c/openstack/kolla-ansible/+/90495912:49
greatgatsbykevko: thanks for it!12:50
kevkomnasiadka: btw, another example of simple patch for downstream users who wants to fix their deployments with kolla patch feature here -> https://review.opendev.org/c/openstack/kolla/+/829295/6 12:50
kevkoi will try to backport it to other versions ...but you know ..it's time consuming ..and it won't be in every oslo release .....for this situations i am using ^^ for several versions without problem 12:52
opendevreviewIvan Halomi proposed openstack/kolla-ansible master: Refactor of docker worker  https://review.opendev.org/c/openstack/kolla-ansible/+/90829512:57
mnasiadkakevko: where is the example? 12:59
opendevreviewDawud proposed openstack/kolla master: Change logic for plugins in grafana image build  https://review.opendev.org/c/openstack/kolla/+/91318413:00
kevkomnasiadka: just saying that https://review.opendev.org/c/openstack/kolla/+/829295/6 is quite important patch ... but it takes some time to get into upstream ... 13:00
kevkomnasiadka: so in those situations i just take a patch , place into my git and patch my kolla images ...13:01
mnasiadkakevko: there are hundreds of important patches13:01
mnasiadkawhich nobody reviews13:01
kevkomnasiadka: I know 13:02
opendevreviewIvan Halomi proposed openstack/kolla-ansible master: Refactor of kolla_container_facts  https://review.opendev.org/c/openstack/kolla-ansible/+/91141713:08
opendevreviewSven Kieske proposed openstack/kolla-ansible master: [docs] remove warning about octavia_network_type  https://review.opendev.org/c/openstack/kolla-ansible/+/92209813:09
opendevreviewRafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default  https://review.opendev.org/c/openstack/kolla-ansible/+/90495914:04
opendevreviewSven Kieske proposed openstack/kolla-ansible stable/2024.1: CI: increase timeout for server resize confirmation  https://review.opendev.org/c/openstack/kolla-ansible/+/92212014:46
opendevreviewSven Kieske proposed openstack/kolla-ansible stable/2023.2: CI: increase timeout for server resize confirmation  https://review.opendev.org/c/openstack/kolla-ansible/+/92212114:47
opendevreviewSven Kieske proposed openstack/kolla-ansible stable/2023.1: CI: increase timeout for server resize confirmation  https://review.opendev.org/c/openstack/kolla-ansible/+/92212214:47
SvenKieskebbezak: why -2 for backport of this? https://review.opendev.org/c/openstack/kolla-ansible/+/921846 the new order of nova role seems better also for older releases, no? or do you think it's a too high risk?14:49
SvenKieskekevko: it would really be nice to provide some feedback instead of simply -1 a patch, that's really not helpful https://review.opendev.org/c/openstack/kolla-ansible/+/92209815:12
*** syd is now known as Guest993715:21
*** syd is now known as Guest994516:19
*** syd is now known as Guest994716:29
ababbittIs bifrost introspection failing on 2024.1 for anyone else on enroll? (Kayobe)16:58
opendevreviewRafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default  https://review.opendev.org/c/openstack/kolla-ansible/+/90495917:39
mnasiadkaababbitt: are you using ironic-inspector or the ironic built in inspection?17:41
mnasiadkaSvenKieske: it's not required for backporting, it only fixes issues during conversion to podman - let's not backport things that are not required (re 921846)17:42
opendevreviewRafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default  https://review.opendev.org/c/openstack/kolla-ansible/+/90495918:15
ababbittmnasiadka: New lab build, with a new kayobe-config from 2024.1 branch. Only options set are ipmi username, pass, and lldp interface name. (And ofc network configs so pxe works). So based on that, it should be using ironic-inspector. Logs here: https://gist.github.com/AndrewBabbitt97/c79d5cfb8773669b289848ea6427e12a18:25
ababbittSeems like it's enabled but not being used?18:25
ababbittAlso just note, the node gets registered in ironic but is missing all inspection rule data. Exception Can not transition from state 'enroll' on event 'wait' (no defined transition)18:33
opendevreviewMichal Nasiadka proposed openstack/kolla master: CI: Move image building to a role  https://review.opendev.org/c/openstack/kolla/+/92214819:49
opendevreviewMichal Nasiadka proposed openstack/kolla master: CI: Move image building to a role  https://review.opendev.org/c/openstack/kolla/+/92214819:51
opendevreviewPierre Riteau proposed openstack/kayobe master: CI: Add non-voting Python 3.12 job  https://review.opendev.org/c/openstack/kayobe/+/92085420:10
opendevreviewRafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default  https://review.opendev.org/c/openstack/kolla-ansible/+/90495923:08

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!