*** lucasagomes has quit IRC | 00:00 | |
*** lucasagomes has joined #openstack-qa | 00:02 | |
*** Suyi_ has quit IRC | 00:05 | |
*** yangyapeng has quit IRC | 00:05 | |
*** Swami has quit IRC | 00:06 | |
*** lucasagomes has quit IRC | 00:07 | |
*** lucasagomes has joined #openstack-qa | 00:10 | |
*** fnaval has quit IRC | 00:10 | |
*** lucasagomes has quit IRC | 00:14 | |
*** eeiden has quit IRC | 00:16 | |
*** lucasagomes has joined #openstack-qa | 00:17 | |
openstackgerrit | Merged openstack-infra/devstack-gate: Handle multiline env vars in reproduce.sh https://review.openstack.org/369633 | 00:24 |
---|---|---|
*** tqtran has quit IRC | 00:25 | |
*** lucasagomes has quit IRC | 00:27 | |
*** krtaylor has joined #openstack-qa | 00:30 | |
*** lucasagomes has joined #openstack-qa | 00:30 | |
*** OferBarber has joined #openstack-qa | 00:32 | |
*** lucasagomes has quit IRC | 00:34 | |
*** lucasagomes has joined #openstack-qa | 00:38 | |
openstackgerrit | Merged openstack-dev/devstack: Don't add deb-* packaging repos to plugin list https://review.openstack.org/368547 | 00:39 |
*** itsuugo has quit IRC | 00:40 | |
*** itsuugo has joined #openstack-qa | 00:40 | |
*** dave-mccowan has joined #openstack-qa | 00:43 | |
*** itsuugo has quit IRC | 00:45 | |
*** lucasagomes has quit IRC | 00:46 | |
*** itsuugo has joined #openstack-qa | 00:46 | |
*** lucasagomes has joined #openstack-qa | 00:48 | |
*** tuanla_ has joined #openstack-qa | 00:49 | |
*** itsuugo has quit IRC | 00:51 | |
*** itsuugo has joined #openstack-qa | 00:53 | |
*** lucasagomes has quit IRC | 00:57 | |
*** lucasagomes has joined #openstack-qa | 01:00 | |
*** eeiden has joined #openstack-qa | 01:01 | |
*** lucasagomes has quit IRC | 01:04 | |
*** lucasagomes has joined #openstack-qa | 01:07 | |
*** gvrangan has quit IRC | 01:09 | |
*** itsuugo has quit IRC | 01:10 | |
*** itsuugo has joined #openstack-qa | 01:11 | |
*** lucasagomes has quit IRC | 01:12 | |
*** lucasagomes has joined #openstack-qa | 01:14 | |
*** gouthamr has quit IRC | 01:17 | |
*** rossella_ has quit IRC | 01:18 | |
*** lucasagomes has quit IRC | 01:19 | |
*** rossella_ has joined #openstack-qa | 01:19 | |
*** lucasagomes has joined #openstack-qa | 01:21 | |
*** itsuugo has quit IRC | 01:26 | |
*** jamielennox is now known as jamielennox|away | 01:26 | |
*** itsuugo has joined #openstack-qa | 01:27 | |
*** lucasagomes has quit IRC | 01:30 | |
*** yangyapeng has joined #openstack-qa | 01:30 | |
*** lucasagomes has joined #openstack-qa | 01:33 | |
*** gyx has joined #openstack-qa | 01:34 | |
*** artom has quit IRC | 01:38 | |
*** itsuugo has quit IRC | 01:39 | |
*** artom has joined #openstack-qa | 01:39 | |
*** salv-orlando has joined #openstack-qa | 01:41 | |
*** itsuugo has joined #openstack-qa | 01:41 | |
*** salv-orl_ has quit IRC | 01:43 | |
*** mriedem has quit IRC | 01:49 | |
*** clu_ has quit IRC | 01:50 | |
*** Apoorva_ has joined #openstack-qa | 01:51 | |
*** itsuugo has quit IRC | 01:52 | |
*** itsuugo has joined #openstack-qa | 01:54 | |
*** yamahata has quit IRC | 01:55 | |
*** Apoorva has quit IRC | 01:55 | |
openstackgerrit | zhufl proposed openstack/tempest: Rename test_list_servers_filter_by_server_status https://review.openstack.org/369775 | 01:56 |
*** Apoorva_ has quit IRC | 01:56 | |
*** yangyapeng has quit IRC | 02:06 | |
*** lucasagomes has quit IRC | 02:06 | |
*** lucasagomes has joined #openstack-qa | 02:07 | |
*** yangyapeng has joined #openstack-qa | 02:07 | |
*** lucasagomes has quit IRC | 02:15 | |
*** yangyape_ has joined #openstack-qa | 02:16 | |
*** lucasagomes has joined #openstack-qa | 02:17 | |
*** yangyapeng has quit IRC | 02:18 | |
*** woodster_ has quit IRC | 02:19 | |
*** lucasagomes has quit IRC | 02:22 | |
*** lucasagomes has joined #openstack-qa | 02:23 | |
*** yangyapeng has joined #openstack-qa | 02:24 | |
*** yangyape_ has quit IRC | 02:27 | |
*** lucasagomes has quit IRC | 02:28 | |
*** lucasagomes has joined #openstack-qa | 02:29 | |
*** csomerville has quit IRC | 02:32 | |
*** lucasagomes has quit IRC | 02:34 | |
*** lucasagomes has joined #openstack-qa | 02:37 | |
*** cody-somerville has joined #openstack-qa | 02:38 | |
*** dave-mccowan has quit IRC | 02:38 | |
*** lucasagomes has quit IRC | 02:41 | |
*** lucasagomes has joined #openstack-qa | 02:43 | |
*** akshai has joined #openstack-qa | 02:48 | |
*** lucasagomes has quit IRC | 02:48 | |
*** akshai_ has joined #openstack-qa | 02:49 | |
*** stevemar_ is now known as stevemar | 02:49 | |
*** lucasagomes has joined #openstack-qa | 02:49 | |
*** akshai has quit IRC | 02:53 | |
*** lucasagomes has quit IRC | 02:57 | |
*** lucasagomes has joined #openstack-qa | 02:58 | |
*** lucasagomes has quit IRC | 03:03 | |
*** lucasagomes has joined #openstack-qa | 03:05 | |
*** lucasagomes has quit IRC | 03:10 | |
*** lucasagomes has joined #openstack-qa | 03:13 | |
*** akshai_ has quit IRC | 03:19 | |
*** lucasagomes has quit IRC | 03:21 | |
*** itsuugo has quit IRC | 03:21 | |
*** tqtran has joined #openstack-qa | 03:22 | |
*** itsuugo has joined #openstack-qa | 03:23 | |
*** lucasagomes has joined #openstack-qa | 03:23 | |
oomichi | sc68cal: I am checking dev status of devstack based on https://etherpad.openstack.org/p/newton-qa-newton-priorities | 03:23 |
oomichi | sc68cal: the "neutron cleanup" seems done, I guess because of https://review.openstack.org/#/c/315218/ https://review.openstack.org/#/c/315221/ and https://review.openstack.org/#/c/350750/ | 03:25 |
oomichi | sc68cal: is it fine to mark it as done now? | 03:25 |
*** tochi has quit IRC | 03:26 | |
oomichi | sc68cal: in most case, cleanup is never ending story though ;) | 03:26 |
*** tqtran has quit IRC | 03:26 | |
*** lucasagomes has quit IRC | 03:28 | |
*** lucasagomes has joined #openstack-qa | 03:29 | |
*** yangyapeng has quit IRC | 03:31 | |
*** itsuugo has quit IRC | 03:32 | |
*** itsuugo has joined #openstack-qa | 03:34 | |
*** lucasagomes has quit IRC | 03:41 | |
*** lucasagomes has joined #openstack-qa | 03:43 | |
*** weshay is now known as weshay_pto | 03:45 | |
*** hoangcx has joined #openstack-qa | 03:46 | |
oomichi | andreaf: I think we can mark https://blueprints.launchpad.net/tempest/+spec/client-manager-refactor as implemented now, right? | 03:47 |
oomichi | andreaf: thanks for doing that, anyways :) | 03:47 |
*** itsuugo has quit IRC | 03:48 | |
*** lucasagomes has quit IRC | 03:48 | |
*** itsuugo has joined #openstack-qa | 03:49 | |
*** lucasagomes has joined #openstack-qa | 03:50 | |
*** yamamoto has joined #openstack-qa | 03:52 | |
*** lucasagomes has quit IRC | 03:54 | |
*** lucasagomes has joined #openstack-qa | 03:56 | |
*** tochi has joined #openstack-qa | 03:58 | |
*** itsuugo has quit IRC | 03:59 | |
*** itsuugo has joined #openstack-qa | 04:00 | |
*** lucasagomes has quit IRC | 04:01 | |
gmann | oomichi: regarding this - https://review.openstack.org/#/c/368326/1 | 04:02 |
gmann | oomichi: actually that is also 1 way to init the existing dir | 04:02 |
gmann | oomichi: can we say this in OR condition | 04:02 |
*** lucasagomes has joined #openstack-qa | 04:04 | |
*** itsuugo has quit IRC | 04:10 | |
*** lucasagomes has quit IRC | 04:12 | |
*** itsuugo has joined #openstack-qa | 04:12 | |
*** lucasagomes has joined #openstack-qa | 04:13 | |
*** hoangcx has left #openstack-qa | 04:16 | |
openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: DNM testing trove with lxd https://review.openstack.org/369810 | 04:16 |
*** lucasagomes has quit IRC | 04:17 | |
*** itsuugo has quit IRC | 04:19 | |
*** lucasagomes has joined #openstack-qa | 04:19 | |
*** itsuugo has joined #openstack-qa | 04:22 | |
*** lucasagomes has quit IRC | 04:24 | |
openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: DNM testing trove with lxd https://review.openstack.org/369810 | 04:25 |
*** lucasagomes has joined #openstack-qa | 04:26 | |
*** gvrangan has joined #openstack-qa | 04:26 | |
*** armax has quit IRC | 04:29 | |
*** Alex_Stef has joined #openstack-qa | 04:30 | |
*** gvrangan has quit IRC | 04:33 | |
*** lucasagomes has quit IRC | 04:33 | |
*** lucasagomes has joined #openstack-qa | 04:35 | |
openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: DNM testing trove with lxd https://review.openstack.org/369810 | 04:38 |
*** gvrangan has joined #openstack-qa | 04:40 | |
*** yangyapeng has joined #openstack-qa | 04:41 | |
*** itsuugo has quit IRC | 04:42 | |
*** yamahata has joined #openstack-qa | 04:43 | |
*** lucasagomes has quit IRC | 04:44 | |
*** itsuugo has joined #openstack-qa | 04:44 | |
*** lucasagomes has joined #openstack-qa | 04:47 | |
*** lucasagomes has quit IRC | 04:51 | |
*** itsuugo has quit IRC | 04:51 | |
*** itsuugo has joined #openstack-qa | 04:53 | |
*** lucasagomes has joined #openstack-qa | 04:53 | |
*** yamamoto has quit IRC | 04:55 | |
*** yamamoto has joined #openstack-qa | 04:57 | |
*** lucasagomes has quit IRC | 04:58 | |
*** lucasagomes has joined #openstack-qa | 04:58 | |
*** lucasagomes has quit IRC | 05:03 | |
*** yangyapeng has quit IRC | 05:04 | |
*** lucasagomes has joined #openstack-qa | 05:04 | |
*** yangyapeng has joined #openstack-qa | 05:06 | |
openstackgerrit | Merged openstack/tempest: Remove non-locking accounts from the config guide https://review.openstack.org/339754 | 05:07 |
*** lucasagomes has quit IRC | 05:09 | |
*** lucasagomes has joined #openstack-qa | 05:11 | |
openstackgerrit | melissaml proposed openstack/tempest: Modify use of assertTrue(A in B) https://review.openstack.org/369824 | 05:13 |
*** armax has joined #openstack-qa | 05:15 | |
*** josdotso has quit IRC | 05:15 | |
*** pgadiya has joined #openstack-qa | 05:15 | |
*** gcheresh has joined #openstack-qa | 05:15 | |
*** phschwartz has quit IRC | 05:15 | |
*** mordred has quit IRC | 05:16 | |
gmann | oomichi: we need this as create, update and show method needs to be fixed for not to do json.load twice - https://review.openstack.org/#/c/369112/2 | 05:16 |
*** AmitGalitz- has joined #openstack-qa | 05:17 | |
*** rossella_ has quit IRC | 05:18 | |
*** phschwartz has joined #openstack-qa | 05:19 | |
*** rossella_ has joined #openstack-qa | 05:19 | |
*** lucasagomes has quit IRC | 05:19 | |
*** mordred has joined #openstack-qa | 05:21 | |
*** lucasagomes has joined #openstack-qa | 05:23 | |
*** k4n0 has joined #openstack-qa | 05:24 | |
*** lucasagomes has quit IRC | 05:27 | |
*** itsuugo has quit IRC | 05:28 | |
*** lucasagomes has joined #openstack-qa | 05:28 | |
*** k4n0 has quit IRC | 05:29 | |
*** itsuugo has joined #openstack-qa | 05:29 | |
*** rcernin has joined #openstack-qa | 05:33 | |
*** lucasagomes has quit IRC | 05:33 | |
*** lucasagomes has joined #openstack-qa | 05:35 | |
*** cody-somerville has quit IRC | 05:35 | |
*** cody-somerville has joined #openstack-qa | 05:38 | |
*** jamielennox|away is now known as jamielennox | 05:40 | |
*** itsuugo has quit IRC | 05:43 | |
*** lucasagomes has quit IRC | 05:45 | |
*** armax has quit IRC | 05:45 | |
*** itsuugo has joined #openstack-qa | 05:45 | |
*** lucasagomes has joined #openstack-qa | 05:47 | |
*** zackf has quit IRC | 05:50 | |
*** lucasagomes has quit IRC | 05:51 | |
*** chandankumar has joined #openstack-qa | 05:52 | |
*** itsuugo has quit IRC | 05:52 | |
*** itsuugo has joined #openstack-qa | 05:53 | |
*** lucasagomes has joined #openstack-qa | 05:55 | |
*** lucasagomes has quit IRC | 05:59 | |
*** lucasagomes has joined #openstack-qa | 06:02 | |
*** artom has quit IRC | 06:02 | |
*** artom has joined #openstack-qa | 06:03 | |
*** itsuugo has quit IRC | 06:07 | |
*** paul-carlton2 has joined #openstack-qa | 06:08 | |
*** lucasagomes has quit IRC | 06:09 | |
*** itsuugo has joined #openstack-qa | 06:09 | |
*** lucasagomes has joined #openstack-qa | 06:10 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Fix credential client to return raw response https://review.openstack.org/369839 | 06:12 |
gmann | oomichi: like this - https://review.openstack.org/#/c/369839/1 | 06:13 |
*** itzikb has joined #openstack-qa | 06:16 | |
*** salv-orlando has quit IRC | 06:18 | |
*** lucasagomes has quit IRC | 06:18 | |
*** sridharg has joined #openstack-qa | 06:19 | |
*** lucasagomes has joined #openstack-qa | 06:20 | |
openstackgerrit | Merged openstack/tempest: Update Available para link of image v1 client https://review.openstack.org/366453 | 06:22 |
*** itsuugo has quit IRC | 06:27 | |
*** edannon has joined #openstack-qa | 06:29 | |
*** itsuugo has joined #openstack-qa | 06:29 | |
*** pcaruana has joined #openstack-qa | 06:32 | |
*** itsuugo has quit IRC | 06:34 | |
*** itsuugo has joined #openstack-qa | 06:35 | |
*** dimtruck is now known as zz_dimtruck | 06:36 | |
*** yamamoto has quit IRC | 06:40 | |
*** itsuugo has quit IRC | 06:40 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/os-testr: Update home-page url https://review.openstack.org/356179 | 06:41 |
*** itsuugo has joined #openstack-qa | 06:41 | |
*** Guest95134 has quit IRC | 06:42 | |
*** gema has joined #openstack-qa | 06:43 | |
*** lucasagomes has quit IRC | 06:44 | |
*** lucasagomes has joined #openstack-qa | 06:46 | |
*** itsuugo has quit IRC | 06:49 | |
*** itsuugo has joined #openstack-qa | 06:50 | |
*** salv-orlando has joined #openstack-qa | 06:50 | |
*** lucasagomes has quit IRC | 06:51 | |
*** lucasagomes has joined #openstack-qa | 06:53 | |
*** EinstCrazy has joined #openstack-qa | 06:55 | |
*** csomerville has joined #openstack-qa | 06:56 | |
*** tesseract- has joined #openstack-qa | 06:57 | |
*** zz_dimtruck is now known as dimtruck | 06:57 | |
*** EinstCra_ has joined #openstack-qa | 06:58 | |
*** EinstCrazy has quit IRC | 06:58 | |
*** cody-somerville has quit IRC | 06:59 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack-dev/devstack: Updated from generate-devstack-plugins-list https://review.openstack.org/356991 | 06:59 |
*** lucasagomes has quit IRC | 07:00 | |
*** lucasagomes has joined #openstack-qa | 07:00 | |
*** csomerville has quit IRC | 07:01 | |
*** EinstCra_ has quit IRC | 07:04 | |
*** lucasagomes has quit IRC | 07:05 | |
*** shardy has joined #openstack-qa | 07:06 | |
*** lucasagomes has joined #openstack-qa | 07:06 | |
*** dimtruck is now known as zz_dimtruck | 07:10 | |
*** Alex_Stef has quit IRC | 07:12 | |
*** lucasagomes has quit IRC | 07:13 | |
*** lucasagomes has joined #openstack-qa | 07:15 | |
*** wendar has quit IRC | 07:15 | |
*** wendar has joined #openstack-qa | 07:15 | |
*** shardy is now known as shardy_mtg | 07:16 | |
*** yaume has joined #openstack-qa | 07:18 | |
*** Ravikiran_K has joined #openstack-qa | 07:19 | |
*** paul-carlton2 has quit IRC | 07:19 | |
*** yamamoto has joined #openstack-qa | 07:19 | |
*** lucasagomes has quit IRC | 07:20 | |
*** paul-carlton2 has joined #openstack-qa | 07:20 | |
*** lucasagomes has joined #openstack-qa | 07:21 | |
*** tochi has quit IRC | 07:22 | |
openstackgerrit | Merged openstack/tempest: Ignore deb-* packaging repos in the plugin list https://review.openstack.org/369753 | 07:22 |
*** markus_z has joined #openstack-qa | 07:24 | |
*** OferBarber has quit IRC | 07:24 | |
*** tqtran has joined #openstack-qa | 07:26 | |
*** EinstCrazy has joined #openstack-qa | 07:27 | |
*** jlanoux has joined #openstack-qa | 07:30 | |
*** tqtran has quit IRC | 07:31 | |
*** lucasagomes has quit IRC | 07:31 | |
*** EinstCrazy has quit IRC | 07:32 | |
*** paul-carlton2 has quit IRC | 07:33 | |
*** lucasagomes has joined #openstack-qa | 07:34 | |
*** zackf has joined #openstack-qa | 07:35 | |
*** cody-somerville has joined #openstack-qa | 07:35 | |
*** cody-somerville has quit IRC | 07:35 | |
*** cody-somerville has joined #openstack-qa | 07:35 | |
*** csomerville has joined #openstack-qa | 07:37 | |
openstackgerrit | guo yunxian proposed openstack/tempest: Remove unnecessary resource_setup() https://review.openstack.org/369871 | 07:37 |
*** abehl has joined #openstack-qa | 07:38 | |
*** jlanoux has quit IRC | 07:38 | |
*** jlanoux has joined #openstack-qa | 07:38 | |
*** lucasagomes has quit IRC | 07:38 | |
*** lucasagomes has joined #openstack-qa | 07:39 | |
*** zackf has quit IRC | 07:39 | |
*** itsuugo has quit IRC | 07:39 | |
*** EinstCrazy has joined #openstack-qa | 07:40 | |
*** zz_dimtruck has quit IRC | 07:40 | |
*** salv-orl_ has joined #openstack-qa | 07:40 | |
*** cody-somerville has quit IRC | 07:41 | |
*** itsuugo has joined #openstack-qa | 07:41 | |
*** EinstCrazy has quit IRC | 07:41 | |
*** jpena|off is now known as jpena | 07:42 | |
*** zz_dimtruck has joined #openstack-qa | 07:42 | |
*** salv-orlando has quit IRC | 07:42 | |
*** zz_dimtruck is now known as dimtruck | 07:43 | |
*** lucasagomes has quit IRC | 07:44 | |
*** paul-carlton2 has joined #openstack-qa | 07:45 | |
*** lucasagomes has joined #openstack-qa | 07:47 | |
*** kbaikov has quit IRC | 07:51 | |
*** lucasagomes has quit IRC | 07:52 | |
*** lucasagomes has joined #openstack-qa | 07:52 | |
*** jdandrea_ has joined #openstack-qa | 07:53 | |
*** jdandrea has quit IRC | 07:56 | |
*** lucasagomes has quit IRC | 07:57 | |
*** lucasagomes has joined #openstack-qa | 07:59 | |
*** itzikb has quit IRC | 08:00 | |
*** artom has quit IRC | 08:00 | |
*** itzikb has joined #openstack-qa | 08:00 | |
openstackgerrit | liwei proposed openstack/tempest: Add get namespace schema test methods https://review.openstack.org/369880 | 08:01 |
*** artom has joined #openstack-qa | 08:01 | |
*** sshnaidm|afk is now known as sshnaidm | 08:01 | |
*** artom has quit IRC | 08:02 | |
*** artom has joined #openstack-qa | 08:02 | |
*** dtantsur is now known as dtantsur|brb | 08:03 | |
*** lucasagomes has quit IRC | 08:03 | |
*** paul-carlton2 has quit IRC | 08:03 | |
*** Alex_Stef has joined #openstack-qa | 08:07 | |
*** lucasagomes has joined #openstack-qa | 08:08 | |
*** gfidente has joined #openstack-qa | 08:09 | |
andreaf | oomichi: yeah, I think it would be nice to close that | 08:20 |
andreaf | oomichi: with the current way aliases are setup we end up always instantiating all tempest clients anyways, but lazy load is true for plugin clients and I think that's the most important part | 08:21 |
andreaf | oomichi: glad I could contribute that change :) | 08:21 |
*** lucasagomes has quit IRC | 08:27 | |
*** itzkb_ has joined #openstack-qa | 08:29 | |
*** itzikb has quit IRC | 08:29 | |
*** paul-carlton2 has joined #openstack-qa | 08:29 | |
*** yamahata has quit IRC | 08:31 | |
*** oomichi has quit IRC | 08:31 | |
*** lucasagomes has joined #openstack-qa | 08:32 | |
*** oomichi has joined #openstack-qa | 08:32 | |
*** lucasagomes has quit IRC | 08:46 | |
*** lucasagomes has joined #openstack-qa | 08:46 | |
*** itsuugo has quit IRC | 08:51 | |
*** itsuugo has joined #openstack-qa | 08:52 | |
*** itsuugo has quit IRC | 09:02 | |
*** itsuugo has joined #openstack-qa | 09:03 | |
*** lucasagomes has quit IRC | 09:05 | |
*** lucasagomes has joined #openstack-qa | 09:08 | |
*** dimtruck is now known as zz_dimtruck | 09:11 | |
*** e0ne has joined #openstack-qa | 09:14 | |
*** lucasagomes has quit IRC | 09:14 | |
*** lucasagomes has joined #openstack-qa | 09:17 | |
*** mvk has quit IRC | 09:19 | |
*** lucasagomes has quit IRC | 09:21 | |
*** lucasagomes_ has joined #openstack-qa | 09:23 | |
*** lucasagomes_ has quit IRC | 09:23 | |
*** cdent has joined #openstack-qa | 09:26 | |
*** tqtran has joined #openstack-qa | 09:28 | |
*** tqtran has quit IRC | 09:32 | |
*** gvrangan has quit IRC | 09:33 | |
*** itsuugo has quit IRC | 09:33 | |
*** larainema has quit IRC | 09:34 | |
*** itsuugo has joined #openstack-qa | 09:35 | |
*** tosky has joined #openstack-qa | 09:41 | |
*** gyx has quit IRC | 09:43 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/tempest: Fix docstrings in Tempest REST client for Ironic https://review.openstack.org/362221 | 09:45 |
*** dtantsur|brb is now known as dtantsur | 09:48 | |
*** mvk has joined #openstack-qa | 09:52 | |
*** yangyapeng has quit IRC | 09:53 | |
*** rfolco has joined #openstack-qa | 09:56 | |
*** pgadiya has quit IRC | 09:57 | |
*** pgadiya has joined #openstack-qa | 09:58 | |
*** yamamoto has quit IRC | 10:12 | |
*** EinstCrazy has joined #openstack-qa | 10:30 | |
*** ramishra has quit IRC | 10:31 | |
*** ramishra has joined #openstack-qa | 10:33 | |
*** gszasz has joined #openstack-qa | 10:36 | |
*** itsuugo has quit IRC | 10:36 | |
*** itsuugo has joined #openstack-qa | 10:37 | |
*** gszasz has quit IRC | 10:38 | |
*** asselin_ has quit IRC | 10:42 | |
*** EinstCrazy has quit IRC | 10:48 | |
*** itsuugo has quit IRC | 10:48 | |
*** itsuugo has joined #openstack-qa | 10:49 | |
*** shardy_mtg is now known as shardy | 10:51 | |
*** gszasz has joined #openstack-qa | 10:52 | |
*** itsuugo has quit IRC | 10:56 | |
*** itsuugo has joined #openstack-qa | 10:58 | |
*** zz_dimtruck is now known as dimtruck | 11:01 | |
*** larainema has joined #openstack-qa | 11:06 | |
*** shardy is now known as shardy_mtg | 11:07 | |
*** yamamoto has joined #openstack-qa | 11:07 | |
*** pgadiya has quit IRC | 11:08 | |
*** sdague has joined #openstack-qa | 11:08 | |
*** pgadiya has joined #openstack-qa | 11:09 | |
*** pgadiya has quit IRC | 11:09 | |
*** itsuugo has quit IRC | 11:10 | |
*** pgadiya has joined #openstack-qa | 11:10 | |
*** dimtruck is now known as zz_dimtruck | 11:11 | |
*** itsuugo has joined #openstack-qa | 11:11 | |
*** jpena is now known as jpena|lunch | 11:17 | |
*** OferBarber has joined #openstack-qa | 11:18 | |
OferBarber | has anyon ever used "create_test_server" | 11:19 |
OferBarber | from the file tempest/common/compute.py ? | 11:19 |
*** itsuugo has quit IRC | 11:24 | |
*** itsuugo has joined #openstack-qa | 11:26 | |
*** tqtran has joined #openstack-qa | 11:29 | |
dmellado | OferBarber: have you checked the imports on any compute tests? | 11:31 |
dmellado | please do follow the pattern on, for example | 11:31 |
*** tuanla_ has quit IRC | 11:32 | |
dmellado | https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_servers.py#L18 | 11:32 |
dmellado | if you back from the imports there you'll find the relationship | 11:32 |
OferBarber | dmellado: great. thanks a lot | 11:34 |
dmellado | np OferBarber | 11:34 |
*** tqtran has quit IRC | 11:34 | |
*** openstackgerrit has quit IRC | 11:34 | |
*** openstackgerrit has joined #openstack-qa | 11:34 | |
OferBarber | dmellado: why did you select the class from test_server.py and not from compute.py ? | 11:38 |
OferBarber | dmellado: how do i know where to select the function from ? | 11:38 |
dmellado | OferBarber if you check the tests over tempest | 11:39 |
dmellado | it's usually split in api/service | 11:39 |
dmellado | and there you've a base class | 11:39 |
dmellado | if you want to write a new test for compute, I'd suggest that you start extending that class directly | 11:39 |
dmellado | and importing the methods from there (unless they've been ported to tempest/lib) | 11:40 |
*** kbaikov has joined #openstack-qa | 11:40 | |
dmellado | which is the tempest stable interface | 11:40 |
dmellado | https://github.com/openstack/tempest/tree/master/tempest/lib | 11:40 |
OferBarber | dmellado: i would like to write a new scenario | 11:41 |
dmellado | OferBarber: did you check existing scenario ones? | 11:42 |
dmellado | you might as well be good to go with the existing methods on the scenario manager | 11:42 |
dmellado | https://github.com/openstack/tempest/blob/master/tempest/scenario/manager.py | 11:43 |
OferBarber | dmellado: yes. someone in my company thinks some scenarios are missing | 11:44 |
OferBarber | i see that i have lots to learn | 11:44 |
OferBarber | should i always prefer the "base.py" file to be used ? | 11:44 |
dmellado | OferBarber: np! please have a look at the scenario manager | 11:44 |
dmellado | also take into consideration that if your scenario test relies too much on an specific project | 11:45 |
dmellado | i.e. neutron 'advanced' capabilities | 11:45 |
dmellado | you might well consider wether to put it directly on the project tempest plugin | 11:45 |
dmellado | instead of in tempest itself | 11:45 |
OferBarber | dmellado: great tips. thank you | 11:47 |
*** dave-mccowan has joined #openstack-qa | 11:47 | |
dmellado | OferBarber: you're welcome ;) | 11:48 |
*** woodster_ has joined #openstack-qa | 11:49 | |
*** itsuugo has quit IRC | 11:54 | |
*** edmondsw has joined #openstack-qa | 11:55 | |
*** markus_zz has joined #openstack-qa | 11:55 | |
*** markus_z has quit IRC | 11:55 | |
*** itsuugo has joined #openstack-qa | 11:56 | |
*** artom has quit IRC | 11:56 | |
*** artom has joined #openstack-qa | 11:57 | |
*** lbeliveau has quit IRC | 11:58 | |
*** lbeliveau has joined #openstack-qa | 12:00 | |
openstackrecheck | Console logs not available after 13:33s for gate-nova-python27-db-ubuntu-xenial 356558,13,07ef7e6 | 12:02 |
*** Kevin_Zheng has quit IRC | 12:06 | |
*** fnaval has joined #openstack-qa | 12:07 | |
*** gouthamr has joined #openstack-qa | 12:10 | |
*** fnaval has quit IRC | 12:12 | |
*** fnaval has joined #openstack-qa | 12:13 | |
*** fnaval has quit IRC | 12:14 | |
*** fnaval has joined #openstack-qa | 12:15 | |
*** yangyapeng has joined #openstack-qa | 12:17 | |
*** AmitGalitz- has quit IRC | 12:18 | |
*** AmitGalitz has joined #openstack-qa | 12:18 | |
*** AmitGalitz has quit IRC | 12:19 | |
*** Pewlet_Hackard has joined #openstack-qa | 12:19 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/tempest: Fix docstrings in Tempest REST client for Ironic https://review.openstack.org/362221 | 12:27 |
*** mriedem has joined #openstack-qa | 12:27 | |
*** pgadiya has quit IRC | 12:28 | |
*** zackf has joined #openstack-qa | 12:29 | |
*** itsuugo has quit IRC | 12:32 | |
*** itsuugo has joined #openstack-qa | 12:33 | |
*** zackf has quit IRC | 12:33 | |
*** itsuugo has quit IRC | 12:38 | |
*** shardy_mtg is now known as shardy | 12:39 | |
*** itsuugo has joined #openstack-qa | 12:39 | |
*** pgadiya has joined #openstack-qa | 12:43 | |
*** pgadiya has quit IRC | 12:43 | |
*** pgadiya has joined #openstack-qa | 12:43 | |
*** zz_dimtruck is now known as dimtruck | 12:50 | |
*** itsuugo has quit IRC | 12:50 | |
*** itsuugo has joined #openstack-qa | 12:52 | |
*** porrua has joined #openstack-qa | 12:54 | |
*** asselin_ has joined #openstack-qa | 12:55 | |
*** jpena|lunch is now known as jpena | 12:55 | |
*** zackf has joined #openstack-qa | 12:56 | |
*** pgadiya has quit IRC | 12:57 | |
*** Kevin_Zheng has joined #openstack-qa | 13:00 | |
*** dkranz has joined #openstack-qa | 13:01 | |
*** zackf has quit IRC | 13:01 | |
openstackgerrit | Jens Rosenboom proposed openstack-dev/devstack: WIP: Test fix for identity-v3 failure https://review.openstack.org/369675 | 13:10 |
*** Pewlet_Hackard has quit IRC | 13:10 | |
*** rossella_ has quit IRC | 13:18 | |
*** rossella_ has joined #openstack-qa | 13:19 | |
*** shardy has quit IRC | 13:23 | |
*** rbak has joined #openstack-qa | 13:23 | |
*** mriedem has quit IRC | 13:25 | |
*** tqtran has joined #openstack-qa | 13:31 | |
*** markus_zz is now known as markus_z | 13:32 | |
*** tqtran has quit IRC | 13:35 | |
*** artom has quit IRC | 13:40 | |
*** salv-orlando has joined #openstack-qa | 13:40 | |
*** artom has joined #openstack-qa | 13:41 | |
*** artom has quit IRC | 13:42 | |
*** yamamoto has quit IRC | 13:42 | |
*** itsuugo has quit IRC | 13:42 | |
*** artom has joined #openstack-qa | 13:42 | |
*** markvoelker has joined #openstack-qa | 13:43 | |
*** gcheresh has quit IRC | 13:43 | |
*** salv-orl_ has quit IRC | 13:43 | |
*** yamamoto has joined #openstack-qa | 13:43 | |
*** itsuugo has joined #openstack-qa | 13:44 | |
*** yamamoto has quit IRC | 13:44 | |
openstackgerrit | Sean Dague proposed openstack-dev/devstack: force en_US.UTF-8 during devstack runs https://review.openstack.org/370132 | 13:48 |
ylobankov | oomichi, afazekas: are you around? | 13:48 |
* afazekas o/ | 13:48 | |
*** mtanino has joined #openstack-qa | 13:49 | |
ylobankov | afazekas: could you please read the comments here https://review.openstack.org/#/c/322118/16//COMMIT_MSG ? What do you think about it? | 13:49 |
*** itsuugo has quit IRC | 13:50 | |
ylobankov | afazekas: I am ok to delete the test, but oomichi would like to keep it for more test coverage. | 13:51 |
*** itsuugo has joined #openstack-qa | 13:52 | |
*** gamado has joined #openstack-qa | 13:52 | |
*** gamado has quit IRC | 13:52 | |
*** ddieterly has joined #openstack-qa | 13:52 | |
*** mmedvede has joined #openstack-qa | 13:53 | |
*** fnaval has quit IRC | 13:54 | |
afazekas | ylobankov, Am ok with both skipping and/or deleting | 13:54 |
*** shardy has joined #openstack-qa | 13:55 | |
ylobankov | afazekas: Thanks. | 13:56 |
*** yamamoto has joined #openstack-qa | 13:58 | |
*** fnaval has joined #openstack-qa | 13:59 | |
*** akshai has joined #openstack-qa | 14:01 | |
*** Swami has joined #openstack-qa | 14:03 | |
*** Swami has quit IRC | 14:03 | |
*** Swami has joined #openstack-qa | 14:03 | |
*** rodrigods has quit IRC | 14:06 | |
*** rodrigods has joined #openstack-qa | 14:06 | |
*** dustins has joined #openstack-qa | 14:07 | |
*** yamamoto has quit IRC | 14:11 | |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/tempest: WIP: Nova: test live migration with serial console https://review.openstack.org/346815 | 14:11 |
*** mriedem has joined #openstack-qa | 14:11 | |
*** mriedem has quit IRC | 14:11 | |
*** Daviey_ is now known as Daviey | 14:12 | |
*** artom has quit IRC | 14:14 | |
*** artom has joined #openstack-qa | 14:16 | |
*** artom has quit IRC | 14:16 | |
*** artom has joined #openstack-qa | 14:17 | |
*** dtantsur is now known as dtantsur|mtg | 14:17 | |
frickler | so https://review.openstack.org/369675 could succeed as a workaround for https://bugs.launchpad.net/glance-store/+bug/1620999 | 14:18 |
openstack | Launchpad bug 1620999 in glance_store "glance_store ignores user_domain_name and project_domain_name settings" [Undecided,New] | 14:18 |
*** rlrossit has joined #openstack-qa | 14:18 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/365633 failed because of: gate-grenade-dsvm-neutron-ubuntu-trusty: https://bugs.launchpad.net/bugs/1449136, gate-grenade-dsvm-ubuntu-trusty: unrecognized error, gate-tempest-dsvm-cells-ubuntu-xenial: unrecognized error, gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial: https://bugs.launchpad.net/bugs/1449136 | 14:18 |
openstack | Launchpad bug 1449136 in OpenStack-Gate "OpenStack pypi mirrors disconnecting connections" [Undecided,New] | 14:18 |
*** raildo has joined #openstack-qa | 14:22 | |
*** Alex_Stef has quit IRC | 14:24 | |
*** mriedem has joined #openstack-qa | 14:25 | |
*** zackf has joined #openstack-qa | 14:26 | |
*** yamahata has joined #openstack-qa | 14:29 | |
*** Swami_ has joined #openstack-qa | 14:35 | |
*** rfolco has quit IRC | 14:37 | |
*** Swami has quit IRC | 14:38 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/365938 failed because of: gate-tempest-dsvm-cells-ubuntu-xenial: https://bugs.launchpad.net/bugs/1449136, gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial: unrecognized error | 14:38 |
openstack | Launchpad bug 1449136 in OpenStack-Gate "OpenStack pypi mirrors disconnecting connections" [Undecided,New] | 14:38 |
*** Swami_ has quit IRC | 14:39 | |
*** pglass has joined #openstack-qa | 14:44 | |
*** amotoki has joined #openstack-qa | 14:45 | |
*** raildo has quit IRC | 14:47 | |
*** raildo has joined #openstack-qa | 14:47 | |
*** Swami has joined #openstack-qa | 14:48 | |
*** erlon has joined #openstack-qa | 14:50 | |
*** xinli has joined #openstack-qa | 14:52 | |
*** ddieterly is now known as ddieterly[away] | 14:54 | |
*** itsuugo has quit IRC | 14:54 | |
*** itsuugo has joined #openstack-qa | 14:56 | |
*** rfolco has joined #openstack-qa | 14:58 | |
*** ddieterly[away] is now known as ddieterly | 14:59 | |
*** akshai has quit IRC | 14:59 | |
*** pglass has quit IRC | 15:00 | |
*** ddieterly has quit IRC | 15:01 | |
*** edannon has quit IRC | 15:01 | |
*** akshai has joined #openstack-qa | 15:01 | |
*** david-lyle_ has joined #openstack-qa | 15:06 | |
*** david-lyle_ has quit IRC | 15:06 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add subunit2sql CLI option to use run wall time https://review.openstack.org/370193 | 15:07 |
mtreinish | cdent: ^^^ that's the patch we were talking about a while ago to get started on adding run time graphs for views that involve entire runs | 15:07 |
*** jlanoux has quit IRC | 15:08 | |
cdent | mtreinish: oh, awesome. I'm happy to see that. Sorry I didn't get to it myself. I've been under a sea of placement api for a very long time | 15:09 |
* cdent sticks it in his review queue | 15:09 | |
mtreinish | cdent: sure, no worries. I figured that's what was going on, I've been seeing all that stuff moving along. That's definitely a priority, especially with a release right around the corner | 15:10 |
*** yamamoto has joined #openstack-qa | 15:11 | |
*** armax has joined #openstack-qa | 15:12 | |
*** ddieterly has joined #openstack-qa | 15:13 | |
cdent | I fear I will be under this sea for THE REST OF MY LIFE | 15:14 |
openstackgerrit | Lance Bragstad proposed openstack/tempest: Allow for wait in _restore_password https://review.openstack.org/367595 | 15:14 |
*** gvrangan has joined #openstack-qa | 15:14 | |
mtreinish | lbragstad: ^^^ more magic sleeps.... | 15:15 |
lbragstad | mtreinish i think they are the same as before - but the _restore_password method is new | 15:16 |
*** ddieterly has quit IRC | 15:17 | |
*** ddieterly has joined #openstack-qa | 15:18 | |
mtreinish | lbragstad: you're not worried that a user can change their password and still access stuff with the old password for a second after keystone says it's been changed successfully | 15:19 |
mtreinish | isn't that enough time to change the password again? | 15:19 |
*** itzkb_ has quit IRC | 15:19 | |
lbragstad | mtreinish it will actually be the opposite - if a user changes their password, keystone will emit a 401 since the token will make the revocation event from the password change | 15:21 |
lbragstad | s/make/match/ | 15:21 |
*** abehl has quit IRC | 15:25 | |
*** yamamoto has quit IRC | 15:25 | |
*** Kevin_Zheng has quit IRC | 15:26 | |
*** sridharg has quit IRC | 15:29 | |
*** vhosakot has joined #openstack-qa | 15:30 | |
*** ddieterly is now known as ddieterly[away] | 15:31 | |
*** yamamoto has joined #openstack-qa | 15:33 | |
*** rcernin has quit IRC | 15:34 | |
*** dtantsur|mtg is now known as dtantsur | 15:35 | |
*** jlanoux has joined #openstack-qa | 15:36 | |
*** jlanoux has quit IRC | 15:36 | |
*** jlanoux has joined #openstack-qa | 15:38 | |
*** pcaruana has quit IRC | 15:40 | |
openstackgerrit | Sean Dague proposed openstack-dev/devstack: force en_US.UTF-8 during devstack runs https://review.openstack.org/370132 | 15:41 |
*** ddieterly[away] has quit IRC | 15:41 | |
*** rlrossit has quit IRC | 15:43 | |
*** shardy has quit IRC | 15:43 | |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Skip test_add_list_remove_router_on_l3_agent because of the bug 1590049 https://review.openstack.org/370236 | 15:46 |
openstack | bug 1590049 in tempest "test_add_list_remove_router_on_l3_agent may fail on envs where there is >= two L3 agents and DVR enabled" [Undecided,In progress] https://launchpad.net/bugs/1590049 - Assigned to Yaroslav Lobankov (ylobankov) | 15:46 |
openstackrecheck | openstack/nova change: https://review.openstack.org/363926 failed because of: gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial: https://bugs.launchpad.net/bugs/1475333 | 15:48 |
openstack | Launchpad bug 1475333 in devstack "gate-tempest-dsvm-full-ceph fails with Insufficient free space for volume creation on host ubuntu@ceph#ceph (requested / avail): 1/0.0" [High,Confirmed] | 15:48 |
*** ddieterly has joined #openstack-qa | 15:50 | |
*** mriedem has quit IRC | 15:51 | |
*** mriedem1 has joined #openstack-qa | 15:51 | |
*** sshnaidm is now known as sshnaidm|afk | 15:51 | |
*** pglass has joined #openstack-qa | 16:01 | |
*** gyee has joined #openstack-qa | 16:01 | |
*** mriedem1 is now known as mriedem | 16:08 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add run_metadata filtering to get run_times function https://review.openstack.org/370255 | 16:08 |
*** krtaylor has quit IRC | 16:10 | |
*** jpena is now known as jpena|off | 16:11 | |
*** ddieterly has quit IRC | 16:16 | |
*** chandankumar has quit IRC | 16:17 | |
openstackgerrit | Graham Hayes proposed openstack-infra/elastic-recheck: Add query for bug 1623576 https://review.openstack.org/370271 | 16:18 |
openstack | bug 1623576 in Designate "test_zone_create_propagates_to_nameservers times out before zone is fully propagated" [Medium,Triaged] https://launchpad.net/bugs/1623576 | 16:18 |
*** markus_z has quit IRC | 16:20 | |
*** Apoorva has joined #openstack-qa | 16:21 | |
*** lucasagomes has joined #openstack-qa | 16:23 | |
*** yamamoto has quit IRC | 16:23 | |
*** yamamoto has joined #openstack-qa | 16:24 | |
*** yamamoto has quit IRC | 16:25 | |
*** amotoki has quit IRC | 16:26 | |
*** ddieterly has joined #openstack-qa | 16:27 | |
*** porrua has quit IRC | 16:29 | |
*** cdent has quit IRC | 16:32 | |
*** jlanoux has quit IRC | 16:32 | |
*** rcernin has joined #openstack-qa | 16:34 | |
*** rlrossit has joined #openstack-qa | 16:35 | |
*** tesseract- has quit IRC | 16:35 | |
*** dtantsur is now known as dtantsur|afk | 16:36 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add new DB API function to get grouped run_time time series https://review.openstack.org/370288 | 16:38 |
openstackgerrit | Mikhail S Medvedev proposed openstack-dev/devstack: Set default properties for Power: virtual console and scsi driver https://review.openstack.org/370289 | 16:38 |
mtreinish | cdent ^^^ that's part 2, the last part will be to use that in the openstack-health api and then graph the data | 16:38 |
*** yamamoto has joined #openstack-qa | 16:39 | |
*** ddieterly is now known as ddieterly[away] | 16:39 | |
mtreinish | masayukig: ^^^ do you want to work on the openstack-health bits on top of that db api change? | 16:41 |
mtreinish | if not I can handle it | 16:42 |
mtreinish | masayukig: it shouldn't be too hard, just add a function call to the existing get run counts endpoint that will run that query (and group by build name) then putting the data in a pandas dataframe run the resampling and stats functions convert it to the output json and add that to the response payload | 16:44 |
mtreinish | there's a lot of examples of doing that in the api code | 16:44 |
*** yamahata has quit IRC | 16:44 | |
*** Swami has quit IRC | 16:44 | |
mtreinish | masayukig: oh, and of course the js graphing half to use the data :) | 16:44 |
*** ddieterly[away] is now known as ddieterly | 16:46 | |
*** sshnaidm|afk is now known as sshnaidm | 16:47 | |
*** mvk has quit IRC | 16:47 | |
*** e0ne has quit IRC | 16:49 | |
*** gszasz has quit IRC | 16:51 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/364829 failed gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial in the gate queue with an unrecognized error | 16:54 |
*** tosky has quit IRC | 16:56 | |
*** krtaylor has joined #openstack-qa | 16:57 | |
mriedem | mtreinish: https://review.openstack.org/370303 | 16:57 |
mriedem | oh no | 16:57 |
mriedem | nvm | 16:57 |
*** porrua has joined #openstack-qa | 16:58 | |
mriedem | wrong bug :) | 16:58 |
mtreinish | mriedem: heh, ok | 16:59 |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Remove query for fixed bug 1616498 https://review.openstack.org/370304 | 16:59 |
openstack | bug 1615710 in neutron "duplicate for #1616498 test_server_multi_create_auto_allocate failing with "Failed to allocate the network(s), not rescheduling"" [Critical,Fix released] https://launchpad.net/bugs/1615710 - Assigned to Kevin Benton (kevinbenton) | 16:59 |
mriedem | that ^ | 16:59 |
mtreinish | mriedem: +A | 17:00 |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add new DB API function to get grouped run_time time series https://review.openstack.org/370288 | 17:03 |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add run_metadata filtering to get run_times function https://review.openstack.org/370255 | 17:03 |
*** Kiall_ is now known as Kiall | 17:03 | |
*** ddieterly is now known as ddieterly[away] | 17:06 | |
oomichi | gmann: ok, https://review.openstack.org/#/c/368326 is dropped | 17:06 |
oomichi | gmann: about https://review.openstack.org/#/c/369112: if doing that, testers cannot know this weird API response on tests side | 17:07 |
oomichi | gmann: how about having some helper method in common? | 17:08 |
openstackgerrit | Pradeep Kumar proposed openstack/tempest: Adding testcases for metadata def properties https://review.openstack.org/370315 | 17:09 |
*** Swami has joined #openstack-qa | 17:10 | |
oomichi | andreaf: oh, cool. I am happy to change the status of the LP to 'implemented' :-) | 17:11 |
*** yamamoto has quit IRC | 17:11 | |
oomichi | mtreinish: how about the status of https://blueprints.launchpad.net/tempest/+spec/tempest-run-cmd ? | 17:15 |
oomichi | mtreinish: can we change the status to "implemented" after https://review.openstack.org/#/c/348511/ ? | 17:15 |
openstackgerrit | Merged openstack-infra/elastic-recheck: Remove query for fixed bug 1616498 https://review.openstack.org/370304 | 17:15 |
openstack | bug 1615710 in neutron "duplicate for #1616498 test_server_multi_create_auto_allocate failing with "Failed to allocate the network(s), not rescheduling"" [Critical,Fix released] https://launchpad.net/bugs/1615710 - Assigned to Kevin Benton (kevinbenton) | 17:15 |
oomichi | mtreinish: I feel that is already good for users | 17:15 |
mtreinish | oomichi: no, after: https://review.openstack.org/#/c/355666/ | 17:16 |
oomichi | mtreinish: heh, that is "super wip" now | 17:16 |
*** chandankumar has joined #openstack-qa | 17:16 | |
mtreinish | we want to *dogfood* the new workflow completly before we say it's done :) | 17:16 |
*** ddieterly[away] has quit IRC | 17:16 | |
mtreinish | right now there is no usage of workspaces at all anywhere | 17:16 |
oomichi | haha | 17:16 |
mtreinish | which is an important part of the spec | 17:16 |
mtreinish | oomichi: I put that on the etherpad for next week already | 17:17 |
oomichi | OK, you are managing this strictly :) | 17:17 |
oomichi | mtreinish: awesome, thanks | 17:17 |
mtreinish | I'm hoping someone can help me and take over the super wip patch. Since I'm gonna be split between like 5 things at the sprint | 17:18 |
*** rossella_ has quit IRC | 17:18 | |
oomichi | mtreinish: you can work 5 times by comparing comeone in the meetup | 17:18 |
*** rossella_ has joined #openstack-qa | 17:19 | |
oomichi | ok, I put a comment on the bp of LP, which patch is the goal of that | 17:19 |
*** Suyi_ has joined #openstack-qa | 17:20 | |
*** yamamoto has joined #openstack-qa | 17:24 | |
*** yamamoto has quit IRC | 17:24 | |
*** lucasagomes is now known as lucas-dinner | 17:25 | |
sdague | are there actually any utf8 testing tempest patches besides the server rename? | 17:25 |
*** ddieterly has joined #openstack-qa | 17:30 | |
*** openstackrecheck has quit IRC | 17:30 | |
*** openstackrecheck has joined #openstack-qa | 17:30 | |
oomichi | sdague: humm, maybe no. IIUC, old tempest had the code, but that didn't work at all. Then we removed that | 17:33 |
oomichi | sdague: let me check the history | 17:33 |
*** tqtran has joined #openstack-qa | 17:34 | |
*** yaume has quit IRC | 17:34 | |
openstackgerrit | Jens Rosenboom proposed openstack-dev/devstack: Work around glance ignoring domain names in config https://review.openstack.org/369675 | 17:35 |
oomichi | sdague: tempest has utf-8 test for nova images api: https://review.openstack.org/#/c/122354/ | 17:38 |
oomichi | sdague: but not for the serve rename. do we need that now? | 17:38 |
*** tqtran has quit IRC | 17:38 | |
*** yamahata has joined #openstack-qa | 17:38 | |
sdague | oomichi: well, it seems like we should have some bucket of tests for utf8. Or make random name put a utf8 name into every name field by default. There are definitely some issues on logging this stuff in the server | 17:40 |
*** ddieterly is now known as ddieterly[away] | 17:40 | |
oomichi | sdague: how about https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_servers.py#L98 ? | 17:41 |
sdague | oomichi: right, that call triggers an odd stack trace now in the gate, which is non fatal, but odd | 17:41 |
sdague | for exactly this reason | 17:41 |
oomichi | sdague: oh, that seems interesting. can I see the actual log of the gate? | 17:43 |
sdague | http://logs.openstack.org/32/370132/2/check/gate-tempest-dsvm-full-ubuntu-xenial/9288e78/logs/screen-n-cpu.txt.gz#_2016-09-14_16_15_12_752 | 17:43 |
oomichi | sdague: oh, https://review.openstack.org/#/c/370132 seems cool | 17:49 |
sdague | oomichi: yeh... though it's not having the intended effect | 17:49 |
sdague | which I'm scratching my head on | 17:49 |
openstackrecheck | openstack/nova change: https://review.openstack.org/314811 failed gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial in the gate queue with an unrecognized error | 17:50 |
*** ddieterly[away] has quit IRC | 17:50 | |
oomichi | sdague: heh, but this situation seems your expectation(detecting unexpected issues) from the commit msg | 17:51 |
*** tqtran has joined #openstack-qa | 17:54 | |
sdague | well, I'm still getting an ascii decode error on the compute service | 17:56 |
sdague | which confuses me | 17:56 |
*** clu_ has joined #openstack-qa | 17:56 | |
sdague | because it should be running in a utf8 locale | 17:56 |
*** hongbin has joined #openstack-qa | 18:03 | |
*** mvk has joined #openstack-qa | 18:03 | |
*** itsuugo has quit IRC | 18:19 | |
*** itsuugo has joined #openstack-qa | 18:20 | |
lbeliveau | mriedem: oomichi: me and moshele have been trying to get this nova migration tempest merged, can you please help (Mellanox CI has tested it successfully) https://review.openstack.org/#/c/339230/ | 18:20 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/qa-specs: Remove stress-api-tracking https://review.openstack.org/370357 | 18:20 |
*** yamamoto has joined #openstack-qa | 18:25 | |
oomichi | lbeliveau: thanks, I prefer the purpose of the test itself. That will be helpful for productions. | 18:28 |
oomichi | lbeliveau: before adding it as the scenario test, it seems good to add it as an API test because tempest doesn't have it on API test even but it has the client code | 18:30 |
oomichi | lbeliveau: scenario test tends to be more complex than API test, and it is easy to debug the problem when we face if implementing it as API test | 18:31 |
mriedem | lbeliveau: comments inline | 18:31 |
lbeliveau | oomichi: thanks, just to be clear, we should have both an API test and a scenario test ? | 18:32 |
mriedem | oomichi: this does feel like a scenario test to me, it's booting a server and setting up networking with a floating IP, then verifying that networking is ok after a cold migration | 18:32 |
mriedem | if it were an api test, i guess it would be gated on CONF.run_validation? | 18:32 |
oomichi | mriedem: yeah, right. current test case fits to scenario test. but Tempest doesn't have more simple test in API test | 18:33 |
*** artom has quit IRC | 18:33 | |
oomichi | mriedem: I just want to see the API test is stable on the gate before adding the complex test | 18:34 |
*** yamamoto has quit IRC | 18:34 | |
*** artom has joined #openstack-qa | 18:35 | |
mugsie | can I get some e-r cores to review https://review.openstack.org/#/c/370271/ please? | 18:35 |
lbeliveau | mriedem: oomichi: thanks ! I'll work on adding an API test | 18:36 |
mriedem | mugsie: on it | 18:38 |
mugsie | mriedem: ty | 18:39 |
mriedem | mugsie: comments inline | 18:43 |
openstackgerrit | Jens Rosenboom proposed openstack-dev/devstack: DNM: Dummy patch to test keystone fernet patch https://review.openstack.org/370369 | 18:45 |
openstackgerrit | Graham Hayes proposed openstack-infra/elastic-recheck: Add query for bug 1623576 https://review.openstack.org/370271 | 18:47 |
openstack | bug 1623576 in Designate "test_zone_create_propagates_to_nameservers times out before zone is fully propagated" [Medium,Triaged] https://launchpad.net/bugs/1623576 | 18:47 |
mugsie | mriedem: I am getting 4 jobs - should I move them to a nested OR block? | 18:48 |
xinli | rodrigods: Could you please help to review the code for 285640? | 18:48 |
xinli | rodrigods: the link: https://review.openstack.org/#/c/285640/ | 18:49 |
openstackgerrit | Graham Hayes proposed openstack-infra/elastic-recheck: Add query for bug 1623576 https://review.openstack.org/370271 | 18:50 |
openstack | bug 1623576 in Designate "test_zone_create_propagates_to_nameservers times out before zone is fully propagated" [Medium,Triaged] https://launchpad.net/bugs/1623576 | 18:50 |
mriedem | mugsie: oh hrm well at least 2 of those are non-voting | 18:51 |
mriedem | which e-r is going to filter out anyway | 18:51 |
mugsie | yeah - I relasied that as I added them | 18:52 |
mriedem | you can filter in kibana with AND voting:1 | 18:52 |
mriedem | which leaves us with 2 | 18:52 |
mugsie | yeap, I just added them there, and removed the test ids | 18:53 |
mriedem | +W | 18:54 |
mugsie | thanks for your help :) | 18:55 |
*** sshnaidm is now known as sshnaidm|afk | 18:55 | |
openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for bug 1623576 https://review.openstack.org/370271 | 18:59 |
openstack | bug 1623576 in Designate "test_zone_create_propagates_to_nameservers times out before zone is fully propagated" [Medium,Triaged] https://launchpad.net/bugs/1623576 | 19:00 |
mriedem | mtreinish: are you super duper happy with this one yet? https://review.openstack.org/#/c/305120/ | 19:02 |
*** Pewlet_Hackard has joined #openstack-qa | 19:06 | |
*** pcaruana has joined #openstack-qa | 19:07 | |
*** chandankumar has quit IRC | 19:15 | |
*** r-mibu has quit IRC | 19:16 | |
mriedem | shouldn't this say create a local.conf under devstack/ http://docs.openstack.org/developer/devstack/#create-a-local-conf ? | 19:16 |
*** r-mibu has joined #openstack-qa | 19:16 | |
*** ddieterly has joined #openstack-qa | 19:20 | |
openstackgerrit | Merged openstack/tempest: Fix the docstring of _default_security_group https://review.openstack.org/369751 | 19:23 |
openstackgerrit | Merged openstack/tempest: Remove unnecessary resource_setup() https://review.openstack.org/369871 | 19:23 |
*** cdent has joined #openstack-qa | 19:25 | |
*** akshai has quit IRC | 19:28 | |
*** gvrangan has quit IRC | 19:31 | |
*** Apoorva_ has joined #openstack-qa | 19:33 | |
*** grumble has joined #openstack-qa | 19:36 | |
*** csomerville has quit IRC | 19:36 | |
*** Apoorva has quit IRC | 19:36 | |
*** csomerville has joined #openstack-qa | 19:37 | |
*** cody-somerville has joined #openstack-qa | 19:38 | |
*** dkranz has quit IRC | 19:39 | |
*** dkranz has joined #openstack-qa | 19:40 | |
*** salv-orl_ has joined #openstack-qa | 19:40 | |
*** csomerville has quit IRC | 19:41 | |
*** salv-orlando has quit IRC | 19:43 | |
*** akshai has joined #openstack-qa | 19:45 | |
*** akshai_ has joined #openstack-qa | 19:46 | |
*** Apoorva_ has quit IRC | 19:47 | |
*** Apoorva has joined #openstack-qa | 19:47 | |
*** xinli has quit IRC | 19:48 | |
*** porrua has quit IRC | 19:48 | |
*** akshai has quit IRC | 19:49 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/363926 failed gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial in the gate queue with an unrecognized error | 19:50 |
*** pcaruana has quit IRC | 20:00 | |
*** Pewlet_Hackard has quit IRC | 20:02 | |
*** gouthamr has quit IRC | 20:07 | |
*** Ravikiran_K has quit IRC | 20:07 | |
*** Apoorva has quit IRC | 20:15 | |
*** openstackrecheck has quit IRC | 20:15 | |
*** Apoorva has joined #openstack-qa | 20:16 | |
*** openstackrecheck has joined #openstack-qa | 20:16 | |
*** cdent has quit IRC | 20:16 | |
*** gvrangan has joined #openstack-qa | 20:23 | |
erlon | hey guys, mriedem, I needed something like a @ddt in tempest, is it possible? | 20:26 |
mriedem | ddt in tempest was brought up awhile back and nacked i think | 20:27 |
mriedem | since you can use testscenarios | 20:27 |
mtreinish | mriedem: well not really that either. We're pretty -1 heavy against any dynamic test creation at this point. It makes debugging very difficult when any of those thigns are used | 20:28 |
*** e0ne has joined #openstack-qa | 20:30 | |
mriedem | right, remember, no PhD in testscenarios (tm) | 20:32 |
mriedem | i thought about getting one from trump univsersity though | 20:32 |
mriedem | *university | 20:32 |
sdague | no, you spelled it right the first time | 20:32 |
mriedem | touche | 20:33 |
erlon | mtreinish: mriedem: hmm, yeah, I was already guessing that tempest guys would have some resistance about that | 20:33 |
mriedem | those guys are terrible | 20:33 |
erlon | mriedem: mtreinish: what I need to do is, Cinder has multiple openSource backends, let say A, B, C, I need to run the same tests against all possible combinations | 20:34 |
erlon | A <-> B, A <-> C, B <-> C | 20:35 |
erlon | mriedem: mtreinish: I could write this inside the test, but that could make the test to be very very long | 20:36 |
*** rcernin has quit IRC | 20:36 | |
*** gfidente has quit IRC | 20:36 | |
mtreinish | erlon: well that use case is the typical one for something like testscenarios or ddt. It's just in practice it makes debugging things when they fail much more difficult | 20:38 |
mtreinish | erlon: don't we already have multibackend cinder tests in tempest? | 20:38 |
erlon | mtreinish: the test is blocked waiting a gate job, but it only tests 2 backends | 20:40 |
*** rcernin has joined #openstack-qa | 20:40 | |
erlon | mtreinish: https://review.openstack.org/#/c/195443 | 20:40 |
*** rcernin has quit IRC | 20:41 | |
erlon | if I pass CONF.volume.backend_names = 'lvm, ceph, gluster, bdd, nfs', for example, it will only test lvm x ceph | 20:41 |
*** rcernin has joined #openstack-qa | 20:41 | |
*** rcernin has quit IRC | 20:42 | |
*** rcernin has joined #openstack-qa | 20:42 | |
*** gouthamr has joined #openstack-qa | 20:44 | |
mtreinish | erlon: so isn't that a bug in the test as proposed (assuming it's supposed to check all the permutations) | 20:44 |
mtreinish | you'd just need 2 for loops right? | 20:45 |
*** e0ne has quit IRC | 20:45 | |
erlon | mtreinish: yes, that's is the option I mentioned, to loop over the backends, but that could take a long time, and will be also hard to debug as you won't know what backends failed | 20:49 |
erlon | mtreinish: so, if was possible to generate tests, we could have test_available_volume_retype_with_migration_a_vs_b, ..a_vs_c,etc | 20:50 |
erlon | mtreinish: which would be much easier to debug | 20:50 |
*** akshai_ has quit IRC | 20:51 | |
erlon | mtreinish: what is the issue you mentioned about debugging tests generated like that? | 20:51 |
mtreinish | erlon: well to start either you have ddt which results in a subunit test id that's almost always unparseable. Or testscenarios which is confusing to most people (we have it used someplaces today and very few people can follow it, it needs to be removed) | 20:53 |
*** gvrangan has quit IRC | 20:53 | |
mtreinish | erlon: but the fundamental issue is that it needs to be explicit and obvious what is going on. Any type of test generation relies on flushing out the internal details behind the covers to make more compact code | 20:53 |
mtreinish | which is optimizing for the wrong problem space | 20:54 |
mtreinish | especially for a test suite like tempest which is running with real resources and can take a long time (unit tests might be a different story though) | 20:54 |
erlon | mtreinish: testscenarios can use string to identify the tests? | 20:55 |
erlon | mtreinish: I understand, but moving the test complexity inside the test it self, like using the loops as you suggested will have the same problem of ussing hardware resources, but does not solve the debbugability, which IMO would be much easier to use @ddt (or something that provided string identification of tests) | 20:58 |
erlon | mtreinish: surely that shouldn't be the kind of thing you do in lots of tests, just some special cases like this | 20:59 |
mtreinish | erlon: yes, that's normally how testscenarios is used: https://github.com/openstack-infra/subunit2sql/blob/master/subunit2sql/tests/db/test_api.py#L27-L36 | 20:59 |
mtreinish | erlon: we have it in special cases now and we're removing it all, because it makes debugging harder. I think any approach where it's viewed as the best solution (which your case is) is maybe not something we should do inside tempest | 21:00 |
mtreinish | erlon: because fundamentally you want to compat test migrations between all the open source backends right? | 21:00 |
mtreinish | erlon: which is completely achievable with a bash script with a for loop updating a line in the tempest.conf and rerunning the one test | 21:01 |
erlon | mtreinish: and the job running this test will be called solo, without all other tempest, to make the job shorter | 21:01 |
mtreinish | erlon: oh, if you're splitting it out into a seperate job then why do we need to bake this logic inside tempest? | 21:01 |
erlon | mtreinish: exactly, I don't think is possible to do that in the gate scripts | 21:03 |
mtreinish | why not? | 21:03 |
*** xinli has joined #openstack-qa | 21:03 | |
erlon | mtreinish: haven't tried actually, my first option was to implement it in tempest | 21:03 |
*** gyee has quit IRC | 21:04 | |
mtreinish | running a test through all the values in a config matrix is something that really shouldn't be done in the test itself. Doing it as part of the runner seems like a better fit, since the test code is the same it's just different config values. (we might eventually want to investigate adding that logic into tempest run, but it doesn't exist now) | 21:04 |
erlon | mtreinish: so, I needed to know if that is an option, and if not, what suggestions you guys would give | 21:04 |
mtreinish | erlon: this is the same use case that andreaf and dwalleck have brought up for running scenario tests with different flavors and/or images (to verify all the valid combinations work) | 21:05 |
*** ddieterly is now known as ddieterly[away] | 21:05 | |
mtreinish | which was a case that testscenarios was used for befor in tempest but was removed for the reasons I outlined before | 21:05 |
mtreinish | so you might want to talk to them about a having a generic framework for that in the future | 21:06 |
erlon | mtreinish: ill need to investigate further the gate scripts to see how, if is possible, and if folks there does not have any problem with that too | 21:06 |
erlon | mtreinish: mhm | 21:06 |
mtreinish | erlon: you just use a gate hook to define a custom job that runs tempest run --regex multibackend (or whatever regex is appropriate) in a loop and do an iniset or 2 between each iteration of the loop | 21:07 |
mtreinish | erlon: here's an example: https://github.com/openstack/nova/blob/master/nova/tests/live_migration/hooks/utils.sh#L3 of using a hook | 21:08 |
mtreinish | erlon: one thing to watch is to make sure you combine all the subunit output from the multiple runs to make sure you track each iteration of the loop | 21:08 |
erlon | mtreinish: ok, ill try that, are those hooks called by the gate scripts? | 21:09 |
mtreinish | erlon: yes: https://github.com/openstack-infra/project-config/blob/51b85065d342ebc8c53df85ca467cec633b2c955/jenkins/jobs/devstack-gate.yaml#L1434 | 21:09 |
erlon | mtreinish: that would not work if I use concurrency will? | 21:10 |
mtreinish | you wouldn't have multiple workers if you were running a single test class with testscenarios or ddt. Tempest only parallelizes itself at the class level not the test method | 21:11 |
erlon | mtreinish: mhm | 21:12 |
erlon | mtreinish: if this is a pre_test_hook, won't the gate scripts call the actual tests again later? | 21:13 |
mtreinish | erlon: you could cheat and backround each 'tempest run' command and run multiple in parallel that way | 21:13 |
mtreinish | erlon: it's a post test hook and you set TEMPEST_NOTESTS in the job definition so devstack-gate doesn't run the tests | 21:13 |
erlon | mtreinish: hmmmm | 21:15 |
erlon | mtreinish: great I think that's enough to diggest for a while :) | 21:15 |
erlon | mtreinish: thank very much | 21:16 |
mtreinish | sure, np | 21:16 |
erlon | mtreinish: ill get back later if (when) I have any problem | 21:16 |
*** rossella_ has quit IRC | 21:18 | |
*** dkranz has quit IRC | 21:19 | |
*** rossella_ has joined #openstack-qa | 21:19 | |
*** edmondsw has quit IRC | 21:21 | |
openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: Explicitly set overlay bridge ints as up https://review.openstack.org/370442 | 21:29 |
*** ddieterly[away] is now known as ddieterly | 21:31 | |
*** gvrangan has joined #openstack-qa | 21:32 | |
*** mriedem has quit IRC | 21:40 | |
*** rlrossit has quit IRC | 21:47 | |
*** dave-mccowan has quit IRC | 21:48 | |
*** Swami has quit IRC | 21:49 | |
*** krtaylor has quit IRC | 22:02 | |
*** rbak has quit IRC | 22:03 | |
*** Gen has joined #openstack-qa | 22:06 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/366872 failed gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial in the gate queue with an unrecognized error | 22:10 |
*** pglass has quit IRC | 22:16 | |
*** gyee has joined #openstack-qa | 22:17 | |
armax | mtreinish: ping | 22:18 |
mtreinish | armax: pong | 22:18 |
armax | mtreinish: wanted to run something by you | 22:18 |
armax | mtreinish: bug 1623691 shows something very unfortunate | 22:19 |
openstack | bug 1623691 in neutron "security group in use conflict during test cleanup" [High,New] https://launchpad.net/bugs/1623691 | 22:19 |
*** yamahata has quit IRC | 22:20 | |
armax | mtreinish: it looks like the default security group is being deleted before the deletion of the port using it is issued | 22:20 |
mtreinish | hmm, that's odd, we should be pretty careful about the ordering there | 22:20 |
armax | I am trying to track down where in tempest the delete_port and delete_security_group happen | 22:20 |
armax | in DynamicCredentialProvider I can see _cleanup_default_secgroup | 22:20 |
armax | but I don’t see the delete_port | 22:21 |
*** yamahata has joined #openstack-qa | 22:21 | |
mtreinish | armax: https://github.com/openstack/tempest/blob/master/tempest/common/dynamic_creds.py#L363-L420 | 22:21 |
mtreinish | any created ports (for servers) would be cleaned up in the tests that created them | 22:21 |
armax | mtreinish: um | 22:22 |
mtreinish | a long time ago we used to delete all ports on a tenant there (because there were too many bugs in doing it any other way) but that was removed at least 2 yrs ago | 22:22 |
armax | the port in question is a compute port | 22:22 |
armax | I wonder if Nova is lagging behind | 22:22 |
mtreinish | armax: hmm, yeah that sounds like an issue in the test or nova | 22:22 |
*** mtanino has quit IRC | 22:23 | |
armax | mtreinish: looking at the exact life cycle of the port in question | 22:24 |
armax | mtreinish: this might be a Neutron bug after all :) | 22:25 |
*** itsuugo has quit IRC | 22:26 | |
armax | what I can see is that the port_delete is interleaving with the security_group deletion | 22:27 |
*** itsuugo has joined #openstack-qa | 22:28 | |
mtreinish | ah, ok so the 2 deletes are racing with each other | 22:29 |
armax | mtreinish: it looks like they do | 22:29 |
mtreinish | does nova wait on the port deletion, or does it just issue an async port delete call (assuming port delete is async) and move on? | 22:29 |
armax | mtreinish: it looks like they might not wait | 22:30 |
armax | http://logs.openstack.org/77/346377/14/check/gate-grenade-dsvm-neutron-dvr-multinode/056b1cf/logs/new/screen-q-svc.txt.gz#_2016-09-14_20_51_33_475 | 22:30 |
mtreinish | because I traced through a bit of tempest's log and it definitely does the issue server delete get a 204 and then poll loop a get on the server until it gets a 404 | 22:30 |
armax | here the plugin is processing the delete | 22:30 |
armax | a few ms after tempest tries to delete the default secgroup | 22:31 |
armax | http://logs.openstack.org/77/346377/14/check/gate-grenade-dsvm-neutron-dvr-multinode/056b1cf/logs/new/screen-q-svc.txt.gz#_2016-09-14_20_51_33_845 | 22:31 |
armax | a few more ms after | 22:31 |
armax | neutron says the port is deleted | 22:31 |
armax | http://logs.openstack.org/77/346377/14/check/gate-grenade-dsvm-neutron-dvr-multinode/056b1cf/logs/new/screen-q-svc.txt.gz#_2016-09-14_20_51_34_337 | 22:31 |
armax | for godo | 22:31 |
armax | good | 22:31 |
mtreinish | hmm, yeah that seems like something is not waiting in the nova <-> neutron communication waiting on it to go away | 22:35 |
armax | mtreinish: that’s the only trace I could find for as long as logstash can remember | 22:36 |
*** Apoorva_ has joined #openstack-qa | 22:36 | |
armax | mtreinish: I’ll tag nova on the bug report, and mark Tempest as invalid | 22:37 |
mtreinish | armax: hmm, it's only hitting on one patch and it's the check queue | 22:37 |
armax | mtreinish: true, but it’s a geniune error I believe | 22:38 |
armax | mtreinish: because the error happens on a job that has nothing to do with the change | 22:38 |
armax | OVS+DVR vs patch being LinuxBridge only | 22:38 |
armax | mtreinish: it’s sheer bad luck I guess | 22:39 |
mtreinish | yeah, I was just looking at the change. lb vs ovs | 22:39 |
mtreinish | it's unlikely to cause the problem :) | 22:39 |
armax | mtreinish: very :) | 22:39 |
*** Apoorva has quit IRC | 22:39 | |
armax | mtreinish: I suspect the vm deletion and port deletion are async | 22:41 |
*** rcernin has quit IRC | 22:43 | |
armax | mtreinish: ok, thanks for the support | 22:43 |
armax | I’ll loop in mriedem | 22:43 |
armax | and try to figure out the best course of action | 22:43 |
*** Swami has joined #openstack-qa | 22:45 | |
mtreinish | armax: so I was quickly tracing through the nova code, but I don't see any wait loops on nova's network api delete. It just calls neutronclient's delete_port() whenever it does a delete port | 22:45 |
mtreinish | so if that doesn't wait for it to go away I could see that being the problem | 22:45 |
armax | mtreinish: yeah, I figured | 22:45 |
armax | mtreinish: I am surprised this went undetected for so long | 22:46 |
*** Swami has quit IRC | 22:46 | |
*** Swami has joined #openstack-qa | 22:46 | |
mtreinish | unless there is a later callback or something that i missed. I still haven't found exactly where in the compute manager the delete_port eventually gets called from | 22:47 |
armax | mtreinish: there’s a network-vif-deleted event emitted by Neutron | 22:49 |
armax | though | 22:49 |
dansmith | armax: what's the issue? | 22:50 |
mtreinish | armax: yeah I wasn't looking for that, just where the delete was issued. I figured there might be a callback to sync things up | 22:50 |
armax | dansmith: bug 1623691 | 22:50 |
openstack | bug 1623691 in neutron "security group in use conflict during test cleanup" [High,New] https://launchpad.net/bugs/1623691 | 22:50 |
armax | dansmith: it looks like the delete port, after a server delete is racing with the deletion of the default security group | 22:51 |
dansmith | armax: that event just informs compute manager if a port is deleted underneath a running instance | 22:52 |
*** itsuugo has quit IRC | 22:52 | |
dansmith | armax: our server delete is generally very aggressive in terms of not waiting for things, and also, the api doesn't wait for the compute manager to even do its stuff before returning | 22:52 |
dansmith | i.e. delete is not a synchronous operation | 22:52 |
dansmith | is that what you are thinking is the problem? | 22:52 |
dansmith | that we don't block tempest until the thing is really gone? | 22:52 |
*** itsuugo has joined #openstack-qa | 22:53 | |
mtreinish | dansmith: tempest does a poll loop waiting for a 404 | 22:53 |
dansmith | the thing we do wait for is instance shutdown and volume detach so we don't corrupt a filesystem in a volume | 22:53 |
*** ddieterly has quit IRC | 22:53 | |
dansmith | mtreinish: oh okay, then I'm confused I guess | 22:53 |
*** zackf has quit IRC | 22:53 | |
armax | dansmith: so if we don’t wait for the networking part to go away too | 22:53 |
armax | I can see the problem | 22:53 |
dansmith | oh, I see, | 22:54 |
dansmith | the instance is marked as deleted, you get a 404, but we're not done waiting internally for cleanup | 22:54 |
armax | in relation to the test cleanup logic | 22:54 |
armax | that attempts to delete the project’s resources like teh default security group | 22:54 |
*** zackf has joined #openstack-qa | 22:54 | |
armax | dansmith: bear in mind everything happened in teh span of less than half a second | 22:55 |
*** xinli has quit IRC | 22:55 | |
dansmith | yup | 22:56 |
armax | dansmith, mtreinish if we want to leave things as is, I can only imagine protecting ourselves from this race if we retry the operation of deleting the default security group | 23:00 |
dansmith | yeah | 23:00 |
*** dustins has quit IRC | 23:00 | |
dansmith | I'm not really sure what else to do | 23:00 |
armax | and fail if the conflict is permanent | 23:00 |
dansmith | yeah | 23:00 |
dansmith | armax: so theoretically we have made the port delete call by the time we set our deleted=true flag | 23:01 |
dansmith | armax: is port delete in neutron synchronous or no? | 23:02 |
armax | dansmith: the DB object goes away when the API responds | 23:02 |
dansmith | hrm, okay | 23:02 |
dansmith | armax: so we do "try" to delete | 23:02 |
*** sdague has quit IRC | 23:02 | |
dansmith | armax: maybe we failed for some reason? | 23:03 |
dansmith | although we should go into error state if that happens, so nevermind | 23:03 |
armax | dansmith: I see no trace in the cpu log | 23:03 |
dansmith | so tbh, I'm not sure how that is happening, really | 23:03 |
armax | dansmith: in this particular instance | 23:07 |
armax | the response to delete port happens after the attempt to delete the security group | 23:07 |
armax | default one | 23:07 |
armax | http://logs.openstack.org/77/346377/14/check/gate-grenade-dsvm-neutron-dvr-multinode/056b1cf/logs/new/screen-q-svc.txt.gz#_2016-09-14_20_51_34_337 | 23:08 |
armax | http://logs.openstack.org/77/346377/14/check/gate-grenade-dsvm-neutron-dvr-multinode/056b1cf/logs/new/screen-q-svc.txt.gz#_2016-09-14_20_51_33_845 | 23:08 |
dansmith | artom: right, but I'm saying that I don't think you could have gotten a 404 on the instance url until after the port delete has already happened | 23:08 |
dansmith | er, armax ^ | 23:09 |
*** itsuugo has quit IRC | 23:10 | |
armax | dansmith: but Nova returns 404 on an active poll from Tempest to check that the instance is gone | 23:10 |
dansmith | mtreinish: you're sure it's 404 and not v_status=DELET(ed,ing) right? | 23:10 |
armax | dansmith: if Nova API does not wait for compute to be done, I could see a problem? | 23:11 |
*** itsuugo has joined #openstack-qa | 23:11 | |
dansmith | armax: it doesn't wait before returning from the delete call, but you won't get a 404 until the compute node has done a lot of work, including deleting the port and detaching the volumes | 23:12 |
mtreinish | dansmith: pretty sure, let me pull up the link | 23:12 |
armax | dansmith: ok | 23:12 |
dansmith | armax: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2271-L2334 | 23:13 |
mtreinish | dansmith: http://logs.openstack.org/77/346377/14/check/gate-grenade-dsvm-neutron-dvr-multinode/056b1cf/logs/tempest.txt.gz?#_2016-09-14_20_51_32_052 | 23:13 |
dansmith | armax: you get a 404 after L2336 has run | 23:13 |
armax | mtreinish: btw I couldn’t find the tempest log for the new run of greande | 23:13 |
armax | grenade | 23:13 |
dansmith | armax: and the network port delete happens in _shutdown_instance() which is called on L2303 | 23:13 |
mtreinish | dansmith: oops that's the wrong test class | 23:13 |
mtreinish | let me find the equiv for the test in question | 23:13 |
mtreinish | armax: it's in the base logs/ dir | 23:14 |
armax | mtreinish: that’s the one for the old run | 23:15 |
armax | mtreinish: if you’re referring to http://logs.openstack.org/77/346377/14/check/gate-grenade-dsvm-neutron-dvr-multinode/056b1cf/logs/ | 23:15 |
armax | mtreinish: that ends at 8.54 | 23:15 |
armax | mtreinish: the test faillure happens 20 mins later | 23:16 |
mtreinish | armax: for old: http://logs.openstack.org/77/346377/14/check/gate-grenade-dsvm-neutron-dvr-multinode/056b1cf/logs/old/tempest.txt.gz | 23:16 |
mtreinish | armax: for new: http://logs.openstack.org/77/346377/14/check/gate-grenade-dsvm-neutron-dvr-multinode/056b1cf/logs/tempest.txt.gz | 23:16 |
mtreinish | armax: the failure traceback prints at the end of the test run | 23:16 |
mtreinish | after all the other tests have finished (on the console output) | 23:16 |
armax | mtreinish: then something if funky when I am opening this in my browser | 23:17 |
oomichi | slowrie: hi, I guess https://blueprints.launchpad.net/tempest/+spec/list-plugins is complete with https://review.openstack.org/#/c/251542/ , right? | 23:17 |
armax | mtreinish: I am looking at the timestamps in the neutron server log | 23:17 |
oomichi | slowrie: or do we need more work on that? | 23:17 |
slowrie | oomichi: yes | 23:17 |
oomichi | slowrie: which for yes ? done? :) | 23:18 |
slowrie | done | 23:18 |
oomichi | slowrie: cool, thanks | 23:18 |
armax | mtreinish: oh never mind | 23:18 |
armax | mtreinish: I am with yu | 23:18 |
armax | *you | 23:18 |
mtreinish | armax: heh, ok | 23:19 |
*** hongbin has quit IRC | 23:20 | |
*** Swami has quit IRC | 23:21 | |
*** Swami has joined #openstack-qa | 23:21 | |
*** jamielennox is now known as jamielennox|away | 23:22 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/qa-specs: Move implemented blueprints to implemented https://review.openstack.org/370476 | 23:25 |
mtreinish | dansmith: hmm, I actually can't find the wait loop in the log | 23:25 |
*** Suyi_ has quit IRC | 23:25 | |
dansmith | mtreinish: I caught you! :) | 23:25 |
mtreinish | oh I think I see the bug now, it's in tempest | 23:26 |
mtreinish | I have no idea how that didn't get caught though | 23:26 |
dansmith | *gasp* | 23:26 |
dansmith | link? | 23:26 |
armax | mtreinish: enlighten me! | 23:27 |
*** ababich has quit IRC | 23:27 | |
dansmith | armax: ahem.. enlighten US | 23:27 |
armax | :) | 23:27 |
*** ababich has joined #openstack-qa | 23:27 | |
oomichi | slowrie: sorry, one more. https://blueprints.launchpad.net/tempest/+spec/centralized-workspaces also is done now, right | 23:27 |
slowrie | tes | 23:28 |
slowrie | yes* | 23:28 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add wait on server delete https://review.openstack.org/370479 | 23:28 |
mtreinish | dansmith, armax: ^^^ | 23:28 |
oomichi | slowrie: thanks again :) | 23:28 |
dansmith | mtreinish: ah | 23:29 |
mtreinish | I'm surprised we're not hitting that more on neutron grenade jobs. Since it should be pretty easy to trigger on jobs that run smoke | 23:29 |
armax | waaat? | 23:29 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/qa-specs: Move implemented blueprints to implemented https://review.openstack.org/370476 | 23:29 |
mtreinish | since there is only one test in that class tagged as smoke | 23:29 |
*** Swami_ has joined #openstack-qa | 23:30 | |
mtreinish | I should have let on of you push that so I could have +2'd it :/ | 23:30 |
armax | mtreinish: abandon :) | 23:31 |
*** vhosakot has quit IRC | 23:31 | |
armax | mtreinish: nice catch btw | 23:31 |
mtreinish | dansmith, armax: heh, actually take over the patch I actually missed it we have the same bug in the other create server call in that class | 23:32 |
mtreinish | then I'll +2 that | 23:32 |
mtreinish | that's kosher right? | 23:32 |
*** Swami has quit IRC | 23:32 | |
armax | mtreinish: I suppose | 23:33 |
mtreinish | I wonder if it would be better to just change it to true here: https://github.com/openstack/tempest/blob/master/tempest/api/volume/base.py#L178 | 23:34 |
mtreinish | I'm trying to think of a scenario where we wouldn't want to wait on the cleanup | 23:34 |
openstackgerrit | Armando Migliaccio proposed openstack/tempest: Add wait on server delete https://review.openstack.org/370479 | 23:34 |
gmann | oomichi: ping | 23:35 |
oomichi | gmann: pong | 23:35 |
gmann | oomichi: on credential client, i am making it to return raw response in https://review.openstack.org/#/c/369839/1 | 23:35 |
*** rfolco has quit IRC | 23:36 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Revert "Define v3 credentials_client as library" https://review.openstack.org/369112 | 23:36 |
*** Apoorva_ has quit IRC | 23:36 | |
*** Apoorva has joined #openstack-qa | 23:37 | |
mtreinish | dansmith, armax: yeah I just did a check we only use that method in 5 places and none of them depend on a server in the deleting state. Lets just remove that useless conditional and always make it wait | 23:37 |
armax | mtreinish: ack | 23:37 |
oomichi | gmann: yeah +2, thanks :) | 23:38 |
gmann | oomichi: Thanks. | 23:38 |
gmann | oomichi: also we need this as base patch - https://review.openstack.org/#/c/369112/ | 23:38 |
gmann | after that i will reiterate the client and if all looks stable then move | 23:39 |
oomichi | gmann: why do we need to revert that? the client returns raw response as you did on the above patch | 23:39 |
gmann | oomichi: so it was moved to lib with incorrect interfaces | 23:40 |
gmann | oomichi: so we decided to revert from lib so that it does not get release | 23:40 |
gmann | oomichi: and after revert from lib then we will fix as per keystone feedback and then move as stable interface | 23:41 |
oomichi | gmann: I have different thinking. Service clients need to return raw response as possible without weird conversions. | 23:41 |
oomichi | gmann: that is consistent on nova service clients also as you know. hypervisors client | 23:42 |
oomichi | gmann: from this viewpoint, v3 credentials_client is good interface | 23:42 |
gmann | oomichi: yea thats my plan, returning raw response but current interface in lib does nto do that | 23:44 |
gmann | oomichi: so i reverted back and then fix | 23:45 |
gmann | oomichi: or you want to fix in lib only | 23:45 |
oomichi | gmann: ah, which tempest.lib client doesn't return raw response? | 23:46 |
gmann | oomichi: this is current one in lib - https://github.com/openstack/tempest/blob/master/tempest/lib/services/identity/v3/credentials_client.py#L39 | 23:46 |
gmann | oomichi: it does not and we were little bit in hurry to merge that in lib | 23:46 |
gmann | oomichi: i noticed that while doing UT for those | 23:46 |
oomichi | gmann: ah, I missed the patch dependency.. | 23:46 |
oomichi | gmann: credentials_client is not released yet, so we can fix it directly without reverting I think. | 23:47 |
gmann | oomichi: ok, we can do that but not good practice seems like :) | 23:48 |
*** mriedem has joined #openstack-qa | 23:48 | |
oomichi | gmann: yeah, I agree. Sorry about that | 23:49 |
gmann | oomichi: ok, so let me fix it in lib only then we can see if anything else also needed | 23:49 |
gmann | oomichi: ah no you should not be sorry, it was me approved that :) | 23:49 |
oomichi | gmann: related to that, can you take a look at https://review.openstack.org/#/c/368147/ ? | 23:49 |
oomichi | gmann: current docstring's link is separated to 2 lines and that is not readable if we create doc | 23:50 |
gmann | oomichi: sure, i will review in afternoon. some management request today :) | 23:50 |
gmann | oomichi: for api-ref link, yea thats true | 23:50 |
oomichi | gmann: the above patch makes them into a single as the second purpose | 23:51 |
openstackrecheck | Console logs not available after 16:18s for gate-grenade-dsvm-neutron-ubuntu-trusty 369110,6,cd8bf24 | 23:51 |
oomichi | gmann: we can see unreadable links on http://docs-draft.openstack.org/41/368241/1/check/gate-tempest-docs-ubuntu-xenial/dc015f7//doc/build/html/library/clients.html#module-tempest.lib.services.compute | 23:52 |
oomichi | we cannot provide actual links if creating doc.. | 23:52 |
gmann | oomichi: ah nice catch. | 23:52 |
gmann | oomichi: and along with that define the mandatory param explicitly right? | 23:53 |
oomichi | gmann: thanks, management request is important *sometimes* :-) | 23:53 |
gmann | oomichi: i am very frustrated from some actually which are just for showcase only no actual meaning :) | 23:53 |
oomichi | gmann: hehe, please don't say company internal thing on public channel. This is logged ;) | 23:54 |
gmann | heh | 23:54 |
gmann | but thats general things for all company | 23:55 |
oomichi | yeah, that is true | 23:56 |
*** hieulq has quit IRC | 23:56 | |
mtreinish | oomichi: even if it wasn't looged most people will log it anyway... I have irclogs locally going back years :) | 23:57 |
*** mmedvede has quit IRC | 23:57 | |
*** itsuugo has quit IRC | 23:57 | |
oomichi | mtreinish: I need to go to your home to break it down ;) | 23:58 |
*** itsuugo has joined #openstack-qa | 23:58 | |
mtreinish | there are many copies | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!