15:00:18 #startmeeting ironic 15:00:18 Meeting started Mon Nov 26 15:00:18 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:19 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:00:19 o/ 15:00:21 The meeting name has been set to 'ironic' 15:00:22 o/ 15:00:22 o/ 15:00:23 \o 15:00:27 o/ 15:00:31 o/ 15:00:32 o/ 15:00:33 Good morning everyone! 15:00:41 Our agenda this week can be found on the wiki. 15:00:43 #link https://wiki.openstack.org/wiki/Meetings/Ironic 15:00:47 o/ 15:01:04 \o 15:01:06 #topic Announcements/Reminders 15:01:06 o/ 15:01:11 \o/ 15:01:20 o/ 15:01:32 #info Tenks is now under Ironic's governance! 15:01:38 \o/ 15:02:00 o/ 15:02:03 * dtantsur|mtg is lurking 15:02:30 yah, tenk you for tenks 15:02:49 #info The OpenStack summit was very positive, and there was lots of discussion about the future being baremetal clouds. As such things might get a bit busier in here. :) 15:03:10 Does anyone else have an announcement or reminder that they would like to raise this week? 15:03:37 Hey everyone o/ You may have seen me hanging out in the IRC. So, I just wanted to introduce myself to the ironic team :) My name is Dhanuka (he/him), and I'm from Sri Lanka. I'm your Outreachy intern for the December - March round. I'll be working with my mentors dtantsur and etingof, on the sushy project. Thanks for being so welcoming and helpful community! Looking forward to contribute and learn 15:03:43 a lot :) 15:03:57 dnuka: Welcome! 15:04:03 welcome =) 15:04:20 dnuka: Welcome to ironic! 15:04:20 TheJulia , iurygregory thank you :) 15:04:24 Anyone else? 15:05:02 awesome, welcome dnuka :) 15:05:16 Well, I guess we can move on then 15:05:22 thank you jroll :) 15:05:29 Merged openstack/sushy-tools master: Make external dependencies optional https://review.openstack.org/611859 15:05:31 Merged openstack/sushy-tools master: Fix wrong `set_server_metadata` args https://review.openstack.org/618110 15:05:31 #topic Review action items from previous meeting 15:06:12 Looks like I never got around to sending out a doodle for clean steps 15:06:33 #action TheJulia to send out a doodle for clean/deploy steps visibility discussion 15:06:45 Nothing besides that 15:06:47 Moving on 15:07:03 #topic Review subteam status reports 15:07:13 #link https://etherpad.openstack.org/p/IronicWhiteBoard 15:07:28 Starting at line 165 15:07:34 thank you rpioso :) 15:09:37 \o/ looks like we've made a little progress on switching over our jobs to be more python3 and zuul native where possible 15:09:59 TheJulia: is the TODO L202&203 now covered by Python3First? 15:10:23 TheJulia: actually, L204&205 too 15:10:50 rloo: Those are more contextual, I guess we can delete the lines. 206 is totally status informational 15:11:12 TheJulia: i was referring to the TODO above that :) 15:11:50 TheJulia: oh, sorry, theTODO is part of Python3 First. 15:11:51 We kind of mashed it all together for us to just get it done. 15:11:58 Yeah 15:12:01 no worries 15:12:45 Great, the enhanced checksum support is done! (L224) 15:12:49 hshiina: regarding UEFI first, I wonder if we could use/need a few more CI job or a new job that explicitly supports running UEFI 15:13:39 kaifeng: is there anything else that your aware of that needs to be done regarding the checksum support? Or do you consider it done? 15:13:59 I think it's done :) 15:14:59 TheJulia: wrt smartnic support (L277). was there a discussion at forum, with neutron folks on it? 15:15:35 rloo: extensive, I think from our standpoint our needs/required changes has largely been dialed back 15:16:03 TheJulia: dialed back --> that could be good? :) 15:16:04 I'll try and follow-up on that this week. Ive seen the patches have been revised in the last few days so there is hope they are already on track. 15:16:12 rloo: yes, very good. 15:16:22 :) 15:16:59 etingof: How goes the task execution work? 15:17:50 hmm, bug stats hasn't been updated since Nov 5. Do we care? 15:17:53 Looks like graphical consoles is going to need more attention :( 15:18:02 rloo: we've not had a meeting since nov 5th 15:18:11 I'm too busy, sorry 15:18:19 wow, how time flies! 15:18:22 I'll get back to bug stats after the current week of meetings 15:18:29 no worries dtantsur|mtg 15:18:33 dtantsur|mtg: thanks! 15:18:33 TheJulia, I have not worked on the tasks subsystem itself much lately, but I am quite optimistic concerning ipmitool killing patch - which should improve our most immediate problem 15:19:16 It does look like one idea seems to have more support than the other, at least when I was glancing through reviewes earlier this morning 15:19:57 if it is blocked cuz of lack of feedback, should we schedule a time to discuss? eg next Monday's meeting? 15:20:15 or today if it is something that we can grok in 1-2 minutes... 15:20:36 mgoddard: dtantsur|mtg: If there are only minor items on the spec, would there be a possibility of revising/reviewing and getting that moving this week? 15:21:54 I think federation discussions likely need a resync from the summit as well. 15:22:06 TheJulia: for which spec? 15:22:23 * dtantsur|mtg the same question 15:22:25 deploy templates 15:22:26 sorry 15:22:29 ah 15:23:02 I don't remember for sure, but I think nothing critical there 15:23:05 yeah, I can try to get an iteration up this week, if not early next week 15:23:54 if nothing is critical, could we just merge and approve the spec and revise it? 15:23:56 not sure how well I understand the consensus around the design changes, but I'll bring those up in the spec 15:24:05 (well, approve if we have consensus) 15:24:24 mgoddard: that is a good question 15:25:14 Looks like neutron event processing needs some eyes as well 15:25:35 But there is no change there from the last meeting. 15:25:39 I don't have time to revisit right now, sorry. I'd prefer mgoddard update it and we go from there. 15:26:30 I'll do that. should have some time on a train tomorrow 15:26:31 Okay, I'm just worried about the inherent cost of a -1 to reviews 15:26:40 mgoddard: thanks! 15:27:19 I'm done going down the list of priorities, is everyone else done and there are no questions seeking further clarification? 15:29:24 I guess we're good to move on... 15:29:38 #topic Deciding on priorities for the coming week 15:30:35 Ruby Loo proposed openstack/ironic stable/rocky: Don't fail when node is in CLEANFAIL state https://review.openstack.org/620086 15:33:17 In shuffling around some of the items, it doesn't look like we have too many outstanding patches that are in a shape to receive reviews and potentially be merged this week :\ 15:34:03 I've trimmed and cleaned up the list. Is there anything anyone feels should be on the list of items to review this week? 15:35:21 https://review.openstack.org/#/c/587698/ can use some eyes if nothing else 15:35:22 patch 587698 - ironic-inspector-specs - Configurable introspection data backends - 7 patch sets 15:36:16 https://review.openstack.org/#/c/619683/ might be a cantidate as well 15:36:16 patch 619683 - ironic - Expose conductors: db and rpc - 2 patch sets 15:37:16 there is another patch on the way, would be better if proposed 15:37:49 * etingof has quite a few pending patches against sushy-tools 15:37:51 kaifeng: will you be re-ordering the patches then? 15:38:02 etingof: priority list of 3 please :) 15:38:10 https://review.openstack.org/#/c/582314/ could use some eyes 15:38:10 patch 582314 - ironic - Add BIOS interface to Redfish hardware type - 7 patch sets 15:38:18 no, this is the first patch, the second will be around api 15:38:54 bdodd: I'll place mine on it. 15:38:57 kaifeng: ack, I went ahead and put that one on the list, and you can add the other once it is there 15:39:05 bdodd: Added 15:39:20 rpioso TheJulia - thanks! 15:39:58 TheJulia thanks:) 15:40:39 Ohh, I'm going to add https://review.openstack.org/#/c/616968/ 15:40:39 patch 616968 - ironic - Let neutron regenerate mac on port unbind - 4 patch sets 15:40:44 Since that is a bug fix 15:41:21 TheJulia, my humble list: https://review.openstack.org/#/c/616998/ https://review.openstack.org/#/c/616998/ https://review.openstack.org/#/c/618112/ 15:41:22 patch 616998 - sushy-tools - Fix UEFI firmware path and connection attribute in... - 6 patch sets 15:41:23 patch 616998 - sushy-tools - Fix UEFI firmware path and connection attribute in... - 6 patch sets 15:41:25 patch 618112 - sushy-tools - Fix HTTP response code on errors - 2 patch sets 15:41:30 ah 15:41:39 I guess one didn't copy? 15:41:45 yep, hold on 15:42:15 TheJulia, https://review.openstack.org/#/c/614316/ 15:42:16 patch 614316 - sushy-tools - Fix libvirt driver to handle domains by UUID - 6 patch sets 15:42:50 okay, added 15:42:56 That looks like a healthy list for this week! 15:44:51 Everyone good to proceed to discussion? 15:45:18 ++ 15:45:29 #topic Discussion 15:45:55 Looks like we have a single discussion topic from tonyb this week 15:46:16 In regards to providing a path forward where a bmc does not support raw commands 15:46:37 Looks like https://review.openstack.org/#/c/616053/ has a +2 and 2x+1 15:46:38 patch 616053 - ironic - Add ipmi_do_timeout_disable to avoid problematic I... - 4 patch sets 15:46:51 and https://review.openstack.org/#/c/583488/ has a -1 15:46:51 patch 583488 - ironic - Continue to set boot device even if timeout config... - 2 patch sets 15:46:59 Seems like consensus has already been built around the first one 15:47:07 * TheJulia is unsure there needs to be much more of a discussion 15:48:03 I'm going to go with what the consensus is behind for now, and we can go from there 15:48:14 seems like kaifeng is even in favor of taking the first 15:48:16 Anyone else have thoughts on this topic? 15:48:28 (last comment in the second, and a +1 on the first) 15:48:33 Yeah 15:48:42 yeah, no discussion needed imo 15:48:43 Yeah, I plans to base my patch on tonyb's 15:48:52 er, s/plans/plan/ 15:48:55 kaifeng: I was kind of guessing that would be the next step :) 15:48:57 sounds good to me 15:49:05 I guess we can move on to Open Discussion 15:49:36 #topic Open Discussion 15:49:57 \o/ conductor group spec in nova merged 15:50:01 * TheJulia dances 15:50:51 I don't really have anything else to discuss 15:50:52 \o/ 15:51:04 Just starting to go through the tons of things from the last two weeks 15:51:14 Does anyone have anything they wish to discuss? 15:52:11 * TheJulia guesses the answer is no :) 15:54:10 Well everyone, Thank you! 15:54:25 CRICKETS 15:54:43 #endmeeting