22:01:29 #startmeeting Horizon 22:01:30 Meeting started Tue Dec 3 22:01:29 2013 UTC and is due to finish in 60 minutes. The chair is david-lyle. Information about MeetBot at http://wiki.debian.org/MeetBot. 22:01:31 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 22:01:33 The meeting name has been set to 'horizon' 22:01:56 hi there 22:02:01 hello 22:02:02 hello 22:02:05 hey 22:02:05 o/ 22:02:07 hi 22:02:08 hello 22:02:11 hello 22:02:13 Hello everyone 22:02:28 So the freeze for Icehouse-1 is today 22:02:41 there are still a few items pending review 22:02:53 o/ 22:03:02 https://launchpad.net/horizon/+milestone/icehouse-1 22:03:25 What doesn't merge in the next few hours will just get moved to i-2 22:03:39 I do think a couple of the items are close though 22:04:16 hello everyone 22:04:19 howdy 22:04:21 david-lyle, darn, the inline-edit patch got some selenium error, thought it passed like 3 hours ago :-) 22:04:36 david-lyle, it is destined to fail :-) 22:04:46 Aside, from that, I spent time this week trying to cobble together a plan for icehouse as a whole 22:05:11 o/ 22:05:30 if anyone feels there's a misalignment or something I overlooked, please change it or contact me to change it 22:06:02 I may have also assigned a couple of owners that weren't, again I can correct if something is incorrect 22:06:17 yo gang, is there any clue on when the notifications stuff that have been talked so much about will land ? 22:06:18 lsmola, I saw that, no worries 22:06:51 ekarlso, I added the real-time work shadower started back in for i-2 22:07:10 landing, not sure, that was mostly a proof of concept 22:07:12 will it land this time during the cycle ? 22:07:16 sigh 22:07:51 ekarlso: it needs eyes and time using 22:08:05 is there any recent patchsets ? 22:08:18 ekarlso, yeah i guess shadower would appreciate some feedback 22:08:25 ekarlso: not that I am aware of 22:08:27 shadower: ? ;) 22:08:30 ekarlso, there weren't many reviews 22:08:42 what about the bug fixes that still didn't make it to master? 22:08:43 it would be a great addition to have :( 22:09:00 ekarlso, yeah, it's the guy assigned to that 22:09:16 casanch1: 66 bug fixes landed in i-1 22:09:22 casanch1, that can't happen 22:09:25 shadower: any recent work on it ? 22:09:33 casanch1, or misunderstood you? 22:09:34 ekarlso, or is there some newer patch? 22:09:42 lsmola: thats 22:09:43 patches are coming in at about 4.9 per day 22:09:51 what I am wondering about ;) 22:09:55 casanch1, you mean a patch landed in havana. but not in icehouse? 22:10:18 yes, I mean the bugs that are still under review 22:10:24 by coming in, I mean up for review 22:10:25 waiting for approval 22:10:51 but if it's ok to postpone them to i-2, there's no problem 22:10:52 ekarlso, no recent work, as far as I know 22:11:30 casanch1: anything that didn't land in i-1 that was targeted to i-1 automatically gets targeted to i-2 22:12:08 oh ok 22:12:51 I understand review time lag can be frustrating, I feel that too 22:13:16 The best thing to speed things up is provide reviews 22:13:36 :) 22:13:54 Yes, please help review other patches, it really helps :) 22:14:36 #topic Agenda 22:14:48 I'd like to review all futur patches about Angular 22:14:49 https://wiki.openstack.org/wiki/Meetings/Horizon 22:15:04 MaxV: +1 22:15:16 MaxV: please do 22:15:21 MaxV: you did an outstanding job with mine 22:15:22 If someone sees one, or make one ping me 22:15:26 your input is greatly valued 22:15:43 #topic Review IA proposal 22:16:17 This was a carry over from last week. I posted a mind map diagram to http://ask-openstackux.rhcloud.com/question/1/openstack-ui-information-architecture/ 22:16:29 I still need to write the prose 22:17:02 So, this is just a reminder to provide feedback if there's anything you disagree with 22:17:26 A lot of what's in there in contingent on the RBAC work which is progressing slowly :) 22:17:42 but I think bdehamer may be coming to assist on that 22:17:54 that's great news 22:18:13 just quickly connected to that 22:18:25 #topic Jasmine (MaxV) 22:18:41 ok, I'll leave it to open discussion :) 22:18:41 MaxV made a proposal on the mailing list 22:18:59 +1 to proposal 22:19:00 +1 22:19:18 you also have a blueprint, if someone wants to check https://blueprints.launchpad.net/horizon/+spec/jasmine-integration 22:19:50 I still need to have a look, but MaxV, how does it integrate with selenium? Are there any issues? 22:19:55 there also are a few POC https://review.openstack.org/#/q/status:open+project:openstack/horizon+branch:master+topic:jasmine-jshint,n,z 22:20:07 jasmine sounds very good, +1 to that 22:20:16 I made it working through selenium 22:20:41 the error messages are retrieved server side 22:20:54 and fire as a failure of the test 22:21:08 I also wrapped jshint 22:21:34 awesome! jshint is very necessary 22:21:40 as you can tell by my flagrant lack of semicolons 22:22:01 It logs the first 10 failures 22:22:20 I also wants to make a runserver-test command 22:22:35 which will brings up a testing server 22:22:48 some people are interested? 22:23:17 selenium will be used for the packaging and checking regressions 22:23:47 I'm a little on fire, you can say calm down :) 22:23:55 MaxV: the test server is for the Jasmine tests only? 22:24:20 What do you mean with testing server, how much does it do? 22:24:25 no, It just run a server with debug = true and compress false 22:24:49 oh, you mean just django, but with uncompressed assets and debugging 22:24:52 it is the entire horizon app 22:24:52 why compress = False? 22:25:00 debugging 22:25:03 javascript 22:25:15 runserver-dev must be a better name 22:25:33 well, that wouldn't test, what will be deployed in production 22:25:48 wouldn't catch issues with compressing 22:25:56 no, but when developing the JavaScript it's useful 22:26:05 no test, my bad, it is a dev server 22:26:16 ah, ok 22:26:19 gates and checkin verification would still run the way they do now 22:26:24 yes 22:26:36 it is just an entry in run_test.sh 22:26:52 or a flag to --runserver 22:27:00 +50000 to that 22:27:47 I would like to take some time to look into Jasmine and the patches you've put up for review 22:27:47 the main discussion was about Jasmine 22:27:54 sure 22:28:29 that said Jasmine sounds reasonable, I would just like to do my homework and let others check it out too 22:28:53 I'll add it as a topic for the next meeting so we can revisit it 22:29:00 david-lyle, +1 22:29:12 Any other questions for MaxV or re: Jasmine in general? 22:29:53 just as a comment, i think it'll be very nice to have a testing framework people who know angular are familiar with (since we now have angular) 22:30:33 #topic Client side validation 22:30:53 last meeting the issue of client side validation was brought up 22:31:08 yes, that was a quick fix I made 22:31:23 a patch https://bugs.launchpad.net/horizon/+bug/1125232 was proposed that used html5 22:31:26 Launchpad bug 1125232 in horizon "Object Upload is validated after object upload" [Medium,In progress] 22:31:56 there was input that angular might provide a better solution once in 22:32:04 it's pending approval https://review.openstack.org/#/c/57499/ 22:32:09 now that it's in, is that the case? 22:32:10 angular has nice inline form validation options 22:32:23 they are very very simple 22:32:40 or should we accept this approach until the angular method is better laid out 22:32:55 it's a question of time, I suppose 22:33:00 david-lyle: is there a thread about this? i can write up an example if we're interested 22:33:10 the fix is really simple, it's just needed to add one line of code in each form 22:33:18 how long will it take for angular? 22:33:46 depends on how many forms you want to set up and how complex the validations are 22:33:48 it seems easy 22:33:59 david-lyle: this html5 method is very unintrusive, so there is no issue with it for now 22:34:31 MaxV: jomara: good read: http://django-angular.readthedocs.org/en/latest/angular-form-validation.html 22:34:39 oh cool, will check that out 22:34:57 it felt clean to me, I just knew there were concerns expressed 22:35:11 casanch1, does your "fix" work with older browsers? Internet explorer? 22:35:49 mrunge: not really, client side validations should be backed up by server side validations still 22:35:50 IE 8 is as far back as we support 22:36:22 not sure about this instance though 22:36:22 it doesn't work on old versions of IE, IE10 for sure, that IE9 or 8, not sure 22:36:25 I can check 22:36:28 jtomasek, point taken. let's make it directly right, no quick fixes 22:36:44 jtomasek: +1 22:36:50 angular support IE8 and form validation is easy 22:37:08 but as mentionned jtomasek we have to check on server side too 22:37:23 it is being checked on server side 22:37:25 yes of course! 22:37:27 that is already in 22:37:42 this is for the missing client-side validation 22:38:05 casanch1: then I think it is good enough 22:38:14 casanch1: as discussed last time 22:38:26 and again, I completely agree that angular is the best final solution, but this fix will prevent wasting resources while uploading large objects 22:38:41 while we wait for angular... 22:38:49 it is merged 22:39:29 MaxV you mean angular is merged 22:39:37 ok, casanch1 and MaxV can you two use that bug to collaborate on an angular based solution 22:39:43 yes 22:39:47 oh, didn't know that Angular was already merged to master 22:39:49 not the qickhack casanch1 made 22:40:02 ok, I can rework the fix to use angular 22:40:16 ok, casanch1, thank you 22:40:34 no problem 22:40:44 MaxV, would be nice to find a way how to generate client side validation from Django validations 22:41:15 good idea 22:41:35 Yeah, the tricky part is making sure your server-side validations stay in sync with your client-side logic. 22:41:54 a linked solution would be ideal 22:42:14 jtomasek's link above talks about that 22:43:07 #topic Follow up on: Updates from I18N team 22:43:20 lsmola: this is the right solution: http://django-angular.readthedocs.org/en/latest/angular-form-validation.html 22:44:17 I think there are just two remaining reviews that need to be merged on stable, although if the deadline is today, that seems unlikely 22:44:17 jtomasek, excellent 22:44:26 thanks jtomasek 22:45:41 #topic Open Discussion 22:45:43 We may be able to backport a translation patch to the milestone-proposed branch if it's proposed within the next couple of days 22:46:13 jpich ok, that would work 22:46:14 but I'm not sure what's the status on Transifex, if the latest strings have been updated or if the i18n has had a chance to translate yet 22:46:32 I'll check with amotoki tomorrow, he may be more up to date than I am 22:47:04 ok thanks jpich 22:47:21 sure 22:47:22 * david-lyle jumps topics a little too quickly 22:47:32 :-) 22:47:46 alright, open season 22:48:43 one quick update 22:48:55 there is updated navigation proposal 22:48:57 http://ask-openstackux.rhcloud.com/question/2/openstack-dashboard-navigation-redesign/?answer=99#post-id-99 22:49:39 so whoever is interested, feel free to give any feedback, everything is welcome 22:50:17 will do 22:50:44 it's mostly reflecting summit discussions 22:50:54 jcoufal: thanks for the info. definitely interested. 22:51:22 jcoufal: thanks for putting that together. I think there is more than one bp in there 22:51:38 the main navigation and header updates is one 22:52:02 yeah, BP are already there AFAIK 22:52:08 breadcrumbs would be another 22:52:17 david-lyle: +1 22:52:39 i think there was already one about breadcrumbs 22:52:48 but it might be obsolete by this time (not sure) 22:53:05 By the way, 2 of the 4 interns who will participate in the next OPW round will be doing work related to Horizon: Sayali will be working with lsmola on sparklines, and Anna will be working with me on the instance actions 22:53:09 ok, I thought I read through -all- of them in the last week, but I may have missed one 22:53:19 The internship starts on December 10, say hello if you see them around :) 22:53:20 yaaay 22:53:25 fantastic 22:53:36 jpich, sayali has already started 22:54:09 one quick update for django-1.6 22:54:18 I have a fix for horizon for revuew 22:54:25 david-lyle: it was pretty old one, maybe that's my mistake 22:54:36 the larger issue is django_openstack_auth 22:54:38 lsmola: Cool!! Let her know about these meetings so she can join in and say hello, if it's at a reasonable time where she lives :) 22:55:19 jpich, ok, will do 22:55:26 mrunge, what's the issue? 22:55:41 jcoufal: could be a ux bp 22:55:44 david-lyle, the review is here: https://review.openstack.org/#/c/58947/ 22:56:09 david-lyle, and the issue is: somethings in authenticate changed 22:56:21 let me check for the bug I filed 22:56:47 https://bugs.launchpad.net/django-openstack-auth/+bug/1256805 22:56:49 Launchpad bug 1256805 in django-openstack-auth "django-1.6: TypeError at /auth/login/" [Undecided,New] 22:57:59 yeah, openstack_auth seems to break every django roll 22:58:20 once authenticated, one can use horizon with 1.6 (with the proposed fix for horizon) 22:58:35 so, the larger issue is openstack_auth 22:58:54 when we fixed that, we could start gating at 1.6 as well ;-) 22:59:14 mrunge, great 22:59:18 mrunge: great 23:00:27 times almost up, any last items 23:01:26 ok, thanks everyone 23:01:30 #endmeeting