Wednesday, 2021-03-10

*** vishalmanchanda has quit IRC00:05
*** tosky has quit IRC00:36
*** iurygregory_ has joined #openstack-horizon01:30
*** another_larsks has joined #openstack-horizon01:31
*** iurygregory has quit IRC01:38
*** larsks has quit IRC01:39
*** jamesdenton has joined #openstack-horizon03:42
*** jamesdenton has quit IRC03:50
*** jamesdenton has joined #openstack-horizon03:51
*** gyee has quit IRC03:58
*** vishalmanchanda has joined #openstack-horizon04:59
*** jamesdenton has quit IRC05:57
*** jamesden_ has joined #openstack-horizon05:58
*** e0ne has joined #openstack-horizon06:31
*** ChanServ sets mode: +o e0ne06:31
*** e0ne has quit IRC06:31
*** e0ne has joined #openstack-horizon07:22
*** ChanServ sets mode: +o e0ne07:22
*** e0ne has quit IRC07:27
*** BakaKuna has joined #openstack-horizon07:46
*** dklyle has quit IRC07:48
*** rdopiera has joined #openstack-horizon07:56
*** rpittau|afk is now known as rpittau08:27
*** tosky has joined #openstack-horizon08:46
*** rcernin has quit IRC09:54
*** BakaKuna has quit IRC10:00
*** BakaKuna has joined #openstack-horizon10:02
*** e0ne has joined #openstack-horizon10:35
*** ChanServ sets mode: +o e0ne10:35
*** iurygregory_ is now known as iurygregory11:33
*** disap has joined #openstack-horizon12:05
*** kevko has joined #openstack-horizon12:06
*** BakaKuna has quit IRC12:53
openstackgerritVishal Manchanda proposed openstack/horizon master: [DNM] Testing Plugin-related unit tests  https://review.opendev.org/c/openstack/horizon/+/77977014:02
*** BakaKuna has joined #openstack-horizon14:06
*** tmazur has joined #openstack-horizon14:07
*** jamesden_ is now known as jamesdenton14:09
*** disap has quit IRC14:15
*** BakaKuna1 has joined #openstack-horizon14:33
*** BakaKuna has quit IRC14:35
*** BakaKuna1 is now known as BakaKuna14:35
*** fnordahl has quit IRC14:42
openstackgerritVishal Manchanda proposed openstack/horizon master: [DNM] Testing Plugin-related unit tests  https://review.opendev.org/c/openstack/horizon/+/77977015:02
*** jamesdenton has quit IRC15:35
*** jamesdenton has joined #openstack-horizon15:36
*** BakaKuna has quit IRC15:37
*** dklyle has joined #openstack-horizon15:43
*** ikanias has joined #openstack-horizon16:00
-amotoki- moving here16:00
tmazurSo what do you think of user-domain-password order?16:01
amotokiif so, I prefer to user-password-domain16:01
ikaniastmazur: I still stick to username/pass/domain16:01
amotokias we don't need to move the extra field16:01
amotokithe usual pattern is to typy username, TAB (to the password field) and type password.16:02
amotokis/typy/type/16:02
ikaniastamotoki: exactly16:02
e0neamotoki: +116:02
tmazure0ne: what is your opinion on user-password-domain?16:02
e0neI'm ok for it16:02
amotokiI am okay to move the domain field at the bottom. it looks like it addresses user frustration.16:03
amotokiit can also avoid a new setting.16:03
tmazurOkay then it seems we have an agreement16:03
ikaniaslike amotoki said it does not seperate username/password16:04
ikaniaswhich is important16:04
tmazurSo I am simply moving Domain field to the bottom, no new setting16:04
ikaniastmazur: +116:04
tmazurAnd we can consider it as a bugfix now16:04
amotokiif we keep the order of domain->user->password and focus "user" by default, users are forced to move the cursor up and down if the domain field is empty16:05
tmazuramotoki: true16:05
e0neamotoki: or we can focus on domain field (the fist field) by default16:05
tmazure0ne: we have it now :)16:06
tmazure0ne: the point was to shift it to user name since we have a customer's request for this16:06
amotokie0ne: but the problem we are tackling is that focusing on domain field is unnecessary when the domain field is already filled (in case of second or later login)16:06
e0neI understand it...16:07
e0neso moving domain field to to bottom sounds good, we just need to be sure that we can submit a form by pressing 'enter' button16:08
amotokiso I see only two choicse: (1) move "domain" to the bottom (2) set the default focus to "domain" if empty and to "user" if "domain" is filled16:08
tmazurLet's vote maybe?16:09
amotokithrough our discussion (1) sounds simpler16:09
ikaniasamotoki: it seems to me the first option is more simple or am i wrong16:09
ikaniasamotoki: testing it will be simpler too :-)16:10
amotokiI just rewind our dissuccion and clarify the options we have as I believe it moves us to the agreement.16:10
amotokiI agree (1) looks simpler, so my vote is (1) now.16:11
e0ne+116:11
ikanias+116:11
tmazur(1) is simple indeed, and we can even simplify the existing code with it16:11
tmazurso +116:11
vishalmanchandavote for option 1.16:11
amotokisounds fine :)16:11
amotokiwe seem to reach the consensus16:12
amotokilet's move the domain field to the bottom16:12
tmazurOk cool :) Thanks everyone for your time!16:12
e0necan we discuss one more patch?16:12
amotokiit was nice to cover various options and compare them :)16:13
amotokie0ne: go ahead16:13
tmazursure16:13
e0nehttps://review.opendev.org/c/openstack/horizon/+/77912516:13
e0nesince we agreed to do it on the last meeting, I'm OK to merge it16:13
e0nebut we shouldn't do such things just before feature freeze in the future16:13
e0nemaybe we should agree to add deprecations only before milestone-216:14
tmazure0ne: will it be Horizon's way or other projects have similar agreements?16:15
amotokie0ne: the deprecation does not mean a corresponding feature will be dropped in the next release, so I think it is acceptable16:15
amotokiIf you see any feature gap, please file a bug to horizon or contact the horizon team.16:15
e0netmazur: I don't know if other projects do this16:15
e0nebut since horizonn does several releases during release cycle it's good to have some agreement on deprecations16:16
amotokiI added the above notice in the release note.16:16
e0neamotoki: sure, I'm not going to block it. just want to raise this topic16:17
amotokie0ne: yeah, good point16:17
e0newe can discuss it on PTG16:17
amotokiin case of neutron, we don't block any deprecation before the feature freeze.16:17
vishalmanchandaI also have a topic to discuss.16:18
amotokieven after the feature freeze, we sometimes add a deprecation notice if needed. they does not change any features.16:18
amotokiDoes discussing it in PTG mean we defer the deprecation to Xena?16:19
e0neno16:19
e0newe agreed to deprecate it16:19
e0neI approved it16:21
amotokiI can add more note like "we plan to drop it in 'Y' release at latest" in the relnote16:21
amotokito clarify we need feedbacks from operators.16:21
-amotoki- "at latest" might be wrong usage....16:22
*** gyee has joined #openstack-horizon16:22
amotokianyway we can decide when we drop it carefully16:23
e0ne+116:23
vishalmanchandaI am investigating a bug today i.e. https://bugs.launchpad.net/horizon/+bug/1866666 .16:25
openstackLaunchpad bug 1866666 in OpenStack Dashboard (Horizon) "Plugin-related unit tests depend on import and test order" [Undecided,New] - Assigned to Ivan Kolodyazhny (e0ne)16:25
amotokiperhaps https://governance.openstack.org/tc/reference/tags/assert_follows-standard-deprecation.html is a good document to read.16:25
vishalmanchandabut I am not able to run these test on my local env. https://github.com/openstack/horizon/blob/master/openstack_dashboard/test/test_plugins/test_panel.py#L5016:26
amotokivishalmanchanda: have you got any finding?16:26
vishalmanchandahere is my error logs https://pastebin.com/tCsE2hQF16:27
amotokivishalmanchanda: this happens when we run horizon UT in parallel (I mean "with muptiple processes")16:27
amotokivishalmanchanda: it depends on the number of processes of pytest.16:27
amotokivishalmanchanda: I don't remember the right number but it happened when I tried the number of processes of pytest from all numbers from 2 to 8.16:28
amotokivishalmanchanda: perhaps one of 2,3,4 triggered the failure.16:29
vishalmanchandaamotoki: ok, do you have any doc. link for that.16:30
vishalmanchandaIf I want to run it on my local env. and want to debug it how can I do that16:30
amotokivishalmanchanda: I cannot remember the right comments on this, but perhaps bug 1809983 referred in the bug you post16:35
openstackbug 1809983 in OpenStack Dashboard (Horizon) "unittest: NoReverseMatch for test panels sometimes happens" [High,Fix released] https://launchpad.net/bugs/1809983 - Assigned to Akihiro Motoki (amotoki)16:35
amotokiin addition,  my comment on testing results in https://review.opendev.org/c/openstack/horizon/+/730289 may help you.16:36
amotokiperhaps the first is more related.16:36
vishalmanchandaamotoki: ok thanks for the links will look into that.16:37
amotokibut both are related in the context that it happens when we run UT in parallel or run a specific set of tests.16:37
*** takamatsu has quit IRC16:37
vishalmanchandaamotoki: ok got your point.16:38
*** rpittau is now known as rpittau|afk17:05
*** takamatsu has joined #openstack-horizon17:38
openstackgerritMerged openstack/horizon master: Deprecate Django launch instance form  https://review.opendev.org/c/openstack/horizon/+/77912518:03
*** takamatsu has quit IRC18:10
*** BakaKuna has joined #openstack-horizon18:22
*** tbarron has joined #openstack-horizon18:37
*** e0ne has quit IRC18:48
*** rdopiera has quit IRC18:49
*** BakaKuna has quit IRC19:29
*** mugsie__ has joined #openstack-horizon19:37
*** tosky_ has joined #openstack-horizon19:37
*** benj_- has joined #openstack-horizon19:37
*** tosky has quit IRC19:42
*** benj_ has quit IRC19:42
*** mugsie has quit IRC19:42
*** irclogbot_2 has quit IRC19:44
*** jamesdenton has quit IRC19:45
*** mugsie__ is now known as mugsie19:47
*** jamesdenton has joined #openstack-horizon19:48
*** ysirndjuro has joined #openstack-horizon19:52
ysirndjuroHello. Is there a way to pregenerate static and cache files for Horizon? We noticed a long build on initial login into a newly provisioned cloud, as well as the first time a user/admin loads each component tab. Theoretically it could be done by using a standalong requests python module to emulate a login and hitting each page, but was wondering if manage.py in the horizon container already supported functionality to bui19:54
*** tosky_ is now known as tosky20:16
*** takamatsu has joined #openstack-horizon20:25
*** tosky has quit IRC20:49
*** tosky has joined #openstack-horizon20:49
openstackgerritTatiana Ovchinnikova proposed openstack/horizon master: Move "Domain" field under "User Name" and "Password"  https://review.opendev.org/c/openstack/horizon/+/77769020:58
*** dklyle has quit IRC21:10
*** NobodyCam has quit IRC21:39
*** NobodyCam has joined #openstack-horizon21:40
*** dklyle has joined #openstack-horizon21:42
*** johnsom has quit IRC21:46
*** johnsom has joined #openstack-horizon21:46
*** vishalmanchanda has quit IRC22:08
*** rcernin has joined #openstack-horizon22:27
*** ysirndjuro has left #openstack-horizon23:20
*** ysirndjuro has quit IRC23:20
*** rpittau|afk has quit IRC23:37
*** rpittau|afk has joined #openstack-horizon23:37

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!