Tuesday, 2024-02-20

mnasiadkaSvenKieske: 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
mnasiadkaor we could rework it so it gets run on the first host in ansible_play_batch08:00
opendevreviewChristian Berendt proposed openstack/kolla master: prometheus: use inovex/prometheus-libvirt-exporter  https://review.opendev.org/c/openstack/kolla/+/90950708:02
mnasiadkafrickler, kevko, bbezak: would like to get some reviews on https://review.opendev.org/c/openstack/kolla/+/908989 if possible08:13
opendevreviewChristian Berendt proposed openstack/kolla master: prometheus: use inovex/prometheus-libvirt-exporter  https://review.opendev.org/c/openstack/kolla/+/90950708:23
opendevreviewUwe Jäger proposed openstack/kolla-ansible master: Allow overriding of Skyline logos  https://review.opendev.org/c/openstack/kolla-ansible/+/90948108:28
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis  https://review.opendev.org/c/openstack/kolla-ansible/+/90918808:55
SvenKieskemnasiadka: thanks for the pointer09:33
halomivahey, 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 kolla09:36
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Adds the neutron_service_limit option  https://review.opendev.org/c/openstack/kolla-ansible/+/87777609:48
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Adds the neutron_service_limit option  https://review.opendev.org/c/openstack/kolla-ansible/+/87777609:49
AntonyCHi all10:34
AntonyCI'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
AntonyCelse seen this?10:40
SvenKieskeAntonyC: yeah, I have a patch in that direction floating around I guess, wait a second..10:52
mattcreesAntonyC: 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 agent10:53
SvenKieskeAntonyC: https://review.opendev.org/c/openstack/networking-baremetal/+/90399510:53
SvenKieskethat's some motivation to finally finish that patch I guess, need to write some docs and bump min requirements version afaik10:53
SvenKieskebut maybe we want the workaround as well? not sure10:54
mattcreesIf 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
AntonyCI'm willing to help test if that helps10:57
AntonyCThanks for the link I'm trying to apply this patch to my setup11:03
SvenKieskeyou might need a newer oslo-messaging lib for that to work, currently figuring out which version is needed11:04
AntonyCI'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 right11:20
SvenKieskeyou need at least oslo.messaging >= 12.13.011:21
AntonyCis that already in the bobcat release?11:33
SvenKieskeyes, this is in yoga already11:37
SvenKieskehttps://docs.openstack.org/releasenotes/oslo.messaging/yoga.html11:37
SvenKieskethe requirements in networking-baremetal were not bumped in a long time it seems11:37
SvenKieskeI'll update the patch in a few minutes, once my release notes are finished and all linters are satisfied :)11:38
AntonyChmmm yes I'm on oslo.messaging==14.2.411:42
AntonyCin my neutron .conf the option is called rabbit_quorum_queue not rabbit_quorum_queues11:45
AntonyC is this where this property is being set from?11:45
opendevreviewMerged openstack/kolla master: Remove the Prometheus HAProxy exporter image  https://review.opendev.org/c/openstack/kolla/+/90912411:55
AntonyCWith that change the patch works for me, Thank you Sven12:09
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis  https://review.opendev.org/c/openstack/kolla-ansible/+/90918812:23
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis  https://review.opendev.org/c/openstack/kolla-ansible/+/90918812:25
SvenKieskeAntonyC: 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
AntonyCI got it from here https://review.opendev.org/c/openstack/networking-baremetal/+/903995/1/networking_baremetal/agent/ironic_neutron_agent.py#5612:31
AntonyCthat has "rabbit_quorum_queues"12:32
opendevreviewRafal Lewandowski proposed openstack/kayobe master: Add Redfish rules to Ironic and Bifrost introspection  https://review.opendev.org/c/openstack/kayobe/+/90277212:32
AntonyCbut kolla-ansible is adding this in the neutron.conf 12:33
AntonyCroot@control0001:~# grep quorum /etc/kolla/ironic-neutron-agent/neutron.conf12:33
AntonyCrabbit_quorum_queue = true12:33
SvenKieskeah thx, that seems to be a typo on my side12:35
SvenKieskeAntonyC: 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.py12:37
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: WIP: Fixes destroys  https://review.opendev.org/c/openstack/kolla-ansible/+/66828812:44
opendevreviewMichal Nasiadka proposed openstack/kolla master: Move curl sources to sources.py  https://review.opendev.org/c/openstack/kolla/+/90898912:44
opendevreviewMichal Nasiadka proposed openstack/kolla master: Move curl sources to sources.py  https://review.opendev.org/c/openstack/kolla/+/90898912:46
opendevreviewMichal Nasiadka proposed openstack/kolla master: Move curl sources to sources.py  https://review.opendev.org/c/openstack/kolla/+/90898912:52
opendevreviewMichal Nasiadka proposed openstack/kolla master: Use py3 as the default runtime for tox  https://review.opendev.org/c/openstack/kolla/+/89038613:35
opendevreviewMichal Nasiadka proposed openstack/kolla master: Use py3 as the default runtime for tox  https://review.opendev.org/c/openstack/kolla/+/89038613:36
AntonyCoh 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 well13:37
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Bump ansible-core versions to 2.15 and 2.16  https://review.opendev.org/c/openstack/kolla-ansible/+/90752213:48
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: doc: Add a note about SLURP upgrades  https://review.opendev.org/c/openstack/kolla-ansible/+/90953913:56
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: doc: Add a note about SLURP upgrades  https://review.opendev.org/c/openstack/kolla-ansible/+/90953913:57
opendevreviewMerged openstack/kolla-ansible master: cinder: Stop using admin service token  https://review.opendev.org/c/openstack/kolla-ansible/+/90840514:24
opendevreviewMichal Nasiadka proposed openstack/kolla master: Move curl sources to sources.py  https://review.opendev.org/c/openstack/kolla/+/90898914:41
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/2023.2: cinder: Stop using admin service token  https://review.opendev.org/c/openstack/kolla-ansible/+/90954214:48
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/2023.1: cinder: Stop using admin service token  https://review.opendev.org/c/openstack/kolla-ansible/+/90954314:49
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/zed: cinder: Stop using admin service token  https://review.opendev.org/c/openstack/kolla-ansible/+/90954414:49
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/zed: cinder: Stop using admin service token  https://review.opendev.org/c/openstack/kolla-ansible/+/90954414: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 you14:59
dawud1_frickler *15:02
opendevreviewWill Szumski proposed openstack/kayobe master: Add vagrant-libvirt support  https://review.opendev.org/c/openstack/kayobe/+/70366715:07
opendevreviewWill Szumski proposed openstack/kayobe master: Add proxy support to Vagrantfile  https://review.opendev.org/c/openstack/kayobe/+/70366615:07
opendevreviewChristian Berendt proposed openstack/kolla master: Use openstack-exporter 1.7.0  https://review.opendev.org/c/openstack/kolla/+/90959115:17
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Bump ansible-core versions to 2.15 and 2.16  https://review.opendev.org/c/openstack/kolla-ansible/+/90752215:30
opendevreviewMichal Nasiadka proposed openstack/kolla master: Move curl sources to sources.py  https://review.opendev.org/c/openstack/kolla/+/90898915:33
opendevreviewMichal Nasiadka proposed openstack/kolla master: Move curl sources to sources.py  https://review.opendev.org/c/openstack/kolla/+/90898915:35
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: doc: Add a note about SLURP upgrades  https://review.opendev.org/c/openstack/kolla-ansible/+/90953915:37
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: doc: Add a note about SLURP upgrades  https://review.opendev.org/c/openstack/kolla-ansible/+/90953915:39
opendevreviewWill Szumski proposed openstack/kayobe master: Fix issue with overcloud service destroy  https://review.opendev.org/c/openstack/kayobe/+/90959415:39
opendevreviewWill Szumski proposed openstack/kayobe master: Fix issue removing docker volumes  https://review.opendev.org/c/openstack/kayobe/+/90959415:40
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis  https://review.opendev.org/c/openstack/kolla-ansible/+/90918815:50
opendevreviewWill Szumski proposed openstack/kayobe master: Fix typo in example  https://review.opendev.org/c/openstack/kayobe/+/90959715:53
opendevreviewWill Szumski proposed openstack/kayobe master: Add vagrant-libvirt support  https://review.opendev.org/c/openstack/kayobe/+/70366716:59
opendevreviewWill Szumski proposed openstack/kayobe master: Add proxy support to Vagrantfile  https://review.opendev.org/c/openstack/kayobe/+/70366616:59
opendevreviewWill Szumski proposed openstack/kayobe master: Support customising telegraf configuration files  https://review.opendev.org/c/openstack/kayobe/+/90960017:07
opendevreviewMerged openstack/kolla-ansible master: Remove the `grafana` volume  https://review.opendev.org/c/openstack/kolla-ansible/+/89913617:25

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