*** rcernin has joined #openstack-ironic | 00:00 | |
openstackgerrit | liuyuanfeng 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/+/783134 | 01:11 |
---|---|---|
openstackgerrit | ankit proposed openstack/ironic master: Add security dashboard clean steps to ilo drivers https://review.opendev.org/c/openstack/ironic/+/782364 | 01:58 |
*** zzzeek has quit IRC | 02:04 | |
*** zzzeek has joined #openstack-ironic | 02:06 | |
*** sshnaidm|afk has quit IRC | 02:06 | |
*** jamesdenton has quit IRC | 02:45 | |
*** jamesdenton has joined #openstack-ironic | 02:45 | |
*** rcernin has quit IRC | 03:06 | |
*** rcernin has joined #openstack-ironic | 03:11 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Automaticaly set cipher suite https://review.opendev.org/c/openstack/ironic/+/770526 | 03:51 |
*** jamesdenton has quit IRC | 04:03 | |
*** jamesden_ has joined #openstack-ironic | 04:03 | |
*** k_mouza has joined #openstack-ironic | 04:07 | |
*** k_mouza has quit IRC | 04:12 | |
*** ricolin has quit IRC | 04:35 | |
*** jamesden_ has quit IRC | 04:37 | |
*** jamesdenton has joined #openstack-ironic | 04:37 | |
*** ricolin has joined #openstack-ironic | 04:48 | |
*** jamesdenton has quit IRC | 05:14 | |
*** jamesdenton has joined #openstack-ironic | 05:16 | |
openstackgerrit | Merged openstack/ironic master: update grub2 file name https://review.opendev.org/c/openstack/ironic/+/780481 | 05:25 |
*** bdodd has quit IRC | 05:25 | |
*** bdodd has joined #openstack-ironic | 05:32 | |
*** bdodd has quit IRC | 05:32 | |
*** bdodd has joined #openstack-ironic | 05:39 | |
*** moshiur has joined #openstack-ironic | 06:06 | |
*** k_mouza has joined #openstack-ironic | 06:10 | |
*** viks____ has joined #openstack-ironic | 06:11 | |
arne_wiebalck | Good morning, ironic! | 06:12 |
*** k_mouza has quit IRC | 06:14 | |
janders | hey arne_wiebalck :) | 06:14 |
*** k_mouza has joined #openstack-ironic | 06:14 | |
arne_wiebalck | hey janders o/ | 06:14 |
*** k_mouza has quit IRC | 06:18 | |
*** uzumaki has quit IRC | 06:41 | |
*** uzumaki has joined #openstack-ironic | 06:44 | |
*** uzumaki has quit IRC | 07:00 | |
*** jamesdenton has quit IRC | 07:10 | |
*** jamesdenton has joined #openstack-ironic | 07:10 | |
*** rpittau|afk is now known as rpittau | 07:15 | |
rpittau | good morning ironic! | 07:15 |
rpittau | JayF: 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/+/783270 | 07:17 |
rpittau | moshiur: hopefully the bug will get your patch moving | 07:21 |
*** rcernin has quit IRC | 07:29 | |
openstackgerrit | Merged openstack/networking-baremetal master: Update master for stable/wallaby https://review.opendev.org/c/openstack/networking-baremetal/+/783432 | 07:36 |
openstackgerrit | Merged openstack/networking-generic-switch master: Update master for stable/wallaby https://review.opendev.org/c/openstack/networking-generic-switch/+/783428 | 07:37 |
openstackgerrit | Arne Wiebalck proposed openstack/ironic master: [doc] Update available software RAID levels. https://review.opendev.org/c/openstack/ironic/+/783611 | 07:37 |
*** nvalingo has quit IRC | 07:39 | |
openstackgerrit | Arne Wiebalck proposed openstack/ironic master: [doc] Warning about out-of-sync ESPs for UEFI software RAID https://review.opendev.org/c/openstack/ironic/+/783614 | 07:48 |
*** ociuhandu has joined #openstack-ironic | 07:59 | |
*** lucasagomes has joined #openstack-ironic | 08:02 | |
openstackgerrit | Yogesh proposed openstack/ironic master: Add idrac HW type IPMI interface support https://review.opendev.org/c/openstack/ironic/+/771862 | 08:06 |
*** ociuhandu has quit IRC | 08:10 | |
*** anuradha1904 has joined #openstack-ironic | 08:14 | |
*** derekh has joined #openstack-ironic | 08:27 | |
*** tosky has joined #openstack-ironic | 08:28 | |
*** derekh has quit IRC | 08:31 | |
*** ociuhandu has joined #openstack-ironic | 08:31 | |
*** derekh has joined #openstack-ironic | 08:31 | |
*** ociuhandu has quit IRC | 08:37 | |
*** ociuhandu has joined #openstack-ironic | 08:38 | |
*** rcernin has joined #openstack-ironic | 08:48 | |
*** rcernin has quit IRC | 09:00 | |
*** k_mouza has joined #openstack-ironic | 09:01 | |
*** tosin has joined #openstack-ironic | 09:03 | |
openstackgerrit | Arun S A G proposed openstack/ironic master: Add agent_state and agent_status params to heartbeat https://review.opendev.org/c/openstack/ironic/+/778058 | 09:07 |
openstackgerrit | Arun S A G proposed openstack/ironic master: Add support for configdrive in anaconda interface https://review.opendev.org/c/openstack/ironic/+/780398 | 09:07 |
openstackgerrit | Merged openstack/networking-generic-switch master: Add Python3 xena unit tests https://review.opendev.org/c/openstack/networking-generic-switch/+/783431 | 09:17 |
openstackgerrit | Merged openstack/networking-generic-switch stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/networking-generic-switch/+/783422 | 09:17 |
openstackgerrit | Merged openstack/networking-generic-switch stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/networking-generic-switch/+/783425 | 09:17 |
*** Qianbiao has joined #openstack-ironic | 09:25 | |
openstackgerrit | Merged openstack/networking-baremetal master: Add Python3 xena unit tests https://review.opendev.org/c/openstack/networking-baremetal/+/783434 | 09:29 |
openstackgerrit | Merged openstack/networking-baremetal stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/networking-baremetal/+/783427 | 09:29 |
openstackgerrit | Merged openstack/networking-baremetal stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/networking-baremetal/+/783430 | 09:29 |
*** dtantsur|afk is now known as dtantsur | 09:41 | |
dtantsur | morning ironic | 09:42 |
rpittau | good morning dtantsur :) | 09:42 |
openstackgerrit | Merged openstack/ironic master: Add configuration mold storage https://review.opendev.org/c/openstack/ironic/+/769928 | 09:45 |
openstackgerrit | Merged openstack/ironic master: [doc] Update available software RAID levels. https://review.opendev.org/c/openstack/ironic/+/783611 | 09:46 |
dtantsur | late the party, but ajya, could you not add idrac-specific code to generic driver.utils and idrac options to [DEFAULT]? | 09:47 |
dtantsur | this is re https://review.opendev.org/c/openstack/ironic/+/769928 | 09:48 |
ajya | dtantsur: 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 drivers | 09:52 |
*** rcernin has joined #openstack-ironic | 09:52 | |
dtantsur | ajya: as a middle ground, we can have ironic.common.molds and a new [molds] section | 09:52 |
dtantsur | this way it's not explicitly idrac-specific, but is also cleanly separated from everything else | 09:53 |
ajya | dtantsur: ok, I'll create a patch | 10:01 |
dtantsur | thanks! | 10:01 |
*** rcernin has quit IRC | 10:07 | |
*** rcernin has joined #openstack-ironic | 10:07 | |
openstackgerrit | Merged openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port https://review.opendev.org/c/openstack/ironic/+/693328 | 10:07 |
*** ildikov has joined #openstack-ironic | 10:10 | |
*** ociuhandu has quit IRC | 10:17 | |
*** sshnaidm|afk has joined #openstack-ironic | 10:18 | |
*** rcernin has quit IRC | 10:20 | |
*** ociuhandu has joined #openstack-ironic | 10:21 | |
*** rcernin has joined #openstack-ironic | 10:27 | |
*** ociuhandu has quit IRC | 10:30 | |
arne_wiebalck | bye everyone o/ | 10:35 |
* arne_wiebalck has two "morning-only" weeks :) | 10:36 | |
janders | nice arne_wiebalck! | 10:37 |
janders | good morning rpittau dtantsur ajya and Ironic o/ | 10:37 |
*** ociuhandu has joined #openstack-ironic | 10:37 | |
*** rcernin has quit IRC | 10:38 | |
openstackgerrit | Merged openstack/ironic master: Automaticaly set cipher suite https://review.opendev.org/c/openstack/ironic/+/770526 | 10:51 |
*** rcernin has joined #openstack-ironic | 10:51 | |
openstackgerrit | Aija Jauntēva proposed openstack/ironic master: Move configuration mold utilities https://review.opendev.org/c/openstack/ironic/+/783630 | 10:54 |
ajya | hey, janders | 10:56 |
*** moshiur has quit IRC | 11:01 | |
*** rcernin has quit IRC | 11:05 | |
*** k_mouza has quit IRC | 11:07 | |
*** mkrai has joined #openstack-ironic | 11:08 | |
*** ociuhandu has quit IRC | 11:11 | |
*** k_mouza has joined #openstack-ironic | 11:17 | |
*** anuradha1904 has quit IRC | 11:24 | |
*** dtantsur is now known as dtantsur|brb | 11:28 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Support port name: API https://review.opendev.org/c/openstack/ironic/+/765569 | 11:34 |
*** mkrai has quit IRC | 11:37 | |
*** mkrai_ has joined #openstack-ironic | 11:37 | |
*** ociuhandu has joined #openstack-ironic | 11:42 | |
openstackgerrit | Aija Jauntēva proposed openstack/ironic master: Move configuration mold utilities https://review.opendev.org/c/openstack/ironic/+/783630 | 11:43 |
*** sshnaidm|afk is now known as sshnaidm|off | 11:45 | |
*** tosin has quit IRC | 11:53 | |
*** mkrai_ has quit IRC | 11:55 | |
iurygregory | good morning Ironic! | 11:57 |
* iurygregory now 5hrs diff from Europe XD | 11:57 | |
*** rh-jelabarre has joined #openstack-ironic | 11:57 | |
*** ociuhandu has quit IRC | 12:12 | |
*** tosin has joined #openstack-ironic | 12:13 | |
janders | hey iurygregory | 12:28 |
iurygregory | janders, o/ | 12:29 |
rpittau | hey iurygregory janders :) | 12:32 |
iurygregory | rpittau, o/ | 12:32 |
*** ociuhandu has joined #openstack-ironic | 12:55 | |
iurygregory | oh I just notice I need to send a follow-up D: | 13:02 |
iurygregory | tks rpittau stevebaker for the reviews | 13:02 |
rpittau | iurygregory: no problem :) | 13:03 |
*** ociuhandu has quit IRC | 13:04 | |
openstackgerrit | Jacob Anders proposed openstack/ironic master: Generic way to configure clean step priorites https://review.opendev.org/c/openstack/ironic/+/744117 | 13:05 |
janders | ^ hopefully this will fix the funny and embarrassing bug that stevebaker spotted, testing to follow | 13:07 |
janders | see you tomorrow Ironic o/ | 13:07 |
iurygregory | bye janders o/ | 13:09 |
*** rloo has joined #openstack-ironic | 13:11 | |
*** dtantsur|brb is now known as dtantsur | 13:15 | |
TheJulia | good morning | 13:18 |
dtantsur | morning TheJulia | 13:19 |
dtantsur | release time is upon us? | 13:20 |
TheJulia | Yeah, this week | 13:21 |
rpittau | daylight release time? | 13:24 |
dtantsur | :) | 13:24 |
TheJulia | ajya: looks like https://review.opendev.org/c/openstack/ironic/+/748696 could use a quick revision... maybe? | 13:25 |
TheJulia | Reviews on https://review.opendev.org/c/openstack/ironic/+/777448 would be appreciated | 13:26 |
ajya | TheJulia: Yogesh is working on updates to Dmitry's comments about backporting, will post updates if not today, then tomorrow | 13:26 |
rpittau | TheJulia: I approved that now, didn't see Arun replied | 13:26 |
dtantsur | would be cool to get https://review.opendev.org/c/openstack/ironic/+/783312 in, but not critical | 13:27 |
dtantsur | (and yes, I still feel awkward to have proposed that) | 13:27 |
TheJulia | For 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 merge | 13:27 |
rpittau | dtantsur: I have that in my list | 13:28 |
dtantsur | TheJulia: not leaving the anaconda work half-finished is a serious enough reason | 13:28 |
dtantsur | although I haven't checked how invasive the patches are | 13:28 |
dtantsur | have we seriously approved a spec with both "state" and "status" in the same payload? O__o | 13:29 |
* dtantsur should probably close this patch before he gets too frustrated | 13:29 | |
rpittau | mmm I see only the agent_status in the specs | 13:30 |
iurygregory | good morning TheJulia =) | 13:30 |
TheJulia | dtantsur: close enough it should be usable but the last one is the invasive patch :\ | 13:31 |
dtantsur | it won't work without the last patch? or work, but imperfectly? | 13:32 |
dtantsur | this is the key difference IMO | 13:32 |
TheJulia | well, there is one patch beyond that patch which is blocked until next cycle | 13:32 |
TheJulia | the 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-ironic | 13:35 | |
*** tzumainn has joined #openstack-ironic | 13:42 | |
* TheJulia pokes her home hardware lab router | 13:46 | |
*** ociuhandu has joined #openstack-ironic | 13:47 | |
TheJulia | dtantsur: w/r/t your network info schema patch, I've added it to the reviewjam list | 13:48 |
*** sdanni has joined #openstack-ironic | 13:54 | |
dtantsur | thx! | 13:57 |
dtantsur | I'll be a bit late, I need a short break after the current meeting | 13:58 |
*** rcernin has joined #openstack-ironic | 14:01 | |
*** rcernin has quit IRC | 14:06 | |
*** ociuhandu has quit IRC | 14:17 | |
*** ociuhandu has joined #openstack-ironic | 14:22 | |
TheJulia | JayF: zer0c00l: Can we move the tuesday review jam up one hour? | 14:23 |
TheJulia | 10 AM instead of 11 AM. | 14:23 |
*** ociuhandu has quit IRC | 14:27 | |
*** kaifeng has joined #openstack-ironic | 14:37 | |
*** sdanni has quit IRC | 14:40 | |
*** sdanni has joined #openstack-ironic | 14:41 | |
TheJulia | rpittau: so it seems *none* of our default config parameters are getting loaded | 14:47 |
TheJulia | for docs that is | 14:47 |
rpittau | ah | 14:47 |
TheJulia | oslo defaults do, I'm starting to wonder if it is an ordering thing | 14:48 |
rpittau | I'm ok in merging that patch, ta least we have the params in the config sample, then we can investigate further | 14:48 |
TheJulia | yeah | 14:48 |
*** ociuhandu has joined #openstack-ironic | 14:51 | |
*** ricolin_ has joined #openstack-ironic | 14:55 | |
TheJulia | #startmeeting ironic | 15:00 |
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
*** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
openstack | The meeting name has been set to 'ironic' | 15:00 |
TheJulia | Good morning everyone! | 15:00 |
rpittau | o/ | 15:00 |
kaifeng | o/ | 15:00 |
erbarr | o/ | 15:00 |
ajya | o/ | 15:00 |
rpioso | o/ | 15:00 |
TheJulia | Our agenda can be found on the Wiki this week | 15:00 |
TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
dtantsur | o/ | 15:01 |
bdodd | o/ | 15:01 |
*** stendulker has joined #openstack-ironic | 15:01 | |
TheJulia | #topic Announcements / Reminders | 15:02 |
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)" | 15:02 | |
rloo | o/ | 15:02 |
stendulker | o/ | 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 23rd | 15:02 |
TheJulia | #link https://april2021-ptg.eventbrite.com/ | 15:02 |
TheJulia | #link https://etherpad.opendev.org/p/ironic-xena-ptg | 15:03 |
TheJulia | One last item, I will likely be taking the week before the PTG off. | 15:03 |
TheJulia | Does anyone have anything to announce or remind us of this week? | 15:03 |
rpioso | Bill Dodd will be retiring. | 15:04 |
rpioso | bdodd :-) | 15:04 |
bdodd | That is true! | 15:04 |
TheJulia | #info bdodd is retiring! \o/ Thank you Bill for all your hard work! | 15:04 |
ajya | thanks bdodd and enjoy! | 15:05 |
bdodd | Thank all of you for your help and support! | 15:05 |
rpioso | Thanks, again, bdodd, for all of your contributions, especially promoting Redfish in ironic :-) | 15:05 |
rloo | Congrats bdodd! wait, now bdodd has more time to contribute to ironic! (just kidding) | 15:05 |
rpioso | LOL | 15:05 |
bdodd | :) | 15:06 |
TheJulia | rloo: I was going to wait a few days before trying the "but now you have time to contribute more! | 15:06 |
TheJulia | approach :) | 15:06 |
rloo | ha ha, good luck with that approach TheJulia! | 15:06 |
TheJulia | Anyway, moving on, we had no action items from last week. | 15:07 |
rloo | or err.. are you bored now bdodd? We've got some fun stuff... | 15:07 |
TheJulia | Do we want to review subteam statuses since it is basically release week? | 15:07 |
bdodd | rloo Not bored yet. But if that changes I know where to come. :) | 15:08 |
TheJulia | bdodd: Thanks :) | 15:08 |
TheJulia | I'm going to take the lack of responses for subteam updates, meaning we should just move on to priorities for the coming week | 15:08 |
dtantsur | yep | 15:09 |
TheJulia | #topic Deciding on priorities for the coming week | 15:09 |
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:09 | |
TheJulia | So 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/IronicWhiteBoard | 15:10 |
TheJulia | Line 112 | 15:10 |
dtantsur | https://review.opendev.org/c/openstack/ironic/+/744117 is already there | 15:10 |
dtantsur | but would be nice to Jacob to get it in, he's spent significant time on it this cycle | 15:10 |
TheJulia | Yeah | 15:11 |
TheJulia | Ideally I'd like to get the repository in ready to go condition by wednesday | 15:11 |
rloo | of those, which ones are highly desirable for the release? | 15:11 |
*** bburns has joined #openstack-ironic | 15:11 | |
TheJulia | I'd like to see the anaconda stuff get in | 15:12 |
dtantsur | ajya: FYI https://review.opendev.org/c/openstack/ironic/+/759428 needs rebasing | 15:12 |
*** jamesdenton has quit IRC | 15:12 | |
ajya | can this be revisited, it got rebased and removed dependency from another patch - https://review.opendev.org/c/openstack/ironic/+/755821 | 15:12 |
TheJulia | The tempest plugin items are not required | 15:12 |
ajya | dtantsur: yup, will rebase today | 15:13 |
TheJulia | ajya: adding hashtag | 15:13 |
*** jamesdenton has joined #openstack-ironic | 15:13 | |
ajya | TheJulia: thanks | 15:13 |
rloo | would it be ok to remove the hashtag for the ones that are not required? (if it helps wrt prioritizing for the release?) | 15:13 |
dtantsur | ajya: approved | 15:13 |
openstackgerrit | Merged openstack/ironic master: Validate the kickstart template and file before use https://review.opendev.org/c/openstack/ironic/+/777448 | 15:14 |
rpioso | :-) | 15:14 |
TheJulia | rloo: sure | 15:14 |
openstackgerrit | Merged 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/+/782881 | 15:14 |
rloo | and the goal is to review/merge by EOD Wed? | 15:14 |
ajya | thanks, dtantsur | 15:14 |
dtantsur | review by EOD Tue | 15:14 |
TheJulia | ++ | 15:15 |
dtantsur | to allow for a recheck party :) | 15:15 |
rloo | oh. okay... | 15:15 |
TheJulia | I'm going to remove the dib item, since the required diskimage-builder patch has not gone anywhere | 15:15 |
TheJulia | well, ipab | 15:16 |
openstackgerrit | Yogesh proposed openstack/ironic master: Add idrac HW type IPMI interface support https://review.opendev.org/c/openstack/ironic/+/771862 | 15:17 |
TheJulia | I think the current state of https://review.opendev.org/q/hashtag:ironic-week-prio+status:open looks good to me | 15:18 |
rpittau | yup | 15:19 |
dtantsur | yeah | 15:19 |
TheJulia | Well, I guess we're good to move on to Discussion? | 15:19 |
rpioso | TheJulia: https://review.opendev.org/c/openstack/ironic/+/771862 is a Dell priority. | 15:20 |
TheJulia | Okay, 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 |
rpioso | TheJulia: Will do. | 15:23 |
dtantsur | if "idrac-ipmitool-shellinabox" in patch: patch.apply(-2) | 15:23 |
TheJulia | Okay... | 15:24 |
TheJulia | Moving on to Discussion since we have a meeting agenda | 15:24 |
TheJulia | #topic Discussion | 15:24 |
*** openstack changes topic to "Discussion (Meeting topic: ironic)" | 15:24 | |
TheJulia | We 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 |
TheJulia | I *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 |
dtantsur | for the morning instance I'd like to keep the same *local* time, even though initially SPUC was announced in UTC timezone | 15:26 |
dtantsur | but I need to check with Jacob if that's okay | 15:26 |
TheJulia | I think that is okay | 15:27 |
TheJulia | Are there any objections to possibly moving the times a little bit? | 15:28 |
TheJulia | arne_wiebalck: anything for the baremetal sig? | 15:28 |
dtantsur | janders: if I forget to ping you tomorrow ^^^ | 15:28 |
* TheJulia hears no objections | 15:29 | |
TheJulia | Moving on then | 15:29 |
TheJulia | #topic Baremetal SIG | 15:29 |
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)" | 15:29 | |
TheJulia | The only item I'm aware of is I volunteered to talk about Secure RBAC in two weeks :) | 15:29 |
TheJulia | Tuesday, April 12th, 2021 @ 2pm UTC | 15:30 |
TheJulia | #link https://etherpad.opendev.org/p/bare-metal-sig | 15:30 |
TheJulia | Looks like arne_wiebalck is AFK today | 15:30 |
TheJulia | Anyhow, if nobody has anything sig related, we can move on to RFE review | 15:30 |
dtantsur | ++ | 15:31 |
TheJulia | #topic RFE Review | 15:31 |
*** openstack changes topic to "RFE Review (Meeting topic: ironic)" | 15:31 | |
TheJulia | We 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 |
dtantsur | oh wow | 15:31 |
TheJulia | #link https://storyboard.openstack.org/#!/story/2008671 | 15:32 |
TheJulia | the tl;dr is to enable support for S3 | 15:32 |
dtantsur | The idea doesn't sound crazy to me, although the implementation probably will be a bit crazy :) | 15:32 |
*** nvalingo has joined #openstack-ironic | 15:32 | |
TheJulia | Which, 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 integrate | 15:33 |
dtantsur | I suspect $HOME won't work, everything else sounds doable | 15:33 |
TheJulia | eh, ENV['HOME'] ;) | 15:34 |
TheJulia | But yeah | 15:34 |
TheJulia | I was kind of thinking the same | 15:34 |
TheJulia | Any objections or concerns out there? | 15:34 |
dtantsur | I put a question to the RFE, but it's not really blocking | 15:35 |
rpittau | not really, I'm very curious to see the implementation :) | 15:35 |
dtantsur | yeah. only virtual media seems in scope, I wonder if everything else of interest | 15:35 |
dtantsur | (I've asked on the RFE) | 15:36 |
TheJulia | It would be nice if it introduces a clean shim that can be leveraged elsewhere | 15:36 |
TheJulia | Anyway, I guess that comes down to what does the code look like | 15:36 |
dtantsur | yep, similar to how swift is handled | 15:37 |
TheJulia | Anyway, I left a comment as well | 15:37 |
ajya | agree, could be used elsewhere too :) | 15:38 |
TheJulia | ++ | 15:38 |
TheJulia | Anyway, I guess that means we're done with RFE review :) | 15:38 |
TheJulia | #topic Open Discussion | 15:38 |
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:38 | |
sdanni | Hi 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 |
TheJulia | Any one have anything they would like to discuss? | 15:39 |
dtantsur | at a risk of getting shouted at: what would folks say about explicit support for CoreOS? | 15:39 |
dtantsur | in a sense that our network_data procedure does not work for it | 15:39 |
TheJulia | sdanni: greetings! I can try to take a look | 15:39 |
TheJulia | dtantsur: like an explict api level field? | 15:40 |
TheJulia | on the provision call? | 15:40 |
dtantsur | More like a different way to inject network_data | 15:40 |
dtantsur | this cpio madness https://github.com/coreos/coreos-installer/blob/master/docs/iso-embed-ignition.md | 15:40 |
TheJulia | I guess it would really depend on the what and how. | 15:40 |
dtantsur | yeah, I haven't really thought it through | 15:40 |
sdanni | TheJulia: thanks! | 15:40 |
dtantsur | more like checking how much non Red Hat people are opposed to the very idea | 15:41 |
TheJulia | dtantsur: wow | 15:41 |
rpittau | interesting | 15:41 |
dtantsur | unrelated: we're receiving a pressure with releases: https://review.opendev.org/c/openstack/releases/+/783665 | 15:42 |
dtantsur | I think bifrost and inspector should be good to go | 15:43 |
*** rh-jelabarre has quit IRC | 15:43 | |
rpittau | that reminds me | 15:43 |
rpittau | there are 2 quick bifrost patches | 15:43 |
*** rh-jelabarre has joined #openstack-ironic | 15:43 | |
TheJulia | I'm really starting to actually feel a great dislike of the release team's behavior | 15:43 |
rpittau | https://review.opendev.org/c/openstack/bifrost/+/782775 https://review.opendev.org/c/openstack/bifrost/+/782700 | 15:43 |
*** k_mouza has quit IRC | 15:44 | |
*** rh-jelabarre has quit IRC | 15:44 | |
*** k_mouza has joined #openstack-ironic | 15:44 | |
*** rh-jlabarre has joined #openstack-ironic | 15:44 | |
TheJulia | dtantsur: I guess such explicit support for ignition would be okay for basic config injection | 15:45 |
openstackgerrit | Merged openstack/ironic master: Move configuration mold utilities https://review.opendev.org/c/openstack/ironic/+/783630 | 15:45 |
TheJulia | what/how that would look like, I don't really know off-hand | 15:46 |
*** vkmc has quit IRC | 15:47 | |
*** vkmc has joined #openstack-ironic | 15:48 | |
dtantsur | me neither. I'm not sure it's even doable without hardcoding too much coreos knowledge | 15:48 |
dtantsur | maybe it's time for a downstream deploy interface | 15:48 |
TheJulia | or a coreos vmedia specific deploy interface | 15:49 |
dtantsur | yeah, I guess my question boils down to whether something like CoreOSDeploy is acceptable by the community | 15:49 |
dtantsur | (and note that I'll accept no as an answer) | 15:50 |
*** Guest76033 is now known as jroll | 15:50 | |
TheJulia | dtantsur: can that discussion wait until the PTG? | 15:51 |
rloo | it seems very specific to coreos. would it be useful for folks not at redhat? | 15:51 |
TheJulia | the "is it acceptable" | 15:51 |
dtantsur | rloo: there seem to be some people using Fedora CoreOS. I've seen one of them in the wild :) | 15:51 |
rloo | or email. does ptg have an operators track? | 15:51 |
dtantsur | but yes, it's very specific, and yes, most of people interested are related to Red Hat | 15:51 |
rloo | seems a bit similar to the anaconda driver? | 15:52 |
dtantsur | re PTG: the schedule is already packed with things that are arguably a higher priority for *upstream* (even if not for me) | 15:52 |
dtantsur | well.. yes. Anaconda is a precedent of adding support for a specific technology. | 15:52 |
rloo | "similar" -- wrt who might be interested in it | 15:52 |
TheJulia | You know, it could be this is a thing people wnat to use and support in a couple years generally | 15:53 |
dtantsur | CoreOS is very far from anaconda. It's a whole world of its own. | 15:53 |
TheJulia | I'm at least mentally thinking devil's advocate with that though | 15:53 |
rloo | maybe send out email then. although dunno what kind/if any response. | 15:53 |
dtantsur | it could be a part of the Forum if/when it happens | 15:54 |
rloo | similar wrt useful for red-hat related folks. | 15:54 |
dtantsur | I don't think we have enough operators reading the ML :( | 15:54 |
TheJulia | I 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 out | 15:54 |
dtantsur | I can do that | 15:55 |
TheJulia | That seems like the "easy" feedback path as well | 15:55 |
TheJulia | It 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 |
TheJulia | Anyway, I guess that is it for the week | 15:58 |
dtantsur | yep. I've got some food for thought, thank you! | 15:58 |
TheJulia | \o/ | 15:59 |
TheJulia | Thanks everyone! Have a wonderful week! | 15:59 |
TheJulia | #endmeeting | 16: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 | |
openstack | Meeting ended Mon Mar 29 16:00:01 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:00 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-29-15.00.html | 16:00 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-29-15.00.txt | 16:00 |
openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-29-15.00.log.html | 16:00 |
*** lucasagomes has quit IRC | 16:02 | |
*** derekh has quit IRC | 16:03 | |
*** derekh has joined #openstack-ironic | 16:03 | |
*** ociuhandu has quit IRC | 16:04 | |
*** bdodd has quit IRC | 16:06 | |
*** stendulker has quit IRC | 16:06 | |
rpittau | going to check if the sun is still there, see you tomorrow! o/ | 16:08 |
*** rpittau is now known as rpittau|afk | 16:08 | |
TheJulia | o/ | 16:08 |
openstackgerrit | Merged openstack/ironic master: Allow running RAID cleaning steps with in-band cleaning https://review.opendev.org/c/openstack/ironic/+/782552 | 16:09 |
openstackgerrit | Merged openstack/ironic master: Allow using per-site network_data schema https://review.opendev.org/c/openstack/ironic/+/783312 | 16:10 |
*** Qianbiao has quit IRC | 16:26 | |
*** kafilat has joined #openstack-ironic | 16:29 | |
*** paras333_ has joined #openstack-ironic | 16:32 | |
openstackgerrit | Aija Jauntēva proposed openstack/ironic master: Add import, export configuration to idrac-redfish https://review.opendev.org/c/openstack/ironic/+/759428 | 16:33 |
ajya | ^ rebased, please take a look | 16:36 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Validate vmedia for vmedia usage https://review.opendev.org/c/openstack/ironic-python-agent/+/783716 | 16:51 |
*** gyee has joined #openstack-ironic | 16:57 | |
openstackgerrit | Aija Jauntēva proposed openstack/ironic-specs master: Followup pre-defined system hw conf in single step https://review.opendev.org/c/openstack/ironic-specs/+/780645 | 17:07 |
*** zzzeek has quit IRC | 17:08 | |
*** zzzeek has joined #openstack-ironic | 17:10 | |
*** gyee has quit IRC | 17:14 | |
*** gyee has joined #openstack-ironic | 17:14 | |
openstackgerrit | Merged openstack/ironic master: Fix idrac-wsman BIOS step async error handling https://review.opendev.org/c/openstack/ironic/+/755821 | 17:25 |
openstackgerrit | Merged openstack/ironic master: Fix webserver_verify_ca config documentation https://review.opendev.org/c/openstack/ironic/+/783475 | 17:26 |
*** ricolin_ has quit IRC | 17:26 | |
*** paras333_ has quit IRC | 17:29 | |
*** gyee has quit IRC | 17:38 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Always add 'boot_method' vmedia in redfish/ilo vmedia boot https://review.opendev.org/c/openstack/ironic/+/783722 | 17:43 |
TheJulia | dtantsur: ^ | 17:44 |
TheJulia | irmc uses purely static isos | 17:44 |
*** uzumaki has joined #openstack-ironic | 17:47 | |
dtantsur | TheJulia: I think ilo is capable of building ISOs too | 17:48 |
dtantsur | ah, you covered it | 17:49 |
dtantsur | TheJulia: IloUefiHttpsBoot, do we count it as vmedia? | 17:49 |
*** gyee has joined #openstack-ironic | 17:49 | |
dtantsur | probably yes, it uses prepare_deploy_iso | 17:50 |
TheJulia | I *thought* I changed both | 17:50 |
dtantsur | you did, I just didn't understand why at first | 17:51 |
dtantsur | now I do | 17:51 |
TheJulia | ahh, okay | 17:51 |
dtantsur | now the IPA change with Depends-On? | 17:51 |
dtantsur | and maybe nits fixed | 17:51 |
* dtantsur is staying online until that is done | 17:51 | |
TheJulia | dtantsur: one minute | 17:53 |
TheJulia | got derailed slightly | 17:54 |
TheJulia | yay for unit tests | 17:57 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Validate vmedia for vmedia usage https://review.opendev.org/c/openstack/ironic-python-agent/+/783716 | 18:02 |
TheJulia | dtantsur: ther eyou go | 18:02 |
dtantsur | thanks! | 18:02 |
dtantsur | I'm a bit too tired to check really carefully, but overall looks good | 18:06 |
dtantsur | maybe stevebaker can land a 2nd +2 on he's in | 18:06 |
dtantsur | s/on/once/ | 18:07 |
*** dtantsur is now known as dtantsur|afk | 18:07 | |
dtantsur|afk | o/ | 18:07 |
*** paras333 has joined #openstack-ironic | 18:07 | |
TheJulia | \o | 18:09 |
*** ayoung has quit IRC | 18:10 | |
*** ayoung has joined #openstack-ironic | 18:13 | |
*** kaifeng has quit IRC | 18:26 | |
*** frigo has joined #openstack-ironic | 18:43 | |
frigo | wow thanks for the review of https://storyboard.openstack.org/#!/story/2008671 ! | 18:44 |
*** tosin has quit IRC | 18:53 | |
*** Goneri has joined #openstack-ironic | 18:58 | |
*** Goneri has quit IRC | 18:59 | |
*** jamesdenton has quit IRC | 19:02 | |
*** jamesden_ has joined #openstack-ironic | 19:02 | |
*** viks____ has quit IRC | 19:04 | |
*** zzzeek has quit IRC | 19:05 | |
zer0c00l | TheJulia: works for me! | 19:05 |
*** zzzeek has joined #openstack-ironic | 19:06 | |
*** uzumaki has quit IRC | 19:08 | |
*** uzumaki has joined #openstack-ironic | 19:08 | |
*** frigo has quit IRC | 19:20 | |
*** frigo has joined #openstack-ironic | 19:29 | |
*** dsneddon has joined #openstack-ironic | 19:30 | |
*** frigo has quit IRC | 19:33 | |
stevebaker | TheJulia: https://review.opendev.org/c/openstack/ironic/+/783722 just failed on a vmedia job | 19:57 |
zer0c00l | TheJulia: i wonder if i should move the release notes to the heartbeat api change? | 20:03 |
TheJulia | zer0c00l: likely | 20:04 |
TheJulia | sigh | 20:06 |
TheJulia | I *think* we can remove the exception entirely | 20:11 |
TheJulia | hmm | 20:16 |
TheJulia | I *think* these need to be disjointed | 20:16 |
TheJulia | stevebaker: 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 time | 20:18 |
TheJulia | so it raises an exception | 20:18 |
TheJulia | ... that was already noted to be removed later | 20:18 |
TheJulia | stevebaker: replied to 783716, I did false because I couldn't see a legitimate case where it could happen now() | 20:21 |
stevebaker | ok | 20:22 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Validate vmedia for vmedia usage https://review.opendev.org/c/openstack/ironic-python-agent/+/783716 | 20:22 |
TheJulia | ^^ might have to just fix IPA first instead of try to force the flag out in all cases in ironic. | 20:23 |
TheJulia | stevebaker: yeah, if that independently passes CI, then we should be good and we can just make that entirely independent of ironic. | 20:26 |
stevebaker | yep, cool cool | 20:27 |
TheJulia | I guess while waiting, might be a good time to go exercise | 20:32 |
*** k_mouza has quit IRC | 20:34 | |
openstackgerrit | Leo McGann proposed openstack/ironic-specs master: Add security interface spec https://review.opendev.org/c/openstack/ironic-specs/+/576718 | 20:47 |
*** derekh has quit IRC | 20:56 | |
*** akahat has quit IRC | 20:56 | |
*** kafilat has quit IRC | 20:57 | |
*** livelace has joined #openstack-ironic | 21:05 | |
openstackgerrit | Leo McGann proposed openstack/ironic-specs master: Add security interface spec https://review.opendev.org/c/openstack/ironic-specs/+/576718 | 21:08 |
*** akahat has joined #openstack-ironic | 21:20 | |
TheJulia | zer0c00l: you around? | 21:29 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix Bandit check https://review.opendev.org/c/openstack/ironic/+/783749 | 21:44 |
TheJulia | zer0c00l: ^^ | 21:44 |
TheJulia | stevebaker: if you could +2+A that, I would be super appreciativ | 21:45 |
TheJulia | since it seems the gate is blocked otherwise | 21:45 |
stevebaker | boop | 21:45 |
TheJulia | err | 21:45 |
TheJulia | need to modify it a little more | 21:45 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix Bandit check https://review.opendev.org/c/openstack/ironic/+/783749 | 21:46 |
TheJulia | I played around trying to figure out why it was so unhappy | 21:46 |
TheJulia | but that seems to be happier now | 21:46 |
TheJulia | stevebaker: looks like the ipa change is good | 21:51 |
TheJulia | it is past all of the voting jobs now | 21:51 |
stevebaker | cool, I'll keep an eye on the other one | 21:53 |
TheJulia | I really should go exercise | 21:53 |
* TheJulia spends time reviewing and can then go exercise | 21:54 | |
*** jamesdenton has joined #openstack-ironic | 21:56 | |
*** jamesden_ has quit IRC | 21:59 | |
*** jamesden_ has joined #openstack-ironic | 22:07 | |
openstackgerrit | Merged openstack/ironic-python-agent stable/victoria: Add fstab pointer to EFI partition https://review.opendev.org/c/openstack/ironic-python-agent/+/782343 | 22:08 |
*** rloo has quit IRC | 22:09 | |
*** jamesdenton has quit IRC | 22:09 | |
TheJulia | rpioso: 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 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Refactor part of image module https://review.opendev.org/c/openstack/ironic-python-agent/+/782348 | 22:12 |
openstackgerrit | Verification 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/+/782349 | 22:12 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: More refactoring of the image module https://review.opendev.org/c/openstack/ironic-python-agent/+/782350 | 22:12 |
openstackgerrit | Verification 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/+/781569 | 22:12 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Prevent broken partition image UEFI deploys https://review.opendev.org/c/openstack/ironic-python-agent/+/782351 | 22:12 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Add fstab pointer to EFI partition https://review.opendev.org/c/openstack/ironic-python-agent/+/782352 | 22:12 |
rpioso | TheJulia: 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 |
TheJulia | Awesome, thanks | 22:20 |
zer0c00l | TheJulia: yes | 22:20 |
zer0c00l | was having lunch. | 22:20 |
TheJulia | I pushed https://review.opendev.org/c/openstack/ironic/+/783749 as your change seemed to intermittently pass bandit checking | 22:21 |
zer0c00l | TheJulia: thanks sorry about that, wonder if it passed when we +1ed. | 22:23 |
TheJulia | I *think* so | 22:23 |
TheJulia | not 100% sure, it just seems really weird | 22:23 |
*** rcernin has joined #openstack-ironic | 22:24 | |
*** rcernin has quit IRC | 22:24 | |
janders | good morning Ironic o/ | 22:24 |
*** rcernin has joined #openstack-ironic | 22:24 | |
TheJulia | o/ janders | 22:24 |
janders | hi TheJulia | 22:25 |
janders | how is it going? | 22:25 |
*** jamesdenton has joined #openstack-ironic | 22:25 | |
TheJulia | good, https://review.opendev.org/c/openstack/ironic/+/744117 may require attention :) | 22:25 |
*** jamesden_ has quit IRC | 22:28 | |
janders | that's the top of my list today :) on it | 22:35 |
*** derekh has joined #openstack-ironic | 22:48 | |
*** livelace has quit IRC | 22:55 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix configuration generation for ironic doc pages https://review.opendev.org/c/openstack/ironic/+/783756 | 22:57 |
rpioso | TheJulia: 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 |
openstackgerrit | Arun S A G proposed openstack/ironic master: Add agent_state and agent_status params to heartbeat https://review.opendev.org/c/openstack/ironic/+/778058 | 23:37 |
openstackgerrit | Arun S A G proposed openstack/ironic master: Add support for configdrive in anaconda interface https://review.opendev.org/c/openstack/ironic/+/780398 | 23:37 |
openstackgerrit | Merged openstack/ironic-python-agent master: Validate vmedia for vmedia usage https://review.opendev.org/c/openstack/ironic-python-agent/+/783716 | 23:38 |
openstackgerrit | Merged openstack/ironic master: [doc] Warning about out-of-sync ESPs for UEFI software RAID https://review.opendev.org/c/openstack/ironic/+/783614 | 23:38 |
iurygregory | morning janders o/ | 23:38 |
openstackgerrit | Richard G. Pioso proposed openstack/ironic master: Add iDRAC management via Redfish to idrac HW type https://review.opendev.org/c/openstack/ironic/+/721593 | 23:38 |
rpioso | TheJulia: Done | 23:51 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!