Wednesday, 2015-10-21

*** stevemar_ has joined #openstack-horizon00:00
*** stevemar_ has quit IRC00:04
*** ybathia has quit IRC00:05
*** Piet has joined #openstack-horizon00:07
*** krotscheck has quit IRC00:11
*** jpomeroy has quit IRC00:11
*** jpomeroy has joined #openstack-horizon00:11
*** rajatvig has quit IRC00:12
openstackgerritNathan Zeplowitz proposed openstack/horizon: WIP: Add Create Volume Action for the Image  https://review.openstack.org/22967700:17
*** dansiwiec has quit IRC00:17
*** nzeplowi has quit IRC00:18
*** SimonChung1 has quit IRC00:18
*** krotscheck has joined #openstack-horizon00:19
*** garthb_ has quit IRC00:22
*** yamamoto has joined #openstack-horizon00:23
*** bpokorny has joined #openstack-horizon00:28
tqtransqchen: can you get matt on irc if he is still around? need to ask him a question about his patch00:28
*** pumaranikar has joined #openstack-horizon00:30
sqchenMatt  is not online atm.00:30
sqchentqtran: ^^00:30
*** bpokorny has quit IRC00:31
*** jpomeroy has quit IRC00:32
*** bpokorny has joined #openstack-horizon00:32
tqtran:( ok00:32
*** kzaitsev_mb has quit IRC00:33
*** bpokorny has quit IRC00:33
*** bpokorny has joined #openstack-horizon00:34
sqchentqtran: I left a message to him.00:34
*** pumaranikar has quit IRC00:35
tqtrancoolios, thanks sean!00:36
*** tqtran has quit IRC00:36
*** ybathia_ has joined #openstack-horizon00:37
*** yamamoto has quit IRC00:42
*** lblanchard has joined #openstack-horizon00:43
*** kmartin has joined #openstack-horizon00:43
*** lblanchard has quit IRC00:47
*** kmartin has quit IRC00:51
*** jpomeroy has joined #openstack-horizon00:54
*** pumaranikar has joined #openstack-horizon01:04
*** bpokorny has quit IRC01:05
*** neelashah has joined #openstack-horizon01:05
*** neelashah1 has quit IRC01:06
*** ybathia_ has quit IRC01:09
openstackgerritRichard Jones proposed openstack/horizon: Adding Identity ng-roles panel  https://review.openstack.org/22282501:11
*** gyee has quit IRC01:15
openstackgerritRichard Jones proposed openstack/horizon: Adding tests for ng identity basePath constant  https://review.openstack.org/22180401:15
*** jwy has quit IRC01:17
openstackgerritMerged openstack/horizon: Improving JS unit test branch coverage  https://review.openstack.org/23598401:18
openstackgerritMerged openstack/horizon: Improving JS unit test branch coverage  https://review.openstack.org/23598401:18
*** josecastroleon has joined #openstack-horizon01:22
*** zhurong has joined #openstack-horizon01:30
*** kzaitsev_mb has joined #openstack-horizon01:30
*** itisha has quit IRC01:31
*** sqchen has quit IRC01:31
*** sqchen has joined #openstack-horizon01:38
*** yamamoto has joined #openstack-horizon01:42
*** jwy has joined #openstack-horizon01:48
*** yamamoto has quit IRC01:48
*** josecastroleon has quit IRC01:52
*** kzaitsev_mb has quit IRC01:54
*** pumaranikar has quit IRC01:55
*** SimonChung has joined #openstack-horizon01:56
*** lhcheng has quit IRC01:59
*** stevemar_ has joined #openstack-horizon02:02
*** SimonChung1 has joined #openstack-horizon02:09
*** SimonChung has quit IRC02:09
*** sqchen has quit IRC02:11
*** clu_ has quit IRC02:12
*** sqchen has joined #openstack-horizon02:14
*** pumaranikar has joined #openstack-horizon02:15
openstackgerritColeman Beasley proposed openstack/horizon: Add unit tests for load-edit directive  https://review.openstack.org/23778802:21
*** mounica has quit IRC02:26
*** jbell8 has joined #openstack-horizon02:28
*** yamamoto has joined #openstack-horizon02:30
*** pumaranikar has quit IRC02:50
*** kzaitsev_mb has joined #openstack-horizon02:50
*** saksham has quit IRC02:51
openstackgerritRajat Vig proposed openstack/horizon: Add angular flavors panel  https://review.openstack.org/20620302:53
*** KunalGan_ has joined #openstack-horizon02:54
*** kzaitsev_mb has quit IRC02:55
openstackgerritRajat Vig proposed openstack/horizon: Add Magic Search Filtering to NG Flavors  https://review.openstack.org/23375502:57
openstackgerritRajat Vig proposed openstack/horizon: Add a Delete Modal Service for deleting entities  https://review.openstack.org/23440802:57
openstackgerritRajat Vig proposed openstack/horizon: API Services for Flavors Panel  https://review.openstack.org/21489602:57
*** KunalGandhi has quit IRC02:58
*** morgan has quit IRC03:00
*** zhurong has quit IRC03:02
*** zhurong has joined #openstack-horizon03:03
*** rajatvig has joined #openstack-horizon03:03
openstackgerritRajat Vig proposed openstack/horizon: Add Multiple and Single delete actions for the flavors panel  https://review.openstack.org/23487303:04
*** garthb has joined #openstack-horizon03:04
*** lhcheng has joined #openstack-horizon03:19
*** jwy has quit IRC03:32
*** KunalGan_ has quit IRC03:35
*** pumaranikar has joined #openstack-horizon03:36
*** rajatvig has quit IRC03:40
*** jbell8 has quit IRC03:44
*** jbell8 has joined #openstack-horizon03:44
*** saksham has joined #openstack-horizon03:45
*** SimonChung has joined #openstack-horizon03:50
*** SimonChung1 has quit IRC03:54
*** SimonChung has quit IRC03:56
*** SimonChung has joined #openstack-horizon03:57
*** pumaranikar has quit IRC03:57
*** pumaranikar has joined #openstack-horizon03:57
*** SimonChung1 has joined #openstack-horizon04:02
*** SimonChung has quit IRC04:05
*** saksham_ has joined #openstack-horizon04:07
*** SimonChung has joined #openstack-horizon04:08
*** SimonChung1 has quit IRC04:08
*** bpokorny has joined #openstack-horizon04:10
*** saksham has quit IRC04:10
*** saksham_ is now known as saksham04:10
*** jbell8 has quit IRC04:15
*** jbell8 has joined #openstack-horizon04:16
*** jbell8 has quit IRC04:18
*** jbell8 has joined #openstack-horizon04:18
*** SimonChung1 has joined #openstack-horizon04:20
*** SimonChung has quit IRC04:21
*** masco has joined #openstack-horizon04:21
*** jbell8 has quit IRC04:32
*** lhcheng has quit IRC04:39
*** rajatvig has joined #openstack-horizon04:41
*** mohankumar has joined #openstack-horizon04:43
*** bpokorny has quit IRC04:52
*** kzaitsev_mb has joined #openstack-horizon04:52
*** bpokorny has joined #openstack-horizon04:53
*** pumaranikar has quit IRC04:54
*** kzaitsev_mb has quit IRC04:57
*** bpokorny has quit IRC04:58
*** lhcheng has joined #openstack-horizon05:04
*** pumaranikar has joined #openstack-horizon05:05
openstackgerritReedip proposed openstack/horizon: Add support for IP Version in Horizon FW-Rule  https://review.openstack.org/23238205:06
*** openstackgerrit has quit IRC05:16
*** openstackgerrit has joined #openstack-horizon05:16
*** morgan has joined #openstack-horizon05:16
*** stevemar_ has quit IRC05:19
*** stevemar_ has joined #openstack-horizon05:19
*** stevemar_ has quit IRC05:22
*** flwang has joined #openstack-horizon05:23
*** flwang has quit IRC05:27
*** saksham has quit IRC05:31
*** lhcheng has quit IRC05:44
*** rajatvig has quit IRC05:44
*** morgan has quit IRC05:45
*** pumaranikar has quit IRC05:52
*** garthb has quit IRC05:52
*** kzaitsev_mb has joined #openstack-horizon05:53
*** aortega has quit IRC05:53
*** flwang has joined #openstack-horizon05:57
*** flwang has quit IRC05:58
*** kzaitsev_mb has quit IRC05:58
*** morgan has joined #openstack-horizon06:01
*** sayan has joined #openstack-horizon06:03
*** josecastroleon has joined #openstack-horizon06:06
*** flwang has joined #openstack-horizon06:07
*** flwang has quit IRC06:11
*** flwang has joined #openstack-horizon06:11
*** rajatvig has joined #openstack-horizon06:12
*** evgenyf has joined #openstack-horizon06:13
openstackgerritReedip proposed openstack/horizon: Add support for IP Version in Horizon FW-Rule  https://review.openstack.org/23238206:13
*** rajatvig_ has joined #openstack-horizon06:14
*** rajatvig has quit IRC06:17
*** flwang has quit IRC06:21
*** flwang has joined #openstack-horizon06:24
openstackgerritMerged openstack/horizon: Add HTTPS to image location help text  https://review.openstack.org/22349906:30
*** pumaranikar has joined #openstack-horizon06:31
*** e0ne has joined #openstack-horizon06:32
*** lsmola_ has quit IRC06:33
*** vgridnev has joined #openstack-horizon06:34
*** pumaranikar has quit IRC06:35
*** jamielennox is now known as jamielennox|away06:37
*** mohankumar has quit IRC06:41
*** mohankumar has joined #openstack-horizon06:41
*** e0ne has quit IRC06:42
*** wolverineav has joined #openstack-horizon06:46
*** josecastroleon has quit IRC06:46
*** lsmola_ has joined #openstack-horizon06:46
*** e0ne has joined #openstack-horizon06:47
*** dansiwiec has joined #openstack-horizon06:53
*** pumaranikar has joined #openstack-horizon06:53
*** rdopiera has joined #openstack-horizon06:54
*** kzaitsev_mb has joined #openstack-horizon06:55
*** wolverineav has quit IRC06:58
*** pumaranikar has quit IRC06:58
*** wolverineav has joined #openstack-horizon06:59
*** josecastroleon has joined #openstack-horizon07:00
*** kzaitsev_mb has quit IRC07:01
*** peristeri has joined #openstack-horizon07:02
*** linkedinyou has quit IRC07:06
*** flwang has quit IRC07:06
*** sqchen has quit IRC07:07
*** bluex has joined #openstack-horizon07:07
openstackgerritDan Siwiec proposed openstack/horizon: Add API services for system information  https://review.openstack.org/20519607:08
*** bluex has quit IRC07:08
*** bluex has joined #openstack-horizon07:08
*** wolverineav has quit IRC07:10
*** rajatvig has joined #openstack-horizon07:13
*** vgridnev has quit IRC07:15
*** wolverineav has joined #openstack-horizon07:16
*** wolverineav has quit IRC07:17
*** wolverineav has joined #openstack-horizon07:18
*** wolverineav has quit IRC07:18
*** rajatvig has quit IRC07:19
*** kzaitsev_mb has joined #openstack-horizon07:21
*** bluex has quit IRC07:22
*** sayan has quit IRC07:24
*** vgridnev has joined #openstack-horizon07:24
*** e0ne has quit IRC07:27
*** jongchoi has joined #openstack-horizon07:30
*** wolverineav has joined #openstack-horizon07:31
*** e0ne has joined #openstack-horizon07:31
openstackgerritDan Siwiec proposed openstack/horizon: Add API services for system information  https://review.openstack.org/20519607:36
*** sayan has joined #openstack-horizon07:37
openstackgerritDan Siwiec proposed openstack/horizon: Add angular system information panel  https://review.openstack.org/20473107:38
*** julim has quit IRC07:39
*** wolverineav has quit IRC07:39
*** dansiwiec has quit IRC07:41
*** sayan has quit IRC07:43
*** sayan has joined #openstack-horizon07:44
*** aortega has joined #openstack-horizon07:46
*** julim has joined #openstack-horizon07:46
*** mohankumar has quit IRC07:48
*** sayan has quit IRC07:48
*** sayan has joined #openstack-horizon07:49
*** mpavlase has joined #openstack-horizon07:53
*** jsheeren has joined #openstack-horizon07:54
*** lhcheng has joined #openstack-horizon07:55
openstackgerritReedip proposed openstack/horizon: Add support for IP Version in Horizon FW-Rule  https://review.openstack.org/23238207:56
*** lhcheng has quit IRC07:59
*** zhurong has quit IRC08:01
*** kzaitsev_mb has quit IRC08:03
*** bluex has joined #openstack-horizon08:07
*** bluex has quit IRC08:07
*** bluex has joined #openstack-horizon08:08
*** kzaitsev_mb has joined #openstack-horizon08:09
*** josecastroleon has quit IRC08:11
*** kzaitsev_mb has quit IRC08:13
*** jsheeren has quit IRC08:14
openstackgerritMerged openstack/horizon: Change ports range in validators and test function  https://review.openstack.org/11650808:18
robcresswellGooood morning everyone08:20
*** josecastroleon has joined #openstack-horizon08:20
*** rajatvig_ has quit IRC08:23
*** jgrassler has joined #openstack-horizon08:23
*** vgridnev has quit IRC08:26
*** Ala has joined #openstack-horizon08:30
*** e0ne has quit IRC08:37
jgrasslerGood morning.08:47
*** sayan has quit IRC08:47
jgrasslerI've got a bit of a problem with the projects dropdown in Horizon on Kilo: https://gist.github.com/anonymous/ab8a68ca9e62411e38b808:47
*** tnovacik has joined #openstack-horizon08:48
jgrasslerWe traced the root cause back to a bad endpoint URL being used08:48
*** jongchoi has quit IRC08:48
jgrasslerHorizon tries to use /tokens when it should use /v2.0/tokens.08:48
jgrasslers/endpoint/identity endpoint/08:48
jgrasslerAs you can see from the local_settings.py in the Gist, OPENSTACK_KEYSTONE_URL is configured to be http://10.0.80.11:5000/v2.0 - as far as I understand it Horizon should then use that URL.08:50
jgrasslerInstead it uses http://10.0.80.11:5000 (unversioned, as you'd get it from `openstack endpoint list`)08:51
*** sayan has joined #openstack-horizon08:51
jgrasslerI verified this is indeed the problem by hacking up backend.py to append the API version if it is missing from the URL (ll. 64, also in the Gist)08:52
jgrasslerWhile this did indeed fix the problem, I'd prefer not to have a wrong endpoint in there in the first place - did I miss the correct way to go about that, or should I file a bug report?08:53
*** tnovacik has quit IRC08:54
robcresswelljgrassler: It sounds like a bug to me. At least file it for now, so those with better keystone knowledge can comment08:57
robcresswelljgrassler: Which version is this one?08:57
jgrasslerrobcresswell: Package version?08:58
tsufievjgrassler: is it the same issue as in https://bugs.launchpad.net/horizon/+bug/1413851 ?08:58
openstackLaunchpad bug 1413851 in OpenStack Dashboard (Horizon) "Horizon Identity Projects Panel doesn't list projects when configured to Keystone v3 RBAC" [Undecided,In progress] - Assigned to Dan Nguyen (daniel-a-nguyen)08:58
robcresswelljgrassler: Version of Horizon, as in Juno/ Kilo/ Liberty etc.08:58
robcresswellI don't know if packages follow those strictly, which might make that more difficult to answer08:59
jgrasslerAh, Kilo (the Ubuntu packages with one or two backported bugfixes of our own)08:59
jgrasslerDefinitely not the latest stable/kilo08:59
jgrasslerKeystone v3...doesn't sound like it's exactly the same problem (we configured Horizon to use v2)09:00
jgrasslerLet me have a quick look...09:00
jgrasslerNo, that looks like a different problem09:01
*** openstackgerrit has quit IRC09:01
jgrasslerThat being said, we got the same 404 when switching Horizon to Keystone v3 experimentally09:01
*** JeanBriceCombebi has joined #openstack-horizon09:01
*** openstackgerrit has joined #openstack-horizon09:01
*** amotoki has joined #openstack-horizon09:01
jgrassler(obviously, since it was still using the same unversioned Keystone endpoint URL)09:02
jgrasslerAnyway, I guess I'll file a bug report then. Thanks!09:03
*** akanksha_ has joined #openstack-horizon09:07
tsufievjgrassler, thank you for filing the bug :)09:14
tsufievrobcresswell, seems we really need to adopt a bug day practice09:15
robcresswelltsufiev: Yes :(09:15
tsufievthere are a lot of bugs with undefined priority09:15
jgrasslertsufiev: No worries, happy to :-)09:15
robcresswelltsufiev: Blueprints also crawling back up again09:16
tsufievrobcresswell, yeah, have to constantly run just to keep once achieved results :/09:17
*** e0ne has joined #openstack-horizon09:17
robcresswelltsufiev: fwiw, both neutron and nova also have 1000+ bugs09:20
tsufievokay, I'm relieved :)09:20
*** ^Gal^ has joined #openstack-horizon09:20
*** aix has joined #openstack-horizon09:21
robcresswelltsufiev: I think our bp list is nearly-manageable now anyway. I think 150 would be quite a good target, but often people will do 3/4 small bps per cycle so difficult to gauge an actual "target"09:22
*** vgridnev has joined #openstack-horizon09:22
robcresswelltsufiev: But bugs , we have 874 that are New/ Confirmed/ Triaged/ In Progress09:22
*** sayan has quit IRC09:27
*** mohankumar has joined #openstack-horizon09:31
*** openstackgerrit has quit IRC09:31
*** openstackgerrit has joined #openstack-horizon09:31
*** ccesario_ has quit IRC09:33
*** kzaitsev_mb has joined #openstack-horizon09:35
*** exploreshaifali has joined #openstack-horizon09:43
tsufievrobcresswell, still seems pretty big amount to me (even keeping in mind Neutron/Nova). Looks like a good topic to discuss on today meeting09:49
*** JeanBriceCombebi has quit IRC09:50
*** JeanBriceCombebi has joined #openstack-horizon09:50
robcresswelltsufiev: The bp amount seems big, or bugs?09:53
tsufievrobcresswell, bugs09:54
robcresswelltsufiev: Oh, I agree, its huge and totally useless09:54
robcresswellNo point having a tool that is just a dumping point09:54
*** JeanBriceCombebi has quit IRC10:05
*** vgridnev has quit IRC10:07
*** vgridnev has joined #openstack-horizon10:07
*** lhcheng has joined #openstack-horizon10:07
*** zhenguo has quit IRC10:10
*** JeanBriceCombebi has joined #openstack-horizon10:12
*** lhcheng has quit IRC10:12
*** ygbo has joined #openstack-horizon10:13
*** JeanBriceCombebi has quit IRC10:21
*** cpallares has quit IRC10:27
*** vkmc has quit IRC10:29
r1chardj0n3sby "today'10:32
r1chardj0n3ss meeting" is that one in about 2.5 hours, or one in about 10 hours?10:32
r1chardj0n3sI've lost track :/10:32
robcresswellr1chardj0n3s: I'd assume the one in 1.5 hours, -drivers10:32
r1chardj0n3syep10:32
r1chardj0n3sI shall endeavor to stay away for that one10:33
*** kzaitsev_mb has quit IRC10:35
openstackgerritVitaly Gridnev proposed openstack/horizon: [Sahara] Get urls of datasources safer  https://review.openstack.org/23797610:35
*** stevemar_ has joined #openstack-horizon10:35
r1chardj0n3ser10:35
r1chardj0n3sstay awake10:35
*** vkmc has joined #openstack-horizon10:35
r1chardj0n3snot away10:35
*** cpallares has joined #openstack-horizon10:36
*** tnovacik has joined #openstack-horizon10:36
*** stevemar_ has quit IRC10:38
*** kzaitsev_mb has joined #openstack-horizon10:40
*** ccesario has joined #openstack-horizon10:43
*** kzaitsev_mb has quit IRC10:45
tsufievr1chardj0n3s, your fingers tell of your real intentions ;)10:49
*** tnovacik has quit IRC10:49
r1chardj0n3sdamn it!!!!! :)10:49
*** rajatvig has joined #openstack-horizon10:49
r1chardj0n3sI am fading fast though, so I think I'm gonna have to read the meeting transcript again, or I won't wake up for the morning meeting10:50
r1chardj0n3s(I'm currently also packing up my house for moving the day after I get back from Tokyo summit, because that's how I roll)10:50
tsufievr1chardj0n3s, anyways, I don't think something important is going to be discussed in ~hour. All important stuff awaits you in Tokyo10:51
r1chardj0n3swell, all the arguing about forking anyway :-D10:51
tsufievlike 'to angular or not to angular?'10:51
r1chardj0n3syup10:52
*** rajatvig has quit IRC10:54
*** kzaitsev_mb has joined #openstack-horizon10:57
robcresswellThat topic confused me a little11:04
robcresswellI didn't think "to angular" was a discussion we were still having11:04
robcresswellmore just "how best to angular"11:04
r1chardj0n3srobcresswell: yep11:04
r1chardj0n3sI read the transcript of the openstack-ironic discussion11:04
r1chardj0n3sso yeah, how best to11:05
robcresswellkfox1111: How are you handling JS testing/linting in your app catalog ui?11:06
*** JeanBriceCombebi has joined #openstack-horizon11:06
*** tnovacik has joined #openstack-horizon11:19
*** doug-fish has joined #openstack-horizon11:20
*** ^Gal^ has quit IRC11:20
*** ^Gal^__ has joined #openstack-horizon11:20
*** ^Gal^__ has quit IRC11:20
*** ^Gal^ has joined #openstack-horizon11:22
*** aix has quit IRC11:24
robcresswelltsufiev: Have to step away, should be back for meeting, give or take 5 mins11:24
tsufievrobcresswell, ok11:25
jgrasslertsufiev: Bug report for the project list issue submitted (https://bugs.launchpad.net/horizon/+bug/1508421 ). Took a little longer since I tried to be comprehensive :-)11:33
openstackLaunchpad bug 1508421 in OpenStack Dashboard (Horizon) "Projects dropdown fails due to incomplete Keystone endpoint URL" [Undecided,New]11:33
*** JeanBriceCombebi has quit IRC11:34
*** JeanBriceCombebi has joined #openstack-horizon11:37
*** e0ne has quit IRC11:37
*** exploreshaifali has quit IRC11:38
*** sayan has joined #openstack-horizon11:39
*** sayan has quit IRC11:41
*** Guest63084 has joined #openstack-horizon11:43
*** kzaitsev_mb has quit IRC11:47
tsufievjgrassler, wow, it looks like a new bug template for Horizon bugs :)11:52
*** e0ne has joined #openstack-horizon11:54
*** kzaitsev_mb has joined #openstack-horizon11:56
*** aix has joined #openstack-horizon12:01
*** Ephur has joined #openstack-horizon12:01
*** manas has joined #openstack-horizon12:03
*** Ephur has quit IRC12:06
*** ducttape_ has joined #openstack-horizon12:09
*** Guest63084 has quit IRC12:10
*** tnovacik has quit IRC12:12
*** sayali has quit IRC12:12
*** ^Gal^__ has joined #openstack-horizon12:16
*** ^Gal^ is now known as Guest2358012:16
*** ^Gal^__ is now known as ^Gal^12:16
*** openstackgerrit has quit IRC12:16
*** openstackgerrit has joined #openstack-horizon12:16
*** ^Gal^ has quit IRC12:17
*** Guest23580 has quit IRC12:17
*** ^Gal^ has joined #openstack-horizon12:18
*** kzaitsev_mb has quit IRC12:20
*** vgridnev has quit IRC12:20
*** yamamoto has quit IRC12:21
*** itisha has joined #openstack-horizon12:22
*** tnovacik has joined #openstack-horizon12:27
*** vgridnev has joined #openstack-horizon12:29
*** lapalm has joined #openstack-horizon12:33
*** ducttape_ has quit IRC12:34
*** JeanBriceCombebi has quit IRC12:46
*** JeanBriceCombebi has joined #openstack-horizon12:48
*** sayali has joined #openstack-horizon12:54
openstackgerritRob Cresswell proposed openstack/horizon: Don't hide Karma coverage reports  https://review.openstack.org/23804312:55
*** mounica has joined #openstack-horizon12:55
*** stevemar_ has joined #openstack-horizon13:03
*** petertr7_away is now known as petertr713:04
openstackgerritVitaly Gridnev proposed openstack/horizon: [Sahara] Get urls of datasources safer  https://review.openstack.org/23797613:05
*** matt-borland has joined #openstack-horizon13:06
*** stevemar_ has quit IRC13:07
tsufievrobcresswell, I like the name Karma developers chose. It leads to meaningful commit titles, like 'Don't hide Karma'13:07
tsufievI presume it looks very funny to Buddhists :)13:08
*** stevemar_ has joined #openstack-horizon13:09
*** kzaitsev_mb has joined #openstack-horizon13:09
robcresswelltsufiev: haha, I hadnt thought of that13:11
*** stevemar_ has quit IRC13:12
*** yamamoto has joined #openstack-horizon13:14
*** manas has quit IRC13:14
*** ducttape_ has joined #openstack-horizon13:16
*** zhurong has joined #openstack-horizon13:17
*** stevemar_ has joined #openstack-horizon13:17
*** ducttape_ has quit IRC13:18
robcresswellCould one of the cores take a look at https://review.openstack.org/#/c/212152/ ? Been around a while13:20
*** kiran-r has joined #openstack-horizon13:21
*** masco has quit IRC13:21
*** jpomeroy has quit IRC13:22
matt-borlandSo robcresswell, I'm both livid and depressed about what I've heard about the feature branch  :(13:25
matt-borlandch13:26
matt-borlandI'm wondering if you can help me understand why I shouldn't be livid and depressed. :)13:26
robcresswellmatt-borland: Go ahead13:26
robcresswellAs in, explain the anger/sadness13:26
matt-borlandso, what I'd heard is that we're just putting Searchlight, Identity, and Image panels in it13:26
matt-borlandand that everything else must wait for merging until that completes13:27
matt-borlandwhich I understood to be around midcycle.13:27
matt-borlandThat means that the 5 panels that have gone through UX review, etc.13:27
robcresswellThat seems to be the current plan. The panels haven't actually been decided, yet, it depends which the developers would like to prioritise13:27
matt-borlandwill have to sit an freeze until sometime in February13:27
robcresswellAs in, who is willing to put resources in13:27
matt-borlandwell13:28
matt-borlandwe've put quite a lot of resources into the panel work13:28
matt-borlandcross-pollinating best practices,13:28
matt-borlandsoliticing reviews13:28
matt-borlandestablishing patterns13:28
matt-borlandthat don't exist in those areas13:28
matt-borlandand now we're told13:28
matt-borlandwe have to wait 3 months before anyone should look at them for merging13:28
matt-borlandI'm just saying, that was not what I wanted out of a feature branch13:29
matt-borlandyou don't need to answer/respond, I'm just registering *extreme* lack of support13:30
*** kiran-r has quit IRC13:30
matt-borlandotherwise things are well :)13:30
robcresswellI was typing, but wondering how to phrase13:30
robcresswellWhat would you expect from a feature branch?13:30
robcresswellor feature branching in general.13:30
matt-borlandwell, first I'd prefer that we look at the work that has passed UX reviews, gone through design reviews, etc., to be reviewed for merging.13:31
matt-borlandThat is my first preference, regardless of FBing.13:31
matt-borlandbut it seems there is a Silence.13:31
*** lblanchard has joined #openstack-horizon13:31
robcresswellMy suggestion for merging btw was around M-2, which is mid- January.13:31
*** rdopiera is now known as rdopiera_afk13:31
matt-borlandsure13:31
matt-borlandso basically, Feb would be when people might start looking at merging panels that have been already heavily reviewed *and* have passed UX reviews.13:32
matt-borlandThat is a problem.13:32
robcresswellIn terms of which panels, frankly that can be pretty much argued amongst HP, since that appears to be most of the angular contributors13:32
matt-borlandWe've got lots of good eyes and reviews on many of those panels.13:32
matt-borlandI think the point of a FB13:33
matt-borlandis to address the gripe that people want to see total functionality.13:33
matt-borlandand13:33
matt-borlandto address the ability to better cross-pollinate between patches, by allowing them to merge to something so we don't have long chains of patches.13:34
robcresswellI agree on both13:34
matt-borlandthose are the advantages of the FB :)13:34
matt-borlandbut if we have a large bulk of work13:34
matt-borlandthat has many good patterns in it13:34
matt-borlandthose should be on the feature branch, too13:34
robcresswellBut coordinating that amongst 3 panels, over the next 3 months, given the summit and christmas, seems like enough work, IMO.13:34
matt-borlandin other words13:34
robcresswellonce it starts getting to 5,6 panels, I'm not sure its manageable, and then we end up in the exact same situation.13:35
matt-borlandthat's not how I see it.13:35
matt-borlandfirst off, I fall back on the philosophy of:13:35
*** masco has joined #openstack-horizon13:35
matt-borlandwe've done the process that was originally agreed to,13:35
matt-borlandso let's merge that work.13:35
matt-borlandI worked with you, other cores, and David to figure out the best way to structure patches13:35
matt-borlandI did that13:35
matt-borlandwe designed a new UX review process that we said everything had to go through13:36
matt-borlandI did that.13:36
robcresswell(I hasten to add, this is far from *my decision*, and I'm not trying to tell people what to do. It's just a suggestion to make both sides of the argument agree on a path)13:36
matt-borlandI know :)13:36
matt-borlandI appreciate you talking with me13:36
matt-borlandthis is not about aiming blame13:36
matt-borlandthis is about planning13:36
robcresswellRight13:36
matt-borlandif we need to deep-freeze those 5 panels because we're waiting for some patterns13:37
matt-borlandI object13:37
matt-borlandbut I'd like to not delude myself either13:37
robcresswellSo the panels I've seen that appear to have good progress so far are Flavors, Users, Projects, Images, and possibly Searchlight13:37
matt-borlandSystem Info and Defaults.13:37
robcresswellAnd also the system info13:37
matt-borlandyeah, and this is all Disabled by default.13:38
robcresswellSo that puts us right now at 713:38
matt-borlandthe worst we're doing is adding some unused modules into the Angular space13:38
matt-borlandso I'd prefer we just merge the "easy" stuff13:38
matt-borlandI don't see conflicting patterns here.13:38
matt-borlandI wouldn't either mind creating...heh heh...ANOTHER feature branch just for those 4-5 panels13:39
doug-fishFWIW unused modules add to the effort of translating Horizon.13:39
matt-borlanddoug-fish, they front-load the work.13:40
matt-borlandand most of the verbiage is redundant13:40
matt-borlandin the areas we're talking about13:40
matt-borlandhere's the thing13:40
doug-fishgood to know - though the way it's captured is going to require a retranslate anyway13:40
matt-borlandI've been given every excuse in the book for Cores not to look at reviewing these panels13:40
matt-borlandwait for X, wait for Y, not sure Angular is fast enough, memory overhead, patterns not established...a long list13:41
matt-borlandsome is good to consider, some is not13:41
matt-borlandbut it's time to p*ss or get off the pot for work that has gone the gauntlet13:41
matt-borlandand has been reviewed positively by the community13:41
matt-borlandbut then faces what can only be called Silence of the Cores.13:42
matt-borlandthis is my opinion as a dev :)13:42
*** Daisy has joined #openstack-horizon13:43
doug-fishsure fair enough - IIRC you'll be at the summit, right?13:43
matt-borlandyep :)13:43
doug-fishwell get to hash it out further there, I'm sure13:43
matt-borlandI appreciate everyone helping talk through this13:43
robcresswellHow would you feel about increasing the size of the FB to incorporate more panels?13:43
matt-borlandand all the support we've got13:43
doug-fishmy point on translation was really not to expect to merge late like we did for the launch instance rewrite13:43
matt-borlandright13:44
robcresswellMy fear is: my branch, add 5 panels, go into refactor hell again, and the community refuses the 3000 lines that come out of it at the end.13:44
matt-borlandso, this is a great time to merge the stuff that has been positively reviewed13:44
matt-borlandof course, I agree robcresswell13:44
robcresswellmy? Not sure what I meant by that13:44
doug-fishFWIW I had to put some code into a feature branch related to keystoneclient. I found it to be a big pain, and ultimately that feature branch never merged13:44
doug-fishso I have a bit of a bad taste for that approach13:45
matt-borlandI think that each FB should have a specific set of goals13:45
matt-borlanddoug-fish, exactly13:45
matt-borlandI'd prefer not to feature branch at all13:45
matt-borlandwe have committed to an in-place upgrade13:45
matt-borland(for better or worse)13:45
robcresswellSo the compromise there is err on side of caution, let companies decide which panels they want to push for and develop those 3 panels and merge fast, refactor/ rewrite, merge early Jan, spend next 2 months adding other panels that fit patterns.13:45
matt-borlandso that means13:45
matt-borlandwe have to incrementally add things into master13:45
matt-borlandand iterate13:46
matt-borlandwe spent most of Liberty establishing infrastructure13:46
matt-borlandthat was painful but necessary13:46
matt-borlandthe amount of refactors we've had to apply since13:46
doug-fishthe in-place upgrade is a real concern to me. It means we have to all agree to make the change together, and once we decide all of our consumer have to change13:46
matt-borlandhas been minimal13:46
matt-borlanddoug-fish13:46
*** petertr7 is now known as petertr7_away13:46
matt-borlandI agree13:46
doug-fishincluding a total rewrite of any plugins/customizaitons they have developed13:46
robcresswellI disagree that it was *necessary*. Thoe style should've been decided much, much earlier.13:46
robcresswellHalf of my agnular reviews have just been renaming files or moving code13:47
matt-borlandrobcresswell, it was necessary given where we were13:47
robcresswellGiven that, yes.13:47
matt-borlandyes13:47
*** zhurong has quit IRC13:47
matt-borlandwe are always living based on circumstance13:47
*** jpomeroy has joined #openstack-horizon13:47
*** neelashah1 has joined #openstack-horizon13:47
matt-borlandwe cannot operate in a vacuum, always have a context13:47
matt-borlandsimilarly,13:47
matt-borlandwe seem to want to have an in-place upgrade rather than a fork/Horizon 2.013:48
robcresswelldoug-fish: Realistically, we'd run both panels for a while, but declare a deprecation I assume. Easiest thing would be to have a "new panel" dict in the settings, so that people can toggle it locally with ease.13:48
matt-borlandI don't know if that's good13:48
doug-fishmaybe we need to define this term "in-place"13:48
matt-borlandsure13:48
matt-borlandI just meant,13:48
*** erhudy has joined #openstack-horizon13:48
matt-borlandHorizon is composed of a mix of implementations as it transitions13:49
doug-fishrobcresswell: why not just make them completely independent panels?13:49
matt-borlandThey are separate panels13:49
matt-borlandseparate names,13:49
robcresswelldoug-fish: Well, they are separate panels. Even now they're declared as ng-<panel>13:49
matt-borlandloaded independently13:49
doug-fishI'm happier already  :-)13:49
robcresswellIndeed.13:49
matt-borlandso customers can choose them13:49
matt-borland:)13:49
robcresswellYep13:50
matt-borlandso, good point doug-fish, not 'in-place' in complete replacement/swap-out13:50
matt-borlandbut rather13:50
*** ducttape_ has joined #openstack-horizon13:50
matt-borlandhaving 1 system with 2 technologies being implemented.13:50
doug-fishI had overlooked that fact, and had the (apparently) incorrect assumption that we were completely replacing existing panels13:50
matt-borlandnope :)13:50
matt-borlandok, cool cool13:50
robcresswelldoug-fish: Replacing in the sense that the navbar would direct you to ng-networks etc13:50
robcresswelldoug-fish: But the panel code would exist for both13:51
doug-fishsure, but that something a deployer can choose right?13:51
matt-borlandyep13:51
robcresswellAs I said, it would be nice to have a settings dict that just toggled new/old, rather than having to manually disabled and enable each one, but it makes little difference I suppose.13:51
doug-fishso if they've invested in a python based customzation to the Images panel they can use that panel13:51
doug-fishpresumably for a deprecation period13:51
matt-borlandyep13:51
*** kzaitsev_mb has quit IRC13:51
matt-borlandexactly exactly13:51
robcresswelldoug-fish: Precisely!13:51
matt-borlandso that's the path we're on13:52
*** lapalm has quit IRC13:52
matt-borlandand that's why all the panels that we've developed are (for merging) disabled by default13:52
matt-borlandbut even given that13:52
*** tnovacik has quit IRC13:52
matt-borlandpeople are seemingly afraid to look at merging them.13:52
doug-fishFWIW I'm not afraid, but time constrained13:52
matt-borlandI understand :)13:52
matt-borlandso that's a factor in determining how to approach this rewrite,13:53
matt-borlandfrom david's perspective13:53
matt-borland"cores don't have enough time"13:53
matt-borlandbut,13:53
*** sayali has quit IRC13:53
matt-borlandhaving said that13:53
doug-fishin fact, I should be doing other work than having this chat now13:53
doug-fishbut I think it's important13:53
matt-borlanddoug-fish: np, thanks for tuning in :)13:53
*** sayali has joined #openstack-horizon13:53
matt-borlandI wanted to chat through this openly because I don't want there to be surprises at the summit13:54
matt-borlandI also really need to plan where to put effort13:54
*** stevemar_ has quit IRC13:54
matt-borlandand if we're freezing existing "completed" panel work13:54
matt-borlandI need to know13:54
matt-borland(and simultaneously object)13:54
doug-fishsure - so what's the case against merging these in master w/o the feature branch?13:54
*** petertr7_away is now known as petertr713:54
matt-borlandand also suggest alternatives13:54
matt-borlanddoug-fish, that is *exactly* what I want to know, too13:54
doug-fishI haven't only given them passing reviews and had the impression they weren't mature yet, mostly based on the discussion13:55
matt-borlandwell, here's the thing.13:55
robcresswelldoug-fish: The current resistance seems to be: We merged a load of angular code at first, then spent another X months re-reviewing the code13:55
doug-fishbut given the approach and matt-borland's assurance of the maturing I think they need a closer look13:55
matt-borlandthey've gone through UX review, unlike most other features13:55
matt-borlandsome of them have been sitting awaiting merge review for over a month13:55
matt-borlandsome parts of them, like the APIs, are mostly cookie-cutter and also have value across patches13:56
*** peristeri has quit IRC13:56
matt-borland*panels13:56
robcresswellmatt-borland: It's worth pointing out that isn't unique, Horizon has a notably low review count across the board, comparing the numbers to other projects.13:56
doug-fishrobcresswell: it sounds like the resistance is that we don't want to refactor?13:56
matt-borlandThat's one fear13:56
robcresswelldoug-fish: Yes. Personally, I don't like reviewing the same code 3 times with different file paths.13:56
matt-borlandbut here's my counter argument13:56
doug-fishIMO Horizon is harder to review - the judgement of what is "right" can't necessarily be covered in a unit/integration test13:57
robcresswellAnd having to double check it doesnt break anything in the process13:57
doug-fish(though that's part of it)13:57
robcresswellThere's a minimum time involved in any reviewing regardless, just in pulling down code and running horizon, even for seemingly trivial changes.13:57
matt-borlandright13:57
doug-fishPrior to liberty the panels were untranslatable - that was a big deal to me, but not the case any more13:57
matt-borlandand so we have to constantly re-review our patches, like the APIs, most of which have settled long ago13:58
matt-borlandpatches which, by our agreement early on, were structured explicitly so we could approve underlying patches and not have immense patches13:58
*** kzaitsev_mb has joined #openstack-horizon13:58
*** mohankumar has quit IRC13:58
matt-borlandnow that agreement is all but ignored,13:59
*** sayali has quit IRC13:59
matt-borlandand generally I get abstract arguments for why no one should merge patches13:59
matt-borlandwithout even looking at them.13:59
matt-borlandI've been following these patches13:59
doug-fishno one should merge patches without looking at them - that's not your point is it?13:59
matt-borlandI've been watching for refactor hell...and it hasn't happened!14:00
matt-borlandbecause we're using standard features.14:00
matt-borlandthat we already established in the last releases.14:00
robcresswellRight, the linting and style guiding has finally settled down14:00
matt-borlandno, the problem is that cores *aren't* generally looking at them14:00
robcresswellAlthough I still see the occasional "this should be called blah.js" patch14:00
matt-borlandok,14:01
matt-borlandso, what is your point with that specifically?14:01
*** vgridnev has quit IRC14:01
matt-borlandI'm talking about panel patches.14:01
doug-fishrobcresswell: the fact it's occasional says that it's pretty mature14:01
robcresswellRight, as I said, its settled down now14:01
matt-borlandyeah14:01
robcresswellWhich is good.14:01
matt-borlandand what I'd heard from cores before the end of Liberty was roughly:14:01
*** sayali has joined #openstack-horizon14:02
matt-borlandafter we have worked on some patches, we'll be able to just turn around patch reviews14:02
robcresswellThe other part which hasn't settled, and I spoke to travis about this and he completely agreed, and this is from his angular-focused core perspective too14:02
matt-borlandso I went and researched actual use cases14:02
robcresswellIs that daily churn is a nightmare14:02
matt-borlandand followed the process14:02
*** vgridnev has joined #openstack-horizon14:02
doug-fishrobcresswell: which daily churn?14:02
*** e0ne has quit IRC14:03
matt-borlandyou see, we haven't had the churn in the panels (except for suggestions), because there's minimal overlap with other patches14:03
robcresswellSo, generally speaking, so that I can approve more patches, I will A) wait until a couple of non-cores have reviewed, and B) wait until the patch has 2/3 days without changing. I don't want to review a patch, only to have to re-review it every day of the week.14:03
matt-borlandrobcresswell, absolutely14:04
*** JeanBriceCombebi has quit IRC14:04
matt-borlandso here's what we've done to shelter all the cores14:04
robcresswellThe sys info API patch, for example, has had 9 opatch sets since yesterday14:04
*** JeanBriceCombebi has joined #openstack-horizon14:05
matt-borlandOK, so how do you want to talk about this,14:05
matt-borlandbecause basiaclly what I do is this"14:05
matt-borlandthis:14:05
matt-borlandAfter the UX review, and after our internal review, I ask for others to review the panel patches, right now, only 3 total patches (not panels) at a time.14:05
matt-borlandso we are not clobbering the community14:05
matt-borlandof course we get feedback on those14:06
robcresswellsure14:06
matt-borlandlike Sysinfo API, frankly that was stable for a couple of weeks14:06
matt-borlanduntil yesterday, when Richard pointed out a mocking issue he'd missed before14:06
matt-borlandit's frankly not the most important thing in the world, but we address it14:06
matt-borlandthen, at what point do we ask cores for reviews?14:07
matt-borland3 days?14:07
*** tristanC has quit IRC14:07
matt-borland2 weeks?14:07
matt-borlandit was a mock.14:07
matt-borlandthe feature itself has been virtually unchanged otherwise14:07
matt-borlandand what of the other APIs that have been sitting ready for review?14:08
matt-borlanddo we decide not to review those because someone else saw that there was movement in another patch?14:08
matt-borlandthis is the kind of abstract argument I don't like14:08
*** jongchoi has joined #openstack-horizon14:08
robcresswellI can only explain the reasons I personally would avoid reviewing something14:09
robcresswellI can't speak for the other cores14:09
matt-borlandI know :)14:09
matt-borlandI am voicing my perspective, not condemning :)14:09
doug-fishmatt-borland: here's why I haven't spent much time yet on the angular patches: Early in the release we had a lot of churn with lint/format type issues and it seemed enough people reviewing that ...14:09
doug-fishlater in the release, I had the impression there was refactoring going on to help sort out key patterns ...14:09
doug-fishthat's where I thought we were14:10
openstackgerritPaul Karikh proposed openstack/horizon: [WIP] Skip UpdatePrijectMembers if user_list failed  https://review.openstack.org/23761514:10
matt-borlandI like specifics14:10
robcresswellSo the API for flavors: 7 patch sets in the last week. API for quotas 10 in the last week. Note, this is often multiple patch sets without comments or revies, where the author is just spontaneously updating it.14:10
doug-fishI've been lightly reviewing and my key concerns have been addressed (translatability and extensibility)14:10
robcresswellIf I go and +2 because I think its good enough based on my knowledge, and then the author updates it 2 hours later, I need to pull the patch again, check their patchset changes etc.14:11
matt-borlandrobcresswell, several things:14:11
matt-borland1) I ask people to put descriptions of their changes in, if they don't do that, bad on them.14:11
doug-fishrobcresswell: rereviewing patches you already know is a good way to boost your review numbers!14:12
matt-borland2) there's a fine balance between static reviews because they're not reviewed, and a well-reviewed patch that has gone dormant14:12
robcresswelldoug-fish: :) gotta get those numbers.14:12
doug-fishyou know you can see the changes between patch sets, right?14:12
robcresswelldoug-fish: Ha, yes of course14:12
doug-fishrobcresswell: yeah, I should work on that.14:12
matt-borlandeither way it's nice to have a description14:12
matt-borlandthe other thing,14:13
matt-borlandagain, is that when I've asked people to officially ask for merge review,14:13
robcresswelldoug-fish: My point was it still takes time, that I could spend looking into other patches14:13
*** masco has quit IRC14:13
*** Daisy has quit IRC14:13
*** pumaranikar has joined #openstack-horizon14:13
matt-borlandso, again, how long of 'quiet time' is needed?14:13
robcresswellcouple days is normally enough for me.14:14
*** Daisy has joined #openstack-horizon14:14
matt-borlandOK14:14
matt-borlandI can work with that.14:14
robcresswellIts not that I have a specific "you shall not pass" rule, more that if someone says "please review this cores" and its had 5 patch sets in the last hour I'll likely leave it for the time being.14:14
matt-borlandI can absolutely find patches that have otherwise been stable.  And frankly, those APIs, if you look at what is changing, they are largely stable and have been.14:15
*** sayali has quit IRC14:15
matt-borlandbut, like any time you ask for a review,14:15
matt-borlandsomeone will find a nit14:15
matt-borlandso you feel compelled to address it14:15
matt-borlandrobcresswell, of course14:16
matt-borlandabout the 5 PS in last hour14:16
matt-borlandI have never asked a core to review such a patch.14:16
doug-fishI'm glad we've had this pre-summit chat - frankly, I think if any of these patches are mature (and it sounds like they are) creating the feature branch will only slow progress14:16
robcresswellWell, that depends on the scope of the nit really. If you've got 500 lines of code and someone points out you've forgotten a '.' in your commit message, ignore it.14:16
doug-fishand require that we get *all* of the panels ready before *any* can merge14:16
*** sayali has joined #openstack-horizon14:17
matt-borlandYou gotta remember, in this case, it's complex, because we're getting feedback that "we're new the community and should be good members,"14:17
matt-borlandso those that are new are more ready to please by making the change14:17
doug-fishironically, chatting about this is taking away time that I could actually do reviews later, so I'm going to step away14:17
robcresswelldoug-fish: I don't really mind how we address it. I like the angular work, but I'm not expert in it, so reviews take time.14:17
matt-borlanddoug-fish, thanks for your participation here14:17
robcresswelldoug-fish: Thanks Doug14:17
matt-borlandI really appreciate the opportunity to go thru this14:17
doug-fishsure no worries. Looking forward to talking further next week.14:17
matt-borlandsee you there!14:17
matt-borlandrobcresswell, sorry for venting publicly, but hadda do it14:18
*** Daisy has quit IRC14:18
matt-borlandI'd rather people know my perspective than I just whisper it14:18
*** rajatvig has joined #openstack-horizon14:19
matt-borlandthis is a very complex situation, I appreciate everyone's work14:19
matt-borlandI don't want it to come off as otherwise :) I just need to express my interests14:19
*** Daisy has joined #openstack-horizon14:22
*** rajatvig has quit IRC14:24
robcresswellmatt-borland: No problem14:24
robcresswellJust making coffee, back in a mo14:25
*** jongchoi has quit IRC14:28
*** mhorban_ has joined #openstack-horizon14:31
matt-borlandrobcresswell, no problem :)14:31
*** lastovich has joined #openstack-horizon14:32
*** aortega has quit IRC14:32
matt-borlandwe have amazing, intelligent, professional people here in Horizon...we have lots of opportunity...let's capitalize on that!  Let none of my concerns belittle that attitude.14:33
*** vkmc is now known as vkmc-afk14:34
*** JeanBriceCombebi has quit IRC14:35
robcresswelldavid-lyle: ping14:37
*** lastovich has quit IRC14:38
*** david_cu has joined #openstack-horizon14:39
ducttape_matt-borland  - I don't seem to fit in any of your categories :P14:39
*** david_cu_ has joined #openstack-horizon14:39
matt-borlandha!14:39
matt-borland:)14:39
matt-borlandit always pays to be unique14:40
*** david_cu_ has quit IRC14:41
*** david_cu has quit IRC14:41
*** lastovich has joined #openstack-horizon14:41
robcresswellducttape_: What's your stance on this?14:42
ducttape_on the reviews in general?14:42
ducttape_like what gets looked at?  when do  things get merged?14:43
robcresswellducttape_: On the angular work and where it should live14:43
robcresswelland the various frustrations shared by people :p14:43
ducttape_I think we have a lot of energy / people working on ng stuff, and it is not making the progress that *everyone* would like14:44
ducttape_but just merging more code in, is not really always the best fix14:44
ducttape_I think part of the reason of the slow uptake, is that what we have is not really ready for broad adoption14:45
ducttape_horizon's success is that you have mostly python library developers who want stuff to show up on a web page14:45
ducttape_if our angular layer would do more work to make it easier for these people, things would be going smoother I think14:46
matt-borlandthanks, good points14:46
*** yamamoto has quit IRC14:46
ducttape_like auto generate most of the angular content, with options to customize angular at certain extension points14:47
ducttape_right now, if we have 10 patches with tons of separate js files - that is starting to sign up for a long term support issue14:48
ducttape_so getting the right pattern at the begining is important, and I'm not sure we have that yet14:48
ducttape_these are my ideas, I'm sure others might see it differently14:48
robcresswellmatt-borland: I had one question. What was the reason for choosing to do ng-include and ng-controller for the index tables, rather than a directive?14:49
matt-borlandmany of the patches don't use ng-include.  The original thought was that that would be a way to make it extensible.14:50
ducttape_tqtran and I had an approach where you could convert all table views to angular, and it would take about 3 lines of code changes per table.   While that approach was not supported, I think we need something like that14:50
*** Daisy has quit IRC14:50
matt-borlandif you want to see extensibility, jpomeroy has a good patch for that14:50
*** Daisy has joined #openstack-horizon14:51
matt-borlandducttape_, I kind of agree with that patch, with several ifs.  I've heard strong arguments against that for other reasons, but I feel it could be a good bridge.14:51
robcresswellducttape_: Which patch is that?14:51
matt-borlandit however doesn't easily deal with variance *if* we want to deviate the UX much.14:51
*** Piet has quit IRC14:51
ducttape_https://review.openstack.org/#/c/94706/14:52
ducttape_it's as old as dirt14:52
*** lapalm has joined #openstack-horizon14:52
*** petertr7 is now known as petertr7_away14:53
ducttape_matt-borland - this is where I would say that adding customization hooks should be done over time.  trying to think of every possible extension point is not an effective use of time14:53
*** Drago has joined #openstack-horizon14:53
matt-borlandexactly!  I agree 100% with that sentiment :)14:54
robcresswellmatt-borland: I don't suppose you have an example of a non-ng-include panel? Sorry14:54
ducttape_and also, sometimes you build extension points that nobody cares about14:54
*** Drago has quit IRC14:54
*** Drago has joined #openstack-horizon14:54
matt-borlandhttps://review.openstack.org/#/c/206203/40/openstack_dashboard/dashboards/admin/static/dashboard/admin/flavors/table/flavors-table.html14:54
matt-borlandif that's what you mean robcresswell14:55
matt-borlandif you use that approach14:55
matt-borlandit's quite transparent how it all works14:55
matt-borlandand if there is a need for extensibility14:55
*** stevemar_ has joined #openstack-horizon14:55
robcresswellAh, I meant why was this pattern chosen; <ng-include src="'{{ STATIC_URL }}dashboard/admin/flavors/table/flavors-table.html'"></ng-include>14:55
robcresswelland then ng-controller in the table.html14:55
*** stevemar_ has quit IRC14:56
*** Daisy has quit IRC14:56
robcresswellvs just having a "flavors table" directive that links the two14:56
*** e0ne has joined #openstack-horizon14:56
matt-borlandwell, let's back up 1 second14:56
matt-borlandthere is a transition here14:56
ducttape_matt-borland  - see that patch as an example14:56
ducttape_you could auto generate that14:56
matt-borlandyep, I've looked at it, exactly14:56
matt-borlandI don't disagree with having that approach on top of other more explicit markup too14:57
matt-borlandin fact14:57
*** lapalm has quit IRC14:57
matt-borlandI'd prefer that we have the 'quickie table' approach so that implementers of either core or extended features can just whip them together14:57
robcresswellIt seems to me that an awful lot of the 'table.html' files is absolutely identical. In fact, bar the column names, you can pretty much s/flavor/user/g and get the same table.14:57
matt-borlandand not have to fuss with markup in every case14:57
matt-borlandrobcresswell, talk with Thai about that approach.14:58
*** sayali has quit IRC14:58
matt-borlandI am not invested in that approach14:58
matt-borlandfrankly14:58
robcresswellI was just wondering if we could end up with a thing like <hz-index-table columns="[]">14:58
ducttape_I'd like it so openstack project XYZ comes in with a new library, can do a bit of python, and needs little in the way of doing much else (unless they are motivated to)14:58
robcresswellSure thing14:58
matt-borlandfor the short term we're in what I call a 17-page-app14:58
matt-borlandthat entire routing can change14:58
matt-borlandand when it does,14:58
matt-borlandthe top level of includes goes away14:58
*** lapalm has joined #openstack-horizon14:58
*** stevemar_ has joined #openstack-horizon14:59
*** lapalm has joined #openstack-horizon14:59
matt-borlandrobcresswell, I've tried working that over with Thai as well; I wrote an hz-table patch a long time ago14:59
robcresswellI know with the magnum UI basic tables, they're all displaying ID/ Name/ Status/ Actions, and they're all carbon copies except for the model names14:59
robcresswellWith the python we just declare a list of names and a few other things, like truncate for example, and the tabel does some magic. This feels way more manual to me.15:00
robcresswelltable*15:00
matt-borlandrobcresswell, there are some issues15:00
robcresswellSure15:00
matt-borlandwhat we see in UX15:00
matt-borlandis that we want much more nested elements15:00
matt-borlandwhich you can of course make programmable thru such a configuration15:00
matt-borlandbut15:00
*** sayali has joined #openstack-horizon15:00
matt-borlandin markup, you just do it.15:00
matt-borlandadd in the tags/elements you want.15:01
matt-borlandunlimited15:01
matt-borlandwant a donut chart?15:01
matt-borlandawesome.15:01
matt-borlandbut if it's more imperative15:01
matt-borlanddriven by python15:01
matt-borlandyou have to either make a million little hooks15:01
matt-borlandor something similar15:01
matt-borlandso,15:01
*** alejandrito has joined #openstack-horizon15:01
matt-borlandthe hz-table IS good, and I'd support having one.15:01
matt-borlandthat produces a "good enough" table15:02
robcresswellI'm not saying we should drive it with python, just that we should be more DRY15:02
*** alejandrito has quit IRC15:02
matt-borlandsure15:02
matt-borlandthat is always a concern15:02
robcresswellThe first 20 lines of https://review.openstack.org/#/c/206203/40/openstack_dashboard/dashboards/admin/static/dashboard/admin/flavors/table/flavors-table.html15:02
*** alejandrito has joined #openstack-horizon15:02
matt-borlanddoesn't mean we should hold up EVERY panel patch15:02
robcresswellare the same in every single table so far15:02
matt-borlandwhich again, is my issue :)15:02
robcresswellSure15:02
matt-borlandif you don't like the pattern, argue it15:02
matt-borlandwith the people that are arguing for it.15:02
* robcresswell spreads -1s throughout the land15:03
matt-borlandthis is something Thai was very keen on15:03
matt-borlandand he explained his reasons15:03
robcresswellYeah, I brought it up in a hangout ages ago15:03
*** JeanBriceCombebi has joined #openstack-horizon15:03
matt-borland:)15:03
robcresswellIt seems really anti-angular to me15:03
matt-borlandso,15:03
matt-borlandthere are some questions.15:03
robcresswellMy studies of it have always turned up with "directive all the things"15:04
matt-borlandrobcresswell, I think that is generally true.15:04
matt-borlandI had written my original patches that way15:04
matt-borlandI was then told that it's just adding indirection15:04
matt-borlandwhich in a way, is true15:05
matt-borlandboth from a design perspective15:05
matt-borlandand from a programming perspective15:05
matt-borlandand it's kind of true.15:05
ducttape_my concern with good-enough is that no one ever comes back to fix things.  this is the same as agile - stuff needs to be done-done15:05
matt-borlandhaving said that15:05
*** jprovazn has joined #openstack-horizon15:06
matt-borlandin terms of patterns, nothing is perfect.15:06
matt-borlandesp in angular, where the patterns are vague and sometimes ambiguous15:06
matt-borlandso at some point you either p*ss or get off the pot.  I explicitly worked on a number of panels so we'd get better perspective.15:07
matt-borlandthey have been available to look at for quite some time15:07
matt-borlandthese are not new arguments.  Somehow we agree to disagree, or we prioritize our disagreements and commit to resolving them.15:08
matt-borlandNOT...wait for the code to just work the way we want it to work.15:08
*** SimonChung1 has quit IRC15:08
*** JeanBriceCombebi has quit IRC15:09
*** yamamoto has joined #openstack-horizon15:09
*** Daisy has joined #openstack-horizon15:09
robcresswellSure15:10
robcresswellI'll ask thai to explain15:10
*** JeanBriceCombebi has joined #openstack-horizon15:11
matt-borlandAnd the point of "never fixing things"15:11
matt-borlandI call shenannigans on that, if only because all of Liberty was refactoring work done in Kilo, to make it more right.15:12
matt-borlandin fact, that is one approach I like.  One release for new content, one release for refactors.15:12
matt-borlandallows consumers to better plan their adoption15:13
robcresswellmatt-borland: As a generally OpenStack trend people have a habit of writing code then disappearing15:13
matt-borlandit's not surprising :)15:13
robcresswellmatt-borland: So letting people write crap code on the promise of "we'll fix it later" is a bad move.15:13
*** stevema__ has joined #openstack-horizon15:13
matt-borlandbut I believe, if there's a process problem and a goal...fix the process.15:13
matt-borlanddon't abandon the goal15:13
matt-borlanddon't wallow in misery about the failed process15:14
matt-borlandI think we established there's a difference between "crap code" and "code that some people have disagreements about"15:14
matt-borlandso let's just be clear, no one is suggesting merging crap code.15:15
robcresswellSure, I'm speaking pretty generally about openstack here15:15
matt-borlandI like specifics :)15:15
robcresswellNot "how I think we should handle tables"15:15
matt-borlandbut that was the extension of that argument, wasn't it?  I just want to explain yet another "reason we shouldn't merge X"15:15
tsufievrobcresswell, doug-fish: gentlemen, could I ask for your opinion in this review: https://review.openstack.org/#/c/115092/ ?15:16
tsufiev(it's not my patch)15:16
robcresswellmatt-borland: Which argument? I was just curious why we decided to do ng-include and not directives, thats all15:16
*** stevemar_ has quit IRC15:16
tsufievit's about the integration tests coverage for admin vs. non-admin roles15:17
matt-borlandI dunno!15:17
matt-borland:)15:17
robcresswelllol, okay15:17
tsufievmatt-borland, sorry to interrupt your lovely discussion15:17
matt-borlandtsufiev, no problem!  :)15:17
*** Ephur has joined #openstack-horizon15:17
robcresswelltsufiev: I need to get on with some actual code for now :)15:18
*** emilyw has joined #openstack-horizon15:19
matt-borlandrobcresswell, thanks again for being a sounding-board. :)15:20
tsufievrobcresswell, oh, my evil plan was to get your judgement w/o reading the code )15:20
tsufievs/evil/cunning/15:20
robcresswelltsufiev: heh, assume good and just +2?15:22
robcresswellI must have a bad reputation :(15:22
tsufievrobcresswell, nope, it's not about +2. It's just about -1 vs. -2 :)15:22
*** kmartin has joined #openstack-horizon15:23
*** ZZelle has joined #openstack-horizon15:23
tsufievrobcresswell, I feel obliged to tnovacik, because he built most of stuff I'm taking credits for fixing now15:24
*** Ala has quit IRC15:25
robcresswelltsufiev: Ahh, I see15:27
*** garthb has joined #openstack-horizon15:28
*** yamamoto_ has joined #openstack-horizon15:29
*** aortega has joined #openstack-horizon15:29
*** yamamoto has quit IRC15:32
*** lhcheng has joined #openstack-horizon15:33
*** bpokorny has joined #openstack-horizon15:37
*** bluex has quit IRC15:38
*** bpokorny has quit IRC15:39
*** bpokorny has joined #openstack-horizon15:39
*** vgridnev has quit IRC15:46
openstackgerritRob Cresswell proposed openstack/horizon: Don't hide Karma coverage reports  https://review.openstack.org/23804315:46
*** evgenyf has quit IRC15:46
*** petertr7_away is now known as petertr715:50
*** JeanBriceCombebi has quit IRC15:52
*** Daisy has quit IRC15:53
*** neelashah1 has quit IRC15:53
*** Daisy has joined #openstack-horizon15:53
*** JeanBriceCombebi has joined #openstack-horizon15:54
*** ygbo has quit IRC15:56
*** Daisy has quit IRC15:58
*** vgridnev has joined #openstack-horizon15:59
*** stevema__ has quit IRC16:00
*** vkmc-afk is now known as vkmc16:01
*** vkmc has quit IRC16:01
*** vkmc has joined #openstack-horizon16:01
*** neelashah1 has joined #openstack-horizon16:01
*** JeanBriceCombebi has quit IRC16:02
*** rajatvig has joined #openstack-horizon16:02
*** exploreshaifali has joined #openstack-horizon16:05
openstackgerritRajat Vig proposed openstack/horizon: API for getting and updating quotas  https://review.openstack.org/20712616:05
openstackgerritRajat Vig proposed openstack/horizon: Add angular defaults panel subflows for admin  https://review.openstack.org/23498616:05
openstackgerritRajat Vig proposed openstack/horizon: Add angular defaults panel for admin  https://review.openstack.org/20529616:05
*** emilyw has quit IRC16:05
pkarikhHi folks! Does anyone know, is there any way to apply patches from gerrit into devstack installation? For example, if I want to check gerrit patch for horizon or keystone from the devstack.16:06
matt-borlandpkarikh, I've done that before16:08
*** john5223 is now known as zz_john522316:09
matt-borlandusually just shuffle around where Horizon has been installed to to be a working dir,16:09
*** sqchen has joined #openstack-horizon16:09
matt-borlandand copy/replace local_settings.py accordingly16:09
pkarikhmatt-borland: hm.. So, there is no some service repo in the devstack, so I can just git pull someones changes?16:10
*** Piet has joined #openstack-horizon16:11
matt-borlandno, I swap out the horizon dir with a working directory16:11
pkarikhmatt-borland: ok, thanks!16:12
david-lylepkarikh: yes you can do it all from there16:12
david-lylenot the best way, but expedient at times16:13
david-lyleit's just a clone of the repo16:13
*** petertr7 is now known as petertr7_away16:13
*** neelashah1 has quit IRC16:13
*** e0ne has quit IRC16:16
*** e0ne has joined #openstack-horizon16:17
*** rdopiera_afk is now known as rdopiera16:19
openstackgerritRajat Vig proposed openstack/horizon: Add angular flavors panel  https://review.openstack.org/20620316:23
openstackgerritRajat Vig proposed openstack/horizon: Add button to Create new Flavor  https://review.openstack.org/23487516:23
openstackgerritRajat Vig proposed openstack/horizon: Add angular flavors panel Edit Subflow  https://review.openstack.org/23487416:23
openstackgerritRajat Vig proposed openstack/horizon: Add Multiple and Single delete actions for the flavors panel  https://review.openstack.org/23487316:23
openstackgerritRajat Vig proposed openstack/horizon: API Services for Flavors Panel  https://review.openstack.org/21489616:23
openstackgerritRajat Vig proposed openstack/horizon: Add a Delete Modal Service for deleting entities  https://review.openstack.org/23440816:23
david-lylepkarikh: the exact command is git review -d <change-id> just for completeness16:24
*** petertr7_away is now known as petertr716:27
*** logan2 has quit IRC16:27
*** clu_ has joined #openstack-horizon16:28
*** lastovich has quit IRC16:29
*** SimonChung has joined #openstack-horizon16:29
*** stevemar_ has joined #openstack-horizon16:36
*** exploreshaifali has quit IRC16:38
*** doug-fish has quit IRC16:40
*** doug-fish has joined #openstack-horizon16:41
*** stevemar_ has quit IRC16:41
*** KunalGandhi has joined #openstack-horizon16:42
*** amotoki has quit IRC16:42
*** mpavlase has quit IRC16:43
*** gyee has joined #openstack-horizon16:45
*** doug-fish has quit IRC16:45
*** logan2 has joined #openstack-horizon16:47
*** jobewan has joined #openstack-horizon16:48
*** petertr7 is now known as petertr7_away16:52
*** lapalm has quit IRC16:55
*** lapalm has joined #openstack-horizon16:55
*** doug-fish has joined #openstack-horizon16:56
*** rhagarty has joined #openstack-horizon16:57
*** zigo_ has quit IRC16:59
*** lapalm has quit IRC17:00
*** zigo has joined #openstack-horizon17:00
openstackgerritRob Cresswell proposed openstack/horizon: Don't hide Karma coverage reports  https://review.openstack.org/23804317:04
*** vgridnev has quit IRC17:11
openstackgerritRajat Vig proposed openstack/horizon: Add API services for system information  https://review.openstack.org/20519617:11
openstackgerritRajat Vig proposed openstack/horizon: Add angular system information panel  https://review.openstack.org/20473117:11
*** ColemanB has joined #openstack-horizon17:15
*** sqchen has quit IRC17:15
*** yamamoto_ has quit IRC17:17
*** cbii has joined #openstack-horizon17:19
*** ColemanB has quit IRC17:19
*** pauloewerton has joined #openstack-horizon17:25
*** yamamoto has joined #openstack-horizon17:25
*** kfox1111 has quit IRC17:25
*** kfox1111 has joined #openstack-horizon17:26
*** kzaitsev_mb has quit IRC17:28
*** Daisy has joined #openstack-horizon17:30
*** stevemar_ has joined #openstack-horizon17:32
openstackgerritDiana Whitten proposed openstack/horizon: Horizon Dropdown now inherits from Bootstrap Theme  https://review.openstack.org/23273517:34
*** sqchen has joined #openstack-horizon17:34
*** cbii has quit IRC17:35
*** ducttape_ has quit IRC17:35
*** lapalm has joined #openstack-horizon17:40
*** lapalm has quit IRC17:40
*** exploreshaifali has joined #openstack-horizon17:40
*** lapalm has joined #openstack-horizon17:40
*** hurgleburgler1 has quit IRC17:41
openstackgerritTony Dunbar proposed openstack/horizon: Made instance state values in the CSV summary translatable  https://review.openstack.org/23818617:41
*** rhagarty_ has joined #openstack-horizon17:43
*** amotoki has joined #openstack-horizon17:43
*** rhagarty has quit IRC17:46
openstackgerritTimur Sufiev proposed openstack/horizon: Avoid multiple calls to Glance happening in api.nova.server_list()  https://review.openstack.org/23818917:46
*** amotoki has quit IRC17:48
*** brad[] has quit IRC17:48
*** brad[] has joined #openstack-horizon17:48
*** e0ne has quit IRC17:49
*** vgridnev has joined #openstack-horizon17:50
*** exploreshaifali has quit IRC17:53
*** yamamoto has quit IRC17:55
*** ybathia has joined #openstack-horizon17:55
*** harlowja has quit IRC17:57
*** jpomeroy has quit IRC17:57
*** saksham has joined #openstack-horizon17:57
*** rdopiera has quit IRC17:58
*** stevemar_ has quit IRC17:58
*** jpomeroy has joined #openstack-horizon18:02
*** harlowja has joined #openstack-horizon18:02
*** Daisy has quit IRC18:02
*** lapalm_ has joined #openstack-horizon18:04
*** lsmola_ has quit IRC18:07
*** lapalm__ has joined #openstack-horizon18:07
*** neelashah1 has joined #openstack-horizon18:07
*** lapalm has quit IRC18:08
*** lapalm_ has quit IRC18:08
*** vgridnev has quit IRC18:10
*** tqtran has joined #openstack-horizon18:10
*** ducttape_ has joined #openstack-horizon18:11
*** aix has quit IRC18:12
*** vgridnev has joined #openstack-horizon18:12
*** vgridnev has quit IRC18:14
*** petertr7_away is now known as petertr718:16
*** Piet has quit IRC18:17
*** julim_ has joined #openstack-horizon18:19
*** julim has quit IRC18:19
*** jwy has joined #openstack-horizon18:21
*** vgridnev has joined #openstack-horizon18:25
*** mounica has quit IRC18:26
*** lhcheng has quit IRC18:28
*** lhcheng has joined #openstack-horizon18:29
*** lhcheng has quit IRC18:30
*** exploreshaifali has joined #openstack-horizon18:31
openstackgerritTimur Sufiev proposed openstack/horizon: Reduce Overview panels default date range to 1 day  https://review.openstack.org/23820418:31
*** zul has quit IRC18:32
*** Daisy has joined #openstack-horizon18:33
*** zul has joined #openstack-horizon18:33
openstackgerritLucas Palm proposed openstack/horizon: Truncate instance creation pop-up message  https://review.openstack.org/23173718:35
openstackgerritAkanksha Srivastava proposed openstack/horizon: Fixes the readability of UUID for Network Instance  https://review.openstack.org/22825218:44
*** amotoki has joined #openstack-horizon18:44
*** amotoki has quit IRC18:49
*** jbell8 has joined #openstack-horizon18:49
*** yamamoto has joined #openstack-horizon18:55
*** petertr7 is now known as petertr7_away18:56
*** Daisy has quit IRC18:59
*** yamamoto has quit IRC19:00
*** stevemar_ has joined #openstack-horizon19:07
*** lhcheng has joined #openstack-horizon19:07
*** lhcheng_ has joined #openstack-horizon19:08
*** sqchen has quit IRC19:09
*** SimonChung has quit IRC19:10
*** SimonChung1 has joined #openstack-horizon19:10
*** lhcheng has quit IRC19:12
*** lapalm__ has quit IRC19:12
*** lapalm has joined #openstack-horizon19:13
*** linkedinyou has joined #openstack-horizon19:15
*** stevemar_ has quit IRC19:15
*** rajatvig has quit IRC19:15
*** stevemar_ has joined #openstack-horizon19:16
*** rajatvig has joined #openstack-horizon19:16
*** sqchen has joined #openstack-horizon19:16
*** lapalm has quit IRC19:17
*** stevemar_ has quit IRC19:20
openstackgerritTony Dunbar proposed openstack/horizon: Made instance state values in the CSV summary translatable  https://review.openstack.org/23818619:21
*** lapalm has joined #openstack-horizon19:22
*** cbader has quit IRC19:23
lapalmrobcresswell: Are you around? ;P19:24
*** zacdev has joined #openstack-horizon19:24
*** vgridnev has quit IRC19:25
*** fifieldt_ has joined #openstack-horizon19:26
*** Daisy has joined #openstack-horizon19:27
*** vgridnev has joined #openstack-horizon19:28
*** fifieldt has quit IRC19:29
*** vgridnev has quit IRC19:29
*** vgridnev has joined #openstack-horizon19:30
robcresswelllapalm: How can I help?19:30
*** mounica has joined #openstack-horizon19:31
openstackgerritAkanksha Srivastava proposed openstack/horizon: Fixes Broken Error Message on Subnet Creation  https://review.openstack.org/23821919:31
lapalmrobcresswell: Oh hey.  Just pestering you about my review that is marked WIP still awaiting your response. ;)19:31
robcresswell?? I replied ot that in IRC last time you asked19:32
lapalmhmmmmm19:32
lapalmReally?19:32
*** jprovazn has quit IRC19:32
lapalmIDK about that lol19:32
lapalmI remember asking you and I think you said that you would take a look.19:33
robcresswellNo I definitely replied in the channel19:34
robcresswellLink? I'll look again19:34
lapalmhah alright19:35
tqtranHi robcresswell , can i pester you to look at https://review.openstack.org/#/c/235497/ and https://review.openstack.org/#/c/222297/ again?19:35
lapalmhttps://review.openstack.org/#/c/226579/19:36
lapalmtqtran: wait your turn ;P19:36
tqtran:D drive-by request haha19:36
robcresswellGoodbye evening :(19:40
*** vgridnev has quit IRC19:40
*** hurgleburgler has joined #openstack-horizon19:42
lapalmrobcresswell: Come back tomorrow with a fresh mind and look at them.19:43
lapalmTake the night off19:43
*** Piet has joined #openstack-horizon19:43
*** pauloewerton has quit IRC19:45
tqtranlapalm: you're too nice! sleep is for the weak!19:45
*** petertr7_away is now known as petertr719:45
robcresswellnah, will look this evening.19:45
tqtranhaha jk :P19:45
*** SimonChung1 has quit IRC19:46
*** emilyw has joined #openstack-horizon19:46
robcresswellholy shit19:47
robcresswellwill people please stop grammar sniping19:47
robcresswell3 -1s on a commit message19:47
matt-borlandgeez19:47
robcresswellAlso why does the inline editor wrap :(19:48
robcresswelldoesnt*19:48
matt-borlandthat is kinda sad!19:48
openstackgerritRob Cresswell proposed openstack/horizon: Don't hide Karma coverage reports  https://review.openstack.org/23804319:48
*** rhagarty_ has quit IRC19:48
*** harlowja has quit IRC19:50
*** pumaranikar has quit IRC19:50
*** cbader has joined #openstack-horizon19:50
*** pauloewerton has joined #openstack-horizon19:53
bpokornyrobcresswell: In defense of my -1 on that one, it was about the docs as opposed to the line wrapping.19:54
*** vgridnev has joined #openstack-horizon19:54
bpokornySorry if that seemed too strong, though.19:54
*** Daisy has quit IRC19:54
robcresswellbpokorny: No, yours was reasonable, and line comment was reasonable too.19:55
robcresswellbpokorny: It's more when people start giving extra -1s for "agree needs moar grammar"19:55
robcresswellI'm ranting.19:55
robcresswellLong day :)19:55
hurgleburglerrobcresswell I agree with you19:56
*** vgridnev has quit IRC19:56
matt-borlandrobcresswell, sorry if I got things off to a bad start.  We're all in this together.19:56
robcresswellmatt-borland: Not at all, discussion earlier was good.19:56
hurgleburglerDocumentation grammar is important, but nit picking in commit message is well … nit picking.19:57
r1chardj0n3smorning19:57
matt-borlandI should align my schedule to yours now r1chardj0n3s19:57
hurgleburglerr1chardj0n3s Morning!19:57
openstackgerritRob Cresswell proposed openstack/horizon: Don't hide Karma coverage reports  https://review.openstack.org/23804319:58
r1chardj0n3shurgleburgler: if people dont get there grammer right in commit messages it bugs me19:58
r1chardj0n3sohai matt-borland19:58
robcresswellr1chardj0n3s: lol19:58
matt-borland:)19:58
r1chardj0n3srobcresswell: alot19:59
r1chardj0n3s:)19:59
robcresswellMeeting time20:00
robcresswell#openstack-meeting-3 folks20:00
*** vgridnev has joined #openstack-horizon20:01
*** vgridnev has quit IRC20:03
*** vgridnev has joined #openstack-horizon20:06
*** openstackstatus has joined #openstack-horizon20:07
*** ChanServ sets mode: +v openstackstatus20:07
*** pauloewerton has quit IRC20:08
*** pauloewerton has joined #openstack-horizon20:09
*** mpavlase has joined #openstack-horizon20:11
*** dansiwiec has joined #openstack-horizon20:19
*** woodm19791 has joined #openstack-horizon20:21
*** Daisy has joined #openstack-horizon20:24
openstackgerritRajat Vig proposed openstack/horizon: Add angular flavors panel  https://review.openstack.org/20620320:24
*** vgridnev has quit IRC20:31
*** vgridnev has joined #openstack-horizon20:32
*** kzaitsev_mb has joined #openstack-horizon20:33
openstackgerritRajat Vig proposed openstack/horizon: Add Magic Search Filtering to NG Flavors  https://review.openstack.org/23375520:34
*** errolpais has joined #openstack-horizon20:35
openstackgerritRajat Vig proposed openstack/horizon: Add a Delete Modal Service for deleting entities  https://review.openstack.org/23440820:37
openstackgerritRajat Vig proposed openstack/horizon: API Services for Flavors Panel  https://review.openstack.org/21489620:37
*** dansiwiec has quit IRC20:38
*** harlowja has joined #openstack-horizon20:39
*** harlowja_ has joined #openstack-horizon20:40
openstackgerritDiana Whitten proposed openstack/horizon: Webroot Theme is no longer needed  https://review.openstack.org/21341120:40
*** SimonChung has joined #openstack-horizon20:40
*** alejandrito has quit IRC20:42
*** harlowja has quit IRC20:44
*** amotoki has joined #openstack-horizon20:45
*** jobewan has quit IRC20:45
*** rajatvig has quit IRC20:47
*** doug-fis_ has joined #openstack-horizon20:50
*** doug-fi__ has joined #openstack-horizon20:50
robcresswellhurgleburgler: ^^ isn't that still needed in devstack?20:51
*** doug-fi__ has quit IRC20:51
robcresswelllooking at ze comments20:51
*** doug-fi__ has joined #openstack-horizon20:51
hurgleburglerWell, devstack is configured to need it20:51
hurgleburglerbut it doesn't actually need it anymore20:51
hurgleburglerwe just need to update devstack though20:51
hurgleburglerright david-lyle ?20:51
*** Daisy has quit IRC20:51
david-lylehurgleburgler: yes, hence the -2 :)20:52
robcresswellSure, I just wondered why the rebasing if its waiting on devstack changes20:52
hurgleburglerI was on the wrong patch page :P20:52
hurgleburglersorry20:52
hurgleburglerclick happy me20:52
*** doug-fish has quit IRC20:53
hurgleburglertried to rebase another patch, which actually needs a manual rebase now20:53
*** david____ has joined #openstack-horizon20:54
*** david____ is now known as david_cu20:54
*** doug-fis_ has quit IRC20:54
*** lapalm has quit IRC20:54
openstackgerritRob Cresswell proposed openstack/horizon: Add firewall detail actions and breadcrumb nav  https://review.openstack.org/23431520:58
robcresswelltqtran: Could you take a look at https://review.openstack.org/#/c/212152 ?20:59
*** Drago1 has joined #openstack-horizon20:59
*** Drago1 has quit IRC20:59
tqtransure thing boss21:00
*** Drago1 has joined #openstack-horizon21:00
*** jobewan has joined #openstack-horizon21:00
*** SimonChung1 has joined #openstack-horizon21:01
*** SimonChung has quit IRC21:01
robcresswelltqtran: Thanks!21:01
*** SimonChung has joined #openstack-horizon21:02
*** SimonChung1 has quit IRC21:02
*** SimonChung1 has joined #openstack-horizon21:02
*** SimonChung has quit IRC21:02
*** Drago has quit IRC21:03
matt-borlandlater folks!21:05
*** matt-borland has quit IRC21:05
*** pauloewerton has quit IRC21:06
*** lblanchard has quit IRC21:07
*** woodm19791 has quit IRC21:10
robcresswellMore of you need this in your lives: http://wiki.znc.in/ZNC21:10
*** jongchoi_ has joined #openstack-horizon21:10
*** stevemar_ has joined #openstack-horizon21:11
*** rajatvig has joined #openstack-horizon21:17
*** darrenc is now known as darrenc_afk21:17
*** Daisy has joined #openstack-horizon21:21
*** petertr7 is now known as petertr7_away21:23
*** neelashah1 has quit IRC21:25
*** rhagarty has joined #openstack-horizon21:28
*** SimonChung has joined #openstack-horizon21:28
*** SimonChung1 has quit IRC21:28
*** mpavlase has quit IRC21:28
*** amotoki has quit IRC21:31
*** zz_john5223 has quit IRC21:31
robcresswelltqtran: Looking at the Create User action21:31
tqtrancoolios21:31
robcresswelltqtran: settings.getSettings on the Default Role always returns the default value, not the setting21:32
*** doug-fi__ has quit IRC21:32
robcresswelltqtran: If I log it, it just gives me the string '_member_' in your patch. I can make that anything though, and it always seems to return that instead of the setting value.21:32
*** doug-fish has joined #openstack-horizon21:33
*** zz_john5223 has joined #openstack-horizon21:33
robcresswelltqtran: I'll look some more probably during summit week :)21:33
tqtranhm... really? can you put a breakpoint on the server side and see if its getting hit at all?21:34
*** doug-fis_ has joined #openstack-horizon21:34
*** doug-fish has quit IRC21:37
*** rajatvig has quit IRC21:37
*** emily_w has joined #openstack-horizon21:37
*** doug-fish has joined #openstack-horizon21:38
openstackgerritTony Dunbar proposed openstack/horizon: Add STATUS_DISPLAY_CHOICE which matches Nova vm state  https://review.openstack.org/23825821:38
*** mounica has quit IRC21:38
*** emilyw has quit IRC21:38
*** doug-fis_ has quit IRC21:39
*** darrenc_afk is now known as darrenc21:39
*** emily-w has joined #openstack-horizon21:41
*** rajatvig has joined #openstack-horizon21:41
*** jobewan has quit IRC21:42
*** doug-fish has quit IRC21:42
*** rajatvig has quit IRC21:43
robcresswelltqtran: Er, break where?21:43
*** emily_w has quit IRC21:43
*** rajatvig has joined #openstack-horizon21:43
robcresswelltqtran: Actually, scrap that, its nearly 11pm, I'm having a cup of tea and sleep :)21:44
robcresswelltqtran: See you in Tokyo!21:44
*** rhagarty_ has joined #openstack-horizon21:45
*** jpomeroy_ has joined #openstack-horizon21:45
tqtransee you in Jp!21:46
*** robcresswell has left #openstack-horizon21:46
*** rhagarty has quit IRC21:46
*** jbell8 has quit IRC21:47
*** robcresswell has joined #openstack-horizon21:47
*** jpomeroy has quit IRC21:48
*** Daisy has quit IRC21:49
*** ducttape_ has quit IRC21:49
*** doug-fish has joined #openstack-horizon21:50
*** doug-fish has quit IRC21:54
*** tqtran has quit IRC21:54
*** tqtran has joined #openstack-horizon21:55
*** jobewan has joined #openstack-horizon22:01
*** jongchoi_ has quit IRC22:02
*** jongchoi_ has joined #openstack-horizon22:05
*** david_cu has quit IRC22:06
*** cody-somerville has quit IRC22:06
*** jamielennox|away is now known as jamielennox22:07
*** zacdev has quit IRC22:07
*** jongchoi_ has quit IRC22:10
openstackgerritDiana Whitten proposed openstack/horizon: Basic tables now inherit from Bootstrap Theme  https://review.openstack.org/21860822:11
*** sqchen has quit IRC22:11
*** sqchen has joined #openstack-horizon22:12
*** lhcheng_ has quit IRC22:17
*** Daisy has joined #openstack-horizon22:18
*** vgridnev has quit IRC22:19
*** jbell8 has joined #openstack-horizon22:23
*** jpomeroy_ has quit IRC22:23
*** jbell8 has quit IRC22:27
*** emily_w has joined #openstack-horizon22:27
*** bpokorny_ has joined #openstack-horizon22:29
*** jbell8 has joined #openstack-horizon22:30
*** emilyw has joined #openstack-horizon22:30
*** bpokorny_ has quit IRC22:30
*** bpokorny_ has joined #openstack-horizon22:30
*** emily-w has quit IRC22:30
*** emily_w has quit IRC22:32
*** lhcheng has joined #openstack-horizon22:33
*** bpokorny has quit IRC22:33
*** Daisy has quit IRC22:39
*** jbell8 has quit IRC22:40
*** jbell8 has joined #openstack-horizon22:41
*** jobewan has quit IRC22:46
*** emilyw has quit IRC22:52
*** emilyw has joined #openstack-horizon22:52
*** jbell8 has quit IRC22:53
*** jbell8 has joined #openstack-horizon22:54
*** Drago1 has quit IRC23:03
*** david_cu has joined #openstack-horizon23:05
*** Daisy has joined #openstack-horizon23:08
*** ZZelle has quit IRC23:08
*** emilyw has quit IRC23:17
*** yamamoto has joined #openstack-horizon23:18
*** ducttape_ has joined #openstack-horizon23:19
*** ducttape_ has quit IRC23:20
*** cbii has joined #openstack-horizon23:23
openstackgerritDiana Whitten proposed openstack/horizon: Horizon Login now inherits from Bootstrap Theme  https://review.openstack.org/21988123:23
*** cbii has left #openstack-horizon23:23
*** cbii has joined #openstack-horizon23:24
*** yamamoto has quit IRC23:26
openstackgerritRajat Vig proposed openstack/horizon: API for getting and updating quotas  https://review.openstack.org/20712623:27
openstackgerritRajat Vig proposed openstack/horizon: Add angular defaults panel subflows for admin  https://review.openstack.org/23498623:27
openstackgerritRajat Vig proposed openstack/horizon: Add angular defaults panel for admin  https://review.openstack.org/20529623:27
*** kzaitsev_mb has quit IRC23:27
*** amotoki has joined #openstack-horizon23:28
*** erhudy has quit IRC23:29
openstackgerritDiana Whitten proposed openstack/horizon: Basic tables now inherit from Bootstrap Theme  https://review.openstack.org/21860823:32
*** amotoki has quit IRC23:33
*** harlowja_ has quit IRC23:33
*** Daisy has quit IRC23:35
*** harlowja has joined #openstack-horizon23:37
*** exploreshaifali has quit IRC23:38
openstackgerritDiana Whitten proposed openstack/horizon: Dropdowns should have a consistent design  https://review.openstack.org/23273423:38
hurgleburglerAny one want a quick review, this one is ready: https://review.openstack.org/#/c/23273423:39
*** cbii has quit IRC23:40
openstackgerritDiana Whitten proposed openstack/horizon: Magic Search is now themable  https://review.openstack.org/23330523:40
tqtrandavid-lyle: if you're still around, can i bug you to +A this through? https://review.openstack.org/#/c/214815/23:42
tqtranits something to be desired for many patches23:43
*** Daisy has joined #openstack-horizon23:43
openstackgerritDiana Whitten proposed openstack/horizon: Magic Search is now themable  https://review.openstack.org/23330523:44
*** harlowja_ has joined #openstack-horizon23:45
*** emilyw has joined #openstack-horizon23:46
*** Daisy has quit IRC23:47
*** Daisy has joined #openstack-horizon23:48
*** harlowja has quit IRC23:49
*** SimonChung1 has joined #openstack-horizon23:51
*** SimonChung has quit IRC23:51
*** SimonChung has joined #openstack-horizon23:52
*** SimonChung1 has quit IRC23:52
*** Daisy has quit IRC23:52
david-lyletqtran: I thought I had, oops23:54
*** ducttape_ has joined #openstack-horizon23:54
*** SimonChung1 has joined #openstack-horizon23:55
*** SimonChung has quit IRC23:55
*** yamamoto has joined #openstack-horizon23:55
*** aortega has quit IRC23:56

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!