| *** achanda has quit IRC | 00:00 | |
| *** achanda has joined #openstack-qa | 00:02 | |
| *** yamahata has joined #openstack-qa | 00:09 | |
| *** weshay has quit IRC | 00:19 | |
| *** gfidente^2nd has joined #openstack-qa | 00:20 | |
| *** gfidente has quit IRC | 00:22 | |
| *** mriedem1 has joined #openstack-qa | 00:23 | |
| *** mriedem has quit IRC | 00:24 | |
| *** gfidente^2nd has quit IRC | 00:25 | |
| *** gfidente has joined #openstack-qa | 00:25 | |
| *** dimsum__ has joined #openstack-qa | 00:35 | |
| *** dims_ has joined #openstack-qa | 00:36 | |
| *** dimsum__ has quit IRC | 00:39 | |
| *** oomichi has joined #openstack-qa | 00:41 | |
| *** bkopilov has quit IRC | 00:48 | |
| *** bkopilov has joined #openstack-qa | 00:51 | |
| *** achanda has quit IRC | 00:52 | |
| *** woodster_ has quit IRC | 01:10 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/148137 failed because of: gate-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/1344513 | 01:14 |
|---|---|---|
| openstack | Launchpad bug 1344513 in Cinder "test_cinder_encryption_type_list fails" [Undecided,New] | 01:14 |
| *** bkopilov has quit IRC | 01:23 | |
| *** bkopilov has joined #openstack-qa | 01:24 | |
| openstackgerrit | Jamie Lennox proposed openstack-dev/devstack: Isolate creating service users https://review.openstack.org/150672 | 01:25 |
| *** zhiyan has quit IRC | 01:40 | |
| *** jecarey has joined #openstack-qa | 01:45 | |
| *** zhiyan has joined #openstack-qa | 01:47 | |
| *** woodster_ has joined #openstack-qa | 01:49 | |
| *** wuhg has joined #openstack-qa | 01:49 | |
| *** annashen has joined #openstack-qa | 01:50 | |
| *** achanda has joined #openstack-qa | 01:52 | |
| *** Yogi1 has joined #openstack-qa | 01:53 | |
| *** Yogi11 has joined #openstack-qa | 01:55 | |
| *** Yogi1 has quit IRC | 01:57 | |
| *** achanda has quit IRC | 01:57 | |
| openstackgerrit | Masayuki Igawa proposed openstack/tempest: Change tempest Conflict exc to tempest-lib exc https://review.openstack.org/148440 | 02:09 |
| openstackgerrit | Masayuki Igawa proposed openstack/tempest: Change tempest OverLimit exc to tempest-lib exc https://review.openstack.org/148439 | 02:09 |
| openstackgerrit | Masayuki Igawa proposed openstack/tempest: Change tempest Unauthorized exc to tempest-lib exc https://review.openstack.org/148417 | 02:09 |
| openstackgerrit | Masayuki Igawa proposed openstack/tempest: Change tempest NotFound exc to tempest-lib exc https://review.openstack.org/148481 | 02:09 |
| *** yamamot__ has joined #openstack-qa | 02:11 | |
| *** annashen has quit IRC | 02:27 | |
| *** mwagner_lap has joined #openstack-qa | 02:28 | |
| *** gfidente has quit IRC | 02:40 | |
| *** kan_ has joined #openstack-qa | 03:08 | |
| *** Longgeek has joined #openstack-qa | 03:11 | |
| *** Longgeek has quit IRC | 03:14 | |
| *** Longgeek has joined #openstack-qa | 03:20 | |
| *** shashankhegde has joined #openstack-qa | 03:22 | |
| *** shashankhegde has quit IRC | 03:23 | |
| *** pcrews has quit IRC | 03:30 | |
| openstackgerrit | Jamie Lennox proposed openstack-dev/devstack: Isolate creating service users https://review.openstack.org/150672 | 03:36 |
| *** mriedem1 has quit IRC | 03:40 | |
| *** dimtruck is now known as zz_dimtruck | 03:40 | |
| *** pal has joined #openstack-qa | 03:41 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack-dev/devstack: plugins.rst: Fix a typo https://review.openstack.org/153915 | 03:43 |
| *** Madan has joined #openstack-qa | 03:50 | |
| *** annashen has joined #openstack-qa | 03:51 | |
| *** rushiagr_away is now known as rushiagr | 03:53 | |
| *** dttocs has quit IRC | 04:00 | |
| *** coolsvap_ is now known as coolsvap | 04:02 | |
| *** Yogi11 has quit IRC | 04:03 | |
| *** sabeen has quit IRC | 04:03 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Separate token client from identity client https://review.openstack.org/151530 | 04:11 |
| *** amotoki has joined #openstack-qa | 04:13 | |
| *** jhenner has quit IRC | 04:22 | |
| *** dims_ has quit IRC | 04:23 | |
| *** rushiagr is now known as rushiagr_away | 04:24 | |
| *** annashen has quit IRC | 04:31 | |
| *** rushiagr_away is now known as rushiagr | 04:31 | |
| *** zz_dimtruck is now known as dimtruck | 04:31 | |
| *** achanda has joined #openstack-qa | 04:39 | |
| *** dimtruck is now known as zz_dimtruck | 04:43 | |
| *** zz_dimtruck is now known as dimtruck | 04:43 | |
| *** nikil89_ has joined #openstack-qa | 04:46 | |
| *** nikil89_ is now known as naggappan | 04:46 | |
| *** yfried|afk is now known as yfried | 04:49 | |
| *** coolsvap is now known as coolsvap_ | 04:55 | |
| *** coolsvap_ is now known as coolsvap | 04:56 | |
| *** SridharG has joined #openstack-qa | 04:57 | |
| *** k4n0 has joined #openstack-qa | 04:59 | |
| *** raghav_mulugu has joined #openstack-qa | 05:12 | |
| *** annashen has joined #openstack-qa | 05:13 | |
| *** k4n0 has quit IRC | 05:23 | |
| *** dimsum__ has joined #openstack-qa | 05:24 | |
| *** k4n0 has joined #openstack-qa | 05:24 | |
| *** dimsum__ has quit IRC | 05:29 | |
| naggappan | Hi is there any existing scripts to convert the nosetests output into html fromate. If we use tox it can be done. But is there any way in nosetes? | 05:34 |
| *** k4n0 has quit IRC | 05:36 | |
| *** yfried has quit IRC | 05:38 | |
| *** achanda has quit IRC | 05:38 | |
| *** Murad_ has quit IRC | 05:48 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Remove CONF values from identity clients https://review.openstack.org/151540 | 05:49 |
| *** k4n0 has joined #openstack-qa | 05:50 | |
| *** BharatK has joined #openstack-qa | 05:50 | |
| openstackgerrit | Merged openstack/tempest: Add os-tenant-networks test https://review.openstack.org/146761 | 05:51 |
| openstackgerrit | Merged openstack/tempest: Add os-tenant-networks test https://review.openstack.org/146761 | 05:51 |
| *** boris-42 has quit IRC | 05:52 | |
| *** SridharG has left #openstack-qa | 05:55 | |
| *** achanda has joined #openstack-qa | 05:57 | |
| *** annashen has quit IRC | 06:06 | |
| *** shashankhegde has joined #openstack-qa | 06:07 | |
| *** shashankhegde has quit IRC | 06:11 | |
| *** k4n0 has quit IRC | 06:13 | |
| *** chandankumar has joined #openstack-qa | 06:13 | |
| *** k4n0 has joined #openstack-qa | 06:14 | |
| *** gamado has joined #openstack-qa | 06:16 | |
| *** anshul has joined #openstack-qa | 06:18 | |
| *** anshul has joined #openstack-qa | 06:19 | |
| *** achanda has quit IRC | 06:22 | |
| *** boris-42 has joined #openstack-qa | 06:26 | |
| *** pcaruana has quit IRC | 06:28 | |
| *** k4n0 has quit IRC | 06:31 | |
| *** k4n0 has joined #openstack-qa | 06:34 | |
| *** SridharG has joined #openstack-qa | 06:46 | |
| *** yfried has joined #openstack-qa | 06:49 | |
| *** itzikb has joined #openstack-qa | 06:50 | |
| *** yfried_ has joined #openstack-qa | 06:53 | |
| *** yfried has quit IRC | 06:53 | |
| *** vigneshvar has joined #openstack-qa | 07:02 | |
| *** gszasz has joined #openstack-qa | 07:10 | |
| *** neeti has joined #openstack-qa | 07:12 | |
| *** achanda has joined #openstack-qa | 07:15 | |
| *** afazekas has joined #openstack-qa | 07:20 | |
| *** nmagnezi has joined #openstack-qa | 07:46 | |
| *** achanda has quit IRC | 07:47 | |
| *** achanda has joined #openstack-qa | 07:51 | |
| *** SridharG1 has joined #openstack-qa | 07:53 | |
| *** SridharG has quit IRC | 07:54 | |
| *** e0ne has joined #openstack-qa | 07:55 | |
| *** k4n0 has quit IRC | 07:57 | |
| *** gszasz has quit IRC | 07:58 | |
| *** k4n0 has joined #openstack-qa | 08:00 | |
| *** gszasz has joined #openstack-qa | 08:00 | |
| *** e0ne has quit IRC | 08:03 | |
| *** gamado has quit IRC | 08:07 | |
| *** ajo has quit IRC | 08:10 | |
| *** woodster_ has quit IRC | 08:10 | |
| *** ajo has joined #openstack-qa | 08:10 | |
| *** pal has quit IRC | 08:11 | |
| *** gamado has joined #openstack-qa | 08:11 | |
| *** luqas has joined #openstack-qa | 08:11 | |
| *** neeti has quit IRC | 08:13 | |
| *** naggappan has quit IRC | 08:16 | |
| *** shardy has joined #openstack-qa | 08:17 | |
| *** jlibosva has joined #openstack-qa | 08:18 | |
| *** neeti has joined #openstack-qa | 08:25 | |
| openstackgerrit | Rohan Kanade proposed openstack/tempest: Split resource_setup for network tests https://review.openstack.org/150312 | 08:25 |
| *** jhenner has joined #openstack-qa | 08:32 | |
| *** safchain has joined #openstack-qa | 08:37 | |
| *** aix has joined #openstack-qa | 08:40 | |
| *** andreykurilin_ has joined #openstack-qa | 08:40 | |
| *** jlibosva has quit IRC | 08:41 | |
| *** oomichi has quit IRC | 08:42 | |
| *** jlibosva has joined #openstack-qa | 08:43 | |
| gmann | k4n0: ping | 08:43 |
| k4n0 | gmann, pong | 08:43 |
| gmann | k4n0: regarding resource cleanup patches | 08:43 |
| k4n0 | gmann, Yes, go on | 08:44 |
| gmann | k4n0: actually there are some issue for credential and net resource creation - https://review.openstack.org/#/c/141424/ | 08:44 |
| gmann | k4n0: i discussed that with andreaf_ and he is working on base patch to make those cleanly. | 08:45 |
| *** xavileon has joined #openstack-qa | 08:45 | |
| k4n0 | gmann, Yes, i have seen that patch, I am only fixing minor issues in my patches, will rebase all of them once andreaf_ gets his patch in | 08:45 |
| gmann | k4n0: may be you need to wait for those base patch | 08:46 |
| *** jlanoux has joined #openstack-qa | 08:46 | |
| k4n0 | gmann, ok, I will only fix some review comments related to the split and then wait | 08:46 |
| gmann | k4n0: cool. thx :) | 08:46 |
| k4n0 | gmann, thanks :) | 08:46 |
| openstackgerrit | Yuiko Takada proposed openstack/tempest: Add os-baremetal-nodes tests https://review.openstack.org/149125 | 08:47 |
| *** markus_z has joined #openstack-qa | 08:47 | |
| *** afazekas has quit IRC | 08:49 | |
| *** achanda has quit IRC | 08:52 | |
| *** mpaolino has joined #openstack-qa | 08:53 | |
| *** jordanP has joined #openstack-qa | 08:54 | |
| *** achanda has joined #openstack-qa | 08:56 | |
| openstackgerrit | Merged openstack-dev/devstack: plugins.rst: Fix a typo https://review.openstack.org/153915 | 08:56 |
| *** nikil89_ has joined #openstack-qa | 08:58 | |
| *** nikil89_ is now known as naggappan | 08:59 | |
| openstackgerrit | Yuiko Takada proposed openstack/tempest: Add os-baremetal-nodes tests https://review.openstack.org/149125 | 08:59 |
| *** jpich has joined #openstack-qa | 09:02 | |
| *** vigneshvar has quit IRC | 09:05 | |
| openstackgerrit | Thomas Bechtold proposed openstack/tempest: Test for id_username instead of "Log In" https://review.openstack.org/153954 | 09:05 |
| *** ylobankov has quit IRC | 09:07 | |
| *** achanda has quit IRC | 09:08 | |
| *** jlibosva has quit IRC | 09:09 | |
| *** jlibosva has joined #openstack-qa | 09:10 | |
| *** ylobankov has joined #openstack-qa | 09:13 | |
| *** vigneshvar has joined #openstack-qa | 09:17 | |
| *** SridharG has joined #openstack-qa | 09:17 | |
| openstackgerrit | Michele Paolino proposed openstack-dev/devstack: Support for libvirt/QEMU tar releases https://review.openstack.org/108714 | 09:18 |
| jordanP | ianw, hey. About https://review.openstack.org/#/c/152082/ (Create a `policy.d` dir in $PROJECT_CONF_DIR) I am not sure to undertand your suggestion | 09:18 |
| *** achanda has joined #openstack-qa | 09:18 | |
| jordanP | the single function thing | 09:18 |
| ianw | jordanP: just have a "create_policy_dir" or something, so that someone reads the comment on that function and realises calls to it can disappear at some point in the future | 09:19 |
| ianw | jordanP: i guess it doesn't matter that much, it's not a lot of code. just it gets checked in and never ever leaves | 09:20 |
| *** SridharG1 has quit IRC | 09:20 | |
| jordanP | ianw, okay, so I put a function "create_policy_dir" in lib/nova, lib/cinder, etc... ? | 09:21 |
| *** k4n0 has quit IRC | 09:21 | |
| ianw | jordanP: no just one in functions-common and call it. | 09:22 |
| jordanP | ianw, yeah, that make more sense | 09:22 |
| *** k4n0 has joined #openstack-qa | 09:23 | |
| *** aix has quit IRC | 09:23 | |
| *** vponomaryov has joined #openstack-qa | 09:24 | |
| *** yamahata has quit IRC | 09:31 | |
| *** yamamot__ has quit IRC | 09:32 | |
| *** andreykurilin_ has quit IRC | 09:41 | |
| openstackgerrit | Rohan Kanade proposed openstack/tempest: Split resource_setup for volume tests https://review.openstack.org/151610 | 09:41 |
| *** gfidente has joined #openstack-qa | 09:42 | |
| *** achanda has quit IRC | 09:46 | |
| *** jhenner has quit IRC | 09:49 | |
| *** zz_johnthetubagu is now known as johnthetubaguy | 10:00 | |
| *** e0ne has joined #openstack-qa | 10:00 | |
| openstackrecheck | openstack/requirements change: https://review.openstack.org/151770 failed gate-grenade-dsvm, gate-requirements-integration-dsvm in the gate queue with an unrecognized error | 10:03 |
| *** jhenner has joined #openstack-qa | 10:08 | |
| *** e0ne has quit IRC | 10:09 | |
| *** e0ne has joined #openstack-qa | 10:09 | |
| *** e0ne is now known as e0ne_ | 10:19 | |
| *** andreaf_ is now known as andreaf | 10:21 | |
| *** e0ne_ has quit IRC | 10:24 | |
| *** e0ne has joined #openstack-qa | 10:24 | |
| *** tochi has quit IRC | 10:28 | |
| *** yfried_ is now known as yfried|afk | 10:32 | |
| *** yfried|afk is now known as yfried_ | 10:35 | |
| *** k4n0 has quit IRC | 10:35 | |
| *** k4n0 has joined #openstack-qa | 10:36 | |
| *** vigneshvar has quit IRC | 10:37 | |
| *** vigneshvar has joined #openstack-qa | 10:40 | |
| openstackgerrit | Rohan Kanade proposed openstack/tempest: Split resource_setup for object_storage tests https://review.openstack.org/152105 | 10:46 |
| *** eezhova has joined #openstack-qa | 10:47 | |
| *** k4n0 has quit IRC | 10:47 | |
| *** yfried_ is now known as yfried|afk | 10:48 | |
| *** erlon has joined #openstack-qa | 10:53 | |
| *** yfried|afk is now known as yfried_ | 11:08 | |
| *** e0ne is now known as e0ne_ | 11:08 | |
| *** e0ne_ has quit IRC | 11:14 | |
| *** adalbas has joined #openstack-qa | 11:24 | |
| openstackgerrit | Rohan Kanade proposed openstack/tempest: Split resource_setup for data_processing tests https://review.openstack.org/152412 | 11:27 |
| *** aysyd has joined #openstack-qa | 11:30 | |
| *** afazekas has joined #openstack-qa | 11:34 | |
| openstackgerrit | Rohan Kanade proposed openstack/tempest: Split resource_setup for baremetal tests https://review.openstack.org/152417 | 11:38 |
| *** afazekas has quit IRC | 11:39 | |
| *** chandankumar has quit IRC | 11:42 | |
| *** e0ne has joined #openstack-qa | 11:52 | |
| *** cdent has joined #openstack-qa | 11:54 | |
| *** andreaf_ has joined #openstack-qa | 11:54 | |
| *** jamielennox is now known as jamielennox|away | 11:56 | |
| *** e0ne is now known as e0ne_ | 11:58 | |
| *** andreaf_ has quit IRC | 11:58 | |
| *** andreaf_ has joined #openstack-qa | 11:59 | |
| *** e0ne_ is now known as e0ne | 11:59 | |
| openstackgerrit | Bob Ball proposed openstack-dev/devstack: XenAPI: Use round-robin mirror for Ubuntu https://review.openstack.org/153792 | 12:04 |
| *** gszasz has quit IRC | 12:05 | |
| *** gszasz has joined #openstack-qa | 12:08 | |
| *** Longgeek has quit IRC | 12:12 | |
| openstackrecheck | openstack/sahara change: https://review.openstack.org/151621 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1373513 | 12:16 |
| openstack | Launchpad bug 1373513 in Cinder "Lvm hang during tempest tests" [Critical,In progress] - Assigned to John Griffith (john-griffith) | 12:16 |
| *** gszasz has quit IRC | 12:17 | |
| *** gszasz has joined #openstack-qa | 12:19 | |
| *** afazekas has joined #openstack-qa | 12:19 | |
| openstackgerrit | Erlon R. Cruz proposed openstack-dev/devstack: Set pip_install default behavior to 'upgrade' https://review.openstack.org/154008 | 12:24 |
| *** aix has joined #openstack-qa | 12:29 | |
| *** rfolco has joined #openstack-qa | 12:32 | |
| *** rushiagr is now known as rushiagr_away | 12:35 | |
| *** rfolco has quit IRC | 12:36 | |
| *** Madan has quit IRC | 12:44 | |
| *** dimsum__ has joined #openstack-qa | 12:44 | |
| *** dims_ has joined #openstack-qa | 12:45 | |
| openstackgerrit | Erlon R. Cruz proposed openstack-dev/devstack: Set pip_install default behavior to 'upgrade' https://review.openstack.org/154008 | 12:46 |
| openstackgerrit | nithya-ganesan proposed openstack/tempest: WIP - Create shared ssh verification by extending RemoteClient https://review.openstack.org/153899 | 12:48 |
| openstackgerrit | Rohan Kanade proposed openstack/tempest: Split resource_setup for image tests https://review.openstack.org/153215 | 12:48 |
| *** dimsum__ has quit IRC | 12:49 | |
| *** rfolco has joined #openstack-qa | 12:49 | |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: remove tuskar https://review.openstack.org/154015 | 12:51 |
| *** rfolco has quit IRC | 12:51 | |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: gantt is a dead project, remove it https://review.openstack.org/154017 | 12:53 |
| openstackgerrit | Erlon R. Cruz proposed openstack-dev/devstack: Set pip_install default behavior to 'upgrade' https://review.openstack.org/154008 | 12:54 |
| openstackgerrit | Erlon R. Cruz proposed openstack-dev/devstack: Set pip_install default behavior to 'upgrade' https://review.openstack.org/154008 | 12:55 |
| *** e0ne is now known as e0ne_ | 12:59 | |
| openstackgerrit | Rohan Kanade proposed openstack/tempest: Split resource_setup for identity tests https://review.openstack.org/153205 | 13:01 |
| *** nmagnezi has quit IRC | 13:04 | |
| *** neeti has quit IRC | 13:05 | |
| *** weshay has joined #openstack-qa | 13:06 | |
| *** e0ne_ is now known as e0ne | 13:06 | |
| *** eezhova has quit IRC | 13:14 | |
| openstackgerrit | Rohan Kanade proposed openstack/tempest: Split resource_setup for messaging tests https://review.openstack.org/153764 | 13:16 |
| *** rushiagr_away is now known as rushiagr | 13:16 | |
| *** BharatK has quit IRC | 13:16 | |
| *** nmagnezi has joined #openstack-qa | 13:19 | |
| *** yfried_ is now known as yfried|afk | 13:23 | |
| *** yfried|afk is now known as yfried_ | 13:24 | |
| *** toure_ has joined #openstack-qa | 13:25 | |
| *** toure_ has quit IRC | 13:25 | |
| *** e0ne is now known as e0ne_ | 13:30 | |
| openstackgerrit | Rohan Kanade proposed openstack/tempest: Split resource_setup for orchestration tests https://review.openstack.org/153765 | 13:31 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/153041 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1353939 https://bugs.launchpad.net/bugs/1342380 | 13:32 |
| openstack | Launchpad bug 1353939 in OpenStack Compute (nova) "Rescue fails with 'Failed to terminate process: Device or resource busy' in the n-cpu log" [High,Confirmed] | 13:32 |
| openstack | Launchpad bug 1353939 in OpenStack Compute (nova) "duplicate for #1342380 Rescue fails with 'Failed to terminate process: Device or resource busy' in the n-cpu log" [High,Confirmed] | 13:32 |
| *** woodster_ has joined #openstack-qa | 13:34 | |
| openstackgerrit | Rohan Kanade proposed openstack/tempest: Split resource_setup for telemetry tests https://review.openstack.org/153766 | 13:35 |
| *** bknudson has quit IRC | 13:36 | |
| *** rfolco has joined #openstack-qa | 13:37 | |
| *** e0ne_ is now known as e0ne | 13:38 | |
| *** jecarey has quit IRC | 13:38 | |
| *** rfolco has quit IRC | 13:40 | |
| *** rushiagr is now known as rushiagr_away | 13:41 | |
| *** Longgeek has joined #openstack-qa | 13:44 | |
| *** weshay has quit IRC | 13:47 | |
| *** afazekas has quit IRC | 13:47 | |
| *** aix has quit IRC | 13:47 | |
| *** weshay has joined #openstack-qa | 13:54 | |
| *** vigneshvar has quit IRC | 13:55 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Initial class creds creation in test base class https://review.openstack.org/153681 | 13:57 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Drop the legacy and un-used _interface https://review.openstack.org/153621 | 13:57 |
| *** Longgeek has quit IRC | 13:57 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Split creds and resources in tenant isolation https://review.openstack.org/153896 | 13:57 |
| *** bknudson has joined #openstack-qa | 13:59 | |
| *** Longgeek has joined #openstack-qa | 14:00 | |
| *** SridharG has left #openstack-qa | 14:00 | |
| *** david-lyle_afk is now known as david-lyle | 14:01 | |
| openstackgerrit | Joseph Lanoux proposed openstack/tempest: Update test_live_migration for return-one-value https://review.openstack.org/154047 | 14:01 |
| *** Yogi1 has joined #openstack-qa | 14:02 | |
| *** krtaylor has quit IRC | 14:03 | |
| *** anshul has quit IRC | 14:04 | |
| *** e0ne has quit IRC | 14:05 | |
| *** e0ne has joined #openstack-qa | 14:05 | |
| *** amotoki has quit IRC | 14:09 | |
| openstackgerrit | Nikolay Fedotov proposed openstack/tempest: Scenario test for vpnaas: ipsec-site-connection https://review.openstack.org/140072 | 14:13 |
| sdague | dtroyer: let me know once you get around today | 14:13 |
| sdague | I want to figure out the elasticsearch pkg thing | 14:13 |
| *** jaypipes has joined #openstack-qa | 14:15 | |
| *** ChanServ sets mode: +o jaypipes | 14:15 | |
| *** dustins has joined #openstack-qa | 14:16 | |
| *** krtaylor has joined #openstack-qa | 14:16 | |
| *** jecarey has joined #openstack-qa | 14:19 | |
| *** cbader has joined #openstack-qa | 14:19 | |
| openstackgerrit | nithya-ganesan proposed openstack/tempest: WIP - Create shared ssh verification by extending RemoteClient https://review.openstack.org/153899 | 14:22 |
| *** e0ne is now known as e0ne_ | 14:23 | |
| *** Longgeek has quit IRC | 14:23 | |
| openstackgerrit | David Kranz proposed openstack/tempest: Change security-group clients to return one value and update tests https://review.openstack.org/153685 | 14:23 |
| openstackgerrit | David Kranz proposed openstack/tempest: Change keypair client to return one value and update tests https://review.openstack.org/153652 | 14:24 |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: add gating up/down script for devstack https://review.openstack.org/152994 | 14:24 |
| *** wuhg has quit IRC | 14:29 | |
| *** rushiagr_away is now known as rushiagr | 14:29 | |
| *** abhirc has joined #openstack-qa | 14:38 | |
| *** xavileon has quit IRC | 14:42 | |
| dkranz | andreaf: ping | 14:48 |
| dkranz | andreaf: Are overrrides of the new methods like skip_checks supposed to call super? I would expect so but see some code that does not. | 14:48 |
| *** e0ne_ is now known as e0ne | 14:49 | |
| *** sabeen has joined #openstack-qa | 14:54 | |
| *** afazekas has joined #openstack-qa | 14:54 | |
| *** aix has joined #openstack-qa | 14:56 | |
| *** bkopilov has quit IRC | 14:58 | |
| *** dimtruck is now known as zz_dimtruck | 15:00 | |
| *** mriedem has joined #openstack-qa | 15:01 | |
| *** Ananci has joined #openstack-qa | 15:04 | |
| *** ChuckC has joined #openstack-qa | 15:06 | |
| *** ChuckC has quit IRC | 15:06 | |
| *** ChuckC has joined #openstack-qa | 15:07 | |
| dtroyer | sdague: g'morning. I should be sufficiently caffeinated by now to do actual thinking… ;) | 15:10 |
| *** amotoki has joined #openstack-qa | 15:10 | |
| sdague | dtroyer: ok, cool | 15:10 |
| sdague | so, the thinking question is what to do about - https://review.openstack.org/#/c/153048/ | 15:10 |
| *** superdan is now known as dansmith | 15:10 | |
| sdague | I'd like to get some solution in place | 15:10 |
| sdague | so that the glance folks can move forward | 15:11 |
| *** Obulpathi has joined #openstack-qa | 15:11 | |
| dtroyer | for the most part I think that one is OK | 15:11 |
| sdague | also, I've started to get agressive with folks and -2ing stuff that should be out of tree plugins, given that we've got a working example of testing with the ec2api up | 15:11 |
| dtroyer | yeah, that's a good direction | 15:12 |
| sdague | and my culling branch - https://review.openstack.org/#/q/status:open+project:openstack-dev/devstack+branch:master+topic:culling,n,z | 15:13 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Initial class creds creation in test base class https://review.openstack.org/153681 | 15:13 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Split creds and resources in tenant isolation https://review.openstack.org/153896 | 15:13 |
| *** rbak has joined #openstack-qa | 15:13 | |
| dtroyer | I think my concern in https://etherpad.openstack.org/p/devstack-package-install-interface is how the packages are called. There's a comment in about making the calling of the package functions a bit more atomic and sharable | 15:14 |
| sdague | dtroyer: oh, that's fine, I can work up function calls today if you like | 15:15 |
| dtroyer | the interface functions are fine, its just how we get there. it feels to me it should be more like distro services than integrated sub-services | 15:16 |
| sdague | yeh, that might be. I'll ponder while I go deal with snow removal. Another Monday, another storm. | 15:16 |
| dtroyer | yuck…I'll not brag about 60+ this weekend then | 15:17 |
| andreaf | dkranz: yes overrides should always call super | 15:18 |
| andreaf | dkranz: I haven't reviewed all those patches yet so it might be there are a few missing around | 15:18 |
| *** Longgeek has joined #openstack-qa | 15:24 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Initial class creds creation in test base class https://review.openstack.org/153681 | 15:25 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Split creds and resources in tenant isolation https://review.openstack.org/153896 | 15:25 |
| dkranz | andreaf: thanks, I already gave -1 since I decided this expectation was generic | 15:28 |
| *** nmagnezi has quit IRC | 15:31 | |
| openstackgerrit | Merged openstack-infra/devstack-gate: Refactor processing of testr artifacts for reuse https://review.openstack.org/153677 | 15:32 |
| *** zz_dimtruck is now known as dimtruck | 15:32 | |
| openstackgerrit | Merged openstack-infra/devstack-gate: Enable preseeding testrepository with a number of streams https://review.openstack.org/153036 | 15:32 |
| openstackgerrit | Merged openstack-infra/devstack-gate: Remove unused variables(PIP_USE_MIRRORS, USE_GET_PIP) https://review.openstack.org/149439 | 15:32 |
| openstackgerrit | Merged openstack-infra/devstack-gate: Add partial-ironic support https://review.openstack.org/148760 | 15:32 |
| *** afazekas has quit IRC | 15:32 | |
| andreaf | dkranz: great, thanks | 15:33 |
| openstackgerrit | nithya-ganesan proposed openstack/tempest: WIP - Create shared ssh verification by extending RemoteClient https://review.openstack.org/153899 | 15:33 |
| openstackrecheck | openstack/sahara change: https://review.openstack.org/151621 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1412923 | 15:34 |
| 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) | 15:34 |
| *** enikanorov has joined #openstack-qa | 15:35 | |
| *** achanda has joined #openstack-qa | 15:35 | |
| *** enikanorov has quit IRC | 15:36 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/152866 failed gate-grenade-dsvm, gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error | 15:36 |
| *** yfried_ is now known as yfried|afk | 15:40 | |
| *** achanda has quit IRC | 15:42 | |
| *** malini has joined #openstack-qa | 15:42 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Initial class creds creation in test base class https://review.openstack.org/153681 | 15:44 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Split creds and resources in tenant isolation https://review.openstack.org/153896 | 15:44 |
| *** Obulpathi has quit IRC | 15:45 | |
| *** afazekas has joined #openstack-qa | 15:46 | |
| *** hemnafk is now known as hemna | 15:46 | |
| *** Longgeek has quit IRC | 15:50 | |
| *** Obulpathi has joined #openstack-qa | 15:51 | |
| *** yfried|afk is now known as yfried_ | 15:51 | |
| *** zz_jgrimm is now known as jgrimm | 15:51 | |
| *** ChuckC has quit IRC | 15:51 | |
| *** ChuckC has joined #openstack-qa | 15:52 | |
| hogepodge | sdague can you give me a sense of how hacking rules work within tempest? Just a high level overview of when and how they're applied. | 15:54 |
| hogepodge | or mtreinish ^^, or anyone else | 15:54 |
| *** Obulpathi has left #openstack-qa | 15:55 | |
| *** rfolco has joined #openstack-qa | 15:55 | |
| mtreinish | hogepodge: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/hacking/checks.py is where the custom tempest rules exist | 15:55 |
| mtreinish | they get run per file using flake8 | 15:55 |
| *** dims_ has quit IRC | 15:56 | |
| mtreinish | hogepodge: flake8 does all the parsing of the file and passes lines of strings to each check method in that file | 15:56 |
| mtreinish | the test file explains it a bit more: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/tests/test_hacking.py | 15:57 |
| hogepodge | I see. | 15:57 |
| hogepodge | I have a first pass at a hacking rule for checking uuid metadata. I'm running tox right now to make sure it passes style, but I'll send it up for review. | 15:59 |
| *** pcrews has joined #openstack-qa | 15:59 | |
| *** beekneemech is now known as bnemec | 15:59 | |
| hogepodge | My concern with my implementation is that previous_logical won't catch it on tests where there is other metadata. | 15:59 |
| *** aix has quit IRC | 16:00 | |
| mtreinish | hogepodge: why not? logical lines treat a multiline statement as one line | 16:00 |
| mtreinish | or do you mean other decorators, because that might be an issue if there is a services decorator or a skip | 16:00 |
| hogepodge | That's exactly what I'm worried about. I based my work off of the services decorator check. | 16:01 |
| *** dimsum__ has joined #openstack-qa | 16:01 | |
| openstackgerrit | John Villalovos proposed openstack-dev/devstack: Use 'https' protocol by default as easier for proxies https://review.openstack.org/153397 | 16:04 |
| mtreinish | hogepodge: yeah's a tricky one, I'm not exactly sure how to proceed there. Maybe we should just enforce a decorator order. Since every test requires a uuid decorator we say that goes on the bottom | 16:05 |
| *** itzikb has quit IRC | 16:05 | |
| *** shashankhegde has joined #openstack-qa | 16:05 | |
| mtreinish | and if it also requires a service decorator use the uuid decorator as the line to check for instead of test_* | 16:05 |
| hogepodge | mtreinish I think that's a good idea. | 16:05 |
| mtreinish | then any other decorators get stacked on top of that. I think that should work | 16:06 |
| *** dimsum__ has quit IRC | 16:08 | |
| *** dimsum__ has joined #openstack-qa | 16:10 | |
| *** dimsum__ has quit IRC | 16:11 | |
| openstackgerrit | Merged openstack/tempest: Remove CONF values from volume clients https://review.openstack.org/149910 | 16:11 |
| openstackgerrit | Merged openstack/tempest: Separate token client from identity client https://review.openstack.org/151530 | 16:12 |
| *** dimsum__ has joined #openstack-qa | 16:12 | |
| openstackgerrit | Merged openstack-dev/devstack: guides: Notes to setup DevStack with Nested KVM Virtualization https://review.openstack.org/148630 | 16:12 |
| *** dimsum__ has quit IRC | 16:13 | |
| openstackgerrit | Merged openstack-dev/devstack: lib/neutron: Update comments about entry points https://review.openstack.org/150648 | 16:13 |
| *** dimsum__ has joined #openstack-qa | 16:13 | |
| *** dimsum__ has quit IRC | 16:14 | |
| andreaf | mtreinish: no test run (because of a bug in the code), but still getting a list of slowest tests (https://jenkins03.openstack.org/job/check-tempest-dsvm-full-icehouse/2807/console) - is this from pre-seeding of avg test run times? | 16:15 |
| *** coolsvap is now known as coolsvap_ | 16:15 | |
| mtreinish | andreaf: ooh, nice catch. Yeah the test times there are probably from the pre-seed | 16:16 |
| mtreinish | andreaf: we probably can fix pretty_tox to catch that case and not run slowest | 16:16 |
| mtreinish | oh wait, maybe because we still want it to run if the test run actually ran tests but failed | 16:16 |
| *** dimsum__ has joined #openstack-qa | 16:17 | |
| mtreinish | it might require changing subunit-trace to return a specific code if there are no tests run | 16:17 |
| andreaf | mtreinish: yes that might be a tricky case to catch :) | 16:17 |
| mtreinish | I think we just use 1 now in subunit-trace, which is too generic probably | 16:17 |
| *** raghav_mulugu has quit IRC | 16:21 | |
| openstackgerrit | Matthew Treinish proposed openstack/tempest-lib: Migrate skip_tracker.py from tempest https://review.openstack.org/152338 | 16:22 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Split creds and resources in tenant isolation https://review.openstack.org/153896 | 16:22 |
| *** sergsh has joined #openstack-qa | 16:26 | |
| sergsh | dkranz, ping | 16:26 |
| dkranz | sergsh: pong | 16:26 |
| sergsh | dkranz, abput https://review.openstack.org/#/c/143806/4/tempest/api/network/test_dhcp_ipv6.py | 16:27 |
| sergsh | dkranz, what do you mean "call super first"? | 16:27 |
| dkranz | sergsh: skip_checks is overriding the method in the superclass | 16:27 |
| dkranz | sergsh: it needs to call super the same way resource_setup does | 16:28 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Initial class creds creation in test base class https://review.openstack.org/153681 | 16:28 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Split creds and resources in tenant isolation https://review.openstack.org/153896 | 16:28 |
| dkranz | sergsh: super(NetworksTestDHCPv6, cls).skip_checks() | 16:28 |
| sergsh | dkranz, ok, got it | 16:28 |
| *** aix has joined #openstack-qa | 16:29 | |
| *** afazekas has quit IRC | 16:33 | |
| *** naggappan has quit IRC | 16:34 | |
| flashgordon | dhellmann: https://review.openstack.org/#/c/153708/ your part of reqs core right? | 16:35 |
| *** flashgordon is now known as jogo | 16:35 | |
| *** gszasz has quit IRC | 16:35 | |
| openstackgerrit | Merged openstack-dev/devstack: Keystone set debug in config file https://review.openstack.org/153876 | 16:35 |
| *** gszasz has joined #openstack-qa | 16:35 | |
| openstackgerrit | Merged openstack-dev/devstack: Removed confusing duplication of ca config for keystone https://review.openstack.org/153742 | 16:36 |
| *** shashankhegde has quit IRC | 16:38 | |
| openstackrecheck | openstack/requirements change: https://review.openstack.org/151037 failed gate-grenade-dsvm, gate-requirements-integration-dsvm in the gate queue with an unrecognized error | 16:39 |
| openstackgerrit | David Kranz proposed openstack/tempest: Change flavors client to return one value and update tests https://review.openstack.org/154114 | 16:41 |
| openstackgerrit | Sergey Shnaidman proposed openstack/tempest: Test attaching router to IPv6 subnets https://review.openstack.org/143806 | 16:42 |
| *** rwsu has joined #openstack-qa | 16:44 | |
| openstackgerrit | Jordan Pittier proposed openstack/tempest: Volumes: strict type and value check of the 'bootable' flag https://review.openstack.org/152148 | 16:45 |
| openstackgerrit | Jordan Pittier proposed openstack/tempest: Volumes: strict type and value check of the 'bootable' flag https://review.openstack.org/152148 | 16:46 |
| *** yfried_ has quit IRC | 16:46 | |
| sergsh | dkranz, fixed it, could you please check it in your time? | 16:46 |
| openstackgerrit | Terry Wilson proposed openstack-dev/devstack: Install python-openvswitch with openvswitch https://review.openstack.org/154118 | 16:47 |
| *** Longgeek has joined #openstack-qa | 16:47 | |
| *** nicodemos has joined #openstack-qa | 16:47 | |
| dkranz | sergsh: sure | 16:48 |
| *** bkopilov has joined #openstack-qa | 16:48 | |
| *** afazekas has joined #openstack-qa | 16:50 | |
| openstackgerrit | Sergey Shnaidman proposed openstack/tempest: Support for IPv6 for API (part 2) https://review.openstack.org/137239 | 16:53 |
| *** vigneshvar has joined #openstack-qa | 16:55 | |
| *** Yogi1 has quit IRC | 16:59 | |
| *** marun has joined #openstack-qa | 17:00 | |
| *** coolsvap_ is now known as coolsvap | 17:02 | |
| *** amotoki has quit IRC | 17:03 | |
| *** jlanoux has quit IRC | 17:03 | |
| openstackgerrit | Chris Hoge proposed openstack/tempest: Add new hacking rule for UUID metatdata https://review.openstack.org/154127 | 17:04 |
| *** Yogi1 has joined #openstack-qa | 17:04 | |
| *** enikanorov has joined #openstack-qa | 17:05 | |
| openstackgerrit | Matthew Treinish proposed openstack-infra/devstack-gate: DO NOT MERGE COPY ALL OF .TESTREPOSITORY to log server https://review.openstack.org/149719 | 17:07 |
| openstackgerrit | Sergey Shnaidman proposed openstack/tempest: Support for IPv6 for API (part 2) https://review.openstack.org/137239 | 17:09 |
| *** Longgeek has quit IRC | 17:10 | |
| *** rwsu has quit IRC | 17:10 | |
| *** rwsu has joined #openstack-qa | 17:11 | |
| dhellmann | jogo: I was just on my way here to ask you about what's up with tempest-lib | 17:12 |
| dhellmann | the requirements for oslo.config in stable/juno seem to be in conflict with the other apps? is there anything I can do to help resolve that | 17:13 |
| dhellmann | jogo: +2a | 17:13 |
| dhellmann | jogo: I guess maybe that's what https://review.openstack.org/#/c/153702/ is for? | 17:14 |
| *** mlavalle has joined #openstack-qa | 17:15 | |
| *** EmilienM is now known as EmilienM|afk | 17:15 | |
| *** toure_ has joined #openstack-qa | 17:15 | |
| *** toure_ has quit IRC | 17:15 | |
| *** jpich has quit IRC | 17:19 | |
| openstackgerrit | Merged openstack/tempest: Change tempest UnprocessableEntity exc to tempest-lib exc https://review.openstack.org/148436 | 17:25 |
| *** btran has joined #openstack-qa | 17:25 | |
| openstackgerrit | Merged openstack/tempest: Change tempest InvalidContentType exc to tempest-lib exc https://review.openstack.org/148437 | 17:25 |
| *** mpaolino has quit IRC | 17:28 | |
| *** zigo has quit IRC | 17:29 | |
| *** otherwiseguy has joined #openstack-qa | 17:30 | |
| mriedem | mtreinish: i have to say, i had a hell of a time trying to find a 100% fail in the gate on that keystone race bug you fixed on friday | 17:30 |
| mriedem | with the duplicate role create | 17:30 |
| mriedem | i think that's what is killing the categorization rates | 17:30 |
| mtreinish | mriedem: heh, really I only saw it once on a change sdague pointed out | 17:31 |
| mtreinish | I didn't think it was that common | 17:31 |
| otherwiseguy | sdague: I'm hoping that this change (https://review.openstack.org/#/c/154118/) isn't too controversial. Just adding the installation of python-openvswitch when installing openvswitch to support my native ovsdb backend for ovs_lib patch. python-openvswitch doesn't rely on any other python packages. | 17:31 |
| mriedem | i had it on a nova change of mine | 17:31 |
| mriedem | otherwiseguy: is neutron finally using python-openvswitch rather than command line? | 17:32 |
| mriedem | b/c that would be nice | 17:32 |
| otherwiseguy | mriedem: I have a patch that basically adds an abstract API with a backend for using ovs-vsctl and one for python-openvswitch. | 17:33 |
| otherwiseguy | That way we can transition. | 17:33 |
| mtreinish | mriedem: the bigger issue with that failure is we don't index the keystone logs so we can't write a good e-r query | 17:33 |
| *** gszasz has quit IRC | 17:33 | |
| clarkb | mtreinish: we should now | 17:34 |
| clarkb | sdague fixed it | 17:34 |
| otherwiseguy | So I'm wanting to test both backends with the same functional test, just configuring which backend is used. But python-openvswitch isn't available in PyPI and it's a bit of a pain to get it ready to put in PyPI. | 17:34 |
| mriedem | mtreinish: ? i was getting hits with tags:"screen-key.txt" and tags:"keystone.txt" | 17:34 |
| sdague | mriedem: that's the minority case | 17:34 |
| sdague | most of the runs don't use that log file | 17:35 |
| mtreinish | clarkb: ah, ok cool | 17:35 |
| sdague | because they are under apache | 17:35 |
| mtreinish | sdague: that's just picking up the eventlet case? | 17:35 |
| sdague | so I actually don't know about the keystone.txt one doing the right thing or not | 17:35 |
| openstackgerrit | David Kranz proposed openstack/tempest: Change limits/quota clients to return one value and update tests https://review.openstack.org/154137 | 17:36 |
| sdague | but we definitely weren't indexing that file until late last week | 17:36 |
| *** Nic has joined #openstack-qa | 17:37 | |
| mriedem | mtreinish: seen this? http://logs.openstack.org/37/151037/1/gate/gate-grenade-dsvm/a767b9a/logs/grenade.sh.txt.gz#_2015-02-09_16_04_55_459 | 17:37 |
| mriedem | pkg_resources.ContextualVersionConflict: (oslo.config 1.4.0 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('oslo.config>=1.6.0'), set(['tempest-lib'])) | 17:37 |
| mtreinish | mriedem: yeah that's breaking the juno gate | 17:38 |
| mriedem | bug reported? | 17:38 |
| mtreinish | jogo: is working on it | 17:38 |
| mriedem | assuming capping tempest-lib on stable/juno? | 17:38 |
| mtreinish | mriedem: it's not as simple because tempest is run from master | 17:39 |
| mtreinish | we already run tempest in a tox venv, the fix is to also do that with javelin | 17:39 |
| mriedem | ok | 17:39 |
| otherwiseguy | mriedem: replied to your question re: PyPI on the review. | 17:41 |
| mtreinish | oh, cool it looks like the fixed preseeding is already working | 17:41 |
| mriedem | mtreinish: jogo: reported a bug in case there isn't one https://bugs.launchpad.net/grenade/+bug/1419913 | 17:41 |
| openstack | Launchpad bug 1419913 in grenade "Grenade fails with ContextualVersionConflict between oslo.config and tempest-lib on juno side" [Undecided,New] | 17:41 |
| mtreinish | that took effect faster then I was expecting | 17:41 |
| *** krtaylor has quit IRC | 17:41 | |
| *** jlibosva has quit IRC | 17:43 | |
| openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for grenade bug 1419913 https://review.openstack.org/154147 | 17:47 |
| openstack | bug 1419913 in grenade "Grenade fails with ContextualVersionConflict between oslo.config and tempest-lib on juno side" [Undecided,New] https://launchpad.net/bugs/1419913 | 17:47 |
| mriedem | mtreinish: ^ | 17:47 |
| mriedem | also seeing this http://logs.openstack.org/37/151037/1/gate/gate-requirements-integration-dsvm/7a9f85f/console.html#_2015-02-09_16_13_26_268 | 17:50 |
| mtreinish | mriedem: looks like the stable release tag broke ceilo | 17:52 |
| mriedem | and keystone | 17:53 |
| mtreinish | they probably just forgot to bump the version string in setup.cfg (although they really don't need one IIRC) | 17:53 |
| mriedem | yup https://github.com/openstack/ceilometer/blob/stable/juno/setup.cfg#L3 | 17:53 |
| mriedem | https://github.com/openstack/keystone/blob/stable/juno/setup.cfg#L3 | 17:53 |
| mriedem | i can push patches for those | 17:54 |
| *** krtaylor has joined #openstack-qa | 17:54 | |
| openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for busted requirements on juno bug 1419919 https://review.openstack.org/154158 | 17:57 |
| openstack | bug 1419919 in Keystone "gate-requirements-integration-dsvm fails on stable/juno with ValueError for missing 2014.2.3" [Undecided,New] https://launchpad.net/bugs/1419919 | 17:57 |
| mriedem | mtreinish: ^ | 17:57 |
| mriedem | mtreinish: i amazingly have no meetings today so i can actually do things | 17:57 |
| morganfainberg | mriedem, is that allowed? | 17:58 |
| mriedem | morganfainberg: someone f'ed up | 17:58 |
| morganfainberg | mriedem, a day with no meetings that is | 17:58 |
| morganfainberg | hehe | 17:58 |
| openstackgerrit | Merged openstack-dev/devstack: implement elasticsearch pkg for devstack https://review.openstack.org/153048 | 17:58 |
| mriedem | mtreinish: hmm, cinder and nova also have hard-coded versions in their setup.cfg on stable/juno | 17:58 |
| openstackgerrit | Merged openstack-dev/devstack: Isolate creating service users https://review.openstack.org/150672 | 17:59 |
| *** dmorita has joined #openstack-qa | 17:59 | |
| *** gfidente has quit IRC | 18:01 | |
| *** yamahata has joined #openstack-qa | 18:03 | |
| *** harlowja has joined #openstack-qa | 18:04 | |
| *** annashen has joined #openstack-qa | 18:05 | |
| *** safchain has quit IRC | 18:06 | |
| *** afazekas has quit IRC | 18:07 | |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for grenade bug 1419913 https://review.openstack.org/154147 | 18:08 |
| openstack | bug 1419913 in grenade "Grenade fails with ContextualVersionConflict between oslo.config and tempest-lib on juno side" [Undecided,Confirmed] https://launchpad.net/bugs/1419913 | 18:08 |
| *** e0ne has quit IRC | 18:09 | |
| mtreinish | mriedem: ^^^ man you didn't even give me a chance to +A it... | 18:11 |
| mriedem | mtreinish: too slow | 18:11 |
| mriedem | i'm bumping the versions on the stable/juno branches for each project | 18:11 |
| mriedem | looks like apavec did that for 2014.2.2 after 2014.2.1 was released but it hasn't been done for 2014.2.2/3 yet | 18:12 |
| *** melwitt has joined #openstack-qa | 18:14 | |
| *** aix has quit IRC | 18:15 | |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for busted requirements on juno bug 1419919 https://review.openstack.org/154158 | 18:15 |
| openstack | bug 1419919 in Keystone "gate-requirements-integration-dsvm fails on stable/juno with ValueError for missing 2014.2.3" [Undecided,New] https://launchpad.net/bugs/1419919 | 18:15 |
| *** yjiang5_ has joined #openstack-qa | 18:16 | |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: add gating up/down script for devstack https://review.openstack.org/152994 | 18:16 |
| yjiang5_ | Hi, all, my multi-host devstack environment failed because https://review.openstack.org/#/c/149288/ will die devstack if I have my controller node in another host. I remember devstack should support multiple host enviornment, right? | 18:16 |
| *** openstackrecheck has quit IRC | 18:17 | |
| *** openstackrecheck has joined #openstack-qa | 18:17 | |
| yjiang5_ | dtroyer: do you have any idea on it? | 18:17 |
| *** rushiagr is now known as rushiagr_away | 18:18 | |
| *** luqas has quit IRC | 18:22 | |
| clarkb | yjiang5_: yes, in fact that change has been reverted | 18:23 |
| clarkb | yjiang5_: bceause we need to not die on multinode installs | 18:23 |
| *** mfalatic has joined #openstack-qa | 18:25 | |
| clarkb | yjiang5_: make sure your checkout of devstack is up to date | 18:25 |
| yjiang5_ | clarkb: aha, thanks. So possibly I forgot to update my devstack env.Thanks. | 18:25 |
| yjiang5_ | clarkb: possibly that patch is still helpful. We only need something to do more accurate check and dies if it's really a mis-configuration. I will try to see if I can cook a patch for it. | 18:26 |
| mriedem | mtreinish: not sure if this is the answer, but https://review.openstack.org/#/q/Ib8a29258d99de75b49a9b19aef36bb99bc5fcac0,n,z | 18:26 |
| mriedem | going to lunch | 18:26 |
| *** krtaylor has quit IRC | 18:27 | |
| *** shashankhegde has joined #openstack-qa | 18:27 | |
| *** yamahata has quit IRC | 18:29 | |
| *** patrickeast has joined #openstack-qa | 18:30 | |
| *** yamahata has joined #openstack-qa | 18:30 | |
| openstackgerrit | MORITA Kazutaka proposed openstack-dev/devstack: Add support Sheepdog cinder backend https://review.openstack.org/154184 | 18:30 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/tempest: Updated from global requirements https://review.openstack.org/154187 | 18:31 |
| sdague | jogo: didn't realize this had wedged juno - https://review.openstack.org/#/c/153080/7 | 18:31 |
| sdague | +2 now | 18:31 |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: remove tuskar https://review.openstack.org/154015 | 18:32 |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: gantt is a dead project, remove it https://review.openstack.org/154017 | 18:32 |
| *** markus_z has quit IRC | 18:33 | |
| sdague | mriedem: once you get back, is this a rush fix - https://review.openstack.org/#/c/153080/ ? | 18:34 |
| openstackgerrit | David Kranz proposed openstack/tempest: Change flavors client to return one value and update tests https://review.openstack.org/154114 | 18:37 |
| *** patrickeast has quit IRC | 18:39 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 18:39 | |
| *** patrickeast has joined #openstack-qa | 18:42 | |
| *** achanda has joined #openstack-qa | 18:43 | |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: add gating up/down script for devstack https://review.openstack.org/152994 | 18:44 |
| *** coolsvap is now known as coolsvap_ | 18:45 | |
| mriedem | sdague: rush fix? | 18:45 |
| *** yfried_ has joined #openstack-qa | 18:45 | |
| sdague | mriedem: that should unwedge juno | 18:46 |
| sdague | I think | 18:46 |
| mriedem | sdague: there are 2 bugs that i know of | 18:46 |
| mriedem | started the ML thread | 18:46 |
| sdague | ok, well I'm asking you, as the started of said thread, you want me to fast approve https://review.openstack.org/#/c/153080/ ? | 18:46 |
| *** coolsvap_ is now known as coolsvap | 18:46 | |
| *** vigneshvar has quit IRC | 18:47 | |
| mriedem | sdague: well it looks like grenade is passing, so i suppose | 18:47 |
| *** coolsvap is now known as coolsvap_ | 18:47 | |
| *** achanda has quit IRC | 18:47 | |
| mriedem | looks like there is more work to do there but if it's busting grenade i say go for it | 18:47 |
| sdague | although, grenade on master seems fine... | 18:48 |
| sdague | and that's a master patch | 18:48 |
| mriedem | http://status.openstack.org/elastic-recheck/gate.html | 18:48 |
| mriedem | so is the problem in juno/icehouse? | 18:48 |
| sdague | yeh, I don't know | 18:49 |
| mriedem | logstash shows those only fail on stable/juno | 18:49 |
| mriedem | not master | 18:49 |
| sdague | I just figured if someone told me which fixes needed potential fast approve, then I'd look at them | 18:49 |
| mriedem | sdague: i think master is fine | 18:49 |
| sdague | ok | 18:49 |
| mriedem | http://status.openstack.org/elastic-recheck/data/uncategorized.html in the last 24 hours is pretty clean anyway | 18:50 |
| *** patrickeast has quit IRC | 18:50 | |
| jogo | sdague: well that needs to be backported to stable first anyway | 18:50 |
| jogo | sdague: but the grenade patch needs to land first I think, so not sure | 18:51 |
| *** ChuckC has quit IRC | 18:53 | |
| mriedem | adam_g: since apevec isn't around, can you tell if this is right? https://review.openstack.org/#/q/Ib8a29258d99de75b49a9b19aef36bb99bc5fcac0,n,z | 18:53 |
| *** achanda has joined #openstack-qa | 18:53 | |
| mriedem | i assume apevec usually runs a script to do those | 18:53 |
| *** ChuckC has joined #openstack-qa | 18:54 | |
| jogo | mriedem: so those can't land due to grenade bug | 18:54 |
| *** Longgeek has joined #openstack-qa | 18:55 | |
| jogo | mriedem: and I take it those are blocking the juno gate as well? | 18:55 |
| mriedem | jogo: yeah | 18:55 |
| mriedem | at least ceilometer and keystone | 18:55 |
| mriedem | hence the fubar email :) | 18:55 |
| jogo | mriedem: why do you care about stable/juno btw? your internal IBM branches? | 18:56 |
| mriedem | jogo: that's offensive sir | 18:56 |
| mriedem | jogo: i started looking into it b/c of the bad categorization %s on the e-r page | 18:56 |
| mriedem | that showed me the grenade failure and then later the other thingy | 18:56 |
| sdague | jogo: if you queue up the list of patches I should review in order, I will in about 20 minutes | 18:56 |
| jogo | mriedem: ahhh | 18:57 |
| mriedem | jogo: i care because i care about the health and well being of openstack our shining savior hallelujah | 18:57 |
| jogo | sdague: not actually sure what patches just yet because there are multiple steps and not sure if we can turn them into one to unwedge the gate | 18:57 |
| jogo | mriedem: good, ok so you saw my two patches | 18:57 |
| mriedem | jogo: yup | 18:57 |
| * cdent sings hosanna | 18:58 | |
| jogo | mriedem: so not installing tempest should fix the issue. But doing so is a a two patch solution | 18:58 |
| jogo | mriedem: https://review.openstack.org/#/c/153702/ won't land due to tempest-lib issue | 18:58 |
| jogo | and we can' | 18:58 |
| jogo | we can't land https://review.openstack.org/#/c/153080/7 on stable/juno without the grenade change | 18:58 |
| jogo | so I am looking for ideas | 18:58 |
| mriedem | oh so we can't merge https://review.openstack.org/#/c/153080/ and then backport to stable/juno? | 18:59 |
| mriedem | jogo: can we cap versions in icehouse/juno for oslo.config or anything? | 19:00 |
| mriedem | or they are already capped and that's the problem | 19:00 |
| mriedem | b/c tempest-lib requires newer oslo.config | 19:00 |
| jogo | mriedem: no because grenade assumes javelin is installed | 19:00 |
| jogo | mriedem: yeah we can put a cap in that should fix it | 19:00 |
| jogo | well hmm not sure due to python dependencies being odd | 19:01 |
| *** cbader has quit IRC | 19:01 | |
| mriedem | in icehouse, oslo.config is capped: https://github.com/openstack/requirements/blob/stable/icehouse/global-requirements.txt#L48 | 19:01 |
| mriedem | but tempest-lib isn't | 19:01 |
| mriedem | https://github.com/openstack/requirements/blob/stable/icehouse/global-requirements.txt#L123 | 19:01 |
| mriedem | can we cap tempest-lib in icehouse | 19:01 |
| mriedem | ? | 19:01 |
| jogo | mtreinish: ^ | 19:02 |
| mriedem | oslo.config is uncapped in stable/juno https://github.com/openstack/requirements/blob/stable/juno/global-requirements.txt#L55 | 19:02 |
| jogo | I don't think so because branchless tempest | 19:02 |
| mriedem | branchless tempest can't work with older tempest-lib? | 19:02 |
| *** toure_ has joined #openstack-qa | 19:03 | |
| jogo | mriedem: AFAIK no because trunk tempest needs the latest tempest-lib | 19:03 |
| mriedem | gdi | 19:03 |
| *** zigo has joined #openstack-qa | 19:03 | |
| jogo | right | 19:03 |
| mriedem | who came up with this crazy branchless tempest idea in the first place?! :P | 19:03 |
| jogo | not installing tempest would fix it but we may need to ninja a patch or two in | 19:04 |
| mriedem | can we temporarily install tempest from a tag? | 19:04 |
| jogo | hmmm there is another solution Ithink | 19:04 |
| mriedem | on stable? | 19:05 |
| mriedem | that's what we do internally, we checkout a given corresponding tempest tag for the release and run from there | 19:05 |
| mriedem | we might miss some fixes but we either skip those or cherry pick them back | 19:05 |
| jogo | a patch to stable/icehouse devstack to stop doing reqs sync for tempest | 19:05 |
| jogo | because we are syncing stable/icehouse reqs to tempest master | 19:06 |
| jogo | which is batshit | 19:06 |
| mriedem | jogo: another possible way out, | 19:07 |
| mriedem | tempest requires tempest-lib>=0.1.0 | 19:07 |
| *** vigneshvar has joined #openstack-qa | 19:07 | |
| mriedem | tempest-lib requires oslo.config>=1.4.0 | 19:07 |
| mriedem | which would work under the cap of oslo.config in stable/icehouse: oslo.config>=1.2.0,<1.5 | 19:08 |
| mriedem | so if we could pin tempest-lib to 0.1.0 on stable/icehouse that might workaround it | 19:08 |
| mriedem | let me try that | 19:08 |
| jogo | mriedem: AFAIK we cannot because we use things from tempest-lib 0.2.0 already | 19:08 |
| mriedem | hmm, then the version req in tempest master is busted | 19:09 |
| *** dimtruck is now known as zz_dimtruck | 19:09 | |
| jogo | mriedem: see Idff90dde0607ad58d6959a8afa32f8bb4f74c6dc | 19:09 |
| mriedem | ha, shit | 19:09 |
| mtreinish | jogo: you can use tempest-lib 0.1.0 on the stable branches | 19:09 |
| mtreinish | there is an edge case with an unhandled exception which will mask real failures | 19:10 |
| jogo | mtreinish: excellent haha | 19:10 |
| mriedem | mtreinish: but once tempest on master picks up https://review.openstack.org/#/c/153708/ and bumps the req won't that break if we pin tempest-lib on stable? | 19:10 |
| mriedem | so this would be a temporary pin right | 19:10 |
| mtreinish | mriedem: well it'll change the min in the reqs file, but besides that exception the first real usage is: https://review.openstack.org/153461 | 19:11 |
| openstackgerrit | David Kranz proposed openstack/tempest: Change data_processing client to return one value and update tests https://review.openstack.org/154202 | 19:11 |
| *** Longgeek has quit IRC | 19:11 | |
| *** EmilienM|afk is now known as EmilienM | 19:12 | |
| *** melwitt has quit IRC | 19:13 | |
| mtreinish | mriedem: if it's still syncing the reqs file like it was before it'll overwrite that (which is really bad) | 19:13 |
| *** melwitt has joined #openstack-qa | 19:13 | |
| mriedem | mtreinish: ok sec | 19:13 |
| dhellmann | sdague, mtreinish : since we're capping oslo requirements in the stable branches, would it make sense to remove the stable-compat jobs from those projects? | 19:14 |
| dhellmann | jogo: let me know if I can help with reviews or implementation on this stable/juno stuff -- several oslo libs are blocked until we have that resolved | 19:14 |
| *** melwitt has quit IRC | 19:15 | |
| mtreinish | jogo: the issue there is it's still syncing the reqs file. You need to stop devstack from syncing the reqs on tempest if we're only going to do it through a venv | 19:15 |
| *** melwitt has joined #openstack-qa | 19:15 | |
| jogo | mtreinish: right, so I actually think the best way forward is to ninja a few patches in | 19:15 |
| mriedem | jogo: mtreinish: sdague: https://review.openstack.org/#/c/154203/ | 19:15 |
| jogo | but lets see if mriedem's thing works first | 19:16 |
| mriedem | so we also need a change to devstack to stop syncing reqs on which branch? | 19:16 |
| mriedem | icehouse? | 19:16 |
| jogo | dhellmann: please don't cut any new oslo libs until you are shure they are capped on stable/* branches | 19:16 |
| jogo | mriedem: doing that but still installing tempest will lead to other issues | 19:17 |
| mriedem | jogo: ok i'm watching 154203 in zuul now then | 19:17 |
| *** zz_dimtruck is now known as dimtruck | 19:17 | |
| yjiang5_ | clarkb: according to https://github.com/openstack-dev/devstack/blob/master/stack.sh#L588 ,seems the https://review.openstack.org/#/c/149288/ is not reverted? | 19:17 |
| *** BharatK has joined #openstack-qa | 19:18 | |
| jogo | mriedem: want to start an etherpad for this? | 19:18 |
| sdague | dhellmann: once it's all working, the answer is yes | 19:18 |
| mriedem | jogo: i don't know where to start | 19:18 |
| jogo | mriedem: pick an etherpad name and I will start filling it in | 19:18 |
| mtreinish | jogo: that's what I was saying before, unless there's an unhandled exception raised by an http call 0.1.0 will work even with installing tempest (at least for now) | 19:18 |
| clarkb | yjiang5_: oh wait, where there multiple die patches | 19:18 |
| mriedem | jogo: https://etherpad.openstack.org/p/wedged-stable-gate-feb-2015 ? | 19:19 |
| jogo | sure | 19:19 |
| dhellmann | jogo: yeah, we're holding off | 19:19 |
| clarkb | yjiang5_: fwiw we do have multi host devstack working right now with latest devstack as of frida | 19:19 |
| clarkb | yjiang5_: there was a similar change that was reverted and I didn't look at the diff just the commit message. | 19:20 |
| jogo | dhellmann: thanks | 19:20 |
| dhellmann | sdague: once it's all working? I'm not sure why we care about master working with old stuff now if we're going to impose lower level caps anyway | 19:20 |
| dhellmann | sdague: removing those jobs would at least unblock current dev work on those libraries | 19:20 |
| dhellmann | and if we don't release them until the caps are in place, it shouldn't introduce any other risk elsewhere | 19:21 |
| yjiang5_ | clarkb: hmm, so possibly something else wrong on my environment? Not sure and will check again. | 19:21 |
| sdague | dhellmann: oh, for stuff that has caps? | 19:21 |
| dhellmann | sdague: yeah, all oslo libs will be capped, so I'm suggesting we just stop testing all oslo libs against stable branches for now | 19:21 |
| dhellmann | s/for now/now/ | 19:21 |
| dhellmann | because I have several where I can't land patches because of the tempest-lib thing jogo is working through | 19:21 |
| dhellmann | and then our next priority will be imposing the caps, and I will have some time to help jogo with that now | 19:22 |
| sdague | dhellmann: sure, sounds fine | 19:22 |
| openstackrecheck | Console logs not available after 13:29s for gate-keystone-python27 151962,7,4c5bdc0 | 19:22 |
| clarkb | yjiang5_: I am looking at what we have now | 19:22 |
| dhellmann | sdague: ok, I'll go set that up | 19:22 |
| yjiang5_ | clarkb: thanks. | 19:23 |
| jogo | dhellmann: I am just worried about breaking stable/juno further right now | 19:23 |
| mtreinish | jogo: another not so crazy option is to remove the syncing and reqs job on tempest-lib master and change the oslo.config option to be >=1.4.0 and push a point release | 19:24 |
| clarkb | yjiang5_: http://logs.openstack.org/79/136179/5/experimental/check-tempest-dsvm-aiopcpu/6f5193e/logs/10.209.161.148-subnode/ you can see the logs and configuration for the compute node there | 19:24 |
| clarkb | yjiang5_: DATABASE_TYPE is still set | 19:24 |
| clarkb | yjiang5_: but mysql is not in the list of enabled services. So that is probably why it is working | 19:24 |
| dkranz | cdent: I noticed that the tempest telemetry client create_sample method is never called. Do you know what's up with that? | 19:24 |
| dhellmann | jogo: yeah, this would only let us make changes to master in the libs themselves, and the oslo team has agreed to not release anything until stable/juno is fixed and we have those caps in place | 19:25 |
| jogo | mtreinish: ++ honestly the syncing bumping min versions when not actually needed is hurting us | 19:25 |
| clarkb | yjiang5_: if you look in the localrc file | 19:25 |
| mtreinish | jogo: wait, tempest-lib shouldn't be using oslo.config for anything directly so it shouldn't be in the reqs file for tempest-lib... | 19:25 |
| * mtreinish checks | 19:25 | |
| *** achanda has quit IRC | 19:25 | |
| jogo | dhellmann: cool works for me then, thanks | 19:25 |
| openstackgerrit | Merged openstack-dev/devstack: Add postgresql dev package as testonly neutron dep https://review.openstack.org/153091 | 19:26 |
| yjiang5_ | clarkb: aha, thanks. So I need set the database_type in localrc. Not sure if it | 19:26 |
| openstackgerrit | Matthew Treinish proposed openstack/tempest-lib: Remove oslo.config from requirements.txt https://review.openstack.org/154210 | 19:26 |
| mtreinish | jogo: ^^^ I think that should fix things... | 19:26 |
| yjiang5_ | clarkb: pressed 'enter' too quickly. I will take this way, thanks. | 19:26 |
| clarkb | yjiang5_: and let me know if that isn't it we can keep digging | 19:27 |
| mtreinish | oh wait, nope it'll fail because of oslo.logging nm | 19:27 |
| *** achanda has joined #openstack-qa | 19:27 | |
| yjiang5_ | clarkb: sure. | 19:27 |
| jogo | mtreinish: sigh | 19:27 |
| *** ChuckC_ has joined #openstack-qa | 19:28 | |
| *** ChuckC has quit IRC | 19:29 | |
| jogo | so lets hope the pin tempest-lib on stable works | 19:30 |
| jogo | because otherwise unwinding this will take time | 19:30 |
| *** e0ne has joined #openstack-qa | 19:30 | |
| * mtreinish hides | 19:30 | |
| jogo | mtreinish: want to help? https://review.openstack.org/#/c/153702/ needs backporting to icehouse | 19:31 |
| *** jordanP has quit IRC | 19:32 | |
| mtreinish | jogo: we don't run grenade on icehouse anymore | 19:32 |
| mtreinish | so we shouldn't need to backport that any further | 19:33 |
| mriedem | right | 19:33 |
| jogo | mtreinish: good, well done then that was easy | 19:33 |
| *** sileht has quit IRC | 19:33 | |
| openstackgerrit | Wayne Okuma proposed openstack-dev/devstack: Adds elasticsearch support as a pre-req for the glance-index. https://review.openstack.org/149442 | 19:35 |
| jogo | sdague: https://review.openstack.org/#/c/151812/ and related can sue some reviews | 19:35 |
| sdague | jogo ... looking | 19:35 |
| jogo | to help us get out of this issue in general | 19:35 |
| openstackgerrit | Joe Gordon proposed openstack-dev/devstack: Stop installing tempest https://review.openstack.org/153080 | 19:37 |
| jogo | mriedem: so if your patch works, we can land my patches to stop installing tempest and that should future proof us | 19:37 |
| sdague | oh, man, I didn't realize in new gerrit ui if you highlight text before you write a comment, it leaves that annotation in there - https://review.openstack.org/#/c/151812/1/inc/python,cm | 19:37 |
| *** andreykurilin_ has joined #openstack-qa | 19:38 | |
| *** afazekas has joined #openstack-qa | 19:41 | |
| *** openstackgerrit has quit IRC | 19:41 | |
| *** sileht has joined #openstack-qa | 19:41 | |
| *** openstackgerrit has joined #openstack-qa | 19:41 | |
| jaypipes | mtreinish: on that wedge etherpad, you wrote: "It won't work because oslo.logging (which is still incubator synced in tempest-lib) requires oslo.logging". Is that what you meant to write? :) | 19:43 |
| dhellmann | jaypipes: I think mtreinish meant the incubated logging code in tempest needs oslo.config | 19:44 |
| mtreinish | jaypipes: s/oslo.logging/oslo.config | 19:44 |
| jaypipes | :) | 19:44 |
| jaypipes | gotcha. | 19:44 |
| mtreinish | sigh, fixed it | 19:44 |
| jogo | mriedem sdague: https://review.openstack.org/#/c/153080/7 should be ready for review | 19:47 |
| jogo | the backport of that should prevent this in the future | 19:47 |
| sdague | jogo: +2 | 19:48 |
| * jogo looks up and notices his train is pulling into the station and goes AFK for a bit | 19:48 | |
| sdague | jogo: I think the venv general patch is far enough off, to just move forward on this | 19:48 |
| *** bkopilov has quit IRC | 19:49 | |
| openstackrecheck | Console logs not available after 13:25s for check-tempest-dsvm-full 154140,2,0052e36 | 19:50 |
| *** abhirc has quit IRC | 19:52 | |
| *** itzikb has joined #openstack-qa | 19:53 | |
| *** Rockyg has joined #openstack-qa | 19:53 | |
| openstackgerrit | Matthew Treinish proposed openstack/tempest-lib: Switch to using oslo.log from library https://review.openstack.org/154210 | 19:54 |
| mtreinish | jogo: ^^^ this probably will just make things more complicated. But if we want to remove oslo.config from the reqs file this is the best way todo it | 19:55 |
| mtreinish | eventually we'll want to land that but I'm sure it'll make the stable branch story just more difficult | 19:55 |
| *** melwitt has quit IRC | 19:55 | |
| *** melwitt has joined #openstack-qa | 19:56 | |
| openstackgerrit | David Kranz proposed openstack/tempest: Change telemetry client to return one value and update tests https://review.openstack.org/154218 | 19:56 |
| *** afazekas has quit IRC | 19:56 | |
| itzikb | mtreinish: hi,Can you please review https://review.openstack.org/#/c/139979/? | 19:57 |
| jogo | mtreinish: well if mriedem's patch works we can unwedge stable and begin landing patches to not install tempest | 19:58 |
| jogo | which should prevent your patch from breaking things | 19:58 |
| *** afazekas has joined #openstack-qa | 19:58 | |
| *** krtaylor has joined #openstack-qa | 19:59 | |
| *** andreykurilin_ has quit IRC | 20:00 | |
| jogo | mriedem mtreinish: https://jenkins07.openstack.org/job/check-tempest-dsvm-full/25612/console | 20:00 |
| jogo | from mriedem's attempted fix https://review.openstack.org/#/c/154203/ | 20:00 |
| jogo | so that is fun | 20:01 |
| mtreinish | jogo: yeah there wasn't anything changed in that code path between 0.1.0 and 0.2.0 | 20:02 |
| cdent | dkranz: sorry i was getting some dinner, done now, what's up with which test? (trans. huh?) | 20:02 |
| mtreinish | I wonder how that got through... | 20:02 |
| jogo | mtreinish: hmm maybe stable/icehouse is broken | 20:03 |
| dkranz | cdent: It wasn't a test. Just that some methods are defined in the client but not used in any test. | 20:03 |
| * cdent looks | 20:03 | |
| *** Nic has quit IRC | 20:03 | |
| *** Nic has joined #openstack-qa | 20:04 | |
| jogo | mtreinish: https://review.openstack.org/#/c/154052/ stable/icehouse is working | 20:04 |
| mtreinish | jogo: oh the first failure is why we pushed out 0.2.0: | 20:04 |
| mtreinish | http://logs.openstack.org/03/154203/2/check/check-tempest-dsvm-full/2339922//console.html#_2015-02-09_19_45_24_984 | 20:04 |
| mtreinish | the status code differs thing I've not seen before | 20:04 |
| *** abhirc has joined #openstack-qa | 20:05 | |
| jogo | mtreinish: hmm ok, so how do we want to proceed? | 20:05 |
| mtreinish | oh really weird it's returning a 400 and the traceback says 200 != 202 | 20:05 |
| cdent | dkranz: looking at the log from the commit that added that line I'd say someone decided they were going to add a full featured telemetry client in preparation for some grand future when ceilometer would be covered by tempest | 20:06 |
| cdent | but then plans changed, but the code did not | 20:06 |
| dkranz | cdent: ok, so what is the grand plan now? | 20:06 |
| *** afazekas has quit IRC | 20:07 | |
| mtreinish | jogo: sigh we can't use 0.1.0 anymore it looks like tempest has landed changes recently that require 0.2.0 | 20:07 |
| *** abhirc has quit IRC | 20:07 | |
| *** afazekas has joined #openstack-qa | 20:07 | |
| mriedem | jogo: mtreinish: e-r said it was bug 1352498 | 20:07 |
| openstack | bug 1352498 in Cinder "Tempest fails spectacularly with "VolumeType c39f18bf-2408-4377-98f6-ef2839d6f70e deletion failed, VolumeType in use."" [Undecided,New] https://launchpad.net/bugs/1352498 | 20:07 |
| mriedem | on stable/icehouse | 20:07 |
| *** catohornet1 has joined #openstack-qa | 20:07 | |
| jogo | mtreinish: can we temporarily pin tempest on stable/icehouse? | 20:08 |
| cdent | Well in the 8 months or so since I've known anything about the existence of tempest the plan on whether or not _any_ ceilometer was going to be run in tempest tests has changed at least twice and I'm not currently certain what the state is because I recently saw a patchset which suggested perhaps some tests were going to be allowed in. | 20:08 |
| mtreinish | mriedem: oh, I was going just by the commit messages, let me check the code of the commits I was looking at | 20:08 |
| mtreinish | jogo: it might not be needed | 20:08 |
| cdent | In other words I don't know. Are you trying to accomplish something dkranz, or just looking to clean things up? | 20:08 |
| dkranz | cdent: I was changing the telemetry client to return one value in prep for moving to tempest-lib | 20:09 |
| jogo | mtreinish: hmm did all the jobs fail the same? | 20:09 |
| dkranz | cdent: https://review.openstack.org/#/c/154218/ | 20:09 |
| mriedem | jogo: yeah | 20:09 |
| dkranz | cdent: I was also adding the response code checking in the client which had not been done | 20:09 |
| *** aysyd has quit IRC | 20:10 | |
| mriedem | Details: Unexpected http success status code 200, The expected status code is 202 | 20:10 |
| mtreinish | jogo: yeah the commits I thought required 0.2.0 should still work on 0.1.0 | 20:10 |
| mriedem | jogo: mtreinish: did cinder change the response code at some point? | 20:10 |
| jogo | hmm therea re a lot of volume failures | 20:10 |
| mtreinish | mriedem: they shouldn't be able to | 20:10 |
| mtreinish | because of branchless tempest... | 20:10 |
| jogo | here are the three outputs: | 20:11 |
| jogo | http://logs.openstack.org/03/154203/2/check/check-tempest-dsvm-full/2339922/logs/testr_results.html.gz | 20:11 |
| jogo | http://logs.openstack.org/03/154203/2/check/check-tempest-dsvm-postgres-full/dc9f6b6/logs/testr_results.html.gz | 20:11 |
| jogo | http://logs.openstack.org/03/154203/2/check/check-tempest-dsvm-neutron/8b79a28/logs/testr_results.html.gz | 20:11 |
| jogo | but that isn't full so ignoringit | 20:11 |
| *** catohornet1 has left #openstack-qa | 20:11 | |
| cdent | ah, okay, dkranz I now remember this coming up back when I made some adjustments and at that time there was discussion about the single/double response code that I guess got dropped for "later": https://review.openstack.org/#/c/115971/ | 20:11 |
| cdent | If, in fact, we want templest-lib to have a working telemetry client then yeah, create_sample needs to be in it. | 20:12 |
| mriedem | mtreinish: jogo: https://github.com/openstack/tempest/commit/d9df38c867d6447dc41ad3fa7bc8b3d732e751b0 | 20:12 |
| dkranz | cdent: That's fine. My patch just changes it with the others. | 20:13 |
| mriedem | mtreinish: jogo: https://review.openstack.org/#/c/114783/ didn't run against icehouse from what i can see | 20:13 |
| dkranz | cdent: thanks for the info | 20:13 |
| mriedem | i guess that would have been juno so old grenade would be icehouse | 20:13 |
| jogo | mriedem: but icehouse jobs aren't failing everywhere | 20:13 |
| cdent | You're welcome. Took me a while to decompress the index for that stuff out the back of my brain. | 20:13 |
| mtreinish | mriedem: it ran icehouse on check | 20:15 |
| *** scottda has joined #openstack-qa | 20:15 | |
| mtreinish | and that just moves the check out of the test code and into the clients | 20:15 |
| jdg | jogo: what's the scoop? | 20:16 |
| jogo | jdg: ^ | 20:16 |
| *** mtanino has joined #openstack-qa | 20:17 | |
| jogo | jdg: trying to figure out why this patch failed https://review.openstack.org/#/c/154203/ | 20:17 |
| jdg | looking... | 20:17 |
| openstackgerrit | David Kranz proposed openstack/tempest: Change flavors client to return one value and update tests https://review.openstack.org/154114 | 20:18 |
| jdg | jogo: weird, the VG was never created | 20:18 |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: update plugin doc https://review.openstack.org/154224 | 20:18 |
| mriedem | jdg: well, the bug is old https://bugs.launchpad.net/bugs/1352498 | 20:18 |
| openstack | Launchpad bug 1352498 in Cinder "Tempest fails spectacularly with "VolumeType c39f18bf-2408-4377-98f6-ef2839d6f70e deletion failed, VolumeType in use."" [Undecided,New] | 20:18 |
| mriedem | jdg: the weird thing is it failed the same in 3 different icehouse jobs | 20:18 |
| jdg | mriedem: no... I mean the Volume Group doesn't exist | 20:19 |
| mriedem | are we missing devstack changes on stable/icehouse? | 20:19 |
| jogo | jdg: ohhh so devstack is not setting something up correctly? | 20:19 |
| jdg | mriedem: lemme look again, I could be wrong | 20:19 |
| jogo | jdg: http://logs.openstack.org/03/154203/2/check/check-tempest-dsvm-postgres-full/dc9f6b6/logs/devstacklog.txt.gz#_2015-02-09_19_43_34_974 | 20:20 |
| jdg | mriedem: jogo ok I was wrong | 20:20 |
| mriedem | well, e-r said all 3 jobs failed with the same bug | 20:20 |
| jogo | jdg: is http://logs.openstack.org/03/154203/2/check/check-tempest-dsvm-postgres-full/dc9f6b6/logs/devstacklog.txt.gz#_2015-02-09_19_43_35_098 bad? | 20:20 |
| jogo | mriedem: lets recheck another stable/icehouse patch | 20:21 |
| jogo | to sort out of this is your patch or unrelated | 20:21 |
| jdg | jogo: mriedem can you trap one of those on failure? | 20:22 |
| jdg | jogo: mriedem I'd be curious to see what's deployed on the VG | 20:22 |
| mriedem | jogo: yeah looking that up now | 20:22 |
| jogo | mriedem: https://review.openstack.org/#/c/143110/3 here is a nice unrelated patch | 20:22 |
| jdg | jogo: mriedem ie like "lvs before and after the run" | 20:22 |
| jogo | mriedem: just ran recheck | 20:22 |
| jogo | jdg: fungi or clarkb can do that if you think that is the issue. | 20:22 |
| jogo | jdg: but we try not to as do that as much as possible | 20:23 |
| mriedem | jogo: you rechecked what? | 20:23 |
| jogo | mriedem: some random stable/icehouse patch | 20:23 |
| mriedem | jogo: ah ok | 20:23 |
| mriedem | i was going to check logstash | 20:23 |
| jogo | making sure the failure is related to your patch | 20:23 |
| jogo | oh good idea | 20:23 |
| jdg | mriedem: jogo I'm not sure why but the VG size appears to be just slightly off | 20:23 |
| jdg | I don't know what's going on with the taskflow calls :( | 20:24 |
| *** shashankhegde has quit IRC | 20:24 | |
| clarkb | jogo: iirc devstack-gate names an lv properly so that cinder just uses it | 20:24 |
| mriedem | jogo: so far just my icehouse change http://goo.gl/2m5yOD | 20:25 |
| fungi | jdg: in many cases you can so some inspection just by proposing throwaway tweaks to tempest, devstack or devstack-gate to spit extra details into the logs | 20:25 |
| clarkb | jdg: previously we set up a volume then dumped a file that was then mounted back in somewhere that cinder used and that was causing more problems that just using a 24GB lv directly | 20:25 |
| mriedem | jogo: the bug in tempest-lib is masking the failure | 20:25 |
| *** smcginnis has joined #openstack-qa | 20:25 | |
| mtreinish | jogo, mriedem: looking through the logs at the first failure on the pin patch full job (which is not cinder related) I see: http://logs.openstack.org/03/154203/2/check/check-tempest-dsvm-full/2339922/logs/screen-n-api.txt.gz?level=TRACE | 20:25 |
| jdg | fungi: good idea.. I'll do try that if I don't find what I'm looking for in a minute here | 20:25 |
| mriedem | mtreinish: fun | 20:26 |
| jdg | clarkb: so I remember some of that, but I don't remember where things left off | 20:26 |
| jdg | clarkb: I *thought* we were back to just creating a loop device? | 20:26 |
| jogo | mtreinish: eww | 20:26 |
| clarkb | jdg: pretty sure thats what we did in the past | 20:26 |
| clarkb | jdg: I will dig up where the magic happens | 20:27 |
| fungi | _soon_ (*cough* cross repo dependencies in zuul *cough*) you can even propose a throwaway change to devstack and then set your patch to some other project to depends-on that and see the results too | 20:27 |
| mriedem | jogo: mtreinish: http://goo.gl/7WHbJQ | 20:27 |
| jdg | clarkb: yeah, sorry I lost track of that a while back | 20:27 |
| mriedem | jogo: mtreinish: message:"ResourceClosedError: This result object does not return rows. It has been closed automatically." AND tags:"screen-n-api.txt" | 20:27 |
| mriedem | has 21 hits in 7 days, stable/icehouse and master | 20:27 |
| clarkb | jdg: https://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/functions.sh#n241 | 20:27 |
| jogo | jdg: mriedem looks like a oslo.db bug? | 20:27 |
| clarkb | jdg: line 263 is he important one aiui | 20:28 |
| mtreinish | mriedem: heh, just what we needed | 20:28 |
| mriedem | mtreinish: jogo: yeah, not sure, checking to see if we cap oslo.db in icehouse | 20:28 |
| mtreinish | another bug trying to unwind all of this... | 20:28 |
| mtreinish | mriedem: we probably do | 20:28 |
| mriedem | mtreinish: jogo: oslo.db isn't in icehouse | 20:28 |
| mriedem | https://github.com/openstack/requirements/blob/stable/icehouse/global-requirements.txt | 20:29 |
| mriedem | so that would be oslo-incubator/nova/sqlalchemy | 20:29 |
| dhellmann | yeah, oslo.db started in juno | 20:29 |
| mtreinish | mriedem: oh, yeah I don't think it was ready 1 yr ago | 20:29 |
| mriedem | message:"ResourceClosedError: This result object does not return rows. It has been closed automatically." AND tags:"screen-n-api.txt" AND build_branch:"master" | 20:30 |
| mriedem | only hits on check queue | 20:30 |
| mriedem | so probably code bugs | 20:30 |
| clarkb | jeblair: these are some large zuul changes :) | 20:30 |
| mriedem | jogo: mtreinish: i'm rechecking https://review.openstack.org/#/c/154203/ on the cinder bug | 20:30 |
| jogo | mriedem mtreinish: so even if we resolve the volume failures we still have AttributeError: 'module' object has no attribute 'Forbidden' | 20:31 |
| jogo | so I don't think we have a plan yet | 20:31 |
| mriedem | jogo: if i remember correctly though, that's just a thing in tempest-lib that surfaces when a test fails | 20:31 |
| clarkb | jeblair: will get back to review after lunch. Any chance I can interest you in the stack that starts at https://review.openstack.org/#/c/153093/ ? | 20:31 |
| mriedem | if the tests pass then we shouldn't see that | 20:31 |
| mriedem | jogo: ^ | 20:31 |
| mriedem | mtreinish would have to confirm | 20:31 |
| clarkb | gah wrong window | 20:31 |
| jdg | mriedem: jogo so there are a number of failures cascading there before the cinder volume create failure | 20:31 |
| jogo | mriedem: odds of 3 jobs failing for the same thing being random | 20:32 |
| jogo | is doubtful | 20:32 |
| jdg | even setUpClass fails which I haven't seen before | 20:32 |
| mriedem | jogo: yeah | 20:32 |
| jogo | mriedem: so if https://jenkins03.openstack.org/job/check-tempest-dsvm-full/25883/console fails (random icehouse patch) | 20:33 |
| jogo | then it means we have yet another blocker | 20:33 |
| mriedem | jogo: yeah, was wondering when the last icehouse change was checked | 20:33 |
| jogo | mriedem: 12 hours ago or so | 20:33 |
| *** tsekiyama has joined #openstack-qa | 20:33 | |
| jogo | but that is enough time for new $version of $thing to break us | 20:33 |
| mriedem | yeah | 20:34 |
| jdg | wow.. quota errors too | 20:34 |
| jogo | last success 2015-02-09T10:27:52.264+00:00 | 20:34 |
| jogo | jdg: those are normal, cinder logging makes me want to poke my eyes out with a rusty nail | 20:35 |
| jdg | jogo: fair enough, point taken | 20:35 |
| mriedem | jdg: yeah those quota warnings have been around forever | 20:35 |
| mriedem | the deprecation warnings | 20:35 |
| mriedem | at least horizon | 20:35 |
| jogo | jdg: you should fix 'em! | 20:35 |
| jdg | mriedem: jogo well, they should be unhandled traces | 20:35 |
| jdg | :( | 20:35 |
| mriedem | *at least havana i meant | 20:36 |
| cdent | dkranz: do you have a pointer to some history on why the tempest clients are better when they only return one response? I ask because it is often the headers I want to inspect when making tests. | 20:36 |
| jdg | mriedem: jogo so last commit from Ciner was Jan 14 FWIW | 20:36 |
| *** toure_ has quit IRC | 20:36 | |
| *** smcginnis has left #openstack-qa | 20:36 | |
| jdg | jogo: yes, I just added it to my list | 20:37 |
| jogo | jdg: dependencies change all the time and break stuff | 20:37 |
| jdg | jogo: I've picked through some of them already | 20:37 |
| *** Guest46399 has joined #openstack-qa | 20:37 | |
| jdg | jogo: right, I was just looking to see if we broke something on the Cinder side explicitly | 20:37 |
| jogo | mriedem: so https://jenkins03.openstack.org/job/check-tempest-dsvm-full/25883/console is running | 20:37 |
| jogo | we will know whats up soon | 20:37 |
| mriedem | jogo: yeah i'm watching it on 2 screens | 20:38 |
| mriedem | console and zuul | 20:38 |
| jogo | mriedem: if icehouse is not broken on its own, we need a new plan of attack. | 20:38 |
| cdent | oh, I see dkranz, it's a magic class. Ugh. | 20:38 |
| mriedem | jogo: can we somehow trick icehouse with a trojan horse and once night falls we pounce?! | 20:39 |
| mriedem | you win icehouse, please accept this gift | 20:39 |
| jdg | LOL | 20:39 |
| jogo | mriedem: if only | 20:39 |
| *** Rockyg has quit IRC | 20:40 | |
| *** afazekas has quit IRC | 20:44 | |
| *** salv-orlando has joined #openstack-qa | 20:44 | |
| dkranz | cdent: The history is that almost all tempest code ignores the response except for checking the response code which is handled in the client | 20:47 |
| *** shashankhegde has joined #openstack-qa | 20:47 | |
| * cdent is glad he wrote gabbi | 20:47 | |
| dkranz | cdent: right, that is a different use case | 20:48 |
| cdent | it's exposing all kinds of ways in which headers are being misused and abused | 20:48 |
| cdent | I dunno: You can't test an HTTP API without testing the headers. | 20:48 |
| dkranz | cdent: a fair number of tempest tests could be replaced by something like gabbi | 20:48 |
| dkranz | cdent: but many of them are more functional in nature | 20:48 |
| cdent | pretty much all the gabbi tests I've written so far would be described as functional because they use a database fixture | 20:49 |
| cdent | s/database/storage/ | 20:49 |
| dkranz | cdent: where you have to make additional api calls to validate the result | 20:49 |
| cdent | Sounds like a bad HTTP API then ;) | 20:49 |
| cdent | one you can't trust | 20:50 |
| * cdent is mostly kidding | 20:50 | |
| *** afazekas has joined #openstack-qa | 20:50 | |
| dkranz | cdent: I know. We need both tests that validate the api and that validate deeper issues. | 20:50 |
| *** andreaf_ has quit IRC | 20:51 | |
| cdent | If there's interest + some guidance I can make a stab at starting the process of translating some of the more boring API tests to somewhere else. Is the hope that they would become in-tree? | 20:51 |
| dkranz | cdent: Yes, exactly | 20:51 |
| dkranz | cdent: those tests are more akin to unit tests for the api IMO | 20:52 |
| * cdent nods | 20:52 | |
| dkranz | cdent: and I would include negative tests in that bucket | 20:52 |
| jogo | mriedem: hmm didn't see any package (pip or dpkg) changes between good and bad changes | 20:53 |
| jogo | mriedem: it may be your patch somewhow | 20:53 |
| mriedem | jogo: if it is, it's something that's changed between tempest-lib 0.1.0 and 0.2.0, or tempest's usage of tempest-lib | 20:54 |
| cdent | dkranz, Is there a heuristic for deciding if a test should or should not be in tree? I've heard different perspectives on the "if it needs another project" aspect. | 20:54 |
| dkranz | cdent: one minute, let me find the etherpad about this | 20:55 |
| jogo | mriedem: not sure, can we confirm by pinning master tempest to tempest-lib 0.1.0? | 20:55 |
| *** BharatK has quit IRC | 20:55 | |
| jogo | and even if it did change how do we move forward etc. | 20:55 |
| cdent | (my negative testing of ceilometer last week revealed a _huge_ problem in wsme ....) | 20:55 |
| cdent | cool thanks dkranz | 20:55 |
| mtreinish | mriedem: the key change between 0.1.0 and 0.2.0 was adding a missing exception | 20:55 |
| mtreinish | which tempest was using | 20:55 |
| mriedem | jogo: not sure, have to figure out what's breaking first | 20:55 |
| jogo | damn https://jenkins03.openstack.org/job/check-tempest-dsvm-full/25883/console passed with flying colors | 20:56 |
| *** malini has quit IRC | 20:56 | |
| mtreinish | mriedem: it's the Forbidden UnAuthorized thing we were looking at before | 20:56 |
| mriedem | jogo: there are still 2 other jobs running on it | 20:56 |
| mriedem | mtreinish: yeah | 20:56 |
| *** Longgeek has joined #openstack-qa | 20:56 | |
| mtreinish | mriedem: there were other changes but I don't think any of them should be used anywhere yet | 20:57 |
| jogo | I don't think pinning tempest-lib on stable is a good way forward at this point | 20:57 |
| jogo | the other candidates that I can think of are: | 20:57 |
| mriedem | mtreinish: http://paste.openstack.org/show/170190/ | 20:57 |
| jogo | * Ninja in some of my patches (this would worry me a bit) | 20:58 |
| jogo | * temp pin tempest on stable branches | 20:58 |
| jogo | * EOL icehouse | 20:58 |
| cdent | +1 to ending life. all of it. | 20:59 |
| mtreinish | jogo: I sent the email about the EOL: http://lists.openstack.org/pipermail/openstack-dev/2015-February/056366.html | 20:59 |
| jdg | jogo: cdent wow... that's bold | 20:59 |
| jdg | :) | 21:00 |
| mriedem | i was trying to think of a witty comeback | 21:00 |
| jogo | mtreinish: I actually think that may be the best step forward at this time | 21:00 |
| jogo | although realistically we cannot EOL it *today*. maybe tomorrow (Tuesday) | 21:01 |
| jogo | which means want to just revisit the breakage tomorrow? | 21:01 |
| mriedem | idk, i still feel like killing stable/icehouse gets branchless tempest off too easy | 21:01 |
| dkranz | cdent: I led this cross-project session in Paris https://etherpad.openstack.org/p/kilo-crossproject-move-func-tests-to-projects and there is also related https://etherpad.openstack.org/p/kilo-summit-tempest-scope-in-the-brave-new-world | 21:01 |
| mtreinish | jogo: we probably should put it on the cross-project meeting agenda for tomorrow | 21:02 |
| mriedem | +1 to that | 21:02 |
| jogo | mtreinish: ohh good idea | 21:02 |
| mriedem | since ttx will be there | 21:02 |
| jogo | mriedem: well we have the path forward, its just blocked by all of this | 21:02 |
| dkranz | cdent: No firm conclusions on many issues. The stuff that Sean pushed is also related | 21:02 |
| mriedem | jogo: oh you mean the changes on stable/juno | 21:02 |
| mriedem | forgot about those... | 21:02 |
| jogo | mriedem: my patches https://review.openstack.org/#/q/I78d51f04ed01da4ce8aa0e127be028f969d3b4f8,n,z | 21:03 |
| mriedem | yeah | 21:03 |
| dkranz | cdent: but the bottom line is that it is still unclear exactly what tests should be in tempest in the long run | 21:03 |
| jdg | jogo: mriedem so seriously, just 'drop' support | 21:03 |
| mriedem | bknudson: ^ maybe you want to read that ML thread :) | 21:03 |
| jogo | but they are circular ... :/ | 21:03 |
| jdg | jogo: mriedem that seems highly lame and exactly the kind of thing to make people HATE us | 21:03 |
| jdg | jogo: mriedem as much as pinning sucks, that seems like a much more reasonable/sane approach | 21:03 |
| jogo | hmm maybe not all of 'em https://review.openstack.org/#/c/154217/ | 21:04 |
| mriedem | jdg: what's lame? | 21:04 |
| jogo | different failure | 21:04 |
| cdent | thank dkranz | 21:04 |
| cdent | s | 21:04 |
| jdg | mriedem: your post regarding just EOL Icehouse now? | 21:04 |
| jdg | mriedem: I'm wondering if I'm missing something? | 21:04 |
| mriedem | jdg: that's mtreinish | 21:04 |
| mriedem | jogo: that looks like the ec2 bug | 21:04 |
| jdg | mriedem: yes, but | 21:04 |
| *** weshay has quit IRC | 21:04 | |
| mriedem | is ec2 capped on stable/icehouse | 21:04 |
| jdg | mriedem: DOHHH | 21:04 |
| jdg | mriedem: sorry... not the first time I've done that | 21:04 |
| mtreinish | heh, it happens again :) | 21:05 |
| jdg | sone of a!! | 21:05 |
| jdg | mtreinish: mriedem one of you change your nic quick!! | 21:05 |
| jogo | but we didn't see that moments ago | 21:05 |
| mriedem | jogo: boto is capped on stable/icehouse... https://github.com/openstack/requirements/blob/stable/icehouse/global-requirements.txt#L6 | 21:05 |
| jogo | WTF | 21:05 |
| jogo | mriedem: what does that mean in this context? | 21:05 |
| mriedem | jogo: your stable/icehouse change is failing on boto failures | 21:06 |
| mtreinish | jdg: for a casual nick friday a while back we switches nicks... | 21:06 |
| mriedem | jogo: when we hit that on master we capped boto | 21:06 |
| jdg | LOL | 21:06 |
| mriedem | jogo: then fixed and uncapped, | 21:06 |
| jdg | mtreinish: so what you're saying is I'm not the only one that does it | 21:06 |
| mriedem | so i'm not sure why this is failing on stable/icehouse | 21:06 |
| jogo | mriedem: ohh I know | 21:06 |
| jogo | tempest/stable prob doesn't have the pin in it | 21:06 |
| jogo | and my patch stops global sync | 21:07 |
| mtreinish | jogo: it would if the sync is run... | 21:07 |
| jogo | err damn | 21:07 |
| jogo | tempest master (branchless) | 21:07 |
| mriedem | jogo: ah yup | 21:07 |
| jogo | ugggg | 21:07 |
| mriedem | branchless tempest again ftw :) | 21:07 |
| jogo | so yeah, fuck | 21:08 |
| jogo | so we go to a venv we break EC2, if we don't venv javelin on icehouse breaks | 21:09 |
| mtreinish | jogo: wait I missed something. What!? | 21:09 |
| *** Longgeek has quit IRC | 21:10 | |
| * dhellmann tries to catch up with the latest | 21:10 | |
| dhellmann | jogo: virtualenvs break ec2? | 21:10 |
| mtreinish | dhellmann: my guess is using latest boto with stable code (which installing master tempest from a venv would do) breaks the tests | 21:11 |
| mtreinish | but I missed it too | 21:11 |
| mriedem | https://review.openstack.org/#/c/154217/1 | 21:11 |
| mriedem | that stops the reqs sync if i understand it, | 21:11 |
| mriedem | but tempest on master has uncapped boto | 21:11 |
| mriedem | but the code in nova can't handle uncapped boto | 21:11 |
| mriedem | that's why boto is capped in global-reqs on icehouse | 21:11 |
| jogo | mriedem: correct | 21:11 |
| mtreinish | ffs | 21:12 |
| mriedem | heh | 21:12 |
| *** andreaf_ has joined #openstack-qa | 21:12 | |
| dhellmann | does master tempest work with the capped boto against icehouse? | 21:12 |
| *** andreaf has quit IRC | 21:12 | |
| dhellmann | could we have the tempest job get the boto requirement from the nova requirements list? | 21:12 |
| *** andreaf_ is now known as andreaf | 21:12 | |
| mriedem | dhellmann: master tempest works with capped boto on icehouse yes b/c boto is capped | 21:13 |
| *** andreaf_ has joined #openstack-qa | 21:13 | |
| mriedem | idk about the latter question | 21:13 |
| dhellmann | ok, I wasn't sure if the internal apis of boto had changed, or just the way it interacts with the rest api | 21:13 |
| mriedem | latest boto started doing new auth signature stuff that ec2 in nova wasn't handling | 21:13 |
| dhellmann | right | 21:13 |
| jogo | dhellmann: yes master tempest works with capped boto | 21:13 |
| dhellmann | ok | 21:13 |
| dhellmann | ok, so how hard would it be to make the tempest job pull the boto requirement from whatever nova is using? | 21:14 |
| mriedem | it does on master, but we capped boto on icehouse/juno | 21:14 |
| mriedem | which then dovetails into the topic of removing ec2 from nova :) | 21:14 |
| jogo | as for the second idea, getting boto version from nova-reqs | 21:14 |
| dhellmann | I assume nova has a boto dependency, but maybe not | 21:14 |
| jogo | that is pretty ugly | 21:14 |
| *** afazekas has quit IRC | 21:14 | |
| mriedem | dhellmann: it does, it's capped from stable global-reqs | 21:14 |
| dhellmann | well, it is, but we're trying to build a job that tests the icehouse api and if we know the boto version needs to be different for those tests, then... | 21:14 |
| dhellmann | so we need to stop thinking about this as installing a thing and its dependencies and start thinking about installing a set of packages to run the test job we're designing, right? | 21:15 |
| mriedem | dhellmann: https://github.com/openstack/nova/blob/stable/icehouse/requirements.txt#L6 | 21:15 |
| dhellmann | cool | 21:15 |
| dhellmann | so theoretically at least we could look there to figure out which boto to use for tempest | 21:15 |
| dhellmann | and if tempest's boto requirement was unbounded, or we update the requirement before installing tempest, there would be no runtime requirement check failure | 21:16 |
| *** dmellado has quit IRC | 21:16 | |
| dhellmann | no pkg_resources error, that is | 21:16 |
| *** dmellado has joined #openstack-qa | 21:16 | |
| jogo | so this is a big issue with branchless tempest | 21:16 |
| dhellmann | indeed | 21:16 |
| jogo | can we just cap boto on master for now and revisit later (I.E. never) | 21:16 |
| mriedem | ha | 21:17 |
| jogo | because saying tempest needs different versions of libs for different branches is fugly | 21:17 |
| mtreinish | jogo: I'm fine with that | 21:17 |
| dhellmann | that seems like another option -- didn't we do that for a while? I see some reqs changes to undo the cap | 21:17 |
| mriedem | but it's boto today and something else tempest requires tomorrow | 21:17 |
| mriedem | we uncapped boto on master b/c the nova code was fixed | 21:17 |
| mriedem | we could backport the nova changes down to stable/icehouse... | 21:17 |
| mriedem | but that's dumb b/c boto is capped in nova on icehouse | 21:17 |
| dhellmann | yeah | 21:18 |
| mriedem | wouldn't be a real world issue | 21:18 |
| dhellmann | sure it is, but the way we're telling people to solve it is by using an older version of boto | 21:18 |
| dhellmann | so if that's what we're saying, we should do that ourselves here, too | 21:18 |
| jogo | don't know of any good answer to this | 21:18 |
| dhellmann | I see 3 options: we could turn off those tests, fix nova to work with newer boto, or make the tests run with older boto | 21:19 |
| mriedem | hmm | 21:20 |
| dhellmann | as mriedem points out, this is just today and there may be something other than boto in the future (although it's less likely than if we were installing everything together in one site-packages) | 21:20 |
| mriedem | can we skip the boto tests in tepmest if boto is < 2.35? | 21:20 |
| mriedem | but that doesn't fix jogo's patch | 21:20 |
| dhellmann | no, that wouldn't help | 21:20 |
| *** achanda has quit IRC | 21:20 | |
| dhellmann | you need to skip if the version of nova you're talking to is whatever version doesn't include the fix for the new boto | 21:20 |
| mtreinish | mriedem: we could, but wouldn't help because the tests aren't broken with boto <2.35 | 21:20 |
| *** ChuckC_ has quit IRC | 21:20 | |
| mriedem | mtreinish: right | 21:21 |
| jogo | what about just skip EC2 tests on stable/icehouse | 21:21 |
| mriedem | dhellmann: yeah, +1 to that | 21:21 |
| mtreinish | dhellmann: +1, but we can't do that in tempest because that's not exposed info from nova | 21:21 |
| *** ChuckC_ has joined #openstack-qa | 21:21 | |
| dhellmann | can you tell whether the tests will passed based on the nova api version? or did the fix not cause a version bump? | 21:21 |
| *** ChuckC_ is now known as ChuckC | 21:21 | |
| dhellmann | the api version isn't exposed? | 21:21 |
| mriedem | mtreinish: we can't get that info from the cli? | 21:21 |
| mriedem | nova-manage --version or whatever | 21:21 |
| mriedem | that should return 2015.1 for kilo | 21:21 |
| dhellmann | mriedem: not all tempest runs will have access to nova-manage | 21:21 |
| mriedem | le sigh | 21:22 |
| dhellmann | but I thought the rest api had an endpoint to get the API version | 21:22 |
| jogo | dhellmann: this isn't the openstack API | 21:22 |
| mtreinish | dhellmann: i don't think nova's boto does | 21:22 |
| jogo | its EC2 | 21:22 |
| jogo | so that is a different version thing | 21:22 |
| dhellmann | jogo: sure, but you could *ask* the openstack api first | 21:22 |
| dhellmann | is the version different between the ec2 that works with old and new botos? | 21:23 |
| jogo | that just sounds wrong, and we don't have microversions yet anyway etc. | 21:23 |
| dhellmann | how is it wrong? | 21:23 |
| jogo | we can always backport the fix to icehouse | 21:23 |
| jogo | dhellmann: if using the EC2 API you shouldn't have to ever look at the OpenStack API | 21:23 |
| dhellmann | jogo: it's a damned test, don't be puritanical | 21:24 |
| mtreinish | mriedem: it wouldn't be hard to change the icehouse job to just exclude the tempest boto tests | 21:24 |
| jogo | dhellmann: oh good word ;) | 21:24 |
| dhellmann | :-) | 21:24 |
| mtreinish | which would be how to skip it | 21:24 |
| dhellmann | mtreinish: that sounds like an easy solution | 21:24 |
| jogo | mtreinish: +++ do it | 21:24 |
| jogo | mriedem: where is the nova patch for boto? | 21:25 |
| mtreinish | ugh, now I have to look where to do that... | 21:25 |
| mriedem | jogo: for the fix? | 21:25 |
| mriedem | mtreinish: i think it's d-g | 21:25 |
| mtreinish | yeah it is | 21:25 |
| *** dmellado has quit IRC | 21:25 | |
| jogo | mriedem: yeah, maybe its a easy backport? | 21:25 |
| mriedem | jogo: it's 2 fixes, you'd have to squash them | 21:25 |
| mriedem | first: https://github.com/openstack/nova/commit/f7b1af9e13df728d086047f6763bd98cb2cad1b2 | 21:25 |
| mriedem | second: https://github.com/openstack/nova/commit/fb588f87db65f28823f9e07a9900c34c7b3576a2 | 21:25 |
| jogo | which is required for keystone to calculate | 21:27 |
| jogo | the correct v4 signature when validating the request. | 21:27 |
| jogo | when did keystone add that? | 21:27 |
| * jogo votes drop EC2 tests on icehouse | 21:27 | |
| morganfainberg | keystone add what? | 21:27 |
| *** nicodemos has quit IRC | 21:27 | |
| jogo | https://github.com/openstack/nova/commit/f7b1af9e13df728d086047f6763bd98cb2cad1b2 | 21:27 |
| morganfainberg | oh this the ec2 signature versions are wrong issue? | 21:28 |
| mriedem | yeah | 21:28 |
| mriedem | does keystone support ec2 v4 in icehouse? | 21:28 |
| morganfainberg | yay for incompatible changes :( | 21:28 |
| morganfainberg | mriedem, uh, might need a fix to do so | 21:28 |
| jogo | morganfainberg: TL;DR EC2 tests in icehouse are blocking us from moving away from installing tempest | 21:28 |
| morganfainberg | there is some weirdness in the signature validator that something boto changed somewhere | 21:28 |
| jogo | morganfainberg: not asking keystone to backport | 21:28 |
| morganfainberg | caused a mismatch | 21:28 |
| jogo | morganfainberg: I think we are moving to dropping the EC2 tests on icehouse | 21:29 |
| morganfainberg | jogo, ok then i support removing tests >.> | 21:29 |
| jogo | mtreinish: is working on a patch I think | 21:29 |
| morganfainberg | jogo, i mean, if it warrants a backport i'm happy to consider | 21:29 |
| morganfainberg | jogo, just saying i know there was some issue around this at oine point | 21:29 |
| * jogo hopes mtreinish's skip EC2 on icehouse patch works | 21:30 | |
| mtreinish | jogo: it'll be a project config patch probably looking at the d-g code | 21:31 |
| mriedem | the actual ec2 sig stuff happens in keystonemiddleware | 21:32 |
| mtreinish | so we won't test it upfront | 21:32 |
| mriedem | or keystoneclinet rather | 21:32 |
| morganfainberg | that makes fixing in icehouse painful | 21:32 |
| morganfainberg | really painful | 21:32 |
| dhellmann | mtreinish: if you do it in project-config, won't you have to touch a bunch of jobs separately? | 21:32 |
| morganfainberg | it could be/is new requirements | 21:32 |
| mriedem | well, python-keystoneclient>=0.7.0,<0.12 | 21:32 |
| * dhellmann is still sorting out how these tempest jobs work | 21:33 | |
| cdent | good luck everyone | 21:33 |
| * cdent waves | 21:33 | |
| *** cdent has quit IRC | 21:33 | |
| mtreinish | dhellmann: the job definitions just need a env variable to set a custom testr regex when running with icehouse | 21:33 |
| mriedem | morganfainberg: keystone on icehouse requires keystoneclient < 0.12 which has the v4 auth support https://github.com/openstack/python-keystoneclient/blob/0.11.2/keystoneclient/contrib/ec2/utils.py#L40 | 21:34 |
| mriedem | so i don't think keystone would be an issue | 21:34 |
| mriedem | we'd just have to backport the nova changes | 21:34 |
| mtreinish | dhellmann: the other way to do it is add a condition in d-g if the branch is icehouse and set that regex | 21:34 |
| mriedem | boto is also capped on stable/juno so we'd have to backport the nova ec2 changes to juno and then icehouse | 21:34 |
| dhellmann | mtreinish: yeah, my point was that doing it in d-g would be one localized change, but I guess doing it in p-c means it's tied to the job definition more closely, even if we have to change more than one job definition | 21:35 |
| mtreinish | dhellmann: the 2nd way is probably less LOC but seemed weird because we already have the mechanism to set custom regexes in place | 21:35 |
| dhellmann | yeah | 21:35 |
| mriedem | how is it already almost 4 | 21:37 |
| *** ChuckC has quit IRC | 21:38 | |
| dhellmann | mtreinish: so something like DEVSTACK_GATE_TEMPEST_REGEX='(?!ec2)' maybe? | 21:38 |
| *** ChuckC has joined #openstack-qa | 21:39 | |
| mtreinish | dhellmann: well I was going to use: '(?!.*\[.*\bslow\b.*\])(^tempest\.(api|scenario|cli))' | 21:40 |
| mtreinish | which is the full job's regex without the thirdparty tests | 21:40 |
| dhellmann | where is that? | 21:40 |
| mtreinish | dhellmann: it's in tempest's tox.ini file | 21:40 |
| *** afazekas has joined #openstack-qa | 21:41 | |
| dhellmann | ah | 21:41 |
| dhellmann | and thirdparty is ec2 in this case? | 21:41 |
| mriedem | yeah | 21:41 |
| mriedem | it's the only thing under thirdparty in tempest | 21:41 |
| mriedem | boto | 21:41 |
| mriedem | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/thirdparty | 21:42 |
| dhellmann | k | 21:42 |
| *** ChuckC_ has joined #openstack-qa | 21:42 | |
| dhellmann | looking at how devstack-vm-gate.sh calls tox with the regex, I'm not sure I see how that works. It looks like it passes 2 args, but pretty_tox.sh only consumes the first one | 21:44 |
| mtreinish | dhellmann: the all job on tempest's tox.ini doesn't have a regex set | 21:45 |
| mtreinish | so when d-g passes it to tox, tox will pass it to pretty_tox which will pass it to testr | 21:45 |
| *** ChuckC has quit IRC | 21:46 | |
| dhellmann | mtreinish: devstack-vm-gate.sh passes 2 arguments to tox, though | 21:46 |
| dhellmann | at least on line 573 | 21:46 |
| dhellmann | http://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/devstack-vm-gate.sh#n573 | 21:46 |
| mtreinish | yeah the concurency option | 21:46 |
| mtreinish | to set it only use as many testr worker procs as d-g specified | 21:47 |
| dhellmann | ah, tox.ini has it quoted, I missed that | 21:47 |
| dhellmann | http://git.openstack.org/cgit/openstack/tempest/tree/tox.ini#n40 | 21:47 |
| dhellmann | ok, sorry for the noise :-) | 21:47 |
| mtreinish | dhellmann: no worries, you might have found a bug. It's been a while since I've touched this code path | 21:48 |
| mtreinish | jogo: do you have a sample icehouse review link? I'm trying to figure out which jobs to modify? | 21:48 |
| jogo | mtreinish: https://review.openstack.org/#/c/154217/1 | 21:50 |
| mriedem | looks like just -full jobs | 21:51 |
| mtreinish | yeah just postgres-full and full | 21:52 |
| mtreinish | neutron back then was just smoke | 21:52 |
| mtreinish | which doesn't include the thirdparty tests | 21:52 |
| mtreinish | jogo, mriedem, dhellmann: https://review.openstack.org/#/c/154251/1 | 21:53 |
| *** afazekas has quit IRC | 21:53 | |
| mriedem | mtreinish: throw bug 1419913 in the commit message for tracking | 21:54 |
| openstack | bug 1419913 in grenade "Grenade fails with ContextualVersionConflict between oslo.config and tempest-lib on juno side" [Undecided,Confirmed] https://launchpad.net/bugs/1419913 | 21:54 |
| *** sergsh has quit IRC | 21:54 | |
| *** achanda has joined #openstack-qa | 21:54 | |
| mtreinish | mriedem: done | 21:55 |
| sdague | mtreinish: https://review.openstack.org/#/c/154251/1 why not just do the venv installation in devstack stable/icehouse? | 21:57 |
| mtreinish | sdague: you can look at the backtrace | 21:57 |
| mtreinish | but that's actually what that patch fixes | 21:57 |
| mtreinish | because if we do that tempest get's latest boto | 21:57 |
| *** itzikb has quit IRC | 21:58 | |
| mtreinish | which will fail against stable icehouse's version of boto | 21:58 |
| sdague | ah, right | 21:58 |
| mriedem | mtreinish: questions in there on the jobs you picked to change | 21:58 |
| dhellmann | this was less ugly than having tempest look at nova's version to figure out which version of boto to install for itself and quicker than backporting the fixes to nova | 21:58 |
| jogo | dhellmann: ++ | 21:59 |
| sdague | or to disable it in devstack? | 21:59 |
| sdague | like completely drop the ec2 api in icehouse because it's actually not testable any more | 22:00 |
| sdague | in the devstack config | 22:00 |
| *** dmellado has joined #openstack-qa | 22:00 | |
| dhellmann | that's another option. would tempest figure out not to run the tests automatically if we did that? | 22:00 |
| sdague | that's a question I don't quite know | 22:00 |
| mriedem | looking | 22:00 |
| mtreinish | dhellmann: that's what I was just checking | 22:01 |
| mtreinish | I think the answer is no, but that's fixable | 22:01 |
| sdague | mtreinish: we could just delete the boto config section, right? | 22:01 |
| mtreinish | sdague: I just looked at the code I think it raises an exception if we do that | 22:02 |
| mtreinish | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/thirdparty/boto/test.py#n70 | 22:02 |
| * mtreinish facepalms | 22:03 | |
| mriedem | should have a feature enabled thing in tempest for not running boto tests | 22:03 |
| mriedem | but i guess you just wouldn't run thirdparty | 22:03 |
| mtreinish | yeah or a service_available flag | 22:03 |
| mriedem | that's what we do internally | 22:03 |
| mriedem | right, something like that | 22:03 |
| dhellmann | yeah, and if these tests are moving out to the stand-alone ec2 repo I wouldn't spend time adding a special flag | 22:03 |
| *** afazekas has joined #openstack-qa | 22:04 | |
| sdague | https://review.openstack.org/154252 | 22:05 |
| sdague | how about that? | 22:05 |
| mtreinish | actually relooking at that code it'll do the right thing, it's just a rats nest of nested try excepts | 22:05 |
| mtreinish | sdague: that should work, let's see if can get through check | 22:06 |
| sdague | mtreinish: yep | 22:06 |
| *** oomichi has joined #openstack-qa | 22:06 | |
| *** afazekas has quit IRC | 22:09 | |
| openstackgerrit | Matt Riedemann proposed openstack/tempest: Fix typos in boto decision_maker() https://review.openstack.org/154253 | 22:09 |
| mriedem | sdague: btw, https://etherpad.openstack.org/p/wedged-stable-gate-feb-2015 | 22:10 |
| mriedem | it's a mess, as most good etherpads are | 22:10 |
| *** abhirc has joined #openstack-qa | 22:15 | |
| *** afazekas has joined #openstack-qa | 22:15 | |
| *** Yogi1 has quit IRC | 22:15 | |
| *** dmellado has quit IRC | 22:20 | |
| jogo | mriedem: looks like we have baby steps forward so that is good | 22:22 |
| *** afazekas has quit IRC | 22:23 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Initial class creds creation in test base class https://review.openstack.org/153681 | 22:23 |
| *** dustins has quit IRC | 22:23 | |
| dhellmann | jogo, mriedem, mtreinish: so are we waiting for sdague's patch to merge before we take the next step? | 22:25 |
| jogo | dhellmann: yeah | 22:25 |
| jogo | dhellmann: but there are patches that can still be reviewed | 22:25 |
| *** vigneshvar has quit IRC | 22:25 | |
| jogo | let me cleanup the etherpad to say that | 22:26 |
| dhellmann | jogo: yeah, do -- I'm going to sign off soon but i can look at a patch or two first | 22:26 |
| sdague | jogo: same, it's about to be family time here | 22:26 |
| *** Ananci has quit IRC | 22:29 | |
| *** scottda has left #openstack-qa | 22:29 | |
| sdague | jogo: I rebased https://review.openstack.org/#/c/154217/ on my devstack patch, so it should be able to get good test results (hopefully) | 22:32 |
| openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for ironic bug 1418833 https://review.openstack.org/154272 | 22:33 |
| openstack | bug 1418833 in Ironic "check-tempest-dsvm-ironic-agent_ssh fails due to ConfigDriveWriteError" [Critical,Fix committed] https://launchpad.net/bugs/1418833 - Assigned to Jay Faulkner (jason-oldos) | 22:33 |
| jogo | sdague https://review.openstack.org/#/c/154216/ | 22:34 |
| jogo | sdague: that is failing in boto as well | 22:34 |
| jogo | ohh because its icehouse devstack on old | 22:35 |
| sdague | well... full fails as well | 22:35 |
| sdague | because no one merged this fast enough - https://review.openstack.org/#/c/151037/ | 22:35 |
| jogo | sdague: ugg | 22:36 |
| sdague | apparently you can't turn of ec2 tests - https://jenkins04.openstack.org/job/check-tempest-dsvm-full/28091/console | 22:40 |
| mriedem | yeah https://review.openstack.org/#/c/154252/1 | 22:40 |
| sdague | ok, well that's my night folks | 22:41 |
| mtreinish | sdague: I'll push a flag to make that will just bypass that crazy decision_maker() function | 22:42 |
| mriedem | it looks like if you goofed with the auth values in tempest.conf for boto it will set these values and skip the tests | 22:42 |
| mriedem | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/thirdparty/boto/test.py#n69 | 22:42 |
| mtreinish | and update your patch to use that instead | 22:42 |
| mriedem | we could also backport the nova changes... | 22:42 |
| mriedem | but that's whack a mole | 22:43 |
| dhellmann | I'm comfortable with us saying we're not testing the ec2 API in icehouse any more | 22:43 |
| mriedem | ... | 22:43 |
| dhellmann | if someone wants to do the work to backport the fix and support it, I would be OK with that happening later, after we're up and running elsewhere again | 22:44 |
| mriedem | what if we backport a security fix to icehouse in the ec2 code? | 22:44 |
| mriedem | yeah, we could disable, backport actual fixes, then enable again | 22:44 |
| dhellmann | mriedem: we need to differentiate between what we're going to do today to unblock everyone and what we want the long-term thing to be | 22:44 |
| dhellmann | right | 22:44 |
| mriedem | so mtreinish will push a tempest patch to add a boto entry to service_available in tempest.conf, then skip the boto tests if that's True, | 22:45 |
| mriedem | then we set devstack on stable/icehouse to set that to True | 22:45 |
| *** dmorita has quit IRC | 22:45 | |
| dhellmann | that sounds about right | 22:46 |
| dhellmann | jogo is writing other things in the etherpad | 22:46 |
| mriedem | i also have to head out for $family | 22:46 |
| *** dmorita has joined #openstack-qa | 22:46 | |
| mriedem | will eavesdrop later | 22:46 |
| jogo | still working out the order of things to not break it further | 22:47 |
| dhellmann | I wonder why no tests were run for https://review.openstack.org/#/c/154251/2 | 22:48 |
| *** toure_ has joined #openstack-qa | 22:49 | |
| dhellmann | I need to drop off, too. I'll check the etherpad for reviews to do in the morning | 22:49 |
| *** mriedem has quit IRC | 22:51 | |
| *** rfolco has quit IRC | 22:51 | |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Add compute feature enabled flag for the ec2 api https://review.openstack.org/154279 | 22:52 |
| jogo | ok I think the order should work | 22:52 |
| mtreinish | mriedem, jogo, sdague: ^^^ | 22:52 |
| *** dmellado has joined #openstack-qa | 22:52 | |
| jogo | mtreinish: can you review the steps I wrote down | 22:53 |
| jogo | mtreinish: as I want to make sure I didn't miss anything | 22:53 |
| *** e0ne has quit IRC | 22:53 | |
| sdague | mtreinish: +A, though... is that mergable in current state of world? | 22:53 |
| jogo | if I did, its very possible I break trunk | 22:53 |
| jogo | sdague: ^ | 22:53 |
| jogo | one step is to break stable/juno grenade further | 22:54 |
| mtreinish | sdague: it should be, the thing which is broken is running javelin on juno grenade | 22:54 |
| sdague | ok | 22:54 |
| mtreinish | everything else is protected because of the tox venv | 22:54 |
| sdague | anyway, I really really need to leave computer now, if there is stuff for me to approve in the morning, let me know | 22:55 |
| *** Longgeek has joined #openstack-qa | 22:55 | |
| jogo | sdague: see the etherpad, it should have all the patches and order needed | 22:55 |
| mtreinish | jogo: sure I'll take a look now | 22:55 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Split creds and resources in tenant isolation https://review.openstack.org/153896 | 22:55 |
| *** dmellado has quit IRC | 22:56 | |
| *** dmellado_ has joined #openstack-qa | 22:56 | |
| *** dmellado_ is now known as dmellado | 22:57 | |
| jogo | all together it should be only 5 or 6 patches lol | 22:57 |
| jogo | but hopefully no ninja patches and it shouldn't break master at all | 22:59 |
| mtreinish | jogo: can you hit the rebase button on your don't install tempest in stable/juno | 23:01 |
| mtreinish | I'll +A it after | 23:01 |
| *** toure_ has quit IRC | 23:01 | |
| jogo | mtreinish: which one? | 23:02 |
| jogo | link | 23:02 |
| mtreinish | jogo: https://review.openstack.org/#/c/154217/ | 23:02 |
| *** andreykurilin_ has joined #openstack-qa | 23:03 | |
| jogo | it needs another reabse? Sean rebased it already | 23:03 |
| jogo | ahh I see | 23:03 |
| mtreinish | jogo: I just pushed a rev for Sean's patch to use the flag I added | 23:03 |
| mtreinish | jogo: that order on the etherpad looks right to me | 23:05 |
| *** jamielennox|away is now known as jamielennox | 23:05 | |
| jogo | mtreinish: thanks for sanity checking, here is the rebase https://review.openstack.org/#/c/154217/ | 23:08 |
| * jogo goes AFK to find food | 23:09 | |
| *** enikanorov_ has quit IRC | 23:09 | |
| *** diegows has joined #openstack-qa | 23:10 | |
| *** toure_ has joined #openstack-qa | 23:10 | |
| openstackgerrit | John Villalovos proposed openstack-dev/devstack: Use 'https' protocol by default as easier for proxies https://review.openstack.org/153397 | 23:10 |
| *** Longgeek has quit IRC | 23:12 | |
| openstackgerrit | Merged openstack-dev/devstack: Stop installing tempest https://review.openstack.org/153080 | 23:15 |
| *** jlibosva has joined #openstack-qa | 23:15 | |
| *** jlibosva has quit IRC | 23:20 | |
| *** EmilienM is now known as EmilienM|afk | 23:28 | |
| *** bknudson has quit IRC | 23:31 | |
| *** jgrimm is now known as zz_jgrimm | 23:35 | |
| *** dmellado has quit IRC | 23:36 | |
| *** toure_ has quit IRC | 23:37 | |
| andreaf | mtreinish: is there a way to mark a compute extension in tempest as disabled without having to provide a list of all the enabled ones? | 23:38 |
| *** dmellado has joined #openstack-qa | 23:38 | |
| mtreinish | andreaf: there's just the list opt | 23:38 |
| mtreinish | there are reasons it's that way | 23:40 |
| andreaf | mtreinish: I guess something related to stable branches | 23:40 |
| andreaf | mtreinish: so it means that implementing and exclusion list on top is not an option? | 23:41 |
| mtreinish | actually no, it's more with what happens if you add an extension on master | 23:41 |
| mtreinish | I'd have to dig up all the old discussions to outline all of the edge cases | 23:42 |
| mtreinish | andreaf: devstack actually supports that, but we will not use that on the gate | 23:42 |
| *** mriedem has joined #openstack-qa | 23:43 | |
| mtreinish | because it relies on the api to get the list of all the extensions | 23:43 |
| *** tochi has joined #openstack-qa | 23:43 | |
| *** mriedem has quit IRC | 23:44 | |
| andreaf | mtreinish: if we had an excluded extensions in tempest that would not require using the API | 23:44 |
| mtreinish | andreaf: it's really only either or, the overlap between an include and exclusion list is why we don't do that | 23:45 |
| *** mriedem has joined #openstack-qa | 23:46 | |
| mtreinish | andreaf: https://review.openstack.org/#/c/116121/ | 23:47 |
| andreaf | mtreinish: uhm, right... ok I'll have to stop being lazy and produce my list of extensions :P | 23:47 |
| mtreinish | andreaf: heh, you can use verify_tempest_config for that if the list is constant | 23:48 |
| *** andreykurilin_ has quit IRC | 23:51 | |
| mriedem | mtreinish: so once https://review.openstack.org/#/c/154279/ is merged you're going to update https://review.openstack.org/#/c/154251 right? | 23:51 |
| *** dimtruck is now known as zz_dimtruck | 23:53 | |
| *** dimsum__ has quit IRC | 23:54 | |
| *** dimsum__ has joined #openstack-qa | 23:55 | |
| openstackgerrit | Maru Newby proposed openstack-infra/devstack-gate: Use sudo to move hook script logs https://review.openstack.org/154292 | 23:55 |
| marun | mtreinish: I'm seeing an issue wiht the refactored testr artifact processing... subunit and testr are not installed by the neutron functional job. | 23:57 |
| marun | mtreinish: any thoughts as to how I should best rectify? | 23:57 |
| marun | mtreinish: Should I manually install it in the environment in our hook script? | 23:57 |
| *** dimsum__ is now known as dims | 23:57 | |
| marun | mtreinish: That's the easy way out, but I'm wondering if a generally usable solution wouldn't be preferable. | 23:58 |
| *** dims is now known as Guest70247 | 23:58 | |
| *** Guest70247 has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!