Wednesday, 2022-05-04

opendevreviewJacob Anders proposed openstack/sushy stable/xena: Follow up to Attempt using Redfish settings resource to get URI  https://review.opendev.org/c/openstack/sushy/+/83984405:15
opendevreviewJacob Anders proposed openstack/sushy stable/xena: Attempt using Redfish settings resource to get URI  https://review.opendev.org/c/openstack/sushy/+/83984305:30
opendevreviewJacob Anders proposed openstack/sushy stable/xena: Follow up to Attempt using Redfish settings resource to get URI  https://review.opendev.org/c/openstack/sushy/+/83984405:37
opendevreviewMerged openstack/ironic master: Fix ironic-lib from source and branch detection in IPA builds  https://review.opendev.org/c/openstack/ironic/+/83998206:11
arne_wiebalckGood morning, Ironic!06:24
rpittaugood morning ironic! o/06:25
*** pmannidi is now known as pmannidi|Training07:38
opendevreviewmitya-eremeev-2 proposed openstack/ironic master: Delete resource provider after node deletion.  https://review.opendev.org/c/openstack/ironic/+/82729507:46
opendevreviewmitya-eremeev-2 proposed openstack/ironic master: Delete resource provider after node deletion.  https://review.opendev.org/c/openstack/ironic/+/82729507:46
opendevreviewMerged openstack/ironic-ui master: Address RemovedInDjango40Warning  https://review.opendev.org/c/openstack/ironic-ui/+/83997109:48
dtantsurfolks, https://review.opendev.org/c/openstack/ironic-python-agent/+/839946 is ready for a 2nd review. fixes a very annoying issue for us.09:55
*** matt__ is now known as matfechner11:04
opendevreviewMerged openstack/ironic master: Trivial: Remove prints in db api layer code  https://review.opendev.org/c/openstack/ironic/+/84000511:12
iurygregorygood morning Ironic11:25
ameya49iurygregory : Thank You for review : https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/826646. Can we please get more reviewers to proceed this patch ahead.11:31
iurygregoryameya49, yw!  I hope another core will have time to take a look at it this week11:34
iurygregoryHappy MayThe4thBeWithYou everyone =D11:58
rpittauMayThe4thBeWithYouToo iurygregory :)12:40
opendevreviewMerged openstack/ironic-python-agent master: Use a pre-defined partition UUID to detect configdrive on GPT  https://review.opendev.org/c/openstack/ironic-python-agent/+/83994612:43
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent stable/yoga: Use a pre-defined partition UUID to detect configdrive on GPT  https://review.opendev.org/c/openstack/ironic-python-agent/+/84034712:55
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent stable/xena: Use a pre-defined partition UUID to detect configdrive on GPT  https://review.opendev.org/c/openstack/ironic-python-agent/+/84034812:55
rpittaudtantsur: can we please also try backports to ironic-python-agent bugfix/8.3 and bugfix/8.1 ?12:58
dtantsura bit later, sorry12:58
rpittausure12:58
iurygregoryI was about to say that XD12:58
dtantsurfg13:00
dtantsurdamn13:00
dtantsurI wonder why keyboard focus keeps switching to the IRC window for me13:00
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.4: Use a pre-defined partition UUID to detect configdrive on GPT  https://review.opendev.org/c/openstack/ironic-python-agent/+/84034913:03
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.3: Use a pre-defined partition UUID to detect configdrive on GPT  https://review.opendev.org/c/openstack/ironic-python-agent/+/84047013:03
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.1: Use a pre-defined partition UUID to detect configdrive on GPT  https://review.opendev.org/c/openstack/ironic-python-agent/+/84047113:03
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent master: Do not try to guess EFI partition path by its number  https://review.opendev.org/c/openstack/ironic-python-agent/+/84045313:06
dtantsurrpittau, iurygregory, here goes the next patch ^^13:06
dtantsurthis is a polished version of the wallaby poc13:06
iurygregorydtantsur, ok o/13:16
TheJuliao/13:34
* TheJulia is soooo tired13:34
*** diablo_rojo is now known as Guest8313:36
*** pmannidi|Training is now known as pmannidi13:55
* dtantsur can relate13:56
*** mat_fechner is now known as matfechner14:05
* TheJulia glares at networking-generic-switch14:11
TheJuliaI was feeling super low energy last night, and then decided to start a project in my electrical panel....14:11
opendevreviewVerification of a change to openstack/ironic-tempest-plugin master failed: Add iDRAC management cleaning steps tests  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/82664614:13
opendevreviewDmitry Tantsur proposed openstack/ironic-lib stable/wallaby: Use a pre-defined partition UUID to detect configdrive on GPT  https://review.opendev.org/c/openstack/ironic-lib/+/83994914:15
opendevreviewDmitry Tantsur proposed openstack/ironic-lib stable/wallaby: DNM add an UEFI job  https://review.opendev.org/c/openstack/ironic-lib/+/83995514:15
TheJuliahttps://usercontent.irccloud-cdn.com/file/TuzZPS8W/whodoesthis.png14:15
dtantsurI'm not sure I understand what is going on there :)14:16
TheJuliaas numbers go negative, I'm pushing electrons to the local electrical grid14:16
dtantsurthat's cool!14:23
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Only use findfs when looking up partitions by UUID  https://review.opendev.org/c/openstack/ironic-python-agent/+/84049914:29
TheJuliaI suspect I'm going to need a lot of coffee once I'm actually able to focus on technical work14:45
* TheJulia goes back to jira14:45
opendevreviewRiccardo Pittau proposed openstack/virtualbmc master: Use zed jobs  https://review.opendev.org/c/openstack/virtualbmc/+/84050114:53
opendevreviewMerged openstack/ironic stable/yoga: Fix ironic-lib from source and branch detection in IPA builds  https://review.opendev.org/c/openstack/ironic/+/83998315:15
opendevreviewRiccardo Pittau proposed openstack/virtualbmc master: Use zed jobs  https://review.opendev.org/c/openstack/virtualbmc/+/84050115:40
opendevreviewRiccardo Pittau proposed openstack/sushy-tools master: Use python Zed tests  https://review.opendev.org/c/openstack/sushy-tools/+/83867415:42
*** pmannidi is now known as pmannidi|Gone15:45
opendevreviewRiccardo Pittau proposed openstack/ironic-tempest-plugin master: Add yoga tests, stop testing victoria  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/84050816:07
rpittaubye everyone! o/16:09
TheJuliaNow, where did my brain go....16:22
opendevreviewJulia Kreger proposed openstack/networking-generic-switch master: Trivial: Remove legacy devstack CI call  https://review.opendev.org/c/openstack/networking-generic-switch/+/84051316:38
rpiosoGood morning, ironic :)16:54
rpiosoiurygregory: The gate's ironic-inspector-tempest-discovery test failed against https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/826646. I believe it's unrelated to the change. Agree? Should I recheck it?16:57
iurygregoryrpioso, double checking now let me see17:00
iurygregoryhumm kernel panic in the virtual bm... but only in the gate job, maybe we were just unlucky since the history of the job is green, should be ok to trigger a recheck17:04
rpiosoiurygregory: Thank you! I'll do that.17:05
JayFSo I'm happy to be able to say publically now that I'm changing jobs, and will be moving to a position at G-Research where I'll be working on Open Source software full time again -- including some time (I think ~5-10% a week) in Ironic :)17:13
JayFStarting in June17:13
iurygregoryJayF, WOW NICE! \o/17:13
iurygregoryCongratulations!17:13
JayFI'm pretty stoked :) it's the rare unicorn of open source positions: no customers, no production environment, just go make code good17:14
iurygregory=O17:14
iurygregory*magic*17:14
rpiosoJayF: Awesome! Congratulations!17:14
JayFNo pre-volunteering me for anything though :P I'll focus for a while on reviewing code and changes since I've been paying less attention to get back up to speed17:15
iurygregoryJayF, ok :D17:16
opendevreviewJulia Kreger proposed openstack/networking-generic-switch master: CI: Alternate logic for multinode key replacement  https://review.opendev.org/c/openstack/networking-generic-switch/+/84051917:20
TheJuliaCongrats JayF!17:22
rpiosoiurygregory: Where did you see the kernel panic?17:46
iurygregoryrpioso, if you check the files under logs/ironic-bm-logs17:46
iurygregoryin the end of the file you can find kernel panic =)17:47
rpiosoiurygregory: Thank you!17:54
opendevreviewMerged openstack/ironic-python-agent stable/yoga: Use a pre-defined partition UUID to detect configdrive on GPT  https://review.opendev.org/c/openstack/ironic-python-agent/+/84034718:13
opendevreviewJulia Kreger proposed openstack/ironic master: Fix v6 CI job - Return it to normal non-voting status  https://review.opendev.org/c/openstack/ironic/+/83908618:30
opendevreviewJulia Kreger proposed openstack/ironic master: DNM: Multinode Grenade  https://review.opendev.org/c/openstack/ironic/+/84053018:30
opendevreviewJulia Kreger proposed openstack/ironic master: Fix v6 CI job - Return it to normal non-voting status  https://review.opendev.org/c/openstack/ironic/+/83908618:33
opendevreviewJulia Kreger proposed openstack/ironic master: DNM: Multinode Grenade  https://review.opendev.org/c/openstack/ironic/+/84053018:33
TheJuliahjensas: as discussed, v6 job should be good to go I think. Making it non-voting to start with back in the normal queue so we can get a feel for it. Also notated the needful in the ironic-jobs.yaml file for when time comes to go to UEFI18:34
hjensasTheJulia: ack, good progress. Well done!19:15
TheJulianow, if grenade multinode would behave, I woudl be happy19:17
TheJuliathe whole grenade model makes it *painfully* difficult to fix these sorts of issues19:17
opendevreviewMerged openstack/ironic-lib master: Add Python3 zed unit tests  https://review.opendev.org/c/openstack/ironic-lib/+/83223319:28
TheJuliarpittau: dtantsur: iurygregory: w/r/t multipath, are we doing other related fixes first?19:40
iurygregoryTheJulia, so for wallaby to have things working we had 3 patches 2 in IPA 1in ironic-lib, I think we started merging the other IPA patch first only because CI is unhappy in https://review.opendev.org/c/openstack/ironic-python-agent/+/837039 19:47
rpittauTheJulia: I don't have the patch in front of me, but if I remember correctly we need to adapt the multipath command to make it generic or at least add a non-redhat alternative19:47
iurygregoryyeah ^19:47
TheJuliarpittau: lets talk tomorrow and figure out next steps then19:48
opendevreviewHarald Jensås proposed openstack/networking-baremetal master: Add LACP support to Netconf OpenConfig driver  https://review.opendev.org/c/openstack/networking-baremetal/+/83537819:59
opendevreviewHarald Jensås proposed openstack/networking-baremetal master: Add support for pre-configured link aggregates  https://review.opendev.org/c/openstack/networking-baremetal/+/84053319:59
stevebaker[m]good morning20:09
TheJuliagood morning stevebaker[m] 20:26
opendevreviewJulia Kreger proposed openstack/ironic master: DNM: Multinode Grenade  https://review.opendev.org/c/openstack/ironic/+/84053020:45
opendevreviewMerged openstack/ironic-tempest-plugin master: Add iDRAC management cleaning steps tests  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/82664621:38

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