*** salv-orlando has joined #openstack-qa | 00:01 | |
jamielennox | mtreinish: the response from a create_user is a credential object? | 00:01 |
---|---|---|
openstackgerrit | gordon chung proposed openstack-dev/devstack: switch to use ceilometermiddleware https://review.openstack.org/155473 | 00:02 |
jamielennox | mtreinish: it's a response body - and i couldn't see an easy way to convert that into a credentials object | 00:02 |
*** Longgeek has joined #openstack-qa | 00:05 | |
*** pkoniszewski has joined #openstack-qa | 00:06 | |
*** rbak has quit IRC | 00:08 | |
*** krtaylor has joined #openstack-qa | 00:10 | |
openstackgerrit | Merged openstack/tempest: Remove auth_version config from get_credentials https://review.openstack.org/151635 | 00:11 |
openstackgerrit | Merged openstack/tempest: Split resource_setup for data_processing tests https://review.openstack.org/152412 | 00:12 |
openstackgerrit | Merged openstack/tempest: Volumes: strict type and value check of the 'bootable' flag https://review.openstack.org/152148 | 00:12 |
openstackgerrit | Merged openstack/tempest: Remove novaclient CLI tests https://review.openstack.org/153716 | 00:12 |
*** pkoniszewski has quit IRC | 00:13 | |
openstackrecheck | Console logs not available after 13:22s for check-tempest-dsvm-postgres-full 155089,1,cd8b37d | 00:16 |
*** mlavalle has quit IRC | 00:19 | |
*** mtanino has quit IRC | 00:26 | |
mtreinish | jamielennox: oh, sorry yeah you need to create a credential object out of the response body | 00:28 |
jamielennox | mtreinish: do you have an example of how to do that | 00:28 |
*** abhirc has quit IRC | 00:28 | |
jamielennox | mtreinish: from what i can see any attempt to create a credential object is going to start loading things from config files | 00:28 |
mtreinish | jamielennox: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/common/accounts.py#n125 | 00:29 |
jamielennox | thta's a config file cred though | 00:30 |
mtreinish | that is an example where it reads in user, password, tenant from a yaml file and creates a cred object from that | 00:30 |
jamielennox | mtreinish: so my need is https://github.com/openstack/tempest/blob/master/tempest/api/identity/admin/v3/test_users.py#L68 | 00:30 |
jamielennox | which creates a user then as the admin user changes that user's password | 00:30 |
jamielennox | in v3 policy we removed the admin right to do that, so the operation has to be performed as the user that was just created | 00:31 |
jamielennox | i have user['id'] and original_password, and i can see you get a token futher down, but i want to create a new IdentityClient with that user credentials | 00:32 |
*** Longgeek has quit IRC | 00:34 | |
*** sabeen1 has quit IRC | 00:34 | |
*** david-lyle is now known as david-lyle_afk | 00:37 | |
mtreinish | jamielennox: super rough: http://paste.openstack.org/show/172491/ | 00:38 |
mtreinish | did you try something like that | 00:38 |
jamielennox | mtreinish: i got to the point of tracing what happens with the get_credentials function and the way i read it it would read the tenant_name from the config file if i didn't provide one | 00:41 |
*** abhirc has joined #openstack-qa | 00:41 | |
openstackrecheck | Console logs not available after 13:26s for check-tempest-dsvm-neutron-full 145317,10,2532aa6 | 00:42 |
*** abhirc has quit IRC | 00:42 | |
mtreinish | jamielennox: you don't have the tenant name? | 00:44 |
jamielennox | i don't think so | 00:45 |
*** yamamoto has joined #openstack-qa | 00:45 | |
jamielennox | looking at that test it's never assigned to a project | 00:45 |
mtreinish | jamielennox: you can do that? | 00:45 |
jamielennox | IMO, you should be able to change password without a project scope - whether thats true or not i've never checked | 00:45 |
jamielennox | i think it'll fail without a tenant | 00:47 |
*** gfidente has quit IRC | 00:49 | |
mtreinish | jamielennox: I'd give it a try and we can go from there | 00:52 |
jamielennox | mtreinish: will do | 00:53 |
mtreinish | jamielennox: andreaf would be a good person to ask when he's around (he wrote most of the auth layer) | 00:53 |
openstackrecheck | Console logs not available after 13:27s for check-tempest-dsvm-full 149930,7,582e2d2 | 00:56 |
openstackgerrit | YAMAMOTO Takashi proposed openstack-dev/devstack: ofagent: Vendor code split https://review.openstack.org/149936 | 01:00 |
*** dmorita has quit IRC | 01:00 | |
openstackgerrit | Steve McLellan proposed openstack-dev/devstack: Glance - configure keystone_authtoken separately https://review.openstack.org/155550 | 01:01 |
*** dmorita has joined #openstack-qa | 01:03 | |
*** dmorita has quit IRC | 01:09 | |
*** bknudson has joined #openstack-qa | 01:13 | |
*** achanda has quit IRC | 01:19 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/tempest: Updated from global requirements https://review.openstack.org/154187 | 01:26 |
*** dims_ has joined #openstack-qa | 01:32 | |
*** dims_ has quit IRC | 01:32 | |
*** dims_ has joined #openstack-qa | 01:32 | |
*** dims__ has quit IRC | 01:33 | |
*** tsekiyam_ has joined #openstack-qa | 01:34 | |
*** dims__ has joined #openstack-qa | 01:36 | |
*** tsekiyama has quit IRC | 01:37 | |
*** tsekiyam_ has quit IRC | 01:38 | |
*** dims_ has quit IRC | 01:39 | |
*** ChuckC has quit IRC | 01:45 | |
*** salv-orl_ has joined #openstack-qa | 01:46 | |
*** melwitt has quit IRC | 01:47 | |
*** salv-orlando has quit IRC | 01:48 | |
*** shashankhegde has quit IRC | 01:50 | |
*** ChuckC has joined #openstack-qa | 01:52 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/tempest: Updated from global requirements https://review.openstack.org/154187 | 02:01 |
*** jamielennox is now known as jamielennox|away | 02:01 | |
openstackrecheck | Console logs not available after 13:23s for check-grenade-dsvm 155552,1,91a4177 | 02:03 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/148038 failed gate-cinder-python27 in the gate queue with an unrecognized error | 02:04 |
mriedem | jogo: is that the big boy? nice job | 02:15 |
*** Yogi1 has joined #openstack-qa | 02:15 | |
*** Yogi11 has joined #openstack-qa | 02:16 | |
*** mriedem is now known as mriedem_zz | 02:16 | |
openstackgerrit | Joe Gordon proposed openstack-infra/elastic-recheck: Remove queries for resolved bugs https://review.openstack.org/155604 | 02:16 |
jogo | mriedem_zz: I snuck in a good commit message | 02:16 |
*** salv-orl_ has quit IRC | 02:17 | |
*** Longgeek has joined #openstack-qa | 02:19 | |
*** Yogi1 has quit IRC | 02:19 | |
openstackgerrit | Merged openstack/tempest: Drop the legacy and un-used _interface https://review.openstack.org/153621 | 02:22 |
openstackrecheck | Console logs not available after 13:28s for check-tempest-dsvm-neutron-full 155526,2,7a578e0 | 02:23 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/151802 failed gate-cinder-python27 in the gate queue with an unrecognized error | 02:23 |
*** gamado has joined #openstack-qa | 02:26 | |
*** patrickeast has quit IRC | 02:29 | |
*** Longgeek has quit IRC | 02:30 | |
clarkb | jogo: the pip things should be mostly corrected in the morning | 02:31 |
*** gamado has quit IRC | 02:31 | |
clarkb | jogo: I have devstack-trusty updated across the board for now | 02:32 |
*** mestery has joined #openstack-qa | 02:35 | |
jogo | clarkb: awesome, thanks! | 02:38 |
*** woodster_ has quit IRC | 02:40 | |
openstackgerrit | Yuiko Takada proposed openstack/tempest: Add os-baremetal-nodes tests https://review.openstack.org/149125 | 02:52 |
openstackrecheck | Console logs not available after 13:21s for gate-tempest-dsvm-neutron-full 154472,1,cd614a6 | 02:54 |
*** Yogi11 has quit IRC | 02:56 | |
*** yamahata has quit IRC | 03:04 | |
*** amotoki has joined #openstack-qa | 03:04 | |
*** shashankhegde has joined #openstack-qa | 03:06 | |
*** dims__ has quit IRC | 03:07 | |
*** salv-orlando has joined #openstack-qa | 03:18 | |
*** harlowja_ is now known as harlowja_away | 03:27 | |
*** sdague has quit IRC | 03:31 | |
*** sdague has joined #openstack-qa | 03:31 | |
openstackgerrit | Armando Migliaccio proposed openstack/tempest: Fix 'Number of new ports: 2' failure mode on test_hotplug_nic https://review.openstack.org/155626 | 03:39 |
*** ChuckC_ has joined #openstack-qa | 03:40 | |
*** pal has joined #openstack-qa | 03:43 | |
*** ChuckC has quit IRC | 03:44 | |
*** mestery has quit IRC | 03:55 | |
*** woodster_ has joined #openstack-qa | 03:58 | |
*** mestery has joined #openstack-qa | 03:58 | |
*** Longgeek has joined #openstack-qa | 04:01 | |
*** mestery has quit IRC | 04:03 | |
*** mestery has joined #openstack-qa | 04:04 | |
*** liusheng has joined #openstack-qa | 04:05 | |
*** dims__ has joined #openstack-qa | 04:07 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove unused CONF from limits/identity_client https://review.openstack.org/155632 | 04:09 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Rename data_processing client https://review.openstack.org/155633 | 04:09 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove CONF values from data_processing client https://review.openstack.org/155634 | 04:09 |
*** dims__ has quit IRC | 04:12 | |
oomichi | cyeoh, gmann: can you take a look at https://review.openstack.org/#/c/149125/ again ? | 04:15 |
cyeoh | oomichi, sure | 04:15 |
oomichi | cyeoh, gmann: the test run on the gate "check-tempest-dsvm-ironic-pxe_ssh" | 04:15 |
oomichi | cyeoh: thanks! | 04:16 |
gmann | oomichi: ll check | 04:17 |
oomichi | gmann: thanks :) | 04:17 |
*** chandankumar has joined #openstack-qa | 04:17 | |
*** Madan has joined #openstack-qa | 04:18 | |
*** amotoki has quit IRC | 04:20 | |
openstackgerrit | Merged openstack/tempest: Updated from global requirements https://review.openstack.org/154187 | 04:21 |
*** liusheng has quit IRC | 04:26 | |
*** liusheng has joined #openstack-qa | 04:26 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove meaningless adminPass reference https://review.openstack.org/155637 | 04:26 |
*** zz_dimtruck is now known as dimtruck | 04:27 | |
gmann | oomichi: done, may be we can remove baremetal client variable from non admin base class | 04:30 |
openstackgerrit | Yuiko Takada proposed openstack/tempest: Add os-baremetal-nodes tests https://review.openstack.org/149125 | 04:34 |
*** yamahata has joined #openstack-qa | 04:35 | |
openstackgerrit | Dean Troyer proposed openstack-dev/devstack: Fix is_keystone_enabled() https://review.openstack.org/155638 | 04:35 |
*** pcrews has quit IRC | 04:45 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove CONF reference from volume client https://review.openstack.org/155642 | 04:55 |
*** shashankhegde has quit IRC | 05:00 | |
*** achanda has joined #openstack-qa | 05:02 | |
*** marrusl has quit IRC | 05:05 | |
*** amotoki has joined #openstack-qa | 05:09 | |
*** salv-orlando has quit IRC | 05:10 | |
*** StevenK has quit IRC | 05:50 | |
*** StevenK has joined #openstack-qa | 05:53 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove CONF reference from volume client https://review.openstack.org/155642 | 05:59 |
*** abehl has joined #openstack-qa | 06:02 | |
*** chandankumar has quit IRC | 06:03 | |
*** achanda has quit IRC | 06:03 | |
openstackgerrit | Dean Troyer proposed openstack-dev/devstack: Build a virtual environment for clients https://review.openstack.org/151513 | 06:03 |
*** nikil89_ has joined #openstack-qa | 06:08 | |
*** achanda has joined #openstack-qa | 06:11 | |
*** yamahata has quit IRC | 06:13 | |
*** yamahata has joined #openstack-qa | 06:14 | |
*** coolsvap_ is now known as coolsvap | 06:18 | |
*** salv-orlando has joined #openstack-qa | 06:18 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove CONF reference from volume client https://review.openstack.org/155642 | 06:26 |
*** abhirc has joined #openstack-qa | 06:37 | |
*** woodster_ has quit IRC | 06:40 | |
*** jamielennox|away is now known as jamielennox | 06:44 | |
*** afazekas_ has joined #openstack-qa | 06:47 | |
*** dims__ has joined #openstack-qa | 07:10 | |
*** pal has quit IRC | 07:12 | |
*** pal has joined #openstack-qa | 07:13 | |
*** dims__ has quit IRC | 07:15 | |
*** pkoniszewski has joined #openstack-qa | 07:21 | |
*** abhirc has quit IRC | 07:26 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Add common wait_for_resource_termination https://review.openstack.org/155664 | 07:31 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Use wait_for_resource_termination for nw client https://review.openstack.org/155665 | 07:31 |
*** nikil89_ is now known as naggappan | 07:34 | |
*** naggappan is now known as nandhu | 07:34 | |
*** nandhu is now known as naggappan | 07:34 | |
*** gamado has joined #openstack-qa | 07:35 | |
*** abehl has quit IRC | 07:39 | |
*** gamado has quit IRC | 07:40 | |
*** andreykurilin_ has joined #openstack-qa | 07:42 | |
*** salv-orlando has quit IRC | 07:43 | |
*** andreykurilin_ has quit IRC | 07:50 | |
*** andreykurilin_ has joined #openstack-qa | 07:50 | |
*** gamado has joined #openstack-qa | 07:52 | |
*** gamado has quit IRC | 07:59 | |
*** dimtruck is now known as zz_dimtruck | 08:06 | |
*** andreykurilin_ has quit IRC | 08:07 | |
*** andreykurilin_ has joined #openstack-qa | 08:11 | |
*** SridharG has joined #openstack-qa | 08:16 | |
*** Longgeek has quit IRC | 08:18 | |
*** Longgeek has joined #openstack-qa | 08:19 | |
*** yamamoto has quit IRC | 08:19 | |
*** oomichi has quit IRC | 08:21 | |
*** achanda has quit IRC | 08:21 | |
*** markus_z has joined #openstack-qa | 08:22 | |
*** safchain has joined #openstack-qa | 08:22 | |
*** achanda has joined #openstack-qa | 08:23 | |
*** ihrachyshka has joined #openstack-qa | 08:24 | |
*** pal has quit IRC | 08:30 | |
*** achanda has quit IRC | 08:38 | |
*** jlanoux has joined #openstack-qa | 08:42 | |
*** openstackgerrit has quit IRC | 08:42 | |
*** openstackgerrit has joined #openstack-qa | 08:42 | |
*** pal has joined #openstack-qa | 08:44 | |
*** achanda has joined #openstack-qa | 08:44 | |
openstackgerrit | Yuki Nishiwaki proposed openstack-dev/devstack: Change config of keystone_authtoken in lib/ironic https://review.openstack.org/155360 | 08:46 |
*** mpaolino has joined #openstack-qa | 08:49 | |
openstackgerrit | Yuki Nishiwaki proposed openstack-dev/devstack: Change config of keystone_authtoken in lib/ironic https://review.openstack.org/155360 | 08:50 |
*** vigneshvar has joined #openstack-qa | 08:50 | |
*** jpich has joined #openstack-qa | 08:51 | |
*** gfidente has joined #openstack-qa | 08:54 | |
*** jlibosva has joined #openstack-qa | 08:56 | |
*** jordanP has joined #openstack-qa | 08:56 | |
*** MasterPiece has joined #openstack-qa | 09:05 | |
*** MasterPiece has quit IRC | 09:11 | |
*** achanda has quit IRC | 09:14 | |
*** tkammer has joined #openstack-qa | 09:14 | |
*** tkammer has quit IRC | 09:19 | |
*** jlibosva has quit IRC | 09:20 | |
*** yamamoto has joined #openstack-qa | 09:20 | |
*** jlibosva has joined #openstack-qa | 09:23 | |
*** yamamoto has quit IRC | 09:23 | |
*** e0ne has joined #openstack-qa | 09:26 | |
*** Longgeek has quit IRC | 09:29 | |
*** amotoki has quit IRC | 09:34 | |
*** tkammer has joined #openstack-qa | 09:34 | |
*** salv-orlando has joined #openstack-qa | 09:40 | |
*** obutenko has joined #openstack-qa | 09:45 | |
*** jlibosva has quit IRC | 09:46 | |
*** jlibosva has joined #openstack-qa | 09:46 | |
*** zz_johnthetubagu is now known as johnthetubaguy | 09:47 | |
*** zz_dimtruck is now known as dimtruck | 09:59 | |
*** mpaolino has quit IRC | 10:00 | |
*** yamamoto has joined #openstack-qa | 10:02 | |
*** madhuri_ has quit IRC | 10:05 | |
*** dimtruck is now known as zz_dimtruck | 10:09 | |
*** MasterPiece has joined #openstack-qa | 10:09 | |
*** dims__ has joined #openstack-qa | 10:12 | |
*** mpaolino has joined #openstack-qa | 10:13 | |
*** yamamoto has quit IRC | 10:14 | |
*** dims__ has quit IRC | 10:17 | |
*** e0ne is now known as e0ne_ | 10:23 | |
*** tkammer has quit IRC | 10:31 | |
*** tosky has joined #openstack-qa | 10:31 | |
*** e0ne_ is now known as e0ne | 10:32 | |
openstackgerrit | nithya-ganesan proposed openstack/tempest: WIP - Create shared ssh verification by extending RemoteClient https://review.openstack.org/153899 | 10:37 |
*** tkammer has joined #openstack-qa | 10:46 | |
openstackgerrit | Merged openstack/tempest: Remove unused CONF from limits/identity_client https://review.openstack.org/155632 | 10:54 |
*** aysyd has joined #openstack-qa | 11:02 | |
*** rfolco has joined #openstack-qa | 11:04 | |
*** salv-orlando has quit IRC | 11:05 | |
*** dims__ has joined #openstack-qa | 11:12 | |
*** yamamoto has joined #openstack-qa | 11:14 | |
*** yamamoto has quit IRC | 11:20 | |
andreaf | mtreinish, jamielennox: mtreinish example was quite good, only you'll have to use username=user['name']. And if you don't specify a domain, it will use the domain from CONF.identity.admin_domain | 11:20 |
*** asselin has quit IRC | 11:21 | |
openstackgerrit | vigneshvar proposed openstack-dev/devstack: Upgrades pip under virtualenv https://review.openstack.org/155710 | 11:25 |
*** aix has joined #openstack-qa | 11:27 | |
openstackgerrit | nithya-ganesan proposed openstack/tempest: xWIP - Create shared ssh verification by extending RemoteClient https://review.openstack.org/153899 | 11:29 |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Initial class creds creation in test base class https://review.openstack.org/153681 | 11:31 |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use cls instead of self in classmethod https://review.openstack.org/154693 | 11:31 |
*** aix has quit IRC | 11:38 | |
*** e0ne is now known as e0ne_ | 11:38 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/153226 failed gate-tempest-dsvm-neutron-full in the gate queue with an unrecognized error | 11:41 |
*** yamahata has quit IRC | 11:43 | |
*** e0ne_ has quit IRC | 11:43 | |
*** rfolco has quit IRC | 11:48 | |
*** e0ne has joined #openstack-qa | 11:52 | |
*** apevec has joined #openstack-qa | 12:01 | |
*** dims__ has quit IRC | 12:01 | |
*** dims__ has joined #openstack-qa | 12:02 | |
*** aix has joined #openstack-qa | 12:04 | |
*** salv-orlando has joined #openstack-qa | 12:04 | |
*** e0ne is now known as e0ne_ | 12:04 | |
*** rfolco has joined #openstack-qa | 12:05 | |
*** dims__ has quit IRC | 12:06 | |
*** dims__ has joined #openstack-qa | 12:07 | |
*** jaypipes has quit IRC | 12:13 | |
*** e0ne_ has quit IRC | 12:14 | |
*** jaypipes has joined #openstack-qa | 12:16 | |
*** ChanServ sets mode: +o jaypipes | 12:16 | |
*** eugene_bag has joined #openstack-qa | 12:17 | |
*** ajo_ has joined #openstack-qa | 12:23 | |
openstackgerrit | nithya-ganesan proposed openstack/tempest: xWIP - Create shared ssh verification by extending RemoteClient https://review.openstack.org/153899 | 12:24 |
*** tochi has quit IRC | 12:24 | |
*** ajo has quit IRC | 12:26 | |
*** ymorkovn has joined #openstack-qa | 12:29 | |
*** pal has quit IRC | 12:30 | |
*** dims__ has quit IRC | 12:32 | |
*** pkoniszewski has quit IRC | 12:38 | |
openstackgerrit | Ahmad Faheem proposed openstack/tempest: clear isolated_creds var after the cleanup in clear_isolated_creds. clear_isolated_creds method is called twice once from base class and once from test.py. But second time it fails as the user and tenant are already deleted. So if first time clear_isolate https://review.openstack.org/155736 | 12:51 |
openstackrecheck | openstack/nova change: https://review.openstack.org/153389 failed because of: gate-grenade-dsvm-partial-ncpu: https://bugs.launchpad.net/bugs/1412923, gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1412923 | 12:53 |
openstack | Launchpad bug 1412923 in devstack "heat api takes too long to come up, tempest tests fail" [Undecided,In progress] - Assigned to Davanum Srinivas (DIMS) (dims-v) | 12:53 |
*** yamamoto has joined #openstack-qa | 12:57 | |
*** Madan has quit IRC | 13:03 | |
*** zz_dimtruck is now known as dimtruck | 13:08 | |
*** pkoniszewski has joined #openstack-qa | 13:11 | |
*** jpich has quit IRC | 13:12 | |
*** dims__ has joined #openstack-qa | 13:13 | |
*** dims___ has joined #openstack-qa | 13:16 | |
*** dims__ has quit IRC | 13:17 | |
*** EmilienM|afk is now known as EmilienM | 13:19 | |
*** yamamoto has quit IRC | 13:27 | |
*** aysyd has quit IRC | 13:27 | |
*** dimtruck is now known as zz_dimtruck | 13:30 | |
*** aysyd has joined #openstack-qa | 13:31 | |
*** pal has joined #openstack-qa | 13:31 | |
*** marrusl has joined #openstack-qa | 13:33 | |
openstackrecheck | Console logs not available after 13:35s for check-tempest-dsvm-neutron-full 153681,18,122f7bd | 13:36 |
*** pkoniszewski has quit IRC | 13:36 | |
*** pal has quit IRC | 13:39 | |
*** vigneshvar has quit IRC | 13:39 | |
*** toure has joined #openstack-qa | 13:47 | |
*** Yogi1 has joined #openstack-qa | 13:54 | |
*** polew has joined #openstack-qa | 13:56 | |
*** SridharG has left #openstack-qa | 13:57 | |
*** Yogi11 has joined #openstack-qa | 13:59 | |
*** Yogi1 has quit IRC | 14:03 | |
*** pkoniszewski has joined #openstack-qa | 14:11 | |
*** dustins has joined #openstack-qa | 14:16 | |
*** gamado has joined #openstack-qa | 14:16 | |
*** david-lyle_afk is now known as david-lyle | 14:22 | |
openstackgerrit | nithya-ganesan proposed openstack/tempest: xWIP - Create shared ssh verification by extending RemoteClient https://review.openstack.org/153899 | 14:22 |
*** e0ne has joined #openstack-qa | 14:26 | |
dkranz | afazekas, mtreinish, andreaf : a +A on https://review.openstack.org/154537 and https://review.openstack.org/155053 would be appreciated. Trying to avoid another rebase | 14:27 |
*** tnurlygayanov has joined #openstack-qa | 14:30 | |
ymorkovn | Hi, colleagues could some core members take a look https://review.openstack.org/#/c/153234/ Thanks | 14:30 |
*** Yogi11 has quit IRC | 14:33 | |
*** annegentle has quit IRC | 14:33 | |
*** Yogi1 has joined #openstack-qa | 14:39 | |
*** mpaolino has quit IRC | 14:40 | |
*** abhirc has joined #openstack-qa | 14:40 | |
*** Ananci has joined #openstack-qa | 14:44 | |
*** mriedem_zz has quit IRC | 14:44 | |
*** mriedem has joined #openstack-qa | 14:44 | |
*** e0ne is now known as e0ne_ | 14:45 | |
*** naggappan has quit IRC | 14:46 | |
*** aysyd has quit IRC | 14:51 | |
*** woodster_ has joined #openstack-qa | 14:53 | |
*** sabeen1 has joined #openstack-qa | 14:54 | |
*** aysyd has joined #openstack-qa | 14:54 | |
*** e0ne_ is now known as e0ne | 14:55 | |
*** eugene_bag1 has joined #openstack-qa | 14:55 | |
*** jaypipes is now known as leakypipes | 14:56 | |
*** zz_jgrimm is now known as jgrimm | 14:56 | |
*** jecarey has joined #openstack-qa | 14:56 | |
openstackgerrit | gordon chung proposed openstack/tempest: add support for ceilometermiddleware data https://review.openstack.org/155766 | 14:56 |
dkranz | mtreinish: thanks! | 14:57 |
*** eugene_bag has quit IRC | 14:57 | |
*** dansmith is now known as superdan | 14:58 | |
dkranz | mtreinish: btw, re swift client, it seems the swift api is based almost completely on headers for response, not json body. The exception being when data is returned as a blob. | 14:58 |
dkranz | mtreinish: So my idea is to just change most client methods to not return a bogus 'body' which is always ignored | 14:59 |
*** malini has joined #openstack-qa | 14:59 | |
dkranz | mtreinish: it would be nice if python treated extra return values as ignored rather than bundling to a tuple, but it does not. | 14:59 |
*** malini has left #openstack-qa | 15:00 | |
*** liusheng has quit IRC | 15:01 | |
*** liusheng has joined #openstack-qa | 15:01 | |
mtreinish | dkranz: on https://review.openstack.org/#/c/154537/ I think we should manually rebase it on the compute one (because I approved that first) | 15:02 |
mtreinish | I'll +A it after | 15:02 |
mtreinish | dkranz: sure, that was how I remembered swift's api working | 15:02 |
mtreinish | dkranz: you cann't just use the ResponseBodyData type and set data to None? | 15:03 |
dkranz | mtreinish: Then you would always have to reference the response value | 15:04 |
dkranz | mtreinish: that is what I was trying to avoid | 15:05 |
dkranz | mtreinish: how do I do a manual rebase on another patch? | 15:05 |
dkranz | mtreinish: never done that before | 15:05 |
mtreinish | dkranz: oh the easiest way for me (might not actually be the easiest) is to checkout the server patch locally with git review -d 155053 | 15:06 |
mtreinish | then cherry-pick the floating-ips one on top of it with git review -x 154537 | 15:07 |
mtreinish | if there is a conflict it'll tell you how to deal with that | 15:07 |
dkranz | mtreinish: k, I'll give it a whirl | 15:07 |
dkranz | mtreinish: no conflicts. So that means manual rebase is not necessary, right? | 15:09 |
*** sabeen1 has quit IRC | 15:11 | |
openstackgerrit | Rossella Sblendido proposed openstack-infra/elastic-recheck: Add e-r query for test_rebuild_availability_range https://review.openstack.org/155234 | 15:13 |
*** bdperkin has quit IRC | 15:13 | |
*** bdperkin has joined #openstack-qa | 15:13 | |
mtreinish | dkranz: probably not, although gerrit uses a different git engine (jgit) so it's never 100% | 15:13 |
mtreinish | dkranz: also on: https://review.openstack.org/#/c/154537/4/tempest/api/compute/floating_ips/test_floating_ips_actions.py | 15:14 |
mtreinish | what was with the addCleanup? | 15:14 |
openstackgerrit | Rossella Sblendido proposed openstack-infra/elastic-recheck: Add e-r query for test_rebuild_availability_range https://review.openstack.org/155234 | 15:14 |
*** MasterPiece has quit IRC | 15:14 | |
*** MasterPiece has joined #openstack-qa | 15:15 | |
*** SridharG has joined #openstack-qa | 15:15 | |
*** dims___ is now known as dimsum__ | 15:15 | |
*** rbak has joined #openstack-qa | 15:15 | |
dkranz | mtreinish: The addCleanUp is not necessary. I didn't remove it because it was unrelated. | 15:16 |
mtreinish | dkranz: oh I see now the gerrit formatting was just weird, on my view it looked like a new line | 15:19 |
mtreinish | but it was already there | 15:19 |
dkranz | mtreinish: right | 15:19 |
dkranz | mtreinish: should I push the rebase? | 15:20 |
mtreinish | dkranz: I just +A'd it you can and I'll redo it | 15:20 |
mtreinish | but it'll have to go through check again | 15:20 |
mtreinish | lets just wait and see. I'm guessing it'll get through without issue | 15:21 |
dkranz | mtreinish: If you already +A it doesn't matter. | 15:21 |
dkranz | mtreinish: gerrit rebase will succeed even if there was a semantic conflict | 15:21 |
dkranz | mtreinish: ok, thanks | 15:21 |
dkranz | mtreinish: git seems to be better at allowing close changes to be merged than other scs I have used :) | 15:22 |
*** jlibosva has quit IRC | 15:22 | |
*** jlibosva has joined #openstack-qa | 15:22 | |
openstackgerrit | Merged openstack-infra/elastic-recheck: Add e-r query for test_rebuild_availability_range https://review.openstack.org/155234 | 15:25 |
andreaf | dkranz, mtreinish: I have a question about the set_network_resources, maybe you have a better idea on how to solve this: https://review.openstack.org/#/c/153681/17 | 15:26 |
*** bdperkin has quit IRC | 15:26 | |
andreaf | dkranz, mtreinish: in my patch I'm moving the credentials creation to setup_credentials in test.py - however test classes not yet migrated to using setup_credentials my call set_network_resources in resource_setup, which is too late now | 15:28 |
andreaf | dkranz, mtreinish: and I have a few alternative implementations in mind: detect a late call of set_network_credentials and fix things accordingly - which does not seem to work reliably | 15:29 |
andreaf | dkranz, mtreinish: fix all 32 cases of calls to set_network_resources in the same patch | 15:29 |
andreaf | dkranz, mtreinish: or drop set_network_resources and use a class level dict instead | 15:30 |
openstackgerrit | Merged openstack/tempest: Split resource_setup for database tests https://review.openstack.org/153158 | 15:30 |
mtreinish | andreaf: what is wrong with just keepin set_network_resources just before the setup_creds call? | 15:31 |
openstackgerrit | Merged openstack/tempest: Rename data_processing client https://review.openstack.org/155633 | 15:31 |
openstackgerrit | Merged openstack/tempest: Remove CONF values from data_processing client https://review.openstack.org/155634 | 15:31 |
openstackgerrit | Merged openstack/tempest: Remove CONF reference from volume client https://review.openstack.org/155642 | 15:31 |
mtreinish | the current solution was made to ensure that the network resource dict was set only once per test class and that the leaf always took prio over it's parent | 15:32 |
andreaf | mtreinish: so, that would be my 2nd option | 15:32 |
*** toure has quit IRC | 15:32 | |
andreaf | mtreinish: the problem is just in the migration from the current with one resource_setup to the target with multiple steps | 15:33 |
mtreinish | andreaf: I think making the separation of resources cleaner around networks and creds is good, but I think that should be a 2nd step after we do the split of what's there now | 15:33 |
mtreinish | or am I missing something? | 15:35 |
andreaf | mtreinish: I guess we can do that, only each patch to a base class will have to deal with all the subclasses calling set_network_resources in the same patch | 15:36 |
*** bdperkin has joined #openstack-qa | 15:36 | |
*** vigneshvar has joined #openstack-qa | 15:37 | |
andreaf | mtreinish: which is the issue ekhugen was hitting | 15:37 |
mtreinish | andreaf: only if the subclass has set_network_resources right? | 15:37 |
andreaf | yes | 15:37 |
mtreinish | I don't think there are too many that do that | 15:37 |
*** asselin has joined #openstack-qa | 15:37 | |
andreaf | 32 in all tempest, yes, not too many | 15:37 |
andreaf | so ok I can do it after | 15:38 |
mtreinish | heh, 32 is nice round number a nice coincidence | 15:38 |
*** e0ne is now known as e0ne_ | 15:38 | |
*** eugene_bag1 has quit IRC | 15:38 | |
andreaf | what do you think about the list of required credentials in https://review.openstack.org/#/c/153681/17 ? | 15:41 |
andreaf | https://review.openstack.org/#/c/153681/17/tempest/test.py,cm | 15:41 |
andreaf | mtreinish, ^^ | 15:41 |
*** marun has joined #openstack-qa | 15:42 | |
andreaf | it's a different approach compared to the set_network_resources, and perhaps we should either use an helper function or class level data for all kind of resources | 15:42 |
andreaf | there will be also validation resources with the ssh-auth-strategy bp | 15:42 |
*** coolsvap is now known as coolsvap_ | 15:43 | |
openstackgerrit | Vladik Romanovsky proposed openstack/tempest: Negative unrealistic NUMA test https://review.openstack.org/143541 | 15:43 |
openstackgerrit | Vladik Romanovsky proposed openstack/tempest: Minimal NUMA support test https://review.openstack.org/143540 | 15:43 |
*** openstackrecheck has quit IRC | 15:43 | |
*** openstackrecheck has joined #openstack-qa | 15:43 | |
*** e0ne_ is now known as e0ne | 15:43 | |
openstackgerrit | Merged openstack/tempest: Change basic server client methods to return one value and update tests https://review.openstack.org/155053 | 15:45 |
mtreinish | andreaf: oh, yeah that does look nicer. I agree moving to that model for everything might be good | 15:45 |
*** toure has joined #openstack-qa | 15:46 | |
andreaf | mtreinish: ok, so that I could do as a dedicated patch including all 32 cases before everything else | 15:47 |
*** vigneshvar has quit IRC | 15:47 | |
*** MasterPiece has quit IRC | 15:48 | |
andreaf | mtreinish: I'll propose a change then | 15:49 |
mtreinish | andreaf: I'm still thinking it would be better to just do 1 migration at a time. So finish the split resources as it is now. Then we can look into improving the cred allocation bits | 15:49 |
mtreinish | otherwise it can get a bit messy if we try to tackle too much change at the same time | 15:50 |
andreaf | mtreinish: if we get rid of set_network_resources in favour of a dict, it will make life easier I think | 15:50 |
andreaf | mtreinish: and we will have to do that change in one go nonetheless | 15:51 |
andreaf | mtreinish: I'll prepare a patch to show what I mean, we can decide whether it goes first or after the other migrations | 15:52 |
mtreinish | andreaf: ok we can give it a shot at least, I still have reservations but at least seeing the patch doesn't hurt | 15:52 |
mtreinish | andreaf: the other thing is there are how many split patches up for review, do we want to block all of those on this? | 15:52 |
andreaf | mtreinish: yes that's a good point | 15:53 |
andreaf | mtreinish: if the patches up for review are ok and pass check we should merge them, I'll work on this change in parallel | 15:54 |
*** MasterPiece has joined #openstack-qa | 15:57 | |
*** pcrews has joined #openstack-qa | 15:59 | |
*** polew has quit IRC | 16:00 | |
*** ajo_ has quit IRC | 16:05 | |
*** Yogi1 has quit IRC | 16:05 | |
*** ajo has joined #openstack-qa | 16:05 | |
*** mriedem1 has joined #openstack-qa | 16:06 | |
*** Shrews has left #openstack-qa | 16:08 | |
*** mriedem has quit IRC | 16:08 | |
*** jgrimm_ has joined #openstack-qa | 16:10 | |
*** timello_ has joined #openstack-qa | 16:10 | |
*** ekhugen_alt has joined #openstack-qa | 16:10 | |
*** cyeoh has quit IRC | 16:10 | |
*** maurosr has quit IRC | 16:11 | |
*** bknudson has quit IRC | 16:12 | |
*** timello has quit IRC | 16:12 | |
*** jgrimm- has joined #openstack-qa | 16:13 | |
*** timello has joined #openstack-qa | 16:13 | |
*** jgrimm has quit IRC | 16:13 | |
*** ekhugen has quit IRC | 16:13 | |
*** ekhugen has joined #openstack-qa | 16:13 | |
*** cyeoh has joined #openstack-qa | 16:13 | |
*** maurosr has joined #openstack-qa | 16:14 | |
openstackgerrit | Dean Troyer proposed openstack-dev/devstack: Build a virtual environment for clients https://review.openstack.org/151513 | 16:14 |
*** timello_ has quit IRC | 16:16 | |
*** ekhugen_alt has quit IRC | 16:17 | |
*** jgrimm_ has quit IRC | 16:17 | |
openstackgerrit | gordon chung proposed openstack/tempest: add support for ceilometermiddleware data https://review.openstack.org/155766 | 16:19 |
*** MasterPiece has quit IRC | 16:20 | |
*** shashankhegde has joined #openstack-qa | 16:22 | |
*** sabeen1 has joined #openstack-qa | 16:23 | |
*** MasterPiece has joined #openstack-qa | 16:24 | |
*** bknudson has joined #openstack-qa | 16:27 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/154537 failed because of: gate-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/1321207, gate-tempest-dsvm-postgres-full: unrecognized error | 16:30 |
openstack | Launchpad bug 1321207 in tempest "test_network_basic_ops.TestNetworkBasicOps.test_hotplug_nic fails" [Undecided,In progress] - Assigned to Armando Migliaccio (armando-migliaccio) | 16:30 |
mtreinish | sdague: heh, so after looking at a few test runs since the testr preseeding has landed I don't think it makes the worker balance any better | 16:33 |
*** Yogi1 has joined #openstack-qa | 16:33 | |
mtreinish | in some cases it looks pretty unbalanced: http://logs.openstack.org/61/153461/2/gate/gate-tempest-dsvm-full/e6d45c3/console.html#_2015-02-11_16_50_53_990 | 16:33 |
sdague | hmmm | 16:34 |
sdague | so, there is no way that's actually using the data | 16:34 |
mtreinish | sdague: oh, you're right the order is unchanged | 16:35 |
sdague | look at the order the tests are running in | 16:35 |
sdague | yeh | 16:35 |
mtreinish | hmm, it's definitely loading the streams: http://logs.openstack.org/61/153461/2/gate/gate-tempest-dsvm-full/e6d45c3/console.html#_2015-02-11_16_27_11_464 | 16:36 |
sdague | so, honestly, I'm not convinced that testr is really optimizing any more | 16:36 |
mtreinish | I guess I'll have to do more digging to see why it's not | 16:36 |
sdague | even locally with nova unit tests | 16:36 |
mtreinish | when I looked at it before (right after it landed) the order was being changed | 16:36 |
*** gfidente has quit IRC | 16:37 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/devstack-gate: Add message about preloading subunit streams https://review.openstack.org/155816 | 16:40 |
openstackgerrit | Matthew Treinish proposed openstack-infra/devstack-gate: DO NOT MERGE COPY ALL OF .TESTREPOSITORY to log server https://review.openstack.org/149719 | 16:41 |
*** ihrachyshka has quit IRC | 16:47 | |
*** polew has joined #openstack-qa | 16:49 | |
*** gfidente has joined #openstack-qa | 16:52 | |
*** afazekas_ has quit IRC | 16:53 | |
*** mlavalle has joined #openstack-qa | 16:54 | |
*** ihrachyshka has joined #openstack-qa | 16:54 | |
*** yamahata has joined #openstack-qa | 16:54 | |
*** pkoniszewski has quit IRC | 16:55 | |
*** MasterPiece has quit IRC | 16:55 | |
*** EmilienM is now known as EmilienM|afk | 16:56 | |
*** e0ne is now known as e0ne_ | 16:58 | |
*** jordanP has quit IRC | 16:59 | |
*** jlibosva has quit IRC | 16:59 | |
*** jlibosva has joined #openstack-qa | 16:59 | |
*** MasterPiece has joined #openstack-qa | 17:00 | |
*** jlanoux has quit IRC | 17:00 | |
*** e0ne_ is now known as e0ne | 17:03 | |
openstackgerrit | Vladik Romanovsky proposed openstack/tempest: Negative unrealistic NUMA test https://review.openstack.org/143541 | 17:03 |
*** gamado has quit IRC | 17:04 | |
*** shashankhegde has quit IRC | 17:04 | |
*** MasterPiece has quit IRC | 17:10 | |
*** e0ne is now known as e0ne_ | 17:11 | |
*** safchain has quit IRC | 17:15 | |
*** e0ne_ is now known as e0ne | 17:16 | |
sc68cal | sdague: How does the gate configure the number of interfaces that a node is spawned with? | 17:18 |
sc68cal | I see DEVSTACK_GATE_TOPOLOGY being defined in the devstack-gate git repo | 17:18 |
sdague | DEVSTACK_GATE_TOPOLOGY is about number of nodes started | 17:18 |
sdague | not about network interfaces | 17:19 |
sdague | mostly | 17:19 |
sc68cal | Gotcha, I'm just following the breadcrumbs to figure out how multi-node neutron is built and run at the gate | 17:19 |
sc68cal | # of interfaces that the controller and compute nodes come up with, etc. | 17:20 |
*** abehl has joined #openstack-qa | 17:20 | |
mtreinish | sc68cal: http://git.openstack.org/cgit/openstack-infra/devstack-gate/plain/nova_network_multinode_diagram.txt | 17:21 |
sc68cal | oh awesome | 17:21 |
mtreinish | clarkb made taht diagram to explain the networking for the multinode setup | 17:21 |
sc68cal | ahh no wonder I didn't see that, my repo for devstack-gate was a bit old | 17:23 |
clarkb | note that is specific to nova network tests, neutron do their own tunneling aiui | 17:24 |
clarkb | and d-g doesn't control what neutron does | 17:24 |
*** Shrews has joined #openstack-qa | 17:25 | |
*** alex_xu_ has joined #openstack-qa | 17:30 | |
sc68cal | clarkb: understood, regarding what neutron does - however is it nodepool that generates the requests to RAX and HP for spawning new nodes? | 17:32 |
sc68cal | like what piece does the actual requests to RAX and HP to spawn the nodes? | 17:32 |
clarkb | sc68cal: yes, nodepool today creates two node envs as a jenkins flavor type, then jobs can state they run on that flavor | 17:32 |
*** xu_alex has quit IRC | 17:32 | |
clarkb | it ensures that all multinode test flavors have nodes from the same region so you won't get one node in rax and the other in hp | 17:33 |
sc68cal | gotcha | 17:33 |
clarkb | and number of interfaces does change depending on cloud. hp is one, rax is 2, for nova net multinode we have to do the artificial layer 2 like that otherwise ebtables got cranky | 17:35 |
*** Shrews has left #openstack-qa | 17:35 | |
* sc68cal continues reading | 17:36 | |
*** yjiang5 is now known as yginger5 | 17:36 | |
sc68cal | clarkb: when you say doing the artificial layer 2, is that done inside devstack-gate or nodepool | 17:36 |
clarkb | sc68cal: devstack-gate | 17:37 |
sc68cal | gotcha, hence that nova_network_multinode_diagram.txt | 17:37 |
clarkb | sc68cal: the devstack-gate diagram shows what devstack-gate does ya | 17:37 |
*** pkoniszewski has joined #openstack-qa | 17:37 | |
*** harlowja_away is now known as harlowja_ | 17:48 | |
*** pkoniszewski has quit IRC | 17:52 | |
*** jogo is now known as flashgordon | 17:54 | |
*** MasterPiece has joined #openstack-qa | 17:55 | |
*** e0ne has quit IRC | 17:56 | |
*** markus_z has quit IRC | 17:57 | |
*** mtanino has joined #openstack-qa | 18:01 | |
*** johnthetubaguy is now known as zz_johnthetubagu | 18:04 | |
*** tsekiyama has joined #openstack-qa | 18:07 | |
*** patrickeast has joined #openstack-qa | 18:17 | |
*** vigneshvar has joined #openstack-qa | 18:17 | |
*** Guest40096 has joined #openstack-qa | 18:21 | |
*** melwitt has joined #openstack-qa | 18:21 | |
*** jlibosva has quit IRC | 18:21 | |
*** Guest40096 is now known as annegentle | 18:22 | |
*** achanda has joined #openstack-qa | 18:25 | |
*** liusheng has quit IRC | 18:25 | |
*** liusheng has joined #openstack-qa | 18:25 | |
*** tkammer has quit IRC | 18:27 | |
*** polew has quit IRC | 18:29 | |
*** EmilienM|afk is now known as EmilienM | 18:32 | |
*** andreykurilin_ has quit IRC | 18:32 | |
*** luqas has quit IRC | 18:34 | |
*** shashankhegde has joined #openstack-qa | 18:34 | |
*** aix has quit IRC | 18:34 | |
*** vigneshvar has quit IRC | 18:38 | |
*** vigneshvar has joined #openstack-qa | 18:39 | |
*** dkranz has quit IRC | 18:40 | |
*** dkranz has joined #openstack-qa | 18:43 | |
*** SridharG has left #openstack-qa | 18:44 | |
hogepodge | mtreinish, working on the hacking rule for uuid | 18:46 |
hogepodge | I've come to the conclusion that a scenario decorator needs to come after uuid decorator | 18:47 |
hogepodge | Wait, nevermind on that. | 18:47 |
hogepodge | tripped myself up on the logic and what was being searched on. | 18:48 |
hogepodge | Although there is a problem in returning the function name of the bad scenario test if the hacking rule fails. | 18:49 |
dkranz | sdague: I just got a gate fail with "raise ResponseNotReady" in a bunch of heat tests. Logstash shows this happens a fair bit and always in the grenade job. Is anything more known about this? | 18:53 |
*** MasterPiece has quit IRC | 18:54 | |
*** sabeen1 has quit IRC | 18:55 | |
*** tosky has quit IRC | 18:57 | |
*** liusheng has quit IRC | 18:57 | |
*** liusheng has joined #openstack-qa | 18:58 | |
*** MasterPiece has joined #openstack-qa | 19:01 | |
*** toure has quit IRC | 19:03 | |
dimsum__ | dkranz: last thing i did was to bump up the number of workers for heat api, did not seem to have helped | 19:05 |
*** polew has joined #openstack-qa | 19:05 | |
dkranz | dimsum__: ok, thanks. reverify is my friend I guess. | 19:05 |
*** abhirc has quit IRC | 19:08 | |
*** yamahata has quit IRC | 19:12 | |
openstackgerrit | Chris Hoge proposed openstack/tempest: Add new hacking rule for UUID metatdata https://review.openstack.org/154127 | 19:15 |
hogepodge | mtreinish ^^ That's going to have to become part of the big bang code push, so it's more of a WIP and needs to be merged into that, but up for review to help get it right. | 19:17 |
hogepodge | Added tests to verify checks. | 19:17 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/154537 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1412923 | 19:22 |
openstack | Launchpad bug 1412923 in devstack "heat api takes too long to come up, tempest tests fail" [Undecided,In progress] - Assigned to Davanum Srinivas (DIMS) (dims-v) | 19:22 |
*** Yogi1 has quit IRC | 19:23 | |
mtreinish | hogepodge: sure, yeah that'll come after we do the migration to make sure we don't slip | 19:33 |
mtreinish | but we can review that the functionally it looks right going up into that | 19:33 |
*** Yogi1 has joined #openstack-qa | 19:35 | |
kashyap | Wow, in my DevStack setup (just a day's old) - I was aghast at seeing the DevStack VM reboot on its serial console, while I boot a *Nova* instance in the DevStack VM. | 19:47 |
kashyap | I could confirm this as I saw it happen *thrice* | 19:47 |
kashyap | Wonder if anyone here even remotely remember seeing something like that | 19:47 |
*** sabeen has joined #openstack-qa | 19:48 | |
*** eugene_bag has joined #openstack-qa | 19:49 | |
*** eugene_bag1 has joined #openstack-qa | 19:51 | |
*** eugene_bag1 has quit IRC | 19:51 | |
* kashyap will check scroll later, need to get back to civilization | 19:52 | |
*** shashankhegde has quit IRC | 19:52 | |
*** eugene_bag has quit IRC | 19:53 | |
*** shashankhegde has joined #openstack-qa | 19:59 | |
*** Sukhdev has joined #openstack-qa | 19:59 | |
mtreinish | sdague, andreaf: if you guys get a sec: https://review.openstack.org/#/c/155103/ | 20:03 |
mtreinish | I think it's about time we improved the docs for tempest-lib | 20:03 |
*** MasterPiece has quit IRC | 20:04 | |
*** rfolco has quit IRC | 20:04 | |
sdague | dkranz: so honestly, we should probably not be running any of the heat tests in smoke / grenade | 20:08 |
sdague | can we get that turned off one way or another | 20:08 |
mtreinish | sdague: does heat run on grenade? The tests would be skipped if they're not | 20:09 |
mtreinish | but I agree there is no reason to have them there | 20:10 |
*** patrickeast has quit IRC | 20:11 | |
openstackgerrit | vigneshvar proposed openstack-dev/devstack: Upgrades pip under virtualenv https://review.openstack.org/155710 | 20:11 |
sdague | oh... well interesting. It looks like we are starting heat on old, but not on new | 20:13 |
mtreinish | sdague: that's weird | 20:13 |
bknudson | question about moving client/cli tests out of tempest -- tempest is branchless, but the client libs aren't.... so are we not going to be testing something now? | 20:15 |
mtreinish | bknudson: well unless you set the jobs up to use the clients against stable branches that will be lost | 20:16 |
mtreinish | but tempest is only running w/ the latest client releases now | 20:17 |
*** diegows has joined #openstack-qa | 20:18 | |
bknudson | right... maybe we'll have stable branches of clients since we're capping in stable/. | 20:18 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: remove heat from starting in grenade https://review.openstack.org/155876 | 20:18 |
*** MasterPiece has joined #openstack-qa | 20:18 | |
mtreinish | bknudson: but that wouldn't be for the cli interface | 20:18 |
mtreinish | bknudson: the capping only goes to clients being used internally for services (or at least thats the long term goal) | 20:18 |
mtreinish | we still want to ensure the latest clients work with old releases | 20:19 |
bknudson | the client functional tests are only for CLI? | 20:19 |
bknudson | our goal is to get rid of keystone CLI in favor of unified cli. | 20:19 |
mtreinish | bknudson: the ones in tempest are | 20:20 |
*** pc_m has joined #openstack-qa | 20:20 | |
pc_m | mtreinish: ping | 20:20 |
mtreinish | bknudson: the clients are woefully undertested | 20:20 |
mtreinish | pc_m: pong | 20:20 |
bknudson | testing clis isn't easy to begin with. | 20:21 |
bknudson | or any human interface | 20:21 |
*** ihrachyshka has quit IRC | 20:21 | |
pc_m | mtreinish: I was trying to run coverage test on dsvm-functional. Made a new target, functional-cover, and added --coverage and --coverage-package-name to the testr command | 20:21 |
mtreinish | bknudson: sure, if you look at the cli tests in tempest they're very manual | 20:22 |
mtreinish | pc_m: ok | 20:22 |
pc_m | mtreinish: When I run, I get a warning about test command found, but not installed in testenv, with a message asking if I forgot a dependency | 20:22 |
mtreinish | pc_m: can you pastebin it? | 20:22 |
pc_m | mtreinish: Then I get an InvocationError at end | 20:22 |
pc_m | sure | 20:22 |
bknudson | so, just wondering -- the functional tests in keystoneclient are only for the CLI and not the python API? | 20:23 |
pc_m | mtreinish: http://paste.openstack.org/show/173174/ | 20:23 |
mtreinish | bknudson: so the ones jamielennox is migrating from tempest-lib are just for the cli | 20:23 |
mtreinish | bknudson: but there is nothing you guys from expanding that to test the python api | 20:23 |
mtreinish | s/from/with | 20:24 |
bknudson | ok, thanks. | 20:24 |
mtreinish | bknudson: we just have any existing groundwork doing that | 20:24 |
mtreinish | if you guys come up with a good model for functionally testing the python api on the clients its worth advertising | 20:24 |
pc_m | mtreinish: Fragment of tox.ini http://paste.openstack.org/show/173175/ | 20:25 |
mtreinish | pc_m: so you need to at least install coverage in your venv. Is it in the test-requirements file | 20:25 |
*** yamahata has joined #openstack-qa | 20:25 | |
openstackgerrit | Dean Troyer proposed openstack-dev/devstack: Fix is_keystone_enabled() https://review.openstack.org/155638 | 20:25 |
pc_m | mtreinish: yes, and the 'cover' target works | 20:26 |
mtreinish | pc_m: so the WARNING:test command thing isn't anything to worry about | 20:27 |
mtreinish | you can add a whitelist arg to your tox.ini to make that go away | 20:27 |
mtreinish | but the issue is '/bin/sh: 1: coverage: not found' | 20:27 |
mtreinish | pc_m: I would activate the tox venv for the functional-coverage job and make sure coverage is installed in the venv as the first step | 20:29 |
pc_m | mtreinish: OK. how do I do that (in the tox.ini, or are you saying to enter the venv first)? | 20:30 |
mtreinish | pc_m: yeah just enter the venv with source .tox/functional-cover/bin/activate and run a pip freeze or something | 20:30 |
mtreinish | just to sanity check that coverage was installed in the venv | 20:31 |
pc_m | mtreinish: If I enter the functional-cover venv, and check pip show it has coerage 3.7.1 | 20:31 |
*** achanda has quit IRC | 20:31 | |
mtreinish | pc_m: hmm ok and what happens if you run coverage manually from inside the venv | 20:31 |
pc_m | mtreinish: Just trying that... nothing. pip show indicates the location as /usr/lib/python2.7/dist-packages, which coverage shows nothing | 20:32 |
pc_m | don't see it in .tox/functional-cover/bin | 20:33 |
mtreinish | pc_m: hmm ok then I'd manually reinstall it in the venv | 20:34 |
mtreinish | pc_m: oh you have site packages enabled | 20:34 |
mtreinish | try removing that line from the tox.ini and recreate the venv | 20:34 |
pc_m | mtreinish: Did a find. It is under pep8, docs, cover, and py27 dirs | 20:34 |
mtreinish | with tox -efunctional-cover -r | 20:34 |
pc_m | there is no site-packages dir in the python2.7 area under functional-cover | 20:35 |
mtreinish | pc_m: it's a line in the tox.ini | 20:35 |
mtreinish | site_packages = true | 20:35 |
mtreinish | remove that line | 20:35 |
mtreinish | and recreate the venv | 20:35 |
pc_m | gotcha | 20:36 |
*** patrickeast has joined #openstack-qa | 20:36 | |
pc_m | Just mentioning that there is no site-packages dir in that venv | 20:36 |
mtreinish | pc_m: having site-packages enabled is probably seeing that you have it installed globally on your system and deciding to use that instead of installing it in the venv | 20:36 |
mtreinish | it'll just link to the system coverage python module | 20:36 |
*** bnemec is now known as beekneemech | 20:37 | |
pc_m | mtreinish: commented out the line, rebuilding venv | 20:38 |
*** yginger5 is now known as yginger5_away | 20:39 | |
*** abhirc has joined #openstack-qa | 20:40 | |
pc_m | mtreinish: So it ran, no errors, but said module neutron_vpnass was never imported. I spelled the module wrong :( | 20:41 |
mtreinish | pc_m: heh, I do a double take everytime is see *aas so I guess that's appropriate :) | 20:42 |
pc_m | mtreinish: :) | 20:42 |
pc_m | mtreinish: So because of the site-packages line, it was using global, but wasn't finding because I don't have it installed on base system? | 20:43 |
*** achanda has joined #openstack-qa | 20:43 | |
mtreinish | pc_m: heh, then I don't know | 20:44 |
mtreinish | but that was my guess based on your symptoms | 20:44 |
mtreinish | and it looks like removing that fixed it | 20:44 |
pc_m | mtreinish: Would make sense as the cover command does not have that line. | 20:44 |
mtreinish | hogepodge: on https://review.openstack.org/#/c/154127/ it looks like your unit tests are failing | 20:44 |
* pc_m trying to understand it all | 20:44 | |
pc_m | Looks like it worked w/o error | 20:45 |
pc_m | mtreinish: ^^ awesome | 20:45 |
mtreinish | pc_m: cool, does the report show what you're looking for? | 20:45 |
openstackgerrit | Merged openstack/tempest-lib: Improve the library's documentation https://review.openstack.org/155103 | 20:45 |
pc_m | mtreinish: Well... that's another question (sorry)... | 20:46 |
pc_m | mtreinish: I see a functional html file in the cover area. On this system, ubuntu server, I haven't setup for web server, etc. Is there a way to get a non-html report? | 20:47 |
mtreinish | pc_m: there is a cli interface: http://nedbatchelder.com/code/coverage/ | 20:48 |
mtreinish | but I can't recall if generating a html report removes the coverage files the cli depends on or not | 20:48 |
mtreinish | pc_m: what I normally end up doing is just scp'ing the files off the remote node I run coverage on | 20:49 |
mtreinish | and view them with firefox on my machine running X | 20:49 |
pc_m | mtreinish: gotcha... no biggy. I guess I can try to setup to forward port 80 to my machine and view that way maybe. | 20:50 |
pc_m | mtreinish: Cool though. You got me going. Appreciate the help! | 20:50 |
mtreinish | pc_m: you can't copy the files off the remote system? | 20:50 |
*** pkoniszewski has joined #openstack-qa | 20:51 | |
pc_m | yeah, I can do that too | 20:51 |
pc_m | probably easier... | 20:51 |
pc_m | mtreinish: Thanks so much for the help! | 20:56 |
openstackrecheck | openstack/trove change: https://review.openstack.org/155593 failed because of: gate-trove-python27: https://bugs.launchpad.net/bugs/1379729 | 20:56 |
openstack | Launchpad bug 1379729 in Trove "test_tranformer unit test fails with ModelNotFoundError: InstanceServiceStatus Not Found" [Medium,Triaged] - Assigned to Nikhil Manchanda (slicknik) | 20:56 |
mtreinish | pc_m: sure, np | 20:57 |
*** dustins has quit IRC | 20:57 | |
*** e0ne has joined #openstack-qa | 21:17 | |
*** abehl has quit IRC | 21:19 | |
*** aysyd has quit IRC | 21:20 | |
pc_m | mtreinish: ping. Had another question. | 21:22 |
*** Yogi1 has quit IRC | 21:22 | |
*** e0ne has quit IRC | 21:26 | |
mtreinish | pc_m: sure | 21:30 |
pc_m | mtreinish: Some of the tests, like coverage and dsvm-functional had the prefix lockutils-wrapper on them. Someone mentioned that this was not needed, so I had removed from cover target. Should I remove from functional-cover and/or other targets? | 21:30 |
mtreinish | pc_m: to be honest I'm not exactly sure what that is or why it's needed. | 21:31 |
mtreinish | it does seem weird to me to have to call something like that before the test runner though | 21:32 |
pc_m | mtreinish: any idea who would know? | 21:32 |
pc_m | or I could ask? | 21:32 |
mtreinish | pc_m: well I'd look at the commit history to see when it was added and why and bug the author of that patch | 21:32 |
mtreinish | or if it's an oslo thing just bug dhellmann | 21:33 |
pc_m | mtreinish: good idea. | 21:33 |
pc_m | See commit 88e2d801 (migrate to oslo.concurrency. | 21:34 |
pc_m | Will see if Ihar is around. | 21:34 |
*** jecarey has quit IRC | 21:35 | |
*** e0ne has joined #openstack-qa | 21:37 | |
openstackgerrit | Chad Roberts proposed openstack-dev/devstack: Replace data_processing with data-procesing https://review.openstack.org/135454 | 21:38 |
*** Ananci has quit IRC | 21:41 | |
*** david-lyle has quit IRC | 21:42 | |
*** david-lyle has joined #openstack-qa | 21:42 | |
*** ajo_ has joined #openstack-qa | 21:43 | |
*** leakypipes has quit IRC | 21:45 | |
*** melwitt has quit IRC | 21:46 | |
*** ajo has quit IRC | 21:46 | |
*** david-lyle has quit IRC | 21:47 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/154845 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 21:51 |
*** gfidente has quit IRC | 21:54 | |
*** e0ne has quit IRC | 21:54 | |
pc_m | marun | 21:55 |
pc_m | marun: ping | 21:55 |
openstackgerrit | Dean Troyer proposed openstack-dev/devstack: Fix is_keystone_enabled() https://review.openstack.org/155638 | 21:56 |
openstackgerrit | Dean Troyer proposed openstack-dev/devstack: Build a virtual environment for clients https://review.openstack.org/151513 | 21:56 |
*** Sukhdev has quit IRC | 21:59 | |
*** mriedem1 has quit IRC | 22:02 | |
*** mriedem has joined #openstack-qa | 22:02 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: remove heat from starting in grenade https://review.openstack.org/155876 | 22:12 |
*** achanda has quit IRC | 22:21 | |
*** achanda has joined #openstack-qa | 22:23 | |
*** mfalatic has quit IRC | 22:23 | |
*** harlowja_ is now known as harlowja_away | 22:24 | |
*** Sukhdev has joined #openstack-qa | 22:25 | |
*** harlowja_away is now known as harlowja_ | 22:28 | |
*** pc_m has quit IRC | 22:36 | |
*** mrda is now known as mrda-travelling | 22:41 | |
openstackgerrit | Joe Gordon proposed openstack-dev/devstack: Disable nova policy_dirs https://review.openstack.org/155910 | 22:45 |
*** pkoniszewski has quit IRC | 22:47 | |
openstackgerrit | Merged openstack/tempest: Change fixed/floating ip clients to return one value and update tests https://review.openstack.org/154537 | 22:47 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/155626 failed gate-tempest-dsvm-neutron-full in the gate queue with an unrecognized error | 22:53 |
*** patrickeast has quit IRC | 22:57 | |
*** jecarey has joined #openstack-qa | 22:58 | |
*** yginger5_away is now known as yjiang5 | 23:01 | |
*** abhirc has quit IRC | 23:01 | |
*** abhirc has joined #openstack-qa | 23:02 | |
*** mriedem1 has joined #openstack-qa | 23:07 | |
*** abhirc has quit IRC | 23:08 | |
*** mriedem has quit IRC | 23:09 | |
*** shashankhegde has quit IRC | 23:13 | |
*** shashankhegde has joined #openstack-qa | 23:17 | |
*** shardy has quit IRC | 23:26 | |
*** mriedem1 has quit IRC | 23:27 | |
*** shashankhegde has quit IRC | 23:27 | |
*** zigo has quit IRC | 23:27 | |
*** zigo has joined #openstack-qa | 23:28 | |
*** mriedem has joined #openstack-qa | 23:29 | |
*** mriedem has quit IRC | 23:34 | |
*** mriedem has joined #openstack-qa | 23:35 | |
*** shashankhegde has joined #openstack-qa | 23:36 | |
*** mriedem has quit IRC | 23:39 | |
*** mriedem has joined #openstack-qa | 23:41 | |
*** MasterPiece has quit IRC | 23:42 | |
*** superdan has quit IRC | 23:44 | |
*** dansmith has joined #openstack-qa | 23:45 | |
openstackgerrit | yunhong-jiang proposed openstack-dev/devstack: Make libvirt live migration transportation type configurable https://review.openstack.org/155087 | 23:48 |
*** melwitt has joined #openstack-qa | 23:53 | |
flashgordon | mtreinish: https://review.openstack.org/#/c/152298/6/zuul/layout.yaml,cm landed and is working | 23:55 |
flashgordon | this is going to be fun ! | 23:55 |
*** dimsum__ has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!