21:00:33 #startmeeting nova 21:00:34 Meeting started Thu Feb 26 21:00:33 2015 UTC and is due to finish in 60 minutes. The chair is mikal. Information about MeetBot at http://wiki.debian.org/MeetBot. 21:00:35 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 21:00:38 The meeting name has been set to 'nova' 21:00:41 o/ 21:00:43 Mornnig all! 21:00:50 #topic Kilo release status 21:01:00 dims_: Thanks. 21:01:01 The next deadline applies to all blueprints: 5th March https://wiki.openstack.org/wiki/FeatureProposalFreeze 21:01:04 o/ 21:01:27 So that's the next step in our progressive freeze 21:01:32 And is disturbingly soon 21:01:38 (i.e. next week) 21:01:39 o/ 21:01:45 So just a reminder that its crunch time... 21:02:02 John asks that when all code is up, please mark blueprint as NeedsCodeReview, then Implemented when all is merged 21:02:22 I also want to note that all the "merge quickly" exceptions from last week merged, so that's cool 21:02:28 Anything else on release status? 21:02:53 Moving on... 21:02:58 #topic Kilo priorities 21:03:09 So, a crazily large amount of API code has landed this week 21:03:15 How are the other priorities going? 21:03:30 cells code could use some reviews 21:03:35 it's on the etherpad 21:03:40 Cool 21:03:43 nova-net to neutron migration has refocused on L release 21:03:43 I will take a look today 21:03:58 There was also a spec for scheduler db stuff which was in review when I went to bed last night 21:04:04 Not sure what people have said about it overnight 21:04:19 mostly good things :) 21:04:32 Still waiting for your stamp of approval, though 21:04:34 anteaya: yeah, thanks for that. So the nova-net migration people are going to take some more time to absorb the feedback from reviews for their prototype, and try for L at this point 21:04:49 obondarev has put up a new patch, if we can get some reviews to figure out if this direction will net results that would be wonderful, patchsize increased from about 350 lines to 700 21:04:51 * mriedem joins late 21:04:58 edleafe: so, is there any chance of the implementation getting proposed in gerrit before 5 March? 21:05:10 some of it is up 21:05:11 I tink 21:05:15 mikal: it's been in gerrit for a couple of weeks now as WIP 21:05:16 * mikal notes he is holding up that review 21:05:18 so mostly we need to know, yes on this direction or subclass the api 21:05:42 anteaya: can we add "don't do this at all" to the list of options? :) 21:05:55 dansmith: if that is what you feel 21:05:58 add away 21:06:04 \o/ 21:06:12 of course I will follow up with a what do you suggest as a next step 21:06:14 that's definitely what I feel, but I shan't add it 21:06:23 So, I'd say we should have a summit session on that bit specifically, except the summit feels too far away 21:06:24 thanks for being honest with me 21:06:26 :) 21:06:28 Dan't has spoken! 21:06:35 ha ha ha 21:06:58 mikal: summit session on edleafe's thing or the migration? 21:07:05 YEah, sorry 21:07:09 We're talking about two things at once 21:07:12 The migration 21:07:13 I don't want a summit session on the migration, but would like a friday table 21:07:20 anteaya: ok 21:07:32 But I think I'd like to see progress before then, I don't want a stall waiting for the summit 21:07:37 the last thing i want is to be trapped in a room of 400 people who never show up to meetings saying do it my way 21:07:42 agreed 21:07:46 anteaya: heh, true dat 21:07:56 Ok, so the other topic... 21:08:01 edleafe: I promise to review your thing today 21:08:05 I would like a friday table session where we can get through any deadlocks 21:08:07 mikal: thx 21:08:10 Or you can send me rotten fruit in the mail 21:08:17 edleafe: I promise to annoy mikal until he does ;P 21:08:33 Is there anything else on priorities or shall we move on? 21:08:34 tonyb: that should help 21:08:42 :) 21:09:08 #topic Liberty release status 21:09:13 Ok, so first off... 21:09:24 We've been asked what we'd like for the summit in terms of rooms etc 21:09:28 Noting we might not get all we ask for 21:09:32 Do we want what we had in Paris? 21:09:42 Big room for a couple of days and then a big table room on Friday? 21:09:47 that was nice 21:09:50 friday meetup was good 21:09:57 I was surprised by how well the Friday worked to be honest 21:10:13 roundtable is good for the meetup rather than audience style 21:10:23 Do we see value in doing _any_ audience style? 21:10:30 It has the advantage of a schedule and time boxing 21:10:43 shrug 21:10:49 I like a big round table or square formation rather than long boardroom style 21:10:50 Audience is OK, but we need the rings 21:10:52 i think the 2 sessions on priorities on thursday were also good 21:10:56 You could add a schedule and time boxing to a roundtable 21:11:22 I personally think we should keep the mix the same as Paris 21:11:30 agreed 21:11:30 * mdbooth found the audience style all but useless unless you're in the front row 21:11:30 Because audience still feels productive to me 21:11:31 You should have some open sessions, but I think we could have 1 more round table day and cut one open session day maybe. 21:11:32 I think that's fine 21:11:37 And I worry that all table would go off the rails 21:11:40 Sounds like people agree? 21:11:42 Or don't care? 21:11:43 :P 21:11:48 ==paris 21:11:49 +1 to paris style 21:11:49 +1 21:11:53 +1 21:11:55 all table would go off teh rails 21:11:58 I think that the meetup style discourages passer-by input 21:11:59 mriedem: oh you just coiled a term forever 21:12:02 er, coined 21:12:13 only if bauzas is around 21:12:14 that may be a good thing but I feel ike Paris worked pretty well 21:12:15 *giggity* 21:12:20 Heh 21:12:27 Ok, I shall let ttx know our desires 21:12:28 Speaking of off the rails.... 21:12:38 And then we'll see how physical reality maps to that 21:12:52 So the other thing is... 21:12:57 Liberty specs are open 21:13:09 For people who got bumped from Kilo, you can retarget your thing right now if you want to 21:13:20 Review wont open for a bit, but it means you can do your bits if you desire 21:13:25 mikal: maybe a ML post on that soon? 21:13:36 I assume we will do a fast track approval thing in L as well, but I note I am not the L PTL 21:13:39 jogo: yep 21:13:48 So I don't want to promise thigns for L until we know who the PTL is 21:14:07 I should figure out those election dates 21:14:16 That would be good! 21:14:30 Actually, I should figure out the extra-atcs thing before the election too 21:14:38 yes please 21:14:47 Do you have a deadline for that being done? 21:14:47 and thanks for checking foundation membership 21:14:55 before teh election roll is fixed 21:14:58 mikal: so who would one talk to about planning mid-cycle meetups if one were so inclined? 21:15:06 which is the end of ptl nominations 21:15:13 mriedem: heh, I should answer your email... 21:15:14 i guess next ptl 21:15:22 mriedem: it is certainly a next PTL call 21:15:32 mriedem: but its also a what the team wants thing I think 21:15:57 mriedem: I promise to answer your email today 21:16:04 yeah, I'd avoid punting just because $ptl, since it's a thing that needs to get going 21:16:12 dansmith: agreed 21:16:23 is there even a liberty release schedule yet? 21:16:31 I just feels dishonest to make a lot of edicts about L when it might be someone else living with them 21:16:33 mriedem:: I promise to annoy mikal until he does ;P 21:16:36 mriedem: I don't think so? 21:16:42 don't see a wiki 21:16:43 so nope 21:16:45 * mikal is making a worrying number of promises today 21:16:59 Let's move on again unless there is something else here... 21:17:00 promise you'll never let go... 21:17:18 lol 21:17:25 mriedem: I will be very strong and not tell you my Rick Astley story of the week 21:17:33 #topic Gate status 21:17:38 gate is gr8 21:17:40 mikal: until after the meeting, right? 21:17:43 So, there was a library thing, yeah? 21:17:46 dansmith: sure 21:17:57 oh right, i was going to release novaclient today 21:17:59 oops 21:18:01 A release of something which did something but then soemthing fixed the something? 21:18:05 * mriedem blames jogo for not reminding 21:18:09 Heh 21:18:10 it's fixed in tree 21:18:18 Oh, I was thinking of the keyring release breaking IIRC 21:18:18 supposed to release 2.21.1 (?) today 21:18:30 there were other stable branch breaks earlier in the week but not us 21:18:31 mriedem: do we think releasing on Thursdays is a good idea? 21:18:35 Or should we wait until Monday? 21:18:39 probably monday at this point 21:18:44 I feel that way too 21:18:46 * jogo points out he was asleep when mriedem should have cut the release 21:18:49 In case we horribly wedge the world 21:18:55 oh come on 21:18:58 that could never happen 21:19:00 LOL 21:19:07 it definitely didn't happen last time 21:19:08 given it's only one change to revert the thing that broke everything before 21:19:09 it should be safe 21:19:11 but monday it is 21:19:18 we can always blame something in oslo :) 21:19:21 I need a "what could possibly go wrong" tshirt 21:19:29 Let's move on again... 21:19:31 #topic Bugs 21:19:33 dims_: we could go ahead and get the blame train moving on o.vo 21:19:41 :) 21:19:42 There is a critical on the agenda... 21:20:00 * mdbooth put that there. Needs eyes. 21:20:42 Fair enough 21:20:59 I might be worth -1'ing the other contender for that migration number of avoid a rebase on a critical fix? 21:21:00 TL;DR Relatively minor misconfiguration of Ironic or VMware can delete all your instances 21:21:03 so this shouldn't be a critical 21:21:12 jogo: greed 21:21:12 this doesn't impact all users 21:21:17 jogo: there's a discussion of that in the bug 21:21:19 sheesh 21:21:20 agreed 21:21:24 I tried and was told I was a liar 21:21:27 Well, nicely 21:21:38 mikal: Glad to be polite :) 21:21:44 Heh 21:21:48 It's critical because it will delete all your instances 21:21:50 i.e. data loss 21:21:59 we have a definition of critical 21:22:02 and this isn't it, IMHO 21:22:07 dansmith: Data loss is in it 21:22:10 * mdbooth gets the link 21:22:23 mdbooth: isn't there a work around? don't run redundant nova computes 21:22:35 may not be a good work around but not critical 21:22:40 jogo: You can do it accidentally 21:22:43 IMHO, this is using nova in a not valid way 21:22:58 you can do a lot of thing accidentally in life 21:23:01 dansmith: It does involve a misconfiguration, that's right. 21:23:04 For reference, https://review.openstack.org/#/c/122240/ is the other contender for migration 277 21:23:09 dansmith: yes, that is the bigger issue 21:23:18 So, here's the thing 21:23:18 so data loss due to misconfiguration of one or two drivers != critical, IMHO 21:23:21 However, this is a relatively small misconfiguration with a very large impact 21:23:27 We're having a disagreement about a single piece of metadata 21:23:38 Couldn't we just review the fix, land it, and get on with our lives? 21:23:44 mikal: +1 21:23:56 Or reject the fix of course if we think its wrong 21:23:57 mikal: we could, but there is a deeper issue which this whole model is just bad 21:24:04 right 21:24:16 This is why it needs discussion. 21:24:26 This proposal enforces an existing assumption 21:24:39 The assumption is already implicit in code 21:24:47 This proposal makes it explode if you break it 21:24:49 This proposal == https://review.openstack.org/#/c/158269/ ? 21:24:57 Instead of corrupting your instance 21:25:07 tonyb: Yea 21:25:08 I don't disagree with revisiting the design of this, but I do think we should be protecting current deployments of ironic from face punching 21:25:34 mikal: Also +1 21:25:50 * mdbooth agrees the design should be revisited 21:25:54 * jogo wants more face punching 21:26:02 Sigh 21:26:07 Ok, so 21:26:07 so what is the hold up between the two options? 21:26:13 * bauzas waves late 21:26:13 I'd love some review on that proposed fix 21:26:17 jogo: Which other option? 21:27:04 mdbooth: I thought this was https://review.openstack.org/#/c/154907/1 vs https://review.openstack.org/#/c/158269 ? 21:27:58 jogo: Ah, yes. That does a similar thing by storing metadata in vsphere. It doesn't cover Ironic, for eg. 21:28:07 Also, it's uglier. 21:28:09 so, 21:28:10 mikal: so, I think https://review.openstack.org/#/c/122240/ should get 277, johanes is good with it, redo this one as 278 and ask people to review 21:28:29 sdague: I would be ok with that 21:28:32 I like a driver-specific bandage for this right off 21:28:44 sdague: but if we're going to land 240 let's pull the trigger now 21:28:46 +W on https://review.openstack.org/#/c/122240/ - so 277 is taken now 21:28:46 dansmith: 2 driver-specific bandages 21:28:47 I'm still trying to digest the supposedly global fix 21:28:49 sdague: my -1 there is no longer valid 21:28:58 mdbooth: driver specific implies "in each driver affected" yes 21:29:13 dansmith: yeah, I like the driver specific thing as well 21:29:15 dansmith: Also, the assumption is in Nova, not in the driver 21:29:23 mdbooth: I'm aware 21:29:25 i.e. Nova core 21:29:30 I also think the assumption is fine 21:29:45 All I'm doing is proposing that Nova's existing assumption is enforced by Nova 21:29:53 Seems safest and cleanest 21:30:01 I know 21:30:17 I think we should (a) make this non-critical and (b) review/continue off the meeting 21:30:29 mdbooth: the bigger issue here is we have drivers that are designed where this is even an issue 21:30:36 dansmith: already took it out of critical 21:30:42 cool, then (b) ? :) 21:30:42 Ok, so 21:30:47 Yeah, let's review this code 21:30:53 And discuss there 21:31:01 I can't see us resolving this in this meeting 21:31:46 #topic Stuck reviews 21:31:52 So, there are none on the agenda 21:31:56 And I don't know of any 21:32:17 Anythign where we will never reach a concensus on gerrit? 21:32:37 how to spell concensus? 21:32:37 No? 21:32:37 well... I could troll with this - https://review.openstack.org/#/c/150929/ 21:32:41 I can't see us ever agreeing on that 21:32:45 dansmith: you're dead to me 21:32:53 heh 21:32:54 sdague: you're in danger as well 21:32:58 ha sdague! 21:32:58 sdague: :D 21:33:02 sdague: take it out of -W 21:33:05 sdague: my +2 is already on it 21:33:12 * mikal rage quits this topic 21:33:17 #topic Open Discussion 21:33:21 Ok, now discuss openly 21:33:25 mikal: so it was only a half troll 21:33:31 Or I call tell you about the shirine to Rick Astley I visited this week 21:33:44 any feed back on trivial patch monkey process is welcome. https://etherpad.openstack.org/p/kilo-nova-priorities-tracking 21:33:55 dims_: I find it annoying 21:34:00 but I want to make sure we're good that if we get the change in the ec2 folks need for out of tree, we can deprecate 21:34:02 I go to review those things, and they're already merged 21:34:02 there is this review for working around a libvirt bug where there's some uncertainty about whether a workaround config option is needed https://review.openstack.org/151953 21:34:10 dims_: been happy with it 21:34:17 mikal: we update it twice a day at least 21:34:22 sdague: oh, sorry. I thought that was the vmware / ironic cluster review again 21:34:25 will try to do it before you wake up :) 21:34:30 dims_: everyone else should review slower 21:34:35 I figure I'll wait until after ops summit to un WIP it just so we can get some face reactions with ops on it 21:35:01 dims_: provided monkeys have time :-) 21:35:03 sdague: oh, so that review is me sucking to be honest 21:35:09 bauzas: haha 21:35:10 dims_: can we remove the list of merged things now? I don't think we need that list to show it works 21:35:16 sdague: I haven't had a chance to sit ttx down and work out a way forward which meets his needs 21:35:25 jogo: happy to 21:35:27 sdague: its on my todo list 21:35:34 dims_: it would make it easier to scroll to the bottom 21:35:37 I am lazy 21:35:41 mikal: ok, no prob. I just wanted to keep it on the front burner so it doesn't get lost before k3 21:35:53 sdague: agreed, thanks for keeping me honest 21:35:58 sdague: mikal we really need quick work on this 21:36:12 jogo: i am going to add a counter instead 21:36:15 do we have anyone attending the ops summit to take notes? 21:36:19 dims_: ++ 21:36:29 jogo: sdague is 21:36:34 mikal: nice 21:36:36 yeh, I'll be there 21:36:53 dansmith: yep I know, I already feel guilty 21:36:54 dims_: counter on what ? 21:37:13 bauzas: count of things merged 21:37:22 dims_: I think a counter is useful, because the board occassionally asks how we're fixing this. Metrics are therefore always nice. 21:37:59 dims_: ack. Please don't provide name stats 21:38:37 Name stats? 21:38:50 individual stats 21:38:55 I mean, count on reviews/monkey 21:39:04 As in who is the best predictor of mergable monkey patches? 21:39:07 mikal: dims_ (11'ity billion) tonyb (0) etc 21:39:16 tonyb: right 21:39:21 right, agreed. But bugs merged or reviews closed, or something 21:39:21 Huh, why not? 21:39:24 Not that I need that stat 21:39:32 Just curious why we explicitly don't want it 21:39:33 mikal: gamification 21:39:47 Oh, ok 21:39:47 mikal : because I think it would corrupt the process 21:39:47 mikal: and because some people are pretty bad at it :P 21:39:50 Sure 21:40:42 * dims_ spies 2 deer sauntering by his window 21:40:47 So... in other news most of the electrical bits of my car don't work. Are we done here so I can go and get it fixed? 21:41:03 What I would normally call an "early mark"? 21:41:32 nova is now oslo namespace free 21:41:40 dims_: \o/ 21:41:44 \o/ 21:41:51 and we have novaclient functional tests for a regression 21:41:57 those are nice wins this week 21:41:59 nice 21:42:10 and o.vo! 21:42:20 yay 21:42:22 Yay team! 21:42:25 woohoo! 21:42:27 and I'm about to make dhellmann eat his napkin 21:42:28 and dansmith drinks ovoltine 21:42:35 I shall eat a cookie as a reward 21:42:37 hahaha 21:42:38 heh 21:42:47 sdague: oh, t-shirt. 21:42:49 mikal: isn't it breakfast time by you? 21:42:53 So, I think that means we're done for this week 21:43:00 jogo: yes, breakfast cookie of victory 21:43:05 Its been lovely typing near you all 21:43:09 dims_: nice, I had a turkey fly past the other day 21:43:09 #endmeeting