*** spatel has joined #openstack-horizon | 00:41 | |
*** btully has joined #openstack-horizon | 01:23 | |
*** btully has quit IRC | 01:28 | |
*** duochao has joined #openstack-horizon | 01:37 | |
*** duochao has quit IRC | 01:46 | |
*** pg588x has joined #openstack-horizon | 01:52 | |
*** duochao has joined #openstack-horizon | 01:53 | |
*** BjoernT has joined #openstack-horizon | 02:04 | |
*** spatel has quit IRC | 02:46 | |
*** dave-mccowan has joined #openstack-horizon | 03:12 | |
*** whoami-rajat has joined #openstack-horizon | 03:14 | |
*** dave-mccowan has quit IRC | 03:47 | |
*** tkajinam has quit IRC | 04:03 | |
*** BjoernT has quit IRC | 04:07 | |
*** BjoernT has joined #openstack-horizon | 04:13 | |
*** duochao has quit IRC | 04:23 | |
*** BjoernT has quit IRC | 04:29 | |
*** BjoernT has joined #openstack-horizon | 04:31 | |
*** duochao has joined #openstack-horizon | 04:51 | |
*** huth has joined #openstack-horizon | 05:10 | |
*** BjoernT has quit IRC | 05:17 | |
*** BjoernT_ has joined #openstack-horizon | 05:17 | |
*** BjoernT_ has quit IRC | 05:17 | |
*** BjoernT has joined #openstack-horizon | 05:18 | |
*** BjoernT has quit IRC | 05:18 | |
*** BjoernT_ has joined #openstack-horizon | 05:18 | |
*** BjoernT_ has quit IRC | 05:19 | |
*** BjoernT_ has joined #openstack-horizon | 05:23 | |
*** BjoernT_ has quit IRC | 05:24 | |
*** BjoernT has joined #openstack-horizon | 05:25 | |
*** BjoernT has quit IRC | 05:25 | |
*** BjoernT_ has joined #openstack-horizon | 05:28 | |
*** BjoernT_ has quit IRC | 05:29 | |
*** BjoernT has joined #openstack-horizon | 05:29 | |
*** BjoernT has quit IRC | 05:30 | |
*** BjoernT has joined #openstack-horizon | 05:32 | |
*** BjoernT has quit IRC | 05:32 | |
*** BjoernT has joined #openstack-horizon | 05:33 | |
*** BjoernT has quit IRC | 05:33 | |
*** BjoernT has joined #openstack-horizon | 05:33 | |
*** BjoernT has quit IRC | 05:34 | |
*** BjoernT_ has joined #openstack-horizon | 05:34 | |
*** BjoernT_ has quit IRC | 05:35 | |
*** BjoernT has joined #openstack-horizon | 05:35 | |
*** BjoernT has quit IRC | 05:37 | |
openstackgerrit | Pallav Gupta proposed openstack/horizon master: Fix skipped integration tests for securitygroup, namespace, usersettings https://review.opendev.org/682564 | 06:01 |
---|---|---|
*** huth has left #openstack-horizon | 06:10 | |
openstackgerrit | Akihiro Motoki proposed openstack/horizon master: Handle partial dict setting https://review.opendev.org/684792 | 06:12 |
vishalmanchanda | amotoki: Could you review this patch https://review.opendev.org/#/c/678185/ | 06:17 |
*** duochao has quit IRC | 06:17 | |
vishalmanchanda | e0ne:^^ | 06:17 |
*** duochao has joined #openstack-horizon | 06:26 | |
*** pcaruana has joined #openstack-horizon | 06:30 | |
*** mkowalski has joined #openstack-horizon | 06:39 | |
*** mkowalski__ has quit IRC | 06:40 | |
*** tosky has joined #openstack-horizon | 06:41 | |
openstackgerrit | Merged openstack/horizon master: Fix navigation hang issue https://review.opendev.org/681175 | 06:48 |
*** whoami-rajat has quit IRC | 06:53 | |
openstackgerrit | Akihiro Motoki proposed openstack/horizon master: Fix skipped integration tests for securitygroup, namespace, usersettings https://review.opendev.org/682564 | 07:05 |
*** tesseract has joined #openstack-horizon | 07:05 | |
openstackgerrit | Akihiro Motoki proposed openstack/horizon master: Fix skipped integration tests for securitygroup, namespace, usersettings https://review.opendev.org/682564 | 07:07 |
openstackgerrit | Pallav Gupta proposed openstack/horizon master: Fix skipped integration tests for securitygroup, namespace, usersettings https://review.opendev.org/682564 | 07:33 |
*** FlorianFa has joined #openstack-horizon | 07:52 | |
*** tesseract has quit IRC | 07:52 | |
*** tesseract has joined #openstack-horizon | 07:53 | |
*** pg588x has quit IRC | 07:53 | |
*** e0ne has joined #openstack-horizon | 08:45 | |
*** ChanServ sets mode: +o e0ne | 08:45 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/horizon master: Imported Translations from Zanata https://review.opendev.org/684974 | 08:46 |
*** rcernin has quit IRC | 09:08 | |
*** Emine has joined #openstack-horizon | 09:24 | |
*** victor286 has joined #openstack-horizon | 09:28 | |
*** whoami-rajat has joined #openstack-horizon | 09:38 | |
*** rcernin has joined #openstack-horizon | 09:38 | |
*** mkowalski has quit IRC | 09:50 | |
*** mkowalski has joined #openstack-horizon | 09:51 | |
*** tesseract has quit IRC | 09:59 | |
*** tesseract has joined #openstack-horizon | 10:00 | |
e0ne | amotoki: hi. I took a look on your patch and it seems good | 10:00 |
e0ne | amotoki: honestly, I don't like a function name but don't want to start bikeshedding session how to name it | 10:01 |
e0ne | maybe 'dict_opt' would be better name, but I'm not sure | 10:02 |
*** BjoernT has joined #openstack-horizon | 10:23 | |
*** btully has joined #openstack-horizon | 10:27 | |
*** btully has quit IRC | 10:31 | |
*** pcaruana has quit IRC | 10:32 | |
*** victor286 has quit IRC | 10:42 | |
openstackgerrit | Merged openstack/horizon master: Fix skipped integration tests for securitygroup, namespace, usersettings https://review.opendev.org/682564 | 10:49 |
*** duochao has quit IRC | 10:51 | |
*** pcaruana has joined #openstack-horizon | 10:58 | |
openstackgerrit | Merged openstack/horizon master: Wrong list of attached vms are shown in manage volume attachments https://review.opendev.org/678185 | 11:21 |
*** BjoernT has quit IRC | 11:23 | |
*** ianychoi has quit IRC | 12:00 | |
*** ianychoi has joined #openstack-horizon | 12:03 | |
*** pcaruana has quit IRC | 12:05 | |
openstackgerrit | Merged openstack/horizon master: Imported Translations from Zanata https://review.opendev.org/684974 | 12:07 |
*** pcaruana has joined #openstack-horizon | 12:19 | |
amotoki | e0ne: ah.... there are several similar names: setting, option, configuration... get_dict_opt would be simpler. | 12:50 |
*** BjoernT has joined #openstack-horizon | 12:52 | |
amotoki | e0ne: the only reason I use 'setting' is that 'setting' is used in horizon and 'opt' or 'config' is less used. there is no strong preference on naming. | 12:53 |
e0ne | amotoki: that's just a very minor note | 12:53 |
e0ne | O | 12:53 |
*** jtomasek_ has quit IRC | 12:53 | |
e0ne | amotoki: I've already +2'ed | 12:54 |
amotoki | e0ne: yeah, I saw it :) | 12:54 |
amotoki | e0ne: btw, during working on this, we have no convenient way to override the default value as oslo.config allows us. I need to prepare three util func per module (openstack_dashboard, horizon, opensatck_auth). this is ugly. | 12:55 |
amotoki | it might be a good reason we would like to access opts via oslo.config directly... | 12:55 |
*** jtomasek has joined #openstack-horizon | 12:56 | |
*** tobberydberg_ is now known as tobberydberg | 13:01 | |
*** whoami-rajat has quit IRC | 13:37 | |
*** priteau has joined #openstack-horizon | 13:50 | |
*** priteau has quit IRC | 13:55 | |
*** BjoernT has quit IRC | 13:57 | |
e0ne | dklyle. rdopiera. vishalmanchanda: I appreciate if you could review this one https://review.opendev.org/#/c/684792/2 | 13:58 |
e0ne | amotoki: what do you think about https://review.opendev.org/#/c/679665/? | 13:59 |
e0ne | amotoki: I'll try to fix it in oslo side in U release | 13:59 |
*** BjoernT has joined #openstack-horizon | 14:00 | |
*** rcernin has quit IRC | 14:06 | |
openstackgerrit | Merged openstack/horizon master: Updated max-width to be dynamic for .member class https://review.opendev.org/681366 | 14:20 |
*** pg588x has joined #openstack-horizon | 14:27 | |
*** victor286 has joined #openstack-horizon | 14:40 | |
*** gyee has joined #openstack-horizon | 14:53 | |
*** whoami-rajat has joined #openstack-horizon | 15:16 | |
amotoki | e0ne: back to online. let me check it from now. as a quick look, your plan sounds reasonable. | 15:18 |
e0ne | amotoki: thanks a lot | 15:18 |
amotoki | e0ne: do you have any bug report on this? | 15:19 |
e0ne | amotoki: unfortunately, no :( | 15:19 |
e0ne | I can file a bug if needed | 15:19 |
e0ne | I just cherry-picked that change from our downstream repo | 15:20 |
*** mvkr has quit IRC | 15:20 | |
amotoki | e0ne: it would be nice to have a bug. it would make the thing easier both when backporting it to stable branches and to propose it to oslo.config. | 15:22 |
e0ne | ok | 15:22 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/horizon master: Handle Permission Denied for policy files https://review.opendev.org/679665 | 15:25 |
e0ne | amotoki: I'm going to approve https://review.opendev.org/#/c/684792/ tonight because it affects plugins | 15:26 |
amotoki | e0ne: thanks | 15:26 |
amotoki | e0ne: regarding your policy file patch, I just commented a nit. it is nice to have some warning message. thought? | 15:29 |
e0ne | sounds reasonable | 15:29 |
e0ne | amotoki: do you want to log exception message too? | 15:31 |
amotoki | e0ne: what in my mind is something like LOG.warning("Cannot load a policy file XXXX due to IOError. One possible reason is permission defined", ....) | 15:32 |
amotoki | e0ne: I think no traceback is needed. | 15:33 |
*** mvkr has joined #openstack-horizon | 15:33 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/horizon master: Handle Permission Denied for policy files https://review.opendev.org/679665 | 15:35 |
amotoki | e0ne: you seem to copy my above one. I made a typo "defined" :-( | 15:44 |
e0ne | amotoki: fixed :) | 15:46 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/horizon master: Handle Permission Denied for policy files https://review.opendev.org/679665 | 15:46 |
*** priteau has joined #openstack-horizon | 15:48 | |
*** tosky has quit IRC | 15:52 | |
*** btully has joined #openstack-horizon | 15:53 | |
*** pcaruana has quit IRC | 15:57 | |
*** btully has quit IRC | 15:58 | |
e0ne | amotoki: I'm heading home, be back in ~2 hours | 16:05 |
*** e0ne has quit IRC | 16:06 | |
*** Emine has quit IRC | 16:11 | |
*** victor286 has quit IRC | 16:14 | |
*** tesseract has quit IRC | 16:19 | |
*** BjoernT has quit IRC | 16:26 | |
*** BjoernT has joined #openstack-horizon | 16:32 | |
*** priteau has quit IRC | 16:37 | |
*** yaawang has quit IRC | 16:41 | |
*** yaawang has joined #openstack-horizon | 16:43 | |
*** pg588x has quit IRC | 16:49 | |
*** dklyle has quit IRC | 17:01 | |
*** dklyle has joined #openstack-horizon | 17:01 | |
*** ChanServ sets mode: +o dklyle | 17:01 | |
*** itlinux has quit IRC | 17:04 | |
*** gary-smith_ has joined #openstack-horizon | 17:50 | |
*** pcaruana has joined #openstack-horizon | 18:04 | |
*** whoami-rajat has quit IRC | 18:15 | |
*** e0ne has joined #openstack-horizon | 18:30 | |
*** ChanServ sets mode: +o e0ne | 18:30 | |
*** pcaruana has quit IRC | 18:36 | |
openstackgerrit | Merged openstack/horizon master: Handle Permission Denied for policy files https://review.opendev.org/679665 | 18:39 |
*** whoami-rajat has joined #openstack-horizon | 18:41 | |
*** BjoernT has quit IRC | 19:05 | |
*** BjoernT has joined #openstack-horizon | 19:07 | |
*** dave-mccowan has joined #openstack-horizon | 19:22 | |
*** e0ne has quit IRC | 19:25 | |
*** dave-mccowan has quit IRC | 19:27 | |
openstackgerrit | Merged openstack/horizon master: Handle partial dict setting https://review.opendev.org/684792 | 19:46 |
*** BjoernT_ has joined #openstack-horizon | 20:04 | |
*** BjoernT has quit IRC | 20:06 | |
*** e0ne has joined #openstack-horizon | 20:22 | |
*** ChanServ sets mode: +o e0ne | 20:22 | |
*** e0ne has quit IRC | 20:24 | |
*** zigo_ is now known as zigo | 20:49 | |
*** whoami-rajat has quit IRC | 20:55 | |
*** btully has joined #openstack-horizon | 21:20 | |
*** btully has quit IRC | 21:24 | |
*** BjoernT_ has quit IRC | 22:01 | |
*** tkajinam has joined #openstack-horizon | 22:57 | |
*** rcernin has joined #openstack-horizon | 23:24 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!