Wednesday, 2016-09-14

*** lucasagomes has quit IRC00:00
*** lucasagomes has joined #openstack-qa00:02
*** Suyi_ has quit IRC00:05
*** yangyapeng has quit IRC00:05
*** Swami has quit IRC00:06
*** lucasagomes has quit IRC00:07
*** lucasagomes has joined #openstack-qa00:10
*** fnaval has quit IRC00:10
*** lucasagomes has quit IRC00:14
*** eeiden has quit IRC00:16
*** lucasagomes has joined #openstack-qa00:17
openstackgerritMerged openstack-infra/devstack-gate: Handle multiline env vars in reproduce.sh  https://review.openstack.org/36963300:24
*** tqtran has quit IRC00:25
*** lucasagomes has quit IRC00:27
*** krtaylor has joined #openstack-qa00:30
*** lucasagomes has joined #openstack-qa00:30
*** OferBarber has joined #openstack-qa00:32
*** lucasagomes has quit IRC00:34
*** lucasagomes has joined #openstack-qa00:38
openstackgerritMerged openstack-dev/devstack: Don't add deb-* packaging repos to plugin list  https://review.openstack.org/36854700:39
*** itsuugo has quit IRC00:40
*** itsuugo has joined #openstack-qa00:40
*** dave-mccowan has joined #openstack-qa00:43
*** itsuugo has quit IRC00:45
*** lucasagomes has quit IRC00:46
*** itsuugo has joined #openstack-qa00:46
*** lucasagomes has joined #openstack-qa00:48
*** tuanla_ has joined #openstack-qa00:49
*** itsuugo has quit IRC00:51
*** itsuugo has joined #openstack-qa00:53
*** lucasagomes has quit IRC00:57
*** lucasagomes has joined #openstack-qa01:00
*** eeiden has joined #openstack-qa01:01
*** lucasagomes has quit IRC01:04
*** lucasagomes has joined #openstack-qa01:07
*** gvrangan has quit IRC01:09
*** itsuugo has quit IRC01:10
*** itsuugo has joined #openstack-qa01:11
*** lucasagomes has quit IRC01:12
*** lucasagomes has joined #openstack-qa01:14
*** gouthamr has quit IRC01:17
*** rossella_ has quit IRC01:18
*** lucasagomes has quit IRC01:19
*** rossella_ has joined #openstack-qa01:19
*** lucasagomes has joined #openstack-qa01:21
*** itsuugo has quit IRC01:26
*** jamielennox is now known as jamielennox|away01:26
*** itsuugo has joined #openstack-qa01:27
*** lucasagomes has quit IRC01:30
*** yangyapeng has joined #openstack-qa01:30
*** lucasagomes has joined #openstack-qa01:33
*** gyx has joined #openstack-qa01:34
*** artom has quit IRC01:38
*** itsuugo has quit IRC01:39
*** artom has joined #openstack-qa01:39
*** salv-orlando has joined #openstack-qa01:41
*** itsuugo has joined #openstack-qa01:41
*** salv-orl_ has quit IRC01:43
*** mriedem has quit IRC01:49
*** clu_ has quit IRC01:50
*** Apoorva_ has joined #openstack-qa01:51
*** itsuugo has quit IRC01:52
*** itsuugo has joined #openstack-qa01:54
*** yamahata has quit IRC01:55
*** Apoorva has quit IRC01:55
openstackgerritzhufl proposed openstack/tempest: Rename test_list_servers_filter_by_server_status  https://review.openstack.org/36977501:56
*** Apoorva_ has quit IRC01:56
*** yangyapeng has quit IRC02:06
*** lucasagomes has quit IRC02:06
*** lucasagomes has joined #openstack-qa02:07
*** yangyapeng has joined #openstack-qa02:07
*** lucasagomes has quit IRC02:15
*** yangyape_ has joined #openstack-qa02:16
*** lucasagomes has joined #openstack-qa02:17
*** yangyapeng has quit IRC02:18
*** woodster_ has quit IRC02:19
*** lucasagomes has quit IRC02:22
*** lucasagomes has joined #openstack-qa02:23
*** yangyapeng has joined #openstack-qa02:24
*** yangyape_ has quit IRC02:27
*** lucasagomes has quit IRC02:28
*** lucasagomes has joined #openstack-qa02:29
*** csomerville has quit IRC02:32
*** lucasagomes has quit IRC02:34
*** lucasagomes has joined #openstack-qa02:37
*** cody-somerville has joined #openstack-qa02:38
*** dave-mccowan has quit IRC02:38
*** lucasagomes has quit IRC02:41
*** lucasagomes has joined #openstack-qa02:43
*** akshai has joined #openstack-qa02:48
*** lucasagomes has quit IRC02:48
*** akshai_ has joined #openstack-qa02:49
*** stevemar_ is now known as stevemar02:49
*** lucasagomes has joined #openstack-qa02:49
*** akshai has quit IRC02:53
*** lucasagomes has quit IRC02:57
*** lucasagomes has joined #openstack-qa02:58
*** lucasagomes has quit IRC03:03
*** lucasagomes has joined #openstack-qa03:05
*** lucasagomes has quit IRC03:10
*** lucasagomes has joined #openstack-qa03:13
*** akshai_ has quit IRC03:19
*** lucasagomes has quit IRC03:21
*** itsuugo has quit IRC03:21
*** tqtran has joined #openstack-qa03:22
*** itsuugo has joined #openstack-qa03:23
*** lucasagomes has joined #openstack-qa03:23
oomichisc68cal: I am checking dev status of devstack based on https://etherpad.openstack.org/p/newton-qa-newton-priorities03:23
oomichisc68cal: 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
oomichisc68cal: is it fine to mark it as done now?03:25
*** tochi has quit IRC03:26
oomichisc68cal: in most case, cleanup is never ending story though ;)03:26
*** tqtran has quit IRC03:26
*** lucasagomes has quit IRC03:28
*** lucasagomes has joined #openstack-qa03:29
*** yangyapeng has quit IRC03:31
*** itsuugo has quit IRC03:32
*** itsuugo has joined #openstack-qa03:34
*** lucasagomes has quit IRC03:41
*** lucasagomes has joined #openstack-qa03:43
*** weshay is now known as weshay_pto03:45
*** hoangcx has joined #openstack-qa03:46
oomichiandreaf: I think we can mark https://blueprints.launchpad.net/tempest/+spec/client-manager-refactor as implemented now, right?03:47
oomichiandreaf: thanks for doing that, anyways :)03:47
*** itsuugo has quit IRC03:48
*** lucasagomes has quit IRC03:48
*** itsuugo has joined #openstack-qa03:49
*** lucasagomes has joined #openstack-qa03:50
*** yamamoto has joined #openstack-qa03:52
*** lucasagomes has quit IRC03:54
*** lucasagomes has joined #openstack-qa03:56
*** tochi has joined #openstack-qa03:58
*** itsuugo has quit IRC03:59
*** itsuugo has joined #openstack-qa04:00
*** lucasagomes has quit IRC04:01
gmannoomichi: regarding this - https://review.openstack.org/#/c/368326/104:02
gmannoomichi: actually that is also 1 way to init the existing dir04:02
gmannoomichi: can we say this in OR condition04:02
*** lucasagomes has joined #openstack-qa04:04
*** itsuugo has quit IRC04:10
*** lucasagomes has quit IRC04:12
*** itsuugo has joined #openstack-qa04:12
*** lucasagomes has joined #openstack-qa04:13
*** hoangcx has left #openstack-qa04:16
openstackgerritClark Boylan proposed openstack-infra/devstack-gate: DNM testing trove with lxd  https://review.openstack.org/36981004:16
*** lucasagomes has quit IRC04:17
*** itsuugo has quit IRC04:19
*** lucasagomes has joined #openstack-qa04:19
*** itsuugo has joined #openstack-qa04:22
*** lucasagomes has quit IRC04:24
openstackgerritClark Boylan proposed openstack-infra/devstack-gate: DNM testing trove with lxd  https://review.openstack.org/36981004:25
*** lucasagomes has joined #openstack-qa04:26
*** gvrangan has joined #openstack-qa04:26
*** armax has quit IRC04:29
*** Alex_Stef has joined #openstack-qa04:30
*** gvrangan has quit IRC04:33
*** lucasagomes has quit IRC04:33
*** lucasagomes has joined #openstack-qa04:35
openstackgerritClark Boylan proposed openstack-infra/devstack-gate: DNM testing trove with lxd  https://review.openstack.org/36981004:38
*** gvrangan has joined #openstack-qa04:40
*** yangyapeng has joined #openstack-qa04:41
*** itsuugo has quit IRC04:42
*** yamahata has joined #openstack-qa04:43
*** lucasagomes has quit IRC04:44
*** itsuugo has joined #openstack-qa04:44
*** lucasagomes has joined #openstack-qa04:47
*** lucasagomes has quit IRC04:51
*** itsuugo has quit IRC04:51
*** itsuugo has joined #openstack-qa04:53
*** lucasagomes has joined #openstack-qa04:53
*** yamamoto has quit IRC04:55
*** yamamoto has joined #openstack-qa04:57
*** lucasagomes has quit IRC04:58
*** lucasagomes has joined #openstack-qa04:58
*** lucasagomes has quit IRC05:03
*** yangyapeng has quit IRC05:04
*** lucasagomes has joined #openstack-qa05:04
*** yangyapeng has joined #openstack-qa05:06
openstackgerritMerged openstack/tempest: Remove non-locking accounts from the config guide  https://review.openstack.org/33975405:07
*** lucasagomes has quit IRC05:09
*** lucasagomes has joined #openstack-qa05:11
openstackgerritmelissaml proposed openstack/tempest: Modify use of assertTrue(A in B)  https://review.openstack.org/36982405:13
*** armax has joined #openstack-qa05:15
*** josdotso has quit IRC05:15
*** pgadiya has joined #openstack-qa05:15
*** gcheresh has joined #openstack-qa05:15
*** phschwartz has quit IRC05:15
*** mordred has quit IRC05:16
gmannoomichi: 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/205:16
*** AmitGalitz- has joined #openstack-qa05:17
*** rossella_ has quit IRC05:18
*** phschwartz has joined #openstack-qa05:19
*** rossella_ has joined #openstack-qa05:19
*** lucasagomes has quit IRC05:19
*** mordred has joined #openstack-qa05:21
*** lucasagomes has joined #openstack-qa05:23
*** k4n0 has joined #openstack-qa05:24
*** lucasagomes has quit IRC05:27
*** itsuugo has quit IRC05:28
*** lucasagomes has joined #openstack-qa05:28
*** k4n0 has quit IRC05:29
*** itsuugo has joined #openstack-qa05:29
*** rcernin has joined #openstack-qa05:33
*** lucasagomes has quit IRC05:33
*** lucasagomes has joined #openstack-qa05:35
*** cody-somerville has quit IRC05:35
*** cody-somerville has joined #openstack-qa05:38
*** jamielennox|away is now known as jamielennox05:40
*** itsuugo has quit IRC05:43
*** lucasagomes has quit IRC05:45
*** armax has quit IRC05:45
*** itsuugo has joined #openstack-qa05:45
*** lucasagomes has joined #openstack-qa05:47
*** zackf has quit IRC05:50
*** lucasagomes has quit IRC05:51
*** chandankumar has joined #openstack-qa05:52
*** itsuugo has quit IRC05:52
*** itsuugo has joined #openstack-qa05:53
*** lucasagomes has joined #openstack-qa05:55
*** lucasagomes has quit IRC05:59
*** lucasagomes has joined #openstack-qa06:02
*** artom has quit IRC06:02
*** artom has joined #openstack-qa06:03
*** itsuugo has quit IRC06:07
*** paul-carlton2 has joined #openstack-qa06:08
*** lucasagomes has quit IRC06:09
*** itsuugo has joined #openstack-qa06:09
*** lucasagomes has joined #openstack-qa06:10
openstackgerritGhanshyam Mann proposed openstack/tempest: Fix credential client to return raw response  https://review.openstack.org/36983906:12
gmannoomichi: like this - https://review.openstack.org/#/c/369839/106:13
*** itzikb has joined #openstack-qa06:16
*** salv-orlando has quit IRC06:18
*** lucasagomes has quit IRC06:18
*** sridharg has joined #openstack-qa06:19
*** lucasagomes has joined #openstack-qa06:20
openstackgerritMerged openstack/tempest: Update Available para link of image v1 client  https://review.openstack.org/36645306:22
*** itsuugo has quit IRC06:27
*** edannon has joined #openstack-qa06:29
*** itsuugo has joined #openstack-qa06:29
*** pcaruana has joined #openstack-qa06:32
*** itsuugo has quit IRC06:34
*** itsuugo has joined #openstack-qa06:35
*** dimtruck is now known as zz_dimtruck06:36
*** yamamoto has quit IRC06:40
*** itsuugo has quit IRC06:40
openstackgerritChangBo Guo(gcb) proposed openstack/os-testr: Update home-page url  https://review.openstack.org/35617906:41
*** itsuugo has joined #openstack-qa06:41
*** Guest95134 has quit IRC06:42
*** gema has joined #openstack-qa06:43
*** lucasagomes has quit IRC06:44
*** lucasagomes has joined #openstack-qa06:46
*** itsuugo has quit IRC06:49
*** itsuugo has joined #openstack-qa06:50
*** salv-orlando has joined #openstack-qa06:50
*** lucasagomes has quit IRC06:51
*** lucasagomes has joined #openstack-qa06:53
*** EinstCrazy has joined #openstack-qa06:55
*** csomerville has joined #openstack-qa06:56
*** tesseract- has joined #openstack-qa06:57
*** zz_dimtruck is now known as dimtruck06:57
*** EinstCra_ has joined #openstack-qa06:58
*** EinstCrazy has quit IRC06:58
*** cody-somerville has quit IRC06:59
openstackgerritOpenStack Proposal Bot proposed openstack-dev/devstack: Updated from generate-devstack-plugins-list  https://review.openstack.org/35699106:59
*** lucasagomes has quit IRC07:00
*** lucasagomes has joined #openstack-qa07:00
*** csomerville has quit IRC07:01
*** EinstCra_ has quit IRC07:04
*** lucasagomes has quit IRC07:05
*** shardy has joined #openstack-qa07:06
*** lucasagomes has joined #openstack-qa07:06
*** dimtruck is now known as zz_dimtruck07:10
*** Alex_Stef has quit IRC07:12
*** lucasagomes has quit IRC07:13
*** lucasagomes has joined #openstack-qa07:15
*** wendar has quit IRC07:15
*** wendar has joined #openstack-qa07:15
*** shardy is now known as shardy_mtg07:16
*** yaume has joined #openstack-qa07:18
*** Ravikiran_K has joined #openstack-qa07:19
*** paul-carlton2 has quit IRC07:19
*** yamamoto has joined #openstack-qa07:19
*** lucasagomes has quit IRC07:20
*** paul-carlton2 has joined #openstack-qa07:20
*** lucasagomes has joined #openstack-qa07:21
*** tochi has quit IRC07:22
openstackgerritMerged openstack/tempest: Ignore deb-* packaging repos in the plugin list  https://review.openstack.org/36975307:22
*** markus_z has joined #openstack-qa07:24
*** OferBarber has quit IRC07:24
*** tqtran has joined #openstack-qa07:26
*** EinstCrazy has joined #openstack-qa07:27
*** jlanoux has joined #openstack-qa07:30
*** tqtran has quit IRC07:31
*** lucasagomes has quit IRC07:31
*** EinstCrazy has quit IRC07:32
*** paul-carlton2 has quit IRC07:33
*** lucasagomes has joined #openstack-qa07:34
*** zackf has joined #openstack-qa07:35
*** cody-somerville has joined #openstack-qa07:35
*** cody-somerville has quit IRC07:35
*** cody-somerville has joined #openstack-qa07:35
*** csomerville has joined #openstack-qa07:37
openstackgerritguo yunxian proposed openstack/tempest: Remove unnecessary resource_setup()  https://review.openstack.org/36987107:37
*** abehl has joined #openstack-qa07:38
*** jlanoux has quit IRC07:38
*** jlanoux has joined #openstack-qa07:38
*** lucasagomes has quit IRC07:38
*** lucasagomes has joined #openstack-qa07:39
*** zackf has quit IRC07:39
*** itsuugo has quit IRC07:39
*** EinstCrazy has joined #openstack-qa07:40
*** zz_dimtruck has quit IRC07:40
*** salv-orl_ has joined #openstack-qa07:40
*** cody-somerville has quit IRC07:41
*** itsuugo has joined #openstack-qa07:41
*** EinstCrazy has quit IRC07:41
*** jpena|off is now known as jpena07:42
*** zz_dimtruck has joined #openstack-qa07:42
*** salv-orlando has quit IRC07:42
*** zz_dimtruck is now known as dimtruck07:43
*** lucasagomes has quit IRC07:44
*** paul-carlton2 has joined #openstack-qa07:45
*** lucasagomes has joined #openstack-qa07:47
*** kbaikov has quit IRC07:51
*** lucasagomes has quit IRC07:52
*** lucasagomes has joined #openstack-qa07:52
*** jdandrea_ has joined #openstack-qa07:53
*** jdandrea has quit IRC07:56
*** lucasagomes has quit IRC07:57
*** lucasagomes has joined #openstack-qa07:59
*** itzikb has quit IRC08:00
*** artom has quit IRC08:00
*** itzikb has joined #openstack-qa08:00
openstackgerritliwei proposed openstack/tempest: Add get namespace schema test methods  https://review.openstack.org/36988008:01
*** artom has joined #openstack-qa08:01
*** sshnaidm|afk is now known as sshnaidm08:01
*** artom has quit IRC08:02
*** artom has joined #openstack-qa08:02
*** dtantsur is now known as dtantsur|brb08:03
*** lucasagomes has quit IRC08:03
*** paul-carlton2 has quit IRC08:03
*** Alex_Stef has joined #openstack-qa08:07
*** lucasagomes has joined #openstack-qa08:08
*** gfidente has joined #openstack-qa08:09
andreafoomichi: yeah, I think it would be nice to close that08:20
andreafoomichi: 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 part08:21
andreafoomichi: glad I could contribute that change :)08:21
*** lucasagomes has quit IRC08:27
*** itzkb_ has joined #openstack-qa08:29
*** itzikb has quit IRC08:29
*** paul-carlton2 has joined #openstack-qa08:29
*** yamahata has quit IRC08:31
*** oomichi has quit IRC08:31
*** lucasagomes has joined #openstack-qa08:32
*** oomichi has joined #openstack-qa08:32
*** lucasagomes has quit IRC08:46
*** lucasagomes has joined #openstack-qa08:46
*** itsuugo has quit IRC08:51
*** itsuugo has joined #openstack-qa08:52
*** itsuugo has quit IRC09:02
*** itsuugo has joined #openstack-qa09:03
*** lucasagomes has quit IRC09:05
*** lucasagomes has joined #openstack-qa09:08
*** dimtruck is now known as zz_dimtruck09:11
*** e0ne has joined #openstack-qa09:14
*** lucasagomes has quit IRC09:14
*** lucasagomes has joined #openstack-qa09:17
*** mvk has quit IRC09:19
*** lucasagomes has quit IRC09:21
*** lucasagomes_ has joined #openstack-qa09:23
*** lucasagomes_ has quit IRC09:23
*** cdent has joined #openstack-qa09:26
*** tqtran has joined #openstack-qa09:28
*** tqtran has quit IRC09:32
*** gvrangan has quit IRC09:33
*** itsuugo has quit IRC09:33
*** larainema has quit IRC09:34
*** itsuugo has joined #openstack-qa09:35
*** tosky has joined #openstack-qa09:41
*** gyx has quit IRC09:43
openstackgerritKyrylo Romanenko proposed openstack/tempest: Fix docstrings in Tempest REST client for Ironic  https://review.openstack.org/36222109:45
*** dtantsur|brb is now known as dtantsur09:48
*** mvk has joined #openstack-qa09:52
*** yangyapeng has quit IRC09:53
*** rfolco has joined #openstack-qa09:56
*** pgadiya has quit IRC09:57
*** pgadiya has joined #openstack-qa09:58
*** yamamoto has quit IRC10:12
*** EinstCrazy has joined #openstack-qa10:30
*** ramishra has quit IRC10:31
*** ramishra has joined #openstack-qa10:33
*** gszasz has joined #openstack-qa10:36
*** itsuugo has quit IRC10:36
*** itsuugo has joined #openstack-qa10:37
*** gszasz has quit IRC10:38
*** asselin_ has quit IRC10:42
*** EinstCrazy has quit IRC10:48
*** itsuugo has quit IRC10:48
*** itsuugo has joined #openstack-qa10:49
*** shardy_mtg is now known as shardy10:51
*** gszasz has joined #openstack-qa10:52
*** itsuugo has quit IRC10:56
*** itsuugo has joined #openstack-qa10:58
*** zz_dimtruck is now known as dimtruck11:01
*** larainema has joined #openstack-qa11:06
*** shardy is now known as shardy_mtg11:07
*** yamamoto has joined #openstack-qa11:07
*** pgadiya has quit IRC11:08
*** sdague has joined #openstack-qa11:08
*** pgadiya has joined #openstack-qa11:09
*** pgadiya has quit IRC11:09
*** itsuugo has quit IRC11:10
*** pgadiya has joined #openstack-qa11:10
*** dimtruck is now known as zz_dimtruck11:11
*** itsuugo has joined #openstack-qa11:11
*** jpena is now known as jpena|lunch11:17
*** OferBarber has joined #openstack-qa11:18
OferBarberhas anyon ever used "create_test_server"11:19
OferBarberfrom the file tempest/common/compute.py ?11:19
*** itsuugo has quit IRC11:24
*** itsuugo has joined #openstack-qa11:26
*** tqtran has joined #openstack-qa11:29
dmelladoOferBarber: have you checked the imports on any compute tests?11:31
dmelladoplease do follow the pattern on, for example11:31
*** tuanla_ has quit IRC11:32
dmelladohttps://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_servers.py#L1811:32
dmelladoif you back from the imports there you'll find the relationship11:32
OferBarberdmellado: great. thanks a lot11:34
dmelladonp OferBarber11:34
*** tqtran has quit IRC11:34
*** openstackgerrit has quit IRC11:34
*** openstackgerrit has joined #openstack-qa11:34
OferBarberdmellado: why did you select the class from test_server.py and not from compute.py ?11:38
OferBarberdmellado: how do i know where to select the function from ?11:38
dmelladoOferBarber if you check the tests over tempest11:39
dmelladoit's usually split in api/service11:39
dmelladoand there you've a base class11:39
dmelladoif you want to write a new test for compute, I'd suggest that you start extending that class directly11:39
dmelladoand importing the methods from there (unless they've been ported to tempest/lib)11:40
*** kbaikov has joined #openstack-qa11:40
dmelladowhich is the tempest stable interface11:40
dmelladohttps://github.com/openstack/tempest/tree/master/tempest/lib11:40
OferBarberdmellado: i would like to write a new scenario11:41
dmelladoOferBarber: did you check existing scenario ones?11:42
dmelladoyou might as well be good to go with the existing methods on the scenario manager11:42
dmelladohttps://github.com/openstack/tempest/blob/master/tempest/scenario/manager.py11:43
OferBarberdmellado: yes. someone in my company thinks some scenarios are missing11:44
OferBarberi see that i have lots to learn11:44
OferBarbershould i always prefer the "base.py" file to be used ?11:44
dmelladoOferBarber: np! please have a look at the scenario manager11:44
dmelladoalso take into consideration that if your scenario test relies too much on an specific project11:45
dmelladoi.e. neutron 'advanced' capabilities11:45
dmelladoyou might well consider wether to put it directly on the project tempest plugin11:45
dmelladoinstead of in tempest itself11:45
OferBarberdmellado: great tips. thank you11:47
*** dave-mccowan has joined #openstack-qa11:47
dmelladoOferBarber: you're welcome ;)11:48
*** woodster_ has joined #openstack-qa11:49
*** itsuugo has quit IRC11:54
*** edmondsw has joined #openstack-qa11:55
*** markus_zz has joined #openstack-qa11:55
*** markus_z has quit IRC11:55
*** itsuugo has joined #openstack-qa11:56
*** artom has quit IRC11:56
*** artom has joined #openstack-qa11:57
*** lbeliveau has quit IRC11:58
*** lbeliveau has joined #openstack-qa12:00
openstackrecheckConsole logs not available after 13:33s for gate-nova-python27-db-ubuntu-xenial 356558,13,07ef7e612:02
*** Kevin_Zheng has quit IRC12:06
*** fnaval has joined #openstack-qa12:07
*** gouthamr has joined #openstack-qa12:10
*** fnaval has quit IRC12:12
*** fnaval has joined #openstack-qa12:13
*** fnaval has quit IRC12:14
*** fnaval has joined #openstack-qa12:15
*** yangyapeng has joined #openstack-qa12:17
*** AmitGalitz- has quit IRC12:18
*** AmitGalitz has joined #openstack-qa12:18
*** AmitGalitz has quit IRC12:19
*** Pewlet_Hackard has joined #openstack-qa12:19
openstackgerritKyrylo Romanenko proposed openstack/tempest: Fix docstrings in Tempest REST client for Ironic  https://review.openstack.org/36222112:27
*** mriedem has joined #openstack-qa12:27
*** pgadiya has quit IRC12:28
*** zackf has joined #openstack-qa12:29
*** itsuugo has quit IRC12:32
*** itsuugo has joined #openstack-qa12:33
*** zackf has quit IRC12:33
*** itsuugo has quit IRC12:38
*** shardy_mtg is now known as shardy12:39
*** itsuugo has joined #openstack-qa12:39
*** pgadiya has joined #openstack-qa12:43
*** pgadiya has quit IRC12:43
*** pgadiya has joined #openstack-qa12:43
*** zz_dimtruck is now known as dimtruck12:50
*** itsuugo has quit IRC12:50
*** itsuugo has joined #openstack-qa12:52
*** porrua has joined #openstack-qa12:54
*** asselin_ has joined #openstack-qa12:55
*** jpena|lunch is now known as jpena12:55
*** zackf has joined #openstack-qa12:56
*** pgadiya has quit IRC12:57
*** Kevin_Zheng has joined #openstack-qa13:00
*** dkranz has joined #openstack-qa13:01
*** zackf has quit IRC13:01
openstackgerritJens Rosenboom proposed openstack-dev/devstack: WIP: Test fix for identity-v3 failure  https://review.openstack.org/36967513:10
*** Pewlet_Hackard has quit IRC13:10
*** rossella_ has quit IRC13:18
*** rossella_ has joined #openstack-qa13:19
*** shardy has quit IRC13:23
*** rbak has joined #openstack-qa13:23
*** mriedem has quit IRC13:25
*** tqtran has joined #openstack-qa13:31
*** markus_zz is now known as markus_z13:32
*** tqtran has quit IRC13:35
*** artom has quit IRC13:40
*** salv-orlando has joined #openstack-qa13:40
*** artom has joined #openstack-qa13:41
*** artom has quit IRC13:42
*** yamamoto has quit IRC13:42
*** itsuugo has quit IRC13:42
*** artom has joined #openstack-qa13:42
*** markvoelker has joined #openstack-qa13:43
*** gcheresh has quit IRC13:43
*** salv-orl_ has quit IRC13:43
*** yamamoto has joined #openstack-qa13:43
*** itsuugo has joined #openstack-qa13:44
*** yamamoto has quit IRC13:44
openstackgerritSean Dague proposed openstack-dev/devstack: force en_US.UTF-8 during devstack runs  https://review.openstack.org/37013213:48
ylobankovoomichi, afazekas: are you around?13:48
* afazekas o/13:48
*** mtanino has joined #openstack-qa13:49
ylobankovafazekas: 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 IRC13:50
ylobankovafazekas: I am ok to delete the test, but oomichi would like to keep it for more test coverage.13:51
*** itsuugo has joined #openstack-qa13:52
*** gamado has joined #openstack-qa13:52
*** gamado has quit IRC13:52
*** ddieterly has joined #openstack-qa13:52
*** mmedvede has joined #openstack-qa13:53
*** fnaval has quit IRC13:54
afazekasylobankov, Am ok with both skipping and/or deleting13:54
*** shardy has joined #openstack-qa13:55
ylobankovafazekas: Thanks.13:56
*** yamamoto has joined #openstack-qa13:58
*** fnaval has joined #openstack-qa13:59
*** akshai has joined #openstack-qa14:01
*** Swami has joined #openstack-qa14:03
*** Swami has quit IRC14:03
*** Swami has joined #openstack-qa14:03
*** rodrigods has quit IRC14:06
*** rodrigods has joined #openstack-qa14:06
*** dustins has joined #openstack-qa14:07
*** yamamoto has quit IRC14:11
openstackgerritMarkus Zoeller (markus_z) proposed openstack/tempest: WIP: Nova: test live migration with serial console  https://review.openstack.org/34681514:11
*** mriedem has joined #openstack-qa14:11
*** mriedem has quit IRC14:11
*** Daviey_ is now known as Daviey14:12
*** artom has quit IRC14:14
*** artom has joined #openstack-qa14:16
*** artom has quit IRC14:16
*** artom has joined #openstack-qa14:17
*** dtantsur is now known as dtantsur|mtg14:17
fricklerso https://review.openstack.org/369675 could succeed as a workaround for https://bugs.launchpad.net/glance-store/+bug/162099914:18
openstackLaunchpad bug 1620999 in glance_store "glance_store ignores user_domain_name and project_domain_name settings" [Undecided,New]14:18
*** rlrossit has joined #openstack-qa14:18
openstackrecheckopenstack/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/144913614:18
openstackLaunchpad bug 1449136 in OpenStack-Gate "OpenStack pypi mirrors disconnecting connections" [Undecided,New]14:18
*** raildo has joined #openstack-qa14:22
*** Alex_Stef has quit IRC14:24
*** mriedem has joined #openstack-qa14:25
*** zackf has joined #openstack-qa14:26
*** yamahata has joined #openstack-qa14:29
*** Swami_ has joined #openstack-qa14:35
*** rfolco has quit IRC14:37
*** Swami has quit IRC14:38
openstackrecheckopenstack/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 error14:38
openstackLaunchpad bug 1449136 in OpenStack-Gate "OpenStack pypi mirrors disconnecting connections" [Undecided,New]14:38
*** Swami_ has quit IRC14:39
*** pglass has joined #openstack-qa14:44
*** amotoki has joined #openstack-qa14:45
*** raildo has quit IRC14:47
*** raildo has joined #openstack-qa14:47
*** Swami has joined #openstack-qa14:48
*** erlon has joined #openstack-qa14:50
*** xinli has joined #openstack-qa14:52
*** ddieterly is now known as ddieterly[away]14:54
*** itsuugo has quit IRC14:54
*** itsuugo has joined #openstack-qa14:56
*** rfolco has joined #openstack-qa14:58
*** ddieterly[away] is now known as ddieterly14:59
*** akshai has quit IRC14:59
*** pglass has quit IRC15:00
*** ddieterly has quit IRC15:01
*** edannon has quit IRC15:01
*** akshai has joined #openstack-qa15:01
*** david-lyle_ has joined #openstack-qa15:06
*** david-lyle_ has quit IRC15:06
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add subunit2sql CLI option to use run wall time  https://review.openstack.org/37019315:07
mtreinishcdent: ^^^ that's the patch we were talking about a while ago to get started on adding run time graphs for views that involve entire runs15:07
*** jlanoux has quit IRC15:08
cdentmtreinish: 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 time15:09
* cdent sticks it in his review queue15:09
mtreinishcdent: 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 corner15:10
*** yamamoto has joined #openstack-qa15:11
*** armax has joined #openstack-qa15:12
*** ddieterly has joined #openstack-qa15:13
cdentI fear I will be under this sea for THE REST OF MY LIFE15:14
openstackgerritLance Bragstad proposed openstack/tempest: Allow for wait in _restore_password  https://review.openstack.org/36759515:14
*** gvrangan has joined #openstack-qa15:14
mtreinishlbragstad: ^^^ more magic sleeps....15:15
lbragstadmtreinish i think they are the same as before - but the _restore_password method is new15:16
*** ddieterly has quit IRC15:17
*** ddieterly has joined #openstack-qa15:18
mtreinishlbragstad: 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 successfully15:19
mtreinishisn't that enough time to change the password again?15:19
*** itzkb_ has quit IRC15:19
lbragstadmtreinish 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 change15:21
lbragstads/make/match/15:21
*** abehl has quit IRC15:25
*** yamamoto has quit IRC15:25
*** Kevin_Zheng has quit IRC15:26
*** sridharg has quit IRC15:29
*** vhosakot has joined #openstack-qa15:30
*** ddieterly is now known as ddieterly[away]15:31
*** yamamoto has joined #openstack-qa15:33
*** rcernin has quit IRC15:34
*** dtantsur|mtg is now known as dtantsur15:35
*** jlanoux has joined #openstack-qa15:36
*** jlanoux has quit IRC15:36
*** jlanoux has joined #openstack-qa15:38
*** pcaruana has quit IRC15:40
openstackgerritSean Dague proposed openstack-dev/devstack: force en_US.UTF-8 during devstack runs  https://review.openstack.org/37013215:41
*** ddieterly[away] has quit IRC15:41
*** rlrossit has quit IRC15:43
*** shardy has quit IRC15:43
openstackgerritYaroslav Lobankov proposed openstack/tempest: Skip test_add_list_remove_router_on_l3_agent because of the bug 1590049  https://review.openstack.org/37023615:46
openstackbug 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
openstackrecheckopenstack/nova change: https://review.openstack.org/363926 failed because of: gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial: https://bugs.launchpad.net/bugs/147533315:48
openstackLaunchpad 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-qa15:50
*** mriedem has quit IRC15:51
*** mriedem1 has joined #openstack-qa15:51
*** sshnaidm is now known as sshnaidm|afk15:51
*** pglass has joined #openstack-qa16:01
*** gyee has joined #openstack-qa16:01
*** mriedem1 is now known as mriedem16:08
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add run_metadata filtering to get run_times function  https://review.openstack.org/37025516:08
*** krtaylor has quit IRC16:10
*** jpena is now known as jpena|off16:11
*** ddieterly has quit IRC16:16
*** chandankumar has quit IRC16:17
openstackgerritGraham Hayes proposed openstack-infra/elastic-recheck: Add query for bug 1623576  https://review.openstack.org/37027116:18
openstackbug 1623576 in Designate "test_zone_create_propagates_to_nameservers times out before zone is fully propagated" [Medium,Triaged] https://launchpad.net/bugs/162357616:18
*** markus_z has quit IRC16:20
*** Apoorva has joined #openstack-qa16:21
*** lucasagomes has joined #openstack-qa16:23
*** yamamoto has quit IRC16:23
*** yamamoto has joined #openstack-qa16:24
*** yamamoto has quit IRC16:25
*** amotoki has quit IRC16:26
*** ddieterly has joined #openstack-qa16:27
*** porrua has quit IRC16:29
*** cdent has quit IRC16:32
*** jlanoux has quit IRC16:32
*** rcernin has joined #openstack-qa16:34
*** rlrossit has joined #openstack-qa16:35
*** tesseract- has quit IRC16:35
*** dtantsur is now known as dtantsur|afk16:36
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add new DB API function to get grouped run_time time series  https://review.openstack.org/37028816:38
openstackgerritMikhail S Medvedev proposed openstack-dev/devstack: Set default properties for Power: virtual console and scsi driver  https://review.openstack.org/37028916:38
mtreinishcdent ^^^ that's part 2, the last part will be to use that in the openstack-health api and then graph the data16:38
*** yamamoto has joined #openstack-qa16:39
*** ddieterly is now known as ddieterly[away]16:39
mtreinishmasayukig: ^^^ do you want to work on the openstack-health bits on top of that db api change?16:41
mtreinishif not I can handle it16:42
mtreinishmasayukig: 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 payload16:44
mtreinishthere's a lot of examples of doing that in the api code16:44
*** yamahata has quit IRC16:44
*** Swami has quit IRC16:44
mtreinishmasayukig: oh, and of course the js graphing half to use the data :)16:44
*** ddieterly[away] is now known as ddieterly16:46
*** sshnaidm|afk is now known as sshnaidm16:47
*** mvk has quit IRC16:47
*** e0ne has quit IRC16:49
*** gszasz has quit IRC16:51
openstackrecheckopenstack/nova change: https://review.openstack.org/364829 failed gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial in the gate queue with an unrecognized error16:54
*** tosky has quit IRC16:56
*** krtaylor has joined #openstack-qa16:57
mriedemmtreinish: https://review.openstack.org/37030316:57
mriedemoh no16:57
mriedemnvm16:57
*** porrua has joined #openstack-qa16:58
mriedemwrong bug :)16:58
mtreinishmriedem: heh, ok16:59
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck: Remove query for fixed bug 1616498  https://review.openstack.org/37030416:59
openstackbug 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
mriedemthat ^16:59
mtreinishmriedem: +A17:00
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add new DB API function to get grouped run_time time series  https://review.openstack.org/37028817:03
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add run_metadata filtering to get run_times function  https://review.openstack.org/37025517:03
*** Kiall_ is now known as Kiall17:03
*** ddieterly is now known as ddieterly[away]17:06
oomichigmann: ok, https://review.openstack.org/#/c/368326 is dropped17:06
oomichigmann: about  https://review.openstack.org/#/c/369112: if doing that, testers cannot know this weird API response on tests side17:07
oomichigmann: how about having some helper method in common?17:08
openstackgerritPradeep Kumar proposed openstack/tempest: Adding testcases for metadata def properties  https://review.openstack.org/37031517:09
*** Swami has joined #openstack-qa17:10
oomichiandreaf: oh, cool. I am happy to change the status of the LP to 'implemented' :-)17:11
*** yamamoto has quit IRC17:11
oomichimtreinish: how about the status of https://blueprints.launchpad.net/tempest/+spec/tempest-run-cmd ?17:15
oomichimtreinish: can we change the status to "implemented" after https://review.openstack.org/#/c/348511/ ?17:15
openstackgerritMerged openstack-infra/elastic-recheck: Remove query for fixed bug 1616498  https://review.openstack.org/37030417:15
openstackbug 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
oomichimtreinish: I feel that is already good for users17:15
mtreinishoomichi: no, after: https://review.openstack.org/#/c/355666/17:16
oomichimtreinish: heh, that is "super wip" now17:16
*** chandankumar has joined #openstack-qa17:16
mtreinishwe want to *dogfood* the new workflow completly before we say it's done :)17:16
*** ddieterly[away] has quit IRC17:16
mtreinishright now there is no usage of workspaces at all anywhere17:16
oomichihaha17:16
mtreinishwhich is an important part of the spec17:16
mtreinishoomichi: I put that on the etherpad for next week already17:17
oomichiOK, you are managing this strictly :)17:17
oomichimtreinish: awesome, thanks17:17
mtreinishI'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 sprint17:18
*** rossella_ has quit IRC17:18
oomichimtreinish: you can work 5 times by comparing comeone in the meetup17:18
*** rossella_ has joined #openstack-qa17:19
oomichiok, I put a comment on the bp of LP, which patch is the goal of that17:19
*** Suyi_ has joined #openstack-qa17:20
*** yamamoto has joined #openstack-qa17:24
*** yamamoto has quit IRC17:24
*** lucasagomes is now known as lucas-dinner17:25
sdagueare there actually any utf8 testing tempest patches besides the server rename?17:25
*** ddieterly has joined #openstack-qa17:30
*** openstackrecheck has quit IRC17:30
*** openstackrecheck has joined #openstack-qa17:30
oomichisdague: humm, maybe no. IIUC, old tempest had the code, but that didn't work at all. Then we removed that17:33
oomichisdague: let me check the history17:33
*** tqtran has joined #openstack-qa17:34
*** yaume has quit IRC17:34
openstackgerritJens Rosenboom proposed openstack-dev/devstack: Work around glance ignoring domain names in config  https://review.openstack.org/36967517:35
oomichisdague: tempest has utf-8 test for nova images api: https://review.openstack.org/#/c/122354/17:38
oomichisdague: but not for the serve rename. do we need that now?17:38
*** tqtran has quit IRC17:38
*** yamahata has joined #openstack-qa17:38
sdagueoomichi: 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 server17:40
*** ddieterly is now known as ddieterly[away]17:40
oomichisdague: how about https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_servers.py#L98 ?17:41
sdagueoomichi: right, that call triggers an odd stack trace now in the gate, which is non fatal, but odd17:41
sdaguefor exactly this reason17:41
oomichisdague: oh, that seems interesting. can I see the actual log of the gate?17:43
sdaguehttp://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_75217:43
oomichisdague: oh, https://review.openstack.org/#/c/370132 seems cool17:49
sdagueoomichi: yeh... though it's not having the intended effect17:49
sdaguewhich I'm scratching my head on17:49
openstackrecheckopenstack/nova change: https://review.openstack.org/314811 failed gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial in the gate queue with an unrecognized error17:50
*** ddieterly[away] has quit IRC17:50
oomichisdague: heh, but this situation seems your expectation(detecting unexpected issues) from the commit msg17:51
*** tqtran has joined #openstack-qa17:54
sdaguewell, I'm still getting an ascii decode error on the compute service17:56
sdaguewhich confuses me17:56
*** clu_ has joined #openstack-qa17:56
sdaguebecause it should be running in a utf8 locale17:56
*** hongbin has joined #openstack-qa18:03
*** mvk has joined #openstack-qa18:03
*** itsuugo has quit IRC18:19
*** itsuugo has joined #openstack-qa18:20
lbeliveaumriedem: 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
openstackgerritKen'ichi Ohmichi proposed openstack/qa-specs: Remove stress-api-tracking  https://review.openstack.org/37035718:20
*** yamamoto has joined #openstack-qa18:25
oomichilbeliveau: thanks, I prefer the purpose of the test itself. That will be helpful for productions.18:28
oomichilbeliveau: 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 code18:30
oomichilbeliveau: 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 test18:31
mriedemlbeliveau: comments inline18:31
lbeliveauoomichi: thanks, just to be clear, we should have both an API test and a scenario test ?18:32
mriedemoomichi: 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 migration18:32
mriedemif it were an api test, i guess it would be gated on CONF.run_validation?18:32
oomichimriedem: yeah, right. current test case fits to scenario test. but Tempest doesn't have more simple test in API test18:33
*** artom has quit IRC18:33
oomichimriedem: I just want to see the  API test is stable on the gate before adding the complex test18:34
*** yamamoto has quit IRC18:34
*** artom has joined #openstack-qa18:35
mugsiecan I get some e-r cores to review https://review.openstack.org/#/c/370271/ please?18:35
lbeliveaumriedem: oomichi: thanks !  I'll work on adding an API test18:36
mriedemmugsie: on it18:38
mugsiemriedem: ty18:39
mriedemmugsie: comments inline18:43
openstackgerritJens Rosenboom proposed openstack-dev/devstack: DNM: Dummy patch to test keystone fernet patch  https://review.openstack.org/37036918:45
openstackgerritGraham Hayes proposed openstack-infra/elastic-recheck: Add query for bug 1623576  https://review.openstack.org/37027118:47
openstackbug 1623576 in Designate "test_zone_create_propagates_to_nameservers times out before zone is fully propagated" [Medium,Triaged] https://launchpad.net/bugs/162357618:47
mugsiemriedem: I am getting 4 jobs - should I move them to a nested OR block?18:48
xinlirodrigods: Could you please help to review the code for 285640?18:48
xinlirodrigods: the link: https://review.openstack.org/#/c/285640/18:49
openstackgerritGraham Hayes proposed openstack-infra/elastic-recheck: Add query for bug 1623576  https://review.openstack.org/37027118:50
openstackbug 1623576 in Designate "test_zone_create_propagates_to_nameservers times out before zone is fully propagated" [Medium,Triaged] https://launchpad.net/bugs/162357618:50
mriedemmugsie: oh hrm well at least 2 of those are non-voting18:51
mriedemwhich e-r is going to filter out anyway18:51
mugsieyeah - I relasied that as I added them18:52
mriedemyou can filter in kibana with AND voting:118:52
mriedemwhich leaves us with 218:52
mugsieyeap, I just added them there, and removed the test ids18:53
mriedem+W18:54
mugsiethanks for your help :)18:55
*** sshnaidm is now known as sshnaidm|afk18:55
openstackgerritMerged openstack-infra/elastic-recheck: Add query for bug 1623576  https://review.openstack.org/37027118:59
openstackbug 1623576 in Designate "test_zone_create_propagates_to_nameservers times out before zone is fully propagated" [Medium,Triaged] https://launchpad.net/bugs/162357619:00
mriedemmtreinish: are you super duper happy with this one yet? https://review.openstack.org/#/c/305120/19:02
*** Pewlet_Hackard has joined #openstack-qa19:06
*** pcaruana has joined #openstack-qa19:07
*** chandankumar has quit IRC19:15
*** r-mibu has quit IRC19:16
mriedemshouldn'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-qa19:16
*** ddieterly has joined #openstack-qa19:20
openstackgerritMerged openstack/tempest: Fix the docstring of _default_security_group  https://review.openstack.org/36975119:23
openstackgerritMerged openstack/tempest: Remove unnecessary resource_setup()  https://review.openstack.org/36987119:23
*** cdent has joined #openstack-qa19:25
*** akshai has quit IRC19:28
*** gvrangan has quit IRC19:31
*** Apoorva_ has joined #openstack-qa19:33
*** grumble has joined #openstack-qa19:36
*** csomerville has quit IRC19:36
*** Apoorva has quit IRC19:36
*** csomerville has joined #openstack-qa19:37
*** cody-somerville has joined #openstack-qa19:38
*** dkranz has quit IRC19:39
*** dkranz has joined #openstack-qa19:40
*** salv-orl_ has joined #openstack-qa19:40
*** csomerville has quit IRC19:41
*** salv-orlando has quit IRC19:43
*** akshai has joined #openstack-qa19:45
*** akshai_ has joined #openstack-qa19:46
*** Apoorva_ has quit IRC19:47
*** Apoorva has joined #openstack-qa19:47
*** xinli has quit IRC19:48
*** porrua has quit IRC19:48
*** akshai has quit IRC19:49
openstackrecheckopenstack/nova change: https://review.openstack.org/363926 failed gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial in the gate queue with an unrecognized error19:50
*** pcaruana has quit IRC20:00
*** Pewlet_Hackard has quit IRC20:02
*** gouthamr has quit IRC20:07
*** Ravikiran_K has quit IRC20:07
*** Apoorva has quit IRC20:15
*** openstackrecheck has quit IRC20:15
*** Apoorva has joined #openstack-qa20:16
*** openstackrecheck has joined #openstack-qa20:16
*** cdent has quit IRC20:16
*** gvrangan has joined #openstack-qa20:23
erlonhey guys, mriedem, I needed something like a @ddt in tempest, is it possible?20:26
mriedemddt in tempest was brought up awhile back and nacked i think20:27
mriedemsince you can use testscenarios20:27
mtreinishmriedem: 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 used20:28
*** e0ne has joined #openstack-qa20:30
mriedemright, remember, no PhD in testscenarios (tm)20:32
mriedemi thought about getting one from trump univsersity though20:32
mriedem*university20:32
sdagueno, you spelled it right the first time20:32
mriedemtouche20:33
erlonmtreinish: mriedem: hmm, yeah, I was already guessing that tempest guys would have some resistance about that20:33
mriedemthose guys are terrible20:33
erlonmriedem: 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 combinations20:34
erlonA <-> B, A <-> C, B <-> C20:35
erlonmriedem: mtreinish: I could write this inside the test, but that could make the test to be very very long20:36
*** rcernin has quit IRC20:36
*** gfidente has quit IRC20:36
mtreinisherlon: 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 difficult20:38
mtreinisherlon: don't we already have multibackend cinder tests in tempest?20:38
erlonmtreinish: the test is blocked waiting a gate job, but it only tests 2 backends20:40
*** rcernin has joined #openstack-qa20:40
erlonmtreinish: https://review.openstack.org/#/c/19544320:40
*** rcernin has quit IRC20:41
erlonif I pass CONF.volume.backend_names = 'lvm, ceph, gluster, bdd, nfs', for example, it will only test lvm x ceph20:41
*** rcernin has joined #openstack-qa20:41
*** rcernin has quit IRC20:42
*** rcernin has joined #openstack-qa20:42
*** gouthamr has joined #openstack-qa20:44
mtreinisherlon: so isn't that a bug in the test as proposed (assuming it's supposed to check all the permutations)20:44
mtreinishyou'd just need 2 for loops right?20:45
*** e0ne has quit IRC20:45
erlonmtreinish: 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 failed20:49
erlonmtreinish: so, if was possible to generate tests, we could have test_available_volume_retype_with_migration_a_vs_b, ..a_vs_c,etc20:50
erlonmtreinish: which would be much easier to debug20:50
*** akshai_ has quit IRC20:51
erlonmtreinish: what is the issue you mentioned about debugging tests generated like that?20:51
mtreinisherlon: 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 IRC20:53
mtreinisherlon: 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 code20:53
mtreinishwhich is optimizing for the wrong problem space20:54
mtreinishespecially 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
erlonmtreinish: testscenarios can use string to identify the tests?20:55
erlonmtreinish: 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
erlonmtreinish: surely that shouldn't be the kind of thing you do in lots of tests, just some special cases like this20:59
mtreinisherlon: yes, that's normally how testscenarios is used: https://github.com/openstack-infra/subunit2sql/blob/master/subunit2sql/tests/db/test_api.py#L27-L3620:59
mtreinisherlon: 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 tempest21:00
mtreinisherlon: because fundamentally you want to compat test migrations between all the open source backends right?21:00
mtreinisherlon: which is completely achievable with a bash script with a for loop updating a line in the tempest.conf and rerunning the one test21:01
erlonmtreinish: and the job running this test will be called solo, without all other tempest, to make the job shorter21:01
mtreinisherlon: oh, if you're splitting it out into a seperate job then why do we need to bake this logic inside tempest?21:01
erlonmtreinish: exactly, I don't think is possible to do that in the gate scripts21:03
mtreinishwhy not?21:03
*** xinli has joined #openstack-qa21:03
erlonmtreinish: haven't tried actually, my first option was to implement it in tempest21:03
*** gyee has quit IRC21:04
mtreinishrunning 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
erlonmtreinish: so, I needed to know if that is an option, and if not, what suggestions you guys would give21:04
mtreinisherlon: 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
mtreinishwhich was a case that testscenarios was used for befor in tempest but was removed for the reasons I outlined before21:05
mtreinishso you might want to talk to them about a having a generic framework for that in the future21:06
erlonmtreinish: 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 too21:06
erlonmtreinish: mhm21:06
mtreinisherlon: 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 loop21:07
mtreinisherlon: here's an example: https://github.com/openstack/nova/blob/master/nova/tests/live_migration/hooks/utils.sh#L3 of using a hook21:08
mtreinisherlon: 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 loop21:08
erlonmtreinish: ok, ill try that, are those hooks called by the gate scripts?21:09
mtreinisherlon: yes: https://github.com/openstack-infra/project-config/blob/51b85065d342ebc8c53df85ca467cec633b2c955/jenkins/jobs/devstack-gate.yaml#L143421:09
erlonmtreinish: that would not work if I use concurrency will?21:10
mtreinishyou 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 method21:11
erlonmtreinish: mhm21:12
erlonmtreinish: if this is a pre_test_hook, won't the gate scripts call the actual tests again later?21:13
mtreinisherlon: you could cheat and backround each 'tempest run' command and run multiple in parallel that way21:13
mtreinisherlon: it's a post test hook and you set TEMPEST_NOTESTS in the job definition so devstack-gate doesn't run the tests21:13
erlonmtreinish: hmmmm21:15
erlonmtreinish: great I think that's enough to diggest for a while :)21:15
erlonmtreinish: thank very much21:16
mtreinishsure, np21:16
erlonmtreinish: ill get back later if (when) I have any problem21:16
*** rossella_ has quit IRC21:18
*** dkranz has quit IRC21:19
*** rossella_ has joined #openstack-qa21:19
*** edmondsw has quit IRC21:21
openstackgerritClark Boylan proposed openstack-infra/devstack-gate: Explicitly set overlay bridge ints as up  https://review.openstack.org/37044221:29
*** ddieterly[away] is now known as ddieterly21:31
*** gvrangan has joined #openstack-qa21:32
*** mriedem has quit IRC21:40
*** rlrossit has quit IRC21:47
*** dave-mccowan has quit IRC21:48
*** Swami has quit IRC21:49
*** krtaylor has quit IRC22:02
*** rbak has quit IRC22:03
*** Gen has joined #openstack-qa22:06
openstackrecheckopenstack/nova change: https://review.openstack.org/366872 failed gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial in the gate queue with an unrecognized error22:10
*** pglass has quit IRC22:16
*** gyee has joined #openstack-qa22:17
armaxmtreinish: ping22:18
mtreinisharmax: pong22:18
armaxmtreinish: wanted to run something by you22:18
armaxmtreinish: bug 1623691 shows something very unfortunate22:19
openstackbug 1623691 in neutron "security group in use conflict during test cleanup" [High,New] https://launchpad.net/bugs/162369122:19
*** yamahata has quit IRC22:20
armaxmtreinish: it looks like the default security group is being deleted before the deletion of the port using it is issued22:20
mtreinishhmm, that's odd, we should be pretty careful about the ordering there22:20
armaxI am trying to track down where in tempest the delete_port and delete_security_group happen22:20
armaxin DynamicCredentialProvider I can see _cleanup_default_secgroup22:20
armaxbut I don’t see the delete_port22:21
*** yamahata has joined #openstack-qa22:21
mtreinisharmax: https://github.com/openstack/tempest/blob/master/tempest/common/dynamic_creds.py#L363-L42022:21
mtreinishany created ports (for servers) would be cleaned up in the tests that created them22:21
armaxmtreinish: um22:22
mtreinisha 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 ago22:22
armaxthe port in question is a compute port22:22
armaxI wonder if Nova is lagging behind22:22
mtreinisharmax: hmm, yeah that sounds like an issue in the test or nova22:22
*** mtanino has quit IRC22:23
armaxmtreinish: looking at the exact life cycle of the port in question22:24
armaxmtreinish: this might be a Neutron bug after all :)22:25
*** itsuugo has quit IRC22:26
armaxwhat I can see is that the port_delete is interleaving with the security_group deletion22:27
*** itsuugo has joined #openstack-qa22:28
mtreinishah, ok so the 2 deletes are racing with each other22:29
armaxmtreinish: it looks like they do22:29
mtreinishdoes 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
armaxmtreinish: it looks like they might not wait22:30
armaxhttp://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_47522:30
mtreinishbecause 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 40422:30
armaxhere the plugin is processing the delete22:30
armaxa few ms after tempest tries to delete the default secgroup22:31
armaxhttp://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_84522:31
armaxa few more ms after22:31
armaxneutron says the port is deleted22:31
armaxhttp://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_33722:31
armaxfor godo22:31
armaxgood22:31
mtreinishhmm, yeah that seems like something is not waiting in the nova <-> neutron communication waiting on it to go away22:35
armaxmtreinish: that’s the only trace I could find for as long as logstash can remember22:36
*** Apoorva_ has joined #openstack-qa22:36
armaxmtreinish: I’ll tag nova on the bug report, and mark Tempest as invalid22:37
mtreinisharmax: hmm, it's only hitting on one patch and it's the check queue22:37
armaxmtreinish: true, but it’s a geniune error I believe22:38
armaxmtreinish: because the error happens on a job that has nothing to do with the change22:38
armaxOVS+DVR vs patch being LinuxBridge only22:38
armaxmtreinish: it’s sheer bad luck I guess22:39
mtreinishyeah, I was just looking at the change. lb vs ovs22:39
mtreinishit's unlikely to cause the problem :)22:39
armaxmtreinish: very :)22:39
*** Apoorva has quit IRC22:39
armaxmtreinish: I suspect the vm deletion and port deletion are async22:41
*** rcernin has quit IRC22:43
armaxmtreinish: ok, thanks for the support22:43
armaxI’ll loop in mriedem22:43
armaxand try to figure out the best course of action22:43
*** Swami has joined #openstack-qa22:45
mtreinisharmax: 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 port22:45
mtreinishso if that doesn't wait for it to go away I could see that being the problem22:45
armaxmtreinish: yeah, I figured22:45
armaxmtreinish: I am surprised this went undetected for so long22:46
*** Swami has quit IRC22:46
*** Swami has joined #openstack-qa22:46
mtreinishunless 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 from22:47
armaxmtreinish: there’s a network-vif-deleted event emitted by Neutron22:49
armaxthough22:49
dansmitharmax: what's the issue?22:50
mtreinisharmax: yeah I wasn't looking for that, just where the delete was issued. I figured there might be a callback to sync things up22:50
armaxdansmith: bug 162369122:50
openstackbug 1623691 in neutron "security group in use conflict during test cleanup" [High,New] https://launchpad.net/bugs/162369122:50
armaxdansmith: it looks like the delete port, after a server delete is racing with the deletion of the default security group22:51
dansmitharmax: that event just informs compute manager if a port is deleted underneath a running instance22:52
*** itsuugo has quit IRC22:52
dansmitharmax: 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 returning22:52
dansmithi.e. delete is not a synchronous operation22:52
dansmithis that what you are thinking is the problem?22:52
dansmiththat we don't block tempest until the thing is really gone?22:52
*** itsuugo has joined #openstack-qa22:53
mtreinishdansmith: tempest does a poll loop waiting for a 40422:53
dansmiththe thing we do wait for is instance shutdown and volume detach so we don't corrupt a filesystem in a volume22:53
*** ddieterly has quit IRC22:53
dansmithmtreinish: oh okay, then I'm confused I guess22:53
*** zackf has quit IRC22:53
armaxdansmith: so if we don’t wait for the networking part to go away too22:53
armaxI can see the problem22:53
dansmithoh, I see,22:54
dansmiththe instance is marked as deleted, you get a 404, but we're not done waiting internally for cleanup22:54
armaxin relation to the test cleanup logic22:54
armaxthat attempts to delete the project’s resources like teh default security group22:54
*** zackf has joined #openstack-qa22:54
armaxdansmith: bear in mind everything happened in teh span of less than half a second22:55
*** xinli has quit IRC22:55
dansmithyup22:56
armaxdansmith, 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 group23:00
dansmithyeah23:00
*** dustins has quit IRC23:00
dansmithI'm not really sure what else to do23:00
armaxand fail if the conflict is permanent23:00
dansmithyeah23:00
dansmitharmax: so theoretically we have made the port delete call by the time we set our deleted=true flag23:01
dansmitharmax: is port delete in neutron synchronous or no?23:02
armaxdansmith: the DB object goes away when the API responds23:02
dansmithhrm, okay23:02
dansmitharmax: so we do "try" to delete23:02
*** sdague has quit IRC23:02
dansmitharmax: maybe we failed for some reason?23:03
dansmithalthough we should go into error state if that happens, so nevermind23:03
armaxdansmith: I see no trace in the cpu log23:03
dansmithso tbh, I'm not sure how that is happening, really23:03
armaxdansmith: in this particular instance23:07
armaxthe response to delete port happens after the attempt to delete the security group23:07
armaxdefault one23:07
armaxhttp://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_33723:08
armaxhttp://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_84523:08
dansmithartom: 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 happened23:08
dansmither, armax ^23:09
*** itsuugo has quit IRC23:10
armaxdansmith: but Nova returns 404 on an active poll from Tempest to check that the instance is gone23:10
dansmithmtreinish: you're sure it's 404 and not v_status=DELET(ed,ing) right?23:10
armaxdansmith: if Nova API does not wait for compute to be done, I could see a problem?23:11
*** itsuugo has joined #openstack-qa23:11
dansmitharmax: 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 volumes23:12
mtreinishdansmith: pretty sure, let me pull up the link23:12
armaxdansmith: ok23:12
dansmitharmax: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2271-L233423:13
mtreinishdansmith: 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_05223:13
dansmitharmax: you get a 404 after L2336 has run23:13
armaxmtreinish: btw I couldn’t find the tempest log for the new run of greande23:13
armaxgrenade23:13
dansmitharmax: and the network port delete happens in _shutdown_instance() which is called on L230323:13
mtreinishdansmith: oops that's the wrong test class23:13
mtreinishlet me find the equiv for the test in question23:13
mtreinisharmax: it's in the base logs/ dir23:14
armaxmtreinish: that’s the one for the old run23:15
armaxmtreinish: if you’re referring to http://logs.openstack.org/77/346377/14/check/gate-grenade-dsvm-neutron-dvr-multinode/056b1cf/logs/23:15
armaxmtreinish: that ends at 8.5423:15
armaxmtreinish: the test faillure happens 20 mins later23:16
mtreinisharmax: for old: http://logs.openstack.org/77/346377/14/check/gate-grenade-dsvm-neutron-dvr-multinode/056b1cf/logs/old/tempest.txt.gz23:16
mtreinisharmax: for new: http://logs.openstack.org/77/346377/14/check/gate-grenade-dsvm-neutron-dvr-multinode/056b1cf/logs/tempest.txt.gz23:16
mtreinisharmax: the failure traceback prints at the end of the test run23:16
mtreinishafter all the other tests have finished (on the console output)23:16
armaxmtreinish: then something if funky when I am opening this in my browser23:17
oomichislowrie: hi, I guess https://blueprints.launchpad.net/tempest/+spec/list-plugins is complete with https://review.openstack.org/#/c/251542/ , right?23:17
armaxmtreinish: I am looking at the timestamps in the neutron server log23:17
oomichislowrie: or do we need more work on that?23:17
slowrieoomichi: yes23:17
oomichislowrie: which for yes ? done? :)23:18
slowriedone23:18
oomichislowrie: cool, thanks23:18
armaxmtreinish: oh never mind23:18
armaxmtreinish: I am with yu23:18
armax*you23:18
mtreinisharmax: heh, ok23:19
*** hongbin has quit IRC23:20
*** Swami has quit IRC23:21
*** Swami has joined #openstack-qa23:21
*** jamielennox is now known as jamielennox|away23:22
openstackgerritKen'ichi Ohmichi proposed openstack/qa-specs: Move implemented blueprints to implemented  https://review.openstack.org/37047623:25
mtreinishdansmith: hmm, I actually can't find the wait loop in the log23:25
*** Suyi_ has quit IRC23:25
dansmithmtreinish: I caught you! :)23:25
mtreinishoh I think I see the bug now, it's in tempest23:26
mtreinishI have no idea how that didn't get caught though23:26
dansmith*gasp*23:26
dansmithlink?23:26
armaxmtreinish: enlighten me!23:27
*** ababich has quit IRC23:27
dansmitharmax: ahem.. enlighten US23:27
armax:)23:27
*** ababich has joined #openstack-qa23:27
oomichislowrie: sorry, one more.  https://blueprints.launchpad.net/tempest/+spec/centralized-workspaces also is done now, right23:27
slowrietes23:28
slowrieyes*23:28
openstackgerritMatthew Treinish proposed openstack/tempest: Add wait on server delete  https://review.openstack.org/37047923:28
mtreinishdansmith, armax: ^^^23:28
oomichislowrie: thanks again :)23:28
dansmithmtreinish: ah23:29
mtreinishI'm surprised we're not hitting that more on neutron grenade jobs. Since it should be pretty easy to trigger on jobs that run smoke23:29
armaxwaaat?23:29
openstackgerritKen'ichi Ohmichi proposed openstack/qa-specs: Move implemented blueprints to implemented  https://review.openstack.org/37047623:29
mtreinishsince there is only one test in that class tagged as smoke23:29
*** Swami_ has joined #openstack-qa23:30
mtreinishI should have let on of you push that so I could have +2'd it :/23:30
armaxmtreinish: abandon :)23:31
*** vhosakot has quit IRC23:31
armaxmtreinish: nice catch btw23:31
mtreinishdansmith, armax: heh, actually take over the patch I actually missed it we have the same bug in the other create server call in that class23:32
mtreinishthen I'll +2 that23:32
mtreinishthat's kosher right?23:32
*** Swami has quit IRC23:32
armaxmtreinish: I suppose23:33
mtreinishI 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#L17823:34
mtreinishI'm trying to think of a scenario where we wouldn't want to wait on the cleanup23:34
openstackgerritArmando Migliaccio proposed openstack/tempest: Add wait on server delete  https://review.openstack.org/37047923:34
gmannoomichi: ping23:35
oomichigmann: pong23:35
gmannoomichi: on credential client, i am making it to return raw response in https://review.openstack.org/#/c/369839/123:35
*** rfolco has quit IRC23:36
openstackgerritGhanshyam Mann proposed openstack/tempest: Revert "Define v3 credentials_client as library"  https://review.openstack.org/36911223:36
*** Apoorva_ has quit IRC23:36
*** Apoorva has joined #openstack-qa23:37
mtreinishdansmith, 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 wait23:37
armaxmtreinish: ack23:37
oomichigmann: yeah +2, thanks :)23:38
gmannoomichi: Thanks.23:38
gmannoomichi: also we need this as base patch  - https://review.openstack.org/#/c/369112/23:38
gmannafter that i will reiterate the client and if all looks stable then move23:39
oomichigmann: why do we need to revert that? the client returns raw response as you did on the above patch23:39
gmannoomichi: so it was moved to lib with incorrect interfaces23:40
gmannoomichi: so we decided to revert from lib so that it does not get release23:40
gmannoomichi: and after revert from lib then we will fix as per keystone feedback and then move as stable interface23:41
oomichigmann: I have different thinking. Service clients need to return raw response as possible without weird conversions.23:41
oomichigmann: that is consistent on nova service clients also as you know. hypervisors client23:42
oomichigmann: from this viewpoint, v3 credentials_client is good interface23:42
gmannoomichi: yea thats my plan, returning raw response but current interface in lib does nto do that23:44
gmannoomichi: so i reverted back and then fix23:45
gmannoomichi: or you want to fix in lib only23:45
oomichigmann: ah, which tempest.lib client doesn't return raw response?23:46
gmannoomichi: this is current one in lib - https://github.com/openstack/tempest/blob/master/tempest/lib/services/identity/v3/credentials_client.py#L3923:46
gmannoomichi: it does not and we were little bit in hurry to merge that in lib23:46
gmannoomichi: i noticed that while doing UT for those23:46
oomichigmann: ah, I missed the patch dependency..23:46
oomichigmann: credentials_client is not released yet, so we can fix it directly without reverting I think.23:47
gmannoomichi: ok, we can do that but not good practice seems like :)23:48
*** mriedem has joined #openstack-qa23:48
oomichigmann: yeah, I agree. Sorry about that23:49
gmannoomichi: ok, so let me fix it in lib only then we can see if anything else also needed23:49
gmannoomichi: ah no you should not be sorry, it was me approved that :)23:49
oomichigmann: related to that, can you take a look at https://review.openstack.org/#/c/368147/ ?23:49
oomichigmann: current docstring's link is separated to 2 lines and that is not readable if we create doc23:50
gmannoomichi: sure, i will review in afternoon. some management request today :)23:50
gmannoomichi: for api-ref link, yea thats true23:50
oomichigmann: the above patch makes them into a single as the second purpose23:51
openstackrecheckConsole logs not available after 16:18s for gate-grenade-dsvm-neutron-ubuntu-trusty 369110,6,cd8bf2423:51
oomichigmann: 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.compute23:52
oomichiwe cannot provide actual links if creating doc..23:52
gmannoomichi: ah nice catch.23:52
gmannoomichi: and along with that define the mandatory param explicitly right?23:53
oomichigmann: thanks, management request is important *sometimes* :-)23:53
gmannoomichi: i am very frustrated from some actually which are just for showcase only no actual meaning :)23:53
oomichigmann: hehe, please don't say company internal thing on public channel. This is logged ;)23:54
gmannheh23:54
gmannbut thats general things for all company23:55
oomichiyeah, that is true23:56
*** hieulq has quit IRC23:56
mtreinishoomichi: even if it wasn't looged most people will log it anyway... I have irclogs locally going back years :)23:57
*** mmedvede has quit IRC23:57
*** itsuugo has quit IRC23:57
oomichimtreinish: I need to go to your home to break it down ;)23:58
*** itsuugo has joined #openstack-qa23:58
mtreinishthere are many copies23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!