| lifeless | sdague: you have seen that testtools 1.3.0 has fixed it all, right ? | 00:01 |
|---|---|---|
| lifeless | sdague: (wondering why you're rechecking etc etc) | 00:01 |
| *** yamamoto has quit IRC | 00:01 | |
| *** yamamoto has joined #openstack-qa | 00:02 | |
| lifeless | sdague: [there is still an issue for trove specifically, because 2.6 is now running setUpClass and their code upcalls, which I'll have fixed shortly) | 00:02 |
| lifeless | sdague: see https://review.openstack.org/#/c/134730/ | 00:03 |
| *** yamamoto has quit IRC | 00:04 | |
| openstackgerrit | Ian Wienand proposed openstack-dev/devstack: Remove nose things https://review.openstack.org/134733 | 00:13 |
| *** dmorita has joined #openstack-qa | 00:29 | |
| *** yamamoto has joined #openstack-qa | 00:31 | |
| *** dims has quit IRC | 00:32 | |
| *** dims has joined #openstack-qa | 00:32 | |
| *** dgonzalez has joined #openstack-qa | 00:33 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/129391 failed because of: gate-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/1392860 | 00:37 |
| *** dgonzalez has quit IRC | 00:41 | |
| *** markmcclain has quit IRC | 00:42 | |
| *** markmcclain has joined #openstack-qa | 00:43 | |
| *** mkollaro has quit IRC | 00:45 | |
| *** topshare has joined #openstack-qa | 00:54 | |
| *** nosnos has joined #openstack-qa | 01:01 | |
| *** weshay has quit IRC | 01:03 | |
| *** Yogi has joined #openstack-qa | 01:07 | |
| *** X019 has joined #openstack-qa | 01:19 | |
| *** achanda has quit IRC | 01:39 | |
| *** wuhg has joined #openstack-qa | 01:40 | |
| *** achanda has joined #openstack-qa | 01:43 | |
| *** yamahata has joined #openstack-qa | 01:46 | |
| *** yamahata has quit IRC | 01:52 | |
| *** yamahata has joined #openstack-qa | 01:53 | |
| *** achanda has quit IRC | 02:13 | |
| openstackgerrit | Masayuki Igawa proposed openstack-dev/grenade: Remove commented out codes and files related to upgrade-packages https://review.openstack.org/128299 | 02:16 |
| *** adalbas has quit IRC | 02:17 | |
| *** dims has quit IRC | 02:18 | |
| *** dims has joined #openstack-qa | 02:19 | |
| *** dims has quit IRC | 02:25 | |
| *** dims has joined #openstack-qa | 02:25 | |
| *** oomichi has joined #openstack-qa | 02:28 | |
| *** pcrews has quit IRC | 02:31 | |
| *** X019 has quit IRC | 02:32 | |
| *** X019 has joined #openstack-qa | 02:44 | |
| *** zz_dimtruck is now known as dimtruck | 02:46 | |
| openstackgerrit | Merged openstack/tempest: Decouple resources and preconditions in secgroup scenario https://review.openstack.org/129542 | 02:46 |
| *** jyuso has joined #openstack-qa | 02:52 | |
| *** salv-orlando has quit IRC | 03:00 | |
| *** Yogi has quit IRC | 03:09 | |
| *** achanda has joined #openstack-qa | 03:14 | |
| *** amotoki has joined #openstack-qa | 03:18 | |
| *** achanda has quit IRC | 03:19 | |
| *** dims_ has joined #openstack-qa | 03:20 | |
| *** dims has quit IRC | 03:23 | |
| *** Longgeek has joined #openstack-qa | 03:28 | |
| *** dims_ has quit IRC | 03:29 | |
| *** dims has joined #openstack-qa | 03:29 | |
| *** Longgeek has quit IRC | 03:33 | |
| *** yamamoto has quit IRC | 03:37 | |
| *** markmcclain has quit IRC | 03:43 | |
| *** markmcclain has joined #openstack-qa | 03:44 | |
| *** nosnos has quit IRC | 03:49 | |
| *** ramishra has joined #openstack-qa | 03:51 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack-dev/devstack: Add TEMPEST_ENABLE_NOVA_XML_API option https://review.openstack.org/134851 | 04:06 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack-dev/devstack: Add TEMPEST_ENABLE_NOVA_XML_API option https://review.openstack.org/134851 | 04:07 |
| *** cmyster has quit IRC | 04:19 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack-infra/devstack-gate: WIP: Specify Nova XML API tests clearly for juno/icehouse https://review.openstack.org/134852 | 04:22 |
| *** rushiagr_away is now known as rushiagr | 04:22 | |
| *** cmyster has joined #openstack-qa | 04:23 | |
| *** wuhg has quit IRC | 04:23 | |
| *** vigneshvar has joined #openstack-qa | 04:25 | |
| *** wuhg has joined #openstack-qa | 04:26 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Disable Nova API XML tests as the default https://review.openstack.org/126815 | 04:28 |
| *** vigneshvar has quit IRC | 04:31 | |
| *** yamahata_ has quit IRC | 04:38 | |
| *** yamahata_ has joined #openstack-qa | 04:39 | |
| *** yamahata_ has quit IRC | 04:40 | |
| *** isq has joined #openstack-qa | 04:40 | |
| *** Longgeek has joined #openstack-qa | 04:44 | |
| *** nosnos has joined #openstack-qa | 04:46 | |
| *** vigneshvar has joined #openstack-qa | 04:46 | |
| *** mmedvede has quit IRC | 04:48 | |
| *** Longgeek has quit IRC | 04:48 | |
| *** mmedvede has joined #openstack-qa | 04:49 | |
| *** vponomaryov has quit IRC | 04:50 | |
| *** vponomaryov has joined #openstack-qa | 04:51 | |
| *** achanda has joined #openstack-qa | 04:53 | |
| *** oomichi has quit IRC | 05:11 | |
| *** pcrews has joined #openstack-qa | 05:17 | |
| *** chandankumar has joined #openstack-qa | 05:20 | |
| *** dimtruck is now known as zz_dimtruck | 05:21 | |
| openstackrecheck | Console logs not available after 13:29s for gate-nova-python27 123309,6,f5738bf | 05:22 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/133165 failed gate-grenade-dsvm in the gate queue with an unrecognized error | 05:22 |
| *** bkopilov has quit IRC | 05:27 | |
| *** jamielennox is now known as jamielennox|away | 05:34 | |
| *** neeti has joined #openstack-qa | 05:35 | |
| *** Longgeek has joined #openstack-qa | 05:37 | |
| *** jamielennox|away is now known as jamielennox | 05:41 | |
| *** ashishg has joined #openstack-qa | 05:41 | |
| *** zz_dimtruck is now known as dimtruck | 05:42 | |
| ashishg | Please review https://review.openstack.org/#/c/109529/ | 05:42 |
| *** bkopilov has joined #openstack-qa | 05:50 | |
| *** nosnos_ has joined #openstack-qa | 05:50 | |
| *** nosnos has quit IRC | 05:51 | |
| *** dimtruck is now known as zz_dimtruck | 05:54 | |
| *** yfried has quit IRC | 05:54 | |
| *** pcrews has quit IRC | 05:54 | |
| *** nosnos_ has quit IRC | 05:57 | |
| *** abehl has joined #openstack-qa | 05:58 | |
| *** nosnos has joined #openstack-qa | 05:59 | |
| *** mdbooth has quit IRC | 05:59 | |
| openstackgerrit | Eli Qiao proposed openstack/tempest: Add Conflict exception in test_floating_ips_actions case https://review.openstack.org/131364 | 06:00 |
| *** hogepodge has quit IRC | 06:01 | |
| openstackgerrit | chandan kumar proposed openstack/tempest: enable cinder v2 api for volumetype test https://review.openstack.org/126213 | 06:03 |
| *** mdbooth has joined #openstack-qa | 06:04 | |
| *** Longgeek has quit IRC | 06:17 | |
| *** Longgeek has joined #openstack-qa | 06:18 | |
| *** Longgeek_ has joined #openstack-qa | 06:18 | |
| *** Longgeek has quit IRC | 06:22 | |
| *** nosnos_ has joined #openstack-qa | 06:23 | |
| *** nosnos has quit IRC | 06:23 | |
| *** hogepodge has joined #openstack-qa | 06:24 | |
| openstackgerrit | lokesh s proposed openstack-dev/devstack: fixes cleaning up of ironic directory from /etc https://review.openstack.org/133413 | 06:26 |
| openstackgerrit | chandan kumar proposed openstack/tempest: Enable cinder v2 api for volume hosts https://review.openstack.org/133947 | 06:28 |
| *** ajo has joined #openstack-qa | 06:35 | |
| *** ajo has quit IRC | 06:36 | |
| *** ajo has joined #openstack-qa | 06:36 | |
| *** nmagnezi has joined #openstack-qa | 06:40 | |
| *** markmcclain has quit IRC | 06:45 | |
| *** markmcclain has joined #openstack-qa | 06:46 | |
| *** Longgeek_ has quit IRC | 06:56 | |
| *** Longgeek has joined #openstack-qa | 06:57 | |
| *** markmcclain has quit IRC | 07:01 | |
| *** achanda has quit IRC | 07:02 | |
| *** markmcclain has joined #openstack-qa | 07:08 | |
| *** yfried has joined #openstack-qa | 07:21 | |
| *** Longgeek has quit IRC | 07:27 | |
| *** Longgeek has joined #openstack-qa | 07:28 | |
| *** yfried is now known as yfried|afk | 07:35 | |
| *** yfried|afk is now known as yfried | 07:40 | |
| *** Longgeek_ has joined #openstack-qa | 07:41 | |
| *** tkammer has joined #openstack-qa | 07:42 | |
| *** safchain has joined #openstack-qa | 07:44 | |
| *** Longgeek has quit IRC | 07:45 | |
| *** ramishra has quit IRC | 07:46 | |
| *** e0ne has joined #openstack-qa | 07:47 | |
| *** e0ne has quit IRC | 07:50 | |
| *** dgonzalez has joined #openstack-qa | 07:50 | |
| yfried | masayukig: that's a doc change that could use a +A https://review.openstack.org/#/c/133471/ | 07:51 |
| openstackgerrit | Cyril Roelandt proposed openstack/tempest: Respect auth.allow_tenant_isolation https://review.openstack.org/134259 | 07:54 |
| *** cmyster has left #openstack-qa | 07:55 | |
| *** sridhar has joined #openstack-qa | 08:02 | |
| *** ajo has quit IRC | 08:03 | |
| *** avico has joined #openstack-qa | 08:05 | |
| *** tkammer has quit IRC | 08:05 | |
| *** SridharG has joined #openstack-qa | 08:11 | |
| *** dgonzalez has quit IRC | 08:14 | |
| *** yamahata has quit IRC | 08:20 | |
| *** markus_z has joined #openstack-qa | 08:21 | |
| *** ajo has joined #openstack-qa | 08:23 | |
| openstackgerrit | Sergey Shnaidman proposed openstack/tempest: Add .idea to .gitignore https://review.openstack.org/134874 | 08:26 |
| *** dgonzalez has joined #openstack-qa | 08:26 | |
| *** salv-orlando has joined #openstack-qa | 08:29 | |
| openstackgerrit | Sergey Shnaidman proposed openstack/tempest-lib: Add .idea to .gitignore https://review.openstack.org/134875 | 08:29 |
| *** dgonzalez has quit IRC | 08:32 | |
| *** jordanP has joined #openstack-qa | 08:33 | |
| *** jordanP has quit IRC | 08:33 | |
| *** jordanP has joined #openstack-qa | 08:33 | |
| *** luqas has joined #openstack-qa | 08:34 | |
| *** tkammer has joined #openstack-qa | 08:37 | |
| *** jlanoux has joined #openstack-qa | 08:46 | |
| openstackgerrit | Jordan Pittier proposed openstack/tempest: Make every swift clients use expected_success https://review.openstack.org/134555 | 08:48 |
| *** jpich has joined #openstack-qa | 08:52 | |
| kashyap | ianw, Hi, when you get a moment, this is with current DevStack git: http://paste.openstack.org/show/134035/ | 08:54 |
| kashyap | It's a kind of yum failure, as PIPESTATUS returned 1, instead of 0. | 08:55 |
| kashyap | Looking at devstack/functions-common, not sure what exactly is yum missing there - from the stack trace. I wonder if you have any clues. | 08:56 |
| *** tkammer has quit IRC | 09:06 | |
| jpich | kashyap: Hey! I had a similar issue with the python-ecdsa package recently, I *think* it's because devstack went from installing it from pip to installing it as a package, so when recycling a devstack VM it causes trouble | 09:07 |
| kashyap | jpich, Morning, | 09:08 |
| jpich | removing the version that's already installed then installing the package should resolve it, and after that devstack should run ok | 09:08 |
| kashyap | jpich, I see, good to know. Thanks for the info, I'm checking what's installed on it from a different guest console. | 09:09 |
| kashyap | jpich, I don't have an installed version of python-ecdsa. I just did something crude: remove all the pip directories from /usr/lib/python2.7/site-packages and re-issued a new DevStack run on my F21. | 09:11 |
| jpich | kashyap: Ok! For a quicker check you can also try installing the package manually rather than re-run devstack | 09:12 |
| *** DaSchab has joined #openstack-qa | 09:14 | |
| kashyap | jpich, Yup, noted. I need this install anyhow :-) | 09:14 |
| jpich | kashyap: Good luck! :-) | 09:14 |
| *** tkammer has joined #openstack-qa | 09:21 | |
| kashyap | jpich, Yes, successful DevStack on my re-run. And, you're right - seems like no python-ecdsa RPM package, and no trace of it in my devstack log either. | 09:23 |
| kashyap | ianw, Please disregard me. | 09:23 |
| *** DaSchab has quit IRC | 09:23 | |
| jpich | kashyap: Glad it worked :) | 09:24 |
| *** ashishg has quit IRC | 09:25 | |
| *** mpaolino has joined #openstack-qa | 09:26 | |
| *** e0ne has joined #openstack-qa | 09:31 | |
| *** ramishra_ has joined #openstack-qa | 09:31 | |
| *** zz_johnthetubagu is now known as johnthetubaguy | 09:40 | |
| *** ramishra_ has joined #openstack-qa | 09:41 | |
| openstackgerrit | yolanda.robla proposed openstack-infra/devstack-gate: Add option for is_debian to support it https://review.openstack.org/130768 | 09:41 |
| *** flaper87 has quit IRC | 09:43 | |
| *** flaper87 has joined #openstack-qa | 09:43 | |
| *** markmcclain has quit IRC | 09:46 | |
| *** markmcclain has joined #openstack-qa | 09:47 | |
| *** dkusidlo has joined #openstack-qa | 09:48 | |
| *** DaSchab has joined #openstack-qa | 09:49 | |
| *** yfried is now known as yfried|afk | 09:52 | |
| *** DaSchab has quit IRC | 09:58 | |
| *** DaSchab has joined #openstack-qa | 09:58 | |
| *** DaSchab has quit IRC | 10:00 | |
| openstackgerrit | chandan kumar proposed openstack/tempest: Enable cinder v2 api tests in volume quotas https://review.openstack.org/134889 | 10:00 |
| *** yfried|afk is now known as yfried | 10:01 | |
| *** yamahata has joined #openstack-qa | 10:02 | |
| *** achanda has joined #openstack-qa | 10:06 | |
| *** achanda has quit IRC | 10:10 | |
| *** dkusidlo has quit IRC | 10:11 | |
| *** tkammer has quit IRC | 10:11 | |
| *** yfried is now known as yfried|afk | 10:12 | |
| *** dkusidlo has joined #openstack-qa | 10:15 | |
| *** dkusidlo has quit IRC | 10:15 | |
| *** DaSchab has joined #openstack-qa | 10:16 | |
| *** yfried|afk is now known as yfried | 10:18 | |
| *** tkammer has joined #openstack-qa | 10:25 | |
| *** yfried is now known as yfried|afk | 10:28 | |
| openstackgerrit | Nikolay Fedotov proposed openstack/tempest: Scenario tests for FWaaS https://review.openstack.org/134896 | 10:28 |
| *** yfried|afk is now known as yfried | 10:30 | |
| *** gfidente has joined #openstack-qa | 10:33 | |
| openstackgerrit | Nikolay Fedotov proposed openstack/tempest: Scenario tests for FWaaS https://review.openstack.org/134896 | 10:34 |
| *** topshare has quit IRC | 10:34 | |
| *** yfried has quit IRC | 10:39 | |
| *** tkammer has quit IRC | 10:39 | |
| *** yfried has joined #openstack-qa | 10:39 | |
| *** abehl has quit IRC | 10:42 | |
| sdague | lifeless: so the fact that 1.3.0 fixes things is good. It still doesn't make 1.2 a working version. So this will just block out that one version. | 10:44 |
| *** aysyd has joined #openstack-qa | 10:49 | |
| *** yfried is now known as yfried|afk | 10:50 | |
| *** yfried|afk is now known as yfried | 10:51 | |
| *** tkammer has joined #openstack-qa | 10:54 | |
| *** dgonzalez has joined #openstack-qa | 10:54 | |
| *** abehl has joined #openstack-qa | 10:56 | |
| *** chandankumar has quit IRC | 10:56 | |
| *** mkollaro has joined #openstack-qa | 10:57 | |
| *** abehl has quit IRC | 10:58 | |
| *** abehl has joined #openstack-qa | 10:58 | |
| *** adalbas has joined #openstack-qa | 11:01 | |
| jordanP | yfried, hi. About https://review.openstack.org/#/c/134615/1/tempest/scenario/test_shelve_instance.py to you suggest to change the definition of get_remote_client in manager.py ? | 11:03 |
| yfried | jordanP: I'll talk to you in a few minutes ok? | 11:05 |
| jordanP | yfried, sure | 11:06 |
| *** DaSchab has quit IRC | 11:06 | |
| *** chandankumar has joined #openstack-qa | 11:08 | |
| *** DaSchab has joined #openstack-qa | 11:12 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/129391 failed because of: gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1374175 https://bugs.launchpad.net/bugs/1297560 | 11:15 |
| *** amotoki has quit IRC | 11:19 | |
| *** cdent has joined #openstack-qa | 11:19 | |
| *** dgonzalez has quit IRC | 11:22 | |
| *** dgonzalez has joined #openstack-qa | 11:23 | |
| *** rfolco has joined #openstack-qa | 11:23 | |
| *** dgonzale_ has joined #openstack-qa | 11:27 | |
| *** dgonzalez has quit IRC | 11:28 | |
| *** yamahata has quit IRC | 11:30 | |
| openstackgerrit | Joseph Lanoux proposed openstack/tempest: Add MultiNic test https://review.openstack.org/134908 | 11:30 |
| *** vigneshvar has quit IRC | 11:30 | |
| *** yamahata has joined #openstack-qa | 11:31 | |
| *** dgonzale_ has quit IRC | 11:34 | |
| *** dgonzalez has joined #openstack-qa | 11:35 | |
| *** rfolco has quit IRC | 11:36 | |
| *** tkammer has quit IRC | 11:38 | |
| *** yamahata has quit IRC | 11:39 | |
| *** dgonzalez has quit IRC | 11:39 | |
| *** yfried_ has joined #openstack-qa | 11:44 | |
| yfried_ | jordanP: I'm back | 11:44 |
| *** yfried has quit IRC | 11:44 | |
| yfried_ | jordanP: I'm backjordanP: re get_remote_client - your code seems reasonable enough to be used every time the method is called | 11:44 |
| yfried_ | jordanP: so either this is already the case by some hidden stuff, or this should be implemented in the manager for all to use | 11:44 |
| jordanP | yfried_, i borrowed this code from tempest/scenario/test_snapshot_pattern.py | 11:45 |
| jordanP | i remember there was a lot of failure with test_snapshot_pattern.py and that the try/catch were added to adresse these failures | 11:46 |
| jordanP | not address but help debug | 11:46 |
| yfried_ | jordanP: so move it from there to manager. Unless you can think of a reason not to test that? | 11:46 |
| jordanP | I can think of no reason... so I will do it in a dedicated patch | 11:47 |
| *** openstackgerrit has quit IRC | 11:48 | |
| *** openstackgerrit has joined #openstack-qa | 11:49 | |
| *** dims has quit IRC | 11:50 | |
| *** tkammer has joined #openstack-qa | 11:50 | |
| *** dims has joined #openstack-qa | 11:50 | |
| jordanP | yfried_, I don't agree with your comment "Please consider adding this as another test case in test_network_advanced_server_ops" so I am pushing a new patchset as is, I am waiting for other review | 11:51 |
| *** DaSchab has quit IRC | 11:52 | |
| yfried_ | jordanP: could you please explain why you don't agree? | 11:53 |
| *** DaSchab has joined #openstack-qa | 11:53 | |
| *** dgonzalez has joined #openstack-qa | 11:53 | |
| *** dgonzalez has quit IRC | 11:53 | |
| jordanP | yfried_, well the shelve/unshelve feature is not related to network | 11:53 |
| *** dgonzalez has joined #openstack-qa | 11:53 | |
| yfried_ | jordanP: yes, but your test seems to do the same thing - modify machine and check connectivity, adding only the timestamp test | 11:54 |
| *** dgonzalez has quit IRC | 11:54 | |
| *** DaSchab has quit IRC | 11:55 | |
| *** dgonzalez has joined #openstack-qa | 11:55 | |
| jordanP | yfried_, I know but same as ./test_stamp_pattern.py and ./test_volume_boot_pattern.py | 11:56 |
| yfried_ | jordanP: we need to refactor the scenario tree to solve this kind of issue | 11:57 |
| yfried_ | jordanP: you should be able to share this code even though these tests are unrelated | 11:57 |
| jordanP | I will support you on that :) but it will take time and in the mean time i'd like to move forward with this... | 11:58 |
| yfried_ | jordanP: I agree. just venting... | 11:58 |
| *** dgonzalez has quit IRC | 11:59 | |
| *** yamahata has joined #openstack-qa | 12:02 | |
| *** markus_z has quit IRC | 12:04 | |
| openstackgerrit | Jordan Pittier proposed openstack/tempest: Scenario manager: catch Exception in get_remote_client https://review.openstack.org/134914 | 12:09 |
| *** rfolco has joined #openstack-qa | 12:10 | |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: add LIBS_FROM_GIT to docs https://review.openstack.org/134915 | 12:11 |
| *** mpaolino has quit IRC | 12:17 | |
| openstackgerrit | Jordan Pittier proposed openstack/tempest: Add a shelve/unshelve scenario https://review.openstack.org/134615 | 12:17 |
| *** jaypipes has joined #openstack-qa | 12:18 | |
| *** ChanServ sets mode: +o jaypipes | 12:18 | |
| *** dgonzalez has joined #openstack-qa | 12:19 | |
| *** markus_z has joined #openstack-qa | 12:19 | |
| *** dgonzalez has quit IRC | 12:23 | |
| *** rfolco has quit IRC | 12:23 | |
| *** jyuso has quit IRC | 12:28 | |
| *** Yogi has joined #openstack-qa | 12:28 | |
| *** dmorita has quit IRC | 12:29 | |
| yfried_ | jordanP: I'm still concerned about the network issue | 12:34 |
| yfried_ | jordanP: seems like you are using the default network_resources provided by isolated_creds. are you intending to do so? | 12:34 |
| *** nosnos_ has quit IRC | 12:39 | |
| *** markmcclain has quit IRC | 12:47 | |
| *** DaSchab has joined #openstack-qa | 12:48 | |
| *** markmcclain has joined #openstack-qa | 12:48 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: Skip additional tests when xml_api_v2 is set to False https://review.openstack.org/134924 | 12:50 |
| jordanP | yfried_, i don't understand, "I'm still concerned about the network issue" ? | 12:59 |
| jordanP | the code is similar to test_stamp_pattern.py | 13:00 |
| *** yfried_ is now known as yfried|afk | 13:03 | |
| *** yfried|afk is now known as yfried_ | 13:04 | |
| yfried_ | jordanP: I'm unfamiliar with that test | 13:04 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack-infra/devstack-gate: Switch to enable/disable Nova V2 XML Support https://review.openstack.org/134714 | 13:05 |
| jordanP | yfried_, me too :) | 13:05 |
| yfried_ | jordanP: did you test it on neutron or nova-network? | 13:05 |
| jordanP | neutron | 13:05 |
| *** DaSchab has quit IRC | 13:05 | |
| yfried_ | jordanP: and it used the tenant network created by isolated_creds? | 13:05 |
| yfried_ | mtreinish: ping | 13:06 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: Switch off Nova V2 XML Support https://review.openstack.org/134685 | 13:06 |
| *** DaSchab has joined #openstack-qa | 13:07 | |
| jordanP | yfried_, I dont know.. | 13:08 |
| *** Yogi has quit IRC | 13:08 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/134751 failed because of: gate-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/1392860 | 13:10 |
| *** dgonzalez has joined #openstack-qa | 13:11 | |
| *** mpaolino has joined #openstack-qa | 13:12 | |
| *** rushiagr is now known as rushiagr_away | 13:12 | |
| *** dgonzalez has quit IRC | 13:12 | |
| sdague | dtroyer: once you get up, some template hacking in shocco I think is required | 13:14 |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: add LIBS_FROM_GIT to docs https://review.openstack.org/134915 | 13:15 |
| *** yfried_ is now known as yfried|afk | 13:16 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: Switch off Nova V2 XML Support https://review.openstack.org/134685 | 13:16 |
| *** dkranz has joined #openstack-qa | 13:26 | |
| *** markus_z has quit IRC | 13:26 | |
| *** tristanC_ is now known as tristanC | 13:27 | |
| openstackgerrit | John Davidge proposed openstack-dev/devstack: Add IPv6 support for tenant data network https://review.openstack.org/87987 | 13:35 |
| openstackgerrit | Joseph Lanoux proposed openstack/tempest: Add MultiNic test https://review.openstack.org/134908 | 13:35 |
| *** yfried|afk is now known as yfried_ | 13:37 | |
| *** Yogi has joined #openstack-qa | 13:38 | |
| *** markus_z has joined #openstack-qa | 13:39 | |
| *** k4n0_ has quit IRC | 13:39 | |
| *** Yogi has quit IRC | 13:44 | |
| *** DaSchab has quit IRC | 13:44 | |
| mtreinish | yfried_: pong | 13:48 |
| *** yfried_ is now known as yfried|afk | 13:48 | |
| *** dims has quit IRC | 13:52 | |
| *** dims has joined #openstack-qa | 13:53 | |
| *** dkranz has quit IRC | 13:55 | |
| *** abehl has quit IRC | 13:56 | |
| *** dkranz has joined #openstack-qa | 14:00 | |
| *** neeti has quit IRC | 14:02 | |
| openstackrecheck | openstack/sahara change: https://review.openstack.org/134336 failed because of: gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1374175 https://bugs.launchpad.net/bugs/1297560 | 14:02 |
| sdague | so heat jobs are failing a lot again | 14:05 |
| *** dgonzalez has joined #openstack-qa | 14:06 | |
| *** dustins has joined #openstack-qa | 14:07 | |
| *** cdent has quit IRC | 14:08 | |
| *** dgonzalez has quit IRC | 14:09 | |
| *** yfried|afk is now known as yfried_ | 14:10 | |
| yfried_ | sdague: https://review.openstack.org/#/c/130411/ is still pending since your -1 | 14:10 |
| *** krtaylor has joined #openstack-qa | 14:11 | |
| *** jgriffit1 is now known as jgriffith | 14:16 | |
| sdague | yfried_: ok, well I'm still -1, but for different reasons | 14:16 |
| sdague | I did really want that squashed into the last commit as well, but it's too later for that | 14:17 |
| *** jgriffith is now known as Guest10117 | 14:17 | |
| *** Guest10117 is now known as jdg | 14:18 | |
| *** Yogi has joined #openstack-qa | 14:20 | |
| *** zz_dimtruck is now known as dimtruck | 14:20 | |
| *** jdg is now known as jgriffith | 14:20 | |
| *** Yogi has quit IRC | 14:21 | |
| *** Yogi has joined #openstack-qa | 14:21 | |
| yfried_ | sdague: tnx | 14:22 |
| *** vigneshvar has joined #openstack-qa | 14:23 | |
| *** toure has joined #openstack-qa | 14:26 | |
| yfried_ | does anyone know why we have both "image_ssh_user" and "ssh_user" in tempest.conf? | 14:26 |
| *** toure_ has joined #openstack-qa | 14:32 | |
| *** mriedem has joined #openstack-qa | 14:34 | |
| *** toure has quit IRC | 14:35 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/134561 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error | 14:36 |
| *** DaSchab has joined #openstack-qa | 14:36 | |
| yfried_ | dkranz: ^ | 14:38 |
| *** toure has joined #openstack-qa | 14:38 | |
| dkranz | yfried_: I am about to submit a fix to not use the compute one in the scenario manager. | 14:38 |
| dkranz | yfried_: I don't know why we have both | 14:39 |
| openstackgerrit | David Kranz proposed openstack/tempest: Do not force use of [compute] image_ssh_user in scenario tests https://review.openstack.org/134952 | 14:39 |
| yfried_ | dkranz: so wouldn't a better fix would be to remove one of them? | 14:39 |
| dkranz | yfried_: It is not clear that it is wrong to have both | 14:40 |
| openstackgerrit | Ashish Kumar Gupta proposed openstack/tempest: Floating IP Negative Tests https://review.openstack.org/109529 | 14:40 |
| dkranz | yfried_: but having the scenario manager use the one in compute is clearly wrong which is what I just proposed a fix for | 14:40 |
| yfried_ | dkranz: so either we make it clear and update the docs, or we find out which is redundant and we remove it? | 14:41 |
| *** toure_ has quit IRC | 14:41 | |
| dkranz | yfried_: The problem is that they are clearly redundant in the gate because they are set to the same value. But that doesn't mean they are redundant more generally. | 14:42 |
| *** cdent has joined #openstack-qa | 14:42 | |
| yfried_ | dkranz: I know. which is why I'm trying to figure this out | 14:42 |
| dkranz | yfried_: You are free to come up with a theory :) | 14:42 |
| yfried_ | dkranz: who would know? mtreinish? | 14:42 |
| dkranz | yfried_: My guess is that this is very old. | 14:42 |
| yfried_ | dkranz: obviously :) | 14:43 |
| mtreinish | dkranz: if you want to converge options you can deprecate one | 14:43 |
| dkranz | yfried_: however I could image wanting to run scenario tests with a real image and the compute tests with cirros | 14:43 |
| *** dimtruck is now known as zz_dimtruck | 14:43 | |
| mtreinish | look at the auth section and how we handled tenant isolation | 14:43 |
| *** rushiagr_away is now known as rushiagr | 14:44 | |
| yfried_ | dkranz: by default, scenarios boot the image from CONF.compute.image_ref, so it makes sense to use its ssh_user as well | 14:45 |
| yfried_ | dkranz: what's up with CONF.input_scenario.ssh_user_regex? | 14:45 |
| *** bknudson has joined #openstack-qa | 14:46 | |
| *** chandankumar has quit IRC | 14:47 | |
| dkranz | yfried_: ok, so I abandoned the change | 14:47 |
| sdague | mtreinish: so... heat is off the rails on tests again? | 14:47 |
| yfried_ | dkranz: please don't. let's figure this out | 14:48 |
| mtreinish | sdague: I really dunno, I've been online for ~1hr | 14:50 |
| mtreinish | I wouldn't be surprised though | 14:50 |
| sdague | so, top 3 recheck bugs | 14:50 |
| sdague | 2 & 3 are heat | 14:50 |
| dkranz | yfried_: I just want to make sure this is correct. When we figure out I can resubmit. | 14:50 |
| sdague | 1 is a console generated negative test | 14:50 |
| sdague | which I thought mriedem was going to propose a skip for | 14:50 |
| dkranz | yfried_: It is clearly wrong that [scenario] specifies an ssh user but not an image | 14:51 |
| mriedem | sdague: except it's a generated test, so wasn't sure how to skip, w/o skipping the entire class | 14:51 |
| mtreinish | dkranz: IIRC the scenario section is mostly used for the scenario test which uses testscenarios | 14:52 |
| dkranz | mtreinish: That is certainly not "obvious" :) | 14:52 |
| sdague | dkranz: can you weigh in a bit on GetConsoleOutputNegativeTestJSON and whether what it's doing here is valid | 14:52 |
| *** weshay has joined #openstack-qa | 14:53 | |
| dkranz | sdague: ok, I need to read back | 14:53 |
| sdague | http://logs.openstack.org/96/134596/1/check/check-tempest-dsvm-neutron-full/01fbe23/console.html.gz | 14:53 |
| sdague | that's happening a bunch | 14:53 |
| * dkranz looking | 14:53 | |
| sdague | but it's auto generated negative testing | 14:53 |
| sdague | which makes figuring out what it's doing, and if it's correctly failing, hard | 14:53 |
| dkranz | sdague: Did this start recently? | 14:53 |
| sdague | yes | 14:53 |
| dkranz | sdague: ok, let me investigate | 14:54 |
| sdague | http://status.openstack.org/elastic-recheck/#1392860 | 14:54 |
| yfried_ | dkranz: Change-Id: Ia86f48772ac02d67faa1c7d764cb9dc0938f6452 by andreaf did something with regexes and scenario.utils but I'm not sure what's being done there | 14:54 |
| mtreinish | sdague: it sends a get console request with a body of {"os-getConsoleOutput": {"length": null}} and gets a response body of {"output": ""} | 14:55 |
| mtreinish | that behavior seems off somehow | 14:55 |
| mtreinish | mriedem, dims: ^^^ weren't you playing with this stuff at some point? | 14:55 |
| mriedem | mtreinish: we added that when we hit timeouts or something | 14:56 |
| dims | mriedem: mtreinish: vaguely recall :) | 14:56 |
| dkranz | sdague: looks real to me | 14:58 |
| dkranz | sdague: like the server is treating the 'null' length as if it were zero | 14:58 |
| sdague | ok, but is that actually considered wrong? | 14:58 |
| dkranz | sdague: according to the schema it is wrong | 14:59 |
| sdague | whose schema | 14:59 |
| dkranz | sdague: but that could be a bug in the schema | 14:59 |
| *** Longgeek_ has quit IRC | 14:59 | |
| dkranz | 'json-schema': {'additionalProperties': False, 'type': 'object', 'properties': {'os-getConsoleOutput': {'type': 'object', 'properties': {'length': {'minimum': 0, 'type': ['integer', 'string']}}}}}, 'resources': [{'expected_result': 404, 'name': 'server'}], 'http-method': 'POST'} | 15:00 |
| dkranz | sdague: ^^^ | 15:00 |
| sdague | right, whose schema is that | 15:00 |
| dkranz | sdague: that is in tempest | 15:01 |
| dkranz | sdague: whether it is right or not, it changed in the past few days | 15:01 |
| sdague | because in nova, this is totally allowed | 15:01 |
| dkranz | sdague: I believe it could be, but this test was presumably returning 404 until very recently | 15:01 |
| sdague | so there aren't any recent changes in nova that I can see here | 15:02 |
| sdague | this code path goes back to 2012 | 15:02 |
| mtreinish | well, shouldn't it be failing on every run if the tempest test is incorrect? | 15:03 |
| mtreinish | why does it only happen sometimes? | 15:03 |
| sdague | it's auto generated, so that's the question | 15:03 |
| yfried_ | dkranz: so server_basic_ops seems to be loading (image, flavor, user) dynamically from [input_scenario] section and use CONF.compute.(image_ref, flavor_ref, ssh_user) as defaults | 15:03 |
| dkranz | sdague: the autogen is deterministic though | 15:03 |
| yfried_ | dkranz: seems like it would be reasonable to use the same in all scenarios? | 15:04 |
| yfried_ | mkoderer: ping | 15:05 |
| sdague | yeh, well is someone looking into the failure, or are we going to delete the tests. Because the test seems provably wrong based on my reading of the nova code. | 15:06 |
| dkranz | sdague: I'm going to look at the logs a bit more now | 15:08 |
| sdague | dkranz: ok | 15:08 |
| yfried_ | mtreinish: https://blueprints.launchpad.net/tempest/+spec/input-scenarios-for-scenario this did a POC with server_basic_ops. was this ever continued? | 15:09 |
| *** marun has joined #openstack-qa | 15:09 | |
| *** zz_dimtruck is now known as dimtruck | 15:09 | |
| *** enikanorov_ has joined #openstack-qa | 15:10 | |
| yfried_ | mtreinish: will we be able to discuss large_ops test at this week's mtg, or was it discussed in summit? | 15:12 |
| mtreinish | yfried_: IIRC it's just the one test which uses it. No one has seen sufficient reason to expand it since the one test landed | 15:13 |
| *** rbak has joined #openstack-qa | 15:13 | |
| mtreinish | yfried_: you can put it on the agenda, but really you should just talk to flashgordon about it. | 15:14 |
| mtreinish | in there current form the large ops tests will eventually just be converted to a nova functional test, because that's what they are | 15:14 |
| yfried_ | mtreinish: about large_ops - will do both | 15:15 |
| yfried_ | flashgordon: ping | 15:15 |
| sdague | 44dce62e23a56a0302efc16ecdfeac700bfefa84 merged about the time the failing started... for what it's worth | 15:15 |
| yfried_ | mtreinish: about the BP - well, it was requested in icehouse summit and added new modules and conf values. I wonder if it's being used anywhere. | 15:15 |
| yfried_ | mtreinish: I'm trying to understand why we have so many ssh_user values | 15:15 |
| yfried_ | mtreinish: so if it's not being used - can we remove/revert it until someone actually requests it? or maybe somehow extract it into tempest-lib? | 15:16 |
| *** Yogi has quit IRC | 15:17 | |
| yfried_ | mtreinish: on the other hand, is there a documentation of this dynamic scenario thing? maybe it's a good thing we can start using, but it's just not public knowledge? | 15:20 |
| mtreinish | yfried_: it is being used, by that one test. If you specify multiple images to run that test with you may need to provide multiple ssh users which is why the option is there | 15:21 |
| mtreinish | it's just not used in the gate because we just run it with the cirros image | 15:21 |
| mtreinish | yfried_: there probably aren't any external docs for it in the tempest tree, just inline comments | 15:22 |
| *** jecarey_ has joined #openstack-qa | 15:22 | |
| yfried_ | mtreinish: so it's for operator who want to run server_basic_ops multiple times against different images? | 15:23 |
| mtreinish | yfried_: yes, IIRC that is how it works | 15:23 |
| *** dgonzalez has joined #openstack-qa | 15:24 | |
| *** jlibosva has joined #openstack-qa | 15:24 | |
| yfried_ | mtreinish: so now I wonder if we could make all the scenarios use it, so you have a single place in conf where you define your scenario image(s?) | 15:24 |
| *** weshay has quit IRC | 15:24 | |
| *** sabeen1 has joined #openstack-qa | 15:24 | |
| mtreinish | sdague: hmm weird, maybe the test was just broken before and that's why it was passing. (I could see recreating creds unecessarily causing the expected error condition) | 15:24 |
| yfried_ | mtreinish: as dkranz said, it makes sense to have a different image for scenario | 15:25 |
| dkranz | mtreinish: But that still does not explain why it is flaky now | 15:26 |
| *** jlibosva has quit IRC | 15:26 | |
| *** jlibosva has joined #openstack-qa | 15:27 | |
| jlibosva | sdague: hi | 15:27 |
| *** superdan is now known as dansmith | 15:28 | |
| *** dgonzalez has quit IRC | 15:29 | |
| *** dgonzalez has joined #openstack-qa | 15:29 | |
| *** topshare has joined #openstack-qa | 15:30 | |
| *** atiwari has joined #openstack-qa | 15:30 | |
| *** pcrews has joined #openstack-qa | 15:33 | |
| sdague | dkranz: because it's creating a test server and not waiting for it | 15:34 |
| dkranz | sdague: ok, so that is the bug | 15:34 |
| sdague | it's at least one of them | 15:34 |
| *** jecarey__ has joined #openstack-qa | 15:35 | |
| sdague | anyway, I think we need to delete the auto generated tempest tests anyway, if you can't read the test and figure out what it's going to do, it makes this sort of thing a mess to go debug. Ask mtreinish is not scalable. | 15:35 |
| *** ramishra_ has joined #openstack-qa | 15:36 | |
| *** ramishra_ has quit IRC | 15:36 | |
| mtreinish | sdague: heh, I'm probably the wrong person to ask about it anyway? :) | 15:37 |
| dkranz | sdague: I don't see how that is the issue here. The bug is right in the test code | 15:37 |
| dkranz | sdague: to me the bug is more that waiting for a server to come up in not the default | 15:38 |
| dkranz | sdague: for create_test_server | 15:38 |
| dkranz | sdague: I presume you are not advocating that we create a set of manual negative tests | 15:39 |
| dkranz | sdague: we discussed moving the negative test stuff to tempest-lib | 15:39 |
| *** markmcclain has quit IRC | 15:40 | |
| dkranz | mtreinish: Do you want to remove these tests, or should I push a fix for this bug? | 15:40 |
| *** markmcclain has joined #openstack-qa | 15:40 | |
| sdague | dkranz: my issue is that bug has been in there for a while, it was blocking other people's stuff, and until I spent a second day chasing people on it it didn't get looked at. | 15:41 |
| sdague | dkranz: so that change should actually just mean the test fails 100% of the time, because the tempest schema is wrong | 15:42 |
| sdague | it only happened to pass because server boots were slow enough that the server didn't get created most of the time | 15:42 |
| dkranz | sdague: what is the right schema? This is why we said we should move neg testing to projects | 15:43 |
| sdague | dkranz: don't know, my interest in this test's usefulness ran out an hour ago. Which is why I said we should delete it :) | 15:45 |
| dkranz | sdague: I'm not objecting to that | 15:45 |
| openstackgerrit | Sean Dague proposed openstack/tempest: delete console negative tests https://review.openstack.org/134976 | 15:47 |
| sdague | ok, that's my proposed solution :) | 15:48 |
| *** achanda has joined #openstack-qa | 15:49 | |
| yfried_ | mtreinish: dkranz: how come "tox" isn't part of tempest requirments.txt? | 15:49 |
| jlibosva | sdague: hi, I made a patch for neutron grenade to become voting https://review.openstack.org/#/c/134532/ as you asked. Can you please have a look? | 15:49 |
| sdague | jlibosva: is it passing on stable/juno? | 15:50 |
| jlibosva | sdague: I think not | 15:50 |
| sdague | ok, so you need to change the exclude for that | 15:51 |
| mtreinish | yfried_: because it's not required | 15:51 |
| sdague | jlibosva: left a comment where it should go | 15:51 |
| jlibosva | sdague: thanks | 15:52 |
| *** dgonzale_ has joined #openstack-qa | 15:52 | |
| mtreinish | yfried_: you can run tempest directly with a test runner, there's also run_tempest.sh. Tox is just what we use for the gate and has some convenient shortcuts for different job types | 15:53 |
| dkranz | mtreinish: you ok with just deleting the negative console test? | 15:54 |
| jlibosva | sdague: updated, I hope that's what is needed | 15:54 |
| mtreinish | dkranz: yes, I think that's fine, we can always add it back if it makes sense | 15:55 |
| mtreinish | dkranz: I already left a comment on sdague's review | 15:55 |
| *** dgonzalez has quit IRC | 15:56 | |
| *** rwsu has joined #openstack-qa | 15:56 | |
| dkranz | mtreinish: ok. | 15:56 |
| yfried_ | mtreinish: tnx | 15:56 |
| *** beekneemech is now known as bnemec | 15:56 | |
| *** dgonzale_ has quit IRC | 15:57 | |
| *** zz_jgrimm is now known as jgrimm | 15:59 | |
| openstackgerrit | Yair Fried proposed openstack/tempest: Makes all scenarios use image details from [input-scenario] https://review.openstack.org/134980 | 15:59 |
| yfried_ | dkranz: mtreinish: please take a look https://review.openstack.org/134980 | 16:00 |
| *** edwa6643 has joined #openstack-qa | 16:00 | |
| sdague | mtreinish: testresources getting used anywhere yet in tempest? | 16:00 |
| *** toure has quit IRC | 16:01 | |
| mtreinish | sdague: no, not yet | 16:01 |
| *** mestery is now known as mestery_afk | 16:01 | |
| sdague | ok, bummer, I was looking for examples to make it make sense in my brain | 16:01 |
| mtreinish | sdague: heh, I think that's part of the reason it hasn't been used anywhere in tempest yet :) | 16:02 |
| *** marun has quit IRC | 16:02 | |
| *** toure has joined #openstack-qa | 16:03 | |
| *** dkusidlo has joined #openstack-qa | 16:03 | |
| sdague | yeh... bummer | 16:04 |
| *** yfried_ has quit IRC | 16:04 | |
| *** dkranz has quit IRC | 16:05 | |
| *** jecarey_ has quit IRC | 16:07 | |
| *** jecarey__ is now known as jecarey | 16:07 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: Switch off Nova XML v2 Tests https://review.openstack.org/134985 | 16:08 |
| *** achanda has quit IRC | 16:09 | |
| *** david-lyle_afk is now known as david-lyle | 16:11 | |
| *** jlibosva has quit IRC | 16:12 | |
| cdent | sdague: I was trying to use testresources in my http test experiment and eventually stalled out in part because it was melting my brain and also because I ran out of time for that task | 16:13 |
| cdent | I was trying to create aribtrary dependencies (ordering of tests while preserving isolation) | 16:14 |
| openstackgerrit | Joseph Lanoux proposed openstack/tempest: Add BlockDeviceMappingV2Boot test https://review.openstack.org/134996 | 16:17 |
| *** SridharG has left #openstack-qa | 16:18 | |
| mtreinish | jlanoux: ^^^ pdb? :) | 16:18 |
| jlanoux | mtreinish: sorry - I'm currently correcting it :( | 16:19 |
| mtreinish | jlanoux: heh, no worries | 16:19 |
| *** marun has joined #openstack-qa | 16:19 | |
| *** dgonzalez has joined #openstack-qa | 16:20 | |
| *** dkranz has joined #openstack-qa | 16:20 | |
| *** mpaolino has quit IRC | 16:20 | |
| openstackgerrit | Joseph Lanoux proposed openstack/tempest: Add BlockDeviceMappingV2Boot test https://review.openstack.org/134996 | 16:21 |
| dims | mtreinish: logged a bug for the extra tests that fail even with the nova v2 xml flag switched off - https://bugs.launchpad.net/tempest/+bug/1393475 | 16:22 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/131132 failed because of: gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1374175 https://bugs.launchpad.net/bugs/1297560 | 16:23 |
| mtreinish | dims: ok thanks, you're patch to fix it was the right approach we just have to be more selective in the skips | 16:23 |
| *** hemna has joined #openstack-qa | 16:25 | |
| *** DaSchab has quit IRC | 16:27 | |
| *** DaSchab has joined #openstack-qa | 16:27 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: Switch off Nova XML v2 Tests https://review.openstack.org/134985 | 16:28 |
| *** hemna has quit IRC | 16:28 | |
| *** DaSchab has quit IRC | 16:28 | |
| mtreinish | dims: ^^^ heh, I meant just rebase it on top of that commit :) | 16:29 |
| dims | mtreinish: :) | 16:30 |
| *** rfolco has joined #openstack-qa | 16:32 | |
| *** mpaolino has joined #openstack-qa | 16:34 | |
| *** chandankumar has joined #openstack-qa | 16:34 | |
| *** rfolco has quit IRC | 16:36 | |
| *** sridhar has quit IRC | 16:37 | |
| *** Yogi has joined #openstack-qa | 16:37 | |
| *** dkusidlo has quit IRC | 16:38 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: Switch off Nova XML v2 Tests https://review.openstack.org/134985 | 16:38 |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Skip additional tests when xml_api_v2 is set to False https://review.openstack.org/134924 | 16:39 |
| mtreinish | dims: ^^^ that should do it per test | 16:40 |
| mtreinish | if you wanted to test it to make sure | 16:40 |
| *** dkusidlo has joined #openstack-qa | 16:40 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: Skip additional tests when xml_api_v2 is set to False https://review.openstack.org/134924 | 16:40 |
| mtreinish | dims: ^^^ :( | 16:41 |
| dims | mtreinish: just fixing commit messages and rebasing them correctly | 16:42 |
| dims | mtreinish: y, will add setup methods with skips, is that ok? | 16:42 |
| dims | mtreinish: for those specific ones i listed in the bug report | 16:42 |
| mtreinish | dims: look at rev 2 on 134924 | 16:42 |
| dims | sorry, stepped on your commit | 16:44 |
| mtreinish | no worries :) I think that approach should only skip where needed | 16:44 |
| mtreinish | I think you could do it with a skipIf decorator too, but it gets kind of interesting to pull the interface class variable in the decorator | 16:46 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: Skip additional tests when xml_api_v2 is set to False https://review.openstack.org/134924 | 16:46 |
| *** hemna has joined #openstack-qa | 16:47 | |
| *** mlavalle has joined #openstack-qa | 16:48 | |
| *** hemna has quit IRC | 16:49 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: Skip additional tests when xml_api_v2 is set to False https://review.openstack.org/134924 | 16:49 |
| openstackrecheck | openstack/sahara change: https://review.openstack.org/134336 failed because of: gate-sahara-pep8: unrecognized error, gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1374175 https://bugs.launchpad.net/bugs/1297560 | 16:50 |
| dims | mtreinish: i added a skipUnless in Nova, to avoid adding setUp methods in multiple classes | 16:50 |
| dims | mtreinish- https://review.openstack.org/#/c/134332/12/nova/test.py,unified | 16:50 |
| mtreinish | dims: there is a skip unless decorator in testtools already | 16:51 |
| mtreinish | https://github.com/testing-cabal/testtools/blob/master/testtools/testcase.py#L897 | 16:52 |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: Enable nova multi host on aiopcpu tests https://review.openstack.org/131604 | 16:53 |
| *** hemna has joined #openstack-qa | 16:54 | |
| *** dkranz has quit IRC | 16:54 | |
| *** beagles is now known as beagles_brb | 16:58 | |
| *** toure has quit IRC | 17:02 | |
| *** toure has joined #openstack-qa | 17:03 | |
| *** chandankumar has quit IRC | 17:03 | |
| *** jlanoux has quit IRC | 17:03 | |
| *** rwsu has quit IRC | 17:03 | |
| *** dkranz has joined #openstack-qa | 17:06 | |
| *** tkammer has quit IRC | 17:13 | |
| *** rwsu has joined #openstack-qa | 17:16 | |
| *** rwsu has quit IRC | 17:18 | |
| *** rwsu has joined #openstack-qa | 17:19 | |
| *** dgonzalez has quit IRC | 17:20 | |
| *** dgonzalez has joined #openstack-qa | 17:20 | |
| *** flashgordon is now known as jogo | 17:22 | |
| *** atiwari has quit IRC | 17:23 | |
| jogo | mtreinish: I saw something in backscroll about large-ops | 17:23 |
| *** atiwari has joined #openstack-qa | 17:23 | |
| *** dgonzalez has quit IRC | 17:24 | |
| *** dkusidlo has quit IRC | 17:24 | |
| *** jpich has quit IRC | 17:24 | |
| dims | mtreinish: that does not work when specified at the class level | 17:26 |
| *** DaSchab has joined #openstack-qa | 17:29 | |
| *** avico has quit IRC | 17:31 | |
| *** atiwari has quit IRC | 17:31 | |
| *** atiwari has joined #openstack-qa | 17:32 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/132950 failed gate-nova-python27 in the gate queue with an unrecognized error | 17:33 |
| *** wuhg has quit IRC | 17:33 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: Switch off Nova XML v2 Tests https://review.openstack.org/134985 | 17:34 |
| *** DaSchab has quit IRC | 17:34 | |
| *** weshay has joined #openstack-qa | 17:34 | |
| *** chandankumar has joined #openstack-qa | 17:38 | |
| mtreinish | jogo: yeah, yfried was asking about it he wanted to make changes | 17:41 |
| *** harlowja_away is now known as harlowja | 17:41 | |
| mtreinish | I think he had an etherpad, let me try to dig it up | 17:41 |
| *** kmartin has joined #openstack-qa | 17:42 | |
| mtreinish | jogo: https://etherpad.openstack.org/p/tempest_large_ops_with_network | 17:42 |
| *** yamahata has quit IRC | 17:53 | |
| openstackgerrit | Ashish Kumar Gupta proposed openstack/tempest: Floating IP Negative Tests https://review.openstack.org/109529 | 17:54 |
| *** jordanP has quit IRC | 17:54 | |
| *** asselin has joined #openstack-qa | 17:55 | |
| *** Yogi has quit IRC | 17:56 | |
| *** suneelb has joined #openstack-qa | 17:57 | |
| *** beagles_brb is now known as beagles | 17:58 | |
| jogo | mtreinish: thanks, commenting on the etherpad | 17:58 |
| dims | jogo: update on the xml stuff - http://markmail.org/message/txgsqmx6kpjiltyx | 17:59 |
| *** yamahata has joined #openstack-qa | 18:00 | |
| openstackgerrit | Adam Gandelman proposed openstack-dev/devstack: Do not hard-code ironic svc port, protocol https://review.openstack.org/135041 | 18:01 |
| jogo | dims: thanks, will look at it later today | 18:04 |
| dims | jogo: no hurries, just reporting progress :) | 18:04 |
| jogo | dims: thanks for spearheading this! | 18:05 |
| *** markus_z has quit IRC | 18:10 | |
| *** topshare has quit IRC | 18:10 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/128191 failed because of: gate-nova-python27: unrecognized error, gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1374175 https://bugs.launchpad.net/bugs/1297560 | 18:12 |
| *** bkopilov has quit IRC | 18:13 | |
| *** topshare has joined #openstack-qa | 18:17 | |
| *** mpaolino has quit IRC | 18:18 | |
| *** yfried_ has joined #openstack-qa | 18:19 | |
| yfried_ | dkranz: ping | 18:20 |
| jogo | yfried_: I commented on the etherpad about large ops | 18:22 |
| openstackgerrit | Joe Gordon proposed openstack-infra/devstack-gate: Setup ssh_known_host files for aiopcpu https://review.openstack.org/134670 | 18:22 |
| openstackgerrit | Joe Gordon proposed openstack-infra/devstack-gate: DO NOT MERGE: testing aiopcpu with tempest full https://review.openstack.org/134639 | 18:22 |
| openstackgerrit | Joe Gordon proposed openstack-infra/devstack-gate: Turn off nova_allow_move_to_same_host if in aiopcpu https://review.openstack.org/135044 | 18:22 |
| *** achanda has joined #openstack-qa | 18:25 | |
| *** chandankumar has quit IRC | 18:25 | |
| *** sabeen1 has quit IRC | 18:26 | |
| *** topshare has quit IRC | 18:27 | |
| dkranz | yfried_: pong | 18:28 |
| *** patrickeast has joined #openstack-qa | 18:28 | |
| *** achanda has quit IRC | 18:30 | |
| yfried_ | dkranz: https://review.openstack.org/134980 were you able to take a look? | 18:32 |
| dkranz | yfried_: not yet | 18:33 |
| *** e0ne has quit IRC | 18:33 | |
| *** achanda has joined #openstack-qa | 18:35 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 18:38 | |
| yfried_ | jogo: tnx. posted some questions on your comments | 18:38 |
| yfried_ | jogo: https://etherpad.openstack.org/p/tempest_large_ops_with_network | 18:39 |
| *** rushiagr is now known as rushiagr_away | 18:40 | |
| *** yamahata has quit IRC | 18:41 | |
| yfried_ | dkranz: mtreinish: mkoderer: cyeoh: can I get some cores' eyes on these 2 patches please? https://review.openstack.org/#/c/124722/ https://review.openstack.org/#/c/124625/ | 18:41 |
| openstackgerrit | Dean Troyer proposed openstack-dev/devstack: Simplify docs build https://review.openstack.org/135048 | 18:47 |
| openstackrecheck | openstack/trove change: https://review.openstack.org/112035 failed because of: gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1374175 https://bugs.launchpad.net/bugs/1297560 | 18:48 |
| *** mestery_afk is now known as mestery | 18:48 | |
| *** yjiang5_away is now known as yjiang5 | 18:50 | |
| dkranz | yfried_: See my comment in https://review.openstack.org/#/c/124722/. Do I misunderstand the meaning of __class__? | 18:51 |
| jogo | yfried_: responded | 18:54 |
| *** weshay has quit IRC | 18:55 | |
| *** luqas has quit IRC | 19:03 | |
| openstackgerrit | Sean Dague proposed openstack/tempest: Delete console negative tests https://review.openstack.org/134976 | 19:03 |
| *** vigneshvar_ has joined #openstack-qa | 19:06 | |
| *** ajo has quit IRC | 19:07 | |
| mtreinish | sdague, dkranz: can you guys take a look at: https://review.openstack.org/121910 it should block regressions on not being able to do a list-tests w/o a config file | 19:10 |
| *** vigneshvar has quit IRC | 19:10 | |
| *** Rockyg has joined #openstack-qa | 19:13 | |
| *** bknudson has quit IRC | 19:15 | |
| *** bknudson has joined #openstack-qa | 19:15 | |
| *** wojtczak has joined #openstack-qa | 19:16 | |
| *** DaSchab has joined #openstack-qa | 19:18 | |
| *** weshay has joined #openstack-qa | 19:18 | |
| mtreinish | dims: the xenserver failure on that patch was real: http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/24/134924/5/31874/run_tests.log | 19:22 |
| *** wojtczak has left #openstack-qa | 19:22 | |
| mtreinish | CONF isn't definied in one of the files | 19:22 |
| *** DaSchab has quit IRC | 19:22 | |
| openstackrecheck | openstack/trove change: https://review.openstack.org/134802 failed because of: gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1374175 https://bugs.launchpad.net/bugs/1297560 | 19:23 |
| openstackgerrit | Adolfo Duarte proposed openstack/tempest: Added 'distributed' attribute to router update https://review.openstack.org/132922 | 19:28 |
| dims | mtreinish: ack, fixing | 19:28 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: Skip additional tests when xml_api_v2 is set to False https://review.openstack.org/134924 | 19:33 |
| sdague | dtroyer: hmmmm.... we lost the ability to run devstack unit tests after the bashate conversion? | 19:33 |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Sync latest log module from oslo-incubator https://review.openstack.org/131601 | 19:36 |
| *** yfried_ has quit IRC | 19:39 | |
| *** dustins_ has joined #openstack-qa | 19:41 | |
| *** sabeen has joined #openstack-qa | 19:42 | |
| sdague | mtreinish: why syncing versus switching to oslo.log? | 19:43 |
| mtreinish | sdague: I didn't think oslo.log was ready yet | 19:44 |
| sdague | ok | 19:44 |
| *** dustins has quit IRC | 19:44 | |
| mtreinish | sdague: also dims was helping me with that patch, because there was an issue with oslo.i8ln being an implicit requirement | 19:44 |
| sdague | ok | 19:45 |
| *** atiwari has quit IRC | 19:49 | |
| *** e0ne has joined #openstack-qa | 19:55 | |
| *** weshay has quit IRC | 19:55 | |
| openstackgerrit | Adolfo Duarte proposed openstack/tempest: Added 'distributed' attribute to router update https://review.openstack.org/132922 | 20:01 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: Switch off Nova XML v2 Tests https://review.openstack.org/134985 | 20:03 |
| *** sabeen2 has joined #openstack-qa | 20:03 | |
| *** sabeen has quit IRC | 20:04 | |
| *** omnibus7 is now known as davidlenwell | 20:06 | |
| *** sabeen1 has joined #openstack-qa | 20:06 | |
| *** ajo has joined #openstack-qa | 20:08 | |
| *** sabeen2 has quit IRC | 20:08 | |
| *** Sukhdev has joined #openstack-qa | 20:12 | |
| *** yjiang5 is now known as yjiang5_away | 20:12 | |
| dtroyer | sdague: the tests/* scripts? | 20:12 |
| sdague | yeh, except, we don't seem to be running them | 20:12 |
| *** e0ne has quit IRC | 20:13 | |
| *** mpaolino has joined #openstack-qa | 20:13 | |
| *** sabeen1 has quit IRC | 20:13 | |
| *** sabeen1 has joined #openstack-qa | 20:14 | |
| dtroyer | I don't recall them being in run_stest.sh before | 20:15 |
| sdague | so, you are probably right | 20:15 |
| dtroyer | they should be though, maybe tme to add them | 20:16 |
| sdague | however, the subtlety, and the screw ups I made with the LIBS_FROM_GIT means that I really want an infrastructure for running unit tests here | 20:16 |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: make run_tests.sh do something again https://review.openstack.org/135065 | 20:16 |
| sdague | so.... ^^^ | 20:16 |
| dtroyer | yeah, that's the ticket | 20:17 |
| sdague | so I'm ok adhocing more testing here, though this looked kind of interesting as well - https://github.com/sstephenson/bats | 20:19 |
| sdague | just to make it a bit more structured | 20:20 |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/96800 failed gate-tempest-dsvm-full, gate-tempest-dsvm-postgres-full, gate-tempest-dsvm-neutron-heat-slow, gate-grenade-dsvm in the gate queue with an unrecognized error | 20:25 |
| openstackgerrit | Adam Gandelman proposed openstack-dev/devstack: Log early boot of Ironic VMs to serial with sgabios https://review.openstack.org/135068 | 20:30 |
| openstackgerrit | Adam Gandelman proposed openstack-dev/devstack: Log early boot of Ironic VMs to serial with sgabios https://review.openstack.org/135068 | 20:32 |
| openstackgerrit | Joe Gordon proposed openstack-infra/devstack-gate: DO NOT MERGE: testing aiopcpu with tempest full https://review.openstack.org/134639 | 20:36 |
| openstackgerrit | Joe Gordon proposed openstack-infra/devstack-gate: Set HOST_IP to the private interface https://review.openstack.org/135070 | 20:36 |
| *** atiwari has joined #openstack-qa | 20:40 | |
| openstackgerrit | Rob Crittenden proposed openstack-dev/devstack: Remove hardcoded protocol from backup_swift_url https://review.openstack.org/135073 | 20:41 |
| openstackgerrit | Adam Gandelman proposed openstack-dev/devstack: Do not hard-code ironic svc port, protocol https://review.openstack.org/135041 | 20:44 |
| *** mriedem has quit IRC | 20:47 | |
| *** openstackgerrit has quit IRC | 20:49 | |
| *** openstackgerrit has joined #openstack-qa | 20:49 | |
| *** mlavalle has quit IRC | 20:49 | |
| *** mlavalle has joined #openstack-qa | 20:51 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/109131 failed gate-nova-python27, gate-tempest-dsvm-full, gate-tempest-dsvm-postgres-full, gate-tempest-dsvm-neutron-full, gate-tempest-dsvm-neutron-heat-slow, gate-grenade-dsvm, gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error | 20:52 |
| *** mkollaro has quit IRC | 20:52 | |
| *** mlavalle has quit IRC | 20:53 | |
| *** achanda has quit IRC | 20:54 | |
| *** sabeen1 has quit IRC | 20:56 | |
| *** sabeen2 has joined #openstack-qa | 20:56 | |
| openstackgerrit | Rob Crittenden proposed openstack/tempest: Fix invalid URL in AccountClientCustomizedHeader https://review.openstack.org/135078 | 20:57 |
| openstackgerrit | Sean Dague proposed openstack/tempest: skip test_server_cfn_init as the failure rate is too high https://review.openstack.org/135079 | 20:58 |
| sdague | mtreinish: ^^ | 20:58 |
| sdague | that's just thrashing the gate right now | 20:58 |
| mtreinish | sdague: sure, makes sense | 21:00 |
| mtreinish | although that was the only running test in the slow job | 21:00 |
| mtreinish | so now it's an empty job definition | 21:00 |
| sdague | hmmm. | 21:00 |
| sdague | also.... it seems that such a thing already went in? | 21:00 |
| sdague | https://jenkins05.openstack.org/job/gate-tempest-dsvm-neutron-heat-slow/6750/console | 21:01 |
| mtreinish | sdague: hmm, i think that's just an import failure because the patch there doesn't touch the heat tests | 21:01 |
| sdague | mtreinish: what triggered the import failure? | 21:02 |
| *** mkollaro has joined #openstack-qa | 21:03 | |
| *** vigneshvar_ has quit IRC | 21:03 | |
| mtreinish | dunno, but if the list tests explodes with subunit-trace I think that is the output, but I'm not 100% (it's been a couple of weeks) | 21:03 |
| mtreinish | it could be in any file, because it does a full list first | 21:04 |
| sdague | we don't display anything else ? | 21:05 |
| mtreinish | I'm not sure, I though it should print the list, I'll dig into it in a sec | 21:05 |
| mtreinish | I'm going to transition home first | 21:06 |
| sdague | mtreinish: one sec, why is test_server_cfn_init running in the main tests? | 21:07 |
| *** DaSchab has joined #openstack-qa | 21:07 | |
| *** DaSchab has quit IRC | 21:11 | |
| openstackrecheck | openstack/swift change: https://review.openstack.org/135008 failed because of: gate-grenade-dsvm: unrecognized error, gate-tempest-dsvm-full: unrecognized error, gate-tempest-dsvm-neutron-heat-slow: unrecognized error, gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1297560 | 21:24 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/113811 failed gate-tempest-dsvm-full, gate-tempest-dsvm-postgres-full, gate-tempest-dsvm-neutron-heat-slow, gate-grenade-dsvm, gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error | 21:24 |
| anteaya | and thoughts on where I would look in the logs to find out more details about the heat slow failures: http://logs.openstack.org/08/135008/2/gate/gate-tempest-dsvm-neutron-heat-slow/8b9eb1e/logs/ | 21:25 |
| anteaya | they aren't in tempest, testr, any of the devstack files or anywhere else I have looked | 21:25 |
| *** achanda has joined #openstack-qa | 21:28 | |
| lifeless | anteaya: it failed without capturing any tests | 21:28 |
| lifeless | possibly dying hard on the first test or test setup, which subunit v1 can't show because it can't multiplex | 21:29 |
| stevebaker | anteaya: could it be that no tests are running in the heat-slow job, and they are all running in the main tests job? | 21:32 |
| *** ajo has quit IRC | 21:34 | |
| *** ajo has joined #openstack-qa | 21:34 | |
| stevebaker | anteaya: ignore me | 21:35 |
| openstackgerrit | Adolfo Duarte proposed openstack/tempest: Adding api tests for dvr routers https://review.openstack.org/133317 | 21:35 |
| *** Rockyg has quit IRC | 21:36 | |
| *** eezhova has quit IRC | 21:37 | |
| *** eezhova has joined #openstack-qa | 21:37 | |
| *** aysyd has quit IRC | 21:43 | |
| *** cdent has quit IRC | 21:44 | |
| sdague | lifeless: ... is it possible in the testtools releases that how regexes worked with testr changed? | 21:48 |
| sdague | stevebaker: yeh, it looks like tempest tests aren't running in the jobs I'd expect them to | 21:50 |
| mtreinish | sdague: do you have a link? | 21:50 |
| sdague | http://logs.openstack.org/65/135065/1/check/check-tempest-dsvm-full/0f5df25/console.html | 21:51 |
| sdague | test_server_cfn_init should be a slow test | 21:51 |
| sdague | so not run there | 21:51 |
| sdague | and equivalently, the heat jobs are failing because they are empty | 21:51 |
| *** safchain has quit IRC | 21:52 | |
| *** salv-orlando has quit IRC | 21:52 | |
| stevebaker | sdague: ah, there it is http://logs.openstack.org/65/135065/1/check/check-tempest-dsvm-full/0f5df25/console.html#_2014-11-17_21_18_30_124 | 21:53 |
| openstackgerrit | Rob Crittenden proposed openstack-dev/devstack: Configure boto.cfg in tempest to point to SSL bundle https://review.openstack.org/135091 | 21:53 |
| mtreinish | sdague: yeah, the other tests tagged as slow are getting run there too | 21:54 |
| sdague | mtreinish: right, so there is a system level failure here | 21:54 |
| mtreinish | sdague: yeah, it's the new testtools, I just did a local test | 21:54 |
| sdague | mtreinish: ok, 1.4.0 breaks the regex? | 21:54 |
| mtreinish | http://paste.openstack.org/show/134187/ | 21:54 |
| sdague | does 1.3.0 as well? | 21:55 |
| mtreinish | dunno, I'll check | 21:55 |
| sdague | lifeless: ^^^ | 21:55 |
| lifeless | sdague: no | 21:55 |
| lifeless | sdague: it shouldn't be possible | 21:55 |
| lifeless | sdague: but I guess perhaps it is. I need to take cynthia swimming now, will dive in as soon as I return | 21:56 |
| mtreinish | nope: http://paste.openstack.org/show/134188/ | 21:56 |
| mtreinish | sdague: so it's just 1.4.0 | 21:56 |
| sdague | mtreinish: ok, want to propose the requirements exclusion? | 21:56 |
| mtreinish | sdague: sure | 21:57 |
| mtreinish | heh, but won't that break trove again? | 21:57 |
| *** ajo has quit IRC | 21:57 | |
| anteaya | lifeless: ah thanks | 21:57 |
| lifeless | sdague: mtreinish: please try to reproduce more narrowly, I've just tested regexes with 1.4.0 here and its fine. | 21:58 |
| lifeless | sdague: mtreinish: last time it was knee-jerk and premature, I'd had to see that happen again. I'll be back in ~90m | 21:58 |
| sdague | lifeless: it wasn't premature, 1.2.0 required things it didn't specify | 21:58 |
| sdague | blocking the problematic version lets people get back to work, and lets you have family time to figure it out when is convenient | 21:59 |
| *** toure has quit IRC | 22:00 | |
| sdague | lifeless: attr regex ? | 22:00 |
| mtreinish | sdague: https://review.openstack.org/#/c/135092/ | 22:01 |
| *** salv-orlando has joined #openstack-qa | 22:02 | |
| sdague | mtreinish: if you testr list-tests | 22:02 |
| sdague | are the tags still there? | 22:02 |
| openstackgerrit | Sean M. Collins proposed openstack-dev/devstack: Documentation: Using Neutron with DevStack https://review.openstack.org/131201 | 22:02 |
| mtreinish | sdague: yes, I think so, let me double check those paste links | 22:02 |
| mtreinish | sdague: yeah on 1.1.0, and 1.3.0 the attrs were still listed in the regexed list | 22:03 |
| mtreinish | oh let met try it with 1.4.0 without a regex filter and see if the attrs are there | 22:03 |
| sdague | right, but in 1.4.0 there are no attrs, right? | 22:03 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack-infra/devstack-gate: Add oslo.context to devstack-vm-gate-wrap.sh https://review.openstack.org/135093 | 22:04 |
| mtreinish | sdague: yep, that looks to be the issue | 22:04 |
| mtreinish | the attrs aren't being appended to the test_id. So the regex is working correctly because there is nothing matching | 22:04 |
| *** safchain has joined #openstack-qa | 22:04 | |
| sdague | mtreinish: can you propose all the backports as well? | 22:04 |
| sdague | mtreinish: yep | 22:05 |
| mtreinish | sdague: sure I'll push the backports one sec | 22:05 |
| openstackrecheck | Console logs not available after 13:27s for gate-tempest-dsvm-postgres-full 134555,4,a009eac | 22:07 |
| *** DaSchab has joined #openstack-qa | 22:07 | |
| *** ianw has quit IRC | 22:08 | |
| *** nmagnezi has quit IRC | 22:08 | |
| *** ianw has joined #openstack-qa | 22:08 | |
| openstackgerrit | Joe Gordon proposed openstack-infra/devstack-gate: Setup ssh_known_host files for aiopcpu https://review.openstack.org/134670 | 22:09 |
| openstackgerrit | Joe Gordon proposed openstack-infra/devstack-gate: Turn off nova_allow_move_to_same_host if in aiopcpu https://review.openstack.org/135044 | 22:09 |
| openstackgerrit | Joe Gordon proposed openstack-infra/devstack-gate: DO NOT MERGE: testing aiopcpu with tempest full https://review.openstack.org/134639 | 22:09 |
| openstackgerrit | Joe Gordon proposed openstack-infra/devstack-gate: Set HOST_IP to the private interface https://review.openstack.org/135070 | 22:09 |
| mtreinish | sdague: https://review.openstack.org/#/c/135096/ and https://review.openstack.org/#/c/135097/ | 22:09 |
| mtreinish | it looks like juno was missing the 1.2.0 exclude | 22:09 |
| *** Sukhdev has quit IRC | 22:12 | |
| *** DaSchab has quit IRC | 22:12 | |
| *** ajo has joined #openstack-qa | 22:15 | |
| *** ajo has quit IRC | 22:15 | |
| openstackgerrit | Jamie Lennox proposed openstack-dev/devstack: Use certmonger for certs https://review.openstack.org/135101 | 22:16 |
| sdague | mtreinish: yeh, it never managed to merge | 22:17 |
| sdague | because ... it takes too long to fix req issues | 22:18 |
| mtreinish | lifeless: it's probably around the fact that tempest tries to inline the decorators: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/test.py#n47 | 22:18 |
| *** sabeen2 has quit IRC | 22:18 | |
| mtreinish | although looking at the testcase.py code I'm not sure why replacing unittest with unittest2 would break that | 22:19 |
| mtreinish | but something looks weird to me in the tempest code, is that really the right way to inline a decorator? | 22:21 |
| *** edwa6643 has quit IRC | 22:22 | |
| openstackrecheck | Console logs not available after 14:37s for check-tempest-dsvm-neutron-heat-slow 135067,1,3403eee | 22:22 |
| *** mpaolino has quit IRC | 22:30 | |
| *** dkranz has quit IRC | 22:31 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/128578 failed gate-tempest-dsvm-full, gate-tempest-dsvm-postgres-full, gate-tempest-dsvm-neutron-full, gate-tempest-dsvm-neutron-heat-slow, gate-grenade-dsvm, gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error | 22:36 |
| *** dustins_ has quit IRC | 22:36 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: Add oslo.context https://review.openstack.org/135105 | 22:37 |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: Setup ssh_known_host files for aiopcpu https://review.openstack.org/134670 | 22:47 |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: DO NOT MERGE: testing aiopcpu with tempest full https://review.openstack.org/134639 | 22:47 |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: Use private addresses everywhere with aiopcpu https://review.openstack.org/135070 | 22:47 |
| openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/134972 failed gate-tempest-dsvm-neutron-heat-slow in the gate queue with an unrecognized error | 22:49 |
| *** salv-orlando has quit IRC | 22:58 | |
| *** mlavalle has joined #openstack-qa | 22:58 | |
| *** salv-orlando has joined #openstack-qa | 23:00 | |
| openstackrecheck | Console logs not available after 13:44s for check-tempest-dsvm-full 135059,1,49cd829 | 23:03 |
| *** marun has quit IRC | 23:04 | |
| mtreinish | sdague: I think I just found a fix for the testtools 1.4.0 thing in tempest | 23:04 |
| mtreinish | did we already merge the exclude? | 23:05 |
| mtreinish | clarkb: ^^^ ? | 23:05 |
| mtreinish | it looks like tempest has the mixin order backwards on the base test classes, I think | 23:05 |
| sdague | mtreinish: so what changed in testtools that made it care? | 23:06 |
| mtreinish | sdague: the only difference between 1.3.0 and 1.4.0 is that the testtools TestCase class inherits from unittest2 instead of unittest: | 23:07 |
| mtreinish | https://github.com/testing-cabal/testtools/commit/d30ee53f2eeb9ce4c4a04b7f6e94063edab72611#diff-1be3d2372bad38ec96fe0bf7b4e66485R28 | 23:07 |
| mtreinish | I'm not sure why that would make a difference though | 23:08 |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: Setup ssh_known_host files for aiopcpu https://review.openstack.org/134670 | 23:08 |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: DO NOT MERGE: testing aiopcpu with tempest full https://review.openstack.org/134639 | 23:08 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: Skip additional tests when xml_api_v2 is set to False https://review.openstack.org/134924 | 23:08 |
| mtreinish | because I thought unittest2 just did a pass through if python was newer than 2.6 | 23:08 |
| clarkb | yes I merged already | 23:08 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: Switch off Nova XML v2 Tests https://review.openstack.org/134985 | 23:08 |
| clarkb | but you can unwind at a leisurely pace now | 23:08 |
| openstackgerrit | Matthew Treinish proposed openstack/tempest-lib: Add a test mixin for users of the cli tests https://review.openstack.org/135108 | 23:10 |
| mtreinish | clarkb: ok, because it'll probably be a 2 stage thing to get the order switched over in tempest, which will require a tempest-lib release | 23:10 |
| *** dimtruck is now known as zz_dimtruck | 23:16 | |
| openstackrecheck | Console logs not available after 14:31s for check-tempest-dsvm-full 135074,2,de23c20 | 23:17 |
| mtreinish | sdague: oh, I'm think afazekas may have hit this before when he added the py2.6 hack in to add unittest2 to the BaseTestCase there. | 23:17 |
| mtreinish | although maybe not because the order was wrong even if the unittest2 usage was conditional | 23:18 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/131132 failed gate-nova-python27, gate-tempest-dsvm-full, gate-tempest-dsvm-postgres-full, gate-tempest-dsvm-neutron-heat-slow, gate-grenade-dsvm, gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error | 23:19 |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Fix inheritance for BaseTestCase https://review.openstack.org/135115 | 23:21 |
| mtreinish | sdague: ^^^ | 23:21 |
| mtreinish | lifeless: ^^^ I think that fixes the issue with testtools 1.4.0 | 23:21 |
| *** markmcclain has quit IRC | 23:22 | |
| openstackgerrit | Matthew Treinish proposed openstack/tempest-lib: Add a test mixin for users of the cli tests https://review.openstack.org/135108 | 23:25 |
| mtreinish | dims: sorry I suck on 134924, so many little copy and paste bugs... | 23:27 |
| *** dims_ has joined #openstack-qa | 23:28 | |
| *** rbak_ has joined #openstack-qa | 23:30 | |
| *** rbak_ has quit IRC | 23:32 | |
| dims_ | mtreinish: i do it *all* the time :) no sweat | 23:32 |
| *** dims has quit IRC | 23:32 | |
| *** rbak has quit IRC | 23:33 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: Setup ssh_known_host files for aiopcpu https://review.openstack.org/134670 | 23:36 |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: DO NOT MERGE: testing aiopcpu with tempest full https://review.openstack.org/134639 | 23:36 |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: Use private addresses everywhere with aiopcpu https://review.openstack.org/135070 | 23:36 |
| sdague | mtreinish: once you figure this out, would be good to figure out what the testtools test case or tempest unit test case would be to prevent future breakage | 23:36 |
| *** yjiang5_away is now known as yjiang5 | 23:36 | |
| openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/134975 failed gate-tempest-dsvm-full, gate-tempest-dsvm-postgres-full, gate-tempest-dsvm-neutron-heat-slow in the gate queue with an unrecognized error | 23:37 |
| lifeless | right, back | 23:37 |
| mtreinish | sdague: hmm, I can add an attr check to the list-tests unit test I've got a patch up to fix | 23:40 |
| mtreinish | I'm not sure it's something we can internally check in testtools, if the usage in tempest was wrong and just happened to work before | 23:41 |
| mtreinish | although it might be a unittest2 thing, I'm not sure I still need to dig further I guess | 23:41 |
| lifeless | mtreinish: that decorator glue seems to be dynamically generating attrs, which is fine if a little awkward | 23:43 |
| mtreinish | lifeless: yeah, I'm pretty sure it's because the mixin order was backwards in tempest | 23:43 |
| mtreinish | it works when I locally tested https://review.openstack.org/#/c/135115/ | 23:44 |
| mtreinish | but I'm not sure why it would have worked in testtools<1.4.0 if that was the cause | 23:44 |
| lifeless | sdague: when I say premature w.r.t. 1.2.0. I mean that there was no analysis of whether it was an upstream issue or not. It was just (AFAICT) fire and forget. | 23:44 |
| lifeless | mtreinish: possibly there is a bug in the unittest2 id() method. | 23:45 |
| lifeless | mtreinish: I will look into that. | 23:45 |
| lifeless | mtreinish: I know there are things in there I need to improve. | 23:45 |
| mtreinish | lifeless: oh ok, yeah it could be, I guess I should probably pull the unittest2 tree. I don't know why I thought if python>=2.7 unittest2 just imported unittest. | 23:49 |
| sdague | lifeless: I spent 2 hrs of my saturday getting to the understanding level I had to send that email | 23:49 |
| sdague | while you can say I didn't understand everything | 23:50 |
| sdague | I am quite annoyed by saying there was no analysis | 23:50 |
| sdague | for you, as the testtools author, to create the minimal test case is a few minutes of work, but that's not true for everyone else | 23:51 |
| lifeless | sdague: I didn't know that. All I knew was what you wrote | 23:51 |
| sdague | my email was a summary, to give people enough info to move forward | 23:51 |
| lifeless | sdague: I'm sorry that a chunk of your time was consumed. | 23:51 |
| sdague | lifeless: yeh, no worries. I just felt like your retort devalued the effort that went into having any path forward. I agree the dependency fix was the right eventual fix, but these things emerge in parts, and we have to accept that | 23:52 |
| sdague | which is why the blocks I think are a good tool here, like I said, it lets us split the problem and get people running again that are working on completely unrelated stuff | 23:54 |
| lifeless | sure | 23:54 |
| lifeless | I'm totally fine with blocks; what got me grumpy fundamentally was the last paragraph in your mail | 23:54 |
| lifeless | which intended or not I took as an attack | 23:55 |
| lifeless | in generall roll-back is a much safer strategy than roll-forward | 23:55 |
| *** melwitt has joined #openstack-qa | 23:55 | |
| openstackrecheck | Console logs not available after 13:57s for gate-tempest-dsvm-neutron-full 133557,1,2705af2 | 23:55 |
| lifeless | and the block is a roll-back strategy | 23:55 |
| *** DaSchab has joined #openstack-qa | 23:56 | |
| sdague | lifeless: yeh, sorry about that | 23:56 |
| *** bknudson has quit IRC | 23:56 | |
| sdague | I didn't mean as such, I'd actually said it in a few other "breaks the world emails" in the past | 23:57 |
| sdague | so it's part of my post amble thinking now | 23:57 |
| openstackrecheck | openstack/sahara change: https://review.openstack.org/134345 failed gate-sahara-pep8, gate-tempest-dsvm-full, gate-tempest-dsvm-postgres-full, gate-tempest-dsvm-neutron-heat-slow, gate-grenade-dsvm in the gate queue with an unrecognized error | 23:57 |
| *** achanda has quit IRC | 23:57 | |
| sdague | but I'll try to do better on that front | 23:57 |
| lifeless | sdague: FWIW one of the reasons I did the release on saturday is that it was more likely than other releases to have fallout, and I'd rather find and fix that during folks weekends than primme time | 23:58 |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: Setup ssh_known_host files for aiopcpu https://review.openstack.org/134670 | 23:58 |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: DO NOT MERGE: testing aiopcpu with tempest full https://review.openstack.org/134639 | 23:58 |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: Use private addresses everywhere with aiopcpu https://review.openstack.org/135070 | 23:58 |
| sdague | ah, see I'd rather you did it at a time when we had a bunch of folks around, so we could figure out what was going on together | 23:59 |
| sdague | because it kind of took a team effort to do that | 23:59 |
| sdague | as there are a lot of moving parts | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!