14:00:53 <edleafe> #startmeeting nova_scheduler
14:00:54 <openstack> Meeting started Mon Mar 13 14:00:53 2017 UTC and is due to finish in 60 minutes.  The chair is edleafe. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:56 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:58 <openstack> The meeting name has been set to 'nova_scheduler'
14:01:04 <bauzas> \o
14:01:05 <macsz> \o
14:01:06 <edleafe> Good UGT morning! Who's here?
14:01:11 <cdent> orz
14:01:33 <jroll> morning :)
14:02:14 <edleafe> With the recent DST shift, I now have time to caffeinate before these meetings :)
14:02:28 * cdent holds on to his hat
14:02:57 <edleafe> cdent: oh, don't worry. It just means I may not be as dimwitted as usual
14:03:16 <jaypipes> o/
14:03:19 <cdent> does that mean I should be holding on to my sides?
14:03:34 <edleafe> Hold on to whatever you like
14:04:16 <bauzas> FWIW, I'm still having 25 mins for you folks before rushing out
14:04:24 <bauzas> at least, till April
14:04:28 <bauzas> and then DST for me
14:04:47 <alex_xu> o/
14:05:28 <edleafe> Let's get started
14:05:31 <edleafe> #topic Specs & Reviews
14:05:52 <jaypipes> traits ahoy
14:05:56 <edleafe> first up: Traits series, starting with:
14:06:06 <edleafe> #link
14:06:11 <edleafe> doh!
14:06:18 <jaypipes> merged.
14:06:23 <jaypipes> :)
14:06:24 <edleafe> #link https://review.openstack.org/#/c/376199/
14:06:25 <alex_xu> #link https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/resource-provider-traits
14:06:39 <alex_xu> the patch series :)
14:07:10 <edleafe> alex_xu: this looks like it's getting very close
14:07:10 <jaypipes> yup, ^^ it highest priority.
14:07:20 <alex_xu> edleafe: thanks
14:07:21 <edleafe> alex_xu: Any issues we need to focus on or discuss?
14:07:25 <jaypipes> alex_xu: I'm reviewing that series this morning.
14:07:42 <alex_xu> 1. I removed the cache
14:08:04 <jaypipes> edleafe, alex_xu: I will have a couple patches up today for adding networking traits to os-traits.
14:08:05 <alex_xu> 2. is there anyway I can separate one microversion into two separated patches?
14:08:26 <jaypipes> alex_xu: not if both patches add/change something in the API.
14:08:31 <alex_xu> jaypipes: cool
14:08:45 <jaypipes> alex_xu: but you can split out patches and isolate only the API-affecting stuff into a single patch.
14:08:46 <alex_xu> jaypipes: ok
14:08:48 <edleafe> alex_xu: +1 to cache removal
14:09:34 <alex_xu> ok, let me merge the last two API patches
14:09:48 <edleafe> jaypipes: those changes won't conflict with anything in alex_xu's series, will they?
14:11:27 <edleafe> Oh, well, guess Jay got pulled in another direction... :)
14:11:30 <edleafe> Moving on...
14:11:31 <edleafe> Ironic inventory is reeeeeally close:
14:11:33 <edleafe> #link https://review.openstack.org/#/c/441544/
14:11:56 <edleafe> Let's all get our eyes on that
14:12:19 <edleafe> Nested RPs needs some attention:
14:12:20 <edleafe> #link https://review.openstack.org/#/c/415920/
14:12:32 <edleafe> But those are waiting for the traits stuff
14:12:52 <edleafe> Finally:
14:12:53 <edleafe> Improve placement API error messages:
14:12:53 <edleafe> #link https://review.openstack.org/#/c/418393/
14:13:11 <edleafe> Comments on any of these?
14:13:17 <jaypipes> edleafe: you mean the os-traits ones?
14:13:27 <edleafe> jaypipes: yes
14:13:29 <jaypipes> edleafe: sorry, no, they won't affect alex's patches.
14:13:44 <edleafe> jaypipes: cool, I didn't imagine they would
14:13:55 <_gryf> edleafe, there was some comments from stephen
14:14:05 <_gryf> which was addressed
14:14:14 <edleafe> _gryf: on which?
14:14:28 <_gryf> edleafe, imporve api error mesgs
14:15:16 <edleafe> I have in my queue to re-review with those changes
14:15:23 <_gryf> cool, hanks
14:15:27 <_gryf> *thanks
14:15:36 <edleafe> Anything else for Specs and Reviews?
14:15:56 <edleafe> #topic Bugs
14:16:05 <edleafe> No new bugs that I'm aware of
14:16:20 * edleafe hasn't searched that deeply, though
14:17:00 <cdent> I looked late last week, and agree
14:17:13 <edleafe> Moving on...
14:17:17 <edleafe> #topic Open discussion
14:17:19 <cdent> (I try to regularly look for things that need a placement or scheduler tag)
14:17:28 <edleafe> Functional testing:
14:17:29 <edleafe> https://etherpad.openstack.org/p/nova-placement-functional
14:17:40 <edleafe> We have some general approaches there
14:17:53 <edleafe> I have some first baby steps outlined at the bottom
14:18:15 <edleafe> I'd like to start working on this this week
14:18:25 <edleafe> Unless anyone has objections
14:18:44 <bauzas> not sure I get the problem
14:18:47 <bauzas> edleafe: ^
14:19:13 <jroll> edleafe: those baby steps lgtm
14:19:29 <bauzas> edleafe: you know that we have a PlacementFixture, right?
14:19:37 <edleafe> bauzas: we lack functional tests, and that prevents us from making changes with any confidence
14:19:37 * jroll reminds folks that he'll be writing specs for some of the flavor changes and ironic transition this week
14:19:58 <edleafe> bauzas: yes
14:20:14 <cdent> jroll++
14:20:15 <edleafe> bauzas: I'd like to use it more
14:20:41 <bauzas> edleafe: okay, you confused me with "jay's fixtures" then
14:21:21 <bauzas> and yeah, I agree with you, we're missing functional coverage
14:21:49 <edleafe> bauzas: the idea was to have a single base instead of several
14:22:59 <edleafe> So I'll probably start on this today, and have a first attempt soon. I'll ping people on -nova to review
14:23:22 <edleafe> Any other comments/concerns about functional testing changes?
14:24:09 <edleafe> whoops, forgot to
14:24:11 <edleafe> #link Functional testing:
14:24:12 <edleafe> https://etherpad.openstack.org/p/nova-placement-functional
14:24:29 * edleafe is having copy/paste issues this morning!
14:24:48 <edleafe> #undo
14:24:49 <openstack> Removing item from minutes: #link https://etherpad.openstack.org/p/nova-placement-functional
14:24:53 <edleafe> #link https://etherpad.openstack.org/p/nova-placement-functional
14:25:17 <jaypipes> edleafe: cool with me. ++
14:25:38 <edleafe> OK, moving on.
14:25:39 <edleafe> Extracting placement from Nova
14:25:40 <edleafe> Dev ML discussion starting with:
14:25:41 <edleafe> #link http://lists.openstack.org/pipermail/openstack-dev/2017-March/113812.html
14:25:53 <edleafe> cdent: wanna run with this one?
14:26:29 <cdent> a) the email pretty much says it: I extracted a working placement and found some few small issues, some of which we can deal with independently of any actual plan to do extraction
14:27:01 <cdent> b) it wasn't my intent to such any timeline or solid plan, just to change guessing to less guessing, because all the guessing was causing some FUD
14:27:25 <jroll> ++
14:27:37 <jaypipes> edleafe: on functional testing (sorry, late), please get with sfinucan, since he's also been working on that for libvirt.
14:28:00 <cdent> c) that said, I continue to think that sooner is better than later: we have a self-contained thing here that is _easy_ to extract (unlike the rest of nova) and we can help both nova and placement by having disconnected review processes
14:28:03 <cdent> EOS
14:28:21 <edleafe> jaypipes: will do
14:29:06 * bauzas bails out
14:29:17 <bauzas> damn kids
14:29:18 <edleafe> cdent: separating it sooner will mean less time for rust to build that will make them harder to separate later
14:29:23 <jaypipes> cdent: I agree with the sentiment, as mentioned on the ML. I think getting the discussion started *now* about a separation in Queens is A Good Thing. dansmith has some concerns on whether to include claims in the list of things to do *before* split-out, but is willing to discuss further about an appropriate delimiting point.
14:29:28 <edleafe> rust == linkages
14:30:38 <edleafe> OK, how about we continue the discussion on the ML, and talk further at our next meeting?
14:30:44 <cdent> +1
14:31:08 <edleafe> It's much too early in the thought process to be able to make any decisions, anyway
14:31:12 <cdent> yes
14:31:19 <_gryf> btw, when is the spec freeze?
14:31:39 <edleafe> But I do think knowing the issues can help us clean up those bits earlier, so thanks cdent
14:31:51 * cdent bows
14:32:06 <edleafe> _gryf: https://releases.openstack.org/pike/schedule.html
14:32:11 <_gryf> thx
14:32:38 <edleafe> Anything else for Opens?
14:34:25 <edleafe> OK, thanks everyone!
14:34:27 <edleafe> #endmeeting