12:00:54 #startmeeting horizondrivers 12:00:55 Meeting started Wed Aug 12 12:00:54 2015 UTC and is due to finish in 60 minutes. The chair is david-lyle. Information about MeetBot at http://wiki.debian.org/MeetBot. 12:00:56 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 12:00:58 The meeting name has been set to 'horizondrivers' 12:01:10 any horizon folks around? 12:01:19 yes! 12:01:26 good morning david-lyle ! 12:01:43 good afternoon mrunge 12:02:24 Afternoon 12:03:22 So.. 3 of us? 12:03:27 I don't see many others online right now, so yeah 12:03:37 makes decisions easier ;-) 12:03:55 good morning all 12:04:43 david-lyle, have you had the time to look at blueprints? 12:05:06 So as discussed in last weeks Horizon team meeting, we wanted to clean up the blueprint backlog in horizon and get back to the point where we can provide more immediate feedback on new blueprints proposed. And to clean out ones that are obsolete or don't make sense any more 12:05:06 or did anyone found out, which blueprints need to be discussed? 12:05:15 just wanted to record that for other 12:05:17 s 12:05:36 mrunge: I am open to plans of attach 12:05:36 yes, we have way too many bps to act on 12:05:40 *attack 12:05:46 Yup. Now would be a good time if anyone has a blueprint they are active on, to mention it. 12:06:18 masco, I know you were about to open a bp 12:06:30 did you already do it actually? 12:06:31 unfortunately, we can't sort bps by date 12:06:39 which would be nice 12:07:09 https://blueprints.launchpad.net/horizon/+spec/replace-sorteddict-with-ordereddict 12:07:42 o/ 12:07:47 tsufiev, o/ 12:08:03 masco, yes! That's something fitting into django-1.8 as well 12:08:07 mrunge, it was created by my fellow developer https://blueprints.launchpad.net/horizon/+spec/service-chaining-support 12:08:20 keeping it separate makes sense as well 12:08:39 * mrunge was talking about https://blueprints.launchpad.net/horizon/+spec/replace-sorteddict-with-ordereddict 12:08:40 I think the bp masco linked makes perfect sense and is non-controversial as python 2.7 is the minimum in OpenStack 12:08:55 yes 12:09:09 Yup. Also 1.9 is supposed to be supported by the M release, so this needs to be done. 12:09:19 I don't see a reason not to approve, anyone? 12:09:27 it doesn't need much work, but i need you guys approval to start 12:09:29 no, makes sense 12:09:42 * robcresswell is getting confused by all the double negatives 12:09:44 and about prio here? 12:09:58 I put medium 12:10:04 great 12:10:23 thanks! 12:10:25 It's a blocker for 1.9 support it seems, so med or high IMO. 12:10:36 masco are you thinking Liberty or M for that? 12:10:54 i can submit in Liberty 12:11:18 just added that 12:11:44 ok, one down :) 12:11:48 thanks masco 12:11:59 thanks all 12:12:02 masco had another one: https://blueprints.launchpad.net/horizon/+spec/service-chaining-support 12:12:22 ok looking 12:12:44 this is a bit more controversial, at least, it's not that simple 12:12:45 me and mohan developing together on this, it in well progress 12:13:09 masco this does not require neutronclient changes? 12:13:14 or neutron changes? 12:13:18 it needs 12:13:29 those would be "outside dependencies" 12:13:37 that is under developing 12:13:53 not sure it may up for review 12:14:06 we can not rely on unreleased features 12:14:38 agree, 12:14:58 please update with patches if available, or at least not that changes are needed in neutron and neutronclient 12:15:05 *note 12:15:11 IMO, that bp cannot be targeted yet as it is waiting on unmerged changes to Neutron 12:15:21 yes 12:15:25 right 12:15:33 i added here to get confirmation from the cores and avoid the duplicate work 12:15:52 i will add a note as you said david-lyle 12:15:57 on the other side, it's no question, we should support that, once its implemented in neutron 12:16:09 priority? 12:16:11 yes 12:16:19 currently? 12:16:33 no, I'd like to prioritize 12:16:48 makes sifting through later easier 12:16:56 medium I'd say 12:17:10 For now I would mark as Low. It's a very specific feature it seems, and can be reevalutated later? 12:17:24 sure 12:17:43 I was thinking low to medium 12:18:08 Either is fine. 12:18:08 set to low, I agree with robcresswell, once it's concrete in neutron we can increase if necessary 12:18:14 cool 12:18:24 :) 12:18:25 should masco try to get guidance from UX folks for this? 12:18:47 I think visualization might be tricky? 12:18:53 That would probably be a good idea 12:18:59 Yeah, if you have an idea on implementation, discuss the layout with Piet & co. 12:19:15 Then when the code is merged to neutron, there is no arguing/ delay on design 12:19:27 sure, i will do 12:19:30 masco, if you need a pointer to Piet, I can make a contact 12:19:56 yes mrunge, i will ask you about this 12:20:02 thanks 12:20:06 will do that later 12:20:12 sure 12:20:49 looking at https://blueprints.launchpad.net/horizon 12:21:02 I was just looking at high prio bps 12:21:16 Is there an API for this? Interface is so useless >.< 12:21:28 launchpad blueprints I mean :) 12:21:31 there's https://blueprints.launchpad.net/horizon/+spec/api-capability-detection 12:21:50 maybe we should pull all bps into a google docs 12:21:59 just to be sure, we're looking at each just once 12:22:14 heh, we just need some better filtering mechanisms 12:23:04 that's an old bp 12:23:09 mrunge, the BP is already approved 12:23:19 yes, but nobody is working on it 12:23:43 I thought it was a candidate for removing 12:23:49 I think it is 12:24:02 until someone is ready to run with it 12:24:14 there is no detail in there 12:24:29 let's drop it then? 12:24:41 I think so 12:25:19 Scrolling through started and unprioritised ones. This could be closed, it looks like it was solved in a patch: https://blueprints.launchpad.net/horizon/+spec/remove-unittest2 12:25:20 we just remove the assignee then only other ppl will try to pick 12:25:24 so, how to remove it... 12:26:00 Mark Obsolete? 12:26:28 ho, i thought cores have permission ;) 12:26:35 yes, they have 12:26:46 but that is really obfuscated in launchpad 12:26:59 ok marked and not added 12:27:05 *note 12:27:10 hey guys, there is an blueprint related for sahara https://blueprints.launchpad.net/horizon/+spec/sahara-node-processes-tab ; implementation is already on review and well reviewed by sahara folks 12:27:22 yay two less to go 12:27:58 vgridnev: Looking 12:29:07 seems widely supported by the sahara team, looking at the patch 12:29:10 Looks like a pretty small bp. I think Liberty, Low priority. It's just a layout redesign. 12:29:50 agree with robcresswell 12:29:51 robcresswell, agreed 12:30:20 +1 12:30:46 ok updated 12:31:01 thanks vgridnev for bringing this up 12:31:02 thanks vgridnev 12:31:32 You missed milestone btw 12:31:43 that's after the fact now 12:31:50 only set when completed 12:31:52 vgridnev, that reorg does not touch tests at all? 12:31:56 Oh, okay 12:32:03 change this release 12:32:12 Gotcha 12:32:17 Is that the same with bugs? 12:32:20 we realized planning was mostly futile 12:32:30 robcresswell: for the most part 12:32:33 Or should I still be targetting the merged ones 12:32:44 bugs that need to be fixed before release should be targetted 12:32:53 Sure okay 12:33:09 Yeah I tend to target any merged ones or critical blockers 12:33:50 i have one more to review https://blueprints.launchpad.net/horizon/+spec/instance-rescue-horizon-support 12:33:55 At some point I would like to dig into a few of the angular related bps, but I think we need a wider audience for htat 12:33:58 *that 12:34:11 implementation is done already 12:34:31 david-lyle: We can do that next week. I'll get a list together. 12:34:42 awesome robcresswell 12:35:14 thanks robcresswell, I think that work is too scattershot at this point, I would like to provide better focus 12:35:36 masco: looking 12:35:56 I think there is actually a fair amount of organisation behind the scenes, but it needs to be better shown to the community. 12:36:12 coming with list BP before meeting will be good 12:36:28 yes 12:36:29 it this meeting meant as well for drawing attention to the bps which are approved and the code is complete, but not reviewed? 12:36:35 unrescue instance 12:36:52 that sounds strange 12:37:03 tsufiev, intention is to clean up abandoned blueprints, review new ones 12:37:24 tsufiev: Not really, this is mainly for bp review and cleanup 12:37:32 mrunge, since bp was approved, we can continue discussions on patch, right? btw, right now it's was tested manually. If you want to have tests there, I will add that 12:37:36 mrunge, robcresswell: ah, okay 12:37:40 tsufiev, no if the bp is approved already, it's outside the scope of this, unless you think it should be abandoned 12:37:41 nova command itself named as unrescue :) 12:37:43 we have alist of 297 blueprints. I'm confident, on all are being worked on 12:37:50 got it 12:38:27 come out from rescue is unrescue 12:38:33 masco: it's like I pulled you out of the flooded river and now I'm going to throw you back in 12:38:36 vgridnev, I agree, tests are out of scope for bp review. I still think, having tests would be beneficial 12:38:50 just sounds odd 12:39:02 :) 12:39:04 but word smithing can happen later 12:40:56 so masco in your bp, we're just setting the state, and the repair happens elsewhere? 12:41:41 yes, it is just enabling the nova rescue cmd in horizon 12:41:54 How is this different to logging in via the terminals that instances already expose btw? 12:42:23 if it went to error state you can't login robcresswell 12:42:55 so you can use other image to access the error instance and do repair 12:43:11 Interesting 12:43:23 seems reasonable to me 12:43:51 Since this is already fully supported by nova, I think medium? Also, it may be sensible to reach out to UX and see if they have any ideas on display. 12:44:15 robcresswell: I think we have a pattern here already for display 12:44:35 yeah, it's described in the bp 12:44:46 it's just a modal saying here's you pass or ssh key to access the rescued instance 12:45:06 I'm inclined to leave it a low 12:45:21 not really a mainline feature 12:45:23 yupp 12:45:27 Sure, I was drawing the comparison with the other feature in my mind 12:45:37 but Low is fine 12:45:53 updated 12:45:54 next? 12:46:04 woohoo! Is that.. 5 down? 12:46:34 2 down from 297 bp overall 12:46:37 thanks all once again :) 12:47:07 we have https://blueprints.launchpad.net/horizon/+spec/realtime-spec and https://blueprints.launchpad.net/horizon/+spec/rpc-listener 12:47:25 I meant from those that were not review/ prioritised, we've looked at several and obsoleted 2. 12:47:28 So far so good. 12:47:30 I'd be inclined to obsolete both 12:47:36 agree, robcresswell 12:47:58 yes, that still needs to happen but probably needs a new start 12:48:01 ah, and https://blueprints.launchpad.net/horizon/+spec/realtime-communication on the same topic 12:48:06 Agree with obsoletion. 12:48:13 so, obsolete all three? 12:48:26 the last one had code 12:48:45 right 12:48:55 First two, havent looked at third yet, one sec 12:49:15 I remembered code 12:49:23 there was code 12:49:42 https://review.openstack.org/#/c/40198/ 12:50:16 david-lyle, I've heard from Travis that he was going to do some study in the realtime communications in M release (for SearchLight) 12:51:03 Hmm. I still think this one (third one) could be obsolete. All of the code is 2 year old demo code with no merges. Would it be picked up again, or better to start over at this point? 12:51:05 tsufiev: yes, just wondering of leveraging some of the work tomas started would be usefule 12:51:06 but most likely nobody would be hurt if the old bp is obsoleted and a new one is created 12:52:39 I marked all 3 as obsolete 12:52:51 but bookmarked the one with code to look at again 12:52:57 +1 12:53:02 great 12:53:07 agree 12:53:27 https://blueprints.launchpad.net/horizon/+spec/user-registration 12:53:34 won't happen 12:53:49 yes 12:53:56 I had it as blocked, but I think it's best just to remove 12:54:00 Nice. If we can keep covering 10 - 15 bps a week this could be very productive. I'll bring a list for angular next week so hopefully we can be even faster. 12:54:14 although has been requested several times, david-lyle 12:54:19 but I completely agree 12:54:43 yeah I've been leaving as a "do not enter" sign 12:54:48 keystone will not allow to create 12:55:09 masco: that's why it's blocked :) 12:55:25 Oh, this does get mentioned often. But the bp itself is not useful/ relevant in its current state 12:55:39 :) 12:56:56 obsolete 12:57:00 a last one? 12:57:00 with note 12:57:09 user-registration 12:57:12 yupp 12:57:15 sure 12:57:22 +1 12:57:27 misunderstood the question 12:57:42 which one mrunge? 12:57:47 uhm, I was looking for a last bp to review/obsolete 12:57:57 e.g this one: https://blueprints.launchpad.net/horizon/+spec/scaling-trigger-ceilometer-integration 12:58:36 sounds useful; although it looks like a heat feature 12:58:58 https://blueprints.launchpad.net/horizon/+spec/dashboards-decoupling has not seen any progress 12:59:39 yupp 12:59:45 obsolete both? 13:00:00 Yes 13:00:05 Neither has code 13:00:14 Nor much detail :) 13:00:45 great 13:00:55 that was quite productive! 13:01:27 Yup. Total reduced by 8 I think, and a few more prioritised. 13:01:37 excellent 13:01:42 I will prep a list of angular bps today so it's ready for next week. 13:01:53 before I forget. Should only take a small while. 13:01:59 Thanks all 13:02:00 robcresswell: when you do, can you put it on the agenda for next week 13:02:00 awesome! 13:02:05 Will do 13:02:31 sounds good 13:02:39 thanks for your time and effort. I feel good about the progress. 13:02:45 #endmeeting