15:00:17 #startmeeting ironic 15:00:17 Meeting started Mon Feb 18 15:00:17 2019 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:00:18 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:00:18 o/ 15:00:20 o/ 15:00:20 \o 15:00:20 The meeting name has been set to 'ironic' 15:00:22 o/ 15:00:26 o/ 15:00:27 o/ 15:00:33 o/ 15:00:36 o/ 15:00:46 o/ 15:00:46 Good morning everyone! 15:00:55 good morning TheJulia 15:01:04 good morning! 15:01:08 Our agenda this morning is fairly light, so hopefully we will get this wrapped up nice and quickly. I hope 15:01:25 #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting 15:01:26 o/ 15:01:38 #topic Announcements / Reminders 15:02:20 o/ 15:02:51 #info This week is R-7, which means next week is our final non-client library release for this cycle. 15:02:54 #link https://releases.openstack.org/stein/schedule.html 15:03:22 which, I guess, includes sushy, ironic-lib and metalsmith (?) 15:03:39 sushy and ironic-lib. Metalsmith likely... 15:04:07 #info The Forum/PTG idea etherpad has been posted and already has some great topics. 15:04:09 #link https://etherpad.openstack.org/p/DEN-train-ironic-brainstorming 15:05:18 I have added a list for people to indicate if they are going on the etherpad. That way we can hopefully logically and efficiently try to plan topics. 15:05:43 #info The Cisco UCS drivers have been marked as of deprecated in a proposed review. 15:05:44 :( 15:05:59 #link https://review.openstack.org/#/c/637241/ 15:06:00 patch 637241 - ironic - Deprecate Cisco drivers - 1 patch set 15:06:05 :/ 15:06:12 Does anyone have anything to announce or remind us of this week? 15:06:41 * dtantsur makes a sad manul face at the cisco deprecation 15:07:11 :( 15:07:24 * TheJulia takes the crickets as indication we can proceed 15:07:28 yep 15:07:35 #topic Review action items from previous meeting: 15:07:39 #undo 15:07:40 Removing item from minutes: #topic Review action items from previous meeting: 15:07:40 #topic Review action items from previous meeting 15:08:23 I still have not gotten a chance to send a summary email from our midcycle call given how busy I have been. I'm unsure if it really matters at this point :( 15:09:00 I'll try to get something sent out in the next few days. Today should... hopefully... be quiet. 15:09:05 maybe we just skip it 15:09:14 We did keep really good notes 15:09:22 Anyway 15:09:22 after all, the etherpad is public (and next time we should remember to record the call) 15:09:47 TheJulia: maybe an email pointing to the etherpad would be nice? I can help there if you'd like 15:09:49 Perhaps... 15:09:51 * arne_wiebalck thinks the notes on the etherpad are good enough 15:09:54 \o 15:09:57 mjturek: that would be awesome 15:10:08 Anyway, time to move on! 15:10:11 #topic Review subteam status reports 15:10:15 you got it 15:10:25 #link https://etherpad.openstack.org/p/IronicWhiteBoard 15:10:42 Starting at line 289 15:11:40 re: py3 and explicit py36 unit testing, I think that is fine, and likely preferred given there is an attempt to land python 3.7 unit tests in various projects. 15:12:07 ++ 15:12:15 jroll: any indication if the conductor grouping stuff is going to make it for nova this cycle? 15:12:20 FYI I tried our unit tests with py38-alpha1 :) 15:12:20 I definitely vote for 3.7 :) 15:12:36 sorry, forgot about this meeting >.> 15:12:46 TheJulia: I hope so, it has a +2 15:13:02 :) 15:13:51 The data storage for deploy templates is in good shape, looks like the remaining patches are still WIP 15:14:08 yeah, left a few reviews on the API one, mostly complaining about /details :) 15:14:22 heh, okay 15:14:38 Looks like smartnics is mostly in a happy state, we have a few small reviews to do on our side of the universe 15:14:53 only two ironicclient patches, right? or is there anything else? 15:15:42 dtantsur: one more ironicclient patch and a follow-up patch 15:16:15 And some of the neutron event processing, at least the receiver in the API (That does nothing) has merged so that can move forward some before the end of the cycle. 15:17:36 I doub't graphical console is going to merge this cycle without rpc/api work. kaifeng raised a concern that should likely be discussed 15:17:52 ++ no point in hurrying it in 15:18:46 I hope to finish the allocation API work (without backfilling) this cyclle 15:18:50 TheJulia, for the graphical console I've been working on RPC/API 15:19:07 rpittau: could you put that note on the status? 15:19:15 yes 15:19:18 rpittau: awesome! still, without any vendor implementing it, we don't have to hurry just yet 15:19:37 I've been really amazed just how many new things we've merged this cycle 15:19:43 yeah 15:19:56 Anyway, I'm done going down the list and I guess we can proceed to priorities for the next week. 15:20:07 Everyone else ready to proceed? 15:20:54 Sanity check: I need to manually copy the kernel and ramdisk used for inspection into the ironic httpboot/tftpboot directories, right? The deploy_kernel and deploy_ramdisk driver_info attributes aren't used for that. 15:21:14 larsks: you're right 15:21:20 TheJulia: ++ to proceed 15:21:22 dtantsur: Thanks. 15:21:41 #topic Deciding on priorities for the coming week 15:21:55 And while everyone is running to the list, I'm going to clean it up and remove the items that we merged this past week 15:22:17 #link https://etherpad.openstack.org/p/IronicWhiteBoard 15:22:20 Starting at line 160 15:23:05 quite a few things have merged indeed 15:23:34 dtantsur: Want any ironic eyes on https://review.openstack.org/636594 ? 15:23:35 patch 636594 - openstacksdk - baremetal: support for allocation API - 3 patch sets 15:23:50 TheJulia: I would not mind, but I don't dare asking given how busy everyone is :) 15:24:00 Alright 15:24:45 I just need to nag mordred at some point :) 15:25:57 Looks like someone moved up a networking-baremetal item 15:27:02 I added some backports. 15:27:19 hjensas: they were already on the list :) 15:27:22 I did move them up 15:27:50 I think we're in good shape list wise. Is there anything anyone feels should be on the list? 15:28:13 TheJulia: oh, ok. Thanks, I removed the line I added. 15:28:19 TheJulia: do we need graphical console in the priorities, if we're not merging it until the API is proposed? 15:28:32 I was just typing something out asking if we want to remove it 15:28:52 Removed 15:28:57 I don't think there will be time anyway for a review 15:29:34 Okay, I guess we're good to proceed then 15:29:47 LGTM 15:29:51 Everyone good to proceed to ?Open Discussion? 15:30:38 it seems we can move forward :) 15:30:40 * TheJulia guesses that is a yes 15:30:45 #topic Open Discussion 15:30:55 Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: DNM - Test https://review.openstack.org/637552 15:30:58 rpittau: Seems like you are seeking some help with fixing python3 support in python-hardware ? 15:31:36 rpittau: note that we can move python-hardware to git.openstack.org without it becoming an official project (or a part of any official project) 15:31:38 TheJulia, yes, so far dtantsur has been bery helpful with reviewing and merging some PRs in that hardware repo 15:31:57 oh ok I didn't know that 15:32:09 but we need agreement from all current maintainers, esp. Erwan (?) 15:32:27 yeah, e.g. ironic-staging-drivers is an independent repository 15:33:15 On the subject of the discussed fast-track feature from ?last week?, I've got some code that should work. I've verified via integration test logs that the API change works, still trying to get a tempest scenario to leverage the entire sequence without any reboots. https://review.openstack.org/#/c/635996/ 15:33:15 patch 635996 - ironic - fast tracked deployment support - 11 patch sets 15:33:30 the idea was to be able to keep the hardware repo in shape as much as possible 15:33:54 TheJulia: I'm sloooooowly getting to it :) was planning on today, but had to wake up early, and my head doesn't cooperate easily :) 15:34:05 :) 15:34:23 rpittau: I'm all for it. but we probably need a formal poll (e.g. email) among the current maintainers. then I can guide you through the process. 15:34:28 dtantsur: no worries. I think once I have an integration test performing discovery -> deployment without rebooting... I'll be super happy 15:34:38 Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Run all defined jobs defined in check and gate https://review.openstack.org/636169 15:34:46 TheJulia: a lot of people will 15:34:56 dtantsur, sounds good 15:35:14 dtantsur: next step.. the world? 15:35:27 absolutely 15:35:27 uh, the fast track means? 15:36:10 kaifeng: from enroll to active with only one reboot 15:36:27 kaifeng: so dtantsur and I have this downstream req to try and speed up discovery to deployment time, and we spent a lot of time rebooting, so in discussion ?last? week, there was a lot of positive feedback to adding such functionality to ironic so we don't have to power off when that is the case 15:36:28 boot the node for discovery, leave it running, clean, leave it running, deploy, reboot into instance 15:37:04 awesome! 15:37:13 good idea! 15:38:11 to accelerate the deployment .. some users might not be happy about additional power consumption by running servers 15:38:36 I am trying to understand how discovery -> deployment can happen, a node is enrolled, it can't be scheduled if we don't put it into available state, and how we know it will be chosen? 15:38:51 Anyway, i'd love some reviews, the main thing right now is the tempest test for me 15:39:12 kaifeng: the state machine will progress as usual, we just won't shut down the machine in between 15:39:14 chris_j: it's opt-in, so shouldn't be any surprises 15:39:25 yep, and totally opt-in 15:39:37 awesome! thanks for clarifying 15:39:47 chris_j: and I'd rather find out my breaker is degraded with the entire rack about to be deployed instead of during some day in production.... 15:39:58 hmm, I guess there is no spec right? maybe I can only find answers from patches :) 15:40:19 kaifeng: yeah, no spec. I put together an rfe, and it really seems fairly simple 15:41:15 kaifeng: it really doesn't move the node through the state machine, we truly just leave the power on so the ramdisk can continue to heartbeat to the ironic api if so configured. If it does not with in... 300 seconds, then the node is treated as if it needs to be powered on 15:41:33 okay, I saw this patch https://review.openstack.org/#/c/636778/ but it's not attached to a story yet 15:41:33 patch 636778 - ironic-inspector - Add ironic API url to inspector IPA config - 2 patch sets 15:42:31 kaifeng: you’re looking for this one? https://storyboard.openstack.org/#!/story/2004965 15:43:13 kaifeng: and https://review.openstack.org/635996 15:43:14 patch 635996 - ironic - fast tracked deployment support - 11 patch sets 15:44:02 got it, thanks 15:44:07 Anyway, do we have anything else to discuss today? 15:45:35 * dtantsur is busy laughing at https://2ctptqj9vf3lafyt2rkh1qto-wpengine.netdna-ssl.com/wp-content/uploads/2017/10/bork.jpg 15:45:37 * TheJulia takes the silence as everyone is ready to go have more coffee 15:45:51 yay more coffee 15:46:13 dtantsur: I think something is lost in translation :( 15:46:38 Okay everyone, have a wonderful week! 15:46:41 TheJulia: rather some history of memes :) 15:46:45 :) 15:47:19 Thanks everyone 15:47:29 thank you :) 15:47:31 ty 15:47:34 Thank you 15:47:39 thanks you :) 15:47:45 #endmeeting