Monday, 2022-01-24

opendevreviewVanou Ishii proposed openstack/ironic master: Deal with iRMC virtual media incompatibility  https://review.opendev.org/c/openstack/ironic/+/82379000:46
opendevreviewVanou Ishii proposed openstack/ironic master: Deal with iRMC virtual media incompatibility  https://review.opendev.org/c/openstack/ironic/+/82379000:50
opendevreviewHarald Jensås proposed openstack/ironic-python-agent-builder master: dhcp-all-interfaces: let NetworkManager doit.  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82598601:00
opendevreviewVanou Ishii proposed openstack/ironic master: Deal with iRMC virtual media incompatibility  https://review.opendev.org/c/openstack/ironic/+/82379001:07
*** melwitt is now known as Guest34402:26
opendevreviewchandan kumar 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:02
opendevreviewchandan kumar 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:40
opendevreviewchandan kumar proposed openstack/ironic-python-agent-builder master: Added element to enable RDO deps repo to install sysbench  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82531303:49
opendevreviewHarald Jensås proposed openstack/ironic-python-agent-builder master: dhcp-all-interfaces: let NetworkManager doit.  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82598607:00
opendevreviewHarald Jensås proposed openstack/ironic-python-agent-builder master: dhcp-all-interfaces: let NetworkManager doit.  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82598607:02
arne_wiebalckGood morning, Ironic!07:26
rpittaugood morning ironic! o/08:21
arne_wiebalckhey rpittau o/08:29
rpittauhey arne_wiebalck :)08:29
opendevreviewArne Wiebalck proposed openstack/ironic-python-agent master: [WIP] SoftwareRAID: Use efibootmgr (and drop grub2-install)  https://review.opendev.org/c/openstack/ironic-python-agent/+/82602608:30
arne_wiebalckdtantsur: this ^^ is  a forward patch of the one I tested in Wallaby (w/ and w/o RAID, on centos7 and stream8)08:31
arne_wiebalckdtantsur: there is one or two things to discuss about it, so if you a moment later, let me know08:32
dtantsurgood morning folks09:35
dtantsurarne_wiebalck: yeah, sure09:35
arne_wiebalckgood morning, dtantsur 09:50
dtantsurarne_wiebalck: I'd rather not change label to include "Ironic". why do you need that?09:56
arne_wiebalckdtantsur: there are 2 reasons09:57
arne_wiebalckdtantsur: one is we need to have unique labels per entry, otherwise we remove what we just added09:57
arne_wiebalckdtantsur: the second one is to make clear "who" created the entry09:58
dtantsurI quite strongly disagree with the 2nd one. It's not about "who created", it's about "what is booted"09:58
dtantsurso it's centos, not ironic that is booted09:58
arne_wiebalckdtantsur: on my boxes I see "CentOS Linux", but it is not clear Ironic created the entry09:58
dtantsur(similarly to how you don't end up with "Anaconda Fedora" or "Kickstart RHEL")09:58
arne_wiebalckdtantsur: the "what is booted" stays09:59
arne_wiebalckdtantsur: let me show you how it looks like in the end ...09:59
dtantsurI'm fine if you add an option with a pattern of labels, so that you can modify it to include Ironic. But I think our default should come from the operating system. Us using "ironic" has led to customer confusion already (and may confuse software like "shim" that relies on labels).10:00
arne_wiebalckdtantsur: this is for a non-RAID installation: https://paste.opendev.org/show/812321/10:00
dtantsurcould you elaborate on the first reason? isn't it an issue already then?10:00
arne_wiebalckdtantsur: on the first reason: we remove entries with the same label, but for RAID we have two ESP (which currently get the same label)10:01
dtantsurah gotcha10:01
arne_wiebalckdtantsur: so we create an entry, and then remove it on the call for the 2nd ESP10:01
dtantsurokay, I'd rather have something like "CentOS Linux (RAID, part 1)"10:01
arne_wiebalckdtantsur: ok10:02
dtantsurI feel quite strongly against "Ironic CentOS Linux". We're not in this business, there is enough of RHEL clones out there :D10:02
arne_wiebalckdtantsur: :-D10:02
arne_wiebalckdtantsur: fair enough10:02
arne_wiebalckdtantsur: I will change the ID we pass10:03
arne_wiebalckdtantsur: and, yes, this is an issue already10:03
arne_wiebalckdtantsur: I just did not realise10:03
arne_wiebalckdtantsur: well, I assume it is an issue, I can check10:03
dtantsur++10:03
arne_wiebalckdtantsur: maybe not, since we do not check the labels in case of grub210:04
arne_wiebalckdtantsur: so, this out of the way, there is a 2nd issue10:04
arne_wiebalckdtantsur: we prepare the ESPs *before* checking for valid bootloaders10:04
arne_wiebalckdtantsur: if we find none, we return and try grub210:05
arne_wiebalckdtantsur: and that path will try to prepare the partitions10:05
arne_wiebalckdtantsur: not sure this is an issue in practice10:06
arne_wiebalckdtantsur: the reason we check the bootloaders only after preparing the partitions is that inside this function we also remove the signature (which does not work when the ESP is mounted)10:06
dtantsurarne_wiebalck: isn't it a matter of just passing a flag prepare_copies or something?10:06
arne_wiebalckdtantsur: yep, or a flag erase_signature10:07
dtantsurI guess we do need to erase the signature on the positive path?10:08
arne_wiebalckdtantsur: or mount, check bootloader, umount, prepare, mount10:08
arne_wiebalckdtantsur: what is the positive path?10:08
dtantsurwhen grub is not needed10:08
arne_wiebalckyes10:08
dtantsuryeah, we can do it this way, sounds logical enough10:08
arne_wiebalckerasing the signatures avoids mount confusion10:09
arne_wiebalckdtantsur: there are several options, wasn't sure which one to go with10:09
dtantsurI think the option without new flags (with umount in-between) is the cleanest? I don't have a strong opinion.10:10
arne_wiebalckdtantsur: we can have a small "erase_signature" function which we call optionally in prepare_partitions10:10
arne_wiebalckdtantsur: ok10:10
arne_wiebalckdtantsur: I think these were the two points I wanted to discuss.10:18
dtantsurcool, thanks for handling this10:18
arne_wiebalckdtantsur: if things look otherwise ok to you, I will go ahead a polish the commit10:18
arne_wiebalckdtantsur: less complicated than I thought (not sure I just said this!)10:18
dtantsurarne_wiebalck: :D at a quick glance everything else looks okay. I'll look deeper later.10:19
arne_wiebalckdtantsur: sure, let me polish things, add tests, a reno, ... 10:20
opendevreviewMerged openstack/ironic stable/victoria: Remove redfish cache entry upon errors  https://review.opendev.org/c/openstack/ironic/+/82093411:32
Dhuldev_Valekariurgregory: Hi GM, just want to discuss few things related to ironic tempest test case failure for basic introspection scenario, can you please let me know your availability for discussion over call?11:43
opendevreviewMerged openstack/ironic stable/xena: Remove redfish cache entry upon errors  https://review.opendev.org/c/openstack/ironic/+/82058811:53
arne_wiebalckdtantsur: https://paste.opendev.org/show/812322/13:06
dtantsurarne_wiebalck: this is much nicer IMO13:06
arne_wiebalckdtantsur: ++13:07
arne_wiebalckdtantsur: grub2-install'ed machines do not seem to be affected by the cleanup13:21
arne_wiebalckdtantsur: so, we seem to be good.13:21
*** rcastillo|out is now known as rcastillo|rover14:09
TheJuliagood morning14:30
dtantsurmorning TheJulia 14:40
opendevreviewMerged openstack/ironic master: Migrates docs from wiki  https://review.opendev.org/c/openstack/ironic/+/82480814:51
TheJuliafeels like I've made lots of progress in eliminating the wiki recently, but friday I mainly moved board meeting notes to a new wiki14:55
dtantsur++14:57
*** Guest295 is now known as dansmith14:58
dtantsurbm.get('/v1/nodes?limit=1&associated=false').json()['next']15:47
dtantsur'http://192.168.122.1:6385/v1/None?sort_key=id&sort_dir=asc&fields=instance_uuid,maintenance,power_state,provision_state,uuid,name&limit=1&marker=4e41df61-84b1-5856-bfb6-6b5f2cd3dd11'15:47
dtantsurthis looks really bad. really.15:47
* TheJulia blinks15:47
dtantsurand I'm not even talking about fields that should arguably not be there15:47
TheJuliathose are the default non-detailed fields15:48
dtantsuryeah, I understand. yet.15:48
TheJuliayeah15:48
TheJuliaI'm more concerned about None15:48
dtantsurmuch more :)15:48
TheJuliathe rest just looks ugly15:48
TheJulialimit1, and marker make sense15:49
dtantsurright. and associated should be there as well, see https://storyboard.openstack.org/#!/story/171872615:49
TheJuliawhere did associated go15:49
dtantsur(I was reproducing this bug when hit the None problem)15:49
TheJuliafun15:51
dtantsurmay take a look after the meeting15:52
* TheJulia wonders how many folks are actually here15:52
dtantsurIury on PTO, judging by Twitter15:53
TheJuliaYeah, I saw the one with the tip of the surf board15:53
dtantsuryep :)15:53
dtantsurArne and Riccardo should be here15:53
dtantsurbtw, we've accumulated a fair amount of merged xena patches15:54
dtantsurwe should consider a series of releases15:54
TheJuliaYeah15:54
TheJulia#startmeeting ironic16:00
opendevmeetMeeting started Mon Jan 24 16:00:25 2022 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
opendevmeetThe meeting name has been set to 'ironic'16:00
TheJuliao/16:00
rpittauo/16:00
arne_wiebalcko/16:00
* TheJulia wonders if we have quorum today16:00
dtantsuro/16:00
rlooo/16:00
rpiosoo/16:00
TheJuliaStarting to look like we kind of do :)16:00
TheJuliaGive it another minute I guess16:01
erbarro/16:01
TheJuliaOkay, well, our agenda this week is fairly bare bones16:02
ajyao/16:02
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting16:02
TheJulia#topic Announcements / Reminders16:02
TheJulia#info CFP for the OpenInfra Summit Berlin 2022 is presently open until I believe the end of the first week in Febuary.16:03
TheJulia#link https://cfp.openinfra.dev/app/berlin-202216:03
bfournieo/16:03
TheJulia#info The TC is seeking nominations on release naming which is still open for a few more hours16:04
TheJulia#link http://lists.openstack.org/pipermail/openstack-discuss/2022-January/026620.html16:04
TheJuliaDoes anyone have anything to announce or remind us of this week?16:04
TheJuliaI guess not16:05
TheJulia#topic Review Action Items for last week16:06
TheJuliaI believe we only had one action item last week which was Iury to follow-up on Ussuri EM releases. I believe he has done so.16:06
TheJuliaMoving on!16:06
TheJulia#topic Review subteam status reports16:07
TheJulia#link https://etherpad.opendev.org/p/IronicWhiteBoard16:07
TheJuliaStarting at line 6216:08
TheJuliaLooks like we're juggling a lot right now16:08
dtantsurso true16:08
dtantsur(for me it's mostly openshift stuff though :( )16:09
TheJuliaI'm not sure there is really much to put down besides no update on a number of items16:09
TheJuliaCommunity + bugs for me16:09
TheJuliaAnd meetings :(16:09
TheJuliaOnward?16:09
dtantsuryeah16:10
TheJulia#topic Deciding on priorities for the coming week16:10
TheJulia#link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio16:10
TheJuliaDo we have anything to add to the priority review list this week?16:10
ajyacan these be added - https://review.opendev.org/c/openstack/ironic/+/823311 https://review.opendev.org/c/openstack/ironic/+/823312 ?16:10
dtantsurI only have https://review.opendev.org/c/openstack/ironic/+/82554416:11
TheJuliaYeah, I think all of those can16:11
TheJuliaajya: if you want, you should be able to add the hashtag to them16:11
ajyaok16:12
TheJuliaIt seems I'm not signed into gerrit :\16:12
TheJuliaAnything else?16:13
TheJuliaVery well then.16:14
TheJuliaWe have no discussion topics, so a brief stop over at the Baremetal SIG first16:14
TheJulia#topic Baremetal SIG16:14
arne_wiebalckNothing really (apart from we should start the prep for Feb 8)16:15
TheJulia#info Nobodycam and I are our next volunteers to discuss scaling. I expect we'll begin a slide deck this week in an attempt to gather thoughts/ideas. If anyone has any questions or thinks of specific topics to cover, then please let us know16:15
TheJuliaarne_wiebalck: ++16:15
TheJuliaSo, onward I guess!16:15
arne_wiebalckyes16:15
dtantsurnice!16:15
TheJulia#topic Open Discussion16:15
TheJuliaAnything to discuss this week? Are we taking over the world appropriately? Should we buy purple paint this week?16:16
TheJuliaAnd yes, yellow for the accents, as I believe that was the agreed upon color16:16
TheJulia:)16:16
dtantsur:)16:16
dtantsurwho wants to request xena releases?16:16
TheJuliavolunteers?!16:17
dtantsurI can do it16:17
* TheJulia hears that echoing in the room16:17
TheJuliayou sure?16:17
rpittauI could as well, if help needed16:18
TheJuliaI could try to commit to doing so, but I got very little time heads down last week16:18
TheJulia:(16:18
dtantsurnow we have 3 volunteers :) rpittau, if you have enough time today, go ahead16:18
*** Guest344 is now known as melwitt16:18
TheJulia\o/16:19
dtantsurwe have so many bugs from like 2020 that need triaging16:19
rpittaudtantsur: I can have a look after the meeting16:19
dtantsurand some of them even seem valid...16:19
dtantsurthx!16:19
TheJuliaSo onward?16:19
dtantsuralways onward!16:19
TheJulia#topic Decide who is going to run our next meeting16:19
TheJuliaI volunteer Iury  ;)16:20
dtantsurIury++ assuming he is back16:20
TheJuliaAnd if all else fails I can run it next week16:20
TheJuliayeah16:20
rpittau:D16:20
TheJuliaOkay, well, Thanks everyone!16:21
dtantsurthank you!16:21
arne_wiebalckthanks TheJulia 16:21
rpittauthanks!16:21
TheJulia#endmeeting16:21
opendevmeetMeeting ended Mon Jan 24 16:21:39 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:21
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-01-24-16.00.html16:21
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-01-24-16.00.txt16:21
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-01-24-16.00.log.html16:21
dtantsurfolks, https://review.opendev.org/c/openstack/ironic-specs/+/808804/ is long but easy reading, and needs a 2nd +216:21
rpittaudtantsur, TheJulia, before releasing ironic I think we should merge https://review.opendev.org/c/openstack/ironic/+/82488116:29
dtantsurfair, +216:37
rpittauprobably not worth to release inspector and sushy, or others except bifrost and ipa16:40
rpittauproposed the 2 and waiting for the merge in ironic16:42
NobodyCamGood Morning Ironic'ers happy Monday16:47
rpittauhey NobodyCam :)17:03
rpittauand good night! o/17:03
NobodyCamhey hey rpittau have a great night17:04
dtantsurmorning NobodyCam 17:06
dtantsurTheJulia: I suspect the None issue has been fixed by https://review.opendev.org/c/openstack/ironic/+/824190 and others17:06
dtantsur(my bifrost didn't have it)17:06
NobodyCamHey hey dtantsur o/17:06
TheJuliadtantsur: ahh, yeah, I had a feeling that was related17:07
dtantsurbut I suspect it's not fixed everywhere, I wonder if there are more patches to make or to merge?17:09
dtantsurarne_wiebalck: you only fixed resource_url in nodes, ports and port groups, right?17:10
TheJuliadtantsur: AFAIK that is correct17:11
TheJuliawell, those locations17:11
dtantsurokie, I'll check the others17:11
TheJuliaI didn't see other areas17:11
TheJuliawell, patches for other areas17:11
arne_wiebalckdtantsur: yes, correct ... are there more?17:13
dtantsurarne_wiebalck: I assume all other resources have pagination17:13
dtantsurI'm on it17:14
arne_wiebalckdtantsur: thanks!17:14
arne_wiebalckdtantsur: hmm, yeah, I think I fixed only those I found during testing 17:14
opendevreviewArne Wiebalck proposed openstack/ironic-python-agent master: [WIP] SoftwareRAID: Use efibootmgr (and drop grub2-install)  https://review.opendev.org/c/openstack/ironic-python-agent/+/82602617:15
arne_wiebalckdtantsur: TheJulia: ^^ s/w RAID with efibootmgr if you want to have a look17:16
arne_wiebalcktested on a Wallaby backport, missing a test for s/w RAID, but I thought it might be better if you spot missing pieces before I spent time on a test I have to redo later :)17:17
TheJuliaarne_wiebalck: okay, I'll try to look actually here in a  little bit17:20
sdanniHi! I'm looking into backup and resetting idrac after a node is returned from lessees. I wonder does "import and export configuration" steps include idrac settings? https://docs.openstack.org/ironic/latest/admin/drivers/idrac.html#import-and-export-configuration 17:23
arne_wiebalckTheJulia: thanks! dtantsur had already a sneak preview earlier today :)17:24
dtantsurugh, node history does have a few issues17:31
TheJuliadtantsur: not entirely suprprising given how much free time I have :(18:02
dtantsurI can imagine18:05
*** sshnaidm is now known as sshnaidm|afk18:05
* TheJulia gets a meeting invite in and has no idea what it is talking about18:05
dtantsurTheJulia: this is a perfect reason to put your corgi on the table, put a headset on him and hide18:06
TheJulialol18:06
arne_wiebalckdtantsur: :-D18:07
arne_wiebalckbye everyone, see you tomorrow o/18:07
dtantsurc u arne_wiebalck 18:07
opendevreviewDmitry Tantsur proposed openstack/ironic master: Fix resource_url in the remaining resources  https://review.opendev.org/c/openstack/ironic/+/82614318:11
dtantsurTheJulia, arne_wiebalck ^^^18:11
dtantsursee you tomorrow18:26
TheJuliao/18:30
opendevreviewMerged openstack/ironic stable/xena: Fix validating input for redfish update_firmware  https://review.opendev.org/c/openstack/ironic/+/82488118:54
*** melwitt is now known as Guest44019:54
*** sdanni_ is now known as sdanni20:14
*** melwitt_ is now known as melwitt20:55
*** melwitt is now known as Guest45020:59
-opendevstatus- NOTICE: review.opendev.org will have a few short outages over the next few hours (beginning 22:00 UTC) while we rename projects and then upgrade to Gerrit 3.4. See https://lists.opendev.org/pipermail/service-announce/2022-January/000030.html for details.21:04
*** melwitt_ is now known as melwitt21:05
*** melwitt is now known as Guest45121:07
*** Guest451 is now known as melwitt21:10
TheJulia stevebaker[m] you mentioned when we spoke you had a few things you need me to review21:10
stevebaker[m]TheJulia: thanks, this one would be good https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82540021:11
TheJuliastevebaker[m]: did you verify that pip version was available during wallaby? :)21:13
TheJuliastevebaker[m]: I'd almost change the release note to indicate this fixes it... kind of ;)(21:14
TheJuliaerr ;)21:14
stevebaker[m]TheJulia: not yet, I guess I'll find out when it gets backported. I'm not clear why there is an explicit pip version at all21:14
* TheJulia looks it up real quick21:14
TheJuliaoctober 21, 202121:15
TheJuliaMaybe more reason to s/upgrades/fixes/21:16
TheJuliastevebaker[m]: done21:17
stevebaker[m]TheJulia: awesome thainks21:18
stevebaker[m]TheJulia: oh landing this would be useful also https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/825411/721:19
TheJuliadone21:20
TheJuliastevebaker[m]: https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82336521:27
stevebaker[m]TheJulia: the depends-on in that change will be replaced with the python2 removal backport. I think I should remove the depends-on and then land 823365?21:29
TheJuliaLikely a good idea21:30
TheJuliaDidn’t think of the dependency21:30
TheJuliaI'm going to lay down for a little bit and maybe eat some lunch21:32
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/+/82336521:32
-opendevstatus- NOTICE: The review.opendev.org maintenance work is beginning now. Expect Gerrit outages over the next couple of hours. See https://lists.opendev.org/pipermail/service-announce/2022-January/000030.html for details.22:01
*** melwitt is now known as Guest45622:04
*** Guest456 is now known as melwitt22:12
*** melwitt is now known as Guest45822:13
*** Guest458 is now known as melwitt22:15
opendevreviewMerged openstack/ironic-python-agent-builder master: Remove centos7 job, python2 support  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82540022:33
opendevreviewPetros Petrou proposed openstack/sushy-tools master: VMware vSphere Systems Resource Driver  https://review.opendev.org/c/openstack/sushy-tools/+/82597822:54
opendevreviewJulia Kreger proposed openstack/ironic-tempest-plugin master: WIP: An idea for rbac positive/negative testing  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/81916523:10
opendevreviewSteve Baker proposed openstack/ironic-python-agent-builder stable/xena: Remove centos7 job, python2 support  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82619723:33
opendevreviewSteve Baker proposed openstack/ironic-python-agent-builder stable/xena: Move centos9 jobs from experimental to check  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82619823:33
opendevreviewSteve Baker proposed openstack/ironic-python-agent-builder stable/wallaby: Remove centos7 job, python2 support  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82619923:36
opendevreviewSteve Baker proposed openstack/ironic-python-agent-builder stable/wallaby: Move centos9 jobs from experimental to check  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82620023:36

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