Monday, 2016-04-18

openstackgerritJamie Lennox proposed openstack-dev/devstack: Allow putting service users in a seperate domain  https://review.openstack.org/27105100:00
*** vhosakot has quit IRC00:03
*** ober37_ has joined #openstack-qa00:05
*** salv-orlando has quit IRC00:06
*** ober37 has quit IRC00:07
*** mylu has quit IRC00:07
*** mylu has joined #openstack-qa00:13
*** outofmemory is now known as reedip00:17
*** ober37 has joined #openstack-qa00:18
*** ober37_ has quit IRC00:18
*** mylu has quit IRC00:19
*** mylu has joined #openstack-qa00:21
*** nwkarste_ has quit IRC00:24
*** Jae00 has quit IRC00:25
*** ober37 has quit IRC00:28
*** ober37 has joined #openstack-qa00:28
*** ober37 has quit IRC00:33
*** ober37 has joined #openstack-qa00:34
openstackgerritzhufl proposed openstack/tempest: Set timeout value in urllib3.poolmanager.PoolManager  https://review.openstack.org/29442500:38
*** vhosakot has joined #openstack-qa00:40
*** lekha has quit IRC00:41
*** mylu has quit IRC00:44
*** ober37 has quit IRC00:44
*** ober37 has joined #openstack-qa00:46
*** mylu has joined #openstack-qa00:47
*** vhosakot has quit IRC01:00
*** ober37 has quit IRC01:04
*** yangyape_ has joined #openstack-qa01:06
*** ober37 has joined #openstack-qa01:14
*** ober37_ has joined #openstack-qa01:17
*** ober37 has quit IRC01:18
*** vhosakot has joined #openstack-qa01:22
*** ober37_ has quit IRC01:22
*** ober37 has joined #openstack-qa01:22
*** yangyape_ has quit IRC01:23
*** nwkarsten has joined #openstack-qa01:36
*** mriedem has quit IRC01:39
*** lekha has joined #openstack-qa01:41
*** nwkarsten has quit IRC01:41
openstackgerritMerged openstack-dev/devstack: Updated from generate-devstack-plugins-list  https://review.openstack.org/30670001:42
*** ober37 has quit IRC01:54
*** ober37 has joined #openstack-qa01:54
*** ober37 has quit IRC01:59
*** dkranz_ has quit IRC02:05
*** r-mibu has quit IRC02:09
*** mylu has quit IRC02:10
*** nwkarsten has joined #openstack-qa02:13
*** r-mibu has joined #openstack-qa02:14
*** nwkarsten has quit IRC02:17
*** alex_xu has quit IRC02:18
*** alex_xu has joined #openstack-qa02:22
openstackgerritMerged openstack-dev/devstack: Be a bit more explicit about the plugin generation  https://review.openstack.org/30139702:26
openstackgerritMerged openstack-dev/devstack: Fix Q_PLUGIN_EXTRA_CONF_FILES usage comment  https://review.openstack.org/30654102:34
*** nadya has joined #openstack-qa02:39
*** nadya has quit IRC02:43
*** dimtruck is now known as zz_dimtruck02:52
*** yamamoto_ has joined #openstack-qa02:59
*** amotoki has joined #openstack-qa03:11
*** yamamoto_ has quit IRC03:11
*** yamamoto_ has joined #openstack-qa03:21
*** binbincong has quit IRC03:25
*** tochi has quit IRC03:26
*** binbincong has joined #openstack-qa03:38
openstackgerritMerged openstack-dev/devstack: Move nova's api_paste_config into wsgi group  https://review.openstack.org/30378603:38
*** yamamoto_ has quit IRC03:43
*** sree has joined #openstack-qa03:49
*** tochi has joined #openstack-qa03:56
*** zackf has quit IRC04:05
*** salv-orlando has joined #openstack-qa04:09
*** vhosakot has quit IRC04:10
*** amotoki has quit IRC04:11
*** salv-orlando has quit IRC04:13
*** itzikb has joined #openstack-qa04:15
*** amotoki has joined #openstack-qa04:22
*** yamamoto_ has joined #openstack-qa04:27
*** amotoki has quit IRC04:30
*** sree has quit IRC04:37
*** amotoki has joined #openstack-qa04:38
*** salv-orlando has joined #openstack-qa04:48
*** Poornima has joined #openstack-qa04:54
*** ober37 has joined #openstack-qa04:56
*** ober37 has quit IRC05:00
*** amotoki has quit IRC05:01
*** mylu has joined #openstack-qa05:03
*** rcernin has joined #openstack-qa05:04
*** Alex_Stef has joined #openstack-qa05:07
*** amotoki has joined #openstack-qa05:09
*** shausy has joined #openstack-qa05:11
*** yangyape_ has joined #openstack-qa05:11
*** mylu has quit IRC05:11
*** VanL has quit IRC05:12
*** VanL has joined #openstack-qa05:12
*** e0ne has joined #openstack-qa05:15
*** e0ne has quit IRC05:18
*** ramishra has joined #openstack-qa05:21
*** shausy has quit IRC05:23
*** mylu has joined #openstack-qa05:24
*** shausy has joined #openstack-qa05:26
*** bkopilov_ has joined #openstack-qa05:27
*** ramishra has quit IRC05:28
*** mylu has quit IRC05:29
*** ramishra has joined #openstack-qa05:30
openstackgerritMasayuki Igawa proposed openstack-infra/subunit2sql: Capitalize first characters in param docs  https://review.openstack.org/30690805:30
*** tpatzig_ has quit IRC05:35
*** zackf has joined #openstack-qa05:36
*** mkoderer has quit IRC05:36
*** mkoderer has joined #openstack-qa05:39
*** zackf has quit IRC05:42
*** davideagnello has joined #openstack-qa05:49
*** davideagnello has quit IRC05:51
*** ramishra has quit IRC05:52
*** ramishra has joined #openstack-qa05:55
*** ober37 has joined #openstack-qa05:57
*** itzikb has quit IRC05:58
*** tpatzig has joined #openstack-qa05:59
*** ober37 has quit IRC06:01
*** fedexo has joined #openstack-qa06:11
*** Poornima has quit IRC06:16
*** cody-somerville has quit IRC06:17
*** apevec has joined #openstack-qa06:18
*** apevec has quit IRC06:18
*** apevec has joined #openstack-qa06:18
*** apevec has left #openstack-qa06:18
openstackgerritGhanshyam Mann proposed openstack/tempest: Fix rest_client's expected_success for non int status  https://review.openstack.org/30691906:22
openstackgerritGhanshyam Mann proposed openstack/tempest: Fix rest_client's expected_success for non int status  https://review.openstack.org/30691906:23
*** sridharg has joined #openstack-qa06:31
*** alex_xu has quit IRC06:40
*** eliqiao has quit IRC06:40
*** itzikb has joined #openstack-qa06:40
*** alex_xu has joined #openstack-qa06:41
*** eliqiao has joined #openstack-qa06:42
openstackgerritOpenStack Proposal Bot proposed openstack-dev/devstack: Updated from generate-devstack-plugins-list  https://review.openstack.org/30693206:53
*** neeti has joined #openstack-qa06:56
*** ttx_ is now known as ttx07:06
*** abehl has joined #openstack-qa07:14
*** dtantsur|afk is now known as dtantsur07:14
*** pcaruana has joined #openstack-qa07:15
*** e0ne has joined #openstack-qa07:16
openstackgerritMasayuki Igawa proposed openstack/os-testr: Fix docs typos  https://review.openstack.org/30694507:22
*** Alex_Stef has quit IRC07:29
*** sridharg has quit IRC07:30
*** ifarkas has joined #openstack-qa07:32
*** jlanoux has joined #openstack-qa07:36
*** lezbar has quit IRC07:39
*** sridharg has joined #openstack-qa07:41
*** nadya has joined #openstack-qa07:48
*** rossella_s has joined #openstack-qa07:48
*** alex_xu has quit IRC07:49
*** alex_xu has joined #openstack-qa07:51
*** vgridnev has joined #openstack-qa07:54
*** vgridnev has quit IRC07:56
*** bdperkin has joined #openstack-qa07:57
*** bdperkin has joined #openstack-qa07:57
*** ober37 has joined #openstack-qa07:59
*** pilgrimstack has joined #openstack-qa08:01
*** ober37 has quit IRC08:03
*** kbaikov has quit IRC08:04
*** shardy has joined #openstack-qa08:06
openstackgerritMasayuki Igawa proposed openstack/qa-specs: Remove tempest run from cli improvements  https://review.openstack.org/30698508:10
*** sshnaidm has quit IRC08:12
*** fedexo has quit IRC08:12
*** lezbar has joined #openstack-qa08:14
*** Serlex has joined #openstack-qa08:18
*** kbaikov has joined #openstack-qa08:22
*** drptbl has joined #openstack-qa08:27
*** gfidente has joined #openstack-qa08:31
openstackgerritDong Ma proposed openstack-infra/subunit2sql: Add db api tests for subunit2sql (1/3)  https://review.openstack.org/30503008:37
*** mpaolino has joined #openstack-qa08:41
*** jpena|off is now known as jpena08:49
*** cdent has joined #openstack-qa08:51
*** sridharg has quit IRC08:53
*** zackf has joined #openstack-qa08:58
*** sridharg has joined #openstack-qa08:59
*** sridharg has quit IRC08:59
*** yamamoto_ has quit IRC09:02
*** yamamoto has joined #openstack-qa09:02
*** zackf has quit IRC09:03
*** sridharg has joined #openstack-qa09:03
*** tesseract has joined #openstack-qa09:03
*** tesseract is now known as Guest3039409:03
*** Deng has joined #openstack-qa09:07
*** drptbl_ has joined #openstack-qa09:08
*** drptbl has quit IRC09:09
gmannoomichi:  mtreinish: hope this is fine w.r.t.backward compatibility - https://review.openstack.org/#/c/306919/209:10
gmannoomichi:  mtreinish:  it will fail for users using that method wrongly.09:10
gmannwith current code it pass falsely09:11
*** neeti has quit IRC09:13
*** neeti has joined #openstack-qa09:24
*** sshnaidm has joined #openstack-qa09:29
*** neeti has quit IRC09:41
*** neeti has joined #openstack-qa09:41
*** cdent has quit IRC09:44
*** mpaolino has quit IRC09:49
*** mpaolino has joined #openstack-qa09:50
*** tosky has joined #openstack-qa09:57
*** Alex_Stef has joined #openstack-qa10:03
*** salv-orl_ has joined #openstack-qa10:04
*** gszasz has joined #openstack-qa10:06
*** salv-orlando has quit IRC10:07
*** yangyap__ has joined #openstack-qa10:09
*** tpatzig has left #openstack-qa10:10
*** yangyape_ has quit IRC10:13
*** ober37 has joined #openstack-qa10:13
*** yangyap__ has quit IRC10:13
*** cdent has joined #openstack-qa10:14
*** sdague has joined #openstack-qa10:15
*** ober37 has quit IRC10:20
*** neeti_ has joined #openstack-qa10:32
*** neeti has quit IRC10:33
openstackgerritBenny Kopilov proposed openstack/tempest: Cinder storage pools tests  https://review.openstack.org/30707810:33
*** bkopilov_ has quit IRC10:40
*** bkopilov_ has joined #openstack-qa10:40
*** jecarey has joined #openstack-qa11:00
*** SlickN1k has joined #openstack-qa11:01
*** marcusvrn__ has joined #openstack-qa11:02
*** zhiyan_ has joined #openstack-qa11:02
*** beekneemech has joined #openstack-qa11:04
*** krotscheck_ has joined #openstack-qa11:04
*** jgriffith_ has joined #openstack-qa11:04
*** lbragstad_ has joined #openstack-qa11:04
*** rodrigods has quit IRC11:04
*** rodrigods has joined #openstack-qa11:05
*** dmellado_ has joined #openstack-qa11:05
*** dalees` has joined #openstack-qa11:05
*** marcusvrn_ has quit IRC11:05
*** melwitt has quit IRC11:05
*** SlickNik has quit IRC11:05
*** jgriffith has quit IRC11:05
*** dtroyer has quit IRC11:05
*** lbragstad has quit IRC11:05
*** anthonyper has quit IRC11:05
*** dalees has quit IRC11:06
*** kinrui has quit IRC11:06
*** zhiyan has quit IRC11:06
*** harlowja has quit IRC11:06
*** krotscheck has quit IRC11:06
*** dtantsur has quit IRC11:06
*** dmellado has quit IRC11:06
*** ianw has quit IRC11:06
*** bnemec has quit IRC11:06
*** jecarey_ has quit IRC11:06
*** jesusaur has quit IRC11:06
*** SlickN1k is now known as SlickNik11:06
*** jgriffith_ is now known as jgriffith11:06
*** jgriffith is now known as Guest7277211:06
*** krotscheck_ is now known as krotscheck11:06
*** marcusvrn__ is now known as marcusvrn_11:07
*** dtroyer has joined #openstack-qa11:07
*** ianw has joined #openstack-qa11:08
*** dougwig has quit IRC11:09
*** sweston has quit IRC11:09
*** dtantsur has joined #openstack-qa11:09
*** anthonyper has joined #openstack-qa11:10
*** tosky_ has joined #openstack-qa11:10
*** gszasz has quit IRC11:10
*** melwitt has joined #openstack-qa11:10
*** zhiyan_ is now known as zhiyan11:10
*** melwitt is now known as Guest6400311:10
*** tosky has quit IRC11:12
*** rcernin has quit IRC11:12
*** Kennan has quit IRC11:13
*** sweston has joined #openstack-qa11:15
*** dougwig has joined #openstack-qa11:15
*** kinrui has joined #openstack-qa11:17
*** rcernin has joined #openstack-qa11:23
*** gszasz has joined #openstack-qa11:24
*** Kennan has joined #openstack-qa11:32
*** lucasagomes is now known as lucas-hungry11:43
*** dave-mccowan has joined #openstack-qa11:44
*** tochi has quit IRC11:46
*** weshay has joined #openstack-qa11:56
*** yamamoto has quit IRC12:01
*** mtreinish has joined #openstack-qa12:03
*** rfolco has joined #openstack-qa12:04
*** aysyd has joined #openstack-qa12:04
*** Poornima has joined #openstack-qa12:10
*** raildo-afk is now known as raildo12:12
*** zz_dimtruck is now known as dimtruck12:13
*** mylu has joined #openstack-qa12:15
*** markvoelker has joined #openstack-qa12:19
*** mylu has quit IRC12:26
*** edmondsw has joined #openstack-qa12:26
*** Poornima_K has joined #openstack-qa12:27
*** ddieterly has joined #openstack-qa12:29
*** ddieterly is now known as ddieterly[away]12:30
*** Poornima has quit IRC12:30
*** mylu has joined #openstack-qa12:33
*** jpena is now known as jpena|lunch12:36
*** yamamoto has joined #openstack-qa12:38
*** mylu has quit IRC12:39
*** ober37 has joined #openstack-qa12:39
*** ober37_ has joined #openstack-qa12:40
*** ober37 has quit IRC12:43
*** Deng has quit IRC12:44
*** lucas-hungry is now known as lucasagomes12:44
*** neeti_ has quit IRC12:44
*** yamamoto has quit IRC12:45
*** yamamoto has joined #openstack-qa12:45
*** jecarey has quit IRC12:50
*** Poornima_K has quit IRC12:50
*** yamamoto has quit IRC12:50
*** ddieterly[away] is now known as ddieterly12:53
*** yamamoto has joined #openstack-qa12:54
*** yamamoto has quit IRC12:56
*** yamamoto has joined #openstack-qa12:57
*** cdent has quit IRC12:58
*** amotoki has quit IRC12:59
*** cdent has joined #openstack-qa12:59
*** yamamoto has quit IRC13:02
*** ddieterly has quit IRC13:07
*** ekhugen has quit IRC13:08
*** maurosr has quit IRC13:08
*** kinrui is now known as fungi13:09
*** yamamoto has joined #openstack-qa13:13
*** asselin__ has joined #openstack-qa13:15
*** drptbl_ has quit IRC13:15
*** amotoki has joined #openstack-qa13:19
*** dtantsur is now known as dtantsur|brb13:19
*** drptbl has joined #openstack-qa13:19
*** nwkarsten has joined #openstack-qa13:20
*** pilgrimstack has quit IRC13:22
*** pilgrimstack1 has joined #openstack-qa13:22
*** tosky_ is now known as tosky13:23
*** dkranz_ has joined #openstack-qa13:24
*** rfolco has quit IRC13:26
*** rfolco has joined #openstack-qa13:26
openstackgerritAndrew Kerr proposed openstack/tempest: Add tests for Cinder user messages v3 API  https://review.openstack.org/30063913:27
*** mriedem has joined #openstack-qa13:28
*** phschwartz_ is now known as pschwartz13:28
*** nwkarsten has quit IRC13:31
*** cdent has quit IRC13:31
*** nwkarsten has joined #openstack-qa13:32
*** amotoki has quit IRC13:34
*** nwkarsten has quit IRC13:36
*** amotoki has joined #openstack-qa13:40
*** fnaval_ has joined #openstack-qa13:41
sc68calsdague: http://lists.openstack.org/pipermail/openstack-dev/2016-April/092415.html :)13:43
*** rbak has joined #openstack-qa13:43
*** pc_m has quit IRC13:44
*** fnaval has quit IRC13:45
*** jaypipes has joined #openstack-qa13:47
*** ChanServ sets mode: +o jaypipes13:47
*** nwkarsten has joined #openstack-qa13:48
*** pc_m has joined #openstack-qa13:49
*** amotoki has quit IRC13:49
*** jpena|lunch is now known as jpena13:49
*** ddieterly has joined #openstack-qa13:51
*** ddieterly is now known as ddieterly[away]13:51
*** ddieterly[away] is now known as ddieterly13:51
*** ddieterly is now known as ddieterly[away]13:52
*** ddieterly[away] is now known as ddieterly13:54
*** Alex_Stef has quit IRC13:57
*** amotoki has joined #openstack-qa13:59
*** bkeller` has quit IRC14:01
*** coreyob_ is now known as coreyob14:02
*** sigmavirus24_awa is now known as sigmavirus2414:03
*** jecarey has joined #openstack-qa14:04
*** jecarey has quit IRC14:04
*** jecarey has joined #openstack-qa14:04
*** sc68cal has quit IRC14:04
*** bkeller` has joined #openstack-qa14:04
*** ddieterly is now known as ddieterly[away]14:06
openstackgerritSergey Nikitin proposed openstack/tempest: Added tests for compute microversion 2.9  https://review.openstack.org/30722214:07
*** dstanek has joined #openstack-qa14:07
*** sc68cal has joined #openstack-qa14:09
jrollmtreinish: so turns out that "ssh commands are hanging" thing we talked about is actually something with ssh - the polling implementation in devstack is broken, the non-polling implementation apparently works fine14:13
mriedemmtreinish: is it just me, or is this compute api test for volume snapshots not doing anything with a snapshot of a server with a volume attached to it? https://github.com/openstack/tempest/blob/master/tempest/api/compute/volumes/test_volume_snapshots.py#L4214:13
mriedemlooks like purely a proxy for the cinder snapshots API via nova14:13
*** mfisher_ora has joined #openstack-qa14:14
mtreinishjroll: the polling implementation in devstack?14:14
mtreinishmriedem: yeah it looks like it tests nova's proxy api for volume snapshots14:16
mriedemyuck14:16
mriedemmtreinish: which is redundant with https://github.com/openstack/tempest/blob/master/tempest/api/volume/test_volumes_snapshots.py#L87 right?14:17
mriedemmtreinish: looks like we should make the compute api one a volume attached to a server and then snapshot the server,14:17
mriedemalthough we already have a scenario test for that, but the scenario test has the ssh and timestamp file wrinkles probably14:17
*** fnaval_ has quit IRC14:17
jrollmtreinish: in tempest-lib: https://github.com/openstack/tempest-lib/blob/master/tempest_lib/common/ssh.py#L127-L16014:18
jrollsorry14:18
mtreinishmriedem: yeah it looks like the same as taht test, just going through nova instead of direct to cinder14:19
mriedemtest_stamp_pattern would test creating a snapshot of a server with a volume attached, except it's skipped :)14:19
mtreinishmriedem: right14:19
mriedemreason i'm looking is i asked the person writing this to provide a tempest test for it https://review.openstack.org/#/c/295865/14:20
mriedemsnapshot a server with a volume attached and the server is paused / suspended when we snapshot14:20
mriedemi figured we already had a test for the volume attached snapshot when the server is active test14:20
mtreinishjroll: you mean: https://github.com/openstack/tempest/blob/master/tempest/lib/common/ssh.py#L127-L154 ? Or are you actually using the tempest-lib version?14:21
mtreinishjroll: there could be a bug there, but fwiw that poll() loop hasn't changed in quite some time14:22
mtreinishmriedem: yeah I think that's probably a good addition14:22
*** ddieterly[away] is now known as ddieterly14:23
*** ddieterly is now known as ddieterly[away]14:24
*** drptbl has quit IRC14:24
*** rlrossit has joined #openstack-qa14:25
*** ajmiller has joined #openstack-qa14:25
*** drptbl has joined #openstack-qa14:25
*** rfolco has quit IRC14:26
*** rfolco has joined #openstack-qa14:26
*** ober37_ has quit IRC14:26
*** ddieterly[away] is now known as ddieterly14:27
*** ober37 has joined #openstack-qa14:28
*** rfolco has quit IRC14:28
jrollmtreinish: I guess I'm not sure which off-hand, but they're the same14:29
jrollmtreinish: and we're trying to track it down, but yeah that's busted for us somehow, absolutely no clue why14:29
*** rfolco has joined #openstack-qa14:29
jrolljust letting you know what we found and curious if you have thuoghts14:30
*** lbragstad_ is now known as lbragstad14:31
mtreinishjroll: heh TBH I'm not sure if I've touched that code since: https://review.openstack.org/#/c/41497/14:32
jrolllol14:32
mtreinishjroll: but, if you have more specifics on where it's falling apart I can try to give a hand14:33
jrollmtreinish: this is most of what I have: https://bugs.launchpad.net/diskimage-builder/+bug/157149214:34
openstackLaunchpad bug 1571492 in Ironic "gate-tempest-dsvm-ironic-pxe_ssh-dib failing : test_baremetal_server_ops ssh timeout" [Critical,Confirmed]14:34
*** ddieterly is now known as ddieterly[away]14:35
*** fnaval has joined #openstack-qa14:35
*** fnaval has quit IRC14:37
*** pglass has joined #openstack-qa14:38
*** ddieterly[away] is now known as ddieterly14:39
*** Guest11043 is now known as redrobot14:40
*** cdent has joined #openstack-qa14:41
*** diana_clarke has left #openstack-qa14:42
*** pilgrimstack1 has quit IRC14:43
*** ekhugen has joined #openstack-qa14:43
*** maurosr has joined #openstack-qa14:44
*** fnaval has joined #openstack-qa14:44
*** erlon_ has joined #openstack-qa14:48
*** pilgrimstack has joined #openstack-qa14:48
*** ajmiller_ has joined #openstack-qa14:48
*** jlanoux has quit IRC14:48
*** Guest64003 is now known as melwitt14:51
*** ajmiller has quit IRC14:52
*** ajmiller has joined #openstack-qa14:53
*** jlwhite_ has joined #openstack-qa14:54
*** jlwhite_ has quit IRC14:55
*** jlwhite_ has joined #openstack-qa14:55
*** ajmiller_ has quit IRC14:56
*** sshnaidm has quit IRC14:58
*** rbak has quit IRC14:59
*** ddieterly is now known as ddieterly[away]15:00
*** breton has joined #openstack-qa15:01
bretonhi. Is this the right place for questions about tempest?15:02
jrollyes15:02
*** ddieterly[away] is now known as ddieterly15:06
*** dtantsur|brb is now known as dtantsur15:07
*** shausy has quit IRC15:08
*** zackf has joined #openstack-qa15:08
breton> Tempest will automatically discover any installed plugins when it is run.15:08
bretonwe have a tempest plugin in keystone. It's in keystone_tempest_plugin/ directory and is named "keystone_tests". How do i run not the entire set of tempest tests, but tests only from this plugin?15:08
toskybreton: it's all a matter of patterns to use with ostestr: http://docs.openstack.org/developer/tempest/overview.html#quickstart15:14
*** itzikb has quit IRC15:15
*** ober37 has quit IRC15:15
*** ober37 has joined #openstack-qa15:16
*** armax has joined #openstack-qa15:17
*** jlanoux has joined #openstack-qa15:18
*** ober37 has quit IRC15:20
*** abehl has quit IRC15:20
bretontosky: so to run only keystone_tests i need to use regex `tempest\.keystone_tests`?15:20
*** cody-somerville has joined #openstack-qa15:22
toskybreton: probably 'keystone_tempest_plugin.*'15:23
toskybreton: check the pattern by listing all the available tests with testr list-tests15:23
*** vhosakot has joined #openstack-qa15:25
*** mtanino has joined #openstack-qa15:25
*** armax has quit IRC15:26
artomA whole lot of jobs in the check pipeline are stuck in queued... Is this normal, am I just seeing trouble where there isn't any?15:28
*** Guest72772 is now known as jgriffith15:28
*** armax has joined #openstack-qa15:29
*** ddieterly is now known as ddieterly[away]15:29
bretontosky: understood, thank you15:29
*** rbak has joined #openstack-qa15:33
*** armax has quit IRC15:33
*** woodster_ has joined #openstack-qa15:34
*** e0ne has quit IRC15:34
*** mrodden has quit IRC15:43
*** armax has joined #openstack-qa15:44
*** mrodden has joined #openstack-qa15:44
*** ddieterly[away] is now known as ddieterly15:46
*** ddieterly is now known as ddieterly[away]15:47
rlrossitmtreinish: I've found something curious with these tempest plugins... if you import tempest.clients within the plugin class's module, stevedore suddenly becomes unable to find the plugin class15:49
rlrossitbut it can only be in the class's module. If I import that anywhere else (after the plugin has been loaded) everything works fine15:50
*** nwkarsten has quit IRC15:50
*** nwkarsten has joined #openstack-qa15:50
mtreinishrlrossit: hmm that's weird, maybe there's an import error or something which is blocking stevedore from loading it?15:52
*** sshnaidm has joined #openstack-qa15:53
rlrossitmtreinish: so this started out with me importing my own modules in there, but after commenting out more and more, I've gotten it down to an import of tempest.clients. If I comment out that import, it loads just fine15:54
*** apoorvad has joined #openstack-qa15:54
rlrossitI don't really want to go through the 50000 imports in tempest.clients either to see if one of those is mucking it up :(15:55
mtreinishrlrossit: I would expect it to log something in tempest.log if there was an error blocking it from being loaded though15:55
mtreinishheh, yeah there are a lot of imports in clients.py15:56
mtreinishit could also be anything executed at import time in the clients.py module15:56
rlrossitmtreinish: the only thing it's currently complaining about is "ERROR tempest.test_discover.plugins [-] Could not load 'plugin_name': 'module' object has no attribute 'CinnamonRolePlugin'"15:57
rlrossitand then it barfs saying module has no attribute CinnamonRolePlugin15:58
rlrossitI'm using a binary search to figure out where the import problem is happening...15:58
rlrossitmeta-algorithms15:58
*** nwkarste_ has joined #openstack-qa15:58
*** ddieterly[away] is now known as ddieterly15:59
*** nwkarsten has quit IRC15:59
*** ddieterly has quit IRC15:59
*** tqtran has joined #openstack-qa16:00
*** ddieterly has joined #openstack-qa16:00
*** ddieterly has quit IRC16:00
mtreinishrlrossit: we probably should make that logging a bit better if we can: https://github.com/openstack/tempest/blob/master/tempest/test_discover/plugins.py#L80-L8216:00
rlrossitmtreinish: the problem wouldn't be in accessing CONF would it?16:00
mtreinishrlrossit: it could be, the CONF stuff should be lazy loaded, so it'll try to read the config file when you do a getattr on CONF16:01
rlrossitmtreinish: oh goodness I think that's what it is16:01
mtreinishthe ordering around conf gets really weird sometimes16:01
rlrossitmtreinish: removing https://github.com/openstack/tempest/blob/master/tempest/clients.py#L188-L201 makes it work16:02
rlrossitso there is some goofy bootstrapping going on with the CONF stuff16:02
rlrossitbecause in order for CONF to work, it needs to find the CONF stuff of all plugins too right?16:02
mtreinishrlrossit: right, when it inits the CONF object it uses the extension manager to get the function calls to add all the plugins options16:03
mtreinishthat is probably it16:04
*** salv-orlando has joined #openstack-qa16:04
rlrossitmaybe I'll just move the import to the plugin's load_tests then?16:04
mtreinishso you were trying to load all the plugins while loading the plugins :)16:04
mtreinishwhy do you need clients.py in the plugin class?16:04
*** Guest30394 has quit IRC16:05
rlrossitmtreinish: so it's at the bottom of a bunch of other imports16:05
rlrossitmtreinish: you ready for my evil master plan with my plugin?16:05
*** ober37 has joined #openstack-qa16:05
* mtreinish is afraid16:06
rlrossityou should be16:06
*** jlwhite_ has quit IRC16:06
*** jlwhite_ has joined #openstack-qa16:06
rlrossitI'm basically trying to dynamically load all tempest test classes, and then wrap each of those in a dynamically created class to run with a different set of users16:06
*** salv-orl_ has quit IRC16:06
rlrossitwith different expected results for each set of users16:07
*** nwkarste_ has quit IRC16:07
*** jlwhite__ has joined #openstack-qa16:07
* rlrossit laughs maniacally16:07
*** ober37 has quit IRC16:07
*** jlwhite__ has left #openstack-qa16:07
*** ober37 has joined #openstack-qa16:07
*** Swami has joined #openstack-qa16:08
rlrossitmtreinish: and because tempest's plugins don't account for my extremely crazy one-off plugin, I'm trying to do a lot of that in load_tests :)16:08
*** nwkarsten has joined #openstack-qa16:08
mtreinishrlrossit: hah, yeah that's pretty out there :)16:08
*** aysyd has quit IRC16:09
rlrossitmtreinish: the goal is when I'm no longer embarrassed about the state of this thing, I'll throw it out in the openstack tent16:09
mtreinishcool, the best part of that is tempest doesn't actually call load_tests, so you wrote a plugin to have the test runner execute a bunch of arbitrary code16:09
rlrossitit's actually really really good at testing policy configurations16:09
mtreinishrlrossit: yeah, I bet a bunch of people with weird policy files are gonna love it16:10
*** nadya has quit IRC16:10
mtreinishmy normal answer for those peopl is: stop breaking interop we won't fix that in tempest...16:10
rlrossitmtreinish: does that make me an enabler?16:11
mtreinishheh, I guess so16:11
*** sridharg has quit IRC16:13
mtreinishrlrossit: http://lists.openstack.org/pipermail/openstack-dev/2016-February/086118.html and https://bugs.launchpad.net/tempest/+bug/1565584 are 2 examples from recent memory16:17
openstackLaunchpad bug 1565584 in tempest "Default tests do not request credentials by role" [Undecided,In progress] - Assigned to Dale Smith (dalees)16:17
rlrossitmtreinish: oh man, these guys are gonna love me16:18
rlrossitI do exactly what is asked in that ML :P16:18
rlrossitor well part of it at least I guess16:18
*** Kiall has quit IRC16:18
mtreinishrlrossit: yeah I don't think your case will work for the ML configuration. They set servers to be scoped to the user not tenant16:20
rlrossitthat bug though... that's me fo sho16:20
*** nwkarsten has quit IRC16:21
*** pcaruana has quit IRC16:21
*** jlwhite_ has quit IRC16:22
*** jlwhite_ has joined #openstack-qa16:22
*** Kiall has joined #openstack-qa16:22
*** jlwhite_ has quit IRC16:22
*** jlwhite_ has joined #openstack-qa16:23
*** aysyd has joined #openstack-qa16:23
*** nwkarsten has joined #openstack-qa16:25
*** Kiall has quit IRC16:27
*** gszasz has quit IRC16:27
*** jlwhite__ has joined #openstack-qa16:28
*** tosky has quit IRC16:29
*** jlwhite__ has quit IRC16:29
*** fnaval has quit IRC16:31
*** jlwhite has quit IRC16:35
*** tosky has joined #openstack-qa16:35
*** jlwhite has joined #openstack-qa16:36
*** fnaval has joined #openstack-qa16:36
*** Kiall has joined #openstack-qa16:38
*** tqtran has quit IRC16:40
*** jlwhite has quit IRC16:41
*** rcernin has quit IRC16:43
*** Kiall has quit IRC16:43
*** Kiall has joined #openstack-qa16:44
openstackgerritJordan Pittier proposed openstack/tempest: Remove iso8601 and anyjson from requirements.txt  https://review.openstack.org/30737016:47
dalees`rlrossit, i'm not caught up on what you're working on, but if you'd like someone to help test/review in the context of https://bugs.launchpad.net/tempest/+bug/1565584 i'd be happy to help :)16:48
openstackLaunchpad bug 1565584 in tempest "Default tests do not request credentials by role" [Undecided,In progress] - Assigned to Dale Smith (dalees)16:48
*** jlwhite_ has quit IRC16:52
*** vgridnev_ has joined #openstack-qa16:52
*** pc_m has quit IRC16:53
*** davideagnello has joined #openstack-qa16:54
*** pc_m has joined #openstack-qa16:55
*** jlwhite has joined #openstack-qa16:58
*** jlwhite has quit IRC17:00
*** jlwhite has joined #openstack-qa17:03
*** jlwhite has quit IRC17:03
*** rossella_s has quit IRC17:03
*** rossella_s has joined #openstack-qa17:04
*** jpena is now known as jpena|off17:07
*** tqtran has joined #openstack-qa17:10
*** Alex_Stef has joined #openstack-qa17:10
*** harlowja has joined #openstack-qa17:11
*** nwkarsten has quit IRC17:14
*** nwkarsten has joined #openstack-qa17:22
*** sigmavirus24 is now known as sigmavirus24_awa17:26
*** jesusaur has joined #openstack-qa17:26
*** sigmavirus24_awa is now known as sigmavirus2417:28
*** yamahata has joined #openstack-qa17:29
*** gszasz has joined #openstack-qa17:31
openstackgerritAlex Meade proposed openstack-dev/devstack: Add volumev3 endpoint for Cinder  https://review.openstack.org/30058517:32
*** kbaikov_ has joined #openstack-qa17:32
*** kbaikov_ has quit IRC17:35
*** gfidente has quit IRC17:35
*** tosky has quit IRC17:36
*** cdent has quit IRC17:38
*** ober37 has left #openstack-qa17:40
*** ober37 has joined #openstack-qa17:41
*** stevemar has quit IRC17:41
ober37Tempest question, what's the stance on negative test case contributions?   Are these on hold until the topic gets settled at Austin?17:41
*** kbaikov_ has joined #openstack-qa17:43
*** stevemar has joined #openstack-qa17:44
*** ifarkas has quit IRC17:44
*** kbaikov_ has quit IRC17:47
*** Serlex has left #openstack-qa17:47
*** jlanoux has quit IRC17:48
*** dtantsur is now known as dtantsur|afk17:49
*** lucasagomes is now known as lucas-dinner17:51
*** kbaikov_ has joined #openstack-qa17:52
*** kbaikov_ has quit IRC17:53
openstackgerritJordan Pittier proposed openstack/tempest: Remove definitions of unused exception classes.  https://review.openstack.org/30739217:54
*** pvaneck has joined #openstack-qa17:56
*** nwkarsten has quit IRC17:59
*** nwkarsten has joined #openstack-qa18:00
*** nwkarsten has quit IRC18:11
*** yamahata has quit IRC18:11
*** ajmiller_ has joined #openstack-qa18:12
*** nwkarsten has joined #openstack-qa18:12
*** salv-orlando has quit IRC18:13
*** salv-orlando has joined #openstack-qa18:13
*** ajmiller has quit IRC18:15
*** salv-orlando has quit IRC18:18
mtreinishober37: I don't think there is an official block on them until summit. But, I think you'll probably have a hard time getting critical review mass until the session18:22
*** nwkarsten has quit IRC18:23
*** nwkarsten has joined #openstack-qa18:23
*** salv-orlando has joined #openstack-qa18:24
*** sambetts is now known as sambetts|afk18:32
*** mpaolino has quit IRC18:35
*** nwkarsten has quit IRC18:35
*** nwkarsten has joined #openstack-qa18:36
openstackgerritMatthew Treinish proposed openstack/tempest: Add generic CONF skip decorators  https://review.openstack.org/30678018:37
oomichigmann: I saw https://review.openstack.org/#/c/306919 , but I cannot fine the merit like fixing actual bug or something.18:39
ober37mtreinish: Thanks!  Yeah that makes sense18:39
oomichigmann: there is a lot of this kind thing on each API18:40
mtreinishoomichi: heh, when I saw the email for that bug I just assumed it was the httplib2->urllib3 migration. But, thinking about it more that couldn't have been because that would've been blocked in the gate18:41
mtreinishoomichi: so I agree, I don't think there is a real bug there18:41
mtreinishoomichi: on https://review.openstack.org/305495 was there something you wanted me to change on that patch? Or is it good to go?18:43
*** nadya has joined #openstack-qa18:43
oomichimtreinish: ah, I am guessing you will separate start_date param from this method and create get_recent_*_runs method with start_dete param. willn't you?18:45
mtreinishoomichi: yes, I was planning to push a follow up patch to add the additional parameter to the other methods soon18:46
oomichimtreinish: or will you keep the param on the existing one and create another method with the param?18:46
mtreinishI am keeping the param on the method added in that patch, because it's needed there18:46
oomichimtreinish: ah, I see your point. you will add the param to *existing* methods18:47
oomichimtreinish: I misunderstood you would add new methods18:47
oomichifor the param.18:47
mtreinishoomichi: oh, no that would be kinda crazy, it's a kwarg so I can just add it to the end of the existing methods in the future18:48
*** ajmiller_ is now known as ajmiller18:48
mtreinishno need to add a separate method for 1 additional option arg18:48
oomichimtreinish: ok, that is extensible because of start_date=None18:48
oomichimtreinish: yeah, I need coffee now. +A18:49
*** nwkarsten has quit IRC18:52
oomichimtreinish: btw how about https://review.openstack.org/#/c/306985/ ?   After that, maybe masayukig will make the exsting tempest-cli-improvements as implemented18:52
oomichimtreinish: then we can see a single clear target: tempest.run18:53
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add optional start_date parameter to get_recent*() methods  https://review.openstack.org/30741618:53
mtreinishoomichi: ^^^ that's what I meant for the  followup patch (that's a wip)18:54
oomichimtreinish: so fast, cool18:54
mtreinishoomichi: well I still need to update the docs, and add tests and release notes18:55
mtreinishbut the functional change is super straightforward18:55
oomichimtreinish: -1 on https://review.openstack.org/30741618:56
mtreinishoomichi: I did say it was a wip...18:56
oomichimtreinish: hehe, I know. but you didn't put it on gerrit ;-)18:57
mtreinishoomichi: you were too fast18:57
*** nwkarsten has joined #openstack-qa19:00
openstackgerritMerged openstack/tempest: "is_resource_deleted" fails to verify delete  https://review.openstack.org/29607219:02
*** bkeller^ has joined #openstack-qa19:04
*** bkeller^ has quit IRC19:04
*** nwkarsten has quit IRC19:04
*** ober37 has quit IRC19:07
*** ober37 has joined #openstack-qa19:08
*** ober37 has quit IRC19:08
oomichimtreinish: oops, you -1d on masayukig's spec patch. isn,t that confused if moving the spec which includes tempest.run also into implemented?19:08
*** ober37 has joined #openstack-qa19:08
mtreinishoomichi: no I think masayukig should make the changes and move the file in 1 patch19:09
*** bkeller^ has joined #openstack-qa19:09
oomichimtreinish: ah, I agree :-)19:09
openstackgerritMerged openstack-infra/subunit2sql: Add api to get recent failed runs by run_metadata  https://review.openstack.org/30549519:16
*** bkeller`- has joined #openstack-qa19:16
*** jlwhite has joined #openstack-qa19:19
*** rfolco has quit IRC19:20
*** bkeller`- has quit IRC19:23
*** bkeller^ has quit IRC19:23
*** bkeller^ has joined #openstack-qa19:23
*** bkeller` has quit IRC19:24
*** bkeller^ is now known as bkeller`19:24
*** cdent has joined #openstack-qa19:25
*** ober37` has joined #openstack-qa19:25
*** rfolco has joined #openstack-qa19:25
openstackgerritMerged openstack-infra/subunit2sql: Fix the typo of docstring  https://review.openstack.org/30594119:25
*** cdent has quit IRC19:26
*** mminesh has joined #openstack-qa19:27
*** jlwhite has quit IRC19:28
*** nwkarsten has joined #openstack-qa19:29
*** weshay has quit IRC19:29
*** jlwhite has joined #openstack-qa19:37
*** yamamoto has quit IRC19:39
*** salv-orlando has quit IRC19:47
*** ddieterly has joined #openstack-qa19:52
*** e0ne has joined #openstack-qa19:52
*** yamamoto has joined #openstack-qa19:54
*** ddieterly is now known as ddieterly[away]20:00
*** alex_xu has quit IRC20:04
*** ddieterly[away] is now known as ddieterly20:05
*** alex_xu has joined #openstack-qa20:06
*** e0ne has quit IRC20:08
*** nwkarsten has quit IRC20:09
*** nwkarsten has joined #openstack-qa20:10
*** beekneemech is now known as bneme20:12
*** bneme is now known as bnemec20:12
*** ddieterly is now known as ddieterly[away]20:23
*** ddieterly[away] is now known as ddieterly20:26
*** ober37 has quit IRC20:27
*** ober37` is now known as ober3720:27
*** nwkarste_ has joined #openstack-qa20:28
*** nwkarsten has quit IRC20:28
openstackgerritMerged openstack-dev/devstack: worlddump: request Guru Mediation reports for neutron agents  https://review.openstack.org/27903520:28
*** nwkarste_ has quit IRC20:29
*** nwkarsten has joined #openstack-qa20:30
*** ober37_ has joined #openstack-qa20:32
*** yamahata has joined #openstack-qa20:33
*** salv-orlando has joined #openstack-qa20:33
*** nwkarsten has quit IRC20:34
*** nadya has quit IRC20:36
*** e0ne has joined #openstack-qa20:38
*** dkranz_ has quit IRC20:40
*** lucas-dinner has quit IRC20:44
*** e0ne has quit IRC20:50
*** lucasagomes has joined #openstack-qa20:53
*** cdent has joined #openstack-qa20:56
*** ober37_ has quit IRC20:56
*** hichihara has joined #openstack-qa20:57
*** fawadkhaliq has joined #openstack-qa20:57
*** mfisher_ora has quit IRC20:57
*** weshay has joined #openstack-qa20:58
*** rossella_s has quit IRC21:03
*** lucasagomes has quit IRC21:03
*** rossella_s has joined #openstack-qa21:04
*** raildo is now known as raildo-afk21:05
*** shardy has quit IRC21:11
*** ddieterly is now known as ddieterly[away]21:12
*** lucasagomes has joined #openstack-qa21:12
*** aysyd has quit IRC21:14
*** gszasz has quit IRC21:16
*** mylu has joined #openstack-qa21:22
*** vhosakot has quit IRC21:22
*** rlrossit has quit IRC21:22
*** mylu has quit IRC21:24
*** mylu has joined #openstack-qa21:25
*** jecarey has quit IRC21:30
*** nwkarsten has joined #openstack-qa21:34
*** ddieterly[away] is now known as ddieterly21:38
*** nwkarsten has quit IRC21:41
*** dimtruck is now known as zz_dimtruck21:42
*** pcrews_ has joined #openstack-qa21:47
*** pcrews has quit IRC21:50
*** nwkarsten has joined #openstack-qa21:53
*** ddieterly is now known as ddieterly[away]21:54
*** mylu has quit IRC21:54
*** pglass has quit IRC21:56
*** fawadkhaliq has quit IRC22:00
openstackgerritMerged openstack-infra/subunit2sql: Rework migration 1679b5bc102c to not use the db api  https://review.openstack.org/30136922:05
*** jecarey has joined #openstack-qa22:07
*** jecarey has quit IRC22:12
*** cdent has quit IRC22:13
*** hichihara has quit IRC22:14
*** ddieterly[away] is now known as ddieterly22:16
*** dave-mccowan has quit IRC22:19
*** ddieterly is now known as ddieterly[away]22:20
*** sigmavirus24 is now known as sigmavirus24_awa22:20
*** nwkarsten has quit IRC22:24
*** nwkarsten has joined #openstack-qa22:25
*** dalees` has quit IRC22:25
*** ddieterly[away] is now known as ddieterly22:26
*** hichihara has joined #openstack-qa22:26
*** yamahata has quit IRC22:27
*** markvoelker has quit IRC22:28
*** vgridnev_ has quit IRC22:30
*** ddieterly is now known as ddieterly[away]22:31
*** yamahata has joined #openstack-qa22:34
*** nwkarsten has quit IRC22:34
*** nwkarsten has joined #openstack-qa22:35
*** ddieterly[away] is now known as ddieterly22:35
*** dalees` has joined #openstack-qa22:38
*** nwkarsten has quit IRC22:39
*** ddieterly is now known as ddieterly[away]22:42
*** ddieterly[away] is now known as ddieterly22:44
*** eliqiao_ has joined #openstack-qa22:44
*** eliqiao has quit IRC22:45
*** sdague has quit IRC22:45
*** ddieterly is now known as ddieterly[away]22:51
*** sdague has joined #openstack-qa22:57
*** ddieterly[away] is now known as ddieterly23:04
*** ddieterly is now known as ddieterly[away]23:04
*** ddieterly[away] has quit IRC23:04
*** vhosakot has joined #openstack-qa23:06
openstackgerritSteve Heyman proposed openstack-dev/grenade: Update README with user parameter for setup-grenade  https://review.openstack.org/30751623:10
openstackgerritSteve Heyman proposed openstack-dev/grenade: Update README with user parameter for setup-grenade  https://review.openstack.org/30751623:12
*** itzikb has joined #openstack-qa23:14
*** raildo-afk is now known as raildo23:18
openstackgerritHongbin Lu proposed openstack-dev/devstack: Support decompressing bz2 image  https://review.openstack.org/30684223:19
*** zz_dimtruck is now known as dimtruck23:22
mriedemmtreinish: the client setup structure for admin tests in tempest totally confuses me23:25
mriedemlike in here it creates a variable for the migrations admin client https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_migrations.py#L3023:26
mriedembut elsewhere in that test class it just uses self.servers_client which must be inherited from the parent class23:26
mriedemwhich would be https://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L378 and that sets,23:27
mriedemcls.availability_zone_admin_client = (                                            cls.os_adm.availability_zone_client)23:27
mriedemwhich is confusing given the az admin client is set in the parent class of that: https://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L8423:27
mriedemso i never know what i'm actually supposed to do in setup_clients23:27
*** markvoelker has joined #openstack-qa23:28
mtreinishmriedem: for the most part setup clients is just used as aliases to save space23:28
mtreinishI normally prefer to just use self.os_admin.client directly23:28
mtreinishit's much clearer23:28
mtreinishbut sometimes there is a 79 character concern which is probably why there is so much aliasing23:29
mtreinishmriedem: so I'm pretty sure L84 in the base compute class is the az client with regular user creds23:30
mtreinish(it must be used for a negative test somewhere)23:30
mtreinishand the admin one in the L378 class is the one to actually use (since it has admin creds)23:31
mriedemok so if i want to create a flavor in an admin test, i need to do https://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L8423:31
mriedemoops23:31
mriedemcls.flavors_admin_client = cls.os_adm.flavors_client?23:31
mtreinishmriedem: yeah, that or just call self.os_adm.flavors_client.create_flavor() directly23:32
mtreinishthe setup_client() stuff is not required23:32
gmannoomichi: but that is bug in method implementation not related to API behavior23:33
*** markvoelker has quit IRC23:33
mriedemmtreinish: ok, i think i have this then23:33
gmannoomichi: that method is expected to compare the two status code which should be int and return the result23:33
gmannoomichi: but if read_code is not int then it would not compare anything and result to pass23:34
oomichigmann: I guess the patch adds just a validation, right?23:34
gmannoomichi: i mean expected_code=202, read_code='201' is true with current implementation23:34
mtreinishgmann: right but how is getting a string? it gets called directly from inside the rest client right?23:34
mtreinishall the clients should use the same path23:34
mtreinishis it never passing an int?23:34
*** ddieterly has joined #openstack-qa23:35
gmannoomichi: no this is called from every service clients23:35
oomichigmann: how about converting string to int like nova does?23:35
*** ddieterly is now known as ddieterly[away]23:35
gmannmtreinish: yea all usage are correct so no issue23:35
*** salv-orlando has quit IRC23:35
gmannmtreinish: oomichi if we pass resp['status'] then it was string23:35
gmannactuall bug was with wrong usage of this method23:36
gmannhttps://review.openstack.org/#/c/304380/13/designate_tempest_plugin/services/dns/v2/json/zone_imports_client.py23:36
gmannwrong usage of this method(passing string) give false pass23:36
*** tochi has joined #openstack-qa23:36
*** nwkarsten has joined #openstack-qa23:36
oomichigmann: ah, nice to know that.23:36
openstackgerritMatt Riedemann proposed openstack/tempest: Add a test for reverting a resize with a deleted flavor  https://review.openstack.org/30751823:36
mriedemdansmith: ^23:36
oomichigmann: I wanted to see this kind of actual report23:36
dansmith`mricool23:37
dansmithwow23:37
dansmithmriedem: cool23:37
*** ddieterly[away] is now known as ddieterly23:37
gmannmtreinish: oomichi currently all tempest service clients use this correctly no issue there23:37
gmannbut in some plugin it was tried wrongly and results were wrong23:37
gmannoomichi: i added that in bug report if that is enough23:37
oomichigmann: but designate tried to use it wrongly before. If your patch was merged, they could know that.23:38
mtreinishgmann: I'm still a bit confused was the dict access always available? or is this fallout from the switch to urllib3?23:38
gmannoomichi: yea23:38
mtreinishgmann: because that's the crux of the issue resp['status'] vs resp.status right?23:38
gmannmtreinish: yea23:38
openstackgerritMatt Riedemann proposed openstack/tempest: Add a test for reverting a resize with a deleted flavor  https://review.openstack.org/30751823:39
oomichigmann: nice to write it on the commit message for me.23:39
gmannmtreinish: i feel that was always, mean  resp['status'] was available always23:39
*** nwkarsten has quit IRC23:41
mtreinishgmann: yeah it looks like it was: https://github.com/httplib2/httplib2/blob/master/python2/httplib2/__init__.py#L1655-L169023:42
mtreinishtbh, I don't understand why they set it as a string from the dict access but leave it as an int from the attr23:43
mtreinishbut the urrlib3 patch was correct in maintaining that weirdness23:43
gmannmtreinish: yea thats not right to get with string at least for this method usage23:43
mtreinishgmann: ok I understand your bugfix more now23:44
*** nwkarsten has joined #openstack-qa23:44
gmannmtreinish: actually we have that in doc string but nobody read those seems :)23:44
mtreinishgmann: I know I don't :)23:45
gmann:)23:45
flwangmtreinish: ping need your help on http://logs.openstack.org/79/302479/11/check/gate-tempest-dsvm-zaqar/6c92f39/23:46
flwangnot sure if you remember our last discussion23:46
gmannoomichi: sure, i will add more in commit msg23:46
flwangi changed the code a bit and now i got a different error23:46
oomichigmann: it is nice to specify change-id on the commit message for future us23:46
* gmann need to learn clear and large commit msg from mtreinish. 23:47
gmannoomichi: yea i always a lazy person in commit msg term :)23:47
gmannoomichi: i will add. Thanks23:47
mtreinishflwang: it looks like it's not completely configured still: http://logs.openstack.org/79/302479/11/check/gate-tempest-dsvm-zaqar/6c92f39/logs/tempest_conf.txt.gz23:47
mtreinishflwang: there isn't an auth url set23:47
mtreinishgmann: heh, something like: https://review.openstack.org/#/c/306779/23:48
mtreinishwhich is a lot of words for a file mv23:48
flwangmtreinish: ah, i see23:49
flwangmtreinish: cheers, man23:49
gmannmtreinish: ah, yea, did not check that yet :) but there are lot of places you have commit msg larger than code changes :)23:49
*** nwkarsten has quit IRC23:49
gmannand well explanatory23:50
mtreinishgmann: normally when writing a commit msg, I to try and answer 2 questions: why someone should care about this patch and why I decided to write it (which are often related if not the same)23:50
mtreinishflwang: are you using devstack to configure tempest?23:51
*** mriedem is now known as mriedem_away23:51
mtreinishflwang: oh right I remember now, this when you're calling tempest in the gate hook before devstack runs. Why are you doing that? (I don't remember)23:52
gmannmtreinish: humm, that's cool, I ll also try that. but issue is laziness for doc or commit msg :) but i can understand that can be much helpful many times23:53
mtreinishgmann: heh, everyone gets lazy with commit msgs sometimes too23:53
*** ddieterly has quit IRC23:53
mtreinishbut I found a long time ago that having a good commit msg helps a lot with getting good reviews23:53
gmannmtreinish: yea totally agree.23:55
*** erlon_ has quit IRC23:55
flwangmtreinish: so should I put those new changes lines under line 36 https://review.openstack.org/#/c/302479/11/devstack/gate/gate_hook.sh ?23:57

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