stromgren | As 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 |
---|---|---|
stromgren | Anyway, 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 |
mnasiadka | greatgatsby: 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 |
mnasiadka | stromgren: just create your own fork and backport this downstream? | 07:16 |
stromgren | mnasiadka: Thanks! | 07:21 |
opendevreview | Tadas Sutkaitis proposed openstack/kolla master: fix: build from git source after cadf plugin introduction https://review.opendev.org/c/openstack/kolla/+/922077 | 08:39 |
SvenKieske | mhm, 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.2024 | 08:49 |
SvenKieske | some trouble with wheel building somehow I guess | 08:50 |
SvenKieske | even before wheel building, just installing requirements | 08:50 |
mnasiadka | is that for all images SvenKieske ? | 09:14 |
SvenKieske | it's failing befora kolla-build is even run :) at least locally, just wanted to rule out local error | 09:17 |
SvenKieske | I have a feeling it is a local error, but pip error messages are not helpful sometimes :) | 09:18 |
SvenKieske | testing master now | 09:19 |
SvenKieske | lol? "venv: failed with kolla-build is not allowed, use allowlist_externals to allow it" | 09:21 |
SvenKieske | mhm, 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-development | 09:22 |
SvenKieske | I 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 |
SvenKieske | patched it for now to move on to the actual problem | 09:24 |
SvenKieske | so 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 guess | 09:26 |
mnasiadka | I'm pretty sure we don't support that since a long time | 09:27 |
SvenKieske | :D okay, then I'll just propose a patch to drop the docs for that | 09:28 |
SvenKieske | do you just start a custom venv for testing kolla patches or how do you do it? | 09:29 |
frickler | the issue is that tox doesn't actually install kolla into .tox/venv, only reqs | 09:30 |
frickler | if I manually run pip install, the tox call works | 09:30 |
SvenKieske | that 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 stuff | 09:31 |
SvenKieske | will do it manually for now, just wanted to debug the "build multiple rmq versions" change, originally | 09:32 |
SvenKieske | alright, works now | 09:35 |
opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default https://review.opendev.org/c/openstack/kolla-ansible/+/904959 | 09:37 |
SvenKieske | weird that it fails in CI though: https://review.opendev.org/c/openstack/kolla/+/918974 | 09:38 |
opendevreview | Sven Kieske proposed openstack/kolla master: [docs] remove not working dev setup https://review.opendev.org/c/openstack/kolla/+/922079 | 09:43 |
opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis https://review.opendev.org/c/openstack/kolla-ansible/+/909188 | 09:50 |
opendevreview | Bartosz Bezak proposed openstack/kayobe master: CI: remove SLURP jobs for D cycle https://review.opendev.org/c/openstack/kayobe/+/922080 | 10:22 |
opendevreview | Bartosz Bezak proposed openstack/kayobe master: remove migrate_rabbitmq_queues in D cycle https://review.opendev.org/c/openstack/kayobe/+/922081 | 10:24 |
opendevreview | Bartosz Bezak proposed openstack/kayobe master: remove migrate_rabbitmq_queues in D cycle https://review.opendev.org/c/openstack/kayobe/+/922081 | 10:24 |
opendevreview | Bartosz Bezak proposed openstack/kayobe master: remove migrate_rabbitmq_queues in D cycle https://review.opendev.org/c/openstack/kayobe/+/922081 | 11:03 |
opendevreview | Bartosz Bezak proposed openstack/kayobe master: remove migrate_rabbitmq_queues in D cycle https://review.opendev.org/c/openstack/kayobe/+/922081 | 11:04 |
kevko | \o | 11:06 |
SvenKieske | o/ | 11:21 |
opendevreview | Michal Arbet proposed openstack/kolla master: Add mechanism for patching files in containers https://review.opendev.org/c/openstack/kolla/+/829295 | 11:22 |
opendevreview | Michal Arbet proposed openstack/kolla master: Add macro for patch files to all images https://review.opendev.org/c/openstack/kolla/+/829296 | 11:22 |
opendevreview | Tadas Sutkaitis proposed openstack/kolla master: fix: build from git source after cadf plugin introduction https://review.opendev.org/c/openstack/kolla/+/922077 | 12:22 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Move mariadb deployment before loadbalancer https://review.opendev.org/c/openstack/kolla-ansible/+/922092 | 12:27 |
greatgatsby | mnasiadka: 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/+/921506 | 12:27 |
kevko | greatgatsby: yep :) that's our patch : | 12:49 |
kevko | greatgatsby: :) | 12:49 |
opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default https://review.opendev.org/c/openstack/kolla-ansible/+/904959 | 12:49 |
greatgatsby | kevko: thanks for it! | 12:50 |
kevko | mnasiadka: 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 |
kevko | i 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 |
opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Refactor of docker worker https://review.opendev.org/c/openstack/kolla-ansible/+/908295 | 12:57 |
mnasiadka | kevko: where is the example? | 12:59 |
opendevreview | Dawud proposed openstack/kolla master: Change logic for plugins in grafana image build https://review.opendev.org/c/openstack/kolla/+/913184 | 13:00 |
kevko | mnasiadka: 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 |
kevko | mnasiadka: so in those situations i just take a patch , place into my git and patch my kolla images ... | 13:01 |
mnasiadka | kevko: there are hundreds of important patches | 13:01 |
mnasiadka | which nobody reviews | 13:01 |
kevko | mnasiadka: I know | 13:02 |
opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Refactor of kolla_container_facts https://review.opendev.org/c/openstack/kolla-ansible/+/911417 | 13:08 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: [docs] remove warning about octavia_network_type https://review.opendev.org/c/openstack/kolla-ansible/+/922098 | 13:09 |
opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default https://review.opendev.org/c/openstack/kolla-ansible/+/904959 | 14:04 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2024.1: CI: increase timeout for server resize confirmation https://review.opendev.org/c/openstack/kolla-ansible/+/922120 | 14:46 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2023.2: CI: increase timeout for server resize confirmation https://review.opendev.org/c/openstack/kolla-ansible/+/922121 | 14:47 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2023.1: CI: increase timeout for server resize confirmation https://review.opendev.org/c/openstack/kolla-ansible/+/922122 | 14:47 |
SvenKieske | bbezak: 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 |
SvenKieske | kevko: 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/+/922098 | 15:12 |
*** syd is now known as Guest9937 | 15:21 | |
*** syd is now known as Guest9945 | 16:19 | |
*** syd is now known as Guest9947 | 16:29 | |
ababbitt | Is bifrost introspection failing on 2024.1 for anyone else on enroll? (Kayobe) | 16:58 |
opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default https://review.opendev.org/c/openstack/kolla-ansible/+/904959 | 17:39 |
mnasiadka | ababbitt: are you using ironic-inspector or the ironic built in inspection? | 17:41 |
mnasiadka | SvenKieske: 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 |
opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default https://review.opendev.org/c/openstack/kolla-ansible/+/904959 | 18:15 |
ababbitt | mnasiadka: 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/c79d5cfb8773669b289848ea6427e12a | 18:25 |
ababbitt | Seems like it's enabled but not being used? | 18:25 |
ababbitt | Also 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 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Move image building to a role https://review.opendev.org/c/openstack/kolla/+/922148 | 19:49 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Move image building to a role https://review.opendev.org/c/openstack/kolla/+/922148 | 19:51 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: CI: Add non-voting Python 3.12 job https://review.opendev.org/c/openstack/kayobe/+/920854 | 20:10 |
opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default https://review.opendev.org/c/openstack/kolla-ansible/+/904959 | 23:08 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!