openstackgerrit | Jamie Lennox proposed openstack-dev/devstack: Allow putting service users in a seperate domain https://review.openstack.org/271051 | 00:00 |
---|---|---|
*** vhosakot has quit IRC | 00:03 | |
*** ober37_ has joined #openstack-qa | 00:05 | |
*** salv-orlando has quit IRC | 00:06 | |
*** ober37 has quit IRC | 00:07 | |
*** mylu has quit IRC | 00:07 | |
*** mylu has joined #openstack-qa | 00:13 | |
*** outofmemory is now known as reedip | 00:17 | |
*** ober37 has joined #openstack-qa | 00:18 | |
*** ober37_ has quit IRC | 00:18 | |
*** mylu has quit IRC | 00:19 | |
*** mylu has joined #openstack-qa | 00:21 | |
*** nwkarste_ has quit IRC | 00:24 | |
*** Jae00 has quit IRC | 00:25 | |
*** ober37 has quit IRC | 00:28 | |
*** ober37 has joined #openstack-qa | 00:28 | |
*** ober37 has quit IRC | 00:33 | |
*** ober37 has joined #openstack-qa | 00:34 | |
openstackgerrit | zhufl proposed openstack/tempest: Set timeout value in urllib3.poolmanager.PoolManager https://review.openstack.org/294425 | 00:38 |
*** vhosakot has joined #openstack-qa | 00:40 | |
*** lekha has quit IRC | 00:41 | |
*** mylu has quit IRC | 00:44 | |
*** ober37 has quit IRC | 00:44 | |
*** ober37 has joined #openstack-qa | 00:46 | |
*** mylu has joined #openstack-qa | 00:47 | |
*** vhosakot has quit IRC | 01:00 | |
*** ober37 has quit IRC | 01:04 | |
*** yangyape_ has joined #openstack-qa | 01:06 | |
*** ober37 has joined #openstack-qa | 01:14 | |
*** ober37_ has joined #openstack-qa | 01:17 | |
*** ober37 has quit IRC | 01:18 | |
*** vhosakot has joined #openstack-qa | 01:22 | |
*** ober37_ has quit IRC | 01:22 | |
*** ober37 has joined #openstack-qa | 01:22 | |
*** yangyape_ has quit IRC | 01:23 | |
*** nwkarsten has joined #openstack-qa | 01:36 | |
*** mriedem has quit IRC | 01:39 | |
*** lekha has joined #openstack-qa | 01:41 | |
*** nwkarsten has quit IRC | 01:41 | |
openstackgerrit | Merged openstack-dev/devstack: Updated from generate-devstack-plugins-list https://review.openstack.org/306700 | 01:42 |
*** ober37 has quit IRC | 01:54 | |
*** ober37 has joined #openstack-qa | 01:54 | |
*** ober37 has quit IRC | 01:59 | |
*** dkranz_ has quit IRC | 02:05 | |
*** r-mibu has quit IRC | 02:09 | |
*** mylu has quit IRC | 02:10 | |
*** nwkarsten has joined #openstack-qa | 02:13 | |
*** r-mibu has joined #openstack-qa | 02:14 | |
*** nwkarsten has quit IRC | 02:17 | |
*** alex_xu has quit IRC | 02:18 | |
*** alex_xu has joined #openstack-qa | 02:22 | |
openstackgerrit | Merged openstack-dev/devstack: Be a bit more explicit about the plugin generation https://review.openstack.org/301397 | 02:26 |
openstackgerrit | Merged openstack-dev/devstack: Fix Q_PLUGIN_EXTRA_CONF_FILES usage comment https://review.openstack.org/306541 | 02:34 |
*** nadya has joined #openstack-qa | 02:39 | |
*** nadya has quit IRC | 02:43 | |
*** dimtruck is now known as zz_dimtruck | 02:52 | |
*** yamamoto_ has joined #openstack-qa | 02:59 | |
*** amotoki has joined #openstack-qa | 03:11 | |
*** yamamoto_ has quit IRC | 03:11 | |
*** yamamoto_ has joined #openstack-qa | 03:21 | |
*** binbincong has quit IRC | 03:25 | |
*** tochi has quit IRC | 03:26 | |
*** binbincong has joined #openstack-qa | 03:38 | |
openstackgerrit | Merged openstack-dev/devstack: Move nova's api_paste_config into wsgi group https://review.openstack.org/303786 | 03:38 |
*** yamamoto_ has quit IRC | 03:43 | |
*** sree has joined #openstack-qa | 03:49 | |
*** tochi has joined #openstack-qa | 03:56 | |
*** zackf has quit IRC | 04:05 | |
*** salv-orlando has joined #openstack-qa | 04:09 | |
*** vhosakot has quit IRC | 04:10 | |
*** amotoki has quit IRC | 04:11 | |
*** salv-orlando has quit IRC | 04:13 | |
*** itzikb has joined #openstack-qa | 04:15 | |
*** amotoki has joined #openstack-qa | 04:22 | |
*** yamamoto_ has joined #openstack-qa | 04:27 | |
*** amotoki has quit IRC | 04:30 | |
*** sree has quit IRC | 04:37 | |
*** amotoki has joined #openstack-qa | 04:38 | |
*** salv-orlando has joined #openstack-qa | 04:48 | |
*** Poornima has joined #openstack-qa | 04:54 | |
*** ober37 has joined #openstack-qa | 04:56 | |
*** ober37 has quit IRC | 05:00 | |
*** amotoki has quit IRC | 05:01 | |
*** mylu has joined #openstack-qa | 05:03 | |
*** rcernin has joined #openstack-qa | 05:04 | |
*** Alex_Stef has joined #openstack-qa | 05:07 | |
*** amotoki has joined #openstack-qa | 05:09 | |
*** shausy has joined #openstack-qa | 05:11 | |
*** yangyape_ has joined #openstack-qa | 05:11 | |
*** mylu has quit IRC | 05:11 | |
*** VanL has quit IRC | 05:12 | |
*** VanL has joined #openstack-qa | 05:12 | |
*** e0ne has joined #openstack-qa | 05:15 | |
*** e0ne has quit IRC | 05:18 | |
*** ramishra has joined #openstack-qa | 05:21 | |
*** shausy has quit IRC | 05:23 | |
*** mylu has joined #openstack-qa | 05:24 | |
*** shausy has joined #openstack-qa | 05:26 | |
*** bkopilov_ has joined #openstack-qa | 05:27 | |
*** ramishra has quit IRC | 05:28 | |
*** mylu has quit IRC | 05:29 | |
*** ramishra has joined #openstack-qa | 05:30 | |
openstackgerrit | Masayuki Igawa proposed openstack-infra/subunit2sql: Capitalize first characters in param docs https://review.openstack.org/306908 | 05:30 |
*** tpatzig_ has quit IRC | 05:35 | |
*** zackf has joined #openstack-qa | 05:36 | |
*** mkoderer has quit IRC | 05:36 | |
*** mkoderer has joined #openstack-qa | 05:39 | |
*** zackf has quit IRC | 05:42 | |
*** davideagnello has joined #openstack-qa | 05:49 | |
*** davideagnello has quit IRC | 05:51 | |
*** ramishra has quit IRC | 05:52 | |
*** ramishra has joined #openstack-qa | 05:55 | |
*** ober37 has joined #openstack-qa | 05:57 | |
*** itzikb has quit IRC | 05:58 | |
*** tpatzig has joined #openstack-qa | 05:59 | |
*** ober37 has quit IRC | 06:01 | |
*** fedexo has joined #openstack-qa | 06:11 | |
*** Poornima has quit IRC | 06:16 | |
*** cody-somerville has quit IRC | 06:17 | |
*** apevec has joined #openstack-qa | 06:18 | |
*** apevec has quit IRC | 06:18 | |
*** apevec has joined #openstack-qa | 06:18 | |
*** apevec has left #openstack-qa | 06:18 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Fix rest_client's expected_success for non int status https://review.openstack.org/306919 | 06:22 |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Fix rest_client's expected_success for non int status https://review.openstack.org/306919 | 06:23 |
*** sridharg has joined #openstack-qa | 06:31 | |
*** alex_xu has quit IRC | 06:40 | |
*** eliqiao has quit IRC | 06:40 | |
*** itzikb has joined #openstack-qa | 06:40 | |
*** alex_xu has joined #openstack-qa | 06:41 | |
*** eliqiao has joined #openstack-qa | 06:42 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack-dev/devstack: Updated from generate-devstack-plugins-list https://review.openstack.org/306932 | 06:53 |
*** neeti has joined #openstack-qa | 06:56 | |
*** ttx_ is now known as ttx | 07:06 | |
*** abehl has joined #openstack-qa | 07:14 | |
*** dtantsur|afk is now known as dtantsur | 07:14 | |
*** pcaruana has joined #openstack-qa | 07:15 | |
*** e0ne has joined #openstack-qa | 07:16 | |
openstackgerrit | Masayuki Igawa proposed openstack/os-testr: Fix docs typos https://review.openstack.org/306945 | 07:22 |
*** Alex_Stef has quit IRC | 07:29 | |
*** sridharg has quit IRC | 07:30 | |
*** ifarkas has joined #openstack-qa | 07:32 | |
*** jlanoux has joined #openstack-qa | 07:36 | |
*** lezbar has quit IRC | 07:39 | |
*** sridharg has joined #openstack-qa | 07:41 | |
*** nadya has joined #openstack-qa | 07:48 | |
*** rossella_s has joined #openstack-qa | 07:48 | |
*** alex_xu has quit IRC | 07:49 | |
*** alex_xu has joined #openstack-qa | 07:51 | |
*** vgridnev has joined #openstack-qa | 07:54 | |
*** vgridnev has quit IRC | 07:56 | |
*** bdperkin has joined #openstack-qa | 07:57 | |
*** bdperkin has joined #openstack-qa | 07:57 | |
*** ober37 has joined #openstack-qa | 07:59 | |
*** pilgrimstack has joined #openstack-qa | 08:01 | |
*** ober37 has quit IRC | 08:03 | |
*** kbaikov has quit IRC | 08:04 | |
*** shardy has joined #openstack-qa | 08:06 | |
openstackgerrit | Masayuki Igawa proposed openstack/qa-specs: Remove tempest run from cli improvements https://review.openstack.org/306985 | 08:10 |
*** sshnaidm has quit IRC | 08:12 | |
*** fedexo has quit IRC | 08:12 | |
*** lezbar has joined #openstack-qa | 08:14 | |
*** Serlex has joined #openstack-qa | 08:18 | |
*** kbaikov has joined #openstack-qa | 08:22 | |
*** drptbl has joined #openstack-qa | 08:27 | |
*** gfidente has joined #openstack-qa | 08:31 | |
openstackgerrit | Dong Ma proposed openstack-infra/subunit2sql: Add db api tests for subunit2sql (1/3) https://review.openstack.org/305030 | 08:37 |
*** mpaolino has joined #openstack-qa | 08:41 | |
*** jpena|off is now known as jpena | 08:49 | |
*** cdent has joined #openstack-qa | 08:51 | |
*** sridharg has quit IRC | 08:53 | |
*** zackf has joined #openstack-qa | 08:58 | |
*** sridharg has joined #openstack-qa | 08:59 | |
*** sridharg has quit IRC | 08:59 | |
*** yamamoto_ has quit IRC | 09:02 | |
*** yamamoto has joined #openstack-qa | 09:02 | |
*** zackf has quit IRC | 09:03 | |
*** sridharg has joined #openstack-qa | 09:03 | |
*** tesseract has joined #openstack-qa | 09:03 | |
*** tesseract is now known as Guest30394 | 09:03 | |
*** Deng has joined #openstack-qa | 09:07 | |
*** drptbl_ has joined #openstack-qa | 09:08 | |
*** drptbl has quit IRC | 09:09 | |
gmann | oomichi: mtreinish: hope this is fine w.r.t.backward compatibility - https://review.openstack.org/#/c/306919/2 | 09:10 |
gmann | oomichi: mtreinish: it will fail for users using that method wrongly. | 09:10 |
gmann | with current code it pass falsely | 09:11 |
*** neeti has quit IRC | 09:13 | |
*** neeti has joined #openstack-qa | 09:24 | |
*** sshnaidm has joined #openstack-qa | 09:29 | |
*** neeti has quit IRC | 09:41 | |
*** neeti has joined #openstack-qa | 09:41 | |
*** cdent has quit IRC | 09:44 | |
*** mpaolino has quit IRC | 09:49 | |
*** mpaolino has joined #openstack-qa | 09:50 | |
*** tosky has joined #openstack-qa | 09:57 | |
*** Alex_Stef has joined #openstack-qa | 10:03 | |
*** salv-orl_ has joined #openstack-qa | 10:04 | |
*** gszasz has joined #openstack-qa | 10:06 | |
*** salv-orlando has quit IRC | 10:07 | |
*** yangyap__ has joined #openstack-qa | 10:09 | |
*** tpatzig has left #openstack-qa | 10:10 | |
*** yangyape_ has quit IRC | 10:13 | |
*** ober37 has joined #openstack-qa | 10:13 | |
*** yangyap__ has quit IRC | 10:13 | |
*** cdent has joined #openstack-qa | 10:14 | |
*** sdague has joined #openstack-qa | 10:15 | |
*** ober37 has quit IRC | 10:20 | |
*** neeti_ has joined #openstack-qa | 10:32 | |
*** neeti has quit IRC | 10:33 | |
openstackgerrit | Benny Kopilov proposed openstack/tempest: Cinder storage pools tests https://review.openstack.org/307078 | 10:33 |
*** bkopilov_ has quit IRC | 10:40 | |
*** bkopilov_ has joined #openstack-qa | 10:40 | |
*** jecarey has joined #openstack-qa | 11:00 | |
*** SlickN1k has joined #openstack-qa | 11:01 | |
*** marcusvrn__ has joined #openstack-qa | 11:02 | |
*** zhiyan_ has joined #openstack-qa | 11:02 | |
*** beekneemech has joined #openstack-qa | 11:04 | |
*** krotscheck_ has joined #openstack-qa | 11:04 | |
*** jgriffith_ has joined #openstack-qa | 11:04 | |
*** lbragstad_ has joined #openstack-qa | 11:04 | |
*** rodrigods has quit IRC | 11:04 | |
*** rodrigods has joined #openstack-qa | 11:05 | |
*** dmellado_ has joined #openstack-qa | 11:05 | |
*** dalees` has joined #openstack-qa | 11:05 | |
*** marcusvrn_ has quit IRC | 11:05 | |
*** melwitt has quit IRC | 11:05 | |
*** SlickNik has quit IRC | 11:05 | |
*** jgriffith has quit IRC | 11:05 | |
*** dtroyer has quit IRC | 11:05 | |
*** lbragstad has quit IRC | 11:05 | |
*** anthonyper has quit IRC | 11:05 | |
*** dalees has quit IRC | 11:06 | |
*** kinrui has quit IRC | 11:06 | |
*** zhiyan has quit IRC | 11:06 | |
*** harlowja has quit IRC | 11:06 | |
*** krotscheck has quit IRC | 11:06 | |
*** dtantsur has quit IRC | 11:06 | |
*** dmellado has quit IRC | 11:06 | |
*** ianw has quit IRC | 11:06 | |
*** bnemec has quit IRC | 11:06 | |
*** jecarey_ has quit IRC | 11:06 | |
*** jesusaur has quit IRC | 11:06 | |
*** SlickN1k is now known as SlickNik | 11:06 | |
*** jgriffith_ is now known as jgriffith | 11:06 | |
*** jgriffith is now known as Guest72772 | 11:06 | |
*** krotscheck_ is now known as krotscheck | 11:06 | |
*** marcusvrn__ is now known as marcusvrn_ | 11:07 | |
*** dtroyer has joined #openstack-qa | 11:07 | |
*** ianw has joined #openstack-qa | 11:08 | |
*** dougwig has quit IRC | 11:09 | |
*** sweston has quit IRC | 11:09 | |
*** dtantsur has joined #openstack-qa | 11:09 | |
*** anthonyper has joined #openstack-qa | 11:10 | |
*** tosky_ has joined #openstack-qa | 11:10 | |
*** gszasz has quit IRC | 11:10 | |
*** melwitt has joined #openstack-qa | 11:10 | |
*** zhiyan_ is now known as zhiyan | 11:10 | |
*** melwitt is now known as Guest64003 | 11:10 | |
*** tosky has quit IRC | 11:12 | |
*** rcernin has quit IRC | 11:12 | |
*** Kennan has quit IRC | 11:13 | |
*** sweston has joined #openstack-qa | 11:15 | |
*** dougwig has joined #openstack-qa | 11:15 | |
*** kinrui has joined #openstack-qa | 11:17 | |
*** rcernin has joined #openstack-qa | 11:23 | |
*** gszasz has joined #openstack-qa | 11:24 | |
*** Kennan has joined #openstack-qa | 11:32 | |
*** lucasagomes is now known as lucas-hungry | 11:43 | |
*** dave-mccowan has joined #openstack-qa | 11:44 | |
*** tochi has quit IRC | 11:46 | |
*** weshay has joined #openstack-qa | 11:56 | |
*** yamamoto has quit IRC | 12:01 | |
*** mtreinish has joined #openstack-qa | 12:03 | |
*** rfolco has joined #openstack-qa | 12:04 | |
*** aysyd has joined #openstack-qa | 12:04 | |
*** Poornima has joined #openstack-qa | 12:10 | |
*** raildo-afk is now known as raildo | 12:12 | |
*** zz_dimtruck is now known as dimtruck | 12:13 | |
*** mylu has joined #openstack-qa | 12:15 | |
*** markvoelker has joined #openstack-qa | 12:19 | |
*** mylu has quit IRC | 12:26 | |
*** edmondsw has joined #openstack-qa | 12:26 | |
*** Poornima_K has joined #openstack-qa | 12:27 | |
*** ddieterly has joined #openstack-qa | 12:29 | |
*** ddieterly is now known as ddieterly[away] | 12:30 | |
*** Poornima has quit IRC | 12:30 | |
*** mylu has joined #openstack-qa | 12:33 | |
*** jpena is now known as jpena|lunch | 12:36 | |
*** yamamoto has joined #openstack-qa | 12:38 | |
*** mylu has quit IRC | 12:39 | |
*** ober37 has joined #openstack-qa | 12:39 | |
*** ober37_ has joined #openstack-qa | 12:40 | |
*** ober37 has quit IRC | 12:43 | |
*** Deng has quit IRC | 12:44 | |
*** lucas-hungry is now known as lucasagomes | 12:44 | |
*** neeti_ has quit IRC | 12:44 | |
*** yamamoto has quit IRC | 12:45 | |
*** yamamoto has joined #openstack-qa | 12:45 | |
*** jecarey has quit IRC | 12:50 | |
*** Poornima_K has quit IRC | 12:50 | |
*** yamamoto has quit IRC | 12:50 | |
*** ddieterly[away] is now known as ddieterly | 12:53 | |
*** yamamoto has joined #openstack-qa | 12:54 | |
*** yamamoto has quit IRC | 12:56 | |
*** yamamoto has joined #openstack-qa | 12:57 | |
*** cdent has quit IRC | 12:58 | |
*** amotoki has quit IRC | 12:59 | |
*** cdent has joined #openstack-qa | 12:59 | |
*** yamamoto has quit IRC | 13:02 | |
*** ddieterly has quit IRC | 13:07 | |
*** ekhugen has quit IRC | 13:08 | |
*** maurosr has quit IRC | 13:08 | |
*** kinrui is now known as fungi | 13:09 | |
*** yamamoto has joined #openstack-qa | 13:13 | |
*** asselin__ has joined #openstack-qa | 13:15 | |
*** drptbl_ has quit IRC | 13:15 | |
*** amotoki has joined #openstack-qa | 13:19 | |
*** dtantsur is now known as dtantsur|brb | 13:19 | |
*** drptbl has joined #openstack-qa | 13:19 | |
*** nwkarsten has joined #openstack-qa | 13:20 | |
*** pilgrimstack has quit IRC | 13:22 | |
*** pilgrimstack1 has joined #openstack-qa | 13:22 | |
*** tosky_ is now known as tosky | 13:23 | |
*** dkranz_ has joined #openstack-qa | 13:24 | |
*** rfolco has quit IRC | 13:26 | |
*** rfolco has joined #openstack-qa | 13:26 | |
openstackgerrit | Andrew Kerr proposed openstack/tempest: Add tests for Cinder user messages v3 API https://review.openstack.org/300639 | 13:27 |
*** mriedem has joined #openstack-qa | 13:28 | |
*** phschwartz_ is now known as pschwartz | 13:28 | |
*** nwkarsten has quit IRC | 13:31 | |
*** cdent has quit IRC | 13:31 | |
*** nwkarsten has joined #openstack-qa | 13:32 | |
*** amotoki has quit IRC | 13:34 | |
*** nwkarsten has quit IRC | 13:36 | |
*** amotoki has joined #openstack-qa | 13:40 | |
*** fnaval_ has joined #openstack-qa | 13:41 | |
sc68cal | sdague: http://lists.openstack.org/pipermail/openstack-dev/2016-April/092415.html :) | 13:43 |
*** rbak has joined #openstack-qa | 13:43 | |
*** pc_m has quit IRC | 13:44 | |
*** fnaval has quit IRC | 13:45 | |
*** jaypipes has joined #openstack-qa | 13:47 | |
*** ChanServ sets mode: +o jaypipes | 13:47 | |
*** nwkarsten has joined #openstack-qa | 13:48 | |
*** pc_m has joined #openstack-qa | 13:49 | |
*** amotoki has quit IRC | 13:49 | |
*** jpena|lunch is now known as jpena | 13:49 | |
*** ddieterly has joined #openstack-qa | 13:51 | |
*** ddieterly is now known as ddieterly[away] | 13:51 | |
*** ddieterly[away] is now known as ddieterly | 13:51 | |
*** ddieterly is now known as ddieterly[away] | 13:52 | |
*** ddieterly[away] is now known as ddieterly | 13:54 | |
*** Alex_Stef has quit IRC | 13:57 | |
*** amotoki has joined #openstack-qa | 13:59 | |
*** bkeller` has quit IRC | 14:01 | |
*** coreyob_ is now known as coreyob | 14:02 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 14:03 | |
*** jecarey has joined #openstack-qa | 14:04 | |
*** jecarey has quit IRC | 14:04 | |
*** jecarey has joined #openstack-qa | 14:04 | |
*** sc68cal has quit IRC | 14:04 | |
*** bkeller` has joined #openstack-qa | 14:04 | |
*** ddieterly is now known as ddieterly[away] | 14:06 | |
openstackgerrit | Sergey Nikitin proposed openstack/tempest: Added tests for compute microversion 2.9 https://review.openstack.org/307222 | 14:07 |
*** dstanek has joined #openstack-qa | 14:07 | |
*** sc68cal has joined #openstack-qa | 14:09 | |
jroll | mtreinish: so turns out that "ssh commands are hanging" thing we talked about is actually something with ssh - the polling implementation in devstack is broken, the non-polling implementation apparently works fine | 14:13 |
mriedem | mtreinish: is it just me, or is this compute api test for volume snapshots not doing anything with a snapshot of a server with a volume attached to it? https://github.com/openstack/tempest/blob/master/tempest/api/compute/volumes/test_volume_snapshots.py#L42 | 14:13 |
mriedem | looks like purely a proxy for the cinder snapshots API via nova | 14:13 |
*** mfisher_ora has joined #openstack-qa | 14:14 | |
mtreinish | jroll: the polling implementation in devstack? | 14:14 |
mtreinish | mriedem: yeah it looks like it tests nova's proxy api for volume snapshots | 14:16 |
mriedem | yuck | 14:16 |
mriedem | mtreinish: which is redundant with https://github.com/openstack/tempest/blob/master/tempest/api/volume/test_volumes_snapshots.py#L87 right? | 14:17 |
mriedem | mtreinish: looks like we should make the compute api one a volume attached to a server and then snapshot the server, | 14:17 |
mriedem | although we already have a scenario test for that, but the scenario test has the ssh and timestamp file wrinkles probably | 14:17 |
*** fnaval_ has quit IRC | 14:17 | |
jroll | mtreinish: in tempest-lib: https://github.com/openstack/tempest-lib/blob/master/tempest_lib/common/ssh.py#L127-L160 | 14:18 |
jroll | sorry | 14:18 |
mtreinish | mriedem: yeah it looks like the same as taht test, just going through nova instead of direct to cinder | 14:19 |
mriedem | test_stamp_pattern would test creating a snapshot of a server with a volume attached, except it's skipped :) | 14:19 |
mtreinish | mriedem: right | 14:19 |
mriedem | reason i'm looking is i asked the person writing this to provide a tempest test for it https://review.openstack.org/#/c/295865/ | 14:20 |
mriedem | snapshot a server with a volume attached and the server is paused / suspended when we snapshot | 14:20 |
mriedem | i figured we already had a test for the volume attached snapshot when the server is active test | 14:20 |
mtreinish | jroll: you mean: https://github.com/openstack/tempest/blob/master/tempest/lib/common/ssh.py#L127-L154 ? Or are you actually using the tempest-lib version? | 14:21 |
mtreinish | jroll: there could be a bug there, but fwiw that poll() loop hasn't changed in quite some time | 14:22 |
mtreinish | mriedem: yeah I think that's probably a good addition | 14:22 |
*** ddieterly[away] is now known as ddieterly | 14:23 | |
*** ddieterly is now known as ddieterly[away] | 14:24 | |
*** drptbl has quit IRC | 14:24 | |
*** rlrossit has joined #openstack-qa | 14:25 | |
*** ajmiller has joined #openstack-qa | 14:25 | |
*** drptbl has joined #openstack-qa | 14:25 | |
*** rfolco has quit IRC | 14:26 | |
*** rfolco has joined #openstack-qa | 14:26 | |
*** ober37_ has quit IRC | 14:26 | |
*** ddieterly[away] is now known as ddieterly | 14:27 | |
*** ober37 has joined #openstack-qa | 14:28 | |
*** rfolco has quit IRC | 14:28 | |
jroll | mtreinish: I guess I'm not sure which off-hand, but they're the same | 14:29 |
jroll | mtreinish: and we're trying to track it down, but yeah that's busted for us somehow, absolutely no clue why | 14:29 |
*** rfolco has joined #openstack-qa | 14:29 | |
jroll | just letting you know what we found and curious if you have thuoghts | 14:30 |
*** lbragstad_ is now known as lbragstad | 14:31 | |
mtreinish | jroll: heh TBH I'm not sure if I've touched that code since: https://review.openstack.org/#/c/41497/ | 14:32 |
jroll | lol | 14:32 |
mtreinish | jroll: but, if you have more specifics on where it's falling apart I can try to give a hand | 14:33 |
jroll | mtreinish: this is most of what I have: https://bugs.launchpad.net/diskimage-builder/+bug/1571492 | 14:34 |
openstack | Launchpad bug 1571492 in Ironic "gate-tempest-dsvm-ironic-pxe_ssh-dib failing : test_baremetal_server_ops ssh timeout" [Critical,Confirmed] | 14:34 |
*** ddieterly is now known as ddieterly[away] | 14:35 | |
*** fnaval has joined #openstack-qa | 14:35 | |
*** fnaval has quit IRC | 14:37 | |
*** pglass has joined #openstack-qa | 14:38 | |
*** ddieterly[away] is now known as ddieterly | 14:39 | |
*** Guest11043 is now known as redrobot | 14:40 | |
*** cdent has joined #openstack-qa | 14:41 | |
*** diana_clarke has left #openstack-qa | 14:42 | |
*** pilgrimstack1 has quit IRC | 14:43 | |
*** ekhugen has joined #openstack-qa | 14:43 | |
*** maurosr has joined #openstack-qa | 14:44 | |
*** fnaval has joined #openstack-qa | 14:44 | |
*** erlon_ has joined #openstack-qa | 14:48 | |
*** pilgrimstack has joined #openstack-qa | 14:48 | |
*** ajmiller_ has joined #openstack-qa | 14:48 | |
*** jlanoux has quit IRC | 14:48 | |
*** Guest64003 is now known as melwitt | 14:51 | |
*** ajmiller has quit IRC | 14:52 | |
*** ajmiller has joined #openstack-qa | 14:53 | |
*** jlwhite_ has joined #openstack-qa | 14:54 | |
*** jlwhite_ has quit IRC | 14:55 | |
*** jlwhite_ has joined #openstack-qa | 14:55 | |
*** ajmiller_ has quit IRC | 14:56 | |
*** sshnaidm has quit IRC | 14:58 | |
*** rbak has quit IRC | 14:59 | |
*** ddieterly is now known as ddieterly[away] | 15:00 | |
*** breton has joined #openstack-qa | 15:01 | |
breton | hi. Is this the right place for questions about tempest? | 15:02 |
jroll | yes | 15:02 |
*** ddieterly[away] is now known as ddieterly | 15:06 | |
*** dtantsur|brb is now known as dtantsur | 15:07 | |
*** shausy has quit IRC | 15:08 | |
*** zackf has joined #openstack-qa | 15:08 | |
breton | > Tempest will automatically discover any installed plugins when it is run. | 15:08 |
breton | we have a tempest plugin in keystone. It's in keystone_tempest_plugin/ directory and is named "keystone_tests". How do i run not the entire set of tempest tests, but tests only from this plugin? | 15:08 |
tosky | breton: it's all a matter of patterns to use with ostestr: http://docs.openstack.org/developer/tempest/overview.html#quickstart | 15:14 |
*** itzikb has quit IRC | 15:15 | |
*** ober37 has quit IRC | 15:15 | |
*** ober37 has joined #openstack-qa | 15:16 | |
*** armax has joined #openstack-qa | 15:17 | |
*** jlanoux has joined #openstack-qa | 15:18 | |
*** ober37 has quit IRC | 15:20 | |
*** abehl has quit IRC | 15:20 | |
breton | tosky: so to run only keystone_tests i need to use regex `tempest\.keystone_tests`? | 15:20 |
*** cody-somerville has joined #openstack-qa | 15:22 | |
tosky | breton: probably 'keystone_tempest_plugin.*' | 15:23 |
tosky | breton: check the pattern by listing all the available tests with testr list-tests | 15:23 |
*** vhosakot has joined #openstack-qa | 15:25 | |
*** mtanino has joined #openstack-qa | 15:25 | |
*** armax has quit IRC | 15:26 | |
artom | A whole lot of jobs in the check pipeline are stuck in queued... Is this normal, am I just seeing trouble where there isn't any? | 15:28 |
*** Guest72772 is now known as jgriffith | 15:28 | |
*** armax has joined #openstack-qa | 15:29 | |
*** ddieterly is now known as ddieterly[away] | 15:29 | |
breton | tosky: understood, thank you | 15:29 |
*** rbak has joined #openstack-qa | 15:33 | |
*** armax has quit IRC | 15:33 | |
*** woodster_ has joined #openstack-qa | 15:34 | |
*** e0ne has quit IRC | 15:34 | |
*** mrodden has quit IRC | 15:43 | |
*** armax has joined #openstack-qa | 15:44 | |
*** mrodden has joined #openstack-qa | 15:44 | |
*** ddieterly[away] is now known as ddieterly | 15:46 | |
*** ddieterly is now known as ddieterly[away] | 15:47 | |
rlrossit | mtreinish: I've found something curious with these tempest plugins... if you import tempest.clients within the plugin class's module, stevedore suddenly becomes unable to find the plugin class | 15:49 |
rlrossit | but it can only be in the class's module. If I import that anywhere else (after the plugin has been loaded) everything works fine | 15:50 |
*** nwkarsten has quit IRC | 15:50 | |
*** nwkarsten has joined #openstack-qa | 15:50 | |
mtreinish | rlrossit: hmm that's weird, maybe there's an import error or something which is blocking stevedore from loading it? | 15:52 |
*** sshnaidm has joined #openstack-qa | 15:53 | |
rlrossit | mtreinish: so this started out with me importing my own modules in there, but after commenting out more and more, I've gotten it down to an import of tempest.clients. If I comment out that import, it loads just fine | 15:54 |
*** apoorvad has joined #openstack-qa | 15:54 | |
rlrossit | I don't really want to go through the 50000 imports in tempest.clients either to see if one of those is mucking it up :( | 15:55 |
mtreinish | rlrossit: I would expect it to log something in tempest.log if there was an error blocking it from being loaded though | 15:55 |
mtreinish | heh, yeah there are a lot of imports in clients.py | 15:56 |
mtreinish | it could also be anything executed at import time in the clients.py module | 15:56 |
rlrossit | mtreinish: the only thing it's currently complaining about is "ERROR tempest.test_discover.plugins [-] Could not load 'plugin_name': 'module' object has no attribute 'CinnamonRolePlugin'" | 15:57 |
rlrossit | and then it barfs saying module has no attribute CinnamonRolePlugin | 15:58 |
rlrossit | I'm using a binary search to figure out where the import problem is happening... | 15:58 |
rlrossit | meta-algorithms | 15:58 |
*** nwkarste_ has joined #openstack-qa | 15:58 | |
*** ddieterly[away] is now known as ddieterly | 15:59 | |
*** nwkarsten has quit IRC | 15:59 | |
*** ddieterly has quit IRC | 15:59 | |
*** tqtran has joined #openstack-qa | 16:00 | |
*** ddieterly has joined #openstack-qa | 16:00 | |
*** ddieterly has quit IRC | 16:00 | |
mtreinish | rlrossit: we probably should make that logging a bit better if we can: https://github.com/openstack/tempest/blob/master/tempest/test_discover/plugins.py#L80-L82 | 16:00 |
rlrossit | mtreinish: the problem wouldn't be in accessing CONF would it? | 16:00 |
mtreinish | rlrossit: it could be, the CONF stuff should be lazy loaded, so it'll try to read the config file when you do a getattr on CONF | 16:01 |
rlrossit | mtreinish: oh goodness I think that's what it is | 16:01 |
mtreinish | the ordering around conf gets really weird sometimes | 16:01 |
rlrossit | mtreinish: removing https://github.com/openstack/tempest/blob/master/tempest/clients.py#L188-L201 makes it work | 16:02 |
rlrossit | so there is some goofy bootstrapping going on with the CONF stuff | 16:02 |
rlrossit | because in order for CONF to work, it needs to find the CONF stuff of all plugins too right? | 16:02 |
mtreinish | rlrossit: right, when it inits the CONF object it uses the extension manager to get the function calls to add all the plugins options | 16:03 |
mtreinish | that is probably it | 16:04 |
*** salv-orlando has joined #openstack-qa | 16:04 | |
rlrossit | maybe I'll just move the import to the plugin's load_tests then? | 16:04 |
mtreinish | so you were trying to load all the plugins while loading the plugins :) | 16:04 |
mtreinish | why do you need clients.py in the plugin class? | 16:04 |
*** Guest30394 has quit IRC | 16:05 | |
rlrossit | mtreinish: so it's at the bottom of a bunch of other imports | 16:05 |
rlrossit | mtreinish: you ready for my evil master plan with my plugin? | 16:05 |
*** ober37 has joined #openstack-qa | 16:05 | |
* mtreinish is afraid | 16:06 | |
rlrossit | you should be | 16:06 |
*** jlwhite_ has quit IRC | 16:06 | |
*** jlwhite_ has joined #openstack-qa | 16:06 | |
rlrossit | I'm basically trying to dynamically load all tempest test classes, and then wrap each of those in a dynamically created class to run with a different set of users | 16:06 |
*** salv-orl_ has quit IRC | 16:06 | |
rlrossit | with different expected results for each set of users | 16:07 |
*** nwkarste_ has quit IRC | 16:07 | |
*** jlwhite__ has joined #openstack-qa | 16:07 | |
* rlrossit laughs maniacally | 16:07 | |
*** ober37 has quit IRC | 16:07 | |
*** jlwhite__ has left #openstack-qa | 16:07 | |
*** ober37 has joined #openstack-qa | 16:07 | |
*** Swami has joined #openstack-qa | 16:08 | |
rlrossit | mtreinish: and because tempest's plugins don't account for my extremely crazy one-off plugin, I'm trying to do a lot of that in load_tests :) | 16:08 |
*** nwkarsten has joined #openstack-qa | 16:08 | |
mtreinish | rlrossit: hah, yeah that's pretty out there :) | 16:08 |
*** aysyd has quit IRC | 16:09 | |
rlrossit | mtreinish: the goal is when I'm no longer embarrassed about the state of this thing, I'll throw it out in the openstack tent | 16:09 |
mtreinish | cool, the best part of that is tempest doesn't actually call load_tests, so you wrote a plugin to have the test runner execute a bunch of arbitrary code | 16:09 |
rlrossit | it's actually really really good at testing policy configurations | 16:09 |
mtreinish | rlrossit: yeah, I bet a bunch of people with weird policy files are gonna love it | 16:10 |
*** nadya has quit IRC | 16:10 | |
mtreinish | my normal answer for those peopl is: stop breaking interop we won't fix that in tempest... | 16:10 |
rlrossit | mtreinish: does that make me an enabler? | 16:11 |
mtreinish | heh, I guess so | 16:11 |
*** sridharg has quit IRC | 16:13 | |
mtreinish | rlrossit: http://lists.openstack.org/pipermail/openstack-dev/2016-February/086118.html and https://bugs.launchpad.net/tempest/+bug/1565584 are 2 examples from recent memory | 16:17 |
openstack | Launchpad bug 1565584 in tempest "Default tests do not request credentials by role" [Undecided,In progress] - Assigned to Dale Smith (dalees) | 16:17 |
rlrossit | mtreinish: oh man, these guys are gonna love me | 16:18 |
rlrossit | I do exactly what is asked in that ML :P | 16:18 |
rlrossit | or well part of it at least I guess | 16:18 |
*** Kiall has quit IRC | 16:18 | |
mtreinish | rlrossit: yeah I don't think your case will work for the ML configuration. They set servers to be scoped to the user not tenant | 16:20 |
rlrossit | that bug though... that's me fo sho | 16:20 |
*** nwkarsten has quit IRC | 16:21 | |
*** pcaruana has quit IRC | 16:21 | |
*** jlwhite_ has quit IRC | 16:22 | |
*** jlwhite_ has joined #openstack-qa | 16:22 | |
*** Kiall has joined #openstack-qa | 16:22 | |
*** jlwhite_ has quit IRC | 16:22 | |
*** jlwhite_ has joined #openstack-qa | 16:23 | |
*** aysyd has joined #openstack-qa | 16:23 | |
*** nwkarsten has joined #openstack-qa | 16:25 | |
*** Kiall has quit IRC | 16:27 | |
*** gszasz has quit IRC | 16:27 | |
*** jlwhite__ has joined #openstack-qa | 16:28 | |
*** tosky has quit IRC | 16:29 | |
*** jlwhite__ has quit IRC | 16:29 | |
*** fnaval has quit IRC | 16:31 | |
*** jlwhite has quit IRC | 16:35 | |
*** tosky has joined #openstack-qa | 16:35 | |
*** jlwhite has joined #openstack-qa | 16:36 | |
*** fnaval has joined #openstack-qa | 16:36 | |
*** Kiall has joined #openstack-qa | 16:38 | |
*** tqtran has quit IRC | 16:40 | |
*** jlwhite has quit IRC | 16:41 | |
*** rcernin has quit IRC | 16:43 | |
*** Kiall has quit IRC | 16:43 | |
*** Kiall has joined #openstack-qa | 16:44 | |
openstackgerrit | Jordan Pittier proposed openstack/tempest: Remove iso8601 and anyjson from requirements.txt https://review.openstack.org/307370 | 16:47 |
dalees` | rlrossit, i'm not caught up on what you're working on, but if you'd like someone to help test/review in the context of https://bugs.launchpad.net/tempest/+bug/1565584 i'd be happy to help :) | 16:48 |
openstack | Launchpad bug 1565584 in tempest "Default tests do not request credentials by role" [Undecided,In progress] - Assigned to Dale Smith (dalees) | 16:48 |
*** jlwhite_ has quit IRC | 16:52 | |
*** vgridnev_ has joined #openstack-qa | 16:52 | |
*** pc_m has quit IRC | 16:53 | |
*** davideagnello has joined #openstack-qa | 16:54 | |
*** pc_m has joined #openstack-qa | 16:55 | |
*** jlwhite has joined #openstack-qa | 16:58 | |
*** jlwhite has quit IRC | 17:00 | |
*** jlwhite has joined #openstack-qa | 17:03 | |
*** jlwhite has quit IRC | 17:03 | |
*** rossella_s has quit IRC | 17:03 | |
*** rossella_s has joined #openstack-qa | 17:04 | |
*** jpena is now known as jpena|off | 17:07 | |
*** tqtran has joined #openstack-qa | 17:10 | |
*** Alex_Stef has joined #openstack-qa | 17:10 | |
*** harlowja has joined #openstack-qa | 17:11 | |
*** nwkarsten has quit IRC | 17:14 | |
*** nwkarsten has joined #openstack-qa | 17:22 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 17:26 | |
*** jesusaur has joined #openstack-qa | 17:26 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 17:28 | |
*** yamahata has joined #openstack-qa | 17:29 | |
*** gszasz has joined #openstack-qa | 17:31 | |
openstackgerrit | Alex Meade proposed openstack-dev/devstack: Add volumev3 endpoint for Cinder https://review.openstack.org/300585 | 17:32 |
*** kbaikov_ has joined #openstack-qa | 17:32 | |
*** kbaikov_ has quit IRC | 17:35 | |
*** gfidente has quit IRC | 17:35 | |
*** tosky has quit IRC | 17:36 | |
*** cdent has quit IRC | 17:38 | |
*** ober37 has left #openstack-qa | 17:40 | |
*** ober37 has joined #openstack-qa | 17:41 | |
*** stevemar has quit IRC | 17:41 | |
ober37 | Tempest question, what's the stance on negative test case contributions? Are these on hold until the topic gets settled at Austin? | 17:41 |
*** kbaikov_ has joined #openstack-qa | 17:43 | |
*** stevemar has joined #openstack-qa | 17:44 | |
*** ifarkas has quit IRC | 17:44 | |
*** kbaikov_ has quit IRC | 17:47 | |
*** Serlex has left #openstack-qa | 17:47 | |
*** jlanoux has quit IRC | 17:48 | |
*** dtantsur is now known as dtantsur|afk | 17:49 | |
*** lucasagomes is now known as lucas-dinner | 17:51 | |
*** kbaikov_ has joined #openstack-qa | 17:52 | |
*** kbaikov_ has quit IRC | 17:53 | |
openstackgerrit | Jordan Pittier proposed openstack/tempest: Remove definitions of unused exception classes. https://review.openstack.org/307392 | 17:54 |
*** pvaneck has joined #openstack-qa | 17:56 | |
*** nwkarsten has quit IRC | 17:59 | |
*** nwkarsten has joined #openstack-qa | 18:00 | |
*** nwkarsten has quit IRC | 18:11 | |
*** yamahata has quit IRC | 18:11 | |
*** ajmiller_ has joined #openstack-qa | 18:12 | |
*** nwkarsten has joined #openstack-qa | 18:12 | |
*** salv-orlando has quit IRC | 18:13 | |
*** salv-orlando has joined #openstack-qa | 18:13 | |
*** ajmiller has quit IRC | 18:15 | |
*** salv-orlando has quit IRC | 18:18 | |
mtreinish | ober37: I don't think there is an official block on them until summit. But, I think you'll probably have a hard time getting critical review mass until the session | 18:22 |
*** nwkarsten has quit IRC | 18:23 | |
*** nwkarsten has joined #openstack-qa | 18:23 | |
*** salv-orlando has joined #openstack-qa | 18:24 | |
*** sambetts is now known as sambetts|afk | 18:32 | |
*** mpaolino has quit IRC | 18:35 | |
*** nwkarsten has quit IRC | 18:35 | |
*** nwkarsten has joined #openstack-qa | 18:36 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add generic CONF skip decorators https://review.openstack.org/306780 | 18:37 |
oomichi | gmann: I saw https://review.openstack.org/#/c/306919 , but I cannot fine the merit like fixing actual bug or something. | 18:39 |
ober37 | mtreinish: Thanks! Yeah that makes sense | 18:39 |
oomichi | gmann: there is a lot of this kind thing on each API | 18:40 |
mtreinish | oomichi: heh, when I saw the email for that bug I just assumed it was the httplib2->urllib3 migration. But, thinking about it more that couldn't have been because that would've been blocked in the gate | 18:41 |
mtreinish | oomichi: so I agree, I don't think there is a real bug there | 18:41 |
mtreinish | oomichi: on https://review.openstack.org/305495 was there something you wanted me to change on that patch? Or is it good to go? | 18:43 |
*** nadya has joined #openstack-qa | 18:43 | |
oomichi | mtreinish: ah, I am guessing you will separate start_date param from this method and create get_recent_*_runs method with start_dete param. willn't you? | 18:45 |
mtreinish | oomichi: yes, I was planning to push a follow up patch to add the additional parameter to the other methods soon | 18:46 |
oomichi | mtreinish: or will you keep the param on the existing one and create another method with the param? | 18:46 |
mtreinish | I am keeping the param on the method added in that patch, because it's needed there | 18:46 |
oomichi | mtreinish: ah, I see your point. you will add the param to *existing* methods | 18:47 |
oomichi | mtreinish: I misunderstood you would add new methods | 18:47 |
oomichi | for the param. | 18:47 |
mtreinish | oomichi: oh, no that would be kinda crazy, it's a kwarg so I can just add it to the end of the existing methods in the future | 18:48 |
*** ajmiller_ is now known as ajmiller | 18:48 | |
mtreinish | no need to add a separate method for 1 additional option arg | 18:48 |
oomichi | mtreinish: ok, that is extensible because of start_date=None | 18:48 |
oomichi | mtreinish: yeah, I need coffee now. +A | 18:49 |
*** nwkarsten has quit IRC | 18:52 | |
oomichi | mtreinish: btw how about https://review.openstack.org/#/c/306985/ ? After that, maybe masayukig will make the exsting tempest-cli-improvements as implemented | 18:52 |
oomichi | mtreinish: then we can see a single clear target: tempest.run | 18:53 |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add optional start_date parameter to get_recent*() methods https://review.openstack.org/307416 | 18:53 |
mtreinish | oomichi: ^^^ that's what I meant for the followup patch (that's a wip) | 18:54 |
oomichi | mtreinish: so fast, cool | 18:54 |
mtreinish | oomichi: well I still need to update the docs, and add tests and release notes | 18:55 |
mtreinish | but the functional change is super straightforward | 18:55 |
oomichi | mtreinish: -1 on https://review.openstack.org/307416 | 18:56 |
mtreinish | oomichi: I did say it was a wip... | 18:56 |
oomichi | mtreinish: hehe, I know. but you didn't put it on gerrit ;-) | 18:57 |
mtreinish | oomichi: you were too fast | 18:57 |
*** nwkarsten has joined #openstack-qa | 19:00 | |
openstackgerrit | Merged openstack/tempest: "is_resource_deleted" fails to verify delete https://review.openstack.org/296072 | 19:02 |
*** bkeller^ has joined #openstack-qa | 19:04 | |
*** bkeller^ has quit IRC | 19:04 | |
*** nwkarsten has quit IRC | 19:04 | |
*** ober37 has quit IRC | 19:07 | |
*** ober37 has joined #openstack-qa | 19:08 | |
*** ober37 has quit IRC | 19:08 | |
oomichi | mtreinish: oops, you -1d on masayukig's spec patch. isn,t that confused if moving the spec which includes tempest.run also into implemented? | 19:08 |
*** ober37 has joined #openstack-qa | 19:08 | |
mtreinish | oomichi: no I think masayukig should make the changes and move the file in 1 patch | 19:09 |
*** bkeller^ has joined #openstack-qa | 19:09 | |
oomichi | mtreinish: ah, I agree :-) | 19:09 |
openstackgerrit | Merged openstack-infra/subunit2sql: Add api to get recent failed runs by run_metadata https://review.openstack.org/305495 | 19:16 |
*** bkeller`- has joined #openstack-qa | 19:16 | |
*** jlwhite has joined #openstack-qa | 19:19 | |
*** rfolco has quit IRC | 19:20 | |
*** bkeller`- has quit IRC | 19:23 | |
*** bkeller^ has quit IRC | 19:23 | |
*** bkeller^ has joined #openstack-qa | 19:23 | |
*** bkeller` has quit IRC | 19:24 | |
*** bkeller^ is now known as bkeller` | 19:24 | |
*** cdent has joined #openstack-qa | 19:25 | |
*** ober37` has joined #openstack-qa | 19:25 | |
*** rfolco has joined #openstack-qa | 19:25 | |
openstackgerrit | Merged openstack-infra/subunit2sql: Fix the typo of docstring https://review.openstack.org/305941 | 19:25 |
*** cdent has quit IRC | 19:26 | |
*** mminesh has joined #openstack-qa | 19:27 | |
*** jlwhite has quit IRC | 19:28 | |
*** nwkarsten has joined #openstack-qa | 19:29 | |
*** weshay has quit IRC | 19:29 | |
*** jlwhite has joined #openstack-qa | 19:37 | |
*** yamamoto has quit IRC | 19:39 | |
*** salv-orlando has quit IRC | 19:47 | |
*** ddieterly has joined #openstack-qa | 19:52 | |
*** e0ne has joined #openstack-qa | 19:52 | |
*** yamamoto has joined #openstack-qa | 19:54 | |
*** ddieterly is now known as ddieterly[away] | 20:00 | |
*** alex_xu has quit IRC | 20:04 | |
*** ddieterly[away] is now known as ddieterly | 20:05 | |
*** alex_xu has joined #openstack-qa | 20:06 | |
*** e0ne has quit IRC | 20:08 | |
*** nwkarsten has quit IRC | 20:09 | |
*** nwkarsten has joined #openstack-qa | 20:10 | |
*** beekneemech is now known as bneme | 20:12 | |
*** bneme is now known as bnemec | 20:12 | |
*** ddieterly is now known as ddieterly[away] | 20:23 | |
*** ddieterly[away] is now known as ddieterly | 20:26 | |
*** ober37 has quit IRC | 20:27 | |
*** ober37` is now known as ober37 | 20:27 | |
*** nwkarste_ has joined #openstack-qa | 20:28 | |
*** nwkarsten has quit IRC | 20:28 | |
openstackgerrit | Merged openstack-dev/devstack: worlddump: request Guru Mediation reports for neutron agents https://review.openstack.org/279035 | 20:28 |
*** nwkarste_ has quit IRC | 20:29 | |
*** nwkarsten has joined #openstack-qa | 20:30 | |
*** ober37_ has joined #openstack-qa | 20:32 | |
*** yamahata has joined #openstack-qa | 20:33 | |
*** salv-orlando has joined #openstack-qa | 20:33 | |
*** nwkarsten has quit IRC | 20:34 | |
*** nadya has quit IRC | 20:36 | |
*** e0ne has joined #openstack-qa | 20:38 | |
*** dkranz_ has quit IRC | 20:40 | |
*** lucas-dinner has quit IRC | 20:44 | |
*** e0ne has quit IRC | 20:50 | |
*** lucasagomes has joined #openstack-qa | 20:53 | |
*** cdent has joined #openstack-qa | 20:56 | |
*** ober37_ has quit IRC | 20:56 | |
*** hichihara has joined #openstack-qa | 20:57 | |
*** fawadkhaliq has joined #openstack-qa | 20:57 | |
*** mfisher_ora has quit IRC | 20:57 | |
*** weshay has joined #openstack-qa | 20:58 | |
*** rossella_s has quit IRC | 21:03 | |
*** lucasagomes has quit IRC | 21:03 | |
*** rossella_s has joined #openstack-qa | 21:04 | |
*** raildo is now known as raildo-afk | 21:05 | |
*** shardy has quit IRC | 21:11 | |
*** ddieterly is now known as ddieterly[away] | 21:12 | |
*** lucasagomes has joined #openstack-qa | 21:12 | |
*** aysyd has quit IRC | 21:14 | |
*** gszasz has quit IRC | 21:16 | |
*** mylu has joined #openstack-qa | 21:22 | |
*** vhosakot has quit IRC | 21:22 | |
*** rlrossit has quit IRC | 21:22 | |
*** mylu has quit IRC | 21:24 | |
*** mylu has joined #openstack-qa | 21:25 | |
*** jecarey has quit IRC | 21:30 | |
*** nwkarsten has joined #openstack-qa | 21:34 | |
*** ddieterly[away] is now known as ddieterly | 21:38 | |
*** nwkarsten has quit IRC | 21:41 | |
*** dimtruck is now known as zz_dimtruck | 21:42 | |
*** pcrews_ has joined #openstack-qa | 21:47 | |
*** pcrews has quit IRC | 21:50 | |
*** nwkarsten has joined #openstack-qa | 21:53 | |
*** ddieterly is now known as ddieterly[away] | 21:54 | |
*** mylu has quit IRC | 21:54 | |
*** pglass has quit IRC | 21:56 | |
*** fawadkhaliq has quit IRC | 22:00 | |
openstackgerrit | Merged openstack-infra/subunit2sql: Rework migration 1679b5bc102c to not use the db api https://review.openstack.org/301369 | 22:05 |
*** jecarey has joined #openstack-qa | 22:07 | |
*** jecarey has quit IRC | 22:12 | |
*** cdent has quit IRC | 22:13 | |
*** hichihara has quit IRC | 22:14 | |
*** ddieterly[away] is now known as ddieterly | 22:16 | |
*** dave-mccowan has quit IRC | 22:19 | |
*** ddieterly is now known as ddieterly[away] | 22:20 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 22:20 | |
*** nwkarsten has quit IRC | 22:24 | |
*** nwkarsten has joined #openstack-qa | 22:25 | |
*** dalees` has quit IRC | 22:25 | |
*** ddieterly[away] is now known as ddieterly | 22:26 | |
*** hichihara has joined #openstack-qa | 22:26 | |
*** yamahata has quit IRC | 22:27 | |
*** markvoelker has quit IRC | 22:28 | |
*** vgridnev_ has quit IRC | 22:30 | |
*** ddieterly is now known as ddieterly[away] | 22:31 | |
*** yamahata has joined #openstack-qa | 22:34 | |
*** nwkarsten has quit IRC | 22:34 | |
*** nwkarsten has joined #openstack-qa | 22:35 | |
*** ddieterly[away] is now known as ddieterly | 22:35 | |
*** dalees` has joined #openstack-qa | 22:38 | |
*** nwkarsten has quit IRC | 22:39 | |
*** ddieterly is now known as ddieterly[away] | 22:42 | |
*** ddieterly[away] is now known as ddieterly | 22:44 | |
*** eliqiao_ has joined #openstack-qa | 22:44 | |
*** eliqiao has quit IRC | 22:45 | |
*** sdague has quit IRC | 22:45 | |
*** ddieterly is now known as ddieterly[away] | 22:51 | |
*** sdague has joined #openstack-qa | 22:57 | |
*** ddieterly[away] is now known as ddieterly | 23:04 | |
*** ddieterly is now known as ddieterly[away] | 23:04 | |
*** ddieterly[away] has quit IRC | 23:04 | |
*** vhosakot has joined #openstack-qa | 23:06 | |
openstackgerrit | Steve Heyman proposed openstack-dev/grenade: Update README with user parameter for setup-grenade https://review.openstack.org/307516 | 23:10 |
openstackgerrit | Steve Heyman proposed openstack-dev/grenade: Update README with user parameter for setup-grenade https://review.openstack.org/307516 | 23:12 |
*** itzikb has joined #openstack-qa | 23:14 | |
*** raildo-afk is now known as raildo | 23:18 | |
openstackgerrit | Hongbin Lu proposed openstack-dev/devstack: Support decompressing bz2 image https://review.openstack.org/306842 | 23:19 |
*** zz_dimtruck is now known as dimtruck | 23:22 | |
mriedem | mtreinish: the client setup structure for admin tests in tempest totally confuses me | 23:25 |
mriedem | like in here it creates a variable for the migrations admin client https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_migrations.py#L30 | 23:26 |
mriedem | but elsewhere in that test class it just uses self.servers_client which must be inherited from the parent class | 23:26 |
mriedem | which would be https://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L378 and that sets, | 23:27 |
mriedem | cls.availability_zone_admin_client = ( cls.os_adm.availability_zone_client) | 23:27 |
mriedem | which is confusing given the az admin client is set in the parent class of that: https://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L84 | 23:27 |
mriedem | so i never know what i'm actually supposed to do in setup_clients | 23:27 |
*** markvoelker has joined #openstack-qa | 23:28 | |
mtreinish | mriedem: for the most part setup clients is just used as aliases to save space | 23:28 |
mtreinish | I normally prefer to just use self.os_admin.client directly | 23:28 |
mtreinish | it's much clearer | 23:28 |
mtreinish | but sometimes there is a 79 character concern which is probably why there is so much aliasing | 23:29 |
mtreinish | mriedem: so I'm pretty sure L84 in the base compute class is the az client with regular user creds | 23:30 |
mtreinish | (it must be used for a negative test somewhere) | 23:30 |
mtreinish | and the admin one in the L378 class is the one to actually use (since it has admin creds) | 23:31 |
mriedem | ok so if i want to create a flavor in an admin test, i need to do https://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L84 | 23:31 |
mriedem | oops | 23:31 |
mriedem | cls.flavors_admin_client = cls.os_adm.flavors_client? | 23:31 |
mtreinish | mriedem: yeah, that or just call self.os_adm.flavors_client.create_flavor() directly | 23:32 |
mtreinish | the setup_client() stuff is not required | 23:32 |
gmann | oomichi: but that is bug in method implementation not related to API behavior | 23:33 |
*** markvoelker has quit IRC | 23:33 | |
mriedem | mtreinish: ok, i think i have this then | 23:33 |
gmann | oomichi: that method is expected to compare the two status code which should be int and return the result | 23:33 |
gmann | oomichi: but if read_code is not int then it would not compare anything and result to pass | 23:34 |
oomichi | gmann: I guess the patch adds just a validation, right? | 23:34 |
gmann | oomichi: i mean expected_code=202, read_code='201' is true with current implementation | 23:34 |
mtreinish | gmann: right but how is getting a string? it gets called directly from inside the rest client right? | 23:34 |
mtreinish | all the clients should use the same path | 23:34 |
mtreinish | is it never passing an int? | 23:34 |
*** ddieterly has joined #openstack-qa | 23:35 | |
gmann | oomichi: no this is called from every service clients | 23:35 |
oomichi | gmann: how about converting string to int like nova does? | 23:35 |
*** ddieterly is now known as ddieterly[away] | 23:35 | |
gmann | mtreinish: yea all usage are correct so no issue | 23:35 |
*** salv-orlando has quit IRC | 23:35 | |
gmann | mtreinish: oomichi if we pass resp['status'] then it was string | 23:35 |
gmann | actuall bug was with wrong usage of this method | 23:36 |
gmann | https://review.openstack.org/#/c/304380/13/designate_tempest_plugin/services/dns/v2/json/zone_imports_client.py | 23:36 |
gmann | wrong usage of this method(passing string) give false pass | 23:36 |
*** tochi has joined #openstack-qa | 23:36 | |
*** nwkarsten has joined #openstack-qa | 23:36 | |
oomichi | gmann: ah, nice to know that. | 23:36 |
openstackgerrit | Matt Riedemann proposed openstack/tempest: Add a test for reverting a resize with a deleted flavor https://review.openstack.org/307518 | 23:36 |
mriedem | dansmith: ^ | 23:36 |
oomichi | gmann: I wanted to see this kind of actual report | 23:36 |
dansmith | `mricool | 23:37 |
dansmith | wow | 23:37 |
dansmith | mriedem: cool | 23:37 |
*** ddieterly[away] is now known as ddieterly | 23:37 | |
gmann | mtreinish: oomichi currently all tempest service clients use this correctly no issue there | 23:37 |
gmann | but in some plugin it was tried wrongly and results were wrong | 23:37 |
gmann | oomichi: i added that in bug report if that is enough | 23:37 |
oomichi | gmann: but designate tried to use it wrongly before. If your patch was merged, they could know that. | 23:38 |
mtreinish | gmann: I'm still a bit confused was the dict access always available? or is this fallout from the switch to urllib3? | 23:38 |
gmann | oomichi: yea | 23:38 |
mtreinish | gmann: because that's the crux of the issue resp['status'] vs resp.status right? | 23:38 |
gmann | mtreinish: yea | 23:38 |
openstackgerrit | Matt Riedemann proposed openstack/tempest: Add a test for reverting a resize with a deleted flavor https://review.openstack.org/307518 | 23:39 |
oomichi | gmann: nice to write it on the commit message for me. | 23:39 |
gmann | mtreinish: i feel that was always, mean resp['status'] was available always | 23:39 |
*** nwkarsten has quit IRC | 23:41 | |
mtreinish | gmann: yeah it looks like it was: https://github.com/httplib2/httplib2/blob/master/python2/httplib2/__init__.py#L1655-L1690 | 23:42 |
mtreinish | tbh, I don't understand why they set it as a string from the dict access but leave it as an int from the attr | 23:43 |
mtreinish | but the urrlib3 patch was correct in maintaining that weirdness | 23:43 |
gmann | mtreinish: yea thats not right to get with string at least for this method usage | 23:43 |
mtreinish | gmann: ok I understand your bugfix more now | 23:44 |
*** nwkarsten has joined #openstack-qa | 23:44 | |
gmann | mtreinish: actually we have that in doc string but nobody read those seems :) | 23:44 |
mtreinish | gmann: I know I don't :) | 23:45 |
gmann | :) | 23:45 |
flwang | mtreinish: ping need your help on http://logs.openstack.org/79/302479/11/check/gate-tempest-dsvm-zaqar/6c92f39/ | 23:46 |
flwang | not sure if you remember our last discussion | 23:46 |
gmann | oomichi: sure, i will add more in commit msg | 23:46 |
flwang | i changed the code a bit and now i got a different error | 23:46 |
oomichi | gmann: it is nice to specify change-id on the commit message for future us | 23:46 |
* gmann need to learn clear and large commit msg from mtreinish. | 23:47 | |
gmann | oomichi: yea i always a lazy person in commit msg term :) | 23:47 |
gmann | oomichi: i will add. Thanks | 23:47 |
mtreinish | flwang: it looks like it's not completely configured still: http://logs.openstack.org/79/302479/11/check/gate-tempest-dsvm-zaqar/6c92f39/logs/tempest_conf.txt.gz | 23:47 |
mtreinish | flwang: there isn't an auth url set | 23:47 |
mtreinish | gmann: heh, something like: https://review.openstack.org/#/c/306779/ | 23:48 |
mtreinish | which is a lot of words for a file mv | 23:48 |
flwang | mtreinish: ah, i see | 23:49 |
flwang | mtreinish: cheers, man | 23:49 |
gmann | mtreinish: ah, yea, did not check that yet :) but there are lot of places you have commit msg larger than code changes :) | 23:49 |
*** nwkarsten has quit IRC | 23:49 | |
gmann | and well explanatory | 23:50 |
mtreinish | gmann: normally when writing a commit msg, I to try and answer 2 questions: why someone should care about this patch and why I decided to write it (which are often related if not the same) | 23:50 |
mtreinish | flwang: are you using devstack to configure tempest? | 23:51 |
*** mriedem is now known as mriedem_away | 23:51 | |
mtreinish | flwang: oh right I remember now, this when you're calling tempest in the gate hook before devstack runs. Why are you doing that? (I don't remember) | 23:52 |
gmann | mtreinish: humm, that's cool, I ll also try that. but issue is laziness for doc or commit msg :) but i can understand that can be much helpful many times | 23:53 |
mtreinish | gmann: heh, everyone gets lazy with commit msgs sometimes too | 23:53 |
*** ddieterly has quit IRC | 23:53 | |
mtreinish | but I found a long time ago that having a good commit msg helps a lot with getting good reviews | 23:53 |
gmann | mtreinish: yea totally agree. | 23:55 |
*** erlon_ has quit IRC | 23:55 | |
flwang | mtreinish: so should I put those new changes lines under line 36 https://review.openstack.org/#/c/302479/11/devstack/gate/gate_hook.sh ? | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!