12:00:55 #startmeeting Horizon 12:00:56 Meeting started Wed Mar 16 12:00:55 2016 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:57 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 12:00:59 The meeting name has been set to 'horizon' 12:01:02 o/ 12:01:17 hi 12:01:52 o/ 12:02:27 \o 12:03:06 * david-lyle tries to fight through the crowd to the front 12:03:15 o/ 12:03:29 Let's get started 12:03:42 RC-1 should happen this week 12:03:57 that's my primary concern right now 12:04:21 I've moved one bp out of RC-1 as it wasn't going to close 12:04:35 The consistency group one is also at risk 12:04:53 and the launch instance one needs a little more work 12:04:57 I don't think volumes is going to make it; still have a concern with some parts, and there's been no other core reviews on it. 12:05:12 Ports I'm just updating, that should be good to go in a couple hours 12:05:53 robcresswell: ok thanks 12:06:25 so I think one bp realistically is still in play 12:06:38 then the remaining bugs 12:07:46 there's only one or two bugs in the RC list that look worthy of slowing the RC for 12:09:18 I assume you are thinking https://bugs.launchpad.net/horizon/+bug/1543891 ? 12:09:19 Launchpad bug 1543891 in OpenStack Dashboard (Horizon) "Launch instance from Volumes Snapshot Page opens LEGACY launch instance, even if LEGACY is set to False in local_settings" [High,In progress] - Assigned to Matt Borland (palecrow) 12:09:46 I would say every one of third High bugs is nice to see fixed in M 12:09:53 *of three 12:10:09 https://bugs.launchpad.net/horizon/+bug/1553314 This one is quite important and a small fix. 12:10:10 Launchpad bug 1553314 in OpenStack Dashboard (Horizon) "Attempting to create a volume after deleting one fails" [High,In progress] - Assigned to Timur Sufiev (tsufiev-x) 12:10:28 yes those two 12:10:59 haven't looked at Angular actions after jQuery refresh recently, does it work already? 12:11:26 I mean https://bugs.launchpad.net/horizon/+bug/1514627 12:11:28 Launchpad bug 1514627 in OpenStack Dashboard (Horizon) "Angular actions not evaluated properly when dependent on row update." [Medium,In progress] - Assigned to Tyr Johanson (tyr-6) 12:11:37 No, and Tyr has only put up a partial fix and no others. 12:11:38 there is a patch merged I think 12:11:40 oh 12:11:54 what views are actually impacted? 12:12:15 just images? or objects as well 12:12:24 The angular actions one? 12:12:31 sorry, yes 12:12:39 every table with Update Metadata Actions, every table with Launch Instance NG action 12:12:44 Anything that can do launch instance (volumes, images, snapshots) 12:13:01 ok, that is a high then 12:13:02 :( 12:13:02 If oyu create one, then try and launch an instance, the button doesnt work. 12:13:15 I invented an ugly ad-hoc solution for that issue: https://review.openstack.org/#/c/289849/1 12:13:25 if we need some desperate measures :) 12:14:04 No, that involves undoing the strict-di bp from earlier 12:14:21 so what did the merged patch help, anything? 12:14:27 yep, that's why I called them desperate 12:14:48 Something to do with images, which isnt even enabled 12:14:59 super 12:15:08 so that one is not in progress 12:15:25 looks like we have an opportunity 12:16:00 moved to unassigned and confirmed 12:16:13 please take if you have an idea 12:16:21 robcresswell: but if we could find a way to inject $injector into legacy JS without undoing ng-strict-di, my solution would be less ugly 12:16:41 Sure 12:16:59 The thing is, there are 2 outstanding bugs against LI behaviour that I would consider blocking issues 12:17:01 I'm having trouble following that, I'll have to look into it tomorrow ;-) 12:17:43 so the other option is go back to legacy, which isn't ideal 12:17:46 If we can't get these sorted in the next day, we'll likely have to consider reverting LI-default. 12:17:49 Yes. 12:18:01 oh 12:18:05 let's see if we can make progress 12:18:07 :( 12:18:43 are there any other issues that people want to mention leading into the RC? 12:18:57 I'll reviews tsufievs patch for https://bugs.launchpad.net/horizon/+bug/1553314 and get Ports updated, then see what I can do about the other bugs. 12:18:58 Launchpad bug 1553314 in OpenStack Dashboard (Horizon) "Attempting to create a volume after deleting one fails" [High,In progress] - Assigned to Timur Sufiev (tsufiev-x) 12:19:07 Oops, sorry for speaking over you there :) 12:19:22 not over me, you're good 12:19:41 the other thing is please continue to test 12:19:59 if we find other issues we have RC-2 to correct if need be 12:20:37 actually this one https://bugs.launchpad.net/horizon/+bug/1528465 could be annoying as well 12:20:40 Launchpad bug 1528465 in OpenStack Dashboard (Horizon) "dashboard project network column display duplicate default public network randomly (with admin)" [Medium,In progress] - Assigned to Akihiro Motoki (amotoki) 12:20:43 I can confirm it 12:21:18 tsufiev: Patch is in the gate 12:21:22 Seems its already fixed? 12:21:41 yeah that one is in the gate 12:21:42 robcresswell, never mind then :) 12:21:59 need new status for LP: Patch is in gate 12:22:42 patch is 3/4 through gate and no failing tests 12:22:56 don't jinx it 12:22:56 would be a helpful status in LP :) 12:23:58 ok, that's the RC 12:24:08 #topic summit space 12:24:50 I was informed there may potentially be extra fishbowl or working session space at the summit 12:25:04 do we desire any additional space? 12:25:19 we were 1 fishbowl and 7 working session 12:25:24 it'd be really good if we could get a cross-project yelling space to sort out xstatic ... 12:25:40 that's a different pool generally 12:25:46 we got *so* close last week 12:25:49 ok 12:26:11 is the stable/mitaka branch cut with with the first RC or with the final one? 12:26:12 if we use one of our times I think we'll have lots of people with conflcts 12:26:19 tsufiev: first one 12:26:29 have we started collecting topics for the summit yet? 12:26:37 doug-fish: not yet 12:26:51 we should probably get the etherpad rolling 12:27:07 I can post today 12:27:19 Might be easier to sort out space needs if we knew the topics. My suspicion is that 8 is plenty, but a topic list might show otherwise 12:27:51 jfyi: I won't be in Austin this summit :/ 12:28:08 :-( 12:28:10 :( 12:28:15 :( tsufiev 12:28:49 ? 12:29:41 well then worse performance is back on the agenda 12:29:52 ;-) 12:29:59 sorry to hear that tsufiev 12:30:08 just more tight budget this time, hope it's the only time I was notified too late to apply for Travel Support program 12:30:36 that's unfortunate 12:31:09 sounds like we're ok on space 12:31:19 moving on 12:31:20 will use the free time to finish OSProfiler work, there are always side benefits :) 12:31:44 tsufiev: a week of uninterrupted work isn't bad 12:32:27 #topic PTL election 12:32:39 seems that Rob has no competitors :) 12:32:46 we're in the final 40 or so hours of nominations 12:32:57 and yes Rob is the only brave soul to step up 12:33:15 s/brave/stupid :) 12:33:21 awesome 12:33:34 robcresswell, aren't you afraid that you'll have to bear this for another 4 cycles ;)? 12:33:48 tsufiev: don't scare him off 12:34:01 haha, okay, I'm shutting up 12:34:58 #topic Horizon and not-sanitized data from other OpenStack services - how should we handle it? 12:35:10 this was the only item on the agenda 12:35:12 ah, yes, that one is mine 12:35:36 I think we should sneer a lot at the services 12:35:48 r1chardj0n3s: +1 12:35:52 we've got a bug recently with some Cinder data being displayed by Horizon - but it hadn't been filled in in Horizon, so it hadn't been sanitized 12:36:14 link? 12:36:23 * tsufiev searching 12:36:24 tsufiev: sanitized in what sense? embedded JS? 12:36:55 or something less sinister 12:37:21 oops, it's a private security bug 12:37:31 ok understood 12:37:43 let me describe it briefly 12:38:06 one can put JS injection into Cinder db using CLI 12:38:15 then it is displayed by Horizon 12:38:35 this is not a horizon security bug? 12:38:35 displayed = executed? 12:39:01 as in, it runs the javascript? 12:39:04 doug-fish, yes 12:39:12 ah, whoopsie 12:39:16 sounds like a Horizon security bug to me 12:39:25 doug-fish: I just don't see it 12:39:33 david-lyle, we've been discussing (internally) who should sanitize it 12:39:47 ah so not even upstream bug yet 12:39:57 as of now this a our downstream repo bug, I'll file it as Horizon security 12:40:06 sorry, should have done this in advance 12:40:24 tsufiev: you can assign both cinder and horizon upstream and have those teams weigh in 12:40:30 that poses a question: do we have cores who are allows to deal with secret security bugs? 12:40:32 but horizon should sanitize it 12:40:35 ok 12:40:41 r1chardj0n3s: horizon-sec 12:40:46 thx 12:40:55 is a subset of horizon-core 12:41:30 if there are cores interested please ping me or the next PTL depending on your timing of wanting in 12:41:46 process is a bit different, so the group is limited 12:42:53 will update bug's status in a couple of minutes, once I discuss it with our security guy 12:43:33 tsufiev: make sure when you file it to mark as private security 12:43:52 and don't put it out for normal review! 12:44:06 If there's time, we should also discuss the new Swift UI. 12:44:13 doug-fish: ++ 12:44:25 all reviews happen as attachments to the bug 12:44:41 once there is a review in jenkins the bug is no longer private 12:44:55 and will be set as such 12:45:24 tsufiev: we have lots of areas where we sanitize data, so we must be missing one 12:45:50 #topic swift views 12:45:53 go robcresswell 12:47:05 So the angular swift views have all merged 12:47:16 (thanks everyone) 12:47:24 :) 12:47:31 Last I checked we hadnt quite decided what to do w/ regards to moving over. 12:47:48 I'm in favour of holding the python content, but making the angular content default. 12:47:59 that was my opinion as well 12:48:01 I have a WIP with what I think is the most palatable approach https://review.openstack.org/#/c/293168/ 12:48:06 As it can't not be better. 12:48:22 I didn't get to work on it today, due to needing to look into the ironic ui and another internal thing 12:48:24 +1 12:49:17 I hope to complete it tomorrow, though I wonder if that's not pushing the deadline a bit 12:50:37 (noting that "tomorrow" is in 7 hours ;-) 12:51:19 I may have time to fix up too 12:51:57 if someone else wants to grab that patch and run with it while I'm asleep, be my guest. 12:52:48 r1chardj0n3s: ok 12:52:59 any other swifty toughts? 12:53:02 *thoughts 12:53:19 nothing from me, other than thanks for the work r1chardj0n3s 12:53:40 lots more to do! 12:53:47 and for persevering with my reviews :) 12:53:53 yes thanks r1chardj0n3s 12:53:54 :-) 12:54:17 #topic hurgleburgler is a new horizon core 12:54:31 \o/ 12:54:45 \o/ 12:54:47 well deserved 12:54:52 more reviews \o/ :) 12:55:09 *.scss -1 12:55:20 I'm sure she has that scripted somewhere. 12:55:26 so congrats to her and welcome 12:56:33 #topic Open Discussion 12:56:45 4 minutes for your deepest thoughts 12:56:59 isn't Horizon looking mighty pretty these days? 12:57:35 r1chardj0n3s, on devstack it is :) 12:57:42 *does 12:58:00 next station is Scale Performance 12:58:38 On that note, I think your nva calls patch is in the gate tsufiev 12:58:40 nova* 12:59:09 robcresswell, r1chardj0n3s: thank you for that, that's a beginning of a loong way :) 12:59:35 will keep going there 13:00:16 Thanks everyone let's wrap up the RC 13:00:21 #endmeeting