| *** achanda_ has joined #openstack-qa | 00:00 | |
| *** mahito has joined #openstack-qa | 00:01 | |
| openstackrecheck | openstack/requirements change: https://review.openstack.org/125651 failed gate-requirements-pypy in the gate queue with an unrecognized error | 00:02 |
|---|---|---|
| *** achanda has quit IRC | 00:04 | |
| *** achanda_ has quit IRC | 00:05 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Testing - please ignore https://review.openstack.org/181260 | 00:06 |
| *** hitalia has quit IRC | 00:14 | |
| *** jhenner has quit IRC | 00:22 | |
| *** salv-orlando has joined #openstack-qa | 00:26 | |
| *** jamielennox|away is now known as jamielennox | 00:29 | |
| *** salv-orlando has quit IRC | 00:30 | |
| *** tsekiyam_ has joined #openstack-qa | 00:31 | |
| *** dmorita has joined #openstack-qa | 00:31 | |
| *** tsekiyama has quit IRC | 00:34 | |
| *** mtanino has quit IRC | 00:36 | |
| *** tsekiyam_ has quit IRC | 00:36 | |
| *** erlon has quit IRC | 00:51 | |
| *** sputnik13 has joined #openstack-qa | 00:54 | |
| *** yamahata has joined #openstack-qa | 01:00 | |
| *** mahito has quit IRC | 01:00 | |
| *** mahito has joined #openstack-qa | 01:02 | |
| *** mahito_ has joined #openstack-qa | 01:05 | |
| *** mkoderer has quit IRC | 01:05 | |
| *** mpopow has joined #openstack-qa | 01:07 | |
| *** mahito has quit IRC | 01:08 | |
| *** mpopow has quit IRC | 01:12 | |
| *** mkoderer has joined #openstack-qa | 01:13 | |
| *** shashankhegde has quit IRC | 01:19 | |
| *** nikita09 has quit IRC | 01:28 | |
| *** aix has quit IRC | 01:31 | |
| openstackgerrit | Merged openstack-dev/devstack: create a more generic work around for cffi & wheels https://review.openstack.org/180549 | 01:44 |
| *** patrickeast has quit IRC | 01:45 | |
| *** fedexo has joined #openstack-qa | 01:51 | |
| openstackgerrit | Merged openstack/tempest-lib: Updated from global requirements https://review.openstack.org/181248 | 01:58 |
| *** dims_ has quit IRC | 02:00 | |
| *** dims has joined #openstack-qa | 02:00 | |
| *** dims has quit IRC | 02:01 | |
| *** david-lyle has joined #openstack-qa | 02:10 | |
| *** shashankhegde has joined #openstack-qa | 02:12 | |
| *** patrickeast has joined #openstack-qa | 02:19 | |
| *** fedexo has quit IRC | 02:21 | |
| *** achanda has joined #openstack-qa | 02:29 | |
| *** mpopow has joined #openstack-qa | 02:38 | |
| *** Poornima has joined #openstack-qa | 02:52 | |
| *** salv-orlando has joined #openstack-qa | 02:54 | |
| *** Swami_ has joined #openstack-qa | 02:57 | |
| *** amotoki has joined #openstack-qa | 02:58 | |
| *** Swami has quit IRC | 02:59 | |
| *** dims has joined #openstack-qa | 03:01 | |
| *** Swami__ has joined #openstack-qa | 03:02 | |
| *** salv-orl_ has joined #openstack-qa | 03:03 | |
| *** salv-orlando has quit IRC | 03:04 | |
| *** Swami_ has quit IRC | 03:05 | |
| *** dims has quit IRC | 03:06 | |
| *** salv-orl_ has quit IRC | 03:08 | |
| *** mpopow has quit IRC | 03:13 | |
| *** akshai has joined #openstack-qa | 03:14 | |
| *** salv-orlando has joined #openstack-qa | 03:18 | |
| *** akshai has quit IRC | 03:19 | |
| *** achanda has quit IRC | 03:22 | |
| *** salv-orlando has quit IRC | 03:22 | |
| *** marun has quit IRC | 03:37 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Add a rule for blocking "-" from rand_name call https://review.openstack.org/166679 | 03:37 |
| *** Swami__ has quit IRC | 03:45 | |
| *** shashankhegde has quit IRC | 03:49 | |
| *** Swami has joined #openstack-qa | 03:50 | |
| *** patrickeast has quit IRC | 03:50 | |
| openstackgerrit | Samuel de Medeiros Queiroz proposed openstack-dev/devstack: Honor the flag for Identity v3 API only jobs https://review.openstack.org/179663 | 03:52 |
| *** Swami has quit IRC | 03:55 | |
| openstackgerrit | Samuel de Medeiros Queiroz proposed openstack-dev/devstack: Honor the flag for Identity v3 API only jobs https://review.openstack.org/179663 | 04:08 |
| *** samueldmq has quit IRC | 04:11 | |
| *** pal has joined #openstack-qa | 04:11 | |
| *** mmedvede has quit IRC | 04:14 | |
| *** achanda has joined #openstack-qa | 04:22 | |
| *** rushiagr_away is now known as rushiagr | 04:27 | |
| *** achanda has quit IRC | 04:28 | |
| *** mmedvede has joined #openstack-qa | 04:28 | |
| *** achanda has joined #openstack-qa | 04:42 | |
| *** Sukhdev has joined #openstack-qa | 04:56 | |
| *** mpopow has joined #openstack-qa | 04:57 | |
| *** Longgeek has joined #openstack-qa | 05:00 | |
| *** Madan has joined #openstack-qa | 05:01 | |
| *** salv-orlando has joined #openstack-qa | 05:19 | |
| *** salv-orlando has quit IRC | 05:22 | |
| *** bkopilov has quit IRC | 05:27 | |
| *** Longgeek has quit IRC | 05:29 | |
| *** Sukhdev has quit IRC | 05:30 | |
| *** mpopow has quit IRC | 05:31 | |
| *** jyuso1 has quit IRC | 05:32 | |
| *** sabeen1 has quit IRC | 05:36 | |
| *** bkopilov has joined #openstack-qa | 05:44 | |
| *** pal has quit IRC | 05:48 | |
| *** pal has joined #openstack-qa | 05:48 | |
| *** kiran-r has joined #openstack-qa | 05:51 | |
| pal | Hi can anyone tell me how to solve this new issue coming while running stack.sh (ValueError: ('Missing distribution spec', '[Call')) | 05:51 |
| *** rossella_s has joined #openstack-qa | 05:52 | |
| *** ibiris_away is now known as ibiris | 05:56 | |
| *** BharatK has joined #openstack-qa | 05:57 | |
| *** rossella_s has quit IRC | 06:00 | |
| *** rossella_s has joined #openstack-qa | 06:01 | |
| *** Kennan has quit IRC | 06:03 | |
| *** Kennan has joined #openstack-qa | 06:03 | |
| *** pkoniszewski has joined #openstack-qa | 06:10 | |
| *** rossella_s has quit IRC | 06:11 | |
| *** salv-orlando has joined #openstack-qa | 06:13 | |
| *** Longgeek has joined #openstack-qa | 06:13 | |
| *** neeti has joined #openstack-qa | 06:23 | |
| *** jyuso1 has joined #openstack-qa | 06:30 | |
| *** anshul has joined #openstack-qa | 06:31 | |
| *** yog__ has joined #openstack-qa | 06:35 | |
| *** kiran-r has quit IRC | 06:37 | |
| *** achanda has quit IRC | 06:38 | |
| *** ibiris is now known as ibiris_away | 06:45 | |
| *** ibiris_away is now known as ibiris | 06:46 | |
| *** bkopilov has quit IRC | 06:46 | |
| *** anshul has quit IRC | 06:47 | |
| *** anshul has joined #openstack-qa | 06:47 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/181179 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1429151 | 07:00 |
| openstack | Launchpad bug 1429151 in Cinder "SimpleReadOnlyCinderClientTest.test_cinder_encryption_type_list fails randomly" [High,Confirmed] | 07:00 |
| *** armax has quit IRC | 07:02 | |
| *** k4n0 has joined #openstack-qa | 07:02 | |
| *** rushiagr is now known as rushiagr_away | 07:04 | |
| *** e0ne has joined #openstack-qa | 07:09 | |
| *** woodster_ has quit IRC | 07:10 | |
| *** hitalia has joined #openstack-qa | 07:12 | |
| *** pkoniszewski has quit IRC | 07:12 | |
| *** markus_z has joined #openstack-qa | 07:14 | |
| *** mpopow has joined #openstack-qa | 07:16 | |
| *** sks has joined #openstack-qa | 07:16 | |
| *** jyuso1 has quit IRC | 07:17 | |
| *** jhenner has joined #openstack-qa | 07:21 | |
| *** luqas has quit IRC | 07:25 | |
| *** mmedvede has quit IRC | 07:25 | |
| *** jyuso1 has joined #openstack-qa | 07:27 | |
| *** e0ne has quit IRC | 07:29 | |
| *** luqas has joined #openstack-qa | 07:32 | |
| *** e0ne has joined #openstack-qa | 07:33 | |
| *** mmedvede has joined #openstack-qa | 07:35 | |
| *** jamielennox is now known as jamielennox|away | 07:36 | |
| openstackgerrit | Yuki Nishiwaki proposed openstack-dev/devstack: Move install_infra before execute build_wheels.sh https://review.openstack.org/181312 | 07:36 |
| *** rushiagr_away is now known as rushiagr | 07:40 | |
| *** kiran-r has joined #openstack-qa | 07:41 | |
| *** jlanoux has joined #openstack-qa | 07:45 | |
| *** mpopow has quit IRC | 07:50 | |
| *** Poornima has quit IRC | 07:54 | |
| *** gfidente has joined #openstack-qa | 08:01 | |
| *** kiranr has joined #openstack-qa | 08:10 | |
| *** kiran-r has quit IRC | 08:10 | |
| *** sks has quit IRC | 08:21 | |
| *** e0ne has quit IRC | 08:23 | |
| *** rossella_s has joined #openstack-qa | 08:38 | |
| *** mahito_ has quit IRC | 08:41 | |
| *** hitalia has quit IRC | 08:48 | |
| *** tochi has quit IRC | 08:49 | |
| *** bkopilov has joined #openstack-qa | 08:51 | |
| *** cdent has joined #openstack-qa | 08:56 | |
| *** e0ne has joined #openstack-qa | 08:58 | |
| *** e0ne is now known as e0ne_ | 08:58 | |
| *** e0ne_ is now known as e0ne | 09:07 | |
| *** apevec has joined #openstack-qa | 09:10 | |
| *** SridharG has joined #openstack-qa | 09:19 | |
| *** e0ne is now known as e0ne_ | 09:21 | |
| kiranr | Is there a way to use admin token in tempest for authentication instead of keystone credentials?? | 09:25 |
| *** kiranr is now known as kiran-r | 09:26 | |
| kiran-r | Is there a way to use admin token in tempest for authentication instead of keystone credentials?? | 09:27 |
| *** mpopow has joined #openstack-qa | 09:34 | |
| *** devvesa has joined #openstack-qa | 09:45 | |
| devvesa | Hi, could a core reviewer to take a look at this patch? https://review.openstack.org/#/c/178132/ | 09:45 |
| *** hichihara has quit IRC | 09:48 | |
| *** ibiris is now known as ibiris_away | 09:51 | |
| *** e0ne_ is now known as e0ne | 09:55 | |
| *** coolsvap|afk is now known as coolsvap | 10:03 | |
| *** mpopow has quit IRC | 10:08 | |
| *** sslypushenko has quit IRC | 10:11 | |
| *** woodster_ has joined #openstack-qa | 10:11 | |
| *** sslypush1nko has joined #openstack-qa | 10:16 | |
| *** sslypush1nko is now known as slypushenko_ | 10:17 | |
| *** pc_m has joined #openstack-qa | 10:20 | |
| *** yfried__ is now known as yfried|afk | 10:22 | |
| *** yfried|afk is now known as yfried__ | 10:26 | |
| *** ibiris_away is now known as ibiris | 10:27 | |
| openstackgerrit | Sean Dague proposed openstack-dev/grenade: further updates to plugins documentation https://review.openstack.org/181358 | 10:45 |
| *** Madan has quit IRC | 10:48 | |
| *** samueldmq has joined #openstack-qa | 10:49 | |
| samueldmq | morning | 10:49 |
| *** weshay has joined #openstack-qa | 10:55 | |
| *** e0ne is now known as e0ne_ | 10:56 | |
| *** yfried__ is now known as yfried|afk | 10:56 | |
| *** dmorita has quit IRC | 10:57 | |
| openstackgerrit | Merged openstack-dev/grenade: Add some more words to PLUGINS.rst https://review.openstack.org/179484 | 11:04 |
| *** ihrachyshka has joined #openstack-qa | 11:05 | |
| *** mwagner_lap has joined #openstack-qa | 11:06 | |
| *** rfolco has joined #openstack-qa | 11:09 | |
| *** ihrachyshka has quit IRC | 11:11 | |
| *** dims has joined #openstack-qa | 11:13 | |
| *** e0ne_ is now known as e0ne | 11:14 | |
| *** jose_lausuch has joined #openstack-qa | 11:16 | |
| *** ihrachyshka has joined #openstack-qa | 11:16 | |
| *** yfried|afk is now known as yfried__ | 11:17 | |
| *** pal has quit IRC | 11:19 | |
| *** jose_lausuch has quit IRC | 11:20 | |
| *** jose_lausuch has joined #openstack-qa | 11:21 | |
| *** yfried__ is now known as yfried|afk | 11:27 | |
| *** dims has quit IRC | 11:28 | |
| *** malini has joined #openstack-qa | 11:28 | |
| *** e0ne is now known as e0ne_ | 11:28 | |
| *** dims has joined #openstack-qa | 11:29 | |
| *** e0ne_ is now known as e0ne | 11:30 | |
| openstackrecheck | openstack/sahara change: https://review.openstack.org/180606 failed because of: gate-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/1283522 | 11:35 |
| openstack | Launchpad bug 1283522 in neutron "DB lock timeout errors with parallel operations" [Critical,Fix released] - Assigned to Kevin Benton (kevinbenton) | 11:35 |
| *** sputnik13 has quit IRC | 11:36 | |
| *** malini has quit IRC | 11:47 | |
| *** malini has joined #openstack-qa | 11:48 | |
| *** tkammer has joined #openstack-qa | 11:52 | |
| *** malini has quit IRC | 11:52 | |
| *** mpopow has joined #openstack-qa | 11:52 | |
| sc68cal | mriedem_away: kevinbenton just pushed a commit to fix https://bugs.launchpad.net/neutron/+bug/1452903 | 11:53 |
| openstack | Launchpad bug 1452903 in neutron "KeyError in ovs_neutron_agent._bind_devices" [Undecided,Fix committed] - Assigned to Kevin Benton (kevinbenton) | 11:53 |
| *** jose_lausuch has quit IRC | 11:53 | |
| *** malini has joined #openstack-qa | 11:53 | |
| *** artom has joined #openstack-qa | 11:54 | |
| *** jose_lausuch has joined #openstack-qa | 11:55 | |
| openstackgerrit | Rohan Kanade proposed openstack/tempest: Adds Database Instance/Actions API Tests https://review.openstack.org/180489 | 11:58 |
| *** amotoki has quit IRC | 11:58 | |
| *** jose_lausuch has quit IRC | 12:00 | |
| *** jose_lausuch has joined #openstack-qa | 12:01 | |
| *** kiranr has joined #openstack-qa | 12:02 | |
| *** kiran-r has quit IRC | 12:02 | |
| *** yfried|afk is now known as yfried__ | 12:03 | |
| *** malini has quit IRC | 12:04 | |
| *** k4n0 has quit IRC | 12:04 | |
| openstackgerrit | Flavio Percoco proposed openstack-dev/devstack: Remove Zaqar from devstack https://review.openstack.org/178955 | 12:05 |
| *** jose_lausuch has quit IRC | 12:06 | |
| *** jose_lausuch has joined #openstack-qa | 12:07 | |
| *** e0ne is now known as e0ne_ | 12:10 | |
| *** Yogi has joined #openstack-qa | 12:13 | |
| *** e0ne_ is now known as e0ne | 12:13 | |
| *** nithyag_ has joined #openstack-qa | 12:16 | |
| *** kiranr is now known as kiran-r | 12:16 | |
| *** NithyaG has quit IRC | 12:18 | |
| *** Yogi has quit IRC | 12:18 | |
| *** mtanino has joined #openstack-qa | 12:20 | |
| *** jose_lausuch has quit IRC | 12:25 | |
| *** tkammer has quit IRC | 12:26 | |
| *** jose_lausuch has joined #openstack-qa | 12:26 | |
| *** mpopow has quit IRC | 12:26 | |
| *** Daviey has joined #openstack-qa | 12:27 | |
| *** ihrachyshka has quit IRC | 12:32 | |
| *** dpaterson has quit IRC | 12:33 | |
| *** rushiagr is now known as rushiagr_away | 12:37 | |
| *** aysyd has joined #openstack-qa | 12:40 | |
| *** jose_lausuch has quit IRC | 12:40 | |
| *** jose_lausuch has joined #openstack-qa | 12:40 | |
| *** dkranz has joined #openstack-qa | 12:40 | |
| *** hop has joined #openstack-qa | 12:42 | |
| *** bknudson has quit IRC | 12:43 | |
| openstackgerrit | Merged openstack-dev/hacking: Updated from global requirements https://review.openstack.org/181196 | 12:44 |
| *** marrusl has quit IRC | 12:48 | |
| *** marrusl has joined #openstack-qa | 12:48 | |
| *** malini has joined #openstack-qa | 12:53 | |
| *** malini has quit IRC | 12:53 | |
| *** malini has joined #openstack-qa | 12:53 | |
| *** e0ne is now known as e0ne_ | 12:56 | |
| *** krtaylor has quit IRC | 12:56 | |
| openstackgerrit | Dmitry Ratushnyy proposed openstack/tempest: Implementation of Scenario tests for port-security ML2 extension https://review.openstack.org/178137 | 12:56 |
| *** e0ne_ is now known as e0ne | 12:57 | |
| hop | hi, can I ask there about tempest ? | 12:59 |
| *** jose_lausuch has quit IRC | 13:00 | |
| *** jose_lausuch has joined #openstack-qa | 13:00 | |
| *** dkranz has quit IRC | 13:01 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: [WIP] Try logging exceptions https://review.openstack.org/181379 | 13:03 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: [WIP] Try logging exceptions https://review.openstack.org/181379 | 13:06 |
| *** krtaylor has joined #openstack-qa | 13:08 | |
| *** rushiagr_away is now known as rushiagr | 13:09 | |
| *** bknudson has joined #openstack-qa | 13:11 | |
| openstackrecheck | openstack/sahara change: https://review.openstack.org/179026 failed gate-tempest-dsvm-neutron-full in the gate queue with an unrecognized error | 13:11 |
| *** cdent has quit IRC | 13:12 | |
| dims | mtreinish: see ^^ (possible fix for missing exceptions in tempest.log) | 13:15 |
| *** EmilienM|afk is now known as EmilienM | 13:15 | |
| hop | in tempest tests there are decorator @test.idempotent_id ? why ? how does it generate ? | 13:16 |
| openstackgerrit | Vladyslav Drok proposed openstack-dev/devstack: Add new options to baremetal config section https://review.openstack.org/178606 | 13:17 |
| openstackgerrit | Vladyslav Drok proposed openstack/tempest: Add scenario for testing ironic without glance https://review.openstack.org/178607 | 13:19 |
| *** dims has quit IRC | 13:19 | |
| *** dims has joined #openstack-qa | 13:20 | |
| *** jose_lausuch has quit IRC | 13:21 | |
| sdague | hop: it is created manually for new tests | 13:22 |
| *** jose_lausuch has joined #openstack-qa | 13:22 | |
| *** yfried__ is now known as yfried|afk | 13:22 | |
| sdague | it exists to give stable ids to tests for frameworks like defcore, so that tempest can change test names internally, but they can still reference the "same" tests over time | 13:22 |
| *** jose_lausuch has quit IRC | 13:22 | |
| *** jose_lausuch has joined #openstack-qa | 13:23 | |
| hop | oh, thanks! | 13:24 |
| *** yfried|afk is now known as yfried__ | 13:25 | |
| *** dpaterson has joined #openstack-qa | 13:25 | |
| *** jose_lausuch has quit IRC | 13:26 | |
| *** jose_lausuch has joined #openstack-qa | 13:27 | |
| *** malini has left #openstack-qa | 13:31 | |
| *** sabeen1 has joined #openstack-qa | 13:32 | |
| *** dansmith is now known as superdan | 13:35 | |
| *** coolsvap is now known as coolsvap|afk | 13:37 | |
| *** mriedem_away is now known as mriedem | 13:37 | |
| *** neeti has quit IRC | 13:39 | |
| mriedem | kevinbenton: thanks for https://review.openstack.org/#/c/181185/ | 13:40 |
| *** mtanino has quit IRC | 13:42 | |
| *** erlon has joined #openstack-qa | 13:44 | |
| mtreinish | dims: cool, thanks. Here's hoping it catches a failure :) | 13:44 |
| openstackgerrit | Jens Rosenboom proposed openstack-dev/devstack: Update default cirros version https://review.openstack.org/166778 | 13:44 |
| *** neeti has joined #openstack-qa | 13:44 | |
| mtreinish | oh, we probably could jsut check experimental it, at least one of those will fail probably | 13:44 |
| dims | mtreinish: one time we want tempest to fail and it wont! | 13:44 |
| *** BharatK has quit IRC | 13:44 | |
| mtreinish | hehe | 13:45 |
| *** jose_lausuch has quit IRC | 13:46 | |
| *** cdent has joined #openstack-qa | 13:46 | |
| *** markus_z has quit IRC | 13:47 | |
| *** jose_lausuch has joined #openstack-qa | 13:47 | |
| *** jose_lausuch has left #openstack-qa | 13:47 | |
| *** coolsvap has joined #openstack-qa | 13:47 | |
| openstackgerrit | yuxingw proposed openstack/tempest-lib: Password is not strong enough failures https://review.openstack.org/178546 | 13:50 |
| *** openstackgerrit has quit IRC | 13:51 | |
| *** openstackgerrit has joined #openstack-qa | 13:51 | |
| *** dims is now known as dimsum__ | 13:51 | |
| *** samueldmq has quit IRC | 13:57 | |
| *** rbak has joined #openstack-qa | 13:58 | |
| openstackgerrit | Matt Riedemann proposed openstack-dev/devstack: nova: remove allow_migrate_to_same_host config usage https://review.openstack.org/181395 | 14:01 |
| *** BharatK has joined #openstack-qa | 14:01 | |
| *** yfried__ is now known as yfried|afk | 14:03 | |
| *** BharatK has quit IRC | 14:07 | |
| mtreinish | andreaf: if you get a sec want to +A this: https://review.openstack.org/#/c/178757/ | 14:08 |
| andreaf | mtreinish: are we actually losing coverage until folks implement tests in the python clients for heat/cinder | 14:09 |
| andreaf | and image | 14:09 |
| andreaf | ? | 14:09 |
| mtreinish | andreaf: http://lists.openstack.org/pipermail/openstack-dev/2015-April/063018.html | 14:10 |
| *** mpopow has joined #openstack-qa | 14:10 | |
| *** pglass has joined #openstack-qa | 14:10 | |
| mtreinish | all the functional tests are in progress, heat has patches up now in response to the thread | 14:10 |
| andreaf | mtreinish: yeah I saw your message, but I guess you don't want to wait for the work to be done on client side | 14:11 |
| andreaf | mtreinish: fair enough we waited for long already | 14:11 |
| mtreinish | andreaf: yeah, I've lost patience | 14:12 |
| andreaf | well the patch looks good to me, and I'm happy to see the CLI test go as well | 14:13 |
| *** bnemec has joined #openstack-qa | 14:13 | |
| sdague | superdan: thanks for the grenade doc update | 14:14 |
| andreaf | it seems to be difficult to honour the not all from the same company policy :D | 14:14 |
| sdague | I stuck another update on top of your patch as well that hopefully dumps out a lot of the rest of the model from my head to paper | 14:14 |
| andreaf | andreaf: but I'm sure no one would argue this was instrumental for HP in any way | 14:14 |
| *** dkranz has joined #openstack-qa | 14:14 | |
| andreaf | mtreinish: ^^^ | 14:14 |
| mtreinish | andreaf: heh, I'll leave it up to you, if you'd rather not +A it I'll bug dkranz and mkoderer | 14:14 |
| andreaf | writing to myself | 14:15 |
| andreaf | I did +A already :) | 14:15 |
| superdan | sdague: np, I was actually expecting feedback on my questions and that I'd revise | 14:15 |
| superdan | sdague: but cool, I'll go look at the follow-on in a bit | 14:15 |
| sdague | so, honestly, with docs I'm a fan of "is this better, commit it" | 14:15 |
| sdague | because there are always more issues to fix | 14:15 |
| superdan | sure | 14:16 |
| *** kiran-r has quit IRC | 14:16 | |
| *** kiran-r has joined #openstack-qa | 14:17 | |
| mtreinish | dimsum__: too bad, looks like your test patch passed everything, I ran experimental on it, something should raise an exception there | 14:18 |
| dimsum__ | mtreinish: cool | 14:18 |
| *** BharatK has joined #openstack-qa | 14:19 | |
| sdague | mtreinish / dtroyer - https://review.openstack.org/#/c/181374/ is the backport of the better cffi work around | 14:20 |
| sdague | because we needed it in stable/kilo as well | 14:20 |
| mtreinish | sdague: +2 | 14:21 |
| mtreinish | I'll trade you for: https://review.openstack.org/181048 | 14:21 |
| mtreinish | which stops installing tempest system wide on kilo | 14:21 |
| mtreinish | so we venv isolate it on the stable branches | 14:21 |
| *** kiran-r has quit IRC | 14:22 | |
| *** neeti has quit IRC | 14:25 | |
| *** armax has joined #openstack-qa | 14:26 | |
| *** SridharG1 has joined #openstack-qa | 14:28 | |
| *** BadCub_Hiding is now known as BadCub | 14:29 | |
| *** SridharG has quit IRC | 14:31 | |
| *** bnemec is now known as beekneemech | 14:32 | |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: Move install_infra before execute build_wheels.sh https://review.openstack.org/181409 | 14:32 |
| sdague | mtreinish: that needs +2 as well | 14:32 |
| mtreinish | sdague: done | 14:33 |
| *** anshul has quit IRC | 14:33 | |
| *** tsekiyama has joined #openstack-qa | 14:37 | |
| *** mpopow has quit IRC | 14:45 | |
| *** bkopilov has quit IRC | 14:46 | |
| *** artom has quit IRC | 14:48 | |
| *** hemnafk is now known as hemna | 14:50 | |
| openstackgerrit | Sean Dague proposed openstack-dev/grenade: wip: make grenade smash resources harder https://review.openstack.org/181420 | 14:54 |
| *** devvesa has quit IRC | 14:54 | |
| dkranz | andreaf: Can you please approve https://review.openstack.org/#/c/179401/ ? It keeps getting reported as a bug. | 14:55 |
| *** ajmiller has joined #openstack-qa | 14:55 | |
| *** e0ne is now known as e0ne_ | 14:55 | |
| openstackgerrit | Marian Horban proposed openstack/tempest: API test for instance's migration added https://review.openstack.org/153005 | 14:56 |
| *** yfried|afk is now known as yfried__ | 14:56 | |
| andreaf | dkranz: done - I was just waiting to see the non-isolated job run :) | 14:56 |
| dkranz | andreaf: thanks! | 14:57 |
| dkranz | andreaf: once https://review.openstack.org/#/c/178757/ merges, non-admin should run clean | 14:58 |
| andreaf | dkranz: great! :) | 14:58 |
| *** mpopow has joined #openstack-qa | 15:01 | |
| openstackgerrit | Vladyslav Drok proposed openstack/tempest: Add scenario for testing ironic without glance https://review.openstack.org/178607 | 15:01 |
| *** mtanino has joined #openstack-qa | 15:03 | |
| *** hop has quit IRC | 15:06 | |
| *** e0ne_ is now known as e0ne | 15:07 | |
| *** yog__ has quit IRC | 15:07 | |
| sdague | superdan: so https://review.openstack.org/181420 is something that will be curious to see if we can grind on enough to get the fail consistently | 15:08 |
| *** ajmiller has quit IRC | 15:13 | |
| *** ajmiller has joined #openstack-qa | 15:13 | |
| superdan | sdague: you want one secgroup per server? | 15:14 |
| *** yfried__ is now known as yfried|afk | 15:15 | |
| sdague | superdan: I think so | 15:15 |
| *** mpopow has quit IRC | 15:15 | |
| sdague | do you think the races will expose more or less often with more security groups? | 15:15 |
| superdan | sdague: I dunno, was just trying to think | 15:15 |
| superdan | sdague: so you're trying to create a lot more servers to make it more likely that we hit this? | 15:16 |
| sdague | I guess I thought it would be more likely to trip it | 15:16 |
| sdague | superdan: yes | 15:16 |
| superdan | I dunno, I think it's probably the same, | 15:16 |
| *** mpopow has joined #openstack-qa | 15:16 | |
| superdan | if we don't create a security group, we still have to save/restore iptables to add the server into the group by adding a rule to jump into that chain for the server's IP, IIRC | 15:16 |
| superdan | I'm not sure if one group or multiple groups is more interesting, or if it's a wash, TBH | 15:16 |
| sdague | ok | 15:17 |
| superdan | if it matters, that seems like relevant information, but I'm not sure which to even guess at | 15:17 |
| superdan | so I guess we do this and if it doesn't increase it, then we switch to them all in a single one? | 15:17 |
| openstackgerrit | Sean Dague proposed openstack-dev/grenade: wip: make grenade smash resources harder https://review.openstack.org/181420 | 15:17 |
| sdague | so, actually, it occurred to me the race was probably going to be slowed down by an added ping check | 15:18 |
| sdague | superdan: so, I think we recheck on this a bit | 15:18 |
| sdague | and see what happens | 15:18 |
| superdan | sdague: maybe, depends on what's racing really | 15:18 |
| sdague | yep | 15:18 |
| superdan | if it's just 1/N times we screw up then it doesn't matter | 15:18 |
| sdague | true | 15:19 |
| superdan | should we ping them all after we create though? | 15:19 |
| superdan | we need to know they're all working before we shut down right? | 15:19 |
| sdague | so, it occurred to me we didn't ping the non volume servers on create in the existing code | 15:19 |
| superdan | ah, hmm | 15:19 |
| sdague | which means it doesn't wait for the iptables rules to be working after the floating ip add | 15:19 |
| superdan | hmm | 15:20 |
| sdague | so I think that might be an important concurency slip | 15:20 |
| sdague | we do verify them all before shutdown, but it's another loop | 15:20 |
| superdan | oh, okay | 15:20 |
| superdan | so, | 15:20 |
| sdague | https://github.com/openstack-dev/grenade/blob/master/projects/60_nova/resources.sh#L93 | 15:20 |
| sdague | that's the existing code | 15:20 |
| superdan | thinking about this one or many secgroup thing.. | 15:21 |
| superdan | when we add an instance to a secgroup, | 15:21 |
| superdan | we make the change directly in the db | 15:21 |
| superdan | and then we fire off a *cast* to the compute node that does the refresh | 15:21 |
| superdan | so we return immediately, but that actual add happens later | 15:22 |
| superdan | and potentially overlapping with other things like other boots or other floating-add operations | 15:22 |
| sdague | ah, yeh, so that seems like it's more likely to trigger the race then | 15:22 |
| superdan | well, no, | 15:22 |
| superdan | because adding a secgroup itself is just a db operation | 15:22 |
| superdan | adding a secgroup doesn't cause any iptables activity | 15:22 |
| sdague | oh, ok | 15:22 |
| superdan | it's only the adding-an-instance operation that does | 15:22 |
| sdague | ok, so it's a wash | 15:23 |
| superdan | well, | 15:23 |
| superdan | maybe, | 15:23 |
| superdan | but not adding a new secgroup each time means that we have a little less latency between the creates | 15:23 |
| sdague | ok | 15:24 |
| superdan | not creating the secgroup doesn't add to the iptables load, so it's just slowing us down | 15:24 |
| sdague | well, lets see how the runs play out | 15:24 |
| superdan | but again, we don't really know what is happening, so I'm not sure it matters | 15:24 |
| sdague | and we can try that approach as well | 15:24 |
| sdague | at some later point | 15:24 |
| superdan | we might need both data points either way, is what I'm trying to say | 15:24 |
| sdague | ok | 15:24 |
| sdague | well, right now I need food, then my afternoon is some errands, so we'll see what this does and I'll poke some more later. | 15:25 |
| superdan | oh, but we're never calling this refresh, we're doing this as boot | 15:25 |
| superdan | okay, so the plan is to just recheck the everlovin crap out of this, yes? | 15:26 |
| openstackgerrit | Marian Horban proposed openstack/tempest: Instance ID was corrected https://review.openstack.org/177277 | 15:27 |
| *** jhenner has quit IRC | 15:27 | |
| *** sks has joined #openstack-qa | 15:29 | |
| openstackgerrit | Merged openstack/tempest: Remove CLI testing once and for all https://review.openstack.org/178757 | 15:30 |
| *** yfried|afk is now known as yfried__ | 15:30 | |
| *** mwagner_lap has quit IRC | 15:35 | |
| mtreinish | ^^^ \o/ | 15:36 |
| *** yfried__ is now known as yfried|afk | 15:41 | |
| *** jaypipes is now known as leakypipes | 15:42 | |
| openstackgerrit | Merged openstack-dev/devstack: Don't install tempest on stable kilo https://review.openstack.org/181048 | 15:44 |
| openstackgerrit | Merged openstack-dev/devstack: Move install_infra before execute build_wheels.sh https://review.openstack.org/181312 | 15:45 |
| *** sks has quit IRC | 15:46 | |
| sdague | superdan: yes, and see if we can expose it | 15:47 |
| *** shashankhegde has joined #openstack-qa | 15:48 | |
| sdague | gah, screwed up a variable name before the post | 15:49 |
| openstackgerrit | Sean Dague proposed openstack-dev/grenade: wip: make grenade smash resources harder https://review.openstack.org/181420 | 15:50 |
| superdan | sdague: if you don't stop touching it, we'll never get a full run.. come on! :) | 15:50 |
| sdague | there were failed results already | 15:50 |
| superdan | oh, heh :) | 15:50 |
| sdague | but it was because it had a missing variable | 15:50 |
| sdague | http://logs.openstack.org/20/181420/2/check/check-grenade-dsvm/0205b91//logs/grenade.sh.txt.gz#_2015-05-08_15_43_34_629 | 15:51 |
| superdan | hmm, I didn't see an email from jenkins | 15:51 |
| *** mlavalle has joined #openstack-qa | 15:51 | |
| sdague | the neutron job was still running | 15:51 |
| superdan | ah | 15:51 |
| sdague | I looked in zuul | 15:51 |
| *** SridharG1 has left #openstack-qa | 15:54 | |
| openstackgerrit | Merged openstack/tempest: Remove unnecessary force_tenant_isolation https://review.openstack.org/179401 | 15:56 |
| *** openstackstatus has quit IRC | 15:56 | |
| *** dkranz has quit IRC | 15:56 | |
| *** patrickeast has joined #openstack-qa | 15:57 | |
| *** openstackstatus has joined #openstack-qa | 15:57 | |
| *** ChanServ sets mode: +v openstackstatus | 15:57 | |
| *** jlanoux has quit IRC | 16:03 | |
| openstackgerrit | Merged openstack-dev/devstack: create a more generic work around for cffi & wheels https://review.openstack.org/181374 | 16:04 |
| *** e0ne is now known as e0ne_ | 16:04 | |
| *** mlavalle has quit IRC | 16:08 | |
| *** Yogi has joined #openstack-qa | 16:08 | |
| *** mlavalle has joined #openstack-qa | 16:09 | |
| *** daniel3_ has joined #openstack-qa | 16:09 | |
| *** yfried|afk has quit IRC | 16:10 | |
| *** shashankhegde has quit IRC | 16:11 | |
| *** Nic has joined #openstack-qa | 16:12 | |
| *** armax has quit IRC | 16:14 | |
| *** kiran-r has joined #openstack-qa | 16:15 | |
| openstackgerrit | Marian Horban proposed openstack/tempest: API test for instance's migration added https://review.openstack.org/153005 | 16:26 |
| *** dboik has joined #openstack-qa | 16:27 | |
| *** dboik_ has quit IRC | 16:27 | |
| *** dboik_ has joined #openstack-qa | 16:30 | |
| *** dboik has quit IRC | 16:33 | |
| *** Swami has joined #openstack-qa | 16:34 | |
| *** Sukhdev has joined #openstack-qa | 16:34 | |
| *** marrusl has quit IRC | 16:37 | |
| *** dustins has joined #openstack-qa | 16:38 | |
| *** ajmiller has quit IRC | 16:38 | |
| *** ajmiller has joined #openstack-qa | 16:39 | |
| *** dboik_ has quit IRC | 16:39 | |
| *** dboik has joined #openstack-qa | 16:40 | |
| *** yamahata has quit IRC | 16:41 | |
| andreaf | mtreinish: I just noticed something weird with tempest glance client - all the methods use URL which include the version e.g. '/v2/images/%s?...', and do not specify an api_version at class level | 16:41 |
| andreaf | mtreinish: the consequence of this is that if the endpoint is versioned in the catalogue you end-up having /v2/v2/... which is broken | 16:42 |
| andreaf | mtreinish: using the api_version at class level should instead support both versioned and un-versioned endpoint | 16:43 |
| andreaf | mtreinish: would you be ok with changing this in principle? | 16:43 |
| mtreinish | andreaf: sure, that's probably fine but I blame your catalog :) | 16:44 |
| andreaf | mtreinish: eh, right | 16:45 |
| andreaf | I'll try to get my catalogue changed, but I think it also makes sense for tempest to be flexible on this | 16:46 |
| mtreinish | andreaf: heh, it's clearly a bug, when I added the glance v2 support I probably didn't even think about it | 16:46 |
| *** Nic has quit IRC | 16:46 | |
| *** Nic has joined #openstack-qa | 16:46 | |
| mtreinish | I was just kidding, your catalog is probably fine, until we start enforcing how projects go in it I don't think that's really an issue | 16:47 |
| *** armax has joined #openstack-qa | 16:49 | |
| *** achanda has joined #openstack-qa | 16:50 | |
| *** Yogi has quit IRC | 16:50 | |
| *** rushiagr is now known as rushiagr_away | 16:53 | |
| *** jogo is now known as fashgordon | 16:55 | |
| *** mwagner_lap has joined #openstack-qa | 16:55 | |
| *** sputnik13 has joined #openstack-qa | 16:56 | |
| *** fashgordon is now known as flashgordon | 16:56 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Tempest image clients use version in URL https://review.openstack.org/181462 | 16:58 |
| andreaf | mtreinish: well I know the keystone folks are promoting un-versioned endpoints | 16:59 |
| andreaf | mtreinish: but I'm not sure where we are in term of enforcement | 16:59 |
| *** luqas has quit IRC | 17:03 | |
| openstackgerrit | Accela Zhao proposed openstack-dev/devstack: Use an actual existing nova scheduler in README.md https://review.openstack.org/181465 | 17:04 |
| *** yamahata has joined #openstack-qa | 17:06 | |
| *** bkopilov has joined #openstack-qa | 17:06 | |
| *** dkranz has joined #openstack-qa | 17:09 | |
| *** Yogi has joined #openstack-qa | 17:13 | |
| openstackgerrit | Accela Zhao proposed openstack-dev/devstack: Fix wrong `sudo ceph -c` command in lib/ceph https://review.openstack.org/181469 | 17:19 |
| *** marrusl has joined #openstack-qa | 17:21 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/tempest: [WIP] Try logging exceptions https://review.openstack.org/181379 | 17:27 |
| *** hitalia has joined #openstack-qa | 17:29 | |
| *** vponomaryov has quit IRC | 17:29 | |
| *** niita09 has joined #openstack-qa | 17:29 | |
| *** artom has joined #openstack-qa | 17:30 | |
| *** niita09 has left #openstack-qa | 17:30 | |
| *** e0ne_ is now known as e0ne | 17:32 | |
| *** anshul has joined #openstack-qa | 17:33 | |
| *** e0ne has quit IRC | 17:34 | |
| *** shashankhegde has joined #openstack-qa | 17:36 | |
| mtreinish | andreaf: there was a cross project spec just pushed up to start that process | 17:38 |
| *** bkopilov has quit IRC | 17:39 | |
| *** Sukhdev has quit IRC | 17:40 | |
| mtreinish | andreaf: https://review.openstack.org/#/c/181393/ | 17:40 |
| *** Sukhdev has joined #openstack-qa | 17:41 | |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Remove common/ssh.py due to migration to tempest_lib https://review.openstack.org/176039 | 17:42 |
| *** anshul has quit IRC | 17:42 | |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Remove common/ssh.py due to migration to tempest_lib https://review.openstack.org/176039 | 17:43 |
| mtreinish | andreaf, sdague, dkranz: ^^^ That should be good to go now | 17:44 |
| dkranz | mtreinish: I'll +2 when it passes | 17:45 |
| mtreinish | dkranz: cool, thanks | 17:46 |
| openstackgerrit | Jeremy Stanley proposed openstack-dev/devstack: Stop installing nose and pylint from distros https://review.openstack.org/181478 | 17:47 |
| openstackgerrit | Jeremy Stanley proposed openstack-dev/devstack: Stop installing nose and pylint from distros https://review.openstack.org/181479 | 17:50 |
| dkranz | mtreinish: I was hearing more complaints about insufficiency of our cleanup for use in real clouds and looked into what rally is proposing https://review.openstack.org/#/c/172967/ Seems like we should share a way to handle this. What do you think of the basic idea there? | 17:51 |
| mtreinish | dkranz: we already have a patch for tempest up doing something like that | 17:52 |
| mtreinish | the one I linked to yesteday | 17:52 |
| mtreinish | we identified that doing a consistent name prefix in conjunction with the other cleanup tooling was probably needed | 17:52 |
| dkranz | mtreinish: ok, I didn't get a chance to look at it. But since tempest is used by rally it would be really silly to have a different mechanism | 17:52 |
| dkranz | mtreinish: nice to agree on naming across all projects for this purpose | 17:53 |
| dkranz | mtreinish: I"ll look at that now. | 17:53 |
| mtreinish | dkranz: well, it's in tempest-lib: http://git.openstack.org/cgit/openstack/tempest-lib/tree/tempest_lib/common/utils/data_utils.py#n40 | 17:53 |
| dkranz | mtreinish: what you pointed to | 17:53 |
| dkranz | mtreinish: and comment in the rally spec | 17:53 |
| mtreinish | that patch from flwang needs some more work before it can land, but it's the basic idea | 17:54 |
| dkranz | mtreinish: ok | 17:55 |
| *** anshul has joined #openstack-qa | 17:56 | |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Update query for bug 1403291 https://review.openstack.org/181150 | 17:58 |
| openstack | bug 1403291 in neutron "VM's fail to receive DHCPOFFER messages" [Critical,Fix released] https://launchpad.net/bugs/1403291 - Assigned to Kevin Benton (kevinbenton) | 17:58 |
| openstackgerrit | Jeremy Stanley proposed openstack-dev/devstack: Stop installing nose and pylint from distros https://review.openstack.org/181485 | 18:00 |
| openstackgerrit | Jeremy Stanley proposed openstack-dev/devstack: Stop installing nose and pylint from distros https://review.openstack.org/181488 | 18:05 |
| *** gfidente has quit IRC | 18:13 | |
| *** coolsvap has quit IRC | 18:15 | |
| *** kiran-r has quit IRC | 18:17 | |
| *** rwsu_ has quit IRC | 18:24 | |
| *** openstackrecheck has quit IRC | 18:25 | |
| *** openstackrecheck has joined #openstack-qa | 18:25 | |
| mtreinish | dkranz: what do you mean by your review on that prefix patch? Are you just looking for a common prefix name between tempest and rally? | 18:31 |
| dkranz | dkranz: definitely that, and there may also be other cool tools that create "temp" resources | 18:31 |
| *** daniel3_ has quit IRC | 18:32 | |
| dkranz | mtreinish: The other issue is that the tempest prefix does not let the admin clean up only a single tempest run | 18:33 |
| dkranz | mtreinish: the rally spec also calls for a task-specific prefix so you can do that | 18:33 |
| openstackgerrit | Merged openstack-dev/devstack: Move install_infra before execute build_wheels.sh https://review.openstack.org/181409 | 18:34 |
| dkranz | mtreinish: otherwise, you can't support multiple uses of rally/tempest/whatever at the same time | 18:34 |
| dkranz | mtreinish: I think the requirmeents in that spec are pretty reasonable and similar to what I was going to propose | 18:34 |
| openstackgerrit | Merged openstack-dev/devstack: Preload updated cirros image https://review.openstack.org/176782 | 18:35 |
| *** apevec has quit IRC | 18:36 | |
| mtreinish | dkranz: hmm, well we don't have a data store for a tempest run, it's not something that's really in scope of tempest so having an incremental run id as part of the prefix isn't something we'll likely be able to do | 18:36 |
| mtreinish | I'm assuming rally will leverage their db for doing that task_id in the prefix bit | 18:37 |
| *** Longgeek has quit IRC | 18:37 | |
| dkranz | mtreinish: yes. | 18:37 |
| *** Longgeek has joined #openstack-qa | 18:37 | |
| dkranz | mtreinish: one idea I had was that if you are using locking accounts, a prefix could be in there | 18:37 |
| mtreinish | dkranz: if you're using locking accounts I'm not sure it really matters | 18:38 |
| mtreinish | because the tenant list is pre defined | 18:38 |
| mtreinish | so you already know what's being created because you look after the run | 18:38 |
| *** Longgeek has quit IRC | 18:39 | |
| *** marun has joined #openstack-qa | 18:39 | |
| dkranz | mtreinish: I need to think about this | 18:39 |
| *** Longgeek has joined #openstack-qa | 18:39 | |
| mtreinish | you also can't do the multiple runs at the same time with test accounts because it would violate the isolation and cause issues | 18:39 |
| dkranz | mtreinish: I didn't mean the same user doing it. | 18:40 |
| dkranz | mtreinish: if you have multiple test account files they would be separate | 18:40 |
| mtreinish | oh, ok | 18:40 |
| mtreinish | well, in that case it's still already isolated to the tenants in the multiple files so you know what was created | 18:41 |
| mtreinish | it actually would be very easy to leverage the existing cleanup tool with test acounts | 18:41 |
| mtreinish | we just need to teach it how to parse the accounts.yaml file | 18:41 |
| dkranz | mtreinish: yes, there was a similar comment in the rally spec | 18:41 |
| dkranz | mtreinish: "normal" cleanup is much easier | 18:41 |
| dkranz | mtreinish: it is the "admin" cleanup that is tricky. Like when you tempest machine died and disappeared while tempest was running | 18:42 |
| *** armax has quit IRC | 18:43 | |
| openstackgerrit | Merged openstack-dev/devstack: Update default cirros version https://review.openstack.org/166778 | 18:43 |
| dkranz | mtreinish: I think this is worth discussing in Vancouver | 18:43 |
| mtreinish | dkranz: I'm not sure I understand there being a distinction. The person who is running tempest will want to do the cleanup right? | 18:44 |
| dkranz | mtreinish: yes, but what if they don't, or their machine dies, or whatever? Then an admin has to clean it up. | 18:44 |
| mtreinish | dkranz: that seems like a really weird edge case to care about, and how does it differ from a normal cloud user having an issue? | 18:45 |
| dkranz | mtreinish: it is the case where you need to cleanup but do not have the tenant id that contains the stranded resources | 18:45 |
| dkranz | mtreinish: maybe it is an edge case. For the normal user you always have the tenant id | 18:46 |
| mordred | mtreinish: how do I make a test in tempest real quick to test image-create? | 18:46 |
| mordred | mtreinish: apparently the image-list test relies on iamge-create | 18:46 |
| mordred | but there is not an explicit test for image-create | 18:46 |
| mordred | so defcore is wanting to toss out image-list | 18:46 |
| mordred | mtreinish: this has made me want to rage-code an image-create test | 18:47 |
| mtreinish | mordred: the stuff in: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/image/v2/test_images.py#n75 doesn't cover what you need? | 18:47 |
| mtreinish | or do you mean a glanceclient test for it? | 18:48 |
| dkranz | mordred: test_create_delete_image | 18:48 |
| mtreinish | mordred: wait what is the defcore objection to the test in question? I'm not sure I understand | 18:49 |
| *** armax has joined #openstack-qa | 18:49 | |
| mordred | mtreinish: they're concerned that list_images relies on upload-image working | 18:50 |
| hogepodge | mtreinish: I believe is that it uses a glance-v1 api for nova functionality | 18:50 |
| hogepodge | they're making claims on what could be valid technical problems, but there are underlying issues about canceling a bunch of tests wholesale. | 18:51 |
| mtreinish | mordred: so we need an isolated list test that doesnt create a resource to check it's presence in the list? | 18:51 |
| hogepodge | if a large number of tests are broken, I'd rather see the process stop so we can fix the tests rather than ignore them and move the process forward. | 18:51 |
| mtreinish | hogepodge, mordred: can I get a link to the test in question? I need a better reference point | 18:52 |
| mtreinish | mordred: but I'll gladly help you rage code any test you need :) | 18:52 |
| hogepodge | mtreinish: look through this, https://review.openstack.org/#/q/project:openstack/defcore+message:flag,n,z | 18:55 |
| hogepodge | some have merged, some haven't | 18:55 |
| mtreinish | hogepodge: so that first 1 with the mordred -1 is the rackspace craziness which should not be flagged because the tasks stuff isn't even upstream | 18:56 |
| mtreinish | hogepodge: so I would require tempest bugs be filed for any of the things making a flag request based on an issue with tempest | 18:58 |
| mordred | mtreinish: apparenty some in defcore disagree with your test design | 19:11 |
| mtreinish | mordred: heh, well then they should have contributed tests to do it the way they wanted. I could never get anyone from glance to write a tempest test... | 19:12 |
| *** salv-orlando has quit IRC | 19:12 | |
| mordred | it's ... VanL thinks that the create-image should be in the fixture | 19:12 |
| mordred | and that if the fixture fails, the test should be skipped | 19:13 |
| clarkb | wat | 19:13 |
| mordred | I just about belly laughed | 19:13 |
| clarkb | "if test fails skip it" | 19:13 |
| *** ajmiller has quit IRC | 19:13 | |
| *** Swami has quit IRC | 19:13 | |
| *** ajmiller has joined #openstack-qa | 19:13 | |
| mtreinish | mordred: oh, yeah that's just crazy | 19:13 |
| mtreinish | mordred: but also there are tests which just do an image create, those won't pass for them either | 19:15 |
| mtreinish | are they just not being run on that list? | 19:15 |
| mordred | mtreinish: I did not find them? | 19:16 |
| *** eglute has joined #openstack-qa | 19:16 | |
| mtreinish | mordred: that link before didn't count? it creates an image then deletes it | 19:17 |
| mtreinish | the test right below that creates an image, then uploads it too | 19:17 |
| mtreinish | mordred: I mean if you think we're missing something we can add whatever | 19:18 |
| *** VanL has joined #openstack-qa | 19:18 | |
| lifeless | wat | 19:19 |
| mordred | hi VanL ! | 19:20 |
| mordred | lifeless, clarkb, mtreinish: VanL and I were just discussing test structure and skipping tests when fixtures fail to run | 19:21 |
| mtreinish | mordred: oh, is it just that none of those create tests are in the defcore list? | 19:21 |
| mordred | mtreinish: possibly? I'll go ask in #defcore | 19:21 |
| mtreinish | I only see one here: http://git.openstack.org/cgit/openstack/defcore/tree/2015.04/2015.04.required.txt | 19:21 |
| mtreinish | L112 | 19:22 |
| lifeless | mordred: I must be missing something, but fixtures are code reuse, skipping is environment detection / policy | 19:22 |
| lifeless | mordred: I'm not sure how they are coupled (but its easy to code if one wants that to happen) | 19:22 |
| mtreinish | lifeless: no, you're not missing anything. That's how it works they shouldn't be coupled at all | 19:22 |
| mtreinish | if they were it would be an important bug to fix | 19:23 |
| *** daniel3_ has joined #openstack-qa | 19:25 | |
| *** mwagner_lap has quit IRC | 19:26 | |
| *** dustins has quit IRC | 19:33 | |
| *** salv-orlando has joined #openstack-qa | 19:38 | |
| *** Sukhdev has quit IRC | 19:41 | |
| *** Swami has joined #openstack-qa | 19:46 | |
| mtreinish | dkranz: if you get a sec those python3 support patches I have up for review should be a quick review | 19:46 |
| dkranz | mtreinish: ok | 19:47 |
| mtreinish | dkranz: awesome, thanks | 19:47 |
| mtreinish | dkranz: fwiw, that branch gets us to like all but 8 unit tests passing on python 3 (after the ssh migration patch works its way through the gate) | 19:48 |
| dkranz | mtreinish: cool | 19:48 |
| *** bkopilov has joined #openstack-qa | 19:49 | |
| mtreinish | I plan to sort out the last few after the current branch lands, and once it's passing I'll switch the py34 experimental job to voting | 19:49 |
| mtreinish | I'm not sure it means we'll be able to do a full run under python3 yet, I haven't had a chance to test that | 19:49 |
| dkranz | mtreinish: I started looking at one earlier where str(x) was replaced by a six.bunch-of-stuff in a whole bunch of places. | 19:49 |
| dkranz | mtreinish: the unicode thing. Maybe we should abstract these things | 19:50 |
| mtreinish | dkranz: sigh, yeah that's what most of the patches are more or less | 19:50 |
| mtreinish | dkranz: well that's what six provides us is that abstraction | 19:50 |
| dkranz | mtreinish: but it is really gross | 19:50 |
| mtreinish | or did you mean wrap it another layer up? | 19:50 |
| dkranz | mtreinish: if possible. grossness like that should only be in one place | 19:50 |
| dkranz | mtreinish: because people writing new code will waste a lot of time getting it wrong I think | 19:50 |
| dkranz | mtreinish: especially those of us not so up on py3 | 19:51 |
| mtreinish | well, that's why we'll make the unit test job gating. But, I'm not sure how much we'd be able to do there because it's differences in builtins to python | 19:51 |
| mtreinish | I agree it isn't pretty | 19:52 |
| mtreinish | I'm just not sure how to make it better | 19:52 |
| *** VanL has quit IRC | 19:52 | |
| dkranz | mtreinish: yeah, I understand. I wish those that created py3 just stuck to making really bad things better | 19:52 |
| dkranz | mtreinish: rather than their image of perfection regardless of cost to those trying to upgrade | 19:53 |
| dkranz | mtreinish: but so it goes | 19:53 |
| mtreinish | sigh, yeah | 19:53 |
| dkranz | mtreinish: actually the whole nova api v2/v3 thing reminded me of that a little, but with a better resolution for our users | 19:53 |
| dkranz | mtreinish: like us saying it was no big deal to deal with the syntax changes for v3 when we are still on py2 six years later... | 19:54 |
| mtreinish | yeah there is a lot of similarity there, heh even the numbers | 19:54 |
| mtreinish | although tbh, I was never quite clear on how we got to nova v2 in the first place. I never remember seeing a nova v1 | 19:56 |
| mtreinish | but that's just probably from before I started working on things | 19:56 |
| openstackgerrit | Merged openstack/tempest: Test live migrate on a paused instance https://review.openstack.org/157054 | 20:00 |
| mtreinish | mriedem, gilliard_afk: ^^^ | 20:01 |
| mriedem | muwahahaha | 20:02 |
| mtreinish | did we just merge a rootkit or something? | 20:02 |
| openstackgerrit | Merged openstack/tempest: Remove common/ssh.py due to migration to tempest_lib https://review.openstack.org/176039 | 20:07 |
| openstackgerrit | Merged openstack-dev/devstack: Stop installing nose and pylint from distros https://review.openstack.org/181485 | 20:07 |
| openstackgerrit | Merged openstack-dev/devstack: Stop installing nose and pylint from distros https://review.openstack.org/181479 | 20:10 |
| *** Sukhdev has joined #openstack-qa | 20:11 | |
| dtroyer | mtreinish: can you give this a final kick? https://review.openstack.org/181488 I wasn't paying attention and merged the stable/juno one already… | 20:15 |
| mtreinish | dtroyer: heh, order matters :) | 20:16 |
| mtreinish | dtroyer: done | 20:16 |
| dtroyer | oh? they didn't merge in order | 20:17 |
| dtroyer | what did I screw up? | 20:17 |
| mtreinish | dtroyer: oh that was a question? I didn't think the merge order actually matters | 20:18 |
| mtreinish | ^^^ there shouldn't be a question mark there | 20:18 |
| dhellmann | I'm seeing errors with pypy dying because of a missing metadata file in a pip dist. I don't see anything in elastic recheck, has anyone else reported that? http://logs.openstack.org/76/179476/3/check/gate-oslo.middleware-pypy/9800dd6/console.html | 20:23 |
| mtreinish | dhellmann: I don't think I've seen that before | 20:24 |
| mtreinish | clarkb, fungi: ^^^ that seems like something you'd have noticed before me | 20:24 |
| dhellmann | mtreinish: I found https://bugs.launchpad.net/openstack-ci/+bug/1315468 but that's an old one | 20:24 |
| openstack | Launchpad bug 1315468 in OpenStack Core Infrastructure "*-pypy Failure: "No such file or directory: '/home/jenkins/workspace/gate-requirements-pypy/.tox/pypy/site-packages/setuptools-2.2.dist-info/METADATA"" [Undecided,Fix released] | 20:24 |
| mtreinish | dhellmann: could it just be a requirement pushed a bad release with pypy? | 20:25 |
| dhellmann | mtreinish: possibly? the missing file seems to be coming out of pip somehow? I'm trying to reproduce it locally | 20:25 |
| * dhellmann is tempted to just remove the pypy job | 20:26 | |
| dhellmann | it passes for me locally, but that pip version looks suspicious: 1.5.4 | 20:27 |
| fungi | mtreinish: doesn't ring a bell | 20:27 |
| fungi | dhellmann: looks like the version of virtualenv in use has pip 1.5.4... | 20:29 |
| dhellmann | fungi: yeah, that jumped out at me, too | 20:29 |
| dhellmann | fungi: it says tox 1.9.2 earlier in the log | 20:30 |
| *** e0ne has joined #openstack-qa | 20:33 | |
| *** mriedem is now known as mriderman | 20:35 | |
| *** Yogi has quit IRC | 20:41 | |
| openstackgerrit | Sean Dague proposed openstack-dev/grenade: wip: make grenade smash resources harder https://review.openstack.org/181420 | 20:41 |
| openstackgerrit | Sean Dague proposed openstack-dev/grenade: wip: version of smash without extra secgroups https://review.openstack.org/181548 | 20:41 |
| *** armax has quit IRC | 20:48 | |
| *** VanL has joined #openstack-qa | 20:53 | |
| superdan | sdague: I suck and haven't been watching, any luck? | 20:53 |
| *** mriderman has quit IRC | 20:54 | |
| superdan | ah, I see | 20:56 |
| sdague | no, the first one died on a programming error I made when I tried to put the cinder thing in a loop | 20:58 |
| *** VanL has quit IRC | 20:58 | |
| sdague | so we didn't get far enough for data | 20:58 |
| sdague | I made a second patch which doesn't isolate security groups for the nova resources | 20:58 |
| sdague | so they can both be rechecked | 20:58 |
| superdan | yeah, I saw cool | 20:59 |
| *** mriedem has joined #openstack-qa | 20:59 | |
| *** e0ne has quit IRC | 21:02 | |
| openstackrecheck | openstack/ceilometer change: https://review.openstack.org/154530 failed gate-ceilometer-python27 in the gate queue with an unrecognized error | 21:03 |
| sdague | I'll probably look at them during the mornings this weekend just to see if there is anything handy out of them | 21:03 |
| sdague | you should also see if this all seems to make sense - https://review.openstack.org/#/c/181358/1/PLUGINS.rst,cm | 21:03 |
| openstackgerrit | Merged openstack-dev/devstack: Stop installing nose and pylint from distros https://review.openstack.org/181478 | 21:06 |
| openstackgerrit | Merged openstack-dev/devstack: Stop installing nose and pylint from distros https://review.openstack.org/181488 | 21:06 |
| *** Longgeek has quit IRC | 21:10 | |
| *** mriedem has quit IRC | 21:11 | |
| *** mriedem has joined #openstack-qa | 21:12 | |
| *** dpaterson has quit IRC | 21:13 | |
| *** artom has quit IRC | 21:17 | |
| superdan | sdague: if I -1 your patch and then fix the things I'm complaining about, will you not hold it against me? | 21:21 |
| superdan | the grenade doc patch that is | 21:21 |
| superdan | well, I think one thing is confusing, so I -1d, but if you agree, I'm happy to fix it up for you | 21:22 |
| *** cbader has quit IRC | 21:25 | |
| *** pc_m has quit IRC | 21:26 | |
| dkranz | mtreinish: I just noticed that verify_tempest_config leaks a tenant because it uses credentials.get_isolated_credentials('verify_tempest_config') but never clears. | 21:27 |
| mtreinish | dkranz: oh, heh nice catch | 21:27 |
| dkranz | mtreinish: does this need a call to clear or is there another way to do it | 21:28 |
| dkranz | mtreinish: I think we want this to behave as if allow_tenant_isolation were false | 21:29 |
| mtreinish | dkranz: yeah you just do isolated_creds.clear_isolated_creds() | 21:29 |
| mtreinish | it'll tear down everything it creates (or allocates in the test accounts case) | 21:29 |
| *** rwsu has joined #openstack-qa | 21:30 | |
| dkranz | mtreinish: the alternative is to add another argument to get_isloated_credentials so it never creates a new tenant | 21:30 |
| *** rwsu has quit IRC | 21:30 | |
| *** rwsu has joined #openstack-qa | 21:31 | |
| dkranz | mtreinish: do you have a preference? | 21:31 |
| mtreinish | well why did we add the isolated creds usage there? | 21:31 |
| mtreinish | I don't remember when we added it | 21:31 |
| *** armax has joined #openstack-qa | 21:31 | |
| dkranz | mtreinish: so it could run abstractly and without admin | 21:31 |
| dkranz | mtreinish: it was fairly recent | 21:31 |
| dkranz | mtreinish: as part of getting non-admion working | 21:32 |
| dkranz | mtreinish: that is the generic way to get a client | 21:32 |
| dkranz | mtreinish: but in this case we are not inside of the tearDownClass paradigm | 21:32 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/173680 failed gate-tempest-dsvm-neutron-full in the gate queue with an unrecognized error | 21:32 |
| *** rwsu has quit IRC | 21:39 | |
| *** erlon has quit IRC | 21:41 | |
| mtreinish | dkranz: yeah we probably will still need to use isolated_creds, because there is no guarantee on how the user is being provided. It could be via accounts | 21:41 |
| mtreinish | instead of a config option | 21:41 |
| mtreinish | for the tenant isolation case though it shouldn't matter because we can use admin creds right? | 21:42 |
| dkranz | mtreinish: yeah, I am pushing a fix now that calls clear | 21:42 |
| dkranz | mtreinish: yeah, it all works but leaks a tenant in that case | 21:42 |
| mtreinish | oh, I meant we don't have to use tenant isolation, we can just use the admin user from config for the verify options if tenant isolation is configured | 21:44 |
| mtreinish | but just using the isolated creds is easier | 21:44 |
| openstackgerrit | David Kranz proposed openstack/tempest: Fix verify_tempest_config to not leak a tenant https://review.openstack.org/181568 | 21:44 |
| dkranz | mtreinish: right, that's why I did that. It was recommended as "the way" to get a client generically | 21:45 |
| dkranz | mtreinish: before I start reviewing the py3 stuff, what do you think about the comment that such ugliness should be in one place? | 21:46 |
| mtreinish | dkranz: well tbh I think we'd just end up trading six.something() for tempest.something() and I'm not sure that's worthwhile | 21:47 |
| mtreinish | or am I missing what you mean? | 21:47 |
| dkranz | mtreinish: replace six.text_type(account).encode('utf-8') with tempest.xxx(account) | 21:48 |
| dkranz | mtreinish: its the boilerplate crap | 21:49 |
| *** aysyd has quit IRC | 21:49 | |
| *** mriedem has quit IRC | 21:49 | |
| dkranz | mtreinish: at least that was the idea | 21:49 |
| *** rfolco has quit IRC | 21:50 | |
| dkranz | mtreinish: I agree it is pretty gross regardless, but if we put these "idioms" in one place people could look for them | 21:51 |
| *** VanL has joined #openstack-qa | 21:51 | |
| mtreinish | dkranz: I mean I'm not opposed to that (I wonder if there's an oslo thing for it already) but I'd prefer to do it as a follow on patch to clean it up | 21:51 |
| dkranz | mtreinish: ok | 21:51 |
| mtreinish | dimsum__, dhellmann: ^^^ | 21:52 |
| *** rwsu has joined #openstack-qa | 21:58 | |
| *** pglass has quit IRC | 22:01 | |
| *** daniel3_ has quit IRC | 22:02 | |
| *** EmilienM is now known as EmilienM|afk | 22:08 | |
| *** Sukhdev has quit IRC | 22:11 | |
| *** VanL has quit IRC | 22:17 | |
| *** beekneemech has quit IRC | 22:18 | |
| *** dimsum__ has quit IRC | 22:21 | |
| *** bknudson has quit IRC | 22:21 | |
| *** cdent has quit IRC | 22:23 | |
| *** Nic has quit IRC | 22:39 | |
| *** rwsu has quit IRC | 22:56 | |
| *** dboik has quit IRC | 23:03 | |
| *** dboik has joined #openstack-qa | 23:04 | |
| *** mlavalle has quit IRC | 23:06 | |
| morganfainberg | mtreinish: going to push a review here shortly that makes devstack do all setup w/ v3 keystone instead of the weird mix. (Wind change heat or other configs assuming v2 explicitly, but all setup will be v3) | 23:11 |
| morganfainberg | mtreinish: specifically around using osc | 23:12 |
| mtreinish | morganfainberg: there's already a patch up that did that I thought | 23:12 |
| mtreinish | one sec, let me check | 23:12 |
| morganfainberg | Oh is there? | 23:12 |
| morganfainberg | I thought we hadn't done that bit yet. Someone might have beaten me to it. | 23:13 |
| mtreinish | morganfainberg: https://review.openstack.org/#/c/179663/ | 23:13 |
| morganfainberg | ;) | 23:13 |
| mtreinish | it's been on my review list for the past week, I haven't had a chance to look at it yet though | 23:13 |
| morganfainberg | mtreinish: I think that is the wrong approach fwiw. | 23:13 |
| morganfainberg | mtreinish: we should stop doing. Setup with v2 period. No conditionals. | 23:14 |
| morganfainberg | V2 can be enabled and used by services. But devstack shouldn't do anything with keystone v2 for tenant/project user etc creation. | 23:14 |
| mtreinish | morganfainberg: ok, sure like I said I haven't looked at it yet, I was just aware of it | 23:14 |
| mtreinish | morganfainberg: sure that makes sense | 23:14 |
| morganfainberg | I'll look that over and -1 and/or push a change to make it not do conditional setup. | 23:15 |
| *** patrickeast has quit IRC | 23:15 | |
| *** mpopow has quit IRC | 23:15 | |
| morganfainberg | I think that is actually 2 changes: 1 don't use v2, 2: honor v3 only for things like heat. | 23:15 |
| morganfainberg | mtreinish: I'll see what k can | 23:15 |
| morganfainberg | Do and ping you when I'm done poking at it. | 23:15 |
| *** mpopow has joined #openstack-qa | 23:16 | |
| mtreinish | morganfainberg: ok, cool | 23:16 |
| * mtreinish assumes k == i :) | 23:16 | |
| morganfainberg | Yes | 23:20 |
| *** rwsu has joined #openstack-qa | 23:22 | |
| mtreinish | morganfainberg: heh, what an interesting world it would be if k and i were interchangable letters :) | 23:24 |
| morganfainberg | Hahahah. | 23:24 |
| morganfainberg | Mobile phones like to swap I and k on me for some reason. :P | 23:24 |
| *** marun has quit IRC | 23:32 | |
| *** mestery has quit IRC | 23:34 | |
| *** heyongli has quit IRC | 23:36 | |
| *** heyongli has joined #openstack-qa | 23:37 | |
| *** hemna is now known as hemnafk | 23:46 | |
| *** ajmiller has quit IRC | 23:48 | |
| *** anshul has quit IRC | 23:50 | |
| *** tsekiyama has quit IRC | 23:51 | |
| *** rwsu has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!