mtreinish | timothyb89, andreaf: ^^^ | 00:00 |
---|---|---|
mtreinish | I haven't tested the tox command yet, but those were the basic local steps I performed | 00:00 |
mtreinish | aside from the workaround I had to do because of bad packages that didn't know what to do with python == python3 on archlinux | 00:00 |
*** yamahata has quit IRC | 00:01 | |
*** edmondsw has quit IRC | 00:01 | |
*** ccarrara has quit IRC | 00:04 | |
*** ccarrara has joined #openstack-qa | 00:06 | |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Add tox env for running a local dev instance of the dashboard https://review.openstack.org/230209 | 00:06 |
*** dramalho has quit IRC | 00:08 | |
*** tsekiyam_ has joined #openstack-qa | 00:09 | |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Add tox env for running a local dev instance of the dashboard https://review.openstack.org/230209 | 00:11 |
*** ccarrara has quit IRC | 00:11 | |
*** mtanino has quit IRC | 00:12 | |
*** tsekiyama has quit IRC | 00:13 | |
*** tsekiyam_ has quit IRC | 00:14 | |
openstackrecheck | Console logs not available after 13:26s for gate-keystone-pep8 230193,1,23327e7 | 00:16 |
*** Kennan has quit IRC | 00:16 | |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Add tox env for running a local dev instance of the dashboard https://review.openstack.org/230209 | 00:19 |
*** jhenner has quit IRC | 00:23 | |
*** markvoelker has joined #openstack-qa | 00:30 | |
*** pvaneck has quit IRC | 00:32 | |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Add tox env for running a local dev instance of the dashboard https://review.openstack.org/230209 | 00:33 |
mtreinish | timothyb89, andreaf: ^^^ ok tested that locally it seems to work fine | 00:34 |
mtreinish | err except on cleanup, it leaves openstack-health running | 00:34 |
mtreinish | I'll have to save that for tomorrow | 00:34 |
*** zz_dimtruck is now known as dimtruck | 00:35 | |
*** markvoelker has quit IRC | 00:35 | |
*** tpeoples has quit IRC | 00:38 | |
*** hoangcx has joined #openstack-qa | 00:40 | |
*** ranger81 has quit IRC | 00:42 | |
*** yamamoto has joined #openstack-qa | 00:43 | |
*** achanda has quit IRC | 00:47 | |
*** yamamoto has quit IRC | 00:48 | |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Use live data on dashboard homepage. (WIP) https://review.openstack.org/224992 | 00:49 |
openstackrecheck | Console logs not available after 13:26s for gate-nova-docs 229964,5,d2d95a4 | 00:52 |
*** mylu has joined #openstack-qa | 00:59 | |
*** hichihara has joined #openstack-qa | 00:59 | |
*** ranger81 has joined #openstack-qa | 01:01 | |
*** sabeen2 has joined #openstack-qa | 01:03 | |
oomichi | mtreinish: hi, sorry for pinging you late night | 01:04 |
*** sabeen1 has quit IRC | 01:05 | |
*** ranger81 has quit IRC | 01:06 | |
mtreinish | oomichi: why apologize it's only 9pm. It's my fault for still being online :) | 01:06 |
*** dimtruck is now known as zz_dimtruck | 01:06 | |
oomichi | mtreinish: yeah, that is late night. you are always online as account | 01:07 |
oomichi | mtreinish: anyways, can you take a look at https://review.openstack.org/#/c/217034 ? | 01:07 |
oomichi | mtreinish: that is blocking service client migration | 01:07 |
oomichi | mtreinish: and LGTM, but all reviewers/writer is my company | 01:07 |
*** yamahata has joined #openstack-qa | 01:08 | |
mtreinish | oomichi: +A | 01:09 |
oomichi | mtreinish: thanks! :) | 01:10 |
mtreinish | oomichi: I'll trade you for: https://review.openstack.org/212691 | 01:11 |
mtreinish | I think it would be good to get that patch in the next release | 01:11 |
mtreinish | which at the rate you're chugging through service client migrations should be pretty soon | 01:12 |
*** thanhnt-z has joined #openstack-qa | 01:12 | |
oomichi | mtreinish: of course, will see it now :) | 01:12 |
oomichi | mtreinish: +A | 01:15 |
*** davideagnello has quit IRC | 01:16 | |
oomichi | mtreinish: btw, proposals are not so many for qa sessions of tokyo summit now | 01:17 |
oomichi | mtreinish: https://etherpad.openstack.org/p/mitaka-qa-summit-topics | 01:18 |
oomichi | mtreinish: 8 proposals in 8 slots | 01:18 |
oomichi | mtreinish: all proposals will be +A in current situation | 01:18 |
*** Swami has quit IRC | 01:18 | |
oomichi | mtreinish: do you have more ideas/proposals? | 01:18 |
*** woodster_ has quit IRC | 01:19 | |
gmann | oomichi: mtreinish : 1 idea but not sure if that can be done in M or later | 01:21 |
*** ranger81 has joined #openstack-qa | 01:21 | |
gmann | This is regarding nova extension option in config | 01:21 |
oomichi | gmann: I'd like to know it :) | 01:21 |
*** hoangcx has quit IRC | 01:22 | |
gmann | actually we have extension list option in config and we do really consider that in all tests | 01:22 |
gmann | oomichi: i remember you enabling that for all tests. means all extension present in nova | 01:22 |
gmann | so as Nova has deprecated the extension concept can we do the same in Tempest and run all Nova tests with considering the all extension enable | 01:23 |
oomichi | gmann: yeah, that should be right direction in long-term | 01:24 |
gmann | or we need to take care of extension enable bits in all tests | 01:24 |
gmann | oomichi: but my worry is as tempest need to support Kilo and Liberty also in M where Nova has extension concept | 01:24 |
mtreinish | oomichi: I came up with most of them I need to figure out more :) | 01:24 |
oomichi | mtreinish: I see, I'd like to add one more(api doc from tempest.log) | 01:25 |
mtreinish | gmann: we have to continue to support extensions as long as we support a nova release that has them | 01:25 |
mtreinish | oomichi: sure put it on the list | 01:25 |
oomichi | gmann: yeah, and we need to support legacy v2 api code of nova in multiple cycles | 01:26 |
gmann | mtreinish: humm yea. | 01:26 |
oomichi | mtreinish: yes, right > support extensions | 01:26 |
gmann | actually i saw fixing tests one by one for extension enable check | 01:26 |
oomichi | gmann: ah, I see. at this time, we should not remove extension cheks from tempest | 01:27 |
gmann | so we should keep other tests who does not consider their extension enable thing as it is and later remove extension thing from Tempest too where all Nova stable branches get rid of extension | 01:27 |
oomichi | gmann: since microversions, new extensions should not be added after Kilo. so it is not so hard to keep extension checks on tempest side | 01:28 |
mtreinish | oomichi: well it might be a bit confusing when we have to support a test that is either extensions or microversions (after the microversion is added that removes extensions) | 01:31 |
mtreinish | but that's an unlikely edge case | 01:31 |
gmann | oomichi: yea. just thinking whether it is worth to fix tests to consider extension enable ? which (extension logic) anyways needs to be go away with extension life | 01:32 |
mtreinish | gmann: oh for cases where we're not checking? | 01:32 |
mtreinish | right now | 01:32 |
gmann | mtreinish: yea | 01:32 |
*** zz_dimtruck is now known as dimtruck | 01:33 | |
oomichi | mtreinish: yeah, extensions vs. microversions are making people confused | 01:33 |
mtreinish | gmann: I think we need to just to support older clouds | 01:33 |
gmann | oomichi: yea it be hard for users to configure both at a time | 01:33 |
mtreinish | because we don't want people running the floating ips bulk test if they say that extension isn't enabled | 01:33 |
*** dims has quit IRC | 01:34 | |
gmann | mtreinish: yea | 01:34 |
*** yamahata has quit IRC | 01:34 | |
*** mudassirlatif has quit IRC | 01:34 | |
oomichi | gmann: I will +1 for the patches adds extension checks even now | 01:34 |
mtreinish | oomichi: not +2 :) | 01:36 |
gmann | :) | 01:36 |
gmann | oomichi: or i was thinking to do those based on how we decide microversion testing and how we will take care of extension in that | 01:37 |
gmann | that's why just wanted to wait on those till microversion topic discussion in summit | 01:38 |
*** jdandrea has quit IRC | 01:39 | |
*** ranger81 has quit IRC | 01:40 | |
*** dimtruck is now known as zz_dimtruck | 01:42 | |
*** mestery has quit IRC | 01:45 | |
*** mestery has joined #openstack-qa | 01:46 | |
oomichi | gmann: yeah, we need to discuss it in microversion test session together | 01:47 |
gmann | oomichi: +1 | 01:48 |
*** achanda has joined #openstack-qa | 01:51 | |
*** ccarrara has joined #openstack-qa | 01:52 | |
*** ccarrara has quit IRC | 01:52 | |
*** moravec has joined #openstack-qa | 01:52 | |
openstackrecheck | Console logs not available after 13:22s for gate-cinder-python34 230199,3,2f86a56 | 01:54 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/227050 failed gate-cinder-python27 in the gate queue with an unrecognized error | 01:55 |
*** ccarrara has joined #openstack-qa | 01:58 | |
*** rfolco has joined #openstack-qa | 02:07 | |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Use live data on dashboard homepage https://review.openstack.org/224992 | 02:12 |
*** mylu has quit IRC | 02:15 | |
*** mylu has joined #openstack-qa | 02:15 | |
*** yamamoto has joined #openstack-qa | 02:21 | |
openstackgerrit | Merged openstack/tempest: Added Test Case for HostsClient https://review.openstack.org/217034 | 02:21 |
*** ccarrara has quit IRC | 02:22 | |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Use live data on dashboard homepage https://review.openstack.org/224992 | 02:23 |
mtreinish | timothyb89: ^^^ I'm still having some trouble getting the gauges working, but the big graph works | 02:25 |
openstackgerrit | Merged openstack/tempest-lib: Provide alternative ssh exec_command for non-linux environments https://review.openstack.org/212691 | 02:25 |
*** mlavalle has quit IRC | 02:25 | |
*** macjack has joined #openstack-qa | 02:27 | |
*** ccarrara has joined #openstack-qa | 02:31 | |
*** markvoelker has joined #openstack-qa | 02:32 | |
oomichi | gmann: btw, can you take a look at https://review.openstack.org/#/c/228672/ ? | 02:32 |
*** yamamoto has quit IRC | 02:34 | |
*** markvoelker has quit IRC | 02:36 | |
gmann | oomichi: looks good, how about changing test_volumes_extensions_client.py name and class name also ? | 02:40 |
openstackgerrit | Reedip proposed openstack/tempest-lib: Migrated hosts_client.py from tempest https://review.openstack.org/230247 | 02:40 |
gmann | it has "extension" word | 02:41 |
oomichi | gmann: nice catch, will update soon | 02:42 |
gmann | oomichi: Thanks | 02:42 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove "extensions" from volumes_extensions_client https://review.openstack.org/228672 | 02:43 |
openstackgerrit | Reedip proposed openstack/tempest-lib: Added a full stop to Autogenerated commit Message https://review.openstack.org/230250 | 02:48 |
openstackgerrit | Reedip proposed openstack/tempest-lib: Added a full stop to auto-generated commit message https://review.openstack.org/230250 | 02:48 |
*** ranger81 has joined #openstack-qa | 02:59 | |
*** hichihara has quit IRC | 02:59 | |
*** Triveni has joined #openstack-qa | 02:59 | |
openstackgerrit | Reedip proposed openstack/tempest-lib: Added a full stop to auto-generated commit message https://review.openstack.org/230250 | 03:00 |
openstackgerrit | Reedip proposed openstack/tempest-lib: Added a full stop to auto-generated commit message https://review.openstack.org/230250 | 03:01 |
*** rfolco has quit IRC | 03:07 | |
*** agireud has joined #openstack-qa | 03:08 | |
openstackrecheck | Console logs not available after 13:23s for gate-nova-python27 230227,1,7d77ea3 | 03:08 |
*** ccarrara has quit IRC | 03:09 | |
openstackgerrit | Kenji Yasui proposed openstack/tempest: Add unit tests for snapshots_client https://review.openstack.org/230254 | 03:12 |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for Pillow 3.0.0 build bug 1501995 https://review.openstack.org/230256 | 03:13 |
openstack | bug 1501995 in OpenStack-Gate "Pillow 3.0.0 breaks the world" [Undecided,Fix released] https://launchpad.net/bugs/1501995 - Assigned to Anita Kuno (anteaya) | 03:13 |
*** macjack has quit IRC | 03:15 | |
*** tpeoples has joined #openstack-qa | 03:15 | |
*** mriedem has quit IRC | 03:15 | |
openstackgerrit | Reedip proposed openstack/tempest: Added Test Cases for ImagesClient https://review.openstack.org/218081 | 03:18 |
*** yamamoto has joined #openstack-qa | 03:23 | |
openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for Pillow 3.0.0 build bug 1501995 https://review.openstack.org/230256 | 03:24 |
openstack | bug 1501995 in OpenStack-Gate "Pillow 3.0.0 breaks the world" [Undecided,Fix released] https://launchpad.net/bugs/1501995 - Assigned to Anita Kuno (anteaya) | 03:24 |
*** salv-orlando has joined #openstack-qa | 03:43 | |
*** salv-orl_ has quit IRC | 03:45 | |
*** tpeoples has quit IRC | 03:57 | |
*** bnemec has quit IRC | 04:02 | |
*** yamahata has joined #openstack-qa | 04:10 | |
*** amotoki has joined #openstack-qa | 04:10 | |
openstackrecheck | openstack/cinder change: https://review.openstack.org/230194 failed gate-tempest-dsvm-full-ceph in the gate queue with an unrecognized error | 04:16 |
*** pcrews has quit IRC | 04:17 | |
*** pcrews has joined #openstack-qa | 04:20 | |
*** yamahata has quit IRC | 04:26 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/devstack-gate: Add script to create a subunit stream for failures before tempest https://review.openstack.org/230263 | 04:26 |
openstackgerrit | Matthew Treinish proposed openstack-infra/devstack-gate: Add script to create a subunit stream for failures before tempest https://review.openstack.org/230263 | 04:31 |
*** markvoelker has joined #openstack-qa | 04:33 | |
openstackgerrit | Reedip proposed openstack/tempest-lib: Added a full stop to auto-generated commit message https://review.openstack.org/230250 | 04:59 |
*** mragupat has joined #openstack-qa | 05:06 | |
openstackrecheck | Console logs not available after 13:24s for gate-nova-tox-functional 217567,28,5605a6c | 05:07 |
*** markvoelker has quit IRC | 05:07 | |
*** mudassirlatif has joined #openstack-qa | 05:11 | |
*** agireud has quit IRC | 05:19 | |
oomichi | gmann: thanks for your review, done on https://review.openstack.org/#/c/228672/ | 05:20 |
openstackrecheck | Console logs not available after 13:21s for gate-tempest-dsvm-neutron-full 220561,14,96c0e07 | 05:20 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove "extensions" from volumes_extensions_client https://review.openstack.org/228672 | 05:22 |
*** agireud has joined #openstack-qa | 05:29 | |
*** ranger81 has quit IRC | 05:31 | |
*** mylu has quit IRC | 05:34 | |
*** ranger81 has joined #openstack-qa | 05:34 | |
*** e0ne has joined #openstack-qa | 05:45 | |
*** hoangcx has joined #openstack-qa | 05:48 | |
*** oomichi has quit IRC | 05:53 | |
*** e0ne has quit IRC | 05:56 | |
*** rossella_s has quit IRC | 06:06 | |
*** rossella_s has joined #openstack-qa | 06:06 | |
*** thanhnt-z has quit IRC | 06:08 | |
*** sabeen2 has quit IRC | 06:10 | |
*** dmorita has joined #openstack-qa | 06:22 | |
*** nadya has joined #openstack-qa | 06:28 | |
*** mragupat has quit IRC | 06:30 | |
gmann | andreaf: replied on that. You are right we need to keep back comp and i agree with your idea of allowing "unscoped" as string | 06:45 |
openstackrecheck | Console logs not available after 13:22s for gate-nova-pep8 212016,14,c922e9e | 06:49 |
*** achanda has quit IRC | 06:51 | |
*** achanda has joined #openstack-qa | 06:59 | |
*** ranger81 has quit IRC | 07:06 | |
*** agireud has quit IRC | 07:08 | |
*** Triveni has quit IRC | 07:10 | |
*** achanda has quit IRC | 07:10 | |
*** agireud has joined #openstack-qa | 07:13 | |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 07:14 |
*** achanda has joined #openstack-qa | 07:15 | |
*** macjack has joined #openstack-qa | 07:16 | |
*** thanhnt-z has joined #openstack-qa | 07:21 | |
*** grafuls has joined #openstack-qa | 07:22 | |
*** mudassirlatif has quit IRC | 07:22 | |
*** shardy has joined #openstack-qa | 07:24 | |
*** gfidente has joined #openstack-qa | 07:34 | |
*** jlanoux has joined #openstack-qa | 07:39 | |
*** cdent has joined #openstack-qa | 07:40 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest-lib: Migrated extensions_client.py from tempest https://review.openstack.org/230298 | 07:41 |
*** achanda has quit IRC | 07:42 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest-lib: Migrated flavors_client.py from tempest https://review.openstack.org/227666 | 07:44 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest-lib: Migrated flavors_client.py from tempest https://review.openstack.org/227666 | 07:45 |
*** mudassirlatif has joined #openstack-qa | 07:45 | |
*** openstackgerrit has quit IRC | 07:46 | |
*** openstackgerrit has joined #openstack-qa | 07:46 | |
*** jordanP has joined #openstack-qa | 07:47 | |
openstackrecheck | Console logs not available after 13:23s for gate-nova-pep8 212016,14,05f8569 | 08:01 |
*** tosky has joined #openstack-qa | 08:03 | |
openstackrecheck | Console logs not available after 13:22s for gate-nova-python27 229578,2,97487a4 | 08:15 |
*** oomichi has joined #openstack-qa | 08:16 | |
openstackgerrit | Marian Horban proposed openstack/tempest: Added processing /compute URL for getting list of versions https://review.openstack.org/230308 | 08:18 |
*** thanhnt-z has quit IRC | 08:27 | |
openstackrecheck | Console logs not available after 13:24s for gate-nova-python34 230295,2,00f84ca | 08:28 |
*** vgridnev has quit IRC | 08:31 | |
*** cazino has joined #openstack-qa | 08:32 | |
openstackgerrit | Kenji Yasui proposed openstack/tempest-lib: Enhance unit test coverage: aggregates_client https://review.openstack.org/230317 | 08:38 |
*** hop|2 has joined #openstack-qa | 08:44 | |
openstackrecheck | Console logs not available after 13:26s for gate-nova-python34 228893,2,0873344 | 08:45 |
*** macjack has quit IRC | 08:51 | |
*** aix has joined #openstack-qa | 08:54 | |
*** apevec has joined #openstack-qa | 08:54 | |
*** apevec has quit IRC | 08:54 | |
*** apevec has joined #openstack-qa | 08:54 | |
*** mudassirlatif has quit IRC | 08:56 | |
*** e0ne has joined #openstack-qa | 08:57 | |
openstackrecheck | Console logs not available after 13:22s for gate-tempest-dsvm-full 221020,5,f289277 | 08:59 |
openstackgerrit | Joe Mills proposed openstack/tempest: Add a simple test for Allowed Address Pairs https://review.openstack.org/230324 | 09:02 |
*** markvoelker has joined #openstack-qa | 09:04 | |
*** macjack has joined #openstack-qa | 09:09 | |
*** markvoelker has quit IRC | 09:09 | |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: DEBUG failures on V3CredsClient https://review.openstack.org/230327 | 09:09 |
*** dmorita has quit IRC | 09:14 | |
*** openstack has joined #openstack-qa | 09:22 | |
*** e0ne has quit IRC | 09:23 | |
*** openstackstatus has joined #openstack-qa | 09:23 | |
*** ChanServ sets mode: +v openstackstatus | 09:23 | |
*** vgridnev has joined #openstack-qa | 09:23 | |
openstackgerrit | ZhiQiang Fan proposed openstack/tempest: add test for volume notification to telemetry https://review.openstack.org/229386 | 09:26 |
*** e0ne has joined #openstack-qa | 09:31 | |
*** vgridnev has quit IRC | 09:31 | |
*** vgridnev_ has joined #openstack-qa | 09:31 | |
*** salv-orlando has quit IRC | 09:33 | |
*** hoangcx has quit IRC | 09:34 | |
*** cazino has left #openstack-qa | 09:34 | |
*** gszasz has joined #openstack-qa | 09:35 | |
yamamoto | jordanP: hi, can you revisit https://review.openstack.org/#/c/222136/ ? thank you. | 09:35 |
*** nadya has quit IRC | 09:37 | |
jordanP | yamamoto, a sec | 09:38 |
yamamoto | jordanP: thank you. no hurry. | 09:39 |
*** jhenner has joined #openstack-qa | 09:39 | |
jordanP | yamamoto, sorry I looked at the diff between patchset 1 and patchset 2 | 09:41 |
jordanP | so I thought the skip mechanism was already there | 09:41 |
jordanP | my mistake | 09:41 |
jordanP | yamamoto, I updated my vote: 2 | 09:44 |
jordanP | +2 | 09:44 |
*** dims__ has joined #openstack-qa | 09:44 | |
*** nadya has joined #openstack-qa | 10:02 | |
*** rossella_s has quit IRC | 10:06 | |
*** rossella_s has joined #openstack-qa | 10:06 | |
*** pcaruana has joined #openstack-qa | 10:13 | |
openstackrecheck | Console logs not available after 13:24s for gate-nova-docs 205766,6,4b1cc7d | 10:15 |
yamamoto | jordanP: thank you | 10:16 |
*** cazino has joined #openstack-qa | 10:18 | |
*** pc_m has joined #openstack-qa | 10:20 | |
*** e0ne has quit IRC | 10:21 | |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: DEBUG failures on V3CredsClient https://review.openstack.org/230327 | 10:23 |
*** nadya has quit IRC | 10:25 | |
openstackgerrit | Sean Dague proposed openstack-dev/devstack: Workaround potential failure to shutdown services https://review.openstack.org/230107 | 10:27 |
openstackgerrit | litong01 proposed openstack-dev/devstack: HOST_IP should not be used in moving address to route https://review.openstack.org/226373 | 10:28 |
*** shardy_ has joined #openstack-qa | 10:28 | |
*** shardy has quit IRC | 10:28 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: DNM: test oslo.service backports https://review.openstack.org/230101 | 10:30 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: DNM: test devstack work around https://review.openstack.org/230108 | 10:30 |
*** nadya has joined #openstack-qa | 10:31 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: update grenade branch logic for mitaka https://review.openstack.org/229363 | 10:31 |
openstackrecheck | Console logs not available after 13:24s for gate-nova-docs 228141,7,1e81a34 | 10:32 |
*** shardy_ has quit IRC | 10:33 | |
*** shardy has joined #openstack-qa | 10:34 | |
*** e0ne has joined #openstack-qa | 10:34 | |
*** mwagner_ has quit IRC | 10:34 | |
*** jaypipes has joined #openstack-qa | 10:36 | |
*** ChanServ sets mode: +o jaypipes | 10:36 | |
*** jaypipes is now known as leakypipes | 10:38 | |
*** yamamoto has quit IRC | 10:43 | |
openstackrecheck | Console logs not available after 13:23s for gate-nova-pep8 207373,16,72d5ca1 | 10:46 |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Fix return value of reserve_fixed_ip client method https://review.openstack.org/230364 | 10:47 |
dims__ | sdague: a small fix in devstack will help us test pika based rabbitmq driver - https://review.openstack.org/#/c/226348/ - please peek when you get a chance | 10:47 |
*** shardy_ has joined #openstack-qa | 10:51 | |
*** frickler has quit IRC | 10:51 | |
sdague | dims__: we no longer need the rabbit section? | 10:51 |
sdague | if so, awesome | 10:51 |
sdague | but I didn't think that was a thing | 10:52 |
*** shardy has quit IRC | 10:52 | |
dims__ | sdague: both will work | 10:52 |
sdague | ok, but options can't be passed in the url still, we need options for that | 10:52 |
dims__ | ah let me ask dukhlov to check on that | 10:53 |
*** ig0r_ has joined #openstack-qa | 10:55 | |
*** ig0r_ has quit IRC | 10:56 | |
*** shardy_ has quit IRC | 10:56 | |
*** e0ne has quit IRC | 10:57 | |
*** shardy has joined #openstack-qa | 10:57 | |
openstackrecheck | Console logs not available after 13:27s for gate-grenade-dsvm-partial-ncpu 221020,7,82cbf24 | 11:00 |
*** markvoelker has joined #openstack-qa | 11:05 | |
*** cazino has quit IRC | 11:05 | |
*** markvoelker has quit IRC | 11:09 | |
*** tosky has quit IRC | 11:10 | |
openstackgerrit | Merged openstack/tempest: Add test for config drive extension https://review.openstack.org/189687 | 11:12 |
openstackrecheck | Console logs not available after 13:26s for gate-tempest-dsvm-neutron-full 230324,1,cacd875 | 11:13 |
*** cdent has quit IRC | 11:14 | |
*** salv-orlando has joined #openstack-qa | 11:15 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/227744 failed because of: gate-nova-docs: https://bugs.launchpad.net/bugs/1501995, gate-nova-pep8: https://bugs.launchpad.net/bugs/1501995, gate-nova-python27: https://bugs.launchpad.net/bugs/1501995, gate-nova-python34: https://bugs.launchpad.net/bugs/1501995, gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1353962 | 11:15 |
openstack | Launchpad bug 1501995 in OpenStack-Gate "Pillow 3.0.0 breaks the world" [Undecided,Fix released] - Assigned to Anita Kuno (anteaya) | 11:15 |
*** salv-orlando has quit IRC | 11:20 | |
*** cazino has joined #openstack-qa | 11:24 | |
openstackrecheck | Console logs not available after 13:28s for gate-nova-python27 228774,1,6467974 | 11:29 |
*** frickler has joined #openstack-qa | 11:29 | |
*** salv-orlando has joined #openstack-qa | 11:32 | |
*** dzamboni has joined #openstack-qa | 11:36 | |
*** aysyd has joined #openstack-qa | 11:40 | |
openstackrecheck | Console logs not available after 13:28s for gate-tempest-dsvm-full-juno 229386,3,6505a68 | 11:42 |
*** yamamoto has joined #openstack-qa | 11:47 | |
*** pcaruana has quit IRC | 11:47 | |
*** nadya has quit IRC | 11:54 | |
*** jffische has joined #openstack-qa | 11:54 | |
*** amotoki has quit IRC | 11:59 | |
openstackrecheck | Console logs not available after 13:24s for gate-tempest-dsvm-neutron-full 230327,4,8a92ca0 | 11:59 |
*** nadya has joined #openstack-qa | 12:00 | |
*** e0ne has joined #openstack-qa | 12:01 | |
*** pradk has joined #openstack-qa | 12:03 | |
*** nadya has quit IRC | 12:04 | |
*** cazino has quit IRC | 12:05 | |
*** markvoelker has joined #openstack-qa | 12:05 | |
*** cazino has joined #openstack-qa | 12:05 | |
openstackgerrit | Andrea Frittoli proposed openstack/tempest-lib: Introduce scope in the auth API https://review.openstack.org/202705 | 12:08 |
*** markvoelker has quit IRC | 12:09 | |
*** markvoelker has joined #openstack-qa | 12:09 | |
*** apevec has quit IRC | 12:11 | |
*** nadya has joined #openstack-qa | 12:11 | |
*** bltavares has joined #openstack-qa | 12:13 | |
openstackrecheck | Console logs not available after 13:22s for gate-nova-pep8 230295,3,d2f38f0 | 12:14 |
*** tosky has joined #openstack-qa | 12:18 | |
*** ranger81 has joined #openstack-qa | 12:18 | |
*** mwagner_ has joined #openstack-qa | 12:19 | |
*** yamamoto has quit IRC | 12:21 | |
*** ranger81 has quit IRC | 12:22 | |
openstackgerrit | Elena Ezhova proposed openstack-dev/grenade: Increase wait_for timeout in ensure_services_stopped https://review.openstack.org/230397 | 12:24 |
openstackgerrit | Sean Dague proposed openstack-dev/grenade: make SERVICE_TIMEOUT variable https://review.openstack.org/230398 | 12:24 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: update grenade branch logic for mitaka https://review.openstack.org/229363 | 12:26 |
*** weshay_xchat has joined #openstack-qa | 12:27 | |
*** weshay_xchat is now known as weshay | 12:28 | |
*** glauco_ has joined #openstack-qa | 12:28 | |
*** ccarrara has joined #openstack-qa | 12:32 | |
openstackrecheck | Console logs not available after 13:23s for gate-tempest-dsvm-full-liberty 230364,1,769ae72 | 12:36 |
*** edmondsw has joined #openstack-qa | 12:37 | |
*** bltavares has quit IRC | 12:38 | |
*** bltavares has joined #openstack-qa | 12:38 | |
*** vgridnev_ has quit IRC | 12:43 | |
*** vgridnev has joined #openstack-qa | 12:44 | |
ccarrara | mtreinish timothyb89: Hi :) Do you guys have any thought how openstack-health will serve the frontend built stuff? | 12:47 |
*** rfolco has joined #openstack-qa | 12:49 | |
*** macjack has joined #openstack-qa | 12:52 | |
*** regXboi has joined #openstack-qa | 12:52 | |
ccarrara | andreaf: yesterday we're asking about how the pages on o-h will be linked. I've tried something here, building the second page and putting the links on home page and second page. https://review.openstack.org/#/c/230135/ | 12:54 |
andreaf | ccarrara: cool - looking | 12:54 |
*** zz_dimtruck is now known as dimtruck | 12:55 | |
*** dtantsur is now known as dtantsur|brb | 12:55 | |
*** dramalho has joined #openstack-qa | 12:55 | |
*** woodster_ has joined #openstack-qa | 12:56 | |
*** dustins has joined #openstack-qa | 12:57 | |
*** dspano has joined #openstack-qa | 12:57 | |
*** dpaterson has joined #openstack-qa | 12:58 | |
openstackgerrit | Tuan Nguyen proposed openstack/tempest-lib: Migrated quotas_client.py from tempest https://review.openstack.org/229120 | 12:59 |
*** salv-orlando has quit IRC | 12:59 | |
*** yamamoto has joined #openstack-qa | 13:07 | |
*** bltavares has quit IRC | 13:07 | |
*** bltavares has joined #openstack-qa | 13:10 | |
*** yamamoto_ has joined #openstack-qa | 13:13 | |
*** yamamoto has quit IRC | 13:16 | |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: DEBUG failures on V3CredsClient https://review.openstack.org/230327 | 13:16 |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 13:16 |
*** dimtruck is now known as zz_dimtruck | 13:17 | |
openstackgerrit | Tuan Nguyen proposed openstack/tempest-lib: Migrated instance_usage_audit_log_client.py from tempest https://review.openstack.org/229910 | 13:19 |
sdague | mtreinish / dansmith / dtroyer : https://review.openstack.org/#/c/229458/ and https://review.openstack.org/#/c/230398/ could use landing, I think we can then do the branch switch over | 13:21 |
*** mriedem has joined #openstack-qa | 13:26 | |
*** mestery_ is now known as mestery | 13:26 | |
*** dramalho has quit IRC | 13:27 | |
*** tpeoples has joined #openstack-qa | 13:28 | |
*** amotoki has joined #openstack-qa | 13:30 | |
*** zz_dimtruck is now known as dimtruck | 13:35 | |
*** cazino has quit IRC | 13:35 | |
*** cazino has joined #openstack-qa | 13:36 | |
*** dramalho has joined #openstack-qa | 13:38 | |
*** cdent has joined #openstack-qa | 13:40 | |
*** mestery has quit IRC | 13:40 | |
*** alaski is now known as lascii | 13:45 | |
*** shardy_ has joined #openstack-qa | 13:49 | |
mtreinish | timothyb89: I got the gauges working on: https://review.openstack.org/#/c/224992/ it turns out it was just an dep install issue on my env, so that should be good to go | 13:49 |
mtreinish | ccarrara: ok, I'll take a look | 13:49 |
*** shardy has quit IRC | 13:51 | |
mtreinish | sdague: +A | 13:51 |
*** artom has joined #openstack-qa | 13:53 | |
ccarrara | mtreinish: thanks :) I'm just asking about serving frontend stuff to know if there is anything in progress or some thought. I'm ready to help you guys :) | 13:53 |
*** shardy_ has quit IRC | 13:54 | |
openstackrecheck | openstack/glance change: https://review.openstack.org/230056 failed gate-tempest-dsvm-large-ops in the gate queue with an unrecognized error | 13:55 |
*** mriedem1 has joined #openstack-qa | 13:55 | |
*** shardy has joined #openstack-qa | 13:55 | |
*** mriedem has quit IRC | 13:55 | |
mtreinish | ccarrara: well I respun the patch last night which actually uses data from the rest server: https://review.openstack.org/#/c/224992/ | 13:57 |
mtreinish | so you can at least test the whole workflow locally | 13:57 |
ccarrara | this is really nice. | 13:57 |
mtreinish | ccarrara: also my patch https://review.openstack.org/#/c/230209/ makes it easy to test things locally, just run 'tox -edeploy-dev' | 13:59 |
mtreinish | I should rebase that on the live data patch | 13:59 |
*** dimtruck is now known as zz_dimtruck | 13:59 | |
*** zz_dimtruck is now known as dimtruck | 14:00 | |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Add tox env for running a local dev instance of the dashboard https://review.openstack.org/230209 | 14:00 |
*** jecarey has joined #openstack-qa | 14:00 | |
*** mriedem1 is now known as mriedem_meeting | 14:00 | |
*** davhou has quit IRC | 14:01 | |
*** apevec has joined #openstack-qa | 14:01 | |
*** rossella_s has quit IRC | 14:06 | |
*** rossella_s has joined #openstack-qa | 14:06 | |
*** bltavares has quit IRC | 14:10 | |
*** bltavares has joined #openstack-qa | 14:13 | |
*** dims__ has quit IRC | 14:14 | |
*** dims__ has joined #openstack-qa | 14:15 | |
sdague | fungi: so... https://review.openstack.org/#/c/229363/ I think is good, with the stack of fixes in flight now | 14:19 |
fungi | sdague: woo! so close | 14:20 |
sdague | fungi: everything in depends-on is in the gate or merged now | 14:20 |
*** macjack has quit IRC | 14:21 | |
*** mfisher_ora has joined #openstack-qa | 14:21 | |
*** ajmiller_ has joined #openstack-qa | 14:23 | |
*** cazino has quit IRC | 14:23 | |
*** ajmiller has quit IRC | 14:25 | |
*** cdent has quit IRC | 14:26 | |
timothyb89 | ccarrara: for serving the frontend, the final build artifact is a flat .html with combined resources, we can put it on any http server | 14:27 |
*** bltavares has quit IRC | 14:27 | |
*** austin81 has joined #openstack-qa | 14:28 | |
*** glauco_ has quit IRC | 14:29 | |
*** bltavares has joined #openstack-qa | 14:29 | |
*** dramalho has quit IRC | 14:29 | |
ccarrara | timothyb89: yes. I do agreed with you. Are there any project and/or openstack way to do that which we must to base on? | 14:29 |
*** mestery has joined #openstack-qa | 14:30 | |
*** glauco_ has joined #openstack-qa | 14:30 | |
*** apevec has quit IRC | 14:31 | |
*** glauco_ has quit IRC | 14:31 | |
*** apevec has joined #openstack-qa | 14:31 | |
timothyb89 | ccarrara: there are some, we have used ironic-webclient as an example for a lot of things | 14:31 |
*** glauco has joined #openstack-qa | 14:31 | |
*** glauco has quit IRC | 14:31 | |
*** glauco_ has joined #openstack-qa | 14:32 | |
timothyb89 | ccarrara: distribution itself should be easy enough, CI integration will be a bit tricker, what with npm being npm | 14:32 |
mtreinish | timothyb89: heh, we'll be CDing it, so we probably want a way to test patches before too long | 14:33 |
timothyb89 | yep, that too | 14:33 |
timothyb89 | actually I think `gulp prod` will have a plain nonzero return value if there's any build issues, like blatant syntax errors etc | 14:34 |
austin81 | mtreinish: so you want to write us some unit tests?? :) | 14:34 |
mtreinish | timothyb89: yeah, and it does some lint checking too | 14:35 |
mtreinish | austin81: hah, no not for the js stuff | 14:35 |
*** dramalho has joined #openstack-qa | 14:35 | |
mtreinish | I just started learning how to do that for real last night | 14:35 |
austin81 | js? isn't it awful? | 14:35 |
mtreinish | my eyes haven't stopped bleeding since I started looking at it | 14:36 |
*** yamahata has joined #openstack-qa | 14:36 | |
timothyb89 | mtreinish: I believe it only treats them as warnings though, we may still want a real gate test at some point | 14:36 |
mtreinish | timothyb89: but we can probably leverage some of what krotscheck has setup with the eslint checks and other things | 14:36 |
timothyb89 | yes, hopefully | 14:36 |
*** cazino has joined #openstack-qa | 14:36 | |
timothyb89 | mtreinish: we do also have infrastructure for unit and integration tests in the project, we just have yet to write any | 14:37 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/tempest-lib: Updated from global requirements https://review.openstack.org/230470 | 14:37 |
*** pglass has joined #openstack-qa | 14:37 | |
krotscheck | What do y'all need help with? | 14:38 |
* krotscheck is reading backscroll | 14:39 | |
krotscheck | Oh, openstack-health | 14:39 |
krotscheck | Neat | 14:39 |
krotscheck | Yes | 14:39 |
mtreinish | krotscheck: yeah I was just saying at some point we need to get the javascript-jobs template running on proposed commits | 14:40 |
krotscheck | mtreinish: It should be pretty easy | 14:41 |
mtreinish | krotscheck: yeah the project-config change looks easy, I was just looking to see what needs to change in repo to support running them | 14:41 |
krotscheck | mtreinish: Lemme go look | 14:42 |
*** tsekiyama has joined #openstack-qa | 14:43 | |
krotscheck | Oh, now I remember, it's a zombie project :) | 14:43 |
krotscheck | Sorry | 14:43 |
krotscheck | frankenstein | 14:43 |
krotscheck | too close to halloween | 14:43 |
*** dansmith is now known as superdan | 14:44 | |
mtreinish | heh, yeah something like that | 14:45 |
*** ranger81 has joined #openstack-qa | 14:45 | |
krotscheck | mtreinish: Ok, a few things likely need to be fixed. | 14:45 |
krotscheck | 1- Right now, project.jsondevDependencies contain things which I think y'all need at runtime. Those need to be moved into 'dependencies' | 14:46 |
*** shardy_ has joined #openstack-qa | 14:46 | |
*** dtantsur|brb is now known as dtantsur | 14:47 | |
*** shardy has quit IRC | 14:48 | |
* krotscheck is waiting for npm install ro resolve | 14:48 | |
krotscheck | wow, y'all have a lot of gulp things in there. | 14:48 |
krotscheck | Are they all strictly necessary? | 14:48 |
timothyb89 | a bit concerned about that actually, will the gate try to install them all from scratch every run? | 14:49 |
mtreinish | I have no idea, timothyb89 ^^^ | 14:49 |
krotscheck | timothyb89: Yep. | 14:49 |
krotscheck | We don't have a proxy yet | 14:49 |
krotscheck | I mean mirror | 14:49 |
krotscheck | ENGLISH | 14:49 |
krotscheck | It's HARD | 14:49 |
timothyb89 | krotscheck: ouch. I have done a lot of pruning as it is, not too sure what else can be removed | 14:49 |
*** ranger81 has quit IRC | 14:49 | |
mtreinish | timothyb89: what are all of those used for? | 14:50 |
krotscheck | timothyb89: Some of the gulp-proxy-to-specific-tool things can be pulled. | 14:51 |
krotscheck | For instance, you can run karma start ./ | 14:51 |
krotscheck | instead of gulp karma | 14:51 |
krotscheck | and hide all that behind npm test | 14:51 |
krotscheck | Once this installs I can show you what I mean | 14:51 |
*** shardy_ has quit IRC | 14:51 | |
krotscheck | Also, I'm using npm v3, which does ALL KINDS of additional install things | 14:52 |
*** shardy has joined #openstack-qa | 14:52 | |
timothyb89 | that's probably for the best, older npm doesn't play nice some some deps | 14:52 |
*** achanda has joined #openstack-qa | 14:53 | |
krotscheck | Hrm, browserify? | 14:53 |
*** tosky has quit IRC | 14:53 | |
openstackrecheck | Console logs not available after 13:26s for gate-cinder-pep8 230429,1,ac3b6b9 | 14:53 |
timothyb89 | it appears that the biggest deps are all test related, probably a bit tricky to clean those unfortunately | 14:54 |
timothyb89 | ehm, yep | 14:54 |
krotscheck | Well, we'll see what we can do :) | 14:54 |
ccarrara | timothyb89: other thing, is it really necessary to use browsersync in the current status of project? | 14:54 |
*** tosky has joined #openstack-qa | 14:54 | |
*** russellb is now known as rustlebee | 14:55 | |
timothyb89 | ccarrara: it isn't strictly "necessary" though it is extremely convenient for dev | 14:55 |
timothyb89 | ccarrara: also it is not used during production builds at all | 14:55 |
krotscheck | what does that do> | 14:56 |
krotscheck | basically gulp watch? | 14:56 |
timothyb89 | it handles automated browser reload and state sync across devices | 14:56 |
timothyb89 | in addition to gulp watch functionality | 14:56 |
*** mlavalle has joined #openstack-qa | 14:56 | |
timothyb89 | useful for demos, etc as well since it will sync mouse clicks and such across browsers | 14:57 |
*** mtanino has joined #openstack-qa | 14:57 | |
krotscheck | Right. Ironic-webclient uses gulp-webserver for that | 14:58 |
*** gszasz has quit IRC | 14:58 | |
ccarrara | krotscheck timothyb89: since we already got this to the project and browsersync is not used during production builds, maybe we can let it. Right? | 14:58 |
krotscheck | For speed's sake, you can remove the 'pretest' call to npm install, the gate will always run it. | 14:58 |
*** mriedem_meeting is now known as mriedem | 14:59 | |
*** cdent has joined #openstack-qa | 14:59 | |
*** pbredenb has quit IRC | 14:59 | |
*** sabeen has joined #openstack-qa | 14:59 | |
timothyb89 | ccarrara: I imagine we can leave it in with "devDependencies" and it won't get downloaded by the gate | 14:59 |
*** ccarmack has joined #openstack-qa | 14:59 | |
krotscheck | npm install installs everything, we don't have a choice over that. | 14:59 |
timothyb89 | ah, hm | 15:00 |
ccarrara | krotscheck: good point :| | 15:00 |
*** pbredenb has joined #openstack-qa | 15:00 | |
*** yamahata has quit IRC | 15:00 | |
timothyb89 | npm install --production? | 15:00 |
*** yamahata has joined #openstack-qa | 15:00 | |
*** achanda has quit IRC | 15:01 | |
krotscheck | timothyb89: How do you distinguish between a necessary dependency for test, a necessary dependency for dev, and a necessary dependency for runtime? | 15:01 |
krotscheck | This is why I use bower for my app dependencies :) | 15:01 |
krotscheck | But even so, using browsersync is not a problem. | 15:01 |
krotscheck | It's a useful tool, let's just leave it. Nitpicking individual packages is just bikeshedding | 15:02 |
ccarrara | krotscheck: I agreed with you :). | 15:02 |
timothyb89 | well, what would it take to "solve" things properly? an npm caching server? | 15:03 |
openstackgerrit | Glauco Oliveira proposed openstack-infra/subunit2sql: Refactor subunit2sql dbapi https://review.openstack.org/228554 | 15:03 |
krotscheck | timothyb89: Nothing, really? What's your tolerance for a longer build? | 15:04 |
krotscheck | timothyb89: I mean yeah, adding an npm mirror into infra would definitely speed things up | 15:04 |
krotscheck | timothyb89: And it might be ncie as the JS ecosystem grows so we don't drive npm Inc out of business | 15:04 |
krotscheck | (though, frankly, I kindof want to so they stop trying to block necessary features in order to try to be profitable) | 15:04 |
mtreinish | krotscheck: things that touch the internet will break eventually. Especially if you get your wish and have moar js | 15:04 |
krotscheck | mtreinish: Did you hear that they emulated the linux kernel in javascript? | 15:05 |
timothyb89 | mtreinish: http://jslinux.org/ | 15:05 |
mtreinish | heh, yeah I've seen that | 15:05 |
openstackrecheck | Console logs not available after 13:25s for gate-horizon-python34 230434,1,3a132aa | 15:07 |
openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/223093 failed gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error | 15:07 |
timothyb89 | I suppose longer builds won't be a huge issue, and realistically the "worst" deps are all things we can't really remove | 15:07 |
timothyb89 | phantomjs, gulp-sass, protractor, karma... | 15:08 |
*** hop|2 has quit IRC | 15:10 | |
*** artom has quit IRC | 15:11 | |
*** grafuls has quit IRC | 15:11 | |
krotscheck | Yep | 15:12 |
krotscheck | timothyb89: What's all this python stuff for? | 15:13 |
timothyb89 | krotscheck: the project namespace is shared with a python module for the rest API | 15:13 |
ccarrara | krotscheck: under openstack-health folder? | 15:13 |
krotscheck | Gotcha, so two projects in one | 15:14 |
timothyb89 | yep | 15:14 |
*** sabeen has quit IRC | 15:15 | |
krotscheck | express? | 15:15 |
timothyb89 | er, is that in there? it probably shouldn't be.. | 15:15 |
openstackgerrit | David Paterson proposed openstack/tempest: Fix init command https://review.openstack.org/224338 | 15:15 |
timothyb89 | apparently it is a depdendency of selenium-webdriver, karma, and a half dozen other libs | 15:17 |
*** jasonsb_ has joined #openstack-qa | 15:18 | |
timothyb89 | ah, but it's downloaded individually where it is needed, we can get rid of our dependency on it at least | 15:20 |
*** davideagnello has joined #openstack-qa | 15:20 | |
*** dramalho has quit IRC | 15:23 | |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: DEBUG failures on V3CredsClient https://review.openstack.org/230327 | 15:23 |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 15:23 |
krotscheck | What? | 15:24 |
krotscheck | That seems silly | 15:24 |
timothyb89 | that's npm :) | 15:24 |
krotscheck | Aaanyway | 15:24 |
krotscheck | Response time this morning is going to be a bit slow until my wife takes the kid | 15:24 |
*** kbaikov has quit IRC | 15:24 | |
timothyb89 | but yeah, we'll need to do a whole lot of pruning, I'll spend some time on that today | 15:24 |
*** ccneill has joined #openstack-qa | 15:25 | |
*** hemnafk is now known as hemna | 15:27 | |
krotscheck | The fact that your gulp tasks require their own includes is probably a good indication that your build is too complicated ;) | 15:28 |
timothyb89 | heh, it ah, does do a fair amount | 15:30 |
openstackrecheck | Console logs not available after 13:24s for gate-tempest-dsvm-postgres-full 230327,5,c4da85c | 15:31 |
*** vgridnev has quit IRC | 15:31 | |
krotscheck | timothyb89: Teh default coverate report output folder for infra is ./cover, not ./coverage | 15:32 |
*** sabeen has joined #openstack-qa | 15:32 | |
*** mlavalle has quit IRC | 15:32 | |
*** scottda has joined #openstack-qa | 15:35 | |
*** cazino has quit IRC | 15:35 | |
scottda | Hi QA folks. | 15:36 |
scottda | I've a change to /etc/cinder/api-paste.ini for my WIP cinder-api-microversions and it needs to be manually copied over from the cinder repo on Devstack in order to get c-api to start.... | 15:36 |
scottda | This is causing Grenade to fail in starting Cinder API. | 15:36 |
scottda | What needs to get done to make the Grenade tests copy over the change to /etc/cinder/api-paste.ini ? | 15:36 |
*** Swami has joined #openstack-qa | 15:37 | |
*** mylu has joined #openstack-qa | 15:38 | |
mtreinish | scottda: that is documented here: http://docs.openstack.org/developer/grenade/readme.html#theory-of-upgrade | 15:39 |
scottda | OK, so does that mean the something like /etc/cinder/api-paste.ini cannot be changed at all? or just that the change must be in the release notes, and somehow Grenade will fail? | 15:41 |
scottda | mriedem: Thanks. I've been pointed to this example for a similar change: | 15:43 |
scottda | https://review.openstack.org/#/c/217625/3 | 15:43 |
mriedem | blarg | 15:43 |
mtreinish | scottda: preferablly you should be able to run without changing the paste file | 15:43 |
openstackgerrit | Michael Krotscheck proposed openstack/openstack-health: Switched from jshint to eslint https://review.openstack.org/230505 | 15:44 |
mtreinish | scottda: but, read the last paragraph of that section, you need to get the ptl to sign off the release notes changed and then we'll be willing to accept a manual upgrade step patch | 15:44 |
mtreinish | but if you can figure out a way to make things work with the old paste that would be best | 15:44 |
openstackgerrit | Michael Krotscheck proposed openstack/openstack-health: Switched from jshint to eslint https://review.openstack.org/230505 | 15:44 |
scottda | Got it. Thanks. The code is not reviewed, and maybe I can do without the change... | 15:44 |
*** dramalho has joined #openstack-qa | 15:44 | |
*** mylu has quit IRC | 15:44 | |
krotscheck | timothyb89: So, from what I can gather, something like ^^ is the only remaining step you need to have the gate switch to using javascript jobs. | 15:44 |
openstackrecheck | Console logs not available after 13:33s for gate-tempest-dsvm-full-juno 221020,8,513b596 | 15:44 |
timothyb89 | krotscheck: ah, good to know, thanks! | 15:45 |
*** mylu has joined #openstack-qa | 15:45 | |
krotscheck | I didn't fix all the linting errors, as some of them are dependent on project governance. I.e. you want to decide whether these rules make sense to you. | 15:45 |
*** mylu has quit IRC | 15:46 | |
timothyb89 | makes sense, I'll look through them | 15:46 |
krotscheck | From an app-design standpoint though, it looks like you're using a lot of require.js syntax, which forces you to use browserify. You may be able to get away without it, and strip out all those dependencies. | 15:46 |
*** mylu has joined #openstack-qa | 15:46 | |
krotscheck | An alternative is here -> https://review.openstack.org/#/c/228630/ | 15:47 |
openstackgerrit | Merged openstack-dev/devstack: Remove unnecessary execute permissions https://review.openstack.org/228100 | 15:47 |
*** jlanoux has quit IRC | 15:47 | |
*** cazino has joined #openstack-qa | 15:48 | |
krotscheck | But, well, all you really have to have for javascript-jobs is for "npm test" and "npm run lint" to work. | 15:48 |
krotscheck | The only other piece is that there's some assumptions that the jobs make about the output structure. The rules are fairly straightforward: | 15:49 |
krotscheck | coverage output goes into ./cover/ | 15:49 |
krotscheck | test reports go into ./reports/ | 15:49 |
krotscheck | Any draft build goes into ./dist/ | 15:50 |
krotscheck | Actually, that's a lie | 15:50 |
krotscheck | Any draft build will be generated off of package.json -> files: [] property | 15:50 |
*** tsekiyama has quit IRC | 15:50 | |
krotscheck | And that's it. | 15:50 |
krotscheck | timothyb89: ^^ that's the basic summary | 15:51 |
timothyb89 | okay, seems pretty straightforward | 15:51 |
krotscheck | Upcoming jobs will try to publish to npm, but that's not relevant to your project. | 15:51 |
krotscheck | If you have suggestions on how to handle translation imports though, or do doc generation, I'm all ears :) | 15:52 |
* krotscheck hasn't tackled that yet. | 15:52 | |
timothyb89 | krotscheck: part of our usage of browserify is so we can have dependencies concat'd + minified, is there a good solution for that as well? | 15:52 |
timothyb89 | that is a good question, I am unfamiliar with that in js | 15:52 |
krotscheck | timothyb89: That patch I sent you generates the concatenation glob patterns for uglify. | 15:52 |
krotscheck | It basically assumes that you have a directory full of ng modules. | 15:52 |
krotscheck | Everything else is pulled from the 'main' property in the bower.json files for each of the bower dependencies (bower-main-files) | 15:53 |
timothyb89 | ah, so it does include bower libs? | 15:53 |
krotscheck | timothyb89: Yep, sec, lemme find the relevant line. | 15:53 |
timothyb89 | huh, okay, that is handy | 15:54 |
krotscheck | timothyb89: It does things like this -> http://git.openstack.org/cgit/openstack/ironic-webclient/tree/gulpfile.js#n101 | 15:54 |
krotscheck | The libs are packaged here -> http://git.openstack.org/cgit/openstack/ironic-webclient/tree/gulpfile.js#n126 | 15:54 |
timothyb89 | ah, okay, that's simple enough | 15:55 |
jordanP | tempest core, we have a disagreement here : https://review.openstack.org/#/c/224944/1/tempest/api_schema/response/compute/v2_1/services.py,cm i'd love to have your input | 15:55 |
krotscheck | And places where the bower.json file is annoying about including dependencies like jquery that we don't want can be overridden directly like this -> http://git.openstack.org/cgit/openstack/ironic-webclient/tree/bower.json#n16 | 15:55 |
jordanP | gmann, you here ? | 15:55 |
sdague | mtreinish: when you added doc stuff to grenade.... | 15:55 |
mtreinish | uh oh | 15:55 |
sdague | are we using any of the sphinx diag stuff | 15:55 |
mtreinish | no | 15:55 |
sdague | because I don't see it | 15:55 |
mtreinish | err, well not that I know of | 15:56 |
sdague | ok, it's blowing up on pillow | 15:56 |
mtreinish | there weren't any diagrams in the docs so I wouldn't have imported that module on purpose | 15:56 |
*** amotoki has quit IRC | 15:56 | |
*** jecarey has quit IRC | 15:56 | |
mtreinish | I might have accidently cargo culted it from somewhere though | 15:56 |
sdague | yeh, probably from devstack | 15:57 |
*** yamamoto_ has quit IRC | 15:57 | |
openstackgerrit | Sean Dague proposed openstack-dev/grenade: remove use of sphinx diag tools https://review.openstack.org/230516 | 15:57 |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Enable running under wsgi container https://review.openstack.org/225004 | 15:57 |
mtreinish | sdague: yeah, that's what I based most of the docs stuff off of | 15:58 |
sdague | ok, that and https://review.openstack.org/230517 | 15:58 |
sdague | we should put through fast | 15:58 |
mtreinish | sdague: just +A'd both | 15:59 |
*** mlavalle has joined #openstack-qa | 15:59 | |
openstackgerrit | Sean Dague proposed openstack-dev/grenade: disable image v2 with osc because it doesn't work https://review.openstack.org/229458 | 16:01 |
openstackgerrit | Sean Dague proposed openstack-dev/grenade: make SERVICE_TIMEOUT variable https://review.openstack.org/230398 | 16:01 |
sdague | ok, so those are just rebases to sit on top of that one | 16:01 |
sdague | I think all the cr votes are kept, if you want to +A those | 16:02 |
mtreinish | already done | 16:02 |
sdague | then we can recheck the d-g change and it might work | 16:02 |
*** mylu has quit IRC | 16:03 | |
openstackrecheck | Console logs not available after 18:00s for gate-tempest-dsvm-cells 212908,20,1f0d904 | 16:03 |
*** mylu has joined #openstack-qa | 16:03 | |
*** jogo is now known as flashgordon | 16:05 | |
*** artom has joined #openstack-qa | 16:07 | |
*** artom has quit IRC | 16:07 | |
*** sabeen3 has joined #openstack-qa | 16:07 | |
*** artom has joined #openstack-qa | 16:07 | |
*** sabeen has quit IRC | 16:08 | |
openstackgerrit | John F. Fischer proposed openstack/tempest-lib: Migrated security_groups_client.py https://review.openstack.org/229914 | 16:09 |
openstackgerrit | Michael Krotscheck proposed openstack/eslint-config-openstack: Removed TODO comments https://review.openstack.org/228915 | 16:09 |
openstackgerrit | Michael Krotscheck proposed openstack/eslint-config-openstack: Upgraded eslint version https://review.openstack.org/228914 | 16:09 |
*** ranger81 has joined #openstack-qa | 16:09 | |
*** jordanP has quit IRC | 16:09 | |
*** ranger81 has quit IRC | 16:10 | |
*** ranger81 has joined #openstack-qa | 16:10 | |
*** jordanP has joined #openstack-qa | 16:11 | |
*** salv-orlando has joined #openstack-qa | 16:11 | |
*** ranger81 has quit IRC | 16:12 | |
*** mylu has quit IRC | 16:13 | |
*** artom has quit IRC | 16:13 | |
*** jasonsb_ has quit IRC | 16:13 | |
*** mylu has joined #openstack-qa | 16:15 | |
*** artom has joined #openstack-qa | 16:15 | |
*** mylu has quit IRC | 16:16 | |
openstackrecheck | Console logs not available after 13:32s for gate-nova-python27 228843,1,e9ecbbe | 16:17 |
*** glauco_ has quit IRC | 16:19 | |
*** dramalho has quit IRC | 16:19 | |
*** bltavares has quit IRC | 16:19 | |
*** vgridnev has joined #openstack-qa | 16:21 | |
*** bltavares has joined #openstack-qa | 16:21 | |
*** dramalho has joined #openstack-qa | 16:21 | |
*** mylu has joined #openstack-qa | 16:21 | |
*** dramalho has quit IRC | 16:21 | |
*** jtrovo has joined #openstack-qa | 16:21 | |
*** nic has joined #openstack-qa | 16:22 | |
*** dtantsur is now known as dtantsur|afk | 16:23 | |
*** dimtruck is now known as zz_dimtruck | 16:23 | |
*** mudassirlatif has joined #openstack-qa | 16:23 | |
*** salv-orlando has quit IRC | 16:24 | |
*** catohornet has joined #openstack-qa | 16:25 | |
*** bltavares has quit IRC | 16:25 | |
*** jtrovo has quit IRC | 16:27 | |
*** cazino has left #openstack-qa | 16:27 | |
openstackgerrit | Sirushti Murugesan proposed openstack-dev/devstack: Enable optional Python 3 support https://review.openstack.org/181165 | 16:28 |
*** austin81 has left #openstack-qa | 16:30 | |
*** e0ne has quit IRC | 16:31 | |
openstackrecheck | Console logs not available after 13:23s for gate-cinder-python34 220561,14,52301cc | 16:31 |
*** jordanP has quit IRC | 16:36 | |
*** mylu has quit IRC | 16:42 | |
*** jasonsb_ has joined #openstack-qa | 16:44 | |
openstackrecheck | Console logs not available after 13:22s for gate-nova-python27 221310,15,ffffb58 | 16:44 |
*** mylu has joined #openstack-qa | 16:48 | |
*** jasonsb_ has quit IRC | 16:49 | |
*** dims__ is now known as dimsum__ | 16:49 | |
*** nadya has quit IRC | 16:50 | |
*** mlavalle has quit IRC | 16:51 | |
*** yamahata has quit IRC | 16:53 | |
*** tosky has quit IRC | 16:56 | |
*** davideagnello has quit IRC | 16:56 | |
*** amotoki has joined #openstack-qa | 16:56 | |
*** nadya has joined #openstack-qa | 16:57 | |
*** yamamoto has joined #openstack-qa | 16:57 | |
*** luqas has quit IRC | 16:59 | |
*** davideagnello has joined #openstack-qa | 17:01 | |
*** amotoki has quit IRC | 17:01 | |
*** yamamoto has quit IRC | 17:02 | |
openstackrecheck | Console logs not available after 13:30s for gate-tempest-python34 221020,9,267f60b | 17:06 |
*** bltavares has joined #openstack-qa | 17:09 | |
*** tpeoples has quit IRC | 17:09 | |
*** dramalho has joined #openstack-qa | 17:09 | |
*** glauco_ has joined #openstack-qa | 17:10 | |
*** jtrovo has joined #openstack-qa | 17:12 | |
*** achanda has joined #openstack-qa | 17:16 | |
*** yamahata has joined #openstack-qa | 17:18 | |
openstackrecheck | Console logs not available after 13:34s for gate-tempest-dsvm-postgres-full 230327,6,3fab617 | 17:21 |
*** shardy has quit IRC | 17:23 | |
*** glauco_ has quit IRC | 17:27 | |
*** bltavares has quit IRC | 17:27 | |
*** mriedem has quit IRC | 17:28 | |
*** aysyd has quit IRC | 17:29 | |
*** aysyd has joined #openstack-qa | 17:29 | |
*** mriedem has joined #openstack-qa | 17:32 | |
*** mylu has quit IRC | 17:33 | |
*** nadya has quit IRC | 17:33 | |
*** jtrovo has quit IRC | 17:34 | |
*** mylu has joined #openstack-qa | 17:35 | |
*** mylu has quit IRC | 17:36 | |
*** mylu has joined #openstack-qa | 17:37 | |
*** macjack has joined #openstack-qa | 17:38 | |
*** mylu has quit IRC | 17:39 | |
*** mylu has joined #openstack-qa | 17:40 | |
*** macjack has left #openstack-qa | 17:42 | |
*** pvaneck has joined #openstack-qa | 17:43 | |
*** dramalho has quit IRC | 17:43 | |
*** mylu has quit IRC | 17:47 | |
*** mylu has joined #openstack-qa | 17:53 | |
openstackrecheck | Console logs not available after 13:28s for gate-tempest-dsvm-full-non-isolated 221020,9,a2e0088 | 17:55 |
*** mylu has quit IRC | 17:57 | |
*** mylu has joined #openstack-qa | 17:57 | |
*** cdent has quit IRC | 17:58 | |
*** mylu has quit IRC | 17:58 | |
*** mylu has joined #openstack-qa | 17:59 | |
*** e0ne has joined #openstack-qa | 18:01 | |
*** yamahata has quit IRC | 18:03 | |
*** rossella_s has quit IRC | 18:06 | |
*** rossella_s has joined #openstack-qa | 18:06 | |
*** nadya has joined #openstack-qa | 18:13 | |
*** mylu has quit IRC | 18:15 | |
*** mlavalle has joined #openstack-qa | 18:17 | |
*** mylu has joined #openstack-qa | 18:19 | |
*** weshay has quit IRC | 18:21 | |
*** weshay_xchat has joined #openstack-qa | 18:21 | |
*** artom has quit IRC | 18:21 | |
*** mylu has quit IRC | 18:26 | |
*** mylu has joined #openstack-qa | 18:28 | |
*** e0ne has quit IRC | 18:28 | |
openstackgerrit | Chuck Carmack proposed openstack/tempest: Add a config option to disable security group usage https://review.openstack.org/225199 | 18:30 |
*** salv-orlando has joined #openstack-qa | 18:31 | |
openstackgerrit | John Warren proposed openstack/tempest: Split out Neutron subnets client https://review.openstack.org/230604 | 18:33 |
*** drptbl has joined #openstack-qa | 18:36 | |
*** mylu has quit IRC | 18:37 | |
*** mylu has joined #openstack-qa | 18:38 | |
*** zz_dimtruck is now known as dimtruck | 18:39 | |
ccarmack | mriedem: wondering what you think about my last comment to https://review.openstack.org/#/c/225199/ | 18:39 |
*** mylu has quit IRC | 18:40 | |
*** mylu has joined #openstack-qa | 18:40 | |
*** dramalho has joined #openstack-qa | 18:41 | |
*** jffische has quit IRC | 18:42 | |
mriedem | ccarmack: lascii said it won't work for cells, but it doesn't break at the parent cell API | 18:43 |
mriedem | i'm not sure we want to explicitly change that to start failing in the parent cell | 18:43 |
mriedem | ccarmack: but if we're going to add an option to tempest called compute-feature-enabled.security_groups and it can be False, i'd expect that the tempest.api.compute.security_groups tests will be skipped | 18:44 |
mriedem | regardless of cells | 18:44 |
ccarmack | was that conversation -nova, I missed it | 18:44 |
*** dramalho has quit IRC | 18:46 | |
*** salv-orlando has quit IRC | 18:47 | |
*** e0ne has joined #openstack-qa | 18:48 | |
mriedem | ccarmack: yeah this morning | 18:49 |
ccarmack | ok | 18:49 |
lascii | I should have added my comment to the review | 18:50 |
*** gfidente has quit IRC | 18:52 | |
*** dzamboni has quit IRC | 18:52 | |
*** glauco_ has joined #openstack-qa | 18:55 | |
openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/223093 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1501366 | 18:55 |
openstack | Launchpad bug 1501366 in OpenStack Compute (nova) "libvirtError: Error while building firewall: Some rules could not be created for interface" [High,Confirmed] | 18:55 |
*** nadya has quit IRC | 18:57 | |
*** nadya has joined #openstack-qa | 18:58 | |
*** dramalho has joined #openstack-qa | 18:59 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add new graph time for variance of individual tests https://review.openstack.org/230611 | 19:02 |
*** bltavares has joined #openstack-qa | 19:03 | |
openstackgerrit | Caio Carrara proposed openstack/openstack-health: Add second page (project) https://review.openstack.org/230135 | 19:03 |
mtreinish | timothyb89: so on https://review.openstack.org/#/c/229682/2 what is need for real review? | 19:07 |
*** ccarrara has quit IRC | 19:08 | |
*** e0ne has quit IRC | 19:08 | |
*** erlon has quit IRC | 19:08 | |
timothyb89 | mtreinish: not much, a quick fix to the line colors and maybe some clarification comments | 19:10 |
mtreinish | timothyb89: ok, cool | 19:11 |
mtreinish | that's up first in a branch of frontend stuff | 19:11 |
mtreinish | which'll be good to get moving forward | 19:11 |
timothyb89 | mtreinish: does your tox env solve the npm version issue introduced with that patch? | 19:11 |
mtreinish | timothyb89: it does, it installs npm from source in the tox env | 19:12 |
timothyb89 | okay cool | 19:12 |
mtreinish | you just run the tox command and it does everything for you | 19:12 |
timothyb89 | just making sure the readme didn't need further updates | 19:13 |
mtreinish | timothyb89: I think I updated the readme on that patch too | 19:13 |
timothyb89 | yup, you did | 19:13 |
mtreinish | regXboi: so why only +1 on https://review.openstack.org/#/c/224992/ ? | 19:13 |
mtreinish | is there something I need to fix? | 19:13 |
regXboi | mtreinish: I haven't been able to test it | 19:14 |
mtreinish | ah ok | 19:14 |
regXboi | so it *looks* good but I can't say it *is* good | 19:14 |
*** davideagnello has quit IRC | 19:14 | |
mtreinish | heh, ok | 19:14 |
mtreinish | there wasnt a comment, so I was just wondering | 19:14 |
regXboi | ah | 19:14 |
regXboi | that was my mistake - I should of said that | 19:14 |
mtreinish | no worries | 19:14 |
*** jecarey has joined #openstack-qa | 19:15 | |
* regXboi pretty much stuck in performance purgatory mode | 19:16 | |
*** tsekiyama has joined #openstack-qa | 19:16 | |
*** dramalho has quit IRC | 19:16 | |
*** dramalho has joined #openstack-qa | 19:17 | |
*** jtrovo has joined #openstack-qa | 19:18 | |
*** dramalho has quit IRC | 19:19 | |
*** pc_m has quit IRC | 19:20 | |
*** mwagner_ has quit IRC | 19:21 | |
*** ccarrara has joined #openstack-qa | 19:21 | |
*** e0ne has joined #openstack-qa | 19:22 | |
openstackgerrit | Merged openstack-dev/grenade: remove use of sphinx diag tools https://review.openstack.org/230516 | 19:24 |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Update flake8 path in tox.ini https://review.openstack.org/230621 | 19:27 |
*** glauco_ has quit IRC | 19:28 | |
*** glauco_ has joined #openstack-qa | 19:28 | |
openstackrecheck | Console logs not available after 13:31s for gate-keystone-pep8 230590,1,f86ff58 | 19:28 |
*** david-lyle has quit IRC | 19:35 | |
*** david-lyle has joined #openstack-qa | 19:35 | |
*** e0ne has quit IRC | 19:40 | |
*** enikanorov_ has quit IRC | 19:40 | |
openstackrecheck | Console logs not available after 13:25s for gate-tempest-python27 224338,10,fe7db31 | 19:42 |
*** mlavalle has quit IRC | 19:42 | |
openstackgerrit | Merged openstack-dev/grenade: disable image v2 with osc because it doesn't work https://review.openstack.org/229458 | 19:44 |
openstackgerrit | Merged openstack-dev/grenade: make SERVICE_TIMEOUT variable https://review.openstack.org/230398 | 19:44 |
openstackgerrit | Merged openstack-infra/devstack-gate: update grenade branch logic for mitaka https://review.openstack.org/229363 | 19:44 |
*** david-ly_ has joined #openstack-qa | 19:45 | |
mtreinish | sdague: ^^^ that looks promising | 19:45 |
sdague | yes, yes it does | 19:46 |
sdague | so, we openned up the timeout from 10 to 30 seconds for service stopping | 19:46 |
sdague | we should keep an eye on grenade fails though | 19:47 |
sdague | to see if that still races some times | 19:47 |
*** yamahata has joined #openstack-qa | 19:47 | |
*** david-lyle has quit IRC | 19:49 | |
*** david-ly_ has quit IRC | 19:49 | |
openstackgerrit | Sean Dague proposed openstack-dev/grenade: stop forcing to cinder v1 https://review.openstack.org/229423 | 19:51 |
openstackrecheck | Console logs not available after 13:24s for gate-nova-docs 228141,7,a07077f | 19:55 |
*** mwagner_ has joined #openstack-qa | 19:57 | |
*** ajmiller_ is now known as ajmiller | 20:05 | |
openstackrecheck | Console logs not available after 13:25s for gate-tempest-dsvm-neutron-full-liberty 230604,1,a5bcb4d | 20:09 |
*** tsekiyama has quit IRC | 20:12 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add new graph time for variance of individual tests https://review.openstack.org/230611 | 20:13 |
*** tsekiyama has joined #openstack-qa | 20:17 | |
*** david-lyle has joined #openstack-qa | 20:24 | |
*** leakypipes has quit IRC | 20:28 | |
*** moravec has quit IRC | 20:29 | |
*** moravec has joined #openstack-qa | 20:29 | |
*** aysyd has quit IRC | 20:30 | |
openstackrecheck | Console logs not available after 13:24s for gate-horizon-npm-run-test 172602,25,2bd6a3e | 20:34 |
*** ccneill has quit IRC | 20:38 | |
*** nadya has quit IRC | 20:40 | |
*** drptbl has quit IRC | 20:41 | |
*** ccneill has joined #openstack-qa | 20:44 | |
*** tsekiyama has quit IRC | 20:47 | |
*** drptbl has joined #openstack-qa | 20:49 | |
*** ccarmack has left #openstack-qa | 20:52 | |
*** nadya has joined #openstack-qa | 20:56 | |
*** amotoki has joined #openstack-qa | 20:58 | |
openstackgerrit | John Warren proposed openstack/tempest: Split out Neutron subnets client https://review.openstack.org/230604 | 20:58 |
*** tsekiyama has joined #openstack-qa | 20:58 | |
openstackgerrit | Glauco Oliveira proposed openstack/openstack-health: Add build failure rate chart on homepage https://review.openstack.org/230643 | 21:01 |
glauco_ | mtreinish: andreaf: timothyb89: we have an idea of how to improve the existing dashboard homepage. we thought of having another chart in there showing the build failure rate. That would be us identifying failure trends independently of the amount of builds running. | 21:03 |
*** amotoki has quit IRC | 21:03 | |
*** bltavares has quit IRC | 21:03 | |
glauco_ | We would love having your opinion on whether this is valuable or not | 21:03 |
*** dimtruck is now known as zz_dimtruck | 21:03 | |
*** tsekiyam_ has joined #openstack-qa | 21:03 | |
glauco_ | https://review.openstack.org/#/c/230643/ | 21:03 |
*** dustins has quit IRC | 21:03 | |
*** mtanino has quit IRC | 21:04 | |
timothyb89 | glauco_: I am wondering if it might be possible to integrate that with the main chart? | 21:06 |
timothyb89 | glauco_: it appears to be roughly the same as what you get by deselecting the "passes" series from the top legend | 21:06 |
glauco_ | timothyb89: they are using different scales | 21:06 |
glauco_ | I think it would become difficult to read it | 21:06 |
*** mylu has quit IRC | 21:07 | |
*** tsekiyama has quit IRC | 21:07 | |
*** tsekiyam_ has quit IRC | 21:07 | |
glauco_ | timothyb89: This new chart shows data 'normalized'. I think it would be a complimentary information for the previous chart. | 21:08 |
ccarrara | timothyb89: you're correct about it being the same data, but it's showed in a different way that, as we believe, can help user see the failures behaviour along the time independently of number of runs. | 21:08 |
ccarrara | timothyb89: mtreinish andreaf it would be awesome to hear what a "real dashboard" user has to say about this kind of things. I mean, the way that data presentation can help them properly. | 21:10 |
timothyb89 | I agree, mtreinish may have some thoughts on that | 21:10 |
mtreinish | ccarrara: I tend to agree, a failure rate is definitely more noticeable than a failure count. if you look at the data on avg our failure rate is like 1% | 21:10 |
mtreinish | so the count is going to be a very small number | 21:11 |
*** regXboi has quit IRC | 21:11 | |
glauco_ | mtreinish: that might be the case of decrease the height of 'Build Failure Rate' chart, since there wouldn't be much variation on the information displayed on it | 21:14 |
*** dpaterson has quit IRC | 21:15 | |
*** pbredenb has quit IRC | 21:16 | |
mtreinish | glauco_: I think this looks fine: http://i.imgur.com/utUW94U.png | 21:16 |
mtreinish | although I kinda liked the blues before | 21:16 |
*** pbredenb has joined #openstack-qa | 21:16 | |
mtreinish | (you probably should isolate your patch to just adding the new graph not changing colors on existing ones) | 21:16 |
ccarrara | timothyb89: how are you going with replacement of highchart to nvd3? Anything we can help? | 21:16 |
glauco_ | mtreinish: we could rollback the color change. We would just like to choose colors that could give it a good contrast between 'good' and 'bad' | 21:17 |
timothyb89 | ccarrara: I am going to make a few quick fixes and resubmit in a bit | 21:17 |
glauco_ | mtreinish: sure! just included everything on the same bundle to give people a taste of what we were talking about. we will split it between smaller patches. | 21:17 |
timothyb89 | https://review.openstack.org/#/c/224992/ will need some re-tweaking as some bits were lost during a rebase, but I hope to have it up today as well | 21:18 |
glauco_ | mtreinish: do you have any suggestion on colors for success and failures that would provide a better contrast? | 21:18 |
mtreinish | timothyb89: well I can handle the rebase | 21:18 |
ccarrara | timothyb89: really good. Just asking because we're starting to have several patches that depends on that and if you need anything, let us know :) | 21:18 |
mtreinish | glauco_: sure it's all personal taste I guess, I personally would have gone blue and red, instead of green and red | 21:19 |
mtreinish | but I don't think it matters too much | 21:19 |
*** salv-orlando has joined #openstack-qa | 21:19 | |
glauco_ | mtreinish: got it! will update it accordingly! ;) | 21:19 |
timothyb89 | mtreinish: actually I may have misread the diff, looks like everything is fine | 21:19 |
mtreinish | ccarrara: heh, I was pestering him about that earlier :) | 21:19 |
clarkb | choosing colors that are color blind friendly is always good | 21:20 |
clarkb | (we havent been good about that historically) | 21:21 |
clarkb | red and green are bad together iirc | 21:21 |
ccarrara | nice point clarkb, thanks for remembering :) | 21:22 |
openstackgerrit | Glauco Oliveira proposed openstack/openstack-health: Add build failure rate chart on homepage https://review.openstack.org/230643 | 21:22 |
*** glauco_ has quit IRC | 21:22 | |
timothyb89 | I can go ahead and fix the coloring in my patch as well | 21:23 |
ccarrara | ^^ removing color changes | 21:23 |
ccarrara | timothyb89: as we change this point we think we should fix | 21:23 |
timothyb89 | changing green to blue? | 21:23 |
ccarrara | we keep all default (blue) | 21:24 |
ccarrara | maybe we should move forward this way for while | 21:24 |
ccarrara | as clarkb mentioned we need to think carefully about the color choices. | 21:25 |
mtreinish | ccarrara: this might be of interest: http://www.research.ibm.com/people/l/lloydt/color/color.HTM | 21:25 |
ccarrara | mtreinish: Wow, looks awesome. Thanks! :) Nice weekend reading. | 21:27 |
*** salv-orlando has quit IRC | 21:27 | |
clarkb | ccarrara: its probably not the most important thing but I know we have had users in the past complain about how some of our tools aren't so great with color | 21:28 |
ccarrara | clarkb: we think developing tools that is really inclusive is an important thing. You're completely right. Maybe we can move forward now with some simplistic choices, but we must to keep this in mind. | 21:30 |
openstackgerrit | Tim Buckley proposed openstack/openstack-health: Replace Highcharts with d3 and Nvd3 charts. https://review.openstack.org/229682 | 21:31 |
*** nadya has quit IRC | 21:31 | |
timothyb89 | ccarrara, mtreinish: I believe it should be good to go | 21:32 |
ccarrara | timothyb89: nice, looking :) | 21:33 |
timothyb89 | mtreinish: it still is not using a focus chart, I think I'd prefer to add that in a separate patch to keep things moving | 21:34 |
mtreinish | timothyb89: ok, there's a merge conflict on the live data one, let me fix that, and then I'll review | 21:34 |
mtreinish | timothyb89: sure thats fine, this gives us a starting point which is good | 21:34 |
ccarrara | timothyb89: I do agreed that is a good point that allow us move forward. Nice work :) | 21:38 |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Use live data on dashboard homepage https://review.openstack.org/224992 | 21:39 |
mtreinish | timothyb89: ^^^ the blue does look better, although it also changed the pie charts | 21:40 |
timothyb89 | I did that intentionally, but if that isn't what we want I can change it back | 21:40 |
mtreinish | no it doesn't really matter | 21:40 |
dspano | For the in-tree tests, are what used to be api now in the functional directory? | 21:41 |
ccarrara | mtreinish: than sounds that we're going to have some live data with d3 workflowed? Or not yet? | 21:42 |
dspano | I'm currently looking at heat's heat_integration directory. | 21:42 |
openstackrecheck | Console logs not available after 13:35s for gate-grenade-dsvm 224338,10,9d783e5 | 21:42 |
dspano | s/heat_integration/heat_integrationtests/ | 21:42 |
mtreinish | ccarrara: that imgur link I pasted above was with live data | 21:42 |
openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/223093 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1446583 | 21:42 |
openstack | Launchpad bug 1446583 in oslo.service "services no longer reliably stop in stable/liberty / master" [Critical,New] | 21:42 |
ccarrara | yes, we run with live data locally too. I was mentioning about the changes be merged into code. | 21:43 |
*** bnemec has joined #openstack-qa | 21:44 | |
mtreinish | oh, yeah we're probably pretty close to getting things merged | 21:44 |
timothyb89 | mtreinish: maybe I'm not crazy, it looks like the interval selector dropdown is missing? | 21:45 |
*** yamahata has quit IRC | 21:45 | |
mtreinish | I dropped it from the patch, I figured we could add it back later | 21:45 |
mtreinish | it's hard coded to hour write now | 21:45 |
timothyb89 | okay, fair enough | 21:45 |
*** ranger81 has joined #openstack-qa | 21:48 | |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Add tox env for running a local dev instance of the dashboard https://review.openstack.org/230209 | 21:50 |
*** dspano has quit IRC | 21:51 | |
*** mriedem has quit IRC | 21:51 | |
openstackgerrit | Tim Buckley proposed openstack/openstack-health: Fix applcation tile in `constants.js`. https://review.openstack.org/230659 | 21:51 |
mtreinish | timothyb89: ^^^ hah, I noticed that yesterday too | 21:51 |
timothyb89 | was starting to become annoying :) | 21:52 |
ccarrara | timothyb89: Are you also thinking that gauge charts are little bit small related to all the white space in panel body? | 21:58 |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Add build failure rate chart on homepage https://review.openstack.org/230643 | 21:58 |
timothyb89 | ccarrara: they are, there's not a good way to work around that with nvd3 that I could find | 21:58 |
*** mfisher_ora has quit IRC | 21:59 | |
timothyb89 | ccarrara: I figured I'd just throw something together quickly and we can replace it with a better gauge chart later | 21:59 |
ccarrara | timothyb89: same here.. I was digging into that, but unsuccessfully. | 21:59 |
mtreinish | timothyb89, ccarrara: the bigger win there would be to have the percentage back | 22:00 |
mtreinish | the size can be a secondary concern | 22:00 |
timothyb89 | that should be pretty simple | 22:00 |
ccarrara | mtreinish: like mockup? | 22:01 |
mtreinish | ccarrara: yeah, I think the mockup had the percentage per project | 22:02 |
ccarrara | mtreinish: yes. In the mockups the percentage is located below the chart. I think with d3 we'll can put the value "inside" the gauge. Like inside the blue and red part. | 22:04 |
mtreinish | david-lyle: so for your readme comment look at https://review.openstack.org/230209 | 22:04 |
timothyb89 | that is supported, I'll look into tweaking the patch | 22:05 |
mtreinish | timothyb89: just do it as a follow up | 22:05 |
timothyb89 | will do | 22:05 |
mtreinish | we've got a pretty good stack going already :) | 22:05 |
*** rossella_s has quit IRC | 22:06 | |
*** bnemec has quit IRC | 22:06 | |
*** rossella_s has joined #openstack-qa | 22:06 | |
openstackgerrit | Merged openstack/openstack-health: Replace Highcharts with d3 and Nvd3 charts. https://review.openstack.org/229682 | 22:06 |
openstackrecheck | Console logs not available after 13:23s for gate-grenade-dsvm 230617,1,2273413 | 22:07 |
*** yamahata has joined #openstack-qa | 22:07 | |
david-lyle | mtreinish: still missing the required piece for the manual steps | 22:08 |
david-lyle | otherwise looks good | 22:08 |
*** dimsum__ has quit IRC | 22:09 | |
mtreinish | david-lyle: oh, I can fix the readme there too, leave a comment with what I should change on it | 22:10 |
mtreinish | david-lyle: but I don't recomend doing it outside the tox job, its really weird. It took me a long time to get it all working on my local env | 22:10 |
mtreinish | I don't like the js packaging story | 22:10 |
david-lyle | mtreinish: I did add it | 22:11 |
david-lyle | I've gotten it working outside tox, but have to deviate from the README, that last addition should allow it to work, but I like the idea of nodeenv better | 22:11 |
ccarrara | mtreinish: do you mind if I get this task to improve readme thing? | 22:12 |
ccarrara | I can do it based on you tox task. | 22:13 |
david-lyle | ccarrara: https://review.openstack.org/#/c/230209/7/README.rst ? | 22:14 |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Add tox env for running a local dev instance of the dashboard https://review.openstack.org/230209 | 22:15 |
mtreinish | ccarrara: oh, sry I just pushed the update | 22:15 |
ccarrara | no problem. | 22:15 |
ccarrara | :) | 22:15 |
mtreinish | david-lyle: I might have gotten the npm syntax wrong | 22:15 |
mtreinish | david-lyle: yeah I had to do it manually too, just to figure out what to do with tox | 22:16 |
ccarrara | mtreinish: a patch I've submitted start show some merge conflicts, do I need to resolve it locally and submit a new patch? here -> https://review.openstack.org/#/c/230135/ | 22:17 |
*** sabeen3 has quit IRC | 22:18 | |
mtreinish | ccarrara: yeah, if there are merge conflicts you'll have to manually rebase things locally | 22:18 |
mtreinish | and then push it up to gerrit again | 22:18 |
ccarrara | Ok, I'll do it. | 22:18 |
david-lyle | mtreinish: I don't believe update works | 22:19 |
david-lyle | install | 22:19 |
david-lyle | just tried | 22:19 |
mtreinish | david-lyle: hmm, ok is it install then? | 22:20 |
openstackrecheck | Console logs not available after 13:22s for gate-grenade-dsvm 138664,9,7a5ed11 | 22:20 |
mtreinish | archlinux packaged a new enough version so I wasn't sure what I needed to use | 22:20 |
ccarrara | mtreinish: do you have any thought about recommend use of nvm in Readme? | 22:22 |
ccarrara | nvm = node version manager | 22:23 |
mtreinish | ccarrara: nope, I've never used it so I can't weigh whether we should recommend it or not | 22:23 |
timothyb89 | ccarrara: I believe it fills the same role as nodeenv? | 22:23 |
timothyb89 | which the tox environment uses now | 22:23 |
timothyb89 | that said we'll probably want to specify real node/npm versions, since we aren't currently compatible with the new node 4.0 release | 22:25 |
ccarrara | nvm do the same thing nodeenv apparently, but cannot integrate with virtualenv and I do not think it (node environment) should. Seems pretty different things and a good idea to keep them separated. | 22:26 |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Enable running under wsgi container https://review.openstack.org/225004 | 22:26 |
david-lyle | mtreinish: only install worked for me | 22:26 |
mtreinish | david-lyle: so change it to 'npm install npm'? | 22:27 |
david-lyle | just change update to install | 22:27 |
mtreinish | ok, will do | 22:27 |
david-lyle | npm install -g npm | 22:27 |
david-lyle | sudo npm install -g npm | 22:28 |
ccarrara | just a sudden thought, 'npm install -g npm' looks really weird. =P | 22:28 |
ccarrara | because if you're running npm, so it's already installed, no? | 22:28 |
openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Add tox env for running a local dev instance of the dashboard https://review.openstack.org/230209 | 22:28 |
mtreinish | david-lyle: ^^^ | 22:29 |
mtreinish | ccarrara: yes, they are different, but the idea is about creating an isolated env for doing everything needed to run things. Which on many systems(like mine) is important | 22:29 |
mtreinish | so with one command you get an isolated python and js env to run things from for dev | 22:29 |
*** ranger81 has left #openstack-qa | 22:30 | |
ccarrara | Yes, this is really important. But as a just personal option, I would prefer nvm. But if nodeenv is working fine to the project, this is most important :) | 22:30 |
mtreinish | ccarrara: well, you don't have to use the tox job. I just wanted a simple option for people not familiar with the workflow to experiment with it | 22:32 |
david-lyle | well npm is a weird package management system at best | 22:32 |
david-lyle | yes polluting the system libraries is not good | 22:32 |
clarkb | +1 | 22:32 |
ccarrara | david-lyle: agree. I was not recommending to use system environment at all, just some ideas about different version managers ;) | 22:34 |
openstackrecheck | Console logs not available after 13:25s for gate-tempest-dsvm-neutron-full-kilo 230604,2,4169e5a | 22:34 |
*** rustlebee is now known as russellb | 22:39 | |
david-lyle | ccarrara: understood, hadn't read all the conversation | 22:39 |
ccarrara | david-lyle: was nice to see we all agree about this subject anyway :) | 22:40 |
david-lyle | :D | 22:40 |
*** david-lyle has quit IRC | 22:45 | |
*** edmondsw has quit IRC | 22:46 | |
*** david-lyle has joined #openstack-qa | 22:47 | |
mtreinish | ccarrara: so you seem to like the js stuff (at least more then me :) ) do you think you could have a try at adding this to the line chart directive: http://www.d3noob.org/2014/04/adding-in-zero-values-into-time-series.html | 22:47 |
mtreinish | probably as an option, because we don't likely want that on the percentage graph only the real time series ones like the success counts | 22:48 |
openstackgerrit | Caio Carrara proposed openstack/openstack-health: Add second page (project) https://review.openstack.org/230135 | 22:50 |
ccarrara | mtreinish: I definitely can try it. Just to see if I understand, you want this only on the first chart (failures and success), right? | 22:55 |
*** amotoki has joined #openstack-qa | 22:59 | |
*** pglass has quit IRC | 23:01 | |
*** dimsum__ has joined #openstack-qa | 23:02 | |
mtreinish | ccarrara: yeah, just on the first chart | 23:02 |
ccarrara | nice, I'm already digging into it. :) | 23:03 |
*** ccneill has quit IRC | 23:03 | |
*** amotoki has quit IRC | 23:04 | |
mtreinish | ccarrara: the line graph is drawing between points even if there is missing data which makes the counts a bit misleading | 23:07 |
mtreinish | but for a percentage view (on the second graph) thats fine and probably desireable | 23:07 |
ccarrara | yes, I got it ;) | 23:08 |
ccarrara | mtreinish: I need to go now, hope bring some nice result related with it. Let us know if you need anything else ;) | 23:11 |
*** jtrovo has quit IRC | 23:12 | |
*** ccarrara has left #openstack-qa | 23:12 | |
*** Swami has quit IRC | 23:17 | |
*** achanda_ has joined #openstack-qa | 23:22 | |
*** hemna is now known as hemnafk | 23:24 | |
*** achanda has quit IRC | 23:25 | |
*** achanda_ has quit IRC | 23:26 | |
*** dimsum__ has quit IRC | 23:46 | |
openstackrecheck | openstack/glance change: https://review.openstack.org/229973 failed because of: gate-tempest-dsvm-full-ceph: https://bugs.launchpad.net/bugs/1475333 | 23:56 |
openstack | Launchpad bug 1475333 in Cinder "gate-tempest-dsvm-full-ceph fails with Insufficient free space for volume creation on host ubuntu@ceph#ceph (requested / avail): 1/0.0" [High,Incomplete] | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!