| *** itzikb has joined #openstack-qa | 00:00 | |
| *** itzikb has quit IRC | 00:01 | |
| openstackrecheck | Console logs not available after 13:26s for gate-tempest-dsvm-cells 279626,9,c48416c | 00:02 |
|---|---|---|
| *** yamamoto has joined #openstack-qa | 00:08 | |
| *** minwang2 has quit IRC | 00:10 | |
| *** yangyape_ has quit IRC | 00:10 | |
| *** ddieterly has joined #openstack-qa | 00:13 | |
| *** vhosakot has joined #openstack-qa | 00:14 | |
| openstackrecheck | Console logs not available after 13:24s for gate-tempest-dsvm-neutron-full 313163,1,c1b4e5a | 00:15 |
| *** dtantsur has joined #openstack-qa | 00:22 | |
| openstackgerrit | John L. Villalovos proposed openstack-dev/devstack: Export the 'short_source' function https://review.openstack.org/313132 | 00:23 |
| openstackgerrit | John L. Villalovos proposed openstack-dev/devstack: Export the 'short_source' function https://review.openstack.org/313132 | 00:24 |
| *** salv-orlando has joined #openstack-qa | 00:26 | |
| *** madhu_ak has quit IRC | 00:28 | |
| openstackrecheck | Console logs not available after 13:27s for gate-horizon-dsvm-integration 304103,4,44a0aab | 00:29 |
| *** salv-orlando has quit IRC | 00:32 | |
| *** dwalleck has quit IRC | 00:35 | |
| *** mriedem has quit IRC | 00:39 | |
| *** akshai has joined #openstack-qa | 00:40 | |
| *** akshai_ has joined #openstack-qa | 00:42 | |
| openstackrecheck | Console logs not available after 13:21s for gate-tempest-dsvm-full-devstack-plugin-ceph 313166,1,acc0686 | 00:42 |
| *** akshai has quit IRC | 00:45 | |
| *** mriedem has joined #openstack-qa | 00:46 | |
| *** dgonzalez has quit IRC | 00:48 | |
| *** frickler has quit IRC | 00:49 | |
| *** frickler has joined #openstack-qa | 00:50 | |
| *** dgonzalez has joined #openstack-qa | 00:50 | |
| *** salv-orlando has joined #openstack-qa | 00:51 | |
| *** mtanino has quit IRC | 00:52 | |
| *** hieulq has joined #openstack-qa | 00:53 | |
| *** hoangcx has joined #openstack-qa | 00:54 | |
| openstackrecheck | Console logs not available after 13:24s for gate-tempest-dsvm-neutron-full 274210,2,748fe03 | 00:55 |
| *** salv-orlando has quit IRC | 00:56 | |
| *** suyash has quit IRC | 00:57 | |
| *** cody-somerville has joined #openstack-qa | 01:03 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Introduce scope in the auth API https://review.openstack.org/313171 | 01:09 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 01:09 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Add admin role on domain for v3 https://review.openstack.org/226297 | 01:09 |
| openstackrecheck | Console logs not available after 13:23s for gate-tempest-dsvm-full 221020,13,71ef162 | 01:09 |
| *** yangyapeng has joined #openstack-qa | 01:12 | |
| *** yangyapeng has quit IRC | 01:21 | |
| *** salv-orlando has joined #openstack-qa | 01:22 | |
| openstackrecheck | Console logs not available after 13:23s for gate-tempest-dsvm-full 226297,7,2dd844d | 01:23 |
| *** dwalleck has joined #openstack-qa | 01:26 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 01:27 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Add admin role on domain for v3 https://review.openstack.org/226297 | 01:27 |
| *** salv-orlando has quit IRC | 01:30 | |
| *** ddieterly has quit IRC | 01:31 | |
| openstackrecheck | Console logs not available after 13:23s for gate-nova-docs 312910,4,eee144a | 01:36 |
| *** vhosakot_ has joined #openstack-qa | 01:38 | |
| *** vhosakot_ has quit IRC | 01:38 | |
| *** yamahata has quit IRC | 01:43 | |
| *** cody-somerville has quit IRC | 01:47 | |
| stevemar | armax: would checking for something like "tempest.api.identity.v3.test_users.IdentityV3UsersTest.test_user_update_own_password [2.128660s] ... FAILED" not work? | 01:48 |
| armax | stevemar: it would, but that could potentially yield to many results | 01:49 |
| *** Swami has quit IRC | 01:49 | |
| stevemar | hmm ... maybe | 01:49 |
| armax | and it would not be an ideal query | 01:49 |
| openstackrecheck | Console logs not available after 13:23s for gate-tempest-dsvm-full 313171,2,d11e83d | 01:49 |
| armax | stevemar: I am saying this based on the feedback I typically got when writing E-R queries | 01:50 |
| stevemar | armax: hmm https://github.com/openstack-infra/elastic-recheck/blob/master/queries/1577558.yaml | 01:55 |
| stevemar | oh nvm unrelated, that was a straight up failure | 01:55 |
| stevemar | this password one is more racy | 01:56 |
| *** salv-orlando has joined #openstack-qa | 01:57 | |
| armax | stevemar: if mriedem if happy with something like that, we can certainly come up with a similar query to catch bug 1578866 | 01:58 |
| openstack | bug 1578866 in OpenStack Identity (keystone) "test_user_update_own_password failing intermittently" [Undecided,New] https://launchpad.net/bugs/1578866 | 01:58 |
| stevemar | armax: message:"testtools.matchers._impl.MismatchError: <bound method V3TokenClient.auth of <tempest.lib.services.identity.v3.token_client.V3TokenClient" only brings up 1 hit :( | 01:59 |
| armax | stevemar: over how long? | 01:59 |
| stevemar | armax: 2 hits over 2+ hrs | 01:59 |
| stevemar | ah there we go, all day gives me 21 hits | 02:00 |
| armax | stevemar: perhaps we can make the testcase emit a better error | 02:00 |
| stevemar | seems transienty enough | 02:00 |
| armax | before trying to catch it with an E-R query | 02:00 |
| armax | esttools.matchers._impl.MismatchError: <bound method V3TokenClient.auth of <tempest.lib.services.identity.v3.token_client.V3TokenClient object at 0x7f77a88c6950>> returned bla bla is nasty enough | 02:01 |
| stevemar | presently just returns a dict :) | 02:01 |
| openstackrecheck | Console logs not available after 13:22s for gate-tempest-dsvm-full 312791,2,88d9fff | 02:03 |
| stevemar | armax: i'll submit an ER patch for the query i'm using, should be good enough *shrugs* | 02:03 |
| armax | stevemar: ack | 02:04 |
| *** anteaya has quit IRC | 02:04 | |
| *** woodster_ has quit IRC | 02:08 | |
| *** rfolco has quit IRC | 02:08 | |
| *** salv-orlando has quit IRC | 02:09 | |
| openstackgerrit | Steve Martinelli proposed openstack-infra/elastic-recheck: add query for bug 1578866 https://review.openstack.org/313205 | 02:10 |
| openstack | bug 1578866 in OpenStack Identity (keystone) "test_user_update_own_password failing intermittently" [Undecided,New] https://launchpad.net/bugs/1578866 | 02:10 |
| stevemar | armax: ^ | 02:11 |
| *** apoorvad has quit IRC | 02:16 | |
| *** dwalleck has quit IRC | 02:16 | |
| openstackrecheck | Console logs not available after 13:23s for gate-horizon-tox-py27dj19 268368,40,a67b745 | 02:16 |
| openstackgerrit | Ryan Moats proposed openstack-dev/devstack: DNM!!! Let's see what breaks when neutron-debug isn't called https://review.openstack.org/313147 | 02:19 |
| *** dwalleck has joined #openstack-qa | 02:19 | |
| *** gouthamr_ has quit IRC | 02:20 | |
| *** vhosakot has quit IRC | 02:25 | |
| mriedem | stevemar: so that's a new regression today? | 02:28 |
| mriedem | or since 4/29 when fernet was made the default in devstack? | 02:28 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22testtools.matchers._impl.MismatchError%3A%20%26lt%3Bbound%20method%20V3TokenClient.auth%20of%20%26lt%3Btempest.lib.services.identity.v3.token_client.V3TokenClient%5C%22%20AND%20tags%3A%5C%22console%5C%22%20AND%20voting%3A1&from=10d | 02:28 |
| openstackrecheck | Console logs not available after 13:24s for gate-horizon-npm-run-test 312287,2,c790f0f | 02:30 |
| mriedem | +W | 02:30 |
| *** vhosakot has joined #openstack-qa | 02:31 | |
| *** vhosakot has quit IRC | 02:33 | |
| openstackgerrit | Merged openstack-infra/elastic-recheck: add query for bug 1578866 https://review.openstack.org/313205 | 02:35 |
| openstack | bug 1578866 in OpenStack Identity (keystone) "test_user_update_own_password failing intermittently" [High,Confirmed] https://launchpad.net/bugs/1578866 | 02:35 |
| *** mriedem has quit IRC | 02:36 | |
| *** openstackrecheck has quit IRC | 02:40 | |
| *** openstackrecheck has joined #openstack-qa | 02:40 | |
| *** dwalleck has quit IRC | 02:43 | |
| *** dwalleck has joined #openstack-qa | 02:43 | |
| *** vhosakot has joined #openstack-qa | 02:47 | |
| *** jasonsb has quit IRC | 02:48 | |
| *** jasonsb has joined #openstack-qa | 02:48 | |
| *** jecarey has joined #openstack-qa | 02:59 | |
| *** jecarey has quit IRC | 03:04 | |
| *** binbincong has quit IRC | 03:04 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/284421 failed gate-horizon-dsvm-integration in the gate queue with an unrecognized error | 03:05 |
| *** fnaval_ has joined #openstack-qa | 03:06 | |
| *** fnaval has quit IRC | 03:09 | |
| *** jrcloud has joined #openstack-qa | 03:09 | |
| *** zackf has quit IRC | 03:10 | |
| *** akshai_ has quit IRC | 03:15 | |
| *** jasonsb has quit IRC | 03:16 | |
| openstackrecheck | Console logs not available after 13:25s for gate-tempest-dsvm-full 304511,22,ab513fe | 03:18 |
| *** markvoelker has quit IRC | 03:20 | |
| *** salv-orlando has joined #openstack-qa | 03:21 | |
| *** yangyapeng has joined #openstack-qa | 03:28 | |
| *** zackf has joined #openstack-qa | 03:29 | |
| *** jrcloud has quit IRC | 03:29 | |
| *** binbincong has joined #openstack-qa | 03:30 | |
| *** salv-orlando has quit IRC | 03:30 | |
| *** jecarey has joined #openstack-qa | 03:31 | |
| openstackrecheck | Console logs not available after 13:28s for gate-tempest-dsvm-neutron-full-liberty 221020,15,04daad6 | 03:32 |
| *** jecarey has quit IRC | 03:40 | |
| *** jecarey has joined #openstack-qa | 03:41 | |
| *** salv-orlando has joined #openstack-qa | 03:44 | |
| *** jecarey has quit IRC | 03:45 | |
| *** salv-orlando has quit IRC | 03:46 | |
| *** jrcloud has joined #openstack-qa | 03:49 | |
| *** dwalleck has quit IRC | 03:50 | |
| *** zackf has quit IRC | 03:50 | |
| *** dwalleck has joined #openstack-qa | 03:51 | |
| *** jasonsb has joined #openstack-qa | 03:52 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 03:57 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: pick Add admin role on domain for v3 https://review.openstack.org/226297 | 03:57 |
| openstackgerrit | Merged openstack/tempest: Added test for router's port update with fixed IP. https://review.openstack.org/202351 | 04:03 |
| *** jrcloud has quit IRC | 04:06 | |
| *** zackf has joined #openstack-qa | 04:07 | |
| *** yfried_ has joined #openstack-qa | 04:07 | |
| *** zz_dimtruck is now known as dimtruck | 04:12 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/309246 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1463631 | 04:13 |
| openstack | Launchpad bug 1463631 in grenade "60_nova/resources.sh:106:ping_check_public fails intermittently" [Undecided,Confirmed] | 04:13 |
| *** markvoelker has joined #openstack-qa | 04:21 | |
| *** dave-mcc_ has quit IRC | 04:26 | |
| *** markvoelker has quit IRC | 04:26 | |
| *** salv-orlando has joined #openstack-qa | 04:28 | |
| *** salv-orlando has quit IRC | 04:32 | |
| *** dimtruck is now known as zz_dimtruck | 04:46 | |
| *** jrcloud has joined #openstack-qa | 04:49 | |
| *** zackf has quit IRC | 04:51 | |
| *** cody-somerville has joined #openstack-qa | 04:54 | |
| *** cody-somerville has joined #openstack-qa | 04:54 | |
| *** dwalleck has quit IRC | 04:58 | |
| *** dwalleck has joined #openstack-qa | 04:58 | |
| openstackrecheck | Console logs not available after 13:26s for gate-tempest-dsvm-lxc-f23 312773,1,061f223 | 05:01 |
| *** salv-orlando has joined #openstack-qa | 05:01 | |
| *** fedexo has joined #openstack-qa | 05:02 | |
| *** salv-orlando has quit IRC | 05:04 | |
| *** Poornima has joined #openstack-qa | 05:07 | |
| openstackrecheck | Console logs not available after 13:27s for gate-tempest-dsvm-neutron-full-liberty 221020,16,db84f21 | 05:15 |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/312000 failed gate-horizon-dsvm-integration in the gate queue with an unrecognized error | 05:15 |
| *** salv-orlando has joined #openstack-qa | 05:17 | |
| *** markvoelker has joined #openstack-qa | 05:22 | |
| *** salv-orlando has quit IRC | 05:24 | |
| *** markvoelker has quit IRC | 05:26 | |
| *** vhosakot has quit IRC | 05:32 | |
| openstackrecheck | Console logs not available after 13:25s for gate-cinder-python27-db 313213,1,eb5fb5b | 05:32 |
| *** salv-orlando has joined #openstack-qa | 05:35 | |
| *** salv-orlando has quit IRC | 05:39 | |
| *** yolanda has joined #openstack-qa | 05:55 | |
| *** jasonsb has quit IRC | 06:08 | |
| *** jasonsb has joined #openstack-qa | 06:08 | |
| *** fnaval_ has quit IRC | 06:09 | |
| *** jasonsb has quit IRC | 06:13 | |
| *** fnaval has joined #openstack-qa | 06:16 | |
| *** jasonsb has joined #openstack-qa | 06:16 | |
| *** markvoelker has joined #openstack-qa | 06:23 | |
| *** itsuugo has quit IRC | 06:25 | |
| *** markvoelker has quit IRC | 06:27 | |
| *** fawadkhaliq has joined #openstack-qa | 06:31 | |
| *** dwalleck has quit IRC | 06:32 | |
| *** fedexo has quit IRC | 06:36 | |
| openstackgerrit | zhufl proposed openstack/tempest: Correct mispell words in comments https://review.openstack.org/313242 | 06:38 |
| *** e0ne has joined #openstack-qa | 06:39 | |
| *** salv-orlando has joined #openstack-qa | 06:46 | |
| *** yolanda has quit IRC | 06:46 | |
| *** yolanda has joined #openstack-qa | 06:47 | |
| *** Serlex has joined #openstack-qa | 06:48 | |
| *** yaume has joined #openstack-qa | 06:49 | |
| *** yaume has quit IRC | 06:50 | |
| *** yaume has joined #openstack-qa | 06:50 | |
| *** itsuugo has joined #openstack-qa | 06:50 | |
| *** e0ne has quit IRC | 06:53 | |
| *** salv-orlando has quit IRC | 06:57 | |
| *** grafuls has joined #openstack-qa | 06:59 | |
| *** e0ne has joined #openstack-qa | 06:59 | |
| *** e0ne has quit IRC | 07:00 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack-dev/devstack: Updated from generate-devstack-plugins-list https://review.openstack.org/312335 | 07:00 |
| *** yolanda has quit IRC | 07:02 | |
| *** yolanda has joined #openstack-qa | 07:06 | |
| *** hieulq has quit IRC | 07:07 | |
| *** hoangcx has quit IRC | 07:08 | |
| *** fawadkhaliq has quit IRC | 07:14 | |
| *** sridharg has joined #openstack-qa | 07:14 | |
| *** e0ne has joined #openstack-qa | 07:16 | |
| *** ifarkas has joined #openstack-qa | 07:21 | |
| *** yolanda has quit IRC | 07:21 | |
| *** markvoelker has joined #openstack-qa | 07:23 | |
| *** nadya has joined #openstack-qa | 07:24 | |
| *** salv-orlando has joined #openstack-qa | 07:26 | |
| *** markvoelker has quit IRC | 07:28 | |
| *** yolanda has joined #openstack-qa | 07:32 | |
| *** fawadkhaliq has joined #openstack-qa | 07:33 | |
| *** sridharg has quit IRC | 07:37 | |
| *** openstackstatus has quit IRC | 07:38 | |
| *** cody-somerville has quit IRC | 07:40 | |
| *** openstackstatus has joined #openstack-qa | 07:40 | |
| *** ChanServ sets mode: +v openstackstatus | 07:40 | |
| *** salv-orlando has quit IRC | 07:40 | |
| *** sridharg has joined #openstack-qa | 07:43 | |
| *** jpena|off is now known as jpena | 07:44 | |
| *** nadya has quit IRC | 07:46 | |
| *** sridharg has quit IRC | 07:46 | |
| *** sridharg has joined #openstack-qa | 07:47 | |
| *** jlanoux has joined #openstack-qa | 07:47 | |
| *** cody-somerville has joined #openstack-qa | 07:52 | |
| *** hoangcx has joined #openstack-qa | 07:54 | |
| *** nadya has joined #openstack-qa | 08:02 | |
| *** mkrcmari__ has joined #openstack-qa | 08:05 | |
| *** e0ne has quit IRC | 08:07 | |
| *** mvk_ has quit IRC | 08:08 | |
| *** pilgrimstack1 has joined #openstack-qa | 08:09 | |
| *** pilgrimstack has quit IRC | 08:09 | |
| *** sridharg has quit IRC | 08:09 | |
| *** BobBall_AWOL is now known as BobBall | 08:11 | |
| *** BobBall has left #openstack-qa | 08:11 | |
| *** lucas-afk is now known as lucasagomes | 08:15 | |
| openstackrecheck | Console logs not available after 13:24s for gate-tempest-dsvm-postgres-full 272118,23,5b99f53 | 08:16 |
| *** salv-orlando has joined #openstack-qa | 08:19 | |
| *** e0ne has joined #openstack-qa | 08:20 | |
| *** sridharg has joined #openstack-qa | 08:21 | |
| *** sridharg has quit IRC | 08:21 | |
| *** sridharg has joined #openstack-qa | 08:22 | |
| *** salv-orlando has quit IRC | 08:24 | |
| *** hieulq has joined #openstack-qa | 08:27 | |
| *** shardy has joined #openstack-qa | 08:28 | |
| openstackrecheck | Console logs not available after 13:23s for gate-horizon-npm-run-lint 313243,1,61da7f1 | 08:30 |
| *** pcaruana has joined #openstack-qa | 08:36 | |
| *** e0ne has quit IRC | 08:40 | |
| *** rossella_s has joined #openstack-qa | 08:41 | |
| *** binbincong has quit IRC | 08:41 | |
| *** binbincong has joined #openstack-qa | 08:42 | |
| *** e0ne has joined #openstack-qa | 08:42 | |
| *** abehl has joined #openstack-qa | 08:43 | |
| *** mkrcmari__ has quit IRC | 08:43 | |
| *** salv-orlando has joined #openstack-qa | 08:44 | |
| *** salv-orlando has quit IRC | 08:48 | |
| *** zeih has joined #openstack-qa | 08:53 | |
| *** rossella_s has quit IRC | 08:54 | |
| *** fawadkhaliq has quit IRC | 08:54 | |
| *** rossella_s has joined #openstack-qa | 08:54 | |
| *** ihrachys has joined #openstack-qa | 08:54 | |
| *** apevec has joined #openstack-qa | 08:55 | |
| *** jordanP has joined #openstack-qa | 08:55 | |
| *** tesseract has joined #openstack-qa | 08:56 | |
| *** tesseract is now known as Guest42930 | 08:56 | |
| *** abehl has quit IRC | 08:57 | |
| *** anshul has joined #openstack-qa | 08:57 | |
| *** anshul is now known as Guest14219 | 08:57 | |
| *** alex_xu has quit IRC | 09:03 | |
| openstackrecheck | Console logs not available after 13:24s for gate-grenade-dsvm 303345,45,e920d95 | 09:03 |
| *** nadya has quit IRC | 09:04 | |
| *** alex_xu has joined #openstack-qa | 09:05 | |
| *** apevec has left #openstack-qa | 09:11 | |
| *** john51 has quit IRC | 09:11 | |
| *** cody-somerville has quit IRC | 09:12 | |
| *** john51 has joined #openstack-qa | 09:12 | |
| *** zeih has quit IRC | 09:12 | |
| *** sambetts|afk is now known as sambetts | 09:14 | |
| *** mkrcmari__ has joined #openstack-qa | 09:28 | |
| *** salv-orlando has joined #openstack-qa | 09:41 | |
| *** tosky has joined #openstack-qa | 09:47 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Add admin role on domain for v3 https://review.openstack.org/226297 | 09:48 |
| *** salv-orlando has quit IRC | 09:49 | |
| jordanP | andreaf, hi. I am here if you can to discuss some comments I left | 09:51 |
| jordanP | s/can/want/ | 09:52 |
| andreaf | jordanP: hi | 09:52 |
| andreaf | jordanP: I'm not sure what else to say about the re-raise - I don't want to lose the original traceback - so I need to use sys.exc_info to capture it | 09:55 |
| andreaf | jordanP: re-raising the exception with traceback works differently in python2 and python3, so I have to use six | 09:56 |
| jordanP | but if you just use "raise" then the traceback is not lost, it it ? | 09:56 |
| andreaf | jordanP: raise wi | 09:56 |
| *** sshnaidm has quit IRC | 09:57 | |
| andreaf | jordanP: raise will raise the last exception, not the one I want | 09:57 |
| andreaf | if there is an exception during the exception handling | 09:57 |
| jordanP | yes but the raise statement would be outside this scope | 09:58 |
| jordanP | (outside the inner exception handling | 09:58 |
| jordanP | ) | 09:58 |
| jordanP | I need to test it | 09:58 |
| jordanP | andreaf, you are correct | 10:00 |
| jordanP | it comes as a surprise to me... that raise behavior | 10:01 |
| andreaf | yes it surprised me as well, that's why I say the code is ugly thanks for python | 10:01 |
| andreaf | I think python could do a much better job here | 10:01 |
| jordanP | I raged +A the patch :) | 10:02 |
| andreaf | I could try and use the oslo.utils re-raise, which probably does the same thing | 10:02 |
| jordanP | yes, it should indeed to the same thing | 10:02 |
| jordanP | let's move on | 10:02 |
| andreaf | jordanP: ok - I can add the @functools.wraps in a follow-up patch if you want | 10:02 |
| andreaf | thanks | 10:03 |
| *** ihrachys has quit IRC | 10:04 | |
| *** sridharg has quit IRC | 10:08 | |
| *** hoangcx has quit IRC | 10:08 | |
| *** dims has quit IRC | 10:20 | |
| andreaf | jordanP: as you are at it, would you mind reviewing https://review.openstack.org/#/c/311283/ as well, which stands between the two ones you +A already | 10:22 |
| *** cdent has joined #openstack-qa | 10:22 | |
| andreaf | jordanP: I will gladly re-use the same logic in the scenario tests as well, as soon as they are untangled enough to do so... at the moment the chain of calls between helpers prevents that | 10:23 |
| jordanP | andreaf, done. But I don't like the overall approach you've chose. You've coupled 2 seperate things: the SSH client and the openstack clients | 10:24 |
| andreaf | jordanP: what do you mean? the fact that I pass the servers client into the RemoteClient? | 10:24 |
| jordanP | so we will never be able to move common/utils/linux/remote_client.py tempest.lib | 10:24 |
| jordanP | yes | 10:24 |
| jordanP | I think this is good for debugging but it should be temporary | 10:25 |
| *** markvoelker has joined #openstack-qa | 10:26 | |
| jordanP | I hope it's going to go away if/when the SSH job will be stable | 10:26 |
| jordanP | you see my point ? | 10:27 |
| *** sdague has joined #openstack-qa | 10:28 | |
| jordanP | imo RemoteClient.exec_command and validate_authentication is about executing a remote command and checking authentication. And that's it | 10:28 |
| jordanP | it shouldn't do more | 10:28 |
| jordanP | if there's a failure than the caller should do something about it | 10:29 |
| andreaf | jordanP: the alternative would be to have an helper function that calls RemoteClient and then does the debugging in case of exception, but it's not so easy because different tests do different things with the remote client | 10:29 |
| andreaf | jordanP: yes but we have may callers and I don't want to repeat that logic in every single test... | 10:30 |
| *** markvoelker has quit IRC | 10:30 | |
| jordanP | then yeah the alternative with a helper function is an option. Any way, I am fine with the current approch because it limited (only a decorator and 2 decorated methods) | 10:31 |
| jordanP | it shouldn"t be generalized imo | 10:31 |
| jordanP | 'different tests do different things with the remote client' that looks like a bug to me | 10:32 |
| *** penchal has joined #openstack-qa | 10:32 | |
| jordanP | the remote_client should be so simple that it should support only one use case | 10:32 |
| *** yangyape_ has joined #openstack-qa | 10:33 | |
| *** grafuls has quit IRC | 10:35 | |
| *** yangyapeng has quit IRC | 10:36 | |
| *** yangyape_ has quit IRC | 10:37 | |
| *** sridharg has joined #openstack-qa | 10:42 | |
| *** gfidente has joined #openstack-qa | 10:44 | |
| *** gfidente has joined #openstack-qa | 10:44 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Introduce scope in the auth API https://review.openstack.org/313171 | 10:49 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 10:49 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: pick Add admin role on domain for v3 https://review.openstack.org/226297 | 10:49 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 10:51 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Add admin role on domain for v3 https://review.openstack.org/226297 | 10:51 |
| openstackrecheck | Console logs not available after 13:29s for gate-tempest-dsvm-cells 309287,4,12fdf0f | 10:53 |
| *** nadya has joined #openstack-qa | 10:55 | |
| *** dims has joined #openstack-qa | 11:02 | |
| *** yamamoto has quit IRC | 11:08 | |
| *** gouthamr has joined #openstack-qa | 11:21 | |
| openstackgerrit | Sean M. Collins proposed openstack-dev/devstack: Begin new lib/neutron https://review.openstack.org/168438 | 11:23 |
| openstackgerrit | Sean M. Collins proposed openstack-dev/devstack: Begin new lib/neutron https://review.openstack.org/168438 | 11:24 |
| *** markvoelker has joined #openstack-qa | 11:27 | |
| *** gouthamr_ has joined #openstack-qa | 11:27 | |
| *** sridharg has quit IRC | 11:29 | |
| *** gouthamr has quit IRC | 11:31 | |
| *** markvoelker has quit IRC | 11:31 | |
| *** alex_xu has quit IRC | 11:42 | |
| *** yolanda has quit IRC | 11:43 | |
| *** jpena is now known as jpena|lunch | 11:46 | |
| *** alex_xu has joined #openstack-qa | 11:47 | |
| *** yolanda has joined #openstack-qa | 11:48 | |
| *** yangyapeng has joined #openstack-qa | 11:49 | |
| *** yamamoto has joined #openstack-qa | 11:56 | |
| *** salv-orlando has joined #openstack-qa | 11:58 | |
| openstackrecheck | Console logs not available after 13:21s for gate-tempest-dsvm-full 312929,2,3363456 | 11:59 |
| *** yamamoto has quit IRC | 12:03 | |
| *** yamamoto has joined #openstack-qa | 12:05 | |
| *** rfolco has joined #openstack-qa | 12:07 | |
| *** aysyd has joined #openstack-qa | 12:09 | |
| openstackgerrit | Sean M. Collins proposed openstack-dev/devstack: Begin new lib/neutron https://review.openstack.org/168438 | 12:10 |
| *** lucasagomes is now known as lucas-hungry | 12:11 | |
| openstackrecheck | Console logs not available after 13:23s for gate-grenade-dsvm-multinode 303345,46,c23a1da | 12:13 |
| *** raildo-afk is now known as raildo | 12:13 | |
| *** markvoelker has joined #openstack-qa | 12:14 | |
| *** yamamoto has quit IRC | 12:17 | |
| *** sridharg has joined #openstack-qa | 12:17 | |
| *** yamamoto has joined #openstack-qa | 12:18 | |
| *** ddieterly has joined #openstack-qa | 12:19 | |
| *** yamamoto has quit IRC | 12:19 | |
| *** yamamoto has joined #openstack-qa | 12:20 | |
| *** ddieterly is now known as ddieterly[away] | 12:20 | |
| *** grafuls has joined #openstack-qa | 12:22 | |
| *** alex_xu has quit IRC | 12:22 | |
| *** binbincong has quit IRC | 12:23 | |
| *** binbincong has joined #openstack-qa | 12:24 | |
| *** alex_xu has joined #openstack-qa | 12:25 | |
| openstackrecheck | Console logs not available after 13:21s for gate-horizon-dsvm-integration 303429,45,327589f | 12:26 |
| *** salv-orlando has quit IRC | 12:28 | |
| *** sridharg has quit IRC | 12:28 | |
| *** rodrigods has quit IRC | 12:29 | |
| *** rodrigods has joined #openstack-qa | 12:29 | |
| *** yamamoto has quit IRC | 12:29 | |
| *** regXboi has joined #openstack-qa | 12:33 | |
| *** yamamoto has joined #openstack-qa | 12:35 | |
| *** dkranz has joined #openstack-qa | 12:35 | |
| *** gszasz has joined #openstack-qa | 12:37 | |
| openstackrecheck | Console logs not available after 13:21s for gate-nova-python27-db 199093,6,18fc710 | 12:40 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/309271 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error | 12:40 |
| jordanP | andreaf, you here ? | 12:43 |
| andreaf | jordanP: more or less... having lunch | 12:43 |
| andreaf | jordanP: what's up? | 12:45 |
| jordanP | one of the complain here https://etherpad.openstack.org/p/tempest-refactor-ideas is "The class hierarchy needs to be squashed down. No more than 2 class inheritance levels allowed". I took a look at "Use scope in v3 identity client". You changed the base class of most of identity clients | 12:45 |
| jordanP | instead, could you add a __init_ method to those clients | 12:45 |
| jordanP | hum that would introduce code duplication if you had to put this long line "super(DomainScopedRestClient, self).__init__( | 12:46 |
| jordanP | auth_provider, service, region, | 12:46 |
| jordanP | endpoint_type=endpoint_type, build_interval=build_interval," | 12:46 |
| andreaf | jordanP: yes that was my initial implementation | 12:46 |
| *** ddieterly[away] is now known as ddieterly | 12:46 | |
| jordanP | in many clients.. | 12:46 |
| jordanP | but the current approach adds yet another level of indirection | 12:47 |
| andreaf | I'm a bit struggling on what's the best balance here between inheritance and code readability | 12:47 |
| andreaf | I though the current option is rather readable, but I'm probably going to change | 12:47 |
| jordanP | what is like with the current rest-client is that it's generic | 12:48 |
| *** yamamoto has quit IRC | 12:48 | |
| jordanP | it's only tied to keystone with the auth_provider argument | 12:48 |
| jordanP | but now you add this "auth_scope" | 12:48 |
| jordanP | ah auth_scope was already there my bad | 12:49 |
| jordanP | ah no, you added it on a dependent patch, okay | 12:49 |
| andreaf | auth_provider was already there, I added auth_scope | 12:49 |
| jordanP | yes, so the less coupling between a generic rest client and keystone, the better | 12:49 |
| jordanP | can"t the auth provider have the scope in it ? | 12:50 |
| *** yamamoto has joined #openstack-qa | 12:50 | |
| andreaf | sure but a single auth_provider can serve all kind of scopes, so it's really a service client thing to decide what scope to use | 12:50 |
| jordanP | one way of doing it, instead of inheritance, is defining a class variable "scope" that each client can set or overwrite | 12:51 |
| andreaf | jordanP: yes I thought about that, but that's exactly the type of magic that sdague didn't like very much about how we deal with credentials and other resources | 12:52 |
| jordanP | ok | 12:52 |
| jordanP | yes, I didn't think much about it sorry | 12:52 |
| jordanP | going to review it more seriously, as a while | 12:53 |
| jordanP | as whole | 12:53 |
| andreaf | jordanP: no worries I welcome your input, I'm a bit unsure yet about what the best approach is | 12:53 |
| *** akshai has joined #openstack-qa | 12:53 | |
| andreaf | jordanP: other ways of doing it would be to have an helper set_scope() which can be invoked on the rest client, or alternatively to pass the scope into every single API method | 12:55 |
| *** ddieterly has quit IRC | 12:56 | |
| jordanP | I am not sure it's better | 12:56 |
| *** sridharg has joined #openstack-qa | 12:57 | |
| andreaf | the latter would imply extending all HTTP methods (get, post, etc) with a scope parameter | 12:57 |
| andreaf | that would be the more complete solution, but I was trying to keep it simple as long as I don't have to go there... | 12:58 |
| jordanP | yes, changing the siagnatures of many methods is usually a bad sign | 12:58 |
| andreaf | one of the goodies of that series of patches is that I finally enabled testing of dynamic cred provider with v3 identity :) | 12:58 |
| jordanP | yep, that's good ! | 13:00 |
| *** ifarkas has quit IRC | 13:01 | |
| *** nadya has quit IRC | 13:01 | |
| *** jaypipes has joined #openstack-qa | 13:02 | |
| *** ChanServ sets mode: +o jaypipes | 13:02 | |
| *** yolanda has quit IRC | 13:02 | |
| *** Gen_ has joined #openstack-qa | 13:05 | |
| *** anteaya has joined #openstack-qa | 13:06 | |
| openstackgerrit | Sean M. Collins proposed openstack-dev/devstack: Begin new lib/neutron https://review.openstack.org/168438 | 13:07 |
| *** yolanda has joined #openstack-qa | 13:07 | |
| *** jpena|lunch is now known as jpena | 13:12 | |
| *** yamamoto has quit IRC | 13:13 | |
| EmilienM | mtreinish: hey, when you have time, can you look https://review.openstack.org/#/c/312916/ please? | 13:13 |
| *** yamamoto has joined #openstack-qa | 13:15 | |
| *** mtanino has joined #openstack-qa | 13:17 | |
| *** fnaval_ has joined #openstack-qa | 13:17 | |
| *** yamamoto has quit IRC | 13:18 | |
| *** lucas-hungry is now known as lucasagomes | 13:18 | |
| *** yamamoto has joined #openstack-qa | 13:19 | |
| *** mriedem has joined #openstack-qa | 13:20 | |
| *** fnaval has quit IRC | 13:20 | |
| *** yamamoto has quit IRC | 13:21 | |
| *** yamamoto has joined #openstack-qa | 13:23 | |
| rodrigods | jordanP, hi.... replied here: https://review.openstack.org/#/c/285541/8 | 13:24 |
| *** pcaruana has quit IRC | 13:25 | |
| jordanP | rodrigods, got you thanks | 13:25 |
| jordanP | rodrigods, could you split test_project_create_with_parent into 2 test cases ? | 13:26 |
| jordanP | it looks like 2 test cases | 13:26 |
| rodrigods | jordanP, yeah, absolutely | 13:26 |
| jordanP | one with parent_id, the other without parent id ? | 13:26 |
| jordanP | thanks ! | 13:26 |
| rodrigods | jordanP, one thing is... without parent id is a regular "create_project", this test is really testing the creation with a parent_id | 13:27 |
| *** yamamoto has quit IRC | 13:27 | |
| *** porrua has joined #openstack-qa | 13:27 | |
| *** yamamoto has joined #openstack-qa | 13:27 | |
| *** mtanino has quit IRC | 13:27 | |
| rodrigods | jordanP, so... in order to create a project with parent, we need to first create the parent itself | 13:27 |
| jordanP | then you can remove all of the test as it's already tested somewhere else, right ? | 13:28 |
| jordanP | *half ! | 13:28 |
| *** ddieterly has joined #openstack-qa | 13:28 | |
| *** ddieterly is now known as ddieterly[away] | 13:28 | |
| jordanP | not all :p | 13:28 |
| jordanP | maybe add a couple of asserts in the "create regular project" test ? | 13:28 |
| jordanP | if that makes sense | 13:28 |
| *** salv-orlando has joined #openstack-qa | 13:28 | |
| *** jecarey has joined #openstack-qa | 13:29 | |
| rodrigods | jordanP, hmm let me try to split here | 13:29 |
| jordanP | I am not sure, I am not familiar with Keystone v3 yet. But currently to test looks like it tries to test 2 things | 13:29 |
| *** jecarey has quit IRC | 13:29 | |
| jordanP | *the test | 13:29 |
| *** jecarey has joined #openstack-qa | 13:29 | |
| rodrigods | jordanP, sure, will try to make more direct | 13:29 |
| *** jecarey has quit IRC | 13:29 | |
| *** jecarey has joined #openstack-qa | 13:30 | |
| *** ddieterly[away] is now known as ddieterly | 13:30 | |
| *** salv-orlando has quit IRC | 13:31 | |
| *** penchal has quit IRC | 13:33 | |
| *** nadya has joined #openstack-qa | 13:33 | |
| *** gouthamr_ is now known as gouthamr | 13:34 | |
| *** Poornima has quit IRC | 13:35 | |
| *** yamamoto has quit IRC | 13:36 | |
| *** yaume has quit IRC | 13:39 | |
| *** yamamoto has joined #openstack-qa | 13:40 | |
| *** ifarkas has joined #openstack-qa | 13:43 | |
| *** sdague has quit IRC | 13:43 | |
| bknudson | does devstack need to continue to support ubuntu 12.04? | 13:44 |
| *** yamamoto has quit IRC | 13:44 | |
| *** yamamoto has joined #openstack-qa | 13:47 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 13:48 | |
| *** ddieterly is now known as ddieterly[away] | 13:48 | |
| openstackgerrit | Rodrigo Duarte proposed openstack/tempest: Add parent_id to create_project https://review.openstack.org/285541 | 13:49 |
| rodrigods | jordanP, ^ updated | 13:49 |
| *** jaypipes has quit IRC | 13:50 | |
| *** yamamoto has quit IRC | 13:51 | |
| jordanP | rodrigods, oh no, now you tried to stuff 2 changes in one ? | 13:52 |
| jordanP | you shouldn"t have renamed tenant to project | 13:52 |
| jordanP | we want this to happen but not now | 13:52 |
| rodrigods | jordanP, what? | 13:52 |
| jordanP | I mean not with a patch that adds a test | 13:52 |
| jordanP | ahh my bad | 13:53 |
| jordanP | lemme double check | 13:53 |
| rodrigods | jordanP, i didn't rename anything, just created a method to handle the creation of root projects | 13:53 |
| jordanP | ah right, no it's because of the rebase | 13:53 |
| jordanP | https://review.openstack.org/#/c/285541/8..9 | 13:53 |
| jordanP | the diff between PS 8 and PS9 looks big | 13:53 |
| jordanP | but that's because of the rebase | 13:54 |
| jordanP | no worries :) | 13:54 |
| jordanP | sorry | 13:54 |
| rodrigods | ah, ok | 13:54 |
| rodrigods | np :) | 13:54 |
| openstackgerrit | Merged openstack/tempest: Extend remote client to allow for better debugging https://review.openstack.org/311270 | 13:54 |
| *** Guest14219 has quit IRC | 13:54 | |
| openstackgerrit | Paulo Ewerton Gomes Fragoso proposed openstack/tempest: WIP: Adding cross-domain tests https://review.openstack.org/313558 | 13:55 |
| *** pauloewerton has joined #openstack-qa | 13:55 | |
| *** edmondsw has joined #openstack-qa | 13:55 | |
| *** yamamoto has joined #openstack-qa | 13:55 | |
| *** ddieterly[away] is now known as ddieterly | 13:56 | |
| openstackgerrit | Merged openstack/tempest: Pass server to RemoteClient in API tests https://review.openstack.org/311283 | 13:56 |
| openstackgerrit | Merged openstack/tempest: ssh before shelve to avoid ssh failures https://review.openstack.org/313041 | 13:56 |
| *** dansmith is now known as snotdan | 13:56 | |
| *** akshai has quit IRC | 13:57 | |
| *** snotdan is now known as superdan | 13:58 | |
| *** akshai has joined #openstack-qa | 13:59 | |
| *** vhosakot has joined #openstack-qa | 14:00 | |
| jordanP | rodrigods, what's the difference between test_project_create_without_parent and test_project_create_with_domain ? | 14:00 |
| rodrigods | jordanP, with reseller, the parent_id will point to the domain_id | 14:01 |
| rodrigods | if not provided | 14:01 |
| *** rbak_ has joined #openstack-qa | 14:02 | |
| *** yamamoto has quit IRC | 14:02 | |
| *** yangyapeng has quit IRC | 14:02 | |
| jordanP | couldn't you just add a self.assertEqual(parent_id, domain_id) in test_project_create_with_domain ? | 14:02 |
| *** yamamoto has joined #openstack-qa | 14:02 | |
| *** fnaval_ has quit IRC | 14:03 | |
| jordanP | I mean my point is, we duplicate a whole test just to add an attribut check, right ? | 14:03 |
| rodrigods | jordanP, sure... but we couldn't since the "with_domain" test would not pass in previous stable branches | 14:03 |
| *** pauloewerton has quit IRC | 14:03 | |
| rodrigods | so we would skip a test for stable branches that currently runs | 14:04 |
| jordanP | what about: | 14:04 |
| rodrigods | jordanP, | 14:04 |
| jordanP | if CONF.identity_feature_enabled.reseller: | 14:04 |
| jordanP | self.assertEqual(parent_id, domain_id) | 14:04 |
| rodrigods | hmm | 14:04 |
| rodrigods | jordanP, this line should be added everywhere | 14:05 |
| rodrigods | it that's the case | 14:05 |
| rodrigods | in all project tests, i mean | 14:05 |
| *** pauloewerton has joined #openstack-qa | 14:05 | |
| rodrigods | jordanP, we could remove the "without_parent" test, since it is redundant for the "with_parent" | 14:05 |
| jordanP | no, once is enough. | 14:05 |
| jordanP | I think so yes | 14:05 |
| rodrigods | but... we fallback to the state the code was before your suggestion | 14:06 |
| rodrigods | ok | 14:06 |
| jordanP | humm I see | 14:06 |
| jordanP | wait a sec :) | 14:06 |
| *** ramishra has quit IRC | 14:07 | |
| *** ramishra has joined #openstack-qa | 14:07 | |
| jordanP | I see now, I prefer PS 8. What I failed to understand back then is that half of the code in PS8 is just to setup the parent project | 14:08 |
| rodrigods | jordanP, ok... will leave the create_root_project method | 14:08 |
| jordanP | but it's needed, it's not related to the test itself, but you need it to like "setup the environment" | 14:08 |
| rodrigods | but will remove the new test | 14:08 |
| jordanP | no | 14:08 |
| rodrigods | yeah... exactly | 14:08 |
| jordanP | just revert to PS8 | 14:08 |
| jordanP | because no need to introduce a new create_root_project that is only used once | 14:09 |
| *** mtanino has joined #openstack-qa | 14:09 | |
| openstackgerrit | Rodrigo Duarte proposed openstack/tempest: Add parent_id to create_project https://review.openstack.org/285541 | 14:10 |
| jordanP | thanks | 14:12 |
| *** nadya has quit IRC | 14:12 | |
| rodrigods | thanks for the help jordanP | 14:14 |
| *** salv-orlando has joined #openstack-qa | 14:15 | |
| -openstackstatus- NOTICE: Zuul is currently recovering from a large number of changes, it will take a few hours until your job is processed. Please have patience and enjoy a great weekend! | 14:16 | |
| *** yaume has joined #openstack-qa | 14:17 | |
| *** vhosakot has quit IRC | 14:19 | |
| *** jaypipes has joined #openstack-qa | 14:20 | |
| *** ChanServ sets mode: +o jaypipes | 14:20 | |
| *** fnaval has joined #openstack-qa | 14:22 | |
| *** zz_dimtruck is now known as dimtruck | 14:23 | |
| *** gouthamr has quit IRC | 14:25 | |
| *** dmsimard has joined #openstack-qa | 14:25 | |
| *** dmsimard has left #openstack-qa | 14:28 | |
| *** dave-mccowan has joined #openstack-qa | 14:35 | |
| openstackgerrit | Sean M. Collins proposed openstack-dev/devstack: Begin new lib/neutron https://review.openstack.org/168438 | 14:40 |
| *** e0ne has quit IRC | 14:43 | |
| *** e0ne has joined #openstack-qa | 14:44 | |
| -openstackstatus- NOTICE: Zuul has been restarted. As a results, we only preserved patches in the gate queue. Be sure to recheck your patches in gerrit if needed. | 14:46 | |
| *** yamamoto has quit IRC | 14:47 | |
| *** nadya has joined #openstack-qa | 14:49 | |
| *** jaypipes has quit IRC | 14:55 | |
| *** grafuls has quit IRC | 15:00 | |
| *** dwalleck has joined #openstack-qa | 15:00 | |
| *** akshai has quit IRC | 15:01 | |
| *** Guest42930 has quit IRC | 15:05 | |
| *** david-lyle has joined #openstack-qa | 15:06 | |
| *** Serlex has quit IRC | 15:07 | |
| *** fnaval_ has joined #openstack-qa | 15:08 | |
| *** fnaval has quit IRC | 15:10 | |
| *** jaypipes has joined #openstack-qa | 15:11 | |
| *** ChanServ sets mode: +o jaypipes | 15:11 | |
| *** r-mibu has quit IRC | 15:13 | |
| *** r-mibu has joined #openstack-qa | 15:13 | |
| *** yamamoto has joined #openstack-qa | 15:14 | |
| jroll | is current grenade run with neutron or n-net? | 15:18 |
| jroll | I'm debugging some things by manually running things | 15:19 |
| jroll | eval $(openstack ip floating create public -f shell -c id -c ip) | 15:19 |
| jroll | looks like -c ip isn't a thing with neutron installed? | 15:19 |
| *** prometheanfire has joined #openstack-qa | 15:23 | |
| prometheanfire | Any devstack dev able to give the other +2 to https://review.openstack.org/#/c/310838 ? | 15:23 |
| *** akshai has joined #openstack-qa | 15:25 | |
| *** vhosakot has joined #openstack-qa | 15:25 | |
| *** yamamoto has quit IRC | 15:25 | |
| openstackgerrit | Jim Rollenhagen proposed openstack-dev/grenade: Allow creating floating ip address with Neutron enabled https://review.openstack.org/313600 | 15:26 |
| jroll | jlvillal: ^ | 15:26 |
| jlvillal | jroll: Thanks! | 15:26 |
| jroll | np | 15:26 |
| jlvillal | I will add it and spin a new test :) | 15:27 |
| jroll | note: untested :) | 15:27 |
| jroll | other than manual things in bash | 15:27 |
| openstackrecheck | openstack/requirements change: https://review.openstack.org/311761 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1577558 | 15:30 |
| openstack | Launchpad bug 1577558 in OpenStack Security Advisory "v2.0 fernet tokens audit ids are inconsistent" [Undecided,Incomplete] | 15:30 |
| *** jordanP has quit IRC | 15:31 | |
| jlvillal | dtroyer: So the non-voting job for devstack 'gate-tempest-dsvm-ironic-agent_ssh-nv' how can we be sure it is actually using the devstack change? | 15:34 |
| jlvillal | For example: https://review.openstack.org/#/c/313132/ | 15:34 |
| jlvillal | Tested locally and things are working. But I still see the message in the gate about 'short_source'. | 15:34 |
| jlvillal | jroll: FYI ^^^ | 15:34 |
| *** dtantsur is now known as dtantsur|afk | 15:35 | |
| *** jaypipes is now known as leakypipes | 15:36 | |
| *** Gen_ has quit IRC | 15:36 | |
| jlvillal | dtroyer: But could be my mistake. Which is likely :) | 15:38 |
| openstackgerrit | Vladyslav Drok proposed openstack/tempest: Skip unnecessary tests when using Ironic virt driver https://review.openstack.org/220223 | 15:40 |
| *** nadya has quit IRC | 15:40 | |
| *** rossella_s has quit IRC | 15:40 | |
| jlvillal | dtroyer: And never mind. I think it is my mistake. | 15:40 |
| *** rossella_s has joined #openstack-qa | 15:41 | |
| dmellado | jpena: I was chatting with mtreinish about your change | 15:42 |
| dmellado | and it seems that eventlet would be breaking tests that use pyopenssl (because it calls out to c code and can't be monkeypatched by eventlet) | 15:42 |
| dmellado | would you care to explain your need for it? | 15:42 |
| *** salv-orlando has quit IRC | 15:43 | |
| *** nadya has joined #openstack-qa | 15:44 | |
| jpena | so in short, we found out that the Puppet CI was failing in the Glance tests when using https. Digging in the issue, I see the Neutron tempest plugin is monkey patching 'socket'. I created the patch based on the similar code found in glanceclient | 15:45 |
| mtreinish | dmellado: that is the current situation, jpena patch switches tempest to use the eventlet provided pyopenssl which can handle the random eventlet thread switching | 15:45 |
| mtreinish | which is the wrong fix, there is no reason to use eventlet in tempest, a plugin or otherwise | 15:45 |
| mtreinish | and doing so isn't behavior we're going to code around in tempest. It's just a broken plugin | 15:46 |
| *** salv-orlando has joined #openstack-qa | 15:47 | |
| dmellado | mtreinish: so then you're implying that those glance failures could be related to the neutron plugin importing the eventlet? I see no relationship, but might be missing something | 15:48 |
| *** nadya has quit IRC | 15:48 | |
| mtreinish | dmellado: they are, that's how eventlet works | 15:48 |
| *** nadya has joined #openstack-qa | 15:48 | |
| *** ddieterly is now known as ddieterly[away] | 15:49 | |
| mtreinish | it's all monkey patching magic :) | 15:49 |
| dmellado | jpena: when not using the neutron plugin, are you fixing those failures? | 15:49 |
| dmellado | if that's because of the init on the neutron plugin, I'll have a look | 15:49 |
| mtreinish | and by starting it up in the neutron plugin side it'll interfere with code running elsewhere in the same process space (in this case a glance test case) | 15:50 |
| jpena | dmellado: let me give it a quick test | 15:50 |
| dmellado | mtreinish: I see, so jpena's patch is basically re-monkey-patching ;) | 15:50 |
| *** dave-mccowan has quit IRC | 15:50 | |
| mtreinish | dmellado: he's making tempest's use of openssl eventlet safe (by using eventlet's ssl wrapper) | 15:51 |
| *** nadya has quit IRC | 15:51 | |
| mtreinish | so yes :) | 15:51 |
| dmellado | https://github.com/openstack/neutron/blob/master/neutron/tests/__init__.py | 15:51 |
| dmellado | so here this would be the 'guilty' | 15:51 |
| dmellado | as the tempest tests (and plugin) | 15:52 |
| dmellado | are under neutron/test/tempests | 15:52 |
| jpena | dmellado: removing the eventlet.monkey_patch() call from https://github.com/openstack/neutron/blob/master/neutron/tests/__init__.py also fixes the issue, yep | 15:52 |
| mtreinish | dmellado: right, it's not even intentionally being invoked by the plugin | 15:52 |
| mtreinish | it's just a side effect of where it lives in the tree | 15:52 |
| dmellado | hm, interesting | 15:52 |
| jpena | the initial justification for having this call is in https://github.com/openstack/neutron/commit/ea76d10e7492eb90f6777940304c67691caa366b | 15:52 |
| mtreinish | it's seriously just a plugin bug | 15:53 |
| mtreinish | jpena: right, it has nothing to do with the tempest plugin | 15:53 |
| tosky | but tempest tests are different, I wonder if they shouldn't move them in a place where this kind of tricks for the "lower level" tests do not affect them | 15:53 |
| dmellado | I was thinking about that | 15:54 |
| mtreinish | EmilienM: ^^^ this also explains why it's only on newton, because there isn't a mitaka plugin | 15:54 |
| dmellado | to either move them to another repo, which would make things easier | 15:54 |
| dmellado | but amuller was against it | 15:54 |
| mtreinish | dmellado: ++ | 15:54 |
| tosky | so move the directory | 15:54 |
| dmellado | or move the directory to a level on top of the monkey patch | 15:54 |
| *** ddieterly[away] is now known as ddieterly | 15:55 | |
| dmellado | jpena: please open a bug in neutron and mame me the assignee | 15:56 |
| *** sridharg has quit IRC | 15:56 | |
| dmellado | I'll tackle this | 15:56 |
| jpena | dmellado: ack | 15:56 |
| EmilienM | dmellado: thanks | 15:56 |
| dmellado | np, thanks mtreinish ;) | 15:56 |
| EmilienM | jpena: so we abandon your patch and fix it in neutron? | 15:57 |
| *** davideagnello has joined #openstack-qa | 15:57 | |
| *** jlanoux has quit IRC | 15:57 | |
| dmellado | EmilienM: +1 | 15:57 |
| jpena | EmilienM: I guess so. Should we repurpose https://bugs.launchpad.net/tempest/+bug/1577019 for this? | 15:58 |
| openstack | Launchpad bug 1577019 in tempest "BasicOperationsImagesTest tempest tests fail" [Undecided,In progress] - Assigned to Javier Peña (jpena-c) | 15:58 |
| dmellado | jpena: is not really a bug in tempest, but in neutron | 15:58 |
| dmellado | now that the tempest plugin is a part of it | 15:58 |
| jpena | dmellado: yes, understood. Initially it was a Glance bug, but it's been changing culprit as we found more information | 15:59 |
| dmellado | in any case I'll speak with assaf and ihar next week about it | 15:59 |
| * dmellado goes off for now, enjoy your weekend | 16:00 | |
| EmilienM | dmellado: thanks | 16:02 |
| EmilienM | dmellado: for us, it's blocking our CI to bump to Newton testing : | 16:02 |
| EmilienM | :( | 16:02 |
| dmellado | EmilienM: I'll keep you in the loop | 16:02 |
| EmilienM | it would be great if we can sort this out next week | 16:03 |
| EmilienM | so we can produce a first Newton release on time by the end of the month | 16:03 |
| dmellado | EmilienM: do you have someone I could use as a helper? | 16:03 |
| *** yolanda has quit IRC | 16:03 | |
| dmellado | my backlog is 'really','really', huge | 16:03 |
| dmellado | so I could use someone from rdo team | 16:03 |
| EmilienM | dmellado: me or jpena can help. If it's too neutron specific, maybe we can ask Assaf | 16:04 |
| dmellado | sounds cool then, I'll ping you back on Mon ;) | 16:04 |
| *** jdandrea has quit IRC | 16:04 | |
| dmellado | jpena: +1 | 16:04 |
| *** jdandrea has joined #openstack-qa | 16:05 | |
| jpena | dmellado: count on me if I can help | 16:05 |
| EmilienM | thanks guys | 16:05 |
| jpena | I have a VM setup where I can reproduce the issue very quickly | 16:05 |
| *** minwang2 has joined #openstack-qa | 16:05 | |
| *** nadya has joined #openstack-qa | 16:06 | |
| *** johnthetubaguy has quit IRC | 16:08 | |
| *** johnthetubaguy has joined #openstack-qa | 16:08 | |
| *** drptbl has quit IRC | 16:09 | |
| dmellado | EmilienM: jpena and me already sync'ed, I'll ping you next week | 16:09 |
| dmellado | enjoy your weekend ;) | 16:09 |
| *** dmellado is now known as dmellado|off | 16:10 | |
| *** moravec1 has quit IRC | 16:11 | |
| EmilienM | see ya | 16:11 |
| *** moravec has joined #openstack-qa | 16:12 | |
| *** pglass has joined #openstack-qa | 16:14 | |
| *** apoorvad has joined #openstack-qa | 16:14 | |
| openstackgerrit | John L. Villalovos proposed openstack-dev/devstack: Export the 'short_source' function https://review.openstack.org/313132 | 16:16 |
| *** woodster_ has joined #openstack-qa | 16:16 | |
| *** yaume has quit IRC | 16:16 | |
| *** yangyapeng has joined #openstack-qa | 16:17 | |
| *** yaume has joined #openstack-qa | 16:17 | |
| *** yangyapeng has quit IRC | 16:18 | |
| *** minwang2 has quit IRC | 16:19 | |
| *** frontrunner has joined #openstack-qa | 16:21 | |
| frontrunner | on a Compute Node, when should i enable n-vol and when should i enable c-vol? | 16:22 |
| *** minwang2 has joined #openstack-qa | 16:22 | |
| *** e0ne has quit IRC | 16:23 | |
| *** Gen_ has joined #openstack-qa | 16:25 | |
| *** salv-orlando has quit IRC | 16:25 | |
| *** beisner has quit IRC | 16:25 | |
| *** yamamoto has joined #openstack-qa | 16:25 | |
| openstackrecheck | Console logs not available after 13:23s for gate-tempest-dsvm-full-devstack-plugin-ceph 313523,1,f0f6fdc | 16:26 |
| *** hieulq has quit IRC | 16:26 | |
| openstackgerrit | Igor Duarte Cardoso proposed openstack-dev/devstack: Improve residual file cleanup of clean.sh https://review.openstack.org/289965 | 16:27 |
| *** hieulq has joined #openstack-qa | 16:27 | |
| *** mrodden has quit IRC | 16:28 | |
| *** gszasz has quit IRC | 16:28 | |
| *** mtreinish has quit IRC | 16:28 | |
| *** johndperkins has quit IRC | 16:29 | |
| openstackgerrit | Igor Duarte Cardoso proposed openstack-dev/devstack: Properly "stop" horizon when unstacking https://review.openstack.org/290445 | 16:29 |
| *** stevebaker has quit IRC | 16:29 | |
| *** dwalleck has quit IRC | 16:30 | |
| *** yamamoto has quit IRC | 16:30 | |
| *** yaume has quit IRC | 16:32 | |
| *** beisner has joined #openstack-qa | 16:32 | |
| *** SergeyLukjanov has quit IRC | 16:32 | |
| *** SergeyLukjanov has joined #openstack-qa | 16:33 | |
| *** mrodden has joined #openstack-qa | 16:33 | |
| *** stevebaker has joined #openstack-qa | 16:33 | |
| *** mtreinish has joined #openstack-qa | 16:35 | |
| *** ddieterly is now known as ddieterly[away] | 16:37 | |
| *** ddieterly[away] is now known as ddieterly | 16:40 | |
| *** gszasz has joined #openstack-qa | 16:41 | |
| *** madhu_ak has joined #openstack-qa | 16:41 | |
| *** johndperkins has joined #openstack-qa | 16:41 | |
| *** yangyapeng has joined #openstack-qa | 16:42 | |
| *** tosky has quit IRC | 16:43 | |
| *** yfried_ has quit IRC | 16:43 | |
| *** nadya has quit IRC | 16:43 | |
| *** tosky has joined #openstack-qa | 16:45 | |
| *** yangyapeng has quit IRC | 16:47 | |
| *** dimtruck is now known as zz_dimtruck | 16:49 | |
| *** yamamoto has joined #openstack-qa | 16:59 | |
| *** shardy has quit IRC | 17:02 | |
| *** yamamoto has quit IRC | 17:06 | |
| mtreinish | sc68cal, sdague, dtroyer: if you get a sec: https://review.openstack.org/#/q/status:open+topic:install-plugins should be good to go | 17:08 |
| *** mugsie has quit IRC | 17:10 | |
| *** jasonsb has quit IRC | 17:11 | |
| *** yamahata has joined #openstack-qa | 17:12 | |
| *** jasonsb has joined #openstack-qa | 17:12 | |
| *** itsuugo has quit IRC | 17:13 | |
| *** jpena is now known as jpena|off | 17:14 | |
| *** skroni has joined #openstack-qa | 17:15 | |
| *** itsuugo has joined #openstack-qa | 17:15 | |
| *** jasonsb has quit IRC | 17:17 | |
| *** dave-mccowan has joined #openstack-qa | 17:17 | |
| *** weshay has quit IRC | 17:18 | |
| *** ddieterly is now known as ddieterly[away] | 17:19 | |
| openstackrecheck | Console logs not available after 13:27s for gate-tempest-dsvm-neutron-full-mitaka 221020,18,7c64af0 | 17:22 |
| *** lezbar has quit IRC | 17:22 | |
| *** ddieterly[away] is now known as ddieterly | 17:32 | |
| openstackrecheck | Console logs not available after 13:25s for gate-tempest-dsvm-neutron-dvr 220223,19,d3f3b09 | 17:35 |
| *** mugsie has joined #openstack-qa | 17:41 | |
| *** dwalleck has joined #openstack-qa | 17:41 | |
| openstackgerrit | John L. Villalovos proposed openstack-dev/devstack: WIP: Die die die https://review.openstack.org/313651 | 17:42 |
| *** gfidente has quit IRC | 17:44 | |
| *** ddieterly is now known as ddieterly[away] | 17:45 | |
| *** yolanda has joined #openstack-qa | 17:46 | |
| openstackrecheck | Console logs not available after 13:22s for gate-cinder-python27-db 147186,21,6ac15cc | 17:48 |
| openstackgerrit | John L. Villalovos proposed openstack-dev/devstack: Export the 'short_source' function & don't keep PS4 in sudo https://review.openstack.org/313132 | 17:52 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Introduce scope in the auth API https://review.openstack.org/313171 | 17:53 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 17:53 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Add admin role on domain for v3 https://review.openstack.org/226297 | 17:53 |
| *** sambetts is now known as sambetts|afk | 17:53 | |
| *** yolanda has quit IRC | 17:54 | |
| *** ddieterly[away] is now known as ddieterly | 17:54 | |
| slowrie | oomichi: I should have another patch up for subunit-describe-calls today and I'll have a quick follow after that lands to add Request & Response data | 17:54 |
| *** ntata has quit IRC | 17:57 | |
| *** kprabhuv- has quit IRC | 17:57 | |
| *** yarkot1 has quit IRC | 17:57 | |
| *** rbak_ has quit IRC | 17:58 | |
| *** ntata has joined #openstack-qa | 17:59 | |
| *** rbak_ has joined #openstack-qa | 17:59 | |
| *** tongl_ has joined #openstack-qa | 18:01 | |
| *** weshay has joined #openstack-qa | 18:02 | |
| openstackrecheck | Console logs not available after 13:35s for gate-tempest-dsvm-neutron-dvr 313171,4,a63480b | 18:03 |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/313521 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1578866 | 18:04 |
| openstack | Launchpad bug 1578866 in OpenStack Identity (keystone) "test_user_update_own_password failing intermittently" [High,Confirmed] | 18:04 |
| *** ddieterly is now known as ddieterly[away] | 18:04 | |
| *** yarkot1 has joined #openstack-qa | 18:07 | |
| *** lucasagomes is now known as lucas-afk | 18:07 | |
| *** gouthamr has joined #openstack-qa | 18:08 | |
| *** tosky has quit IRC | 18:08 | |
| *** suyash has joined #openstack-qa | 18:08 | |
| *** gouthamr_ has joined #openstack-qa | 18:13 | |
| *** gszasz has quit IRC | 18:14 | |
| *** mriedem1 has joined #openstack-qa | 18:15 | |
| *** tongl_ has quit IRC | 18:16 | |
| *** mriedem has quit IRC | 18:17 | |
| *** gouthamr has quit IRC | 18:17 | |
| *** mriedem1 is now known as mriedem | 18:20 | |
| *** salv-orlando has joined #openstack-qa | 18:24 | |
| *** gouthamr_ is now known as gouthamr | 18:26 | |
| *** nadya has joined #openstack-qa | 18:27 | |
| *** nadya has quit IRC | 18:35 | |
| *** e0ne has joined #openstack-qa | 18:39 | |
| openstackgerrit | Andrew Kerr proposed openstack/tempest: Add tests for Cinder user messages v3 API https://review.openstack.org/300639 | 18:46 |
| *** mkrcmari__ has quit IRC | 18:46 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/tempest: Add a test for cinder multiattach https://review.openstack.org/266605 | 18:52 |
| openstackrecheck | Console logs not available after 13:24s for gate-nova-python27-db 307698,3,a46197c | 19:02 |
| *** ddieterly[away] has quit IRC | 19:04 | |
| *** asselin has joined #openstack-qa | 19:07 | |
| notmyname | if https://review.openstack.org/#/c/312718/ lands (is approved), what's the incentive to write a tempest plugin? | 19:07 |
| notmyname | mtreinish: ^ | 19:07 |
| *** ifarkas has quit IRC | 19:08 | |
| *** dave-mccowan has quit IRC | 19:09 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/tempest: Add a test for cinder multiattach https://review.openstack.org/266605 | 19:10 |
| *** dwalleck has quit IRC | 19:11 | |
| *** e0ne has quit IRC | 19:11 | |
| mtreinish | dims: if you get a sec: https://review.openstack.org/312172 trying to start use only 1 http client in tempest | 19:11 |
| *** ddieterly has joined #openstack-qa | 19:12 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/tempest: Add a test for cinder multiattach https://review.openstack.org/266605 | 19:13 |
| dims | mtreinish : ack will look in a bit | 19:14 |
| mtreinish | dims: cool, thanks | 19:14 |
| openstackrecheck | Console logs not available after 13:25s for gate-tempest-dsvm-postgres-full 272118,24,c655ba8 | 19:16 |
| openstackgerrit | Rob Crittenden proposed openstack-dev/devstack: Add time_elapsed to timekeeping functions https://review.openstack.org/313673 | 19:17 |
| *** cody-somerville has joined #openstack-qa | 19:22 | |
| *** madhu_ak has quit IRC | 19:22 | |
| *** dave-mccowan has joined #openstack-qa | 19:25 | |
| *** rcernin has joined #openstack-qa | 19:26 | |
| openstackrecheck | Console logs not available after 13:31s for gate-tempest-dsvm-neutron-full 313654,1,cab3b78 | 19:30 |
| *** mkrcmari__ has joined #openstack-qa | 19:31 | |
| *** yangyapeng has joined #openstack-qa | 19:31 | |
| *** madhu_ak has joined #openstack-qa | 19:33 | |
| *** yangyapeng has quit IRC | 19:35 | |
| mtreinish | timothyb89: so I'm working on integrating the elastic-recheck data into o-h | 19:37 |
| mtreinish | I forgot how slow it is | 19:37 |
| mtreinish | like way too slow to do dynamically | 19:37 |
| mtreinish | I'm not sure how to best integrate it now, but it's still going at >3min | 19:39 |
| *** salv-orlando has quit IRC | 19:39 | |
| mtreinish | and that's with limiting it to looking at the last hour only | 19:40 |
| *** apoorvad has quit IRC | 19:40 | |
| *** salv-orlando has joined #openstack-qa | 19:42 | |
| mtreinish | timothyb89: it took 4min 27sec, but worked well: http://paste.openstack.org/show/496381/ | 19:43 |
| openstackrecheck | Console logs not available after 13:26s for gate-tempest-dsvm-postgres-full 313171,5,2e6255d | 19:43 |
| mtreinish | granted only 1 match found, but that's something | 19:44 |
| *** cody-somerville has quit IRC | 19:46 | |
| openstackgerrit | Matthew Treinish proposed openstack/openstack-health: WIP: Add elastic-recheck data querying https://review.openstack.org/311919 | 19:46 |
| mtreinish | david-lyle: do you want to push the tempest patch to remove the horizon test (with a depends on to your project-config change) | 19:51 |
| david-lyle | mtreinish: I only put the test as non-voting on check to make sure I had the mechanics worked out | 19:52 |
| david-lyle | once I'm confident in that, I'll change it to check/gate and make it voting | 19:52 |
| david-lyle | the patch you describe would depend on that one | 19:53 |
| mtreinish | david-lyle: that's good enough for me to remove it, but if you want to hold off thats fine | 19:53 |
| * david-lyle thinking | 19:53 | |
| mtreinish | I just wanted to make sure you get dibs on the tempest removal | 19:53 |
| david-lyle | as long as the check job passes, I'm fine with removing | 19:54 |
| david-lyle | I can upload the patch and mark WIP until I have some confirmation I didn't make a mess | 19:54 |
| mtreinish | david-lyle: sure, that works | 19:54 |
| david-lyle | ok, will put up shortly | 19:54 |
| david-lyle | mtreinish: should I just deprecate the config options? | 19:55 |
| david-lyle | related to dashboard | 19:55 |
| mtreinish | normally we just remove them, there's nothing using them | 19:55 |
| mtreinish | it's not exactly friendly, but no one has complained about it before :) | 19:55 |
| david-lyle | ok, didn't want to get hasty | 19:55 |
| david-lyle | but I will happily remove more code :) | 19:55 |
| david-lyle | I'll add a releasenote item too | 19:56 |
| *** gszasz has joined #openstack-qa | 19:57 | |
| mtreinish | oh, good. I've been bad about the reno stuff in tempest | 19:57 |
| openstackrecheck | Console logs not available after 13:27s for gate-grenade-dsvm-multinode 221020,19,1016ec2 | 19:57 |
| mtreinish | timothyb89: after waiting ~4min: http://i.imgur.com/d2qZXnO.png | 19:59 |
| mtreinish | the formatting still needs some work (the patch is still wip) | 19:59 |
| mtreinish | but that's the basic idea | 19:59 |
| mtreinish | but I don't think we can move forward there if it's this slow | 19:59 |
| *** pauloewerton has quit IRC | 19:59 | |
| openstackgerrit | Stephen Lowrie proposed openstack/os-testr: Add subunit-describe-calls https://review.openstack.org/312275 | 20:07 |
| slowrie | mtreinish: oomichi: ^^ | 20:08 |
| *** mvk_ has joined #openstack-qa | 20:08 | |
| mtreinish | slowrie: https://wiki.openstack.org/wiki/GitCommitMessages | 20:09 |
| openstackrecheck | Console logs not available after 13:26s for gate-tempest-dsvm-neutron-dvr 226297,14,7d4909b | 20:10 |
| mtreinish | slowrie: it needs a real commit message, especially for something adding a new command | 20:11 |
| slowrie | mtreinish: Ok, fixing the docs real quick and then I'll push it back up with a full message | 20:11 |
| mtreinish | slowrie: ok, thanks | 20:11 |
| *** mkrcmari__ has quit IRC | 20:12 | |
| *** salv-orlando has quit IRC | 20:13 | |
| *** apoorvad has joined #openstack-qa | 20:13 | |
| *** aysyd has quit IRC | 20:14 | |
| timothyb89 | mtreinish: huh, that looks pretty cool ... I'll take a look and see if there might be some places we can save time | 20:16 |
| mtreinish | asselin, mriedem: what do you think about publishing elastic-recheck on pypi? | 20:16 |
| mtreinish | I'm working on e-r support for openstack health, and right now it's all installed via git | 20:16 |
| mtreinish | I'm not sure it solves any problems though | 20:16 |
| mtreinish | just thinking out loud | 20:16 |
| mtreinish | timothyb89: well right now it's looping over all 10 failures serially we could make that happen in parallel | 20:17 |
| *** jecarey has quit IRC | 20:17 | |
| mtreinish | timothyb89: but the bottleneck is really elastic search | 20:17 |
| mriedem | meh | 20:17 |
| openstackgerrit | Stephen Lowrie proposed openstack/os-testr: Add subunit-describe-calls https://review.openstack.org/312275 | 20:17 |
| mtreinish | timothyb89: oh, and I just realized I broke the details view (when you expand the test run) | 20:18 |
| mtreinish | but it's still wip :) | 20:18 |
| mtreinish | mriedem: yeah, I'm leaning more towards that now too | 20:18 |
| mtreinish | because packaging the queries is weird (and openstack-health will need those too) | 20:18 |
| *** cody-somerville has joined #openstack-qa | 20:19 | |
| asselin | I think it would be nice to do that. | 20:20 |
| asselin | but we'd still | 20:20 |
| asselin | still have to use our local fork. Queries and web stuff would be nice to isolate and make more reusable | 20:21 |
| mtreinish | asselin: right, data files in python packages are a mess, so I'd avoid doing that | 20:21 |
| mtreinish | it would just be for the python code | 20:21 |
| mtreinish | asselin: just fyi: https://review.openstack.org/#/c/311919/ | 20:22 |
| openstackrecheck | Console logs not available after 13:25s for gate-cinder-python34-db 280492,8,9723e02 | 20:24 |
| *** e0ne has joined #openstack-qa | 20:29 | |
| *** openstackgerrit has quit IRC | 20:32 | |
| *** openstackgerrit has joined #openstack-qa | 20:33 | |
| *** ddieterly is now known as ddieterly[away] | 20:34 | |
| *** rfolco has quit IRC | 20:35 | |
| *** jecarey has joined #openstack-qa | 20:35 | |
| openstackrecheck | Console logs not available after 13:23s for gate-tempest-dsvm-neutron-full 309192,4,3cff330 | 20:37 |
| *** cody-somerville has quit IRC | 20:38 | |
| *** e0ne has quit IRC | 20:39 | |
| *** jecarey has quit IRC | 20:39 | |
| *** e0ne has joined #openstack-qa | 20:40 | |
| *** e0ne has quit IRC | 20:40 | |
| asselin | mtreinish, thanks for the reviews. I will try to make an effort to resync our local changes. Moving the queries out to a different repo would help that too. | 20:40 |
| asselin | elastic-recheck-queries | 20:40 |
| *** aysyd has joined #openstack-qa | 20:44 | |
| *** porrua has quit IRC | 20:47 | |
| openstackgerrit | Merged openstack-dev/devstack: Adjust region name for images, networks and volumes during deployment https://review.openstack.org/310431 | 20:48 |
| openstackgerrit | Merged openstack-dev/devstack: Update to bashate 0.5.1 https://review.openstack.org/310122 | 20:49 |
| openstackgerrit | Merged openstack-dev/devstack: Suppressed echoing of the line. https://review.openstack.org/311513 | 20:49 |
| openstackgerrit | Merged openstack-dev/devstack: Added fix to install zookeeper for fedora https://review.openstack.org/312461 | 20:49 |
| openstackgerrit | Merged openstack-dev/devstack: removed dead link to stack.sh.html https://review.openstack.org/312728 | 20:50 |
| openstackgerrit | Merged openstack-dev/devstack: Update auth params in Nova Hypervisor-Ironic https://review.openstack.org/300151 | 20:50 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Introduce scope in the auth API https://review.openstack.org/313171 | 20:52 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 20:52 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Add admin role on domain for v3 https://review.openstack.org/226297 | 20:52 |
| *** ddieterly[away] is now known as ddieterly | 20:58 | |
| *** ddieterly is now known as ddieterly[away] | 20:59 | |
| *** pcrews__ has joined #openstack-qa | 20:59 | |
| openstackgerrit | Merged openstack/tempest: Add option to tempest init to show global conf dir https://review.openstack.org/309732 | 21:00 |
| *** raildo is now known as raildo-afk | 21:01 | |
| mtreinish | asselin: I wasn't thinking about moving the queries to another repo (there's a lot of overhead with a move like that) | 21:01 |
| mtreinish | but if we're thinking baout packaging it for real, we need to make the split more obvious | 21:02 |
| mtreinish | and publish the queries in a different format | 21:02 |
| *** pcrews_ has quit IRC | 21:02 | |
| *** ddieterly[away] is now known as ddieterly | 21:02 | |
| asselin | mtreinish, It's not that difficult. It's all in one folder, we can use git sub-tree to do it. We have the process documented in an infra spec. | 21:03 |
| asselin | but it would require some updates to e.g. testing & puppet | 21:03 |
| openstackgerrit | Ryan Tidwell proposed openstack/tempest: Ensure tempest tests don't assume IP address allocation strategy https://review.openstack.org/312771 | 21:04 |
| asselin | it would definitely make the repo cleaner | 21:04 |
| openstackgerrit | Stephen Lowrie proposed openstack/tempest: Centralized Workspaces https://review.openstack.org/283770 | 21:05 |
| *** ddieterly is now known as ddieterly[away] | 21:08 | |
| *** ddieterly[away] is now known as ddieterly | 21:12 | |
| *** skroni has quit IRC | 21:12 | |
| *** regXboi has quit IRC | 21:21 | |
| mminesh | trying to run git review for the first time and getting "fatal: ICLA contributor agreement requires current contact information. | 21:24 |
| mminesh | Please review your contact information: | 21:24 |
| mminesh | https://review.openstack.org/#/settings/contact" | 21:24 |
| mminesh | I have tried updating the info on the link but getting server error.... | 21:24 |
| mminesh | any help? | 21:24 |
| *** aysyd has quit IRC | 21:24 | |
| *** skroni has joined #openstack-qa | 21:26 | |
| *** skroni has quit IRC | 21:26 | |
| *** skroni has joined #openstack-qa | 21:27 | |
| *** ddieterly has quit IRC | 21:28 | |
| *** skroni has quit IRC | 21:28 | |
| *** leakypipes has quit IRC | 21:35 | |
| *** Gen_ has quit IRC | 21:36 | |
| openstackrecheck | Console logs not available after 13:22s for gate-nova-pep8 312193,2,2f5346b | 21:37 |
| *** sigmavirus24 is now known as sigmavirus24_awa | 21:38 | |
| andreaf | mminesh: welcome :) you may have more luck in the openstack-infra room, they run gerrit so they may be able to help | 21:38 |
| mminesh | thanks andreaf. | 21:38 |
| *** mriedem has quit IRC | 21:44 | |
| *** gszasz has quit IRC | 21:46 | |
| openstackrecheck | Console logs not available after 13:24s for gate-tempest-dsvm-full 156923,39,4643e83 | 21:50 |
| *** salv-orlando has joined #openstack-qa | 21:51 | |
| *** davideagnello has quit IRC | 21:53 | |
| *** pglbutt has joined #openstack-qa | 22:01 | |
| openstackgerrit | Ryan Tidwell proposed openstack/tempest: Ensure tempest tests don't assume IP address allocation strategy https://review.openstack.org/312771 | 22:02 |
| openstackrecheck | Console logs not available after 13:23s for gate-tempest-dsvm-postgres-full 188321,24,de4be3c | 22:03 |
| *** pglass has quit IRC | 22:04 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/284421 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1577558 | 22:04 |
| openstack | Launchpad bug 1577558 in OpenStack Security Advisory "v2.0 fernet tokens audit ids are inconsistent" [Undecided,Incomplete] | 22:04 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Introduce scope in the auth API https://review.openstack.org/313171 | 22:14 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 22:14 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Add admin role on domain for v3 https://review.openstack.org/226297 | 22:14 |
| openstackgerrit | Victor Morales proposed openstack/tempest: Fix tempest_tests.plugin no module error https://review.openstack.org/313710 | 22:15 |
| openstackgerrit | David Lyle proposed openstack/tempest: Remove integrated dashboard tests https://review.openstack.org/313713 | 22:15 |
| *** drptbl has joined #openstack-qa | 22:15 | |
| openstackgerrit | David Lyle proposed openstack/tempest: Remove integrated dashboard tests https://review.openstack.org/313713 | 22:17 |
| openstackrecheck | Console logs not available after 13:22s for gate-nova-pep8 303345,47,c50e6bc | 22:18 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr: Updated from global requirements https://review.openstack.org/313749 | 22:21 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/tempest-lib: Updated from global requirements https://review.openstack.org/305246 | 22:23 |
| *** drptbl_ has joined #openstack-qa | 22:24 | |
| *** drptbl has quit IRC | 22:27 | |
| openstackrecheck | Console logs not available after 13:25s for gate-tempest-dsvm-neutron-dvr 226297,15,0c91067 | 22:31 |
| *** david-lyle has quit IRC | 22:35 | |
| *** david-lyle has joined #openstack-qa | 22:37 | |
| openstackgerrit | David Lyle proposed openstack/tempest: Remove integrated dashboard tests https://review.openstack.org/313713 | 22:38 |
| *** pglbutt has quit IRC | 22:38 | |
| *** cdent has quit IRC | 22:44 | |
| openstackrecheck | Console logs not available after 13:22s for gate-tempest-dsvm-full 313682,2,c4a66b3 | 22:45 |
| *** vhosakot has quit IRC | 22:45 | |
| *** markvoelker has quit IRC | 22:47 | |
| *** markvoelker has joined #openstack-qa | 22:48 | |
| openstackgerrit | Matthew Treinish proposed openstack/openstack-health: WIP: Add elastic-recheck data querying https://review.openstack.org/311919 | 22:53 |
| mtreinish | timothyb89: ^^^ made the elastic search part run in parallel and it takes ~20secs now | 22:54 |
| mtreinish | still not great, but probably liveable. Definitely better than 4min | 22:54 |
| kevinbenton | hi, any devstack experts around? The recent removal of VPNaaS from the devstack code has partially broken heat's grendade job | 22:55 |
| mtreinish | kevinbenton: just blame sc68cal he knew it was gonna break someone :) | 22:56 |
| kevinbenton | :) | 22:56 |
| mtreinish | kevinbenton: do you have a link? | 22:56 |
| kevinbenton | mtreinish: no link. everything is in my head at this point :) | 22:56 |
| mtreinish | kevinbenton: but it's late on fri, so my attention is waning | 22:56 |
| mtreinish | kevinbenton: heh, ok | 22:56 |
| kevinbenton | mtreinish: so this is the bug | 22:57 |
| kevinbenton | mtreinish: https://bugs.launchpad.net/neutron/+bug/1579235 | 22:57 |
| openstack | Launchpad bug 1579235 in neutron "FloatingIP is at status: ACTIVE. failed to reach status: DOWN" [Undecided,New] | 22:57 |
| kevinbenton | mtreinish: but that won't help you much | 22:57 |
| kevinbenton | mtreinish: it ultimately winds down to VPNaaS is leaving an agent running in the grenade upgrade | 22:57 |
| kevinbenton | mtreinish: because nothing tells it to stop anymore | 22:57 |
| kevinbenton | mtreinish: so then there are two agents running and they interfere with each other | 22:57 |
| mtreinish | kevinbenton: oh, are we starting vpnaas on the old side? | 22:57 |
| kevinbenton | mtreinish: yes, and on the new side | 22:58 |
| kevinbenton | mtreinish: but in between the grenade logic to tell services to stop | 22:58 |
| kevinbenton | mtreinish: calls functions in devstack that never communicate this info to plugins | 22:58 |
| kevinbenton | mtreinish: so the vpnaas plugin doesn't get the signal to stop its agent | 22:58 |
| *** fnaval_ is now known as fnaval | 22:59 | |
| kevinbenton | mtreinish: https://github.com/openstack-dev/grenade/blob/master/projects/50_neutron/shutdown.sh#L19-L20 | 22:59 |
| kevinbenton | mtreinish: neither of those result in calls to devstack plugins | 23:00 |
| mtreinish | kevinbenton: that's because shutdown.sh is per project/plugin, if vpnaas has moved to a devstack plugin it'll also need a grenade plugin | 23:02 |
| mtreinish | we probably should backport the vpnaas removal to mitaka too, to prevent it from starting without master support | 23:03 |
| mtreinish | and if we want it enabled use the grenade plugin | 23:03 |
| mtreinish | (after it's added of course) | 23:03 |
| *** yamahata has quit IRC | 23:04 | |
| mtreinish | kevinbenton: why isn't this an issue everywhere? The normal multinode grenade jobs seem to be fine | 23:04 |
| *** yangyapeng has joined #openstack-qa | 23:07 | |
| openstackgerrit | Matthew Treinish proposed openstack-dev/devstack: Remove vpnaas code from devstack https://review.openstack.org/313792 | 23:10 |
| mtreinish | kevinbenton: ^^^ see if that fixes things for you | 23:11 |
| *** yangyapeng has quit IRC | 23:12 | |
| mtreinish | although, I'm not sure if that's a good idea (or if it'll be landable) | 23:12 |
| mtreinish | but I'm pretty sure the fix here is to stop starting vpnaas on mitaka devstack in the grenade runs | 23:12 |
| *** ddieterly has joined #openstack-qa | 23:16 | |
| mtreinish | timothyb89: I'm playing a bit more with it, I think 20sec might be ok. Most of the use loads fast, and just the test failed form last 10 runs takes the extra time | 23:20 |
| kevinbenton | mtreinish: sorry about the delay | 23:20 |
| kevinbenton | mtreinish: i don't think that will fix it. the problem with a devstack plugin is that nothing calls it to shutdown on an upgrade | 23:20 |
| kevinbenton | mtreinish: the reason we don't hit this right now in our main grenade job is because we don't use vpnaas there | 23:21 |
| mtreinish | kevinbenton: right, they need to add the grenade plugin so grenade knows that vpnaas is being installed | 23:21 |
| mtreinish | because it's a plugin on master | 23:22 |
| mtreinish | timothyb89: we might be able to do better on the speed though, I'll need to do some more tests to see how long each es query is actually taking | 23:22 |
| kevinbenton | mtreinish: the other option is to stop using vpnaas in the heat jobs | 23:23 |
| mtreinish | kevinbenton: at a min we need to add a grenade plugin to vpnaas so it knows how to shut it down | 23:23 |
| mtreinish | kevinbenton: yeah, that's more or less what I was trying to do by removing it from mitaka :) | 23:23 |
| kevinbenton | mtreinish: well they are still using it for master as well | 23:24 |
| kevinbenton | mtreinish: let me see if heat has vpn resources it tests | 23:24 |
| *** ddieterly is now known as ddieterly[away] | 23:24 | |
| mtreinish | kevinbenton: right, so if it's needed for the heat jobs, you need to add the upgrade dir (ie grenade plugin) to vpnaas and add using that to the job config | 23:25 |
| kevinbenton | mtreinish: yep, looks like they do have a vpnresource hook so i can't pull the rug from under them. | 23:25 |
| mtreinish | heat already does grenade via it's own plugin so you can follow that as an example | 23:25 |
| kevinbenton | mtreinish: i'll cook up a plugin for vpnaas | 23:25 |
| kevinbenton | mtreinish: so can i have the devstack plugin also configure the grenade plugin inside of neutron-vpnaas? | 23:26 |
| mtreinish | if you add the plugin to tell grenade how to shut things down and register it for the job that should fix the issue | 23:26 |
| kevinbenton | mtreinish: or does it explicitly have to be added to every job as well | 23:26 |
| mtreinish | kevinbenton: http://docs.openstack.org/developer/grenade/plugins.html#out-of-tree-plugins | 23:27 |
| mtreinish | I'm not sure that answers your question though | 23:27 |
| mtreinish | are you talk about in project config? or something else | 23:27 |
| kevinbenton | mtreinish: i was wondering if we could save some effort of having to define enable_grenade_plugin for everyone that uses vpnaas | 23:28 |
| kevinbenton | mtreinish: by putting that login in the devstack plugin in vpnaas | 23:28 |
| mtreinish | I'm not sure it works that way, but tbh I haven't played too much with it | 23:28 |
| kevinbenton | mtreinish: ok. thx for the pointer | 23:29 |
| mtreinish | I think you'll have an ordering problem, the registration call has to be set early enough to be picked up | 23:29 |
| mtreinish | kevinbenton: https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L200 | 23:30 |
| mtreinish | which happens before the old side devstack is run | 23:30 |
| mtreinish | so I don't think you can | 23:30 |
| kevinbenton | ah | 23:30 |
| *** edmondsw has quit IRC | 23:34 | |
| *** akshai has quit IRC | 23:35 | |
| *** rossella_s has quit IRC | 23:40 | |
| *** rossella_s has joined #openstack-qa | 23:41 | |
| *** woodster_ has quit IRC | 23:48 | |
| *** cody-somerville has joined #openstack-qa | 23:49 | |
| *** salv-orlando has quit IRC | 23:56 | |
| *** akshai has joined #openstack-qa | 23:57 | |
| *** akshai_ has joined #openstack-qa | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
