Monday, 2017-05-15

*** openstack has joined #openstack-horizon07:01
Balajithank you @robcresswell07:02
*** Balaji has left #openstack-horizon07:06
*** shuyingya has joined #openstack-horizon07:08
*** belmoreira has joined #openstack-horizon07:12
*** Daisy has joined #openstack-horizon07:22
*** adriant has quit IRC07:29
*** rcernin has joined #openstack-horizon07:47
*** pcaruana has joined #openstack-horizon07:47
*** tosky has joined #openstack-horizon07:48
zioprotorobcresswell: hey there, on what timezone are you ? :) I have done some work in this review https://review.openstack.org/#/c/425283/ now it should be quite okay I guess :D07:51
*** e0ne has joined #openstack-horizon07:56
*** jpich has joined #openstack-horizon07:57
robcresswellzioproto: I'm London based, so UTC / UTC+108:10
*** faizy_ has quit IRC08:16
*** faizy_ has joined #openstack-horizon08:16
*** e0ne has quit IRC08:21
openstackgerritLajos Katona proposed openstack/horizon master: New readonly panel for trunks  https://review.openstack.org/44921708:24
*** e0ne has joined #openstack-horizon08:26
*** pjm6_ has quit IRC08:27
zioprotorobcresswell: great, in am in Zurich. I guess you met my colleague in Boston last week08:36
*** Daisy has quit IRC08:36
*** Daisy has joined #openstack-horizon08:37
*** Daisy_ has joined #openstack-horizon08:39
*** Daisy_ has quit IRC08:41
*** Daisy_ has joined #openstack-horizon08:42
*** Daisy has quit IRC08:42
*** ZZelle_ has quit IRC08:51
*** ZZelle has joined #openstack-horizon08:51
*** ZZelle is now known as Guest6838308:52
*** Guest68383 is now known as ZZelle_08:54
openstackgerritDebo Zhang proposed openstack/horizon master: Use a def instend of a lambda expression.  https://review.openstack.org/46457509:01
openstackgerritLajos Katona proposed openstack/horizon master: New readonly panel for trunks  https://review.openstack.org/44921709:07
*** simon-AS559 has joined #openstack-horizon09:23
openstackgerritRob Cresswell proposed openstack/horizon master: Add a new management cmd to pull translations  https://review.openstack.org/46458109:23
*** e0ne has quit IRC09:26
*** e0ne has joined #openstack-horizon09:26
*** e0ne has quit IRC09:26
*** e0ne has joined #openstack-horizon09:27
*** e0ne has quit IRC09:27
*** e0ne has joined #openstack-horizon09:27
*** e0ne has quit IRC09:28
*** e0ne has joined #openstack-horizon09:28
*** e0ne has quit IRC09:29
*** e0ne has joined #openstack-horizon09:29
*** e0ne has quit IRC09:30
openstackgerritDebo Zhang proposed openstack/horizon master: Use a def instend of a lambda expression.  https://review.openstack.org/46457509:30
*** pjm6 has joined #openstack-horizon09:47
*** Daisy_ has quit IRC09:53
*** Daisy has joined #openstack-horizon09:53
*** Daisy has quit IRC09:58
*** mariusv has quit IRC10:17
*** mariusv_ has quit IRC10:20
*** mariusv has joined #openstack-horizon10:25
*** mariusv has quit IRC10:25
*** mariusv has joined #openstack-horizon10:25
*** lunarlamp has joined #openstack-horizon10:26
*** belmoreira has quit IRC10:28
*** ZZelle_ is now known as ZZelle10:58
*** mvk has quit IRC11:11
*** shuyingya has quit IRC11:32
*** shuyingy_ has joined #openstack-horizon11:32
*** mariusv has quit IRC11:32
*** lunarlamp is now known as mariusv11:32
*** mvk has joined #openstack-horizon11:40
*** simon-AS559 has quit IRC12:01
*** wolverineav has joined #openstack-horizon12:25
*** erhudy has joined #openstack-horizon12:36
*** ratailor has quit IRC12:40
*** belmoreira has joined #openstack-horizon12:43
*** dave-mccowan has joined #openstack-horizon12:51
*** shuyingy_ has quit IRC12:59
*** jtriley has quit IRC13:04
*** ducttape_ has joined #openstack-horizon13:18
openstackgerritMerged openstack/manila-ui master: Updated from global requirements  https://review.openstack.org/46439913:21
*** faizy has joined #openstack-horizon13:22
*** makowals_ has joined #openstack-horizon13:24
*** faizy_ has quit IRC13:25
*** makowals_ has quit IRC13:25
*** makowals has quit IRC13:26
*** makowals has joined #openstack-horizon13:28
*** aortega has quit IRC13:30
*** lblanchard has joined #openstack-horizon13:36
*** masco has quit IRC13:36
*** jtriley has joined #openstack-horizon13:38
*** aortega has joined #openstack-horizon13:42
robcresswellzioproto: Ah sorry, I missed the reponse there. Er, I'm sure I did, I met a lot of people :)13:48
robcresswellzioproto: A few people have asked for that fix, I'll take a look13:49
*** weezS has joined #openstack-horizon14:01
*** shuyingya has joined #openstack-horizon14:04
*** shuyingya has quit IRC14:09
*** TravT has joined #openstack-horizon14:12
*** ChanServ sets mode: +v TravT14:12
*** lcastell has joined #openstack-horizon14:14
zioprotorobcresswell: as soon as it is merged I will backport to Ocata and Newton14:15
zioprotoand make the ubuntu packages14:15
robcresswellzioproto: Ocata should be okay, Newton is a bit tenuous. I'll need to see what the other cores think.14:15
*** TravT_ has quit IRC14:15
robcresswellNewton is critical/security, and this isn't really either.14:15
zioprotoWell, I will backport it anyway for our deploy,ent14:18
robcresswellzioproto: Yeah, makes sense :)14:19
zioprotois Mateusz Kowalski in this channel ?14:27
zioprotoI did reply to the comment in the review, I am not sure how I have to change that line14:27
zioprotoif he uploads a new patch set it is fine for me14:27
*** ratailor has joined #openstack-horizon14:28
robcresswellzioproto: Thats makowals14:31
makowalsYep, I'm here14:31
makowalsI'm just taking a look at this patch, but at this moment I believe what it does is wrong :<14:32
makowalsI will try to put a more detailed comment so someone else can analyze14:33
makowalsBut I see using $scope.model.allowCreateVolumeFromImage is wrong in the scenario we want to achieve14:33
makowalshttps://review.openstack.org/#/c/425283/8/openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/launch-instance-model.service.js -- in this file we cannot execute "onBootToVolumeSupported" ONLY IF "config.enable_default_create_volume = True"14:35
*** shuyingya has joined #openstack-horizon14:38
makowalsI'm very confused with the name "$scope.model.allowCreateVolumeFromImage" as we use it more like ".model.createVolumeByDefault"14:39
makowalsBut I'm not really eager just to rename it as it's used widely in already existing tests14:39
makowalsI would just introduce new one, ".model.createVolumeByDefault", set it according to config file and then use properly14:40
robcresswellThose are different settings though (or sound like it)14:41
robcresswellallowing something vs enabling by default14:41
* robcresswell reads code again14:41
zioprotoI will be soon leaving my office14:44
zioprotoplease feel free to add more patchsets on top of mine14:44
robcresswellHmm, makowals, zioproto, it seems like the usage of enable_default_create_volume is just wrong anyway. If you look at the model.js file, we check whether it should be enabled by default, then we check if it is even supported14:45
robcresswellWhich is clearly the wrong way round14:45
robcresswellI think we could simplify it quite a lot...14:46
robcresswelllet me finish this other patch then I'll take a look14:46
*** aortega has quit IRC14:46
*** zackf has joined #openstack-horizon14:48
*** itlinux has joined #openstack-horizon14:53
zioprotorobcresswell: thanks ! I am not a web developer at all, I just tried to rebase to master the work proposed by somebody else.14:53
*** jeremy_moffitt has quit IRC15:06
*** rcernin has quit IRC15:07
makowals@zioproto: It's totally cool, at least it created some noise so the patch got more visibility15:08
*** jeremy_moffitt has joined #openstack-horizon15:17
*** pcaruana has quit IRC15:25
*** aortega has joined #openstack-horizon15:29
*** jtriley has quit IRC15:30
*** makowals has quit IRC15:31
*** makowals has joined #openstack-horizon15:34
*** shuyingya has quit IRC15:36
*** aortega has quit IRC15:37
*** itlinux has quit IRC15:45
*** itlinux has joined #openstack-horizon15:47
*** itlinux has quit IRC15:52
*** jtriley has joined #openstack-horizon15:53
*** jose-phi_ has quit IRC15:55
*** jose-phillips has joined #openstack-horizon15:56
*** gyee has joined #openstack-horizon15:58
*** ratailor has quit IRC16:02
*** jpich has quit IRC16:11
*** belmoreira has quit IRC16:16
*** e0ne has joined #openstack-horizon16:31
*** jose-phillips has quit IRC16:31
*** jose-phillips has joined #openstack-horizon16:34
*** harlowja has joined #openstack-horizon16:44
*** david-lyle has quit IRC16:46
*** dklyle has joined #openstack-horizon16:46
*** ChanServ sets mode: +o dklyle16:46
*** itlinux has joined #openstack-horizon16:48
*** TravT_ has joined #openstack-horizon16:57
*** ChanServ sets mode: +v TravT_16:57
*** TravT has quit IRC17:00
*** e0ne has quit IRC17:01
*** mvk has quit IRC17:07
*** weezS has quit IRC17:08
*** jose-phillips has quit IRC17:21
*** e0ne has joined #openstack-horizon17:25
*** jose-phillips has joined #openstack-horizon17:26
*** dklyle is now known as david-lyle17:37
*** jose-phillips has quit IRC17:39
*** mvk has joined #openstack-horizon17:41
*** e0ne has quit IRC17:45
*** tosky has quit IRC17:47
*** jose-phillips has joined #openstack-horizon17:48
*** itlinux has quit IRC17:50
openstackgerritRob Cresswell proposed openstack/horizon master: Add the Snapshots tab on the Volume Details page  https://review.openstack.org/27046117:57
*** aortega has joined #openstack-horizon17:57
openstackgerritJulie Gravel proposed openstack/horizon master: Fix Status dropdown initial value  https://review.openstack.org/46145318:13
openstackgerritValeriy Ponomaryov proposed openstack/manila-ui master: [WIP] Refactor dashboards  https://review.openstack.org/46471618:18
*** marlinc has quit IRC18:18
*** TravT_ has quit IRC18:29
*** jose-phillips has quit IRC18:41
*** marlinc has joined #openstack-horizon18:43
openstackgerritValeriy Ponomaryov proposed openstack/manila-ui master: [WIP] Refactor dashboards  https://review.openstack.org/46471618:44
openstackgerritLajos Katona proposed openstack/horizon master: WIP: Details for trunks  https://review.openstack.org/46472718:57
*** TravT has joined #openstack-horizon18:57
*** ChanServ sets mode: +v TravT18:57
*** jose-phillips has joined #openstack-horizon19:00
*** zackf has quit IRC19:07
*** zackf has joined #openstack-horizon19:10
*** zackf has quit IRC19:14
*** zackf has joined #openstack-horizon19:23
*** zul has quit IRC19:32
*** weezS has joined #openstack-horizon19:33
openstackgerritJulie Gravel proposed openstack/horizon master: Fix Status dropdown initial value  https://review.openstack.org/46145319:37
*** TravT has quit IRC19:44
*** aortega has quit IRC19:52
*** zul has joined #openstack-horizon19:52
*** zackf has quit IRC19:53
*** zackf has joined #openstack-horizon19:56
*** zackf has quit IRC20:00
*** TravT has joined #openstack-horizon20:07
*** ChanServ sets mode: +v TravT20:07
*** belmoreira has joined #openstack-horizon20:08
*** lblanchard has quit IRC20:08
*** belmoreira has quit IRC20:08
*** ducttap__ has joined #openstack-horizon20:13
*** ducttape_ has quit IRC20:14
*** TravT has quit IRC20:18
*** jose-phillips has quit IRC20:19
*** ducttap__ has quit IRC20:23
*** ducttape_ has joined #openstack-horizon20:23
*** ducttap__ has joined #openstack-horizon20:24
*** ducttape_ has quit IRC20:25
openstackgerritJulie Gravel proposed openstack/horizon master: Use same form for Create Router in network topology  https://review.openstack.org/46231820:30
*** aortega has joined #openstack-horizon20:31
*** jose-phillips has joined #openstack-horizon20:32
*** trevormc has joined #openstack-horizon20:33
*** gugl has quit IRC20:36
*** tjones has joined #openstack-horizon20:40
*** tonygunk has quit IRC20:40
tjoneshi - nova moved to policy.yaml instead of json.  does horizon support the policy.yaml file too?20:40
robcresswelltjones: No, you'll need to convert it20:41
tjonesok thanks20:41
*** tonygunk has joined #openstack-horizon20:44
tjoneshurrah for anisble - it will do it for me!20:46
robcresswell:)20:46
*** trevormc has left #openstack-horizon20:47
*** dave-mccowan has quit IRC20:52
*** jtriley has quit IRC20:58
openstackgerritRob Cresswell proposed openstack/horizon master: Add config for default create colume option  https://review.openstack.org/42528320:58
*** zackf has joined #openstack-horizon20:59
openstackgerritRob Cresswell proposed openstack/horizon master: Add config for default create colume option  https://review.openstack.org/42528321:00
robcresswellzioproto, makowals, mnaser: https://review.openstack.org/#/c/425283/21:00
robcresswellThats a little cleaner, IMO, and does what it says on the tin.21:01
*** anupn has joined #openstack-horizon21:04
*** jose-phillips has quit IRC21:05
anupnHi All, i am having trouble seeing horizon dashboard in my browser. I have devstack setup inside vagrant. Do we have to do some extra network settings?21:07
anupnwhile trying to run horizon with devstack vagrant?21:07
*** jose-phillips has joined #openstack-horizon21:18
*** gugl has joined #openstack-horizon21:18
*** zackf has quit IRC21:27
*** zackf has joined #openstack-horizon21:32
*** zackf has quit IRC21:37
tjonesrobcresswell: it looks like openstack_auth uses oslo.policy which does understand both json and yaml formats.  so it might possibly work - im trying it now…..21:52
*** adriant has joined #openstack-horizon21:57
tjoneshm no21:59
tjonesi get a bunch of errors like DEBUG:oslo_policy.policy:Rule [compute_extension:hypervisors] does not exist21:59
tjonesyeah does not work  - as you said22:01
*** zackf has joined #openstack-horizon22:02
*** ducttap__ has quit IRC22:04
*** ducttape_ has joined #openstack-horizon22:06
*** ducttape_ has quit IRC22:10
*** aortega has quit IRC22:11
*** ducttape_ has joined #openstack-horizon22:16
*** weezS has quit IRC22:29
tjonesand it's not a matter of just converting yaml to json, the structure has changed too22:32
tjonesim not sure how this is supposed to work22:32
*** zackf has quit IRC22:34
david-lyletjones we recently merged to the new format of nova policy, at least as to how the rules are specified22:38
david-lylebut still in json and only on master22:38
tjonesoh ok - so if i grab that patch and convert to json it should work?  (im on ocata)22:39
david-lylehttps://github.com/openstack/horizon/commit/c61ae4f0834253e523c4443cecb3ce5eb06bf89b was the patch22:39
tjonesoslo.policy will read either json or yaml22:39
tjonesdavid-lyle: thanks!22:40
david-lyletjones, I've never tried yaml with d-o-a22:40
david-lyleso you're in uncharted waters22:40
david-lyleat least on that part22:40
tjones:-)22:40
tjonesthanks this helps a lot22:41
*** tjones1 has joined #openstack-horizon22:43
*** tjones has quit IRC22:47
*** tjones1 has quit IRC22:47
*** zackf has joined #openstack-horizon23:04
*** wolverineav has quit IRC23:23
*** wolverineav has joined #openstack-horizon23:25
*** wolverineav has quit IRC23:30
*** zackf has quit IRC23:34
*** ducttape_ has quit IRC23:34
*** ducttape_ has joined #openstack-horizon23:38
*** ducttape_ has quit IRC23:43
*** harlowja has quit IRC23:46
*** lcastell has quit IRC23:54
*** hoonetorg has quit IRC23:54

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