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