Monday, 2023-05-22

opendevreviewDr. Jens Harbott proposed openstack/kolla-ansible master: CI: Test some more scenarios on debian  https://review.opendev.org/c/openstack/kolla-ansible/+/87913205:12
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373806:40
opendevreviewMichal Nasiadka proposed openstack/kayobe master: Bump up Ansible supported versions to 6.x/7.x  https://review.opendev.org/c/openstack/kayobe/+/87820706:42
opendevreviewMichal Nasiadka proposed openstack/kayobe master: Bump up Ansible supported versions to 6.x/7.x  https://review.opendev.org/c/openstack/kayobe/+/87820706:43
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373807:08
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: baremetal: remove warn: false  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88305307:12
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: baremetal: remove warn: false  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88305307:12
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373807:21
opendevreviewPierre Riteau proposed openstack/kayobe master: [DNM] Test pinning nova-libvirt to yoga image  https://review.opendev.org/c/openstack/kayobe/+/88379107:37
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373807:37
opendevreviewMichal Nasiadka proposed openstack/kayobe master: Bump up Ansible supported versions to 6.x/7.x  https://review.opendev.org/c/openstack/kayobe/+/87820707:44
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373807:48
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373807:50
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373807:55
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373807:57
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373808:03
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373808:04
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373808:10
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373808:25
mnasiadkageez, works08:33
opendevreviewMerged openstack/kayobe master: Add neutron-ovn-agent group to Kolla inventory  https://review.opendev.org/c/openstack/kayobe/+/88374408:52
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: baremetal: remove warn: false  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88305308:55
opendevreviewMark Goddard proposed openstack/kolla stable/yoga: opensearch: move to yum/apt repos  https://review.opendev.org/c/openstack/kolla/+/88371609:06
opendevreviewMark Goddard proposed openstack/kolla stable/yoga: opensearch: move to yum/apt repos  https://review.opendev.org/c/openstack/kolla/+/88371609:08
opendevreviewMark Goddard proposed openstack/kolla stable/yoga: opensearch: move to yum/apt repos  https://review.opendev.org/c/openstack/kolla/+/88371609:10
opendevreviewMark Goddard proposed openstack/kolla-ansible master: Performance: Split out prometheus node exporter and cadvisor  https://review.opendev.org/c/openstack/kolla-ansible/+/74008209:32
mnasiadkafrickler: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/883053/7 - we need that for kolla-ansible rc if you're around10:25
fricklertoo bad it is still failing10:27
mnasiadkaah crap, haven't noticed10:40
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: baremetal: remove warn: false  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88305310:41
mnasiadkanow it's going to pass10:41
mnasiadka(and probably fail on the rocky9 kayobe job doing bare metal tests)10:41
mnasiadkafrickler: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/883738/13 but this one needs your review :)10:48
SvenKieskemgoddard: do you happen to know if stackhpc considers backporting any of the rabbitmq reliability patches, especially the oslo-messaging patches mentioned in this article? https://www.stackhpc.com/rabbitmq-reliability.html if matt can answer that better it would be very kind if you could forward this question.11:01
opendevreviewMark Goddard proposed openstack/kayobe master: Introduce max fail percentage to playbooks  https://review.opendev.org/c/openstack/kayobe/+/81828811:20
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373811:41
opendevreviewMark Goddard proposed openstack/kayobe master: Introduce max fail percentage to playbooks  https://review.opendev.org/c/openstack/kayobe/+/81828811:42
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373811:43
opendevreviewMark Goddard proposed openstack/kolla-ansible master: Support Ansible max_fail_percentage  https://review.opendev.org/c/openstack/kolla-ansible/+/80559811:45
opendevreviewMark Goddard proposed openstack/kayobe master: Introduce max fail percentage to playbooks  https://review.opendev.org/c/openstack/kayobe/+/81828811:45
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: baremetal: remove warn: false  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88305311:48
opendevreviewMark Goddard proposed openstack/kolla-ansible master: Support Ansible max_fail_percentage  https://review.opendev.org/c/openstack/kolla-ansible/+/80559812:00
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373812:10
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: baremetal: remove warn: false  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88305312:15
SvenKieskemattcrees: are there any plans on backporting the oslo messaging patches on your side, regarding your awesome rabbitmq blog post? :)12:22
mattcreesSvenKieske: looks like there's backports proposed for the oslo.messaging patch already, back as far as Xena: https://review.opendev.org/c/openstack/oslo.messaging/+/88353912:24
SvenKieskeyeah, I saw these. is your company involved there or do you maybe happen to now if they will get merged eventually? or should I ask in oslo project? we're interested in these backports to happen.12:25
SvenKieskejudging by the involved mails/people it seems the answer is that you're not involved :)12:26
SvenKieskejust wanted to double check12:26
mattcreesWe're not involved in this really, but given the regression exists from a previous backport to Xena I'd assume these patches will get merged12:27
SvenKieskeseems reasonable, I asked on the bug report; I also asked if it can get reassigned to the correct component, because it's still assigned to charmed-rabbitmq-server; I don't seem to be able to change that myself.12:31
mnasiadkait's assigned to 4 different projects, but one of them is oslo.messaging, so I guess that's ok ;)12:34
mnasiadkathere's also https://review.opendev.org/c/openstack/oslo.messaging/+/876318 and it's backports12:35
greatgatsbymattcrees: can you share the link to the rabbitmq blog post here please?12:35
mattcreesSure thing: https://www.stackhpc.com/rabbitmq-reliability.html12:35
SvenKieskeah, right, didn't know that was a thing in launchpad; I'm more familiar with bugzilla, where this is not possible, interesting.12:36
greatgatsbythanks, always looking for anything to improve rabbitmq stability12:36
SvenKieskereally excellent blogpost! even as I was aware of the whole situation it's good to have all the patches and issues streamlined in one place to lookup. It's rather complicated to keep in one12:36
SvenKieske's head the whole mess.12:37
mnasiadkahttps://review.opendev.org/c/openstack/oslo.messaging/+/87561512:37
mnasiadkathat one is also interesting12:37
mnasiadkaso many years of running RMQ, and still bugs :)12:37
SvenKieskeI'm personally still not sure about the reliability vs performance tradeoff12:38
SvenKieskeI almost always favor reliability12:38
greatgatsbyour team as well.  I'm not sure what the consequences of reduced performance are, but having other services failing due to rabbitmq is troubling12:45
opendevreviewMark Goddard proposed openstack/kolla-ansible master: Performance: Split out prometheus node exporter and cadvisor  https://review.opendev.org/c/openstack/kolla-ansible/+/74008212:46
SvenKieskecan someone comment on https://review.opendev.org/c/openstack/kolla-ansible/+/878270/13/ansible/roles/neutron/templates/neutron.conf.j2#67 ?12:47
SvenKieskenot sure if we want some redundant checks to make them more explicit, I would vouch for "no", but I'm the newbie here :)12:48
opendevreviewPierre Riteau proposed openstack/kayobe master: CI: Enable bare metal testing for Ubuntu upgrades  https://review.opendev.org/c/openstack/kayobe/+/88383512:51
mnasiadkathat's too complicated for reading :D12:51
opendevreviewPierre Riteau proposed openstack/kayobe master: CI: Enable bare metal testing for Ubuntu upgrades  https://review.opendev.org/c/openstack/kayobe/+/88383512:53
SvenKieskeI replied in the review, hopefully that explains what I mean more clearly.12:54
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373812:55
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: baremetal: remove warn: false  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88305312:55
opendevreviewPierre Riteau proposed openstack/kayobe master: CI: Disable bare metal testing on RL9/c9s  https://review.opendev.org/c/openstack/kayobe/+/88383612:57
opendevreviewVerification of a change to openstack/kolla-ansible stable/xena failed: Add flags for RabbitMQ message TTL & queue expiry  https://review.opendev.org/c/openstack/kolla-ansible/+/88031514:09
SvenKieskeis the bot broken? above message seems not to be correct? the verification of the change didn't fail?14:56
fricklerSvenKieske: it failed earlier and then the arm64 results came in while there still was a V-2 on the patch, causing this message to be generated, this is a known issue15:00
SvenKieskeah okay; yeah I saw the failure, but thought that it can't be about that failure, given the time delay. weird.15:02
opendevreviewMerged openstack/ansible-collection-kolla master: Bump ansible-lint  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88373815:16
mnasiadkafrickler: can you use your powers to get https://review.opendev.org/c/opendev/system-config/+/883467 merged? Currently we're not using UCA local mirrors15:25
mnasiadkaI would be grateful15:25
opendevreviewMerged openstack/ansible-collection-kolla master: baremetal: remove warn: false  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/88305315:33
opendevreviewMerged openstack/kolla-ansible stable/xena: Add flags for RabbitMQ message TTL & queue expiry  https://review.opendev.org/c/openstack/kolla-ansible/+/88031515:36
opendevreviewMark Goddard proposed openstack/kayobe master: Remove firewall-debian role  https://review.opendev.org/c/openstack/kayobe/+/88385815:46
fricklermnasiadka: done. did you see my comment on https://review.opendev.org/c/opendev/system-config/+/883468 ?17:22
opendevreviewMerged openstack/kayobe master: CI: Enable bare metal testing for Ubuntu upgrades  https://review.opendev.org/c/openstack/kayobe/+/88383517:34
opendevreviewVerification of a change to openstack/kayobe master failed: CI: Disable bare metal testing on RL9/c9s  https://review.opendev.org/c/openstack/kayobe/+/88383617:34
mnasiadkafrickler: yes, will update tomorrow18:00
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Show an error when *_bridge_ports is not a list  https://review.opendev.org/c/openstack/kayobe/+/88386718:48
opendevreviewPierre Riteau proposed openstack/kayobe stable/zed: CI: Enable bare metal testing for Ubuntu upgrades  https://review.opendev.org/c/openstack/kayobe/+/88372718:51
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Show an error when *_bridge_ports is not a list  https://review.opendev.org/c/openstack/kayobe/+/88386719:05
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Show an error when *_bridge_ports is not a list  https://review.opendev.org/c/openstack/kayobe/+/88386719:16
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Show an error when *_bridge_ports is not a list  https://review.opendev.org/c/openstack/kayobe/+/88386720:32
opendevreviewMerged openstack/kayobe master: CI: Disable bare metal testing on RL9/c9s  https://review.opendev.org/c/openstack/kayobe/+/88383622:15
opendevreviewMerged openstack/kayobe master: CI: Remove disable_selinux_do_reboot flag  https://review.opendev.org/c/openstack/kayobe/+/88337222:15
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Show an error when *_bridge_ports is not a list  https://review.opendev.org/c/openstack/kayobe/+/88386723:42
*** gregoryo is now known as clayjar23:51

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