15:00:05 <TheJulia> #startmeeting ironic
15:00:05 <openstack> Meeting started Mon Jul 23 15:00:05 2018 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:00:06 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:07 <TheJulia> o/
15:00:09 <openstack> The meeting name has been set to 'ironic'
15:00:15 <etingof> o/
15:00:25 <kaifeng> o/
15:00:26 <TheJulia> Good morning everyone! Its time for an ironic meeting!
15:00:30 <rpioso> o/
15:00:31 <stendulker> o/
15:00:34 <rloo> o/
15:00:37 <hshiina> o/
15:00:50 <yolanda> o/
15:01:19 <dtantsur> o/
15:01:28 <mjturek> o/
15:01:43 <TheJulia> Our agenda for this week can be found on the wiki
15:01:46 <TheJulia> #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting
15:02:12 <TheJulia> #topic Announcements/Reminders
15:02:41 <vdrok> o/
15:02:49 <TheJulia> #Info This week is R-5, final release for client libraries, requirement freezes and feature freezes take effect at the end of the week.
15:03:12 <TheJulia> #info ironic-lib and sushy were released last week for the Rocky cycle. Thanks everyone!
15:03:54 <dtantsur> \o/
15:03:54 <rloo> ++ thanks!
15:04:02 <TheJulia> #info The stein PTG planning etherpad remains up, and has had a few minor items added based on where we are at with the Rocky cycle so we discuss them at the PTG.
15:04:06 <TheJulia> #link https://etherpad.openstack.org/p/ironic-stein-ptg
15:04:29 <TheJulia> Does anyone have anything else to announce or raise as a reminder?
15:04:51 <TheJulia> Oh, before I forget
15:05:56 <TheJulia> #info xclarity CI has made progress and is now talking to gerrit. Thanks apparently goes to rajinir for lending a helping hand.
15:06:15 <jroll> \o
15:06:25 <rloo> thx rajinir!
15:06:32 <rajinir> yw
15:06:45 <rloo> we had the midcycle virtual meet up last week, thx to all that attended
15:07:40 <TheJulia> #info Virtual midcycle was last week
15:07:43 <TheJulia> #link https://etherpad.openstack.org/p/ironic-rocky-midcycle
15:08:01 <TheJulia> I think we're good to move on if nobody has anything else?
15:09:02 <TheJulia> It appears that we had no action items form our weekly meeting last week, so I think we can just move along to status updates.
15:09:42 <TheJulia> Moving on...
15:09:45 <TheJulia> #topic Review subteam status reports
15:09:59 <TheJulia> #link https://etherpad.openstack.org/p/IronicWhiteBoard
15:10:12 <TheJulia> Starting at line 159
15:11:49 <TheJulia> Looks like we have some initial documentation posted for deploy steps \o/
15:12:53 <rloo> TheJulia: you OK if I delete the 'Management interface refactoring' part (L237ish)
15:12:56 <TheJulia> zshi, yolanda, hshiina: Do any of you know if someone is working on the API ref updates for bios config gramework?
15:13:15 <TheJulia> rloo: yes, as well as the nova version change
15:13:40 <rloo> done, deletions-are-my-specialty TM
15:13:48 <yolanda> TheJulia, i miss context sorry, which ref updates?
15:14:06 <TheJulia> yolanda: sorry, bios interface
15:14:12 <TheJulia> rloo: \o/
15:14:33 <yolanda> so rest api is done
15:14:47 <TheJulia> dtantsur: I'm going to go ahead and add the python-ironicclient patch to the list for the week since you just started that
15:14:55 <rloo> i hate to ask this, but wrt Deploying with Apache (L299). where are we at with that? still working on API/worker split for inspector?
15:14:58 <dtantsur> yep
15:15:44 <yolanda> there are a pair of patches missing review from zshi
15:15:57 <TheJulia> And I see it is alrady there
15:16:08 <yolanda> https://review.openstack.org/577343 and https://review.openstack.org/576471 ... last one got a -1 from jslagle
15:16:09 <patchbot> patch 577343 - tripleo-docs - Add BIOS settings doc
15:16:10 <patchbot> patch 576471 - python-tripleoclient - Create commands for configuring BIOS on given nodes
15:18:46 <dtantsur> rloo: yep
15:19:17 <rloo> dtantsur: ok, so it stays there.
15:19:47 <rloo> dtantsur: but i guess api/worker split has stalled?
15:20:12 <openstackgerrit> Dmitry Tantsur proposed openstack/python-ironicclient master: Add support for conductor groups  https://review.openstack.org/584901
15:20:12 <dtantsur> rloo: there was a new patch recently
15:20:13 <hshiina> TheJulia, yolanda as far as i know there is not a patch for ironic/api-ref about bios config
15:20:17 <dtantsur> but it's not moving really fast
15:20:43 <jroll> whoops
15:20:48 <rloo> dtantsur: i will link to the story for that
15:21:19 <yolanda> i don't have the knowledge for it... so we miss some feature for the bios story?
15:21:23 <jroll> do we wait for julia to return or continue without her?
15:21:32 <yolanda> hshiina?
15:22:02 <hshiina> TheJulia, yolanda i or someone from fujitsu can work for bios conf api-ref
15:22:08 <dtantsur> yolanda: nothing major IIRC
15:22:16 <rloo> jroll: we continue -- we need to understand the status of bios / update that first
15:22:19 <jroll> yolanda: api-ref is just docs :)
15:22:29 <hshiina> yolanda, we need to add documentation to https://developer.openstack.org/api-ref/baremetal/
15:22:47 <yolanda> ah, i'll be happy to work on it
15:23:22 <rloo> yolanda: i'll put your name down then, thx
15:23:29 <hshiina> yolanda, thanks
15:23:30 <yolanda> ++
15:23:43 <jroll> rloo: yeah, I meant past that I guess :)
15:24:02 <rloo> jroll: heh. ok, someone take over
15:24:17 <rloo> fwiw, i'm good with status reports
15:24:25 <jroll> ditto
15:24:42 <jroll> #topic Review subteam status reports
15:24:50 <jroll> doh, don't have chair status
15:24:58 <jroll> we can continue anyway ¯\_(ツ)_/¯
15:25:06 <rloo> dtantsur: do you have chair status?
15:25:14 <jroll> it's set per-meeting
15:25:21 <dtantsur> nope
15:25:22 <jroll> priorities are at line 100
15:25:29 * jroll removes any merged things
15:25:32 <dtantsur> in API SIG we add all 4 cores to chairs every meeting
15:25:55 <jroll> dtantsur: anything left for interface reset?
15:26:05 <dtantsur> jroll: IIRC no
15:26:10 <dtantsur> well, maybe api-ref
15:26:23 <rloo> was documentation updated for that interface reset?
15:26:25 <dtantsur> maybe even some good docs
15:26:31 <jroll> probably not a priority for this week, right?
15:26:41 <jroll> we should focus on client / features
15:26:41 <dtantsur> yeah, but I'll mark it somewhere
15:26:50 <rloo> not a priority but mention in the subteam stuff
15:27:00 <jroll> someone please add it to subteam things
15:27:06 <rloo> anyone know if there are any client bugs we ought to get in?
15:27:48 <rloo> i added it to the classic driver removal stuff, put down dtantsur's name :)
15:27:52 <jroll> maybe https://review.openstack.org/#/c/532979/ ?
15:27:53 <patchbot> patch 532979 - python-ironicclient - Raise exception for unsupported microversion
15:28:12 <jroll> that's the only patch open I see that seems important and isn't listed elsewhere
15:28:16 <dtantsur> thanks rloo
15:28:18 <jroll> (well, that has been updated this year)
15:28:52 <rloo> jroll: but TheJulia doesn't seem to think it is worth doing. let's ask her
15:29:10 <jroll> oh, I see
15:29:16 <rloo> jroll: so the only client one is for your conductor_group change?
15:29:21 <jroll> looks like it
15:29:33 <jroll> anything else that needs to be prioritized this week?
15:30:25 <openstackgerrit> Dmitry Tantsur proposed openstack/python-ironicclient master: Add support for conductor groups  https://review.openstack.org/584901
15:30:26 <dtantsur> seems ready now ^^^
15:30:31 <jroll> :)
15:30:49 * jroll hears crickets for priorities
15:30:55 <rloo> that boot-from-ramdisk. we'd need to approve spec AND code if we want that in this week. it is feature freeze this week too, right?
15:31:02 <openstackgerrit> Merged openstack/ironic master: Improve pep8 checking along with hacking  https://review.openstack.org/578696
15:31:05 <openstackgerrit> Merged openstack/ironic master: Replace port 35357 with 5000 for "auth_url"  https://review.openstack.org/583539
15:31:07 <jroll> tis feature freeze, I don't see it landing this cycle
15:31:15 <jroll> maybe should remove it then?
15:31:37 <MrBerns> Hi guys, I am having some trouble with deploying nodes in bifrost at the moment. When I attempt to deploy a node using the playbook I get this error: http://paste.openstack.org/show/iGyjBweDzfZLK5SOFipD/
15:31:37 <rloo> i think the spec is ready. don't know about the code.
15:31:42 <dtantsur> rloo: yep, Thu is FF
15:31:54 <dtantsur> MrBerns: sorry, we're in a meeting right now. could you please wait until the end of the hour?
15:31:56 <jroll> we can leave the spec on the list, I suppose
15:32:05 <MrBerns> dtantsur, Pardon, didnt realise
15:32:23 <rloo> i don't know about the ipxe boot interface.
15:32:31 <rloo> maybe we should remove that. L120ish
15:32:47 <dtantsur> rloo: long-term goal of splitting pxe and ipxe boot interfaces..
15:32:48 <jroll> yeah, no clue why that's there
15:32:49 <rloo> although it isn't clear that it is part of the weekly priorities.
15:32:54 <jroll> julia herself agreed it's too late to land
15:33:01 <rloo> maybe TheJulia put it there to mention it.
15:33:07 <dtantsur> ++
15:33:15 <dtantsur> I think it's just to get attention to it
15:33:23 * jroll removed
15:33:45 <rloo> that looks good. i think after this week, we need to focus on bugs/docs, right?
15:33:56 <rloo> week of aug 9 isn't too far off
15:33:56 <jroll> yep
15:34:14 <jroll> and ptg prep
15:34:26 <jroll> ok, moving on?
15:34:36 <rloo> ++ moving on
15:34:42 <jroll> #topic Open Discussion
15:34:50 <jroll> just a note, I'm out wed-fri this week
15:34:59 <rloo> oh, we didn't mention it, but this week is ptl candidate whatever
15:35:30 <jroll> ah, yep
15:35:34 <jroll> nominations are due Jul 31, 2018 23:45 UTC
15:35:54 <TheJulia> okay... So I lost connectivity it seems :\
15:36:02 <dtantsur> indeed
15:36:06 <jroll> welcome back TheJulia :)
15:36:09 <dtantsur> we did mess with priorities a tiny bit
15:36:09 <jroll> we're in open discussion
15:36:49 * jroll has nothing else to add
15:36:54 <TheJulia> #topic Open Discussion
15:37:20 <TheJulia> Looking at the log, I raised a question... I guess when I was getting disconnected about if anyone cared about pxe/ipxe split since it would take a long deprecation cycle
15:37:36 <TheJulia> and I put the patch up to get it started, as the spec has been long approved as well.
15:37:38 <TheJulia> but *shrugs*
15:38:03 <jroll> TheJulia: ah, didn't see the question here
15:38:03 <rloo> TheJulia: is it only one patch to do the split?
15:38:22 <jroll> my biggest concern is it's a huge change and we're three days from FF
15:38:27 <rloo> TheJulia: I haven't looked at it, so don't know if it is 'safe' to do so late in the cycle
15:38:27 <TheJulia> rloo: to start, yes.. there is tons of common code that will need to be refactored
15:38:44 <TheJulia> but it starts the process at least and gives us the ability to continue next cycle
15:39:05 <TheJulia> jroll: Agreed, I have the same concern, but it was also fairly clean and the BFV job passed which is a good sign :)
15:39:08 <rloo> TheJulia: so the first patch is just common code being refactored?
15:39:42 <TheJulia> rloo: We can't refactor the common code without beginning to pull the two interfaces apart to see how the common code should end up looking since there is still overlap and there will ultimately always be some overlap
15:40:12 <TheJulia> first patch creates the interface, minor doc update, removes some of the conditionals in the interface code, maps through to the underlying common code
15:40:14 <rloo> I guess, if the patch isn't to deprecate, I don't see the need to do it/land it this week. Again, I haven't looked to see how risky it is.
15:41:01 <TheJulia> It deprecates the config option which is the master switch for the pxe interface
15:41:37 <rloo> Hmm. We need two cores to review/approve that this week. Is it fair to ask if there are two cores willing to do so?
15:41:44 <TheJulia> Anyway.. That was the only thing I really had for open discussion, and wanted to make a note that I need to take one of the fur children to the vet later today, so I'll be slightly MIA this afternoon
15:42:09 <dtantsur> I can try reviewing the ipxe thingy
15:42:11 <jroll> I won't be able to get to that patch
15:42:20 <TheJulia> rloo: If people want to punt on it, I'm okay with punting
15:42:52 <rloo> let's punt.
15:43:01 <TheJulia> ack, will go block it
15:43:27 <rloo> i'd rather get the boot-from-ramdisk in... but doubtful that will make it
15:44:09 <TheJulia> rloo: next on my todo list before taking the cat to the vet. :)
15:44:19 <rloo> TheJulia: :)
15:45:06 <TheJulia> Do we have anything else to discuss today?
15:45:17 * jroll doesn't
15:45:27 <jroll> TheJulia: oh since you may have missed it, I'm out wed-fri
15:45:40 <rloo> crickets
15:45:42 <TheJulia> jroll: I picked up on it, thanks :)
15:45:48 <jroll> :)
15:46:39 <kaifeng> i want to note that inspector gate is broken, i have put a patch to fix it
15:46:40 <TheJulia> Okay, Thanks everyone!
15:46:42 <kaifeng> https://review.openstack.org/#/c/584273/
15:46:43 <patchbot> patch 584273 - ironic-inspector - Fix gate failure due to recent oslo.policy release
15:46:54 <TheJulia> kaifeng: thanks!
15:47:03 <kaifeng> if anyone has time, just take a look :)
15:47:14 * jroll finds some lunch
15:47:23 <TheJulia> kaifeng: I'll try to later today
15:47:29 <TheJulia> Have a wonderful day everyone!
15:47:44 <TheJulia> #endmeeting