mnasiadka | SvenKieske: we usually add a check if we're running with limit - then we fail if it makes sense (like in the keystone role) | 07:59 |
---|---|---|
mnasiadka | or we could rework it so it gets run on the first host in ansible_play_batch | 08:00 |
opendevreview | Christian Berendt proposed openstack/kolla master: prometheus: use inovex/prometheus-libvirt-exporter https://review.opendev.org/c/openstack/kolla/+/909507 | 08:02 |
mnasiadka | frickler, kevko, bbezak: would like to get some reviews on https://review.opendev.org/c/openstack/kolla/+/908989 if possible | 08:13 |
opendevreview | Christian Berendt proposed openstack/kolla master: prometheus: use inovex/prometheus-libvirt-exporter https://review.opendev.org/c/openstack/kolla/+/909507 | 08:23 |
opendevreview | Uwe Jäger proposed openstack/kolla-ansible master: Allow overriding of Skyline logos https://review.opendev.org/c/openstack/kolla-ansible/+/909481 | 08:28 |
opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis https://review.opendev.org/c/openstack/kolla-ansible/+/909188 | 08:55 |
SvenKieske | mnasiadka: thanks for the pointer | 09:33 |
halomiva | hey, i want to ask about bumping version of docker-py because some tests will fail on cgroupns which doesnt exists until 6.0 ( like this one https://zuul.opendev.org/t/openstack/build/a124e9716d284baa8dc71cfb08cd8ff0/logs ) and also I couldn't find minimal version of docker that is required to run kolla | 09:36 |
opendevreview | Will Szumski proposed openstack/kolla-ansible master: Adds the neutron_service_limit option https://review.opendev.org/c/openstack/kolla-ansible/+/877776 | 09:48 |
opendevreview | Will Szumski proposed openstack/kolla-ansible master: Adds the neutron_service_limit option https://review.opendev.org/c/openstack/kolla-ansible/+/877776 | 09:49 |
AntonyC | Hi all | 10:34 |
AntonyC | I'm trying out the new om_enable_rabbitmq_quorum_queues in the stable/2023.1 branch of kolla-ansible and I'm encountering some problems with just ironic-neutron-agent. I'm on Ubuntu and I'm building the containers using kolla-build. When I enable this feature the Ironic Neutron agent tries to create a new quorum queue with the auto-delete property which causes it to fail and then retry in a loop. Has anyone | 10:40 |
AntonyC | else seen this? | 10:40 |
SvenKieske | AntonyC: yeah, I have a patch in that direction floating around I guess, wait a second.. | 10:52 |
mattcrees | AntonyC: Sounds like you're hitting this bug: https://bugs.launchpad.net/networking-baremetal/+bug/2046962 There's an open patch linked in there, and Michal has suggested a workaround in KA config too to disable auto_delete in the agent | 10:53 |
SvenKieske | AntonyC: https://review.opendev.org/c/openstack/networking-baremetal/+/903995 | 10:53 |
SvenKieske | that's some motivation to finally finish that patch I guess, need to write some docs and bump min requirements version afaik | 10:53 |
SvenKieske | but maybe we want the workaround as well? not sure | 10:54 |
mattcrees | If your bugfix gets merged+backported quickly, then I don't think we'd need the workaround. Depends how fast the review velocity is for the neutron-baremetal project :) | 10:56 |
AntonyC | I'm willing to help test if that helps | 10:57 |
AntonyC | Thanks for the link I'm trying to apply this patch to my setup | 11:03 |
SvenKieske | you might need a newer oslo-messaging lib for that to work, currently figuring out which version is needed | 11:04 |
AntonyC | I'm seeing this "2024-02-20 11:19:25.312 7 ERROR neutron oslo_config.cfg.NoSuchOptError: no such option rabbit_quorum_queues in group [oslo_messaging_rabbit] | 11:20 |
AntonyC | " so I think you are right | 11:20 |
SvenKieske | you need at least oslo.messaging >= 12.13.0 | 11:21 |
AntonyC | is that already in the bobcat release? | 11:33 |
SvenKieske | yes, this is in yoga already | 11:37 |
SvenKieske | https://docs.openstack.org/releasenotes/oslo.messaging/yoga.html | 11:37 |
SvenKieske | the requirements in networking-baremetal were not bumped in a long time it seems | 11:37 |
SvenKieske | I'll update the patch in a few minutes, once my release notes are finished and all linters are satisfied :) | 11:38 |
AntonyC | hmmm yes I'm on oslo.messaging==14.2.4 | 11:42 |
AntonyC | in my neutron .conf the option is called rabbit_quorum_queue not rabbit_quorum_queues | 11:45 |
AntonyC | is this where this property is being set from? | 11:45 |
opendevreview | Merged openstack/kolla master: Remove the Prometheus HAProxy exporter image https://review.opendev.org/c/openstack/kolla/+/909124 | 11:55 |
AntonyC | With that change the patch works for me, Thank you Sven | 12:09 |
opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis https://review.opendev.org/c/openstack/kolla-ansible/+/909188 | 12:23 |
opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis https://review.opendev.org/c/openstack/kolla-ansible/+/909188 | 12:25 |
SvenKieske | AntonyC: were did you get that "rabbit_quorum_queues" instead of "rabbit_quorum_queue" from? was that a typo in your config or something upstream? | 12:31 |
AntonyC | I got it from here https://review.opendev.org/c/openstack/networking-baremetal/+/903995/1/networking_baremetal/agent/ironic_neutron_agent.py#56 | 12:31 |
AntonyC | that has "rabbit_quorum_queues" | 12:32 |
opendevreview | Rafal Lewandowski proposed openstack/kayobe master: Add Redfish rules to Ironic and Bifrost introspection https://review.opendev.org/c/openstack/kayobe/+/902772 | 12:32 |
AntonyC | but kolla-ansible is adding this in the neutron.conf | 12:33 |
AntonyC | root@control0001:~# grep quorum /etc/kolla/ironic-neutron-agent/neutron.conf | 12:33 |
AntonyC | rabbit_quorum_queue = true | 12:33 |
SvenKieske | ah thx, that seems to be a typo on my side | 12:35 |
SvenKieske | AntonyC: this was corrected already in the second patchset from 19.12.2023 though? https://review.opendev.org/c/openstack/networking-baremetal/+/903995/1..2/networking_baremetal/agent/ironic_neutron_agent.py | 12:37 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: WIP: Fixes destroys https://review.opendev.org/c/openstack/kolla-ansible/+/668288 | 12:44 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Move curl sources to sources.py https://review.opendev.org/c/openstack/kolla/+/908989 | 12:44 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Move curl sources to sources.py https://review.opendev.org/c/openstack/kolla/+/908989 | 12:46 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Move curl sources to sources.py https://review.opendev.org/c/openstack/kolla/+/908989 | 12:52 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Use py3 as the default runtime for tox https://review.opendev.org/c/openstack/kolla/+/890386 | 13:35 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Use py3 as the default runtime for tox https://review.opendev.org/c/openstack/kolla/+/890386 | 13:36 |
AntonyC | oh sorry Sven, I just followed the link you sent earlier and didn't think to dig any further so I didn't see that second patchset. That does work perfectly when backported to 2023.1 as well | 13:37 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Bump ansible-core versions to 2.15 and 2.16 https://review.opendev.org/c/openstack/kolla-ansible/+/907522 | 13:48 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: doc: Add a note about SLURP upgrades https://review.opendev.org/c/openstack/kolla-ansible/+/909539 | 13:56 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: doc: Add a note about SLURP upgrades https://review.opendev.org/c/openstack/kolla-ansible/+/909539 | 13:57 |
opendevreview | Merged openstack/kolla-ansible master: cinder: Stop using admin service token https://review.opendev.org/c/openstack/kolla-ansible/+/908405 | 14:24 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Move curl sources to sources.py https://review.opendev.org/c/openstack/kolla/+/908989 | 14:41 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2023.2: cinder: Stop using admin service token https://review.opendev.org/c/openstack/kolla-ansible/+/909542 | 14:48 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2023.1: cinder: Stop using admin service token https://review.opendev.org/c/openstack/kolla-ansible/+/909543 | 14:49 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/zed: cinder: Stop using admin service token https://review.opendev.org/c/openstack/kolla-ansible/+/909544 | 14:49 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/zed: cinder: Stop using admin service token https://review.opendev.org/c/openstack/kolla-ansible/+/909544 | 14:50 |
dawud1_ | Would any non-stackhpc core be able to review this please so that we can make some progress on it? https://review.opendev.org/c/openstack/kolla-ansible/+/899136 perhaps fickler or kevko ? Thank you | 14:59 |
dawud1_ | frickler * | 15:02 |
opendevreview | Will Szumski proposed openstack/kayobe master: Add vagrant-libvirt support https://review.opendev.org/c/openstack/kayobe/+/703667 | 15:07 |
opendevreview | Will Szumski proposed openstack/kayobe master: Add proxy support to Vagrantfile https://review.opendev.org/c/openstack/kayobe/+/703666 | 15:07 |
opendevreview | Christian Berendt proposed openstack/kolla master: Use openstack-exporter 1.7.0 https://review.opendev.org/c/openstack/kolla/+/909591 | 15:17 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Bump ansible-core versions to 2.15 and 2.16 https://review.opendev.org/c/openstack/kolla-ansible/+/907522 | 15:30 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Move curl sources to sources.py https://review.opendev.org/c/openstack/kolla/+/908989 | 15:33 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Move curl sources to sources.py https://review.opendev.org/c/openstack/kolla/+/908989 | 15:35 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: doc: Add a note about SLURP upgrades https://review.opendev.org/c/openstack/kolla-ansible/+/909539 | 15:37 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: doc: Add a note about SLURP upgrades https://review.opendev.org/c/openstack/kolla-ansible/+/909539 | 15:39 |
opendevreview | Will Szumski proposed openstack/kayobe master: Fix issue with overcloud service destroy https://review.opendev.org/c/openstack/kayobe/+/909594 | 15:39 |
opendevreview | Will Szumski proposed openstack/kayobe master: Fix issue removing docker volumes https://review.opendev.org/c/openstack/kayobe/+/909594 | 15:40 |
opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis https://review.opendev.org/c/openstack/kolla-ansible/+/909188 | 15:50 |
opendevreview | Will Szumski proposed openstack/kayobe master: Fix typo in example https://review.opendev.org/c/openstack/kayobe/+/909597 | 15:53 |
opendevreview | Will Szumski proposed openstack/kayobe master: Add vagrant-libvirt support https://review.opendev.org/c/openstack/kayobe/+/703667 | 16:59 |
opendevreview | Will Szumski proposed openstack/kayobe master: Add proxy support to Vagrantfile https://review.opendev.org/c/openstack/kayobe/+/703666 | 16:59 |
opendevreview | Will Szumski proposed openstack/kayobe master: Support customising telegraf configuration files https://review.opendev.org/c/openstack/kayobe/+/909600 | 17:07 |
opendevreview | Merged openstack/kolla-ansible master: Remove the `grafana` volume https://review.opendev.org/c/openstack/kolla-ansible/+/899136 | 17:25 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!