Tuesday, 2022-01-18

opendevreviewVerification of a change to openstack/ironic master failed: Set resource_url when getting all ports or portgroups  https://review.opendev.org/c/openstack/ironic/+/82494200:39
opendevreviewSteve Baker proposed openstack/ironic-python-agent-builder stable/wallaby: Replace genisoimage with xorriso  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82336701:01
opendevreviewSteve Baker proposed openstack/ironic-python-agent-builder stable/wallaby: Preliminary work to support CentOS 9 Stream  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82336501:01
opendevreviewSteve Baker proposed openstack/ironic-python-agent-builder stable/wallaby: [DNM] testing dib-centos9 experimental job  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82413601:01
opendevreviewSteve Baker proposed openstack/ironic-python-agent-builder stable/wallaby: Preliminary work to support CentOS 9 Stream  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82336503:26
opendevreviewSteve Baker proposed openstack/ironic-python-agent-builder stable/wallaby: [DNM] testing dib-centos9 experimental job  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82413603:26
opendevreviewMerged openstack/ironic master: Set resource_url when getting all ports or portgroups  https://review.opendev.org/c/openstack/ironic/+/82494204:13
opendevreviewMerged openstack/ironic-python-agent-builder stable/wallaby: Replace genisoimage with xorriso  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82336704:54
arne_wiebalckGood morning, Ironic!07:20
rpittaugood morning ironic! o/08:21
rpittaustevebaker[m]: about python3-virtualenv on cs9, it can be found in epel908:34
arne_wiebalckHey rpittau o/08:35
arne_wiebalckrpittau: https://review.opendev.org/c/openstack/ironic/+/824190 is the original issue and the "same" as the one you reviewed yesterday ... if you have a moment.08:36
stevebaker[m]rpittau: ah that could be an option, if dib can do epel things08:44
rpittauhey arne_wiebalck :)08:58
rpittaustevebaker[m]: normally it can, not sure for cs9 as it works a bit differently https://docs.fedoraproject.org/en-US/epel/#_el908:58
dtantsurgood morning folks08:59
rpittaugood morning dtantsur :)08:59
rpittauarne_wiebalck: done08:59
arne_wiebalckrpittau: thanks!09:23
opendevreviewDmitry Tantsur proposed openstack/sushy master: Stop logging MessageRegistry objects  https://review.opendev.org/c/openstack/sushy/+/82507109:26
opendevreviewRadosław Piliszek proposed openstack/bifrost bugfix/10.0: [CI] Move Kolla testing to CentOS Stream  https://review.opendev.org/c/openstack/bifrost/+/82505310:28
opendevreviewRadosław Piliszek proposed openstack/bifrost bugfix/10.0: [CI] Move Kolla testing to CentOS Stream  https://review.opendev.org/c/openstack/bifrost/+/82505310:28
opendevreviewRadosław Piliszek proposed openstack/bifrost bugfix/10.1: [CI] Move Kolla testing to CentOS Stream  https://review.opendev.org/c/openstack/bifrost/+/82505410:28
opendevreviewMerged openstack/bifrost master: Do not make password files world-readable  https://review.opendev.org/c/openstack/bifrost/+/82470010:36
iurygregorygood morning Ironic o/10:45
opendevreviewMerged openstack/ironic master: Set resource_url when getting all nodes  https://review.opendev.org/c/openstack/ironic/+/82419010:46
dtantsurmorning iurygregory 11:28
opendevreviewDmitry Tantsur proposed openstack/bifrost stable/xena: Do not make password files world-readable  https://review.opendev.org/c/openstack/bifrost/+/82505511:29
opendevreviewDmitry Tantsur proposed openstack/bifrost stable/wallaby: Do not make password files world-readable  https://review.opendev.org/c/openstack/bifrost/+/82505611:29
rpittaudtantsur: we probably need bugfix/10.1 as well, also maybe we should discuss if we want to keep bugfix branches for bifrost11:34
opendevreviewArne Wiebalck proposed openstack/ironic stable/xena: Set resource_url when getting all nodes  https://review.opendev.org/c/openstack/ironic/+/82509011:34
opendevreviewArne Wiebalck proposed openstack/ironic stable/wallaby: Set resource_url when getting all nodes  https://review.opendev.org/c/openstack/ironic/+/82509111:35
dtantsurrpittau: yeah, this is why I haven't proposed a change..11:35
opendevreviewDmitry Tantsur proposed openstack/bifrost bugfix/12.0: Do not make password files world-readable  https://review.opendev.org/c/openstack/bifrost/+/82505711:36
dtantsur12.0 apparently, not 10.1 :)11:36
rpittauoh whoops yeah!11:36
* dtantsur back in a couple of hours11:36
opendevreviewArne Wiebalck proposed openstack/ironic stable/xena: Set resource_url when getting all ports or portgroups  https://review.opendev.org/c/openstack/ironic/+/82509311:38
opendevreviewArne Wiebalck proposed openstack/ironic stable/wallaby: Set resource_url when getting all ports or portgroups  https://review.opendev.org/c/openstack/ironic/+/82509411:38
rpittauarne_wiebalck: can you please also do backports for 19.0 and 18.1 when you get the chance ?11:39
arne_wiebalckrpittau: sure!11:40
rpittauthanks! :)11:40
opendevreviewArne Wiebalck proposed openstack/ironic bugfix/19.0: Set resource_url when getting all nodes  https://review.opendev.org/c/openstack/ironic/+/82509511:41
opendevreviewArne Wiebalck proposed openstack/ironic bugfix/18.1: Set resource_url when getting all nodes  https://review.opendev.org/c/openstack/ironic/+/82509611:41
opendevreviewArne Wiebalck proposed openstack/ironic bugfix/19.0: Set resource_url when getting all ports or portgroups  https://review.opendev.org/c/openstack/ironic/+/82509711:42
opendevreviewSagar Nanda proposed openstack/ironic-tempest-plugin master: Updated the file as per review comments: Deployment using iPXE boot and agent deploy driver permutations driver  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/82509811:43
opendevreviewArne Wiebalck proposed openstack/ironic bugfix/18.1: Set resource_url when getting all ports or portgroups  https://review.opendev.org/c/openstack/ironic/+/82509911:43
arne_wiebalckrpittau: there is also this older victoria backport for the partx issue which could probably be moved forward: https://review.opendev.org/c/openstack/ironic-python-agent/+/82178711:45
rpittauyep, done, thanks for the reminder11:46
arne_wiebalckrpittau: +111:46
iurygregorywe do have some commits that are worth a release in ussuri before moving to EM13:06
iurygregory:D13:06
rpittauiurygregory: cool! should we also try and merge https://review.opendev.org/c/openstack/ironic/+/820936 ?13:20
opendevreviewMerged openstack/ironic-python-agent stable/victoria: Re-read the partition table with partx -a  https://review.opendev.org/c/openstack/ironic-python-agent/+/82178713:23
opendevreviewMerged openstack/ironic-python-agent stable/victoria: Re-read the partition table with partx -a, part 2  https://review.opendev.org/c/openstack/ironic-python-agent/+/82178813:23
opendevreviewMerged openstack/ironic bugfix/18.1: Trivial: log current state when continuing cleaning  https://review.opendev.org/c/openstack/ironic/+/82061413:23
opendevreviewRiccardo Pittau proposed openstack/bifrost master: Add CentOS Stream 9 keystone integration job  https://review.opendev.org/c/openstack/bifrost/+/82419313:24
rpittau^ probably won't pass, still missing correct context13:25
iurygregoryrpittau, ++13:28
iurygregoryfeel free to +W I've +213:37
rpittauyep,done already :)13:42
iurygregoryty!13:44
iurygregoryI wish my mother had told me that the house would going through renovation today -.-'13:46
rpittau:)13:47
TheJuliagood morning14:09
arne_wiebalckGood morning, TheJulia o/14:15
rpittaugood morning TheJulia :)14:19
TheJuliaso I think I finally got my sushy session auth patch sorted late yesterday afternoon14:28
TheJuliaI feel like that is dance worthy14:28
MahnoorAsgharGood morning! Good to hear ^-^14:29
TheJuliaoooh ahh, and the redfish cache cleanups have been moving forward \o/14:32
TheJuliaiurygregory: much appreciated14:32
* TheJulia unfortunately needs to figure why train is not happy14:32
iurygregoryTheJulia, np! 14:32
iurygregoryalso, good morning =)14:33
opendevreviewMerged openstack/ironic bugfix/18.1: Set resource_url when getting all nodes  https://review.opendev.org/c/openstack/ironic/+/82509614:38
iurygregoryMicrosoft to buy Activision Blizzard =O14:50
TheJuliaYeah, that is kind of surprising14:50
opendevreviewMerged openstack/ironic bugfix/19.0: Set resource_url when getting all nodes  https://review.opendev.org/c/openstack/ironic/+/82509514:59
opendevreviewMerged openstack/ironic stable/xena: Set resource_url when getting all nodes  https://review.opendev.org/c/openstack/ironic/+/82509014:59
rpittauno comment on that :)15:00
holtgreweHi. Whenever I boot a node into IPA, my devices get an IPv6 address which generates ICMPv6 traffic. Can this be deactivated?15:13
opendevreviewMerged openstack/ironic stable/wallaby: Set resource_url when getting all nodes  https://review.opendev.org/c/openstack/ironic/+/82509115:13
opendevreviewMerged openstack/ironic stable/ussuri: Remove redfish cache entry upon errors  https://review.opendev.org/c/openstack/ironic/+/82093615:13
holtgreweAlso, I'm seeing a port, apparently for DHCP, that gets an IPv6 address and also generates ICMPv6 traffic. Can I get the DHCP agent up without IPv6?15:15
opendevreviewRiccardo Pittau proposed openstack/ironic master: [WIP] replace mkisofs with pycdlib  https://review.opendev.org/c/openstack/ironic/+/81912115:21
opendevreviewVerification of a change to openstack/ironic stable/wallaby failed: Remove redfish cache entry upon errors  https://review.opendev.org/c/openstack/ironic/+/82059715:45
* dtantsur back15:51
TheJuliaholtgrewe: I'd disable v6 on the kernel command line I guess16:02
TheJuliaholtgrewe: what distro are you using?16:02
holtgreweRocky Linux16:12
holtgreweIPv6 is disabled via sysctl16:12
holtgreweShould that be enough? Maybe not!16:12
dtantsurmmm, isn't DHCP coming from the subnet of the Neutron network?16:13
holtgreweip_version=4, ipv6_address_mode=None, ipv6_ra_mode=None on my subnets16:15
TheJuliaHmmmm16:28
TheJuliaIs rocky linux NetworkManager based?16:28
dtantsurit should match whatever RHEL 8 is16:28
TheJuliaI *think* it uses an internal client in rhel816:29
holtgreweI have deployed OS with Kolla and they are using centos 8 stream internally.16:30
holtgreweSorry, I always forget this extra layer.16:31
holtgreweOK, I will look into disabling ipv6 on the kernel command lien16:31
holtgrewereboot16:31
TheJuliawell16:31
holtgreweThe docker containers are running with host network mode, I believe.16:31
holtgreweOtherwise, I'll have to live with that very few ICMv6 packages...16:31
TheJuliaultimately, the dhcp clients use raw sockets, so if they are independently issuing icmpv6 solicitations and the neutron router has it enabled somehow, it may still generate ICMPv6 packets for discovey/response16:32
holtgreweDo I see the neutron router with the `openstack` commands or is it something configured in `/etc`?16:33
TheJuliaholtgrewe: so neutron puts those into namespaces16:33
TheJuliaso on the host(s) running neutron, `ip netns list` to see them16:33
TheJuliaand then `ip netns exec [namespace-id] ip addr`16:33
TheJuliaits all dynamic and neutron configured16:33
dtantsurhuh, bifrost stable branches broken? :-/16:36
rpittaummm linters16:42
rpittauoh a missing import, some new lib version broke it?16:42
holtgreweTheJulia: thanks! Will look into this a bit more and try to wrap my head around it.16:42
rpittaudtantsur: https://github.com/Textualize/rich/commit/ad9debf2eb5a04cea709d35a7bc03f316bf0431c16:43
rpittauthat's just great16:43
dtantsursweeeet!16:44
rpittau:/16:44
NobodyCamWait I volunteered for something16:45
NobodyCamAnd good morning ironic’ers16:45
rpittauhey NobodyCam :)16:45
NobodyCamGood Morning rpittau 16:46
iurygregorymorning NobodyCam o/16:47
NobodyCamMorning iurygregory 0/ happy not monday16:51
rpittaudtantsur: the issue is in ansible-lint, it was fixed in 5.3.2, we use <5 up until yoga16:54
dtantsurrpittau: is it possible for us to use 5.3.2?16:55
rpittauI'll have to test it16:55
dtantsurI wonder how we ended up pulling in a major version on stable branches..16:59
opendevreviewRiccardo Pittau proposed openstack/bifrost stable/xena: Fix incompatibility with recent rich library  https://review.opendev.org/c/openstack/bifrost/+/82514116:59
rpittaulet's see if this ^ works16:59
rpittauaaaand meeting17:00
rpittauoh yeah, ansible-lint doesn't have an upper limit to rich version :/17:02
dtantsurbut we should be using upper-constraints..17:06
rpittaurich is not in upper-constraints17:15
rpittaugood night! o/17:29
arne_wiebalckbye everyone o/17:48
dtantsuriurygregory, TheJulia, just opened an ML thread about bifrost bugfix branches19:10
TheJuliaso I think it ended up that way becasue the release team was trying to also be a bit heavy handed and force things that don't cut releases at least 2x a year to be on the openstack project wide release structure with RC releases19:25
TheJuliabut tl;dr +1 kill the bugfix branch on bifrost with fire19:25
TheJuliaPlease just don't burn everything down in the process. The cats will be sad.19:25
dtantsurdefinitely not the cat tree!19:28
dtantsuranyway, we can still release 3x a cycle. will lead to a bit more discipline and better release notes :)19:28
TheJulia++19:30
stevebaker[m]good morning19:34
dtantsurmorning stevebaker[m], a rare occasion to meet you online :)19:35
dtantsur(which means I should probably go eat some dinner)19:36
stevebaker[m]dtantsur: good evening!19:36
dtantsursee you tomorrow folks o/19:37
stevebaker[m]TheJulia: could I trouble you for a dib review? https://review.opendev.org/c/openstack/diskimage-builder/+/82501819:45
TheJuliastevebaker[m]: done20:01
stevebaker[m]TheJulia: thanks20:02

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