Monday, 2021-03-29

« Sunday, 2021-03-28 Index Tuesday, 2021-03-30 »
*** rcernin has joined #openstack-ironic00:00
openstackgerritliuyuanfeng proposed openstack/ironic master: when we use smart card, we set boot_device to bios to wait agent to do something.But after agent finishes,we need to set boot_device to disk. If we do not do so，when ironic finishes deploy, node will boot to bois.  https://review.opendev.org/c/openstack/ironic/+/78313401:11
openstackgerritankit proposed openstack/ironic master: Add security dashboard clean steps to ilo drivers  https://review.opendev.org/c/openstack/ironic/+/78236401:58
*** zzzeek has quit IRC02:04
*** zzzeek has joined #openstack-ironic02:06
*** sshnaidm|afk has quit IRC02:06
*** jamesdenton has quit IRC02:45
*** jamesdenton has joined #openstack-ironic02:45
*** rcernin has quit IRC03:06
*** rcernin has joined #openstack-ironic03:11
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Automaticaly set cipher suite  https://review.opendev.org/c/openstack/ironic/+/77052603:51
*** jamesdenton has quit IRC04:03
*** jamesden_ has joined #openstack-ironic04:03
*** k_mouza has joined #openstack-ironic04:07
*** k_mouza has quit IRC04:12
*** ricolin has quit IRC04:35
*** jamesden_ has quit IRC04:37
*** jamesdenton has joined #openstack-ironic04:37
*** ricolin has joined #openstack-ironic04:48
*** jamesdenton has quit IRC05:14
*** jamesdenton has joined #openstack-ironic05:16
openstackgerritMerged openstack/ironic master: update grub2 file name  https://review.opendev.org/c/openstack/ironic/+/78048105:25
*** bdodd has quit IRC05:25
*** bdodd has joined #openstack-ironic05:32
*** bdodd has quit IRC05:32
*** bdodd has joined #openstack-ironic05:39
*** moshiur has joined #openstack-ironic06:06
*** k_mouza has joined #openstack-ironic06:10
*** viks____ has joined #openstack-ironic06:11
arne_wiebalckGood morning, ironic!06:12
*** k_mouza has quit IRC06:14
jandershey arne_wiebalck :)06:14
*** k_mouza has joined #openstack-ironic06:14
arne_wiebalckhey janders o/06:14
*** k_mouza has quit IRC06:18
*** uzumaki has quit IRC06:41
*** uzumaki has joined #openstack-ironic06:44
*** uzumaki has quit IRC07:00
*** jamesdenton has quit IRC07:10
*** jamesdenton has joined #openstack-ironic07:10
*** rpittau|afk is now known as rpittau07:15
rpittaugood morning ironic!07:15
rpittauJayF: re devstack: I usually run it on ubuntu (focal and bionic), it's been a while I ran it on centos/rhel8 exactly because the gate is on ubuntu, so usually we just suggest to run it on ubuntu. I recently submitted a patch to update the basic configuration (tested on ubuntu as well) -> https://review.opendev.org/c/openstack/ironic/+/78327007:17
rpittaumoshiur: hopefully the bug will get your patch moving07:21
*** rcernin has quit IRC07:29
openstackgerritMerged openstack/networking-baremetal master: Update master for stable/wallaby  https://review.opendev.org/c/openstack/networking-baremetal/+/78343207:36
openstackgerritMerged openstack/networking-generic-switch master: Update master for stable/wallaby  https://review.opendev.org/c/openstack/networking-generic-switch/+/78342807:37
openstackgerritArne Wiebalck proposed openstack/ironic master: [doc] Update available software RAID levels.  https://review.opendev.org/c/openstack/ironic/+/78361107:37
*** nvalingo has quit IRC07:39
openstackgerritArne Wiebalck proposed openstack/ironic master: [doc] Warning about out-of-sync ESPs for UEFI software RAID  https://review.opendev.org/c/openstack/ironic/+/78361407:48
*** ociuhandu has joined #openstack-ironic07:59
*** lucasagomes has joined #openstack-ironic08:02
openstackgerritYogesh proposed openstack/ironic master: Add idrac HW type IPMI interface support  https://review.opendev.org/c/openstack/ironic/+/77186208:06
*** ociuhandu has quit IRC08:10
*** anuradha1904 has joined #openstack-ironic08:14
*** derekh has joined #openstack-ironic08:27
*** tosky has joined #openstack-ironic08:28
*** derekh has quit IRC08:31
*** ociuhandu has joined #openstack-ironic08:31
*** derekh has joined #openstack-ironic08:31
*** ociuhandu has quit IRC08:37
*** ociuhandu has joined #openstack-ironic08:38
*** rcernin has joined #openstack-ironic08:48
*** rcernin has quit IRC09:00
*** k_mouza has joined #openstack-ironic09:01
*** tosin has joined #openstack-ironic09:03
openstackgerritArun S A G proposed openstack/ironic master: Add agent_state and agent_status params to heartbeat  https://review.opendev.org/c/openstack/ironic/+/77805809:07
openstackgerritArun S A G proposed openstack/ironic master: Add support for configdrive in anaconda interface  https://review.opendev.org/c/openstack/ironic/+/78039809:07
openstackgerritMerged openstack/networking-generic-switch master: Add Python3 xena unit tests  https://review.opendev.org/c/openstack/networking-generic-switch/+/78343109:17
openstackgerritMerged openstack/networking-generic-switch stable/wallaby: Update .gitreview for stable/wallaby  https://review.opendev.org/c/openstack/networking-generic-switch/+/78342209:17
openstackgerritMerged openstack/networking-generic-switch stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby  https://review.opendev.org/c/openstack/networking-generic-switch/+/78342509:17
*** Qianbiao has joined #openstack-ironic09:25
openstackgerritMerged openstack/networking-baremetal master: Add Python3 xena unit tests  https://review.opendev.org/c/openstack/networking-baremetal/+/78343409:29
openstackgerritMerged openstack/networking-baremetal stable/wallaby: Update .gitreview for stable/wallaby  https://review.opendev.org/c/openstack/networking-baremetal/+/78342709:29
openstackgerritMerged openstack/networking-baremetal stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby  https://review.opendev.org/c/openstack/networking-baremetal/+/78343009:29
*** dtantsur|afk is now known as dtantsur09:41
dtantsurmorning ironic09:42
rpittaugood morning dtantsur :)09:42
openstackgerritMerged openstack/ironic master: Add configuration mold storage  https://review.opendev.org/c/openstack/ironic/+/76992809:45
openstackgerritMerged openstack/ironic master: [doc] Update available software RAID levels.  https://review.opendev.org/c/openstack/ironic/+/78361109:46
dtantsurlate the party, but ajya, could you not add idrac-specific code to generic driver.utils and idrac options to [DEFAULT]?09:47
dtantsurthis is re https://review.opendev.org/c/openstack/ironic/+/76992809:48
ajyadtantsur: just on time late :) hm, could move it to drac section. This originated there as potentially this _could_ be reused by other drivers, but I guess if that happens, then can move to generic drivers09:52
*** rcernin has joined #openstack-ironic09:52
dtantsurajya: as a middle ground, we can have ironic.common.molds and a new [molds] section09:52
dtantsurthis way it's not explicitly idrac-specific, but is also cleanly separated from everything else09:53
ajyadtantsur: ok, I'll create a patch10:01
dtantsurthanks!10:01
*** rcernin has quit IRC10:07
*** rcernin has joined #openstack-ironic10:07
openstackgerritMerged openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port  https://review.opendev.org/c/openstack/ironic/+/69332810:07
*** ildikov has joined #openstack-ironic10:10
*** ociuhandu has quit IRC10:17
*** sshnaidm|afk has joined #openstack-ironic10:18
*** rcernin has quit IRC10:20
*** ociuhandu has joined #openstack-ironic10:21
*** rcernin has joined #openstack-ironic10:27
*** ociuhandu has quit IRC10:30
arne_wiebalckbye everyone o/10:35
* arne_wiebalck has two "morning-only" weeks :)10:36
jandersnice arne_wiebalck!10:37
jandersgood morning rpittau dtantsur ajya and Ironic o/10:37
*** ociuhandu has joined #openstack-ironic10:37
*** rcernin has quit IRC10:38
openstackgerritMerged openstack/ironic master: Automaticaly set cipher suite  https://review.opendev.org/c/openstack/ironic/+/77052610:51
*** rcernin has joined #openstack-ironic10:51
openstackgerritAija Jauntēva proposed openstack/ironic master: Move configuration mold utilities  https://review.opendev.org/c/openstack/ironic/+/78363010:54
ajyahey, janders10:56
*** moshiur has quit IRC11:01
*** rcernin has quit IRC11:05
*** k_mouza has quit IRC11:07
*** mkrai has joined #openstack-ironic11:08
*** ociuhandu has quit IRC11:11
*** k_mouza has joined #openstack-ironic11:17
*** anuradha1904 has quit IRC11:24
*** dtantsur is now known as dtantsur|brb11:28
openstackgerritKaifeng Wang proposed openstack/ironic master: Support port name: API  https://review.opendev.org/c/openstack/ironic/+/76556911:34
*** mkrai has quit IRC11:37
*** mkrai_ has joined #openstack-ironic11:37
*** ociuhandu has joined #openstack-ironic11:42
openstackgerritAija Jauntēva proposed openstack/ironic master: Move configuration mold utilities  https://review.opendev.org/c/openstack/ironic/+/78363011:43
*** sshnaidm|afk is now known as sshnaidm|off11:45
*** tosin has quit IRC11:53
*** mkrai_ has quit IRC11:55
iurygregorygood morning Ironic!11:57
* iurygregory now 5hrs diff from Europe XD11:57
*** rh-jelabarre has joined #openstack-ironic11:57
*** ociuhandu has quit IRC12:12
*** tosin has joined #openstack-ironic12:13
jandershey iurygregory12:28
iurygregoryjanders, o/12:29
rpittauhey iurygregory janders :)12:32
iurygregoryrpittau, o/12:32
*** ociuhandu has joined #openstack-ironic12:55
iurygregoryoh I just notice I need to send a follow-up D:13:02
iurygregorytks rpittau stevebaker for the reviews13:02
rpittauiurygregory: no problem :)13:03
*** ociuhandu has quit IRC13:04
openstackgerritJacob Anders proposed openstack/ironic master: Generic way to configure clean step priorites  https://review.opendev.org/c/openstack/ironic/+/74411713:05
janders^ hopefully this will fix the funny and embarrassing bug that stevebaker spotted, testing to follow13:07
janderssee you tomorrow Ironic o/13:07
iurygregorybye janders o/13:09
*** rloo has joined #openstack-ironic13:11
*** dtantsur|brb is now known as dtantsur13:15
TheJuliagood morning13:18
dtantsurmorning TheJulia13:19
dtantsurrelease time is upon us?13:20
TheJuliaYeah, this week13:21
rpittaudaylight release time?13:24
dtantsur:)13:24
TheJuliaajya: looks like https://review.opendev.org/c/openstack/ironic/+/748696 could use a quick revision... maybe?13:25
TheJuliaReviews on https://review.opendev.org/c/openstack/ironic/+/777448 would be appreciated13:26
ajyaTheJulia: Yogesh is working on updates to Dmitry's comments about backporting, will post updates if not today, then tomorrow13:26
rpittauTheJulia: I approved that now, didn't see Arun replied13:26
dtantsurwould be cool to get https://review.opendev.org/c/openstack/ironic/+/783312 in, but not critical13:27
dtantsur(and yes, I still feel awkward to have proposed that)13:27
TheJuliaFor anaconda, that leaves https://review.opendev.org/c/openstack/ironic/+/778058/10 which is super late for the release but... if we want anaconda to merge13:27
rpittaudtantsur: I have that in my list13:28
dtantsurTheJulia: not leaving the anaconda work half-finished is a serious enough reason13:28
dtantsuralthough I haven't checked how invasive the patches are13:28
dtantsurhave we seriously approved a spec with both "state" and "status" in the same payload? O__o13:29
* dtantsur should probably close this patch before he gets too frustrated13:29
rpittaummm I see only the agent_status in the specs13:30
iurygregorygood morning TheJulia =)13:30
TheJuliadtantsur: close enough it should be usable but the last one is the invasive patch :\13:31
dtantsurit won't work without the last patch? or work, but imperfectly?13:32
dtantsurthis is the key difference IMO13:32
TheJuliawell, there is one patch beyond that patch which is blocked until next cycle13:32
TheJuliathe invasive patch, I don't think it will entirely work without, but maybe someone could find it useful if we don't end up merging it.13:33
*** zaneb has joined #openstack-ironic13:35
*** tzumainn has joined #openstack-ironic13:42
* TheJulia pokes her home hardware lab router 13:46
*** ociuhandu has joined #openstack-ironic13:47
TheJuliadtantsur: w/r/t your network info schema patch, I've added it to the reviewjam list13:48
*** sdanni has joined #openstack-ironic13:54
dtantsurthx!13:57
dtantsurI'll be a bit late, I need a short break after the current meeting13:58
*** rcernin has joined #openstack-ironic14:01
*** rcernin has quit IRC14:06
*** ociuhandu has quit IRC14:17
*** ociuhandu has joined #openstack-ironic14:22
TheJuliaJayF: zer0c00l: Can we move the tuesday review jam up one hour?14:23
TheJulia10 AM instead of 11 AM.14:23
*** ociuhandu has quit IRC14:27
*** kaifeng has joined #openstack-ironic14:37
*** sdanni has quit IRC14:40
*** sdanni has joined #openstack-ironic14:41
TheJuliarpittau: so it seems *none* of our default config parameters are getting loaded14:47
TheJuliafor docs that is14:47
rpittauah14:47
TheJuliaoslo defaults do,  I'm starting to wonder if it is an ordering thing14:48
rpittauI'm ok in merging that patch, ta least we have the params in the config sample, then we can investigate further14:48
TheJuliayeah14:48
*** ociuhandu has joined #openstack-ironic14:51
*** ricolin_ has joined #openstack-ironic14:55
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Mar 29 15:00:24 2021 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
TheJuliaGood morning everyone!15:00
rpittauo/15:00
kaifengo/15:00
erbarro/15:00
ajyao/15:00
rpiosoo/15:00
TheJuliaOur agenda can be found on the Wiki this week15:00
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
dtantsuro/15:01
bdoddo/15:01
*** stendulker has joined #openstack-ironic15:01
TheJulia#topic Announcements / Reminders15:02
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)"15:02
rlooo/15:02
stendulkero/15:02
TheJulia#info This week is R-2, in other words ironic most likely must release for the cycle this week.15:02
TheJulia#info PTG will be April 19th through 23rd15:02
TheJulia#link https://april2021-ptg.eventbrite.com/15:02
TheJulia#link https://etherpad.opendev.org/p/ironic-xena-ptg15:03
TheJuliaOne last item, I will likely be taking the week before the PTG off.15:03
TheJuliaDoes anyone have anything to announce or remind us of this week?15:03
rpiosoBill Dodd will be retiring.15:04
rpiosobdodd :-)15:04
bdoddThat is true!15:04
TheJulia#info bdodd is retiring! \o/  Thank you Bill for all your hard work!15:04
ajyathanks bdodd and enjoy!15:05
bdoddThank all of you for your help and support!15:05
rpiosoThanks, again, bdodd, for all of your contributions, especially promoting Redfish in ironic :-)15:05
rlooCongrats bdodd! wait, now bdodd has more time to contribute to ironic! (just kidding)15:05
rpiosoLOL15:05
bdodd:)15:06
TheJuliarloo: I was going to wait a few days before trying the "but now you have time to contribute more!15:06
TheJuliaapproach :)15:06
rlooha ha, good luck with that approach TheJulia!15:06
TheJuliaAnyway, moving on, we had no action items from last week.15:07
rlooor err.. are you bored now bdodd? We've got some fun stuff...15:07
TheJuliaDo we want to review subteam statuses since it is basically release week?15:07
bdoddrloo Not bored yet. But if that changes I know where to come. :)15:08
TheJuliabdodd: Thanks :)15:08
TheJuliaI'm going to take the lack of responses for subteam updates, meaning we should just move on to priorities for the coming week15:08
dtantsuryep15:09
TheJulia#topic Deciding on priorities for the coming week15:09
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:09
TheJuliaSo we've basically entirely moved over to the hashtag in gerrit from the whiteboard, but are there any items that need attention or need visibility this week?15:10
TheJulia#link https://etherpad.opendev.org/p/IronicWhiteBoard15:10
TheJuliaLine 11215:10
dtantsurhttps://review.opendev.org/c/openstack/ironic/+/744117 is already there15:10
dtantsurbut would be nice to Jacob to get it in, he's spent significant time on it this cycle15:10
TheJuliaYeah15:11
TheJuliaIdeally I'd like to get the repository in ready to go condition by wednesday15:11
rlooof those, which ones are highly desirable for the release?15:11
*** bburns has joined #openstack-ironic15:11
TheJuliaI'd like to see the anaconda stuff get in15:12
dtantsurajya: FYI https://review.opendev.org/c/openstack/ironic/+/759428 needs rebasing15:12
*** jamesdenton has quit IRC15:12
ajyacan this be revisited, it got rebased and removed dependency from another patch - https://review.opendev.org/c/openstack/ironic/+/75582115:12
TheJuliaThe tempest plugin items are not required15:12
ajyadtantsur: yup, will rebase today15:13
TheJuliaajya: adding hashtag15:13
*** jamesdenton has joined #openstack-ironic15:13
ajyaTheJulia: thanks15:13
rloowould it be ok to remove the hashtag for the ones that are not required? (if it helps wrt prioritizing for the release?)15:13
dtantsurajya: approved15:13
openstackgerritMerged openstack/ironic master: Validate the kickstart template and file before use  https://review.opendev.org/c/openstack/ironic/+/77744815:14
rpioso:-)15:14
TheJuliarloo: sure15:14
openstackgerritMerged openstack/ironic-python-agent stable/train: Do not try to read raw disk data as unicode  https://review.opendev.org/c/openstack/ironic-python-agent/+/78288115:14
rlooand the goal is to review/merge by EOD Wed?15:14
ajyathanks, dtantsur15:14
dtantsurreview by EOD Tue15:14
TheJulia++15:15
dtantsurto allow for a recheck party :)15:15
rloooh. okay...15:15
TheJuliaI'm going to remove the dib item, since the required diskimage-builder patch has not gone anywhere15:15
TheJuliawell, ipab15:16
openstackgerritYogesh proposed openstack/ironic master: Add idrac HW type IPMI interface support  https://review.opendev.org/c/openstack/ironic/+/77186215:17
TheJuliaI think the current state of https://review.opendev.org/q/hashtag:ironic-week-prio+status:open looks good to me15:18
rpittauyup15:19
dtantsuryeah15:19
TheJuliaWell, I guess we're good to move on to Discussion?15:19
rpiosoTheJulia: https://review.opendev.org/c/openstack/ironic/+/771862 is a Dell priority.15:20
TheJuliaOkay, but it has two -2's. You need to engage discussion with those reviewers to get them to reconsider or re-evaluate their stance based on the current state of the patch.15:22
rpiosoTheJulia: Will do.15:23
dtantsurif "idrac-ipmitool-shellinabox" in patch: patch.apply(-2)15:23
TheJuliaOkay...15:24
TheJuliaMoving on to Discussion since we have a meeting agenda15:24
TheJulia#topic Discussion15:24
*** openstack changes topic to "Discussion (Meeting topic: ironic)"15:24
TheJuliaWe have one discussion item today, specifically adjustment of standing items like Friday SPUC and Tuesday review jam since EU daylight savings time has struck everyone in the northern hemesphere.15:25
TheJuliaI *believe* we want to try and see if there is consensus to adjust the times forward by an ?hour? or 2 for each?15:25
dtantsurfor the morning instance I'd like to keep the same *local* time, even though initially SPUC was announced in UTC timezone15:26
dtantsurbut I need to check with Jacob if that's okay15:26
TheJuliaI think that is okay15:27
TheJuliaAre there any objections to possibly moving the times a little bit?15:28
TheJuliaarne_wiebalck: anything for the baremetal sig?15:28
dtantsurjanders: if I forget to ping you tomorrow ^^^15:28
* TheJulia hears no objections15:29
TheJuliaMoving on then15:29
TheJulia#topic Baremetal SIG15:29
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)"15:29
TheJuliaThe only item I'm aware of is I volunteered to talk about Secure RBAC in two weeks :)15:29
TheJuliaTuesday, April 12th, 2021 @ 2pm UTC15:30
TheJulia#link https://etherpad.opendev.org/p/bare-metal-sig15:30
TheJuliaLooks like arne_wiebalck is AFK today15:30
TheJuliaAnyhow, if nobody has anything sig related, we can move on to RFE review15:30
dtantsur++15:31
TheJulia#topic RFE Review15:31
*** openstack changes topic to "RFE Review (Meeting topic: ironic)"15:31
TheJuliaWe have on item to review today. A contributor has proposed improving publish_image to be friendlier to non-openstack environments. Or at least, non-pure openstack environments.15:31
dtantsuroh wow15:31
TheJulia#link https://storyboard.openstack.org/#!/story/200867115:32
TheJuliathe tl;dr is to enable support for S315:32
dtantsurThe idea doesn't sound crazy to me, although the implementation probably will be a bit crazy :)15:32
*** nvalingo has joined #openstack-ironic15:32
TheJuliaWhich, to me, actually makes a lot of sense because they are absolutely correct, to use swift, or even the swift model APIs, it is all predicated on using keystone as the authentication source, so it becomes harder to integrate15:33
dtantsurI suspect $HOME won't work, everything else sounds doable15:33
TheJuliaeh, ENV['HOME'] ;)15:34
TheJuliaBut yeah15:34
TheJuliaI was kind of thinking the same15:34
TheJuliaAny objections or concerns out there?15:34
dtantsurI put a question to the RFE, but it's not really blocking15:35
rpittaunot really, I'm very curious to see the implementation :)15:35
dtantsuryeah. only virtual media seems in scope, I wonder if everything else of interest15:35
dtantsur(I've asked on the RFE)15:36
TheJuliaIt would be nice if it introduces a clean shim that can be leveraged elsewhere15:36
TheJuliaAnyway, I guess that comes down to what does the code look like15:36
dtantsuryep, similar to how swift is handled15:37
TheJuliaAnyway, I left a comment as well15:37
ajyaagree, could be used elsewhere too :)15:38
TheJulia++15:38
TheJuliaAnyway, I guess that means we're done with RFE review :)15:38
TheJulia#topic Open Discussion15:38
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:38
sdanniHi folks! lmcgann and I plan to submit a new change on security interface spec later today. Please take a look if you have time!15:39
TheJuliaAny one have anything they would like to discuss?15:39
dtantsurat a risk of getting shouted at: what would folks say about explicit support for CoreOS?15:39
dtantsurin a sense that our network_data procedure does not work for it15:39
TheJuliasdanni: greetings! I can try to take a look15:39
TheJuliadtantsur: like an explict api level field?15:40
TheJuliaon the provision call?15:40
dtantsurMore like a different way to inject network_data15:40
dtantsurthis cpio madness https://github.com/coreos/coreos-installer/blob/master/docs/iso-embed-ignition.md15:40
TheJuliaI guess it would really depend on the what and how.15:40
dtantsuryeah, I haven't really thought it through15:40
sdanniTheJulia: thanks!15:40
dtantsurmore like checking how much non Red Hat people are opposed to the very idea15:41
TheJuliadtantsur: wow15:41
rpittauinteresting15:41
dtantsurunrelated: we're receiving a pressure with releases: https://review.opendev.org/c/openstack/releases/+/78366515:42
dtantsurI think bifrost and inspector should be good to go15:43
*** rh-jelabarre has quit IRC15:43
rpittauthat reminds me15:43
rpittauthere are 2 quick bifrost patches15:43
*** rh-jelabarre has joined #openstack-ironic15:43
TheJuliaI'm really starting to actually feel a great dislike of the release team's behavior15:43
rpittauhttps://review.opendev.org/c/openstack/bifrost/+/782775 https://review.opendev.org/c/openstack/bifrost/+/78270015:43
*** k_mouza has quit IRC15:44
*** rh-jelabarre has quit IRC15:44
*** k_mouza has joined #openstack-ironic15:44
*** rh-jlabarre has joined #openstack-ironic15:44
TheJuliadtantsur: I guess such explicit support for ignition would be okay for basic config injection15:45
openstackgerritMerged openstack/ironic master: Move configuration mold utilities  https://review.opendev.org/c/openstack/ironic/+/78363015:45
TheJuliawhat/how that would look like, I don't really know off-hand15:46
*** vkmc has quit IRC15:47
*** vkmc has joined #openstack-ironic15:48
dtantsurme neither. I'm not sure it's even doable without hardcoding too much coreos knowledge15:48
dtantsurmaybe it's time for a downstream deploy interface15:48
TheJuliaor a coreos vmedia specific deploy interface15:49
dtantsuryeah, I guess my question boils down to whether something like CoreOSDeploy is acceptable by the community15:49
dtantsur(and note that I'll accept no as an answer)15:50
*** Guest76033 is now known as jroll15:50
TheJuliadtantsur: can that discussion wait until the PTG?15:51
rlooit seems very specific to coreos. would it be useful for folks not at redhat?15:51
TheJuliathe "is it acceptable"15:51
dtantsurrloo: there seem to be some people using Fedora CoreOS. I've seen one of them in the wild :)15:51
rlooor email. does ptg have an operators track?15:51
dtantsurbut yes, it's very specific, and yes, most of people interested are related to Red Hat15:51
rlooseems a bit similar to the anaconda driver?15:52
dtantsurre PTG: the schedule is already packed with things that are arguably a higher priority for *upstream* (even if not for me)15:52
dtantsurwell.. yes. Anaconda is a precedent of adding support for a specific technology.15:52
rloo"similar" -- wrt who might be interested in it15:52
TheJuliaYou know, it could be this is a thing people wnat to use and support in a couple years generally15:53
dtantsurCoreOS is very far from anaconda. It's a whole world of its own.15:53
TheJuliaI'm at least mentally thinking devil's advocate with that though15:53
rloomaybe send out email then. although dunno what kind/if any response.15:53
dtantsurit could be a part of the Forum if/when it happens15:54
rloosimilar wrt useful for red-hat related folks.15:54
dtantsurI don't think we have enough operators reading the ML :(15:54
TheJuliaI do like the send out an email, and from there we could ping the operator's group on twitter and see if they can echo that out15:54
dtantsurI can do that15:55
TheJuliaThat seems like the "easy" feedback path as well15:55
TheJuliaIt occcurs to me if maybe a coreos ignition deploy interface could possibly also pass the additional parameters needed for early hardware initialization , but I suspect just networking is the bare minimum needed at the super low level.15:56
TheJuliaAnyway, I guess that is it for the week15:58
dtantsuryep. I've got some food for thought, thank you!15:58
TheJulia\o/15:59
TheJuliaThanks everyone! Have a wonderful week!15:59
TheJulia#endmeeting16:00
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list."16:00
openstackMeeting ended Mon Mar 29 16:00:01 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:00
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-29-15.00.html16:00
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-29-15.00.txt16:00
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-29-15.00.log.html16:00
*** lucasagomes has quit IRC16:02
*** derekh has quit IRC16:03
*** derekh has joined #openstack-ironic16:03
*** ociuhandu has quit IRC16:04
*** bdodd has quit IRC16:06
*** stendulker has quit IRC16:06
rpittaugoing to check if the sun is still there, see you tomorrow! o/16:08
*** rpittau is now known as rpittau|afk16:08
TheJuliao/16:08
openstackgerritMerged openstack/ironic master: Allow running RAID cleaning steps with in-band cleaning  https://review.opendev.org/c/openstack/ironic/+/78255216:09
openstackgerritMerged openstack/ironic master: Allow using per-site network_data schema  https://review.opendev.org/c/openstack/ironic/+/78331216:10
*** Qianbiao has quit IRC16:26
*** kafilat has joined #openstack-ironic16:29
*** paras333_ has joined #openstack-ironic16:32
openstackgerritAija Jauntēva proposed openstack/ironic master: Add import, export configuration to idrac-redfish  https://review.opendev.org/c/openstack/ironic/+/75942816:33
ajya^ rebased, please take a look16:36
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Validate vmedia for vmedia usage  https://review.opendev.org/c/openstack/ironic-python-agent/+/78371616:51
*** gyee has joined #openstack-ironic16:57
openstackgerritAija Jauntēva proposed openstack/ironic-specs master: Followup pre-defined system hw conf in single step  https://review.opendev.org/c/openstack/ironic-specs/+/78064517:07
*** zzzeek has quit IRC17:08
*** zzzeek has joined #openstack-ironic17:10
*** gyee has quit IRC17:14
*** gyee has joined #openstack-ironic17:14
openstackgerritMerged openstack/ironic master: Fix idrac-wsman BIOS step async error handling  https://review.opendev.org/c/openstack/ironic/+/75582117:25
openstackgerritMerged openstack/ironic master: Fix webserver_verify_ca config documentation  https://review.opendev.org/c/openstack/ironic/+/78347517:26
*** ricolin_ has quit IRC17:26
*** paras333_ has quit IRC17:29
*** gyee has quit IRC17:38
openstackgerritJulia Kreger proposed openstack/ironic master: Always add 'boot_method' vmedia in redfish/ilo vmedia boot  https://review.opendev.org/c/openstack/ironic/+/78372217:43
TheJuliadtantsur: ^17:44
TheJuliairmc uses purely static isos17:44
*** uzumaki has joined #openstack-ironic17:47
dtantsurTheJulia: I think ilo is capable of building ISOs too17:48
dtantsurah, you covered it17:49
dtantsurTheJulia: IloUefiHttpsBoot, do we count it as vmedia?17:49
*** gyee has joined #openstack-ironic17:49
dtantsurprobably yes, it uses prepare_deploy_iso17:50
TheJuliaI *thought* I changed both17:50
dtantsuryou did, I just didn't understand why at first17:51
dtantsurnow I do17:51
TheJuliaahh, okay17:51
dtantsurnow the IPA change with Depends-On?17:51
dtantsurand maybe nits fixed17:51
* dtantsur is staying online until that is done17:51
TheJuliadtantsur: one minute17:53
TheJuliagot derailed slightly17:54
TheJuliayay for unit tests17:57
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Validate vmedia for vmedia usage  https://review.opendev.org/c/openstack/ironic-python-agent/+/78371618:02
TheJuliadtantsur: ther eyou go18:02
dtantsurthanks!18:02
dtantsurI'm a bit too tired to check really carefully, but overall looks good18:06
dtantsurmaybe stevebaker can land a 2nd +2 on he's in18:06
dtantsurs/on/once/18:07
*** dtantsur is now known as dtantsur|afk18:07
dtantsur|afko/18:07
*** paras333 has joined #openstack-ironic18:07
TheJulia\o18:09
*** ayoung has quit IRC18:10
*** ayoung has joined #openstack-ironic18:13
*** kaifeng has quit IRC18:26
*** frigo has joined #openstack-ironic18:43
frigowow thanks for the review of https://storyboard.openstack.org/#!/story/2008671 !18:44
*** tosin has quit IRC18:53
*** Goneri has joined #openstack-ironic18:58
*** Goneri has quit IRC18:59
*** jamesdenton has quit IRC19:02
*** jamesden_ has joined #openstack-ironic19:02
*** viks____ has quit IRC19:04
*** zzzeek has quit IRC19:05
zer0c00lTheJulia: works for me!19:05
*** zzzeek has joined #openstack-ironic19:06
*** uzumaki has quit IRC19:08
*** uzumaki has joined #openstack-ironic19:08
*** frigo has quit IRC19:20
*** frigo has joined #openstack-ironic19:29
*** dsneddon has joined #openstack-ironic19:30
*** frigo has quit IRC19:33
stevebakerTheJulia: https://review.opendev.org/c/openstack/ironic/+/783722 just failed on a vmedia job19:57
zer0c00lTheJulia: i wonder if i should move the release notes to the heartbeat api change?20:03
TheJuliazer0c00l: likely20:04
TheJuliasigh20:06
TheJuliaI *think* we can remove the exception entirely20:11
TheJuliahmm20:16
TheJuliaI *think* these need to be disjointed20:16
TheJuliastevebaker: so what is happening, is there the current code after changes, basically always expects something to be there even thoguh it might have been embedded already in the vmedia. IPA's code doesn't handle that and the signifier flag at the same time20:18
TheJuliaso it raises an exception20:18
TheJulia... that was already noted to be removed later20:18
TheJuliastevebaker: replied to 783716, I did false because I couldn't see a legitimate case where it could happen now()20:21
stevebakerok20:22
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Validate vmedia for vmedia usage  https://review.opendev.org/c/openstack/ironic-python-agent/+/78371620:22
TheJulia^^ might have to just fix IPA first instead of try to force the flag out in all cases in ironic.20:23
TheJuliastevebaker: yeah, if that independently passes CI, then we should be good and we can just make that entirely independent of ironic.20:26
stevebakeryep, cool cool20:27
TheJuliaI guess while waiting, might be a good time to go exercise20:32
*** k_mouza has quit IRC20:34
openstackgerritLeo McGann proposed openstack/ironic-specs master: Add security interface spec  https://review.opendev.org/c/openstack/ironic-specs/+/57671820:47
*** derekh has quit IRC20:56
*** akahat has quit IRC20:56
*** kafilat has quit IRC20:57
*** livelace has joined #openstack-ironic21:05
openstackgerritLeo McGann proposed openstack/ironic-specs master: Add security interface spec  https://review.opendev.org/c/openstack/ironic-specs/+/57671821:08
*** akahat has joined #openstack-ironic21:20
TheJuliazer0c00l: you around?21:29
openstackgerritJulia Kreger proposed openstack/ironic master: Fix Bandit check  https://review.opendev.org/c/openstack/ironic/+/78374921:44
TheJuliazer0c00l: ^^21:44
TheJuliastevebaker: if you could +2+A that, I would be super appreciativ21:45
TheJuliasince it seems the gate is blocked otherwise21:45
stevebakerboop21:45
TheJuliaerr21:45
TheJulianeed to modify it a little more21:45
openstackgerritJulia Kreger proposed openstack/ironic master: Fix Bandit check  https://review.opendev.org/c/openstack/ironic/+/78374921:46
TheJuliaI played around trying to figure out why it was so unhappy21:46
TheJuliabut that seems to be happier now21:46
TheJuliastevebaker: looks like the ipa change is good21:51
TheJuliait is past all of the voting jobs now21:51
stevebakercool, I'll keep an eye on the other one21:53
TheJuliaI really should go exercise21:53
* TheJulia spends time reviewing and can then go exercise21:54
*** jamesdenton has joined #openstack-ironic21:56
*** jamesden_ has quit IRC21:59
*** jamesden_ has joined #openstack-ironic22:07
openstackgerritMerged openstack/ironic-python-agent stable/victoria: Add fstab pointer to EFI partition  https://review.opendev.org/c/openstack/ironic-python-agent/+/78234322:08
*** rloo has quit IRC22:09
*** jamesdenton has quit IRC22:09
TheJuliarpioso: trying to remember, was https://review.opendev.org/c/openstack/ironic/+/748696 one of the ones you were going to update today since it needs a rebase?22:10
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Refactor part of image module  https://review.opendev.org/c/openstack/ironic-python-agent/+/78234822:12
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Mount all vfat partitions before calling grub2  https://review.opendev.org/c/openstack/ironic-python-agent/+/78234922:12
openstackgerritVerification of a change to openstack/ironic-python-agent failed: More refactoring of the image module  https://review.opendev.org/c/openstack/ironic-python-agent/+/78235022:12
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Option to enable bootloader config failure bypass  https://review.opendev.org/c/openstack/ironic-python-agent/+/78156922:12
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Prevent broken partition image UEFI deploys  https://review.opendev.org/c/openstack/ironic-python-agent/+/78235122:12
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Add fstab pointer to EFI partition  https://review.opendev.org/c/openstack/ironic-python-agent/+/78235222:12
rpiosoTheJulia: Nope, the owner will. He's also addressing dtantsur's comment about backporting it to stable/ussuri. That should be completed very early tomorrow morning in North America, hopefully, while we're asleep.22:14
TheJulia\o/22:20
TheJuliaAwesome, thanks22:20
zer0c00lTheJulia: yes22:20
zer0c00lwas having lunch.22:20
TheJuliaI pushed https://review.opendev.org/c/openstack/ironic/+/783749 as your change seemed to intermittently pass bandit checking22:21
zer0c00lTheJulia:  thanks sorry about that, wonder if it passed when we +1ed.22:23
TheJuliaI *think* so22:23
TheJulianot 100% sure, it just seems really weird22:23
*** rcernin has joined #openstack-ironic22:24
*** rcernin has quit IRC22:24
jandersgood morning Ironic o/22:24
*** rcernin has joined #openstack-ironic22:24
TheJuliao/ janders22:24
jandershi TheJulia22:25
jandershow is it going?22:25
*** jamesdenton has joined #openstack-ironic22:25
TheJuliagood, https://review.opendev.org/c/openstack/ironic/+/744117 may require attention :)22:25
*** jamesden_ has quit IRC22:28
jandersthat's the top of my list today :) on it22:35
*** derekh has joined #openstack-ironic22:48
*** livelace has quit IRC22:55
openstackgerritJulia Kreger proposed openstack/ironic master: Fix configuration generation for ironic doc pages  https://review.opendev.org/c/openstack/ironic/+/78375622:57
rpiosoTheJulia: I created tag 2.1.0 in sushy-oem-idrac. Zuul is running the release pipeline job. Once it is published to PyPI, I will push a new patch set for https://review.opendev.org/c/openstack/ironic/+/721593, which addresses the comments we discussed this morning.23:26
openstackgerritArun S A G proposed openstack/ironic master: Add agent_state and agent_status params to heartbeat  https://review.opendev.org/c/openstack/ironic/+/77805823:37
openstackgerritArun S A G proposed openstack/ironic master: Add support for configdrive in anaconda interface  https://review.opendev.org/c/openstack/ironic/+/78039823:37
openstackgerritMerged openstack/ironic-python-agent master: Validate vmedia for vmedia usage  https://review.opendev.org/c/openstack/ironic-python-agent/+/78371623:38
openstackgerritMerged openstack/ironic master: [doc] Warning about out-of-sync ESPs for UEFI software RAID  https://review.opendev.org/c/openstack/ironic/+/78361423:38
iurygregorymorning janders o/23:38
openstackgerritRichard G. Pioso proposed openstack/ironic master: Add iDRAC management via Redfish to idrac HW type  https://review.opendev.org/c/openstack/ironic/+/72159323:38
rpiosoTheJulia: Done23:51
« Sunday, 2021-03-28 Index Tuesday, 2021-03-30 »

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