Monday, 2018-04-23

« Sunday, 2018-04-22 Index Tuesday, 2018-04-24 »
*** germs has quit IRC00:00
*** yamamoto has joined #openstack-horizon00:20
*** yamamoto has quit IRC00:24
*** lblanchard has joined #openstack-horizon00:38
*** yamamoto has joined #openstack-horizon00:41
*** hoangcx has joined #openstack-horizon00:46
*** lblanchard has quit IRC00:51
*** dave-mccowan has joined #openstack-horizon01:00
*** dave-mcc_ has joined #openstack-horizon01:03
*** dave-mccowan has quit IRC01:05
openstackgerritAdrian Turjak proposed openstack/horizon master: Create new wsgi.py file and deprecate old file  https://review.openstack.org/56180201:14
*** namnh has joined #openstack-horizon01:17
*** sapd has joined #openstack-horizon01:43
openstackgerritAdrian Turjak proposed openstack/horizon master: Create new wsgi.py file and deprecate old file  https://review.openstack.org/56180202:14
openstackgerritAdrian Turjak proposed openstack/horizon master: Create new wsgi.py file and deprecate old file  https://review.openstack.org/56180202:37
openstackgerritAdrian Turjak proposed openstack/horizon master: Create new wsgi.py file and deprecate old file  https://review.openstack.org/56180202:38
*** dklyle has joined #openstack-horizon02:41
*** ChanServ sets mode: +o dklyle02:41
adriantrdopiera, amotoki: updated the wsgi patch. please review when you can :)02:46
*** dklyle has quit IRC03:00
*** dangtrinhnt has quit IRC03:09
*** rcernin has quit IRC03:31
*** rcernin has joined #openstack-horizon03:35
*** dave-mcc_ has quit IRC04:00
*** shu-mutow-AWAY is now known as shu-mutow04:22
*** btully has joined #openstack-horizon04:33
*** btully has quit IRC04:38
*** jtomasek has joined #openstack-horizon05:12
*** btully has joined #openstack-horizon05:39
*** btully has quit IRC05:59
*** e0ne has joined #openstack-horizon06:02
*** ChanServ sets mode: +o e0ne06:02
*** lajoskatona has joined #openstack-horizon06:14
*** cshen has joined #openstack-horizon06:27
*** masco has joined #openstack-horizon06:32
*** ircuser-1 has joined #openstack-horizon06:39
*** e0ne has quit IRC06:49
*** tesseract has joined #openstack-horizon07:04
*** katyafervent_awa has quit IRC07:06
*** katyafervent_awa has joined #openstack-horizon07:10
*** sapd has quit IRC07:13
*** rcernin has quit IRC07:19
*** pcaruana has joined #openstack-horizon07:25
*** cshen has quit IRC07:26
*** tosky has joined #openstack-horizon07:28
*** btully has joined #openstack-horizon07:44
*** btully has quit IRC07:48
*** nguyenhai has joined #openstack-horizon07:54
*** jpich has joined #openstack-horizon07:58
*** shu-mutow is now known as shu-mutow-AWAY08:04
*** threestrands has quit IRC08:17
*** sapd has joined #openstack-horizon08:53
*** e0ne has joined #openstack-horizon09:03
*** ChanServ sets mode: +o e0ne09:03
*** namnh has quit IRC09:22
*** namnh has joined #openstack-horizon09:23
e0neamotoki: thanks a lot for the review my patch with django test runner09:25
*** hoangcx has quit IRC09:26
*** hoangcx has joined #openstack-horizon09:26
openstackgerritIvan Kolodyazhny proposed openstack/horizon master: Use default Django test runner instead of nose  https://review.openstack.org/54429609:51
*** rmart04 has joined #openstack-horizon09:53
*** e0ne changes topic to "OpenStack Horizon | Rocky PTG: https://etherpad.openstack.org/p/horizon-ptg-rocky | Rocky-2 priorities: https://launchpad.net/horizon/+milestone/rocky-2 | Docs: https://docs.openstack.org/horizon/latest/"10:05
openstackgerritIvan Kolodyazhny proposed openstack/horizon master: Fix selenuim-headless tests  https://review.openstack.org/55846710:08
*** hoangcx has quit IRC10:09
amotokie0ne: https://review.openstack.org/#/c/558048/10:12
amotokie0ne: if it lands I will send an announcement to the ML following my mail last month on mox removal plan in horizon10:13
*** namnh has quit IRC10:17
e0neamotoki: looking on it10:21
*** pcaruana has quit IRC10:26
*** yamamoto has quit IRC10:31
*** pcaruana has joined #openstack-horizon10:33
*** e0ne has quit IRC10:35
*** e0ne_ has joined #openstack-horizon10:35
*** ChanServ sets mode: +o e0ne_10:35
openstackgerritMerged openstack/horizon master: Disable mox usage by default in test helper  https://review.openstack.org/55804810:37
amotoki^^^ yay11:01
*** hoonetorg has quit IRC11:03
e0ne_I tried to remove nose from horizon plugins11:05
amotokie0ne_: which topic are you using?11:05
e0ne_https://review.openstack.org/#/q/topic:bp/improve-horizon-testing+(status:open+OR+status:merged)11:05
e0ne_everything went good, except freezer and murano11:06
e0ne_murano-dashboard tests are broken for py3511:06
*** e0ne_ is now known as e0ne11:07
amotokie0ne: a mail on mox removal update: http://lists.openstack.org/pipermail/openstack-dev/2018-April/129648.html11:08
amotokie0ne: so what is the plan of your patch series? you propose patches so that horizon plugins consumes horizon.test.settings.11:14
amotokiafter that, does this mean that test runners of all horizon plugins will be changed if we change the test runner in horizon.test.settings?11:14
e0neamotoki: it should be up to plugin maintainers to override test runner. I remove nose because it does a lot of  false positive reports11:15
e0neso we can't be sure that something works even if nose reports OK11:16
*** hoonetorg has joined #openstack-horizon11:16
e0nethat's why I remove nose as a first step to see what is going11:16
e0neI want to have cross-project jobs to test plugins pear each horizon tests11:17
e0newe can use unit-tests as a beginning and selenium tests in the futrure11:17
e0neso plugin maintainers could implement own selenium based tests11:18
amotokie0ne: I am a bit confused. in case of your designate-dashboard patch, https://review.openstack.org/#/c/562525/3/designatedashboard/tests/settings.py11:18
amotokie0ne: L.21 imports all attributes in horizon.test.settings11:18
amotokiand horizon.test.settings defines TEST_RUNNER at https://github.com/openstack/horizon/blob/master/horizon/test/settings.py#L10811:18
amotokiso I wonder horizon plugin maintainers can choose test runner without overriding settings related to testing.11:19
e0nesure. they need to override horizon's settings to change  test runner or anythin else11:20
e0nes/anythin/anything11:20
*** hoonetorg has quit IRC11:21
*** masco has quit IRC11:21
amotokie0ne: horizon.test.settings requires django_nose in INSTALLED_APPS, so if other horizon plugins try to use other test runners. does they need to do something like L.47 in https://review.openstack.org/#/c/558197/3/neutron_vpnaas_dashboard/test/settings.py ?11:22
*** btully has joined #openstack-horizon11:22
amotokie0ne: I just would like to clarify what is the recommended way during this test runner switch.11:23
e0neamotoki: yes, it works in the same way as local_settings.py11:23
e0neamotoki: I think, we need to add a note to the documentation about test runners11:24
e0nemy idea is to get a rid of nose, because it's unsupported and has a lot of bugs11:25
e0nebut the plugins teams could user anything they want in the future11:25
e0nehorizon should provides good enough default test runner to cover most cases11:26
*** btully has quit IRC11:26
e0nedoes it make sense?11:26
mugsiee0ne: amotoki it is worth noting a lot of the plugins cargo cult from one another, and have no idea how any of this works :) - so any extra docs would be ++11:27
*** yamamoto has joined #openstack-horizon11:31
e0nemugsie: good point11:32
e0nemugsie: now I remove nose only because it works not like expected:(11:32
*** yamamoto has quit IRC11:37
*** hoonetorg has joined #openstack-horizon11:39
openstackgerritAkihiro Motoki proposed openstack/horizon master: Drop Flavor Edit completely  https://review.openstack.org/56358611:48
*** pcaruana has quit IRC11:54
*** yamamoto has joined #openstack-horizon12:05
*** yamamoto has quit IRC12:06
*** pcaruana has joined #openstack-horizon12:10
*** doxa has quit IRC12:10
*** vladiskuz has joined #openstack-horizon12:13
vladiskuzamotoki: Hi! are you here?12:14
*** vladiskuz has quit IRC12:14
*** vladiskuz has joined #openstack-horizon12:15
*** yamamoto has joined #openstack-horizon12:37
*** yamamoto has quit IRC12:42
*** dave-mccowan has joined #openstack-horizon12:49
*** yamamoto has joined #openstack-horizon12:51
*** yamamoto has quit IRC12:55
*** vladiskuz has quit IRC13:14
*** jpich has quit IRC13:30
*** lvdombrkr has joined #openstack-horizon13:30
*** r-daneel has joined #openstack-horizon13:37
*** felipemonteiro_ has joined #openstack-horizon13:38
*** jpich has joined #openstack-horizon13:38
*** felipemonteiro__ has joined #openstack-horizon13:44
*** felipemonteiro_ has quit IRC13:47
*** jtriley has joined #openstack-horizon13:48
*** r-daneel has quit IRC13:51
*** yamamoto has joined #openstack-horizon13:51
*** rmart04 has quit IRC13:51
*** rmart04 has joined #openstack-horizon13:52
*** yamamoto has quit IRC13:57
*** rmart04 has quit IRC13:57
*** rmart04 has joined #openstack-horizon13:57
*** felipemonteiro_ has joined #openstack-horizon14:00
*** felipemonteiro__ has quit IRC14:00
*** tsmith2 has joined #openstack-horizon14:05
amotokie0ne: final cleanup of "Edit Flavor" https://review.openstack.org/#/c/563586/14:11
amotokie0ne: we missed it when your patch dropped the feature.14:11
*** lblanchard has joined #openstack-horizon14:11
e0neamotoki: thanks! it's on my TODOs list for tonight14:11
amotokie0ne: no worries. it is not urgent.14:12
*** lblanchard has quit IRC14:12
*** felipemonteiro__ has joined #openstack-horizon14:20
*** felipemonteiro_ has quit IRC14:24
*** lvdombrkr89 has joined #openstack-horizon14:37
*** lvdombrkr has quit IRC14:40
*** lblanchard has joined #openstack-horizon14:46
*** lvdombrkr89 has quit IRC14:47
*** felipemonteiro__ has quit IRC14:51
*** felipemonteiro__ has joined #openstack-horizon14:51
*** yamamoto has joined #openstack-horizon14:53
*** r-daneel has joined #openstack-horizon14:53
*** yamamoto has quit IRC14:58
*** btully has joined #openstack-horizon14:59
*** btully has quit IRC15:04
*** rmart04 has quit IRC15:06
*** felipemonteiro_ has joined #openstack-horizon15:09
*** felipemonteiro__ has quit IRC15:13
*** jtriley has quit IRC15:16
*** pcaruana has quit IRC15:19
amotokiis the meeting agenda page still honored?15:20
*** felipemonteiro_ has quit IRC15:20
*** felipemonteiro_ has joined #openstack-horizon15:20
amotokithe page is not shared at recent meetings, so I wonder what is the best way to add some to the agenda.15:21
e0neamotoki: I use https://wiki.openstack.org/wiki/Meetings/Horizon for meetings agenda15:21
amotokie0ne: thanks. I will add some topics to the page.15:22
amotoki(though I am not 100% sure I can join the meeting this week)15:22
e0ne:(15:22
amotokiI have a local meetup as usual this week15:22
amotokis/this week/this Wed/15:23
amotokiI have neutron drivers meeting on every Friday night, so I tend to schedule local meetups or parties on Wednesday15:24
amotokithat's the reason I asked you whether we change day of week or not when rearranging the meeting time slot.15:25
e0neI understand...15:25
amotokiI will try to share the background in advance.15:25
e0neit's really hard to get all of us in one time15:26
*** lajoskatona has quit IRC15:27
*** dklyle has joined #openstack-horizon15:32
*** ChanServ sets mode: +o dklyle15:32
amotokiyeah exactly.15:33
*** jtriley has joined #openstack-horizon15:46
toskya new contributor posted nice patches to remove mox support from sahara-dashboard, but they are failing in the lower-constraints.txt test, because it forces horizon==13.0.015:47
toskyand I suspest that the patches relies on some feature from horizon master15:47
toskydid anyone already face this issue in out-of-horizon dashboards and know how to fix lower-constraints.txt for this to work?15:48
tosky(or I will nag dhellmann...)15:48
e0netosky: could you please provide a link to the patch?15:48
*** dklyle has quit IRC15:49
e0netosky: need to head off. I can take a look on it in ~3 hours15:51
toskye0ne: https://review.openstack.org/#/c/562892/15:51
*** yamamoto has joined #openstack-horizon15:54
*** e0ne has quit IRC15:54
*** yamamoto has quit IRC16:00
*** gyee has joined #openstack-horizon16:03
*** pcaruana has joined #openstack-horizon16:13
openstackgerritAkihiro Motoki proposed openstack/horizon master: Drop Flavor Edit completely  https://review.openstack.org/56358616:19
*** jpich has quit IRC16:30
*** btully has joined #openstack-horizon16:48
*** btully has quit IRC16:53
*** yamamoto has joined #openstack-horizon16:56
*** yamamoto has quit IRC17:01
amotokitosky: required-projects need to be added. https://github.com/openstack/neutron-vpnaas-dashboard/blob/master/.zuul.yaml is for neutron-vpnaas-dashboard17:06
toskyamotoki: to which job?17:12
amotokitosky: lower-constraints17:12
amotokitosky: tox-siblings will install horizon master if required-projects is specified to lower-constraints job.17:13
toskyamotoki: oh, I see - because that would enable LIBS_FROM_GIT for horizon? But isn't cheating a bit, overriding lower-constraints in fact?17:13
amotokitosky: ?? LIBS_FROM_GIT only works for devstack. lower-constraints is not devstack job but tox job.17:14
toskyright17:14
toskybut the idea is similar17:14
amotokiright17:14
toskyso basically it will ignore the value of lower-constraints, so I though it's a bit cheating because then lower-constraints wouldn't reflect the real value17:15
toskyand we would need to update the value before the release17:15
toskyor we may forget it17:15
toskyand everything would continue to work, but lower-constraints.txt for the horizon dependency wouldn't be true anymore17:16
toskythe value of horizon in lower-constraints.txt*17:16
amotokihorizon==13.0.0 is ignored if required-projcts is specified17:16
toskyyes, exactly: but let's assume that I add required-project; everything will work, then horizon 14 is released and I need it17:17
amotokibut it is due to the some transient situation I think17:17
toskyso I should update lower-constraints.txt, but nothing will remember me to do it17:17
amotokithe infra was changed suddenly regardless of the current horizon release model17:17
toskyso I may forget it, everything will continue to work, but the contents of lower-constraints.txt won't be correct anymore17:17
toskyI guess I will ping dhellmann :D17:18
amotokiand it seems they suggest to switch horizon rel models to cycle-with-intermediate17:18
toskyoh, that will make sense too17:18
amotokior  to split out common parts which plugins depends on into some libraries.17:18
amotokiit will take time17:18
toskyso you would release an intermediate version, then we can switch to that, everyone is happy17:18
amotokiI have no good idea on how to move this forward in short term17:19
toskybut we may need to remember to remove required-project, or the change would be hidden17:19
amotokiI would like to see more wide consensus on this17:19
toskyso do you think it's something I could re-raise again with -infra? Because other projects may be affected17:19
amotokiI understand the whole picture on what is happening, but personally I have no time to move this forward.17:20
toskyI understand that17:20
amotokitosky: yeah it needs to be discussed with the infra team17:20
amotokiversioning change is a bit surprising to consumers.17:21
amotoki13.0.0 is Queens release, but for example 15.6.0 can be Rocky release.17:21
toskybut thanks for the explanation and the hints; I can easily add required-project, but I'd like a more robust solution17:21
*** mvk has quit IRC17:22
amotoki* if we switch a versioning model to the semantic versioning more strictly.17:22
amotokiEach backward-incompat change (i.e., dropping some feature) will bump major version.17:22
*** tesseract has quit IRC17:23
*** e0ne has joined #openstack-horizon17:26
*** ChanServ sets mode: +o e0ne17:26
*** yamamoto has joined #openstack-horizon17:58
*** r-daneel has quit IRC18:00
*** r-daneel has joined #openstack-horizon18:01
*** yamamoto has quit IRC18:02
*** felipemonteiro__ has joined #openstack-horizon18:08
*** felipemonteiro_ has quit IRC18:12
*** darland has quit IRC18:35
*** harlowja has joined #openstack-horizon18:44
*** yamamoto has joined #openstack-horizon18:59
*** yamamoto has quit IRC19:03
*** felipemonteiro__ has quit IRC19:06
*** felipemonteiro__ has joined #openstack-horizon19:07
*** harlowja has quit IRC19:07
*** dklyle has joined #openstack-horizon19:11
*** ChanServ sets mode: +o dklyle19:11
*** imacdonn has quit IRC19:13
*** imacdonn has joined #openstack-horizon19:14
toskyamotoki: I may have temporarily solved by pushing the requirement on the 14.0.0.0b1, available on pypi: https://review.openstack.org/#/c/563733/19:47
toskyamotoki: apart from that, you may have read the discussion on -infra19:47
*** yamamoto has joined #openstack-horizon19:59
*** yamamoto has quit IRC20:05
*** dave-mccowan has quit IRC20:14
*** btully has joined #openstack-horizon20:26
*** felipemonteiro__ has quit IRC20:29
*** btully has quit IRC20:30
*** jtomasek has quit IRC20:43
*** mvk has joined #openstack-horizon20:45
*** idlemind has quit IRC20:59
*** lblanchard has quit IRC21:06
*** jtriley has quit IRC21:10
*** dklyle has quit IRC21:31
*** r-daneel_ has joined #openstack-horizon21:35
*** r-daneel has quit IRC21:36
*** r-daneel_ is now known as r-daneel21:36
*** yamamoto has joined #openstack-horizon22:02
*** yamamoto has quit IRC22:07
*** jrist has quit IRC22:08
*** jrist has joined #openstack-horizon22:12
*** jrist has quit IRC22:12
*** jrist has joined #openstack-horizon22:12
*** e0ne has quit IRC22:14
*** lbragstad has quit IRC22:18
openstackgerritMerged openstack/horizon master: Allow OVA upload for images  https://review.openstack.org/55457222:19
openstackgerritMerged openstack/horizon master: Add identity API version discovery  https://review.openstack.org/55942922:20
*** dklyle has joined #openstack-horizon22:26
*** ChanServ sets mode: +o dklyle22:26
*** rcernin has joined #openstack-horizon22:31
*** dklyle has quit IRC22:39
*** threestrands has joined #openstack-horizon22:41
*** threestrands has quit IRC22:41
*** threestrands has joined #openstack-horizon22:41
*** felipemonteiro__ has joined #openstack-horizon22:42
*** lblanchard has joined #openstack-horizon23:03
*** yamamoto has joined #openstack-horizon23:04
*** felipemonteiro__ has quit IRC23:07
*** yamamoto has quit IRC23:10
*** r-daneel has quit IRC23:21
*** yamamoto has joined #openstack-horizon23:26
*** tosky has quit IRC23:54
« Sunday, 2018-04-22 Index Tuesday, 2018-04-24 »

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