Wednesday, 2019-09-11

*** gyee has quit IRC00:42
*** Tianhao_Hu has joined #openstack-horizon01:20
*** spatel has joined #openstack-horizon01:24
*** tkajinam has quit IRC01:25
*** tkajinam has joined #openstack-horizon01:26
openstackgerritMerged openstack/horizon master: django22: Replace 'render_to_response' with 'render'  https://review.opendev.org/67875701:43
openstackgerritMerged openstack/horizon master: django22: django.test.client.encode_multipart no longer accepts None  https://review.opendev.org/67866601:43
*** early has quit IRC02:00
*** early has joined #openstack-horizon02:03
*** FlorianFa has quit IRC02:15
*** early has quit IRC02:15
*** early has joined #openstack-horizon02:16
*** FlorianFa has joined #openstack-horizon02:35
openstackgerritGloria Gu proposed openstack/horizon master: Updated max-width to be dynamic for .member class  https://review.opendev.org/68136602:53
openstackgerritGloria Gu proposed openstack/horizon master: Updated max-width to be dynamic for .member class  https://review.opendev.org/68136602:55
openstackgerritMerged openstack/horizon master: django22: Call 'is_safe_url' with 'allowed_hosts'  https://review.opendev.org/67866502:55
*** spatel has quit IRC03:03
openstackgerritAkihiro Motoki proposed openstack/horizon master: trivial: Remove vendored 'is_safe_url' function  https://review.opendev.org/67866903:33
openstackgerritAkihiro Motoki proposed openstack/horizon master: Make horizon-openstack-tox-python3-django22 voting  https://review.opendev.org/68122703:34
*** samc-bbc has quit IRC03:45
*** samc-bbc has joined #openstack-horizon03:47
*** ianychoi_ has joined #openstack-horizon04:24
*** ianychoi has quit IRC04:27
*** dave-mccowan has quit IRC04:29
openstackgerritMerged openstack/horizon master: trivial: Add TODOs for remaining function-based views  https://review.opendev.org/67865804:40
*** pcaruana has joined #openstack-horizon04:42
openstackgerritMerged openstack/horizon master: trivial: Remove 'cache_control' decorator  https://review.opendev.org/67867004:46
openstackgerritMerged openstack/horizon master: trivial: Remove dead code  https://review.opendev.org/67866804:46
openstackgerritMerged openstack/horizon master: trivial: Mark inner method as such  https://review.opendev.org/67866704:46
*** pcaruana has quit IRC05:12
*** jtomasek has joined #openstack-horizon05:13
*** rcernin has quit IRC05:22
*** redrobot has quit IRC05:25
*** rcernin has joined #openstack-horizon05:38
*** rcernin has quit IRC05:51
*** Tianhao_Hu has joined #openstack-horizon06:05
*** rcernin has joined #openstack-horizon06:09
openstackgerritMerged openstack/horizon master: trivial: Remove vendored 'is_safe_url' function  https://review.opendev.org/67866906:14
*** pcaruana has joined #openstack-horizon06:21
*** mlycka has joined #openstack-horizon06:40
*** awalende has joined #openstack-horizon07:08
*** tesseract has joined #openstack-horizon07:11
openstackgerritMerged openstack/horizon master: Make horizon-openstack-tox-python3-django22 voting  https://review.opendev.org/68122707:31
openstackgerritAkihiro Motoki proposed openstack/horizon master: django22: release note and docs update  https://review.opendev.org/68139507:32
*** rcernin has quit IRC07:38
*** e0ne has joined #openstack-horizon07:56
*** ChanServ sets mode: +o e0ne07:56
*** e0ne has quit IRC08:12
*** tkajinam has quit IRC08:27
*** jawad_axd has joined #openstack-horizon08:29
*** e0ne has joined #openstack-horizon08:40
*** ChanServ sets mode: +o e0ne08:40
*** priteau has joined #openstack-horizon08:53
*** shilpasd has joined #openstack-horizon09:03
*** ianychoi_ has quit IRC09:09
shilpasdHi All: can someone help me to understand test f/w in horizon09:09
shilpasdat https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/networks/ports/tests.py#L26809:09
*** rcernin has joined #openstack-horizon09:10
shilpasdif i have self.request.user, it gives me django's AnonymousUser09:10
shilpasdbut i am not getting where exactly this user is set in request in test case09:11
shilpasdcan sombody help me to understand this further09:11
mlyckashilpasd: Hi. I'm not an expert on this in any sense of the word, but I'd look around for openstack_auth.utils::get_user() usage in test. I can see it used in a Request factory to set the user in openstack_dashboard.test.helpers and manipulated in TestCase in the same file.09:27
*** rcernin has quit IRC09:41
shilpasdmlycka: openstack_auth.utils::get_user(), returns backend.get_user(user_id) or models.AnonymousUser()09:42
shilpasdbut in request where user is set that question still not getting it09:42
shilpasdin test case mentioned above, its returning AnonymousUser()09:43
shilpasdActually i am working on test cases for https://review.opendev.org/#/c/679967/09:44
shilpasdHere pending test is i want to verify functionality for user which is not a owner of network09:44
*** Tianhao_Hu has quit IRC09:52
*** ianychoi has joined #openstack-horizon10:30
*** ianychoi has quit IRC10:45
*** ianychoi has joined #openstack-horizon10:45
*** shilpasd has quit IRC11:07
*** spsurya has joined #openstack-horizon11:58
*** dave-mccowan has joined #openstack-horizon12:04
*** awalende has quit IRC12:22
*** awalende has joined #openstack-horizon12:23
*** awalende has quit IRC12:27
amotokivishalmanchanda: I -1'ed your backport to stable/rocky https://review.opendev.org/#/c/681454/ and created a new backport with the original Change-Id in https://review.opendev.org/#/c/681454/12:27
amotokivishalmanchanda: if you have questions on the backport process, feel free to ask me.12:28
vishalmanchandaamotoki: no problem. Thanks for fixing it.12:28
vishalmanchandaamotoki: +1.12:28
-amotoki- spent too much time on horizon this week and needs to spend other projects in the rest of the week :)12:29
*** happyhemant has joined #openstack-horizon12:30
*** awalende has joined #openstack-horizon12:34
amotokivishalmanchanda: you propose several backports. Have you looked thru the stable branch section of the project team guide?12:36
*** awalende has quit IRC12:38
*** e0ne has quit IRC12:46
*** e0ne has joined #openstack-horizon12:52
*** ChanServ sets mode: +o e0ne12:52
e0neamotoki, vishalmanchanda, rdopiera: could you please review https://review.opendev.org/#/c/673807/ and the depended patch? it would be good to have it before FF13:23
vishalmanchandae0ne: ok reviewing.13:35
e0nevishalmanchanda: thanks13:36
amotokie0ne: I will look at yours a bit later. btw, what are our pending "feature" reviews? (I know my defaults.py patches hit merge conflicts with dj22 patches....)13:46
vishalmanchandaamotoki: e0ne : Could you help in this patch https://review.opendev.org/#/c/678185/.13:47
vishalmanchandaso it can also land before feature freeze.13:48
amotokie0ne: vishalmanchanda: can we collect "milestone-3 priorities" in https://etherpad.openstack.org/p/horizon-release-priorities?13:49
e0neI'll do it right now13:49
amotokie0ne: thanks. I will be back in an hour later.13:49
*** redrobot has joined #openstack-horizon13:50
amotokiI also proposed an update on grafana.o.o. https://review.opendev.org/#/c/681361/13:53
amotokithe integration test job fails frequently and I'd like to know the failure rate13:53
*** vkmc has joined #openstack-horizon13:54
vkmco/13:54
vkmchi folks, quick question, is there a configuration option to handle user access in horizon?13:55
*** jawad_axd has quit IRC13:55
vkmcsay, I have a user vkmc in keystone that can interact with the APIs13:55
vkmcbut I don't want vkmc to access the dashboard13:55
vkmccan I block the user 'vkmc'?13:55
vkmcamotoki, e0ne, maybe you have a quick answer for that ^13:56
-amotoki- is on train13:56
e0nevkmc: hi. AFAIR, we don't have such feature13:57
vkmce0ne, k k13:57
mlyckavkmc: You can block access to different features using policy files13:57
vkmcI was reading this https://docs.openstack.org/horizon/latest/contributor/topics/policy.html13:57
vkmcmlycka, was about to bring it up :)13:57
mlyckaRight right :)13:58
vkmcso... how a policy for that would look like?13:58
mlyckaTherre should be some in openstack_dashboard/conf14:00
mlyckaI'm not sure how complete that feature is though, unfortunately.14:00
mlyckaThere's also a permissions system for panels and such, but I'm not entirely sure how that works; I only brushed up against it and that was ages ago.14:03
vkmchmm yeah, looking at that seems you can configure for the services itself14:03
vkmcnot horizon14:03
vkmcservices being nova, glance, cinder, keystone14:03
mlyckaRight, but horizon will display and hide and enable and disable things based on those settings; they're similar, but they exist separately from the policy files used by the services themselves.14:04
mlyckaThe images panel, for example, should only be displayed to a user if he has 'get_images' in Horizon's glance.policy file.14:05
mlycka*glance_policy.json14:06
*** BjoernT has joined #openstack-horizon14:17
e0nevishalmanchanda: I'll test your patch later tonight or will early morning tomorrow14:32
*** priteau has quit IRC14:34
*** KeithMnemonic has joined #openstack-horizon14:35
KeithMnemonicamotoki: or anyone else, is it possible to get a WF+1 here https://review.opendev.org/#/c/679859/ ? Thanks!14:37
amotokiKeithMnemonic: could you wait a bit as we are busy for milestone-3 of train release (i.e. feature freeze) and we run out of bw to review stable branches .....14:39
KeithMnemonicok that is fine, is next week a reasonable ETA?14:40
amotokiif no upstream development, backport reviews would be speedy but it would mean no backports :p14:40
KeithMnemonici understand just looking for any estimates14:41
mlyckae0ne: amotoki: all: Hey, guys, I replied back into the thread on discuss. I'd really appreciate it if you could throw in your two cents.15:22
*** gyee has joined #openstack-horizon15:23
amotokimlycka: we are in the team meeting at #openstack-meeting-alt15:24
amotokimlycka: I know you are waiting our replies. only Radomir replied to your thread.15:26
mlyckaamotoki: Yeah, no problem. I'd just like your opinions so that it doesn't just become me and Radomir exchanging opinions.15:26
openstackgerritMerged openstack/horizon master: Remove unused 'not_list' template argument  https://review.opendev.org/68103415:27
openstackgerritMerged openstack/horizon master: django22: release note and docs update  https://review.opendev.org/68139515:27
*** e0ne has quit IRC15:42
*** e0ne has joined #openstack-horizon15:42
*** ChanServ sets mode: +o e0ne15:42
*** mlycka has quit IRC15:47
*** spatel has joined #openstack-horizon16:02
*** spatel has quit IRC16:07
vishalmanchandae0ne: will review your patch https://review.opendev.org/#/c/673807/ tomorrow morning.16:41
e0nevishalmanchanda: thank you16:41
vishalmanchandamy lappy battery will run out16:41
vishalmanchandaand i forgot my charger in office.16:41
e0nethat's why I've got two chargers16:44
openstackgerritMerged openstack/horizon master: PDF documentation build  https://review.opendev.org/66796816:53
*** e0ne has quit IRC16:54
*** tesseract has quit IRC16:57
*** awalende has joined #openstack-horizon17:03
*** awalende has quit IRC17:07
*** spsurya has quit IRC17:32
*** e0ne has joined #openstack-horizon17:55
*** ChanServ sets mode: +o e0ne17:55
*** e0ne has quit IRC17:59
*** e0ne has joined #openstack-horizon18:17
*** ChanServ sets mode: +o e0ne18:17
*** early has quit IRC18:37
*** early has joined #openstack-horizon18:41
*** rdopiera has quit IRC19:32
*** vishalmanchanda has quit IRC19:48
*** e0ne has quit IRC20:06
*** BjoernT has quit IRC20:08
*** e0ne has joined #openstack-horizon20:15
*** ChanServ sets mode: +o e0ne20:15
*** e0ne has quit IRC20:16
*** e0ne has joined #openstack-horizon20:16
*** ChanServ sets mode: +o e0ne20:16
*** mvkr has quit IRC20:27
*** e0ne has quit IRC20:29
*** e0ne_ has joined #openstack-horizon20:29
*** mvkr has joined #openstack-horizon20:32
*** BjoernT has joined #openstack-horizon20:47
*** BjoernT has quit IRC20:51
*** BjoernT_ has joined #openstack-horizon20:51
*** rcernin has joined #openstack-horizon21:15
*** e0ne_ has quit IRC21:41
*** whoami-rajat has quit IRC22:11
*** threestrands has joined #openstack-horizon22:37
*** dtruong has quit IRC22:55
*** dtruong has joined #openstack-horizon22:55
*** tkajinam has joined #openstack-horizon23:04
*** BjoernT_ has quit IRC23:18
openstackgerritGloria Gu proposed openstack/horizon master: Updated max-width to be dynamic for .member class  https://review.opendev.org/68136623:26
*** BjoernT has joined #openstack-horizon23:39
*** BjoernT has quit IRC23:48

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