15:00:17 <TheJulia> #startmeeting ironic
15:00:17 <openstack> 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 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:18 <TheJulia> o/
15:00:20 <iurygregory> o/
15:00:20 <dtantsur> \o
15:00:20 <openstack> The meeting name has been set to 'ironic'
15:00:22 <rpittau> o/
15:00:26 <hjensas> o/
15:00:27 <arne_wiebalck> o/
15:00:33 <dnuka> o/
15:00:36 <kaifeng> o/
15:00:46 <stendulker> o/
15:00:46 <TheJulia> Good morning everyone!
15:00:55 <dnuka> good morning TheJulia
15:01:04 <rpittau> good morning!
15:01:08 <TheJulia> Our agenda this morning is fairly light, so hopefully we will get this wrapped up nice and quickly. I hope
15:01:25 <TheJulia> #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting
15:01:26 <rpioso> o/
15:01:38 <TheJulia> #topic Announcements / Reminders
15:02:20 <mjturek> o/
15:02:51 <TheJulia> #info This week is R-7, which means next week is our final non-client library release for this cycle.
15:02:54 <TheJulia> #link https://releases.openstack.org/stein/schedule.html
15:03:22 <dtantsur> which, I guess, includes sushy, ironic-lib and metalsmith (?)
15:03:39 <TheJulia> sushy and ironic-lib. Metalsmith likely...
15:04:07 <TheJulia> #info The Forum/PTG idea etherpad has been posted and already has some great topics.
15:04:09 <TheJulia> #link https://etherpad.openstack.org/p/DEN-train-ironic-brainstorming
15:05:18 <TheJulia> 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 <TheJulia> #info The Cisco UCS drivers have been marked as of deprecated in a proposed review.
15:05:44 <TheJulia> :(
15:05:59 <TheJulia> #link https://review.openstack.org/#/c/637241/
15:06:00 <patchbot> patch 637241 - ironic - Deprecate Cisco drivers - 1 patch set
15:06:05 <rpittau> :/
15:06:12 <TheJulia> 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 <TheJulia> :(
15:07:24 * TheJulia takes the crickets as indication we can proceed
15:07:28 <dtantsur> yep
15:07:35 <TheJulia> #topic Review action items from previous meeting:
15:07:39 <TheJulia> #undo
15:07:40 <openstack> Removing item from minutes: #topic Review action items from previous meeting:
15:07:40 <TheJulia> #topic Review action items from previous meeting
15:08:23 <TheJulia> 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 <TheJulia> I'll try to get something sent out in the next few days. Today should... hopefully... be quiet.
15:09:05 <dtantsur> maybe we just skip it
15:09:14 <TheJulia> We did keep really good notes
15:09:22 <TheJulia> Anyway
15:09:22 <dtantsur> after all, the etherpad is public (and next time we should remember to record the call)
15:09:47 <mjturek> TheJulia: maybe an email pointing to the etherpad would be nice? I can help there if you'd like
15:09:49 <TheJulia> Perhaps...
15:09:51 * arne_wiebalck thinks the notes on the etherpad are good enough
15:09:54 <cdearborn> \o
15:09:57 <TheJulia> mjturek: that would be awesome
15:10:08 <TheJulia> Anyway, time to move on!
15:10:11 <TheJulia> #topic Review subteam status reports
15:10:15 <mjturek> you got it
15:10:25 <TheJulia> #link https://etherpad.openstack.org/p/IronicWhiteBoard
15:10:42 <TheJulia> Starting at line 289
15:11:40 <TheJulia> 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 <dtantsur> ++
15:12:15 <TheJulia> jroll: any indication if the conductor grouping stuff is going to make it for nova this cycle?
15:12:20 <dtantsur> FYI I tried our unit tests with py38-alpha1 :)
15:12:20 <rpittau> I definitely vote for 3.7 :)
15:12:36 <jroll> sorry, forgot about this meeting >.>
15:12:46 <jroll> TheJulia: I hope so, it has a +2
15:13:02 <TheJulia> :)
15:13:51 <TheJulia> The data storage for deploy templates is in good shape, looks like the remaining patches are still WIP
15:14:08 <dtantsur> yeah, left a few reviews on the API one, mostly complaining about /details :)
15:14:22 <TheJulia> heh, okay
15:14:38 <TheJulia> 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 <dtantsur> only two ironicclient patches, right? or is there anything else?
15:15:42 <TheJulia> dtantsur: one more ironicclient patch and a follow-up patch
15:16:15 <TheJulia> 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 <TheJulia> 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 <dtantsur> ++ no point in hurrying it in
15:18:46 <dtantsur> I hope to finish the allocation API work (without backfilling) this cyclle
15:18:50 <rpittau> TheJulia, for the  graphical console I've been working on RPC/API
15:19:07 <TheJulia> rpittau: could you put that note on the status?
15:19:15 <rpittau> yes
15:19:18 <dtantsur> rpittau: awesome! still, without any vendor implementing it, we don't have to hurry just yet
15:19:37 <TheJulia> I've been really amazed just how many new things we've merged this cycle
15:19:43 <rpittau> yeah
15:19:56 <TheJulia> Anyway, I'm done going down the list and I guess we can proceed to priorities for the next week.
15:20:07 <TheJulia> Everyone else ready to proceed?
15:20:54 <larsks> 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 <dtantsur> larsks: you're right
15:21:20 <dtantsur> TheJulia: ++ to proceed
15:21:22 <larsks> dtantsur: Thanks.
15:21:41 <TheJulia> #topic Deciding on priorities for the coming week
15:21:55 <TheJulia> 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 <TheJulia> #link https://etherpad.openstack.org/p/IronicWhiteBoard
15:22:20 <TheJulia> Starting at line 160
15:23:05 <dtantsur> quite a few things have merged indeed
15:23:34 <TheJulia> dtantsur: Want any ironic eyes on https://review.openstack.org/636594 ?
15:23:35 <patchbot> patch 636594 - openstacksdk - baremetal: support for allocation API - 3 patch sets
15:23:50 <dtantsur> TheJulia: I would not mind, but I don't dare asking given how busy everyone is :)
15:24:00 <TheJulia> Alright
15:24:45 <dtantsur> I just need to nag mordred at some point :)
15:25:57 <TheJulia> Looks like someone moved up a networking-baremetal item
15:27:02 <hjensas> I added some backports.
15:27:19 <TheJulia> hjensas: they were already on the list :)
15:27:22 <TheJulia> I did move them up
15:27:50 <TheJulia> I think we're in good shape list wise. Is there anything anyone feels should be on the list?
15:28:13 <hjensas> TheJulia: oh, ok. Thanks, I removed the line I added.
15:28:19 <dtantsur> TheJulia: do we need graphical console in the priorities, if we're not merging it until the API is proposed?
15:28:32 <TheJulia> I was just typing something out asking if we want to remove it
15:28:52 <TheJulia> Removed
15:28:57 <rpittau> I don't think there will be time anyway for a review
15:29:34 <TheJulia> Okay, I guess we're good to proceed then
15:29:47 <dtantsur> LGTM
15:29:51 <TheJulia> Everyone good to proceed to ?Open Discussion?
15:30:38 <rpittau> it seems we can move forward :)
15:30:40 * TheJulia guesses that is a yes
15:30:45 <TheJulia> #topic Open Discussion
15:30:55 <openstackgerrit> Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: DNM - Test  https://review.openstack.org/637552
15:30:58 <TheJulia> rpittau: Seems like you are seeking some help with fixing python3 support in python-hardware ?
15:31:36 <dtantsur> 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 <rpittau> TheJulia, yes, so far dtantsur has been bery helpful with reviewing and merging some PRs in that hardware repo
15:31:57 <rpittau> oh ok I didn't know that
15:32:09 <dtantsur> but we need agreement from all current maintainers, esp. Erwan (?)
15:32:27 <dtantsur> yeah, e.g. ironic-staging-drivers is an independent repository
15:33:15 <TheJulia> 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 <patchbot> patch 635996 - ironic - fast tracked deployment support - 11 patch sets
15:33:30 <rpittau> the idea was to be able to keep the hardware repo in shape as much as possible
15:33:54 <dtantsur> 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 <dnuka> :)
15:34:23 <dtantsur> 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 <TheJulia> dtantsur: no worries. I think once I have an integration test performing discovery -> deployment without rebooting... I'll be super happy
15:34:38 <openstackgerrit> 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 <dtantsur> TheJulia: a lot of people will
15:34:56 <rpittau> dtantsur, sounds good
15:35:14 <TheJulia> dtantsur: next step.. the world?
15:35:27 <dtantsur> absolutely
15:35:27 <kaifeng> uh, the fast track means?
15:36:10 <dtantsur> kaifeng: from enroll to active with only one reboot
15:36:27 <TheJulia> 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 <dtantsur> boot the node for discovery, leave it running, clean, leave it running, deploy, reboot into instance
15:37:04 <kaifeng> awesome!
15:37:13 <chris_j> good idea!
15:38:11 <chris_j> to accelerate the deployment .. some users might not be happy about additional power consumption by running servers
15:38:36 <kaifeng> 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 <TheJulia> Anyway, i'd love some reviews, the main thing right now is the tempest test for me
15:39:12 <dtantsur> kaifeng: the state machine will progress as usual, we just won't shut down the machine in between
15:39:14 <jroll> chris_j: it's opt-in, so shouldn't be any surprises
15:39:25 <dtantsur> yep, and totally opt-in
15:39:37 <chris_j> awesome! thanks for clarifying
15:39:47 <TheJulia> 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 <kaifeng> hmm, I guess there is no spec right? maybe I can only find answers from patches :)
15:40:19 <TheJulia> kaifeng: yeah, no spec. I put together an rfe, and it really seems fairly simple
15:41:15 <TheJulia> 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 <kaifeng> okay, I saw this patch https://review.openstack.org/#/c/636778/ but it's not attached to a story yet
15:41:33 <patchbot> patch 636778 - ironic-inspector - Add ironic API url to inspector IPA config - 2 patch sets
15:42:31 <arne_wiebalck> kaifeng: you’re looking for this one? https://storyboard.openstack.org/#!/story/2004965
15:43:13 <TheJulia> kaifeng: and https://review.openstack.org/635996
15:43:14 <patchbot> patch 635996 - ironic - fast tracked deployment support - 11 patch sets
15:44:02 <kaifeng> got it, thanks
15:44:07 <TheJulia> 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 <iurygregory> yay more coffee
15:46:13 <TheJulia> dtantsur: I think something is lost in translation :(
15:46:38 <TheJulia> Okay everyone, have a wonderful week!
15:46:41 <dtantsur> TheJulia: rather some history of memes :)
15:46:45 <dnuka> :)
15:47:19 <TheJulia> Thanks everyone
15:47:29 <rpittau> thank you :)
15:47:31 <iurygregory> ty
15:47:34 <rpioso> Thank you
15:47:39 <dnuka> thanks you :)
15:47:45 <TheJulia> #endmeeting