| ianw | stevebaker: that's all the reviews, right? | 00:01 |
|---|---|---|
| *** ekhugen_ has joined #openstack-qa | 00:03 | |
| stevebaker | ianw: \o/ yes thanks! | 00:05 |
| *** yjiang5 has left #openstack-qa | 00:13 | |
| *** harlowja has quit IRC | 00:15 | |
| *** harlowja_ has joined #openstack-qa | 00:15 | |
| *** danfinch has quit IRC | 00:16 | |
| *** dims has joined #openstack-qa | 00:16 | |
| *** rbak has joined #openstack-qa | 00:18 | |
| *** rbak has quit IRC | 00:24 | |
| *** mriedem has joined #openstack-qa | 00:25 | |
| *** suneelb has quit IRC | 00:26 | |
| *** yamahata has joined #openstack-qa | 00:28 | |
| *** aimon has quit IRC | 00:30 | |
| *** dmorita has joined #openstack-qa | 00:32 | |
| *** yamahata has quit IRC | 00:33 | |
| *** yamahata has joined #openstack-qa | 00:34 | |
| *** aimon has joined #openstack-qa | 00:55 | |
| *** topshare has joined #openstack-qa | 01:04 | |
| *** topshare_ has joined #openstack-qa | 01:07 | |
| *** topshare has quit IRC | 01:09 | |
| *** yamahata has quit IRC | 01:09 | |
| *** yamahata has joined #openstack-qa | 01:09 | |
| *** danfinch has joined #openstack-qa | 01:10 | |
| *** rfolco has joined #openstack-qa | 01:16 | |
| *** mwagner_lap has joined #openstack-qa | 01:29 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Fix "mutable" object as default value https://review.openstack.org/100132 | 01:31 |
| *** topshare_ has quit IRC | 01:33 | |
| *** amotoki has joined #openstack-qa | 01:34 | |
| *** topshare has joined #openstack-qa | 01:36 | |
| *** ArxCruz has quit IRC | 01:37 | |
| *** arnaud has quit IRC | 01:41 | |
| *** topshare has quit IRC | 01:42 | |
| *** dims has quit IRC | 01:44 | |
| *** dims has joined #openstack-qa | 01:44 | |
| *** nosnos has joined #openstack-qa | 01:44 | |
| *** zaro has joined #openstack-qa | 01:44 | |
| *** dims has quit IRC | 01:45 | |
| *** dims_ has joined #openstack-qa | 01:45 | |
| *** topshare has joined #openstack-qa | 01:46 | |
| *** topshare has quit IRC | 01:50 | |
| *** topshare has joined #openstack-qa | 01:51 | |
| *** topshare_ has joined #openstack-qa | 01:55 | |
| *** topshare has quit IRC | 01:58 | |
| *** dims_ has quit IRC | 02:02 | |
| *** dims has joined #openstack-qa | 02:03 | |
| *** dims has quit IRC | 02:12 | |
| *** dims has joined #openstack-qa | 02:12 | |
| *** arnaud has joined #openstack-qa | 02:19 | |
| *** dims has quit IRC | 02:22 | |
| *** dims has joined #openstack-qa | 02:23 | |
| *** dims has quit IRC | 02:27 | |
| *** dims has joined #openstack-qa | 02:27 | |
| *** dims has quit IRC | 02:35 | |
| *** topshare_ has quit IRC | 02:36 | |
| *** topshare has joined #openstack-qa | 02:36 | |
| *** dims has joined #openstack-qa | 02:36 | |
| *** dims has quit IRC | 02:40 | |
| *** harlowja_ is now known as harlowja_away | 02:42 | |
| openstackgerrit | Kevin Benton proposed a change to openstack/tempest: Catch BadRequest exception from correct module https://review.openstack.org/115851 | 02:48 |
| *** dimtruck is now known as zz_dimtruck | 02:52 | |
| *** ekhugen_ has quit IRC | 02:54 | |
| *** mriedem has quit IRC | 03:02 | |
| *** adalbas has quit IRC | 03:03 | |
| *** Poornima has joined #openstack-qa | 03:08 | |
| *** Sukhdev has joined #openstack-qa | 03:17 | |
| *** chandankumar has joined #openstack-qa | 03:27 | |
| *** zz_dimtruck is now known as dimtruck | 03:37 | |
| *** chandankumar has quit IRC | 03:44 | |
| *** dimtruck is now known as zz_dimtruck | 03:47 | |
| *** ekhugen_ has joined #openstack-qa | 03:50 | |
| openstackgerrit | ajay proposed a change to openstack/tempest: Improve nova CLI existing tests https://review.openstack.org/101767 | 03:52 |
| *** SridharG has joined #openstack-qa | 04:03 | |
| *** arnaud has quit IRC | 04:10 | |
| *** Madan has joined #openstack-qa | 04:11 | |
| *** arnaud has joined #openstack-qa | 04:12 | |
| *** rbak has joined #openstack-qa | 04:22 | |
| *** rbak has quit IRC | 04:22 | |
| openstackgerrit | Kevin Benton proposed a change to openstack/tempest: Catch BadRequest exception from correct module https://review.openstack.org/115851 | 04:23 |
| openstackgerrit | Puneet Arora proposed a change to openstack/tempest: Added keystone cli test case https://review.openstack.org/114885 | 04:36 |
| *** cmyster has joined #openstack-qa | 04:38 | |
| Santosh | mkoderer: hi | 04:50 |
| *** nmagnezi has joined #openstack-qa | 04:54 | |
| *** nmagnezi has quit IRC | 05:00 | |
| *** Poornima has quit IRC | 05:09 | |
| *** topshare has quit IRC | 05:13 | |
| *** pcrews has quit IRC | 05:16 | |
| *** topshare has joined #openstack-qa | 05:17 | |
| *** Sukhdev has quit IRC | 05:17 | |
| openstackgerrit | Preethi Dsilva proposed a change to openstack/tempest: Added test to check DVR interface add using subnet https://review.openstack.org/113440 | 05:19 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Ironic and baremetal install diskimage-builder with pip https://review.openstack.org/92257 | 05:21 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/110110 | 05:22 |
| *** topshare has quit IRC | 05:24 | |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Updated telemetry client for bp client-checks-success https://review.openstack.org/115871 | 05:24 |
| *** chandankumar has joined #openstack-qa | 05:25 | |
| ianw | stevebaker: hey, one thing i noticed with https://review.openstack.org/#/c/92256/9/lib/dib is we have a setup_package in functions-common. maybe it should use that | 05:28 |
| stevebaker | ianw: I think setup_package does something a bit different; it installs the project to the local system (usually with -e developer mode). What dib needs is a way to build pip packages, and a way to deliver those packages into the custom image building process | 05:34 |
| *** topshare has joined #openstack-qa | 05:34 | |
| ianw | stevebaker: but install_dib is installing to the local system? | 05:37 |
| stevebaker | oh, i see, for local dib usage | 05:37 |
| *** zz_dimtruck is now known as dimtruck | 05:38 | |
| ianw | stevebaker: yeah, just as a cleanup. you can also add yourself in MAINTAINERS.rst for d-i-b :) | 05:39 |
| stevebaker | ianw: developer mode (-e) will never work with dib (its actually a collection of shell scripts) so I'm not sure its worth calling setup_package when it just calls vanilla pip_install anyway | 05:41 |
| *** SridharG has quit IRC | 05:42 | |
| *** Poornima has joined #openstack-qa | 05:43 | |
| ianw | stevebaker: no -e because the scripts won't know where to look? | 05:43 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Install heat-cfntools and heat-templates repos https://review.openstack.org/92258 | 05:43 |
| *** vaidy-nick has joined #openstack-qa | 05:43 | |
| stevebaker | ianw: yeah, dev mode is python specific | 05:44 |
| openstackgerrit | A change was merged to openstack-dev/devstack: A service to install diskimage-builder https://review.openstack.org/92256 | 05:46 |
| *** dimtruck is now known as zz_dimtruck | 05:48 | |
| *** cnesa7 has joined #openstack-qa | 05:48 | |
| ianw | stevebaker: huh, i had it in my head that the stuff in bin/ was python driving the shell parts. i see not | 05:48 |
| StevenK | ianw: Yeah, dib is about 80% shell | 05:50 |
| openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Migrate test_encrypted_cinder_volumes to tempest client https://review.openstack.org/111905 | 05:53 |
| *** Sukhdev has joined #openstack-qa | 05:56 | |
| *** lordd_ has joined #openstack-qa | 05:58 | |
| openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Migrate test_encrypted_cinder_volumes to tempest client https://review.openstack.org/111905 | 06:09 |
| *** jlibosva has joined #openstack-qa | 06:15 | |
| *** SridharG has joined #openstack-qa | 06:15 | |
| *** flaper87|afk is now known as flaper87 | 06:21 | |
| *** arnaud has quit IRC | 06:25 | |
| *** k4n0 has joined #openstack-qa | 06:25 | |
| *** vaidy-nick has quit IRC | 06:25 | |
| *** Santosh has quit IRC | 06:28 | |
| openstackgerrit | ajay proposed a change to openstack/tempest: Improve nova CLI existing tests https://review.openstack.org/101767 | 06:38 |
| *** Santosh_ has joined #openstack-qa | 06:44 | |
| *** vaidy-nick has joined #openstack-qa | 06:47 | |
| *** ekhugen_ has quit IRC | 06:49 | |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Updated telemetry client for bp client-checks-success https://review.openstack.org/115871 | 06:54 |
| kashyap | afazekas, Heya, this seems to be spiking from ER-- https://bugs.launchpad.net/bugs/1323658 I see there's a fair bit of investigation there already | 06:54 |
| afazekas | kashyap: I will relogin in 30 min | 06:55 |
| *** afazekas is now known as __afazekas | 06:55 | |
| kashyap | Sure. No rush. | 06:55 |
| __afazekas | good to see you :) | 06:56 |
| __afazekas | bye | 06:56 |
| kashyap | See ya. | 06:58 |
| *** Sukhdev has quit IRC | 07:01 | |
| *** markus_z has joined #openstack-qa | 07:03 | |
| *** gfidente has joined #openstack-qa | 07:04 | |
| *** wuhg has joined #openstack-qa | 07:19 | |
| *** ildikov has joined #openstack-qa | 07:30 | |
| *** jlanoux has joined #openstack-qa | 07:34 | |
| *** zz_dimtruck is now known as dimtruck | 07:37 | |
| openstackgerrit | Nikita Koshikov proposed a change to openstack-dev/devstack: Coraid cinder driver added https://review.openstack.org/115887 | 07:38 |
| *** lordd_ has quit IRC | 07:39 | |
| *** gszasz has joined #openstack-qa | 07:43 | |
| *** afazekas has joined #openstack-qa | 07:43 | |
| afazekas | kashyap: FYI: https://review.openstack.org/#/c/113856/ | 07:44 |
| *** dimtruck is now known as zz_dimtruck | 07:47 | |
| *** jpich has joined #openstack-qa | 07:51 | |
| *** shardy_afk is now known as shardy | 07:52 | |
| *** aix has joined #openstack-qa | 07:53 | |
| *** sergeysh has joined #openstack-qa | 07:56 | |
| *** jordanP has joined #openstack-qa | 07:59 | |
| *** Longgeek has joined #openstack-qa | 08:10 | |
| *** Longgeek has quit IRC | 08:11 | |
| *** Longgeek has joined #openstack-qa | 08:12 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/tempest: javelin: add network resources https://review.openstack.org/100169 | 08:20 |
| *** samuelbercovici has quit IRC | 08:28 | |
| kashyap | afazekas, Thanks; will look. | 08:28 |
| afazekas | kashyap: what was your question ? | 08:29 |
| mkoderer | andreaf: I reviewed https://review.openstack.org/#/c/107562 | 08:31 |
| mkoderer | andreaf: I found just some nits so I am ok with merging it | 08:31 |
| kashyap | afazekas, I was just looking at the ER for top Gate bugs, and stumbled on the one I mentioned, and wondered if there's any fix in progress. | 08:31 |
| mkoderer | andreaf: are you still working on it or should I +A? | 08:31 |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Updated telemetry client for bp client-checks-success https://review.openstack.org/115871 | 08:33 |
| *** cdent has joined #openstack-qa | 08:34 | |
| andreaf | mkoderer: no I'm not working on it - let me check your comments :) | 08:35 |
| andreaf | mkoderer: ok I just noticed that there is one change in L801 which should not be there - however it seems that that code path is not used as tests are still passing | 08:40 |
| andreaf | mkoderer, given the fact that that code will be dropped anyways once this bp is complete, I think it should be ok to +A | 08:41 |
| andreaf | mkoderer: but if you prefer I can fix that and the os.path thing - I would prefer not to rename the addCleanup to add_cleanup | 08:42 |
| *** sdmonov has joined #openstack-qa | 08:43 | |
| *** cmyster has quit IRC | 08:44 | |
| afazekas | kashyap: can you show me the link again ? | 08:44 |
| kashyap | https://bugs.launchpad.net/bugs/1323658 | 08:44 |
| *** aix has quit IRC | 08:46 | |
| *** lucasagomes has joined #openstack-qa | 08:46 | |
| lucasagomes | hi all, can I get some eyes at this tempest patch here please https://review.openstack.org/#/c/113276/ ? it's pretty small (62 lines) and it's blocking a bp to be marked as implemented in Ironic | 08:47 |
| lucasagomes | that's the last bit | 08:47 |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/tempest: javelin: add network resources https://review.openstack.org/100169 | 08:48 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Fix use of nonexistent class variable in accounts https://review.openstack.org/114636 | 08:54 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Make class setup safe in compute base admin class https://review.openstack.org/115123 | 08:54 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 08:54 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 08:54 |
| *** sdmonov has quit IRC | 08:55 | |
| andreaf | mtreinish: we did get through the get_hash in clear_credentials now, but the call to remove_hash had an extra parameter - fixed that now ^^^^ | 08:56 |
| *** lordd_ has joined #openstack-qa | 08:59 | |
| *** zz_johnthetubagu is now known as johnthetubaguy | 09:01 | |
| *** lordd_ has quit IRC | 09:03 | |
| *** ildikov has quit IRC | 09:07 | |
| mkoderer | andreaf: btw, are you working on unit test for that? | 09:08 |
| mkoderer | andreaf: +A | 09:09 |
| andreaf | mkoderer: thanks - unit tests for? | 09:13 |
| andreaf | mkoderer: we don't have unit tests for test classes | 09:13 |
| coolsvap | andreaf, https://review.openstack.org/90306 https://review.openstack.org/115238 https://review.openstack.org/115671 https://review.openstack.org/115871 | 09:15 |
| *** bkopilov has quit IRC | 09:17 | |
| mkoderer | andreaf: mh ok your right.. | 09:18 |
| andreaf | coolsvap, ok | 09:18 |
| mkoderer | just thougth 350 new LOC and no gurantee that they are really working ;) | 09:19 |
| andreaf | mkoderer, uhm I know it's a large change but most of them should be in use by the scenario test so tested that way | 09:19 |
| andreaf | mkoderer and it's not 100% new code in fact modifications are not that much - it's just that it's copied because we need to run both on official clients as well as tempest clients until migration is done | 09:20 |
| mkoderer | andreaf: ok I see | 09:21 |
| andreaf | mkoderer, but we could consider moving some of the utils into a common class and unit test them - | 09:21 |
| mkoderer | andreaf: that's what I was thinking of, but let's do that later ;) | 09:21 |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Updated telemetry client for bp client-checks-success https://review.openstack.org/115871 | 09:24 |
| *** aix has joined #openstack-qa | 09:24 | |
| *** sdmonov has joined #openstack-qa | 09:25 | |
| openstackgerrit | Rajkumar proposed a change to openstack/tempest: Move API response success check to Neutron client https://review.openstack.org/114186 | 09:27 |
| openstackgerrit | Rajkumar proposed a change to openstack/tempest: Add Test to Create Port with no security groups https://review.openstack.org/115908 | 09:27 |
| openstackgerrit | Jakub Libosvar proposed a change to openstack-infra/devstack-gate: TEST ONLY: Run javelin change with neutron and grenade https://review.openstack.org/115909 | 09:30 |
| *** sdmonov has quit IRC | 09:30 | |
| *** topshare has quit IRC | 09:32 | |
| *** topshare has joined #openstack-qa | 09:33 | |
| *** Madan has quit IRC | 09:34 | |
| openstackgerrit | Preethi Dsilva proposed a change to openstack/tempest: Added test to check DVR interface add using subnet https://review.openstack.org/113440 | 09:36 |
| *** Madan has joined #openstack-qa | 09:38 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack-dev/grenade: TEST: Run grenade with tempest change 100169 https://review.openstack.org/115910 | 09:38 |
| *** topshare has quit IRC | 09:44 | |
| openstackgerrit | Flavio Percoco proposed a change to openstack-dev/devstack: Install mongodb for Zaqar https://review.openstack.org/115911 | 09:44 |
| flaper87 | sdague: chmouel_ hey guys, this is currently blocking patches for zaqar, if you guys can take a look at it, I'd really appreaciate it. ^ | 09:46 |
| *** topshare has joined #openstack-qa | 09:49 | |
| openstackgerrit | Dina Belova proposed a change to openstack/tempest: [WIP] Revert "Skip telemetry_notification_api test due to bug 1336755" https://review.openstack.org/115212 | 09:53 |
| *** tosky has joined #openstack-qa | 09:54 | |
| *** dmorita has quit IRC | 09:55 | |
| *** yamahata has quit IRC | 09:58 | |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Remove functionality to load json files https://review.openstack.org/115918 | 10:00 |
| *** Santosh_ has quit IRC | 10:08 | |
| openstackgerrit | Rohan Kanade proposed a change to openstack/tempest: Move API response success check to Neutron client https://review.openstack.org/114186 | 10:18 |
| *** mkollaro has joined #openstack-qa | 10:19 | |
| *** qba73 has joined #openstack-qa | 10:25 | |
| *** bkopilov has joined #openstack-qa | 10:28 | |
| *** topshare has quit IRC | 10:28 | |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: Install sqlalchemy-migrate from master if requested https://review.openstack.org/114224 | 10:30 |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: (DO NOT MERGE) test run using mysqlconnector https://review.openstack.org/114207 | 10:31 |
| * gmann is away: AFW | 10:31 | |
| *** eezhova has quit IRC | 10:33 | |
| *** eezhova has joined #openstack-qa | 10:34 | |
| *** Longgeek has quit IRC | 10:35 | |
| *** Longgeek has joined #openstack-qa | 10:36 | |
| openstackgerrit | Kirill Shileev proposed a change to openstack/tempest: Add sec-group rules for ping6 and ssh6 https://review.openstack.org/115927 | 10:37 |
| *** Longgeek has quit IRC | 10:40 | |
| *** topshare has joined #openstack-qa | 10:42 | |
| *** topshare has quit IRC | 10:44 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Fix use of nonexistent class variable in accounts https://review.openstack.org/114636 | 10:46 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Make class setup safe in compute base admin class https://review.openstack.org/115123 | 10:46 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 10:46 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 10:46 |
| *** cdent_ has joined #openstack-qa | 10:52 | |
| *** cdent has quit IRC | 10:53 | |
| *** cdent_ is now known as cdent | 10:53 | |
| *** tkammer has joined #openstack-qa | 10:54 | |
| *** jaypipes has quit IRC | 10:55 | |
| *** lordd_ has joined #openstack-qa | 10:59 | |
| *** lordd_ has quit IRC | 11:04 | |
| *** Longgeek has joined #openstack-qa | 11:06 | |
| *** Longgeek has quit IRC | 11:09 | |
| *** Longgeek has joined #openstack-qa | 11:09 | |
| *** e0ne has joined #openstack-qa | 11:13 | |
| e0ne | hi all! did anyone try to run tempest with cinder+ceph backend? | 11:14 |
| openstackgerrit | Jakub Libosvar proposed a change to openstack-dev/grenade: TEST: Run grenade with tempest change 100169 https://review.openstack.org/115910 | 11:18 |
| *** e0ne has quit IRC | 11:18 | |
| *** dims has joined #openstack-qa | 11:20 | |
| *** dims has quit IRC | 11:20 | |
| *** e0ne has joined #openstack-qa | 11:20 | |
| *** dims has joined #openstack-qa | 11:20 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: implement resource destruction https://review.openstack.org/104986 | 11:24 |
| *** amotoki has quit IRC | 11:28 | |
| *** Poornima has quit IRC | 11:29 | |
| *** vaidy-nick has quit IRC | 11:29 | |
| openstackgerrit | Swami Reddy proposed a change to openstack/tempest: Add "reboot_instance" EC2 API test case https://review.openstack.org/115944 | 11:36 |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Updated telemetry client for bp client-checks-success https://review.openstack.org/115871 | 11:37 |
| *** ArxCruz has joined #openstack-qa | 11:43 | |
| *** gszasz has quit IRC | 11:51 | |
| *** ildikov has joined #openstack-qa | 11:51 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack-dev/grenade: TEST: Run grenade with tempest change 100169 https://review.openstack.org/115910 | 11:54 |
| *** gszasz has joined #openstack-qa | 11:56 | |
| *** tosky has quit IRC | 11:58 | |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: (DO NOT MERGE) test run using mysqlconnector https://review.openstack.org/114207 | 11:58 |
| *** Longgeek has quit IRC | 12:03 | |
| *** jhenner1 has quit IRC | 12:04 | |
| *** tkammer has quit IRC | 12:04 | |
| *** adalbas has joined #openstack-qa | 12:05 | |
| andreaf | mtreinish: https://review.openstack.org/#/c/114578/ is looking better now, but there are cases where tearDownClass fails (in the test code) because it tries to delete attributes which are not defined. This issue is introduced by the patch that adds the safe_setup everywhere probably. And it's a good way to identity issues in tearDownClass implementations | 12:06 |
| *** qba73 has quit IRC | 12:07 | |
| *** jhenner has joined #openstack-qa | 12:07 | |
| *** cmyster has joined #openstack-qa | 12:08 | |
| *** Longgeek has joined #openstack-qa | 12:12 | |
| *** Longgeek has quit IRC | 12:13 | |
| *** nosnos has quit IRC | 12:13 | |
| *** Longgeek has joined #openstack-qa | 12:14 | |
| *** ArxCruz has quit IRC | 12:14 | |
| *** aysyd has joined #openstack-qa | 12:18 | |
| *** tkammer has joined #openstack-qa | 12:20 | |
| *** dkranz has joined #openstack-qa | 12:20 | |
| *** ArxCruz has joined #openstack-qa | 12:20 | |
| *** ekhugen_ has joined #openstack-qa | 12:24 | |
| *** zz_dimtruck is now known as dimtruck | 12:27 | |
| *** tosky has joined #openstack-qa | 12:33 | |
| *** lordd_ has joined #openstack-qa | 12:33 | |
| *** jhenner has quit IRC | 12:36 | |
| *** dkranz has quit IRC | 12:36 | |
| *** tkammer has quit IRC | 12:37 | |
| *** ekhugen_ has quit IRC | 12:41 | |
| *** salv-orlando has quit IRC | 12:42 | |
| *** flaper87 is now known as flaper87|afk | 12:44 | |
| *** tkammer has joined #openstack-qa | 12:49 | |
| *** jhenner has joined #openstack-qa | 12:51 | |
| *** dkranz has joined #openstack-qa | 12:51 | |
| *** dims has quit IRC | 12:54 | |
| *** dims has joined #openstack-qa | 12:54 | |
| openstackgerrit | Chris Dent proposed a change to openstack/tempest: Add scenario test for swift-proxy ceilometer middleware https://review.openstack.org/115971 | 12:55 |
| openstackgerrit | Swami Reddy proposed a change to openstack/tempest: Add "reboot_instance" EC2 API test case https://review.openstack.org/115944 | 12:56 |
| *** aix has quit IRC | 12:57 | |
| *** Madan has quit IRC | 12:59 | |
| *** dkranz has quit IRC | 13:03 | |
| *** xianghuihui has joined #openstack-qa | 13:06 | |
| *** jaypipes has joined #openstack-qa | 13:07 | |
| *** ChanServ sets mode: +o jaypipes | 13:07 | |
| *** andreaf has quit IRC | 13:08 | |
| *** xianghui has quit IRC | 13:09 | |
| *** xianghuihui has quit IRC | 13:13 | |
| *** mriedem has joined #openstack-qa | 13:18 | |
| *** dkranz has joined #openstack-qa | 13:18 | |
| *** pcrews has joined #openstack-qa | 13:20 | |
| *** openstackgerrit has quit IRC | 13:21 | |
| *** salv-orlando has joined #openstack-qa | 13:22 | |
| *** malini has joined #openstack-qa | 13:23 | |
| *** jhenner has quit IRC | 13:28 | |
| *** salv-orlando_ has joined #openstack-qa | 13:29 | |
| *** malini has quit IRC | 13:31 | |
| *** salv-orlando has quit IRC | 13:31 | |
| *** salv-orlando_ is now known as salv-orlando | 13:31 | |
| *** malini has joined #openstack-qa | 13:32 | |
| mtreinish | andreaf: ok cool, yeah I figured that would be an issue | 13:32 |
| mtreinish | a lot of the tearDownClasses were probably written assuming setUpClass completed (which normally is a fine assumption to make) | 13:32 |
| *** dustins has joined #openstack-qa | 13:33 | |
| mtreinish | andreaf: looking at the branch you had a rebase mistake: https://review.openstack.org/#/c/114636/9/tempest/common/accounts.py | 13:35 |
| mtreinish | the fallback stuff should be in the test patch... | 13:36 |
| *** malini has quit IRC | 13:39 | |
| *** krtaylor has quit IRC | 13:41 | |
| *** ArxCruz has quit IRC | 13:46 | |
| *** mestery has joined #openstack-qa | 13:47 | |
| *** ArxCruz has joined #openstack-qa | 13:48 | |
| *** jamielennox is now known as jamielennox|away | 13:48 | |
| *** k4n0 has quit IRC | 13:55 | |
| *** lordd_ has quit IRC | 13:57 | |
| *** openstackgerrit has joined #openstack-qa | 13:59 | |
| *** samuelbercovici has joined #openstack-qa | 14:00 | |
| *** mestery has quit IRC | 14:00 | |
| *** toure has joined #openstack-qa | 14:00 | |
| *** mestery has joined #openstack-qa | 14:01 | |
| *** salv-orlando has quit IRC | 14:01 | |
| *** rbak has joined #openstack-qa | 14:07 | |
| *** mestery has quit IRC | 14:10 | |
| *** mestery_ has joined #openstack-qa | 14:10 | |
| *** rbak has quit IRC | 14:10 | |
| *** rbak has joined #openstack-qa | 14:11 | |
| *** mestery_ is now known as mestery | 14:11 | |
| *** krtaylor has joined #openstack-qa | 14:17 | |
| *** aix has joined #openstack-qa | 14:20 | |
| cdent | jlanoux: Thanks for the comments on my swift-proxy ceilometer stuff, I'll get that fixed pronto. In a related Q do you know what I need to do or can do to get the ceilometer middleware turned on in the swift-proxy in the various configs? | 14:22 |
| openstackgerrit | Matt Riedemann proposed a change to openstack-dev/devstack: Add graphviz package to general list for building docs https://review.openstack.org/115989 | 14:22 |
| *** jlanoux_ has joined #openstack-qa | 14:23 | |
| cdent | jlanoux_: Thanks for the comments on my swift-proxy ceilometer stuff, I'll get that fixed pronto. In a related Q do you know what I need to do or can do to get the ceilometer middleware turned on in the swift-proxy in the various configs? | 14:23 |
| *** jlanoux has quit IRC | 14:26 | |
| jlanoux_ | cdent: No problem. For your question, I dont' know. I didn't have the opportunity to do much work on swift. | 14:26 |
| cdent | It's not really a question about swift, it's about how/where to modify gate job configurations. | 14:27 |
| *** lordd_ has joined #openstack-qa | 14:28 | |
| clarkb | cdent the configs are all done by devstack as controlled by devstack-gate | 14:29 |
| clarkb | so if devstack does it or can do it we can toggle it | 14:29 |
| cdent | Thanks clarkb, so if I propose to change something I make a patch against devstack-gate? | 14:30 |
| clarkb | that said ceilometer is intentionally disabled there last I checked so we will need to make sure adding it back in is ok | 14:30 |
| cdent | last I heard it was on its way back in or back in | 14:30 |
| clarkb | cdent if devstack already does it and just needs a bit flipped then yes | 14:30 |
| *** lordd_ has quit IRC | 14:34 | |
| cdent | It appears that what I want is enabled if ceilometer itself is enabled | 14:35 |
| clarkb | that can probably just go in devstack then | 14:37 |
| *** jgrimm has joined #openstack-qa | 14:37 | |
| clarkb | and devstack-gate can (and already does) toggle the main ceilometer flag(s) | 14:37 |
| *** cmyster has quit IRC | 14:40 | |
| *** jecarey has quit IRC | 14:41 | |
| *** samuelbercovici has quit IRC | 14:41 | |
| lucasagomes | hi all, can I get some eyes at this tempest patch here please https://review.openstack.org/#/c/113276/ ? it's pretty small (62 lines) and it's blocking a bp to be marked as implemented in Ironic. Thanks | 14:42 |
| *** samuelbercovici has joined #openstack-qa | 14:42 | |
| *** lordd_ has joined #openstack-qa | 14:45 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-dev/devstack: Change API_RATE_LIMIT default to False to match the nova code https://review.openstack.org/115999 | 14:45 |
| e0ne | hi here. did anyone try to run tempest with cinder+ceph backend? | 14:46 |
| *** jhenner has joined #openstack-qa | 14:48 | |
| clarkb | I think russellb may have been working on it | 14:48 |
| e0ne | thanks, clarkb | 14:48 |
| *** mestery has quit IRC | 14:52 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/devstack-gate: Remove API_RATE_LIMIT setting for Tempest runs https://review.openstack.org/116001 | 14:55 |
| mtreinish | mriedem: btw, did I show you: https://review.openstack.org/114371 yet? | 14:57 |
| mriedem | mtreinish: you didn't | 14:57 |
| mriedem | but we talked about it | 14:57 |
| mriedem | mtreinish: any numbers yet? | 14:58 |
| mriedem | mtreinish: i've still been meaning to do that internally | 14:58 |
| *** coolsvap has quit IRC | 14:58 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: DO NOT MERGE test tempest in the gate without xml https://review.openstack.org/114371 | 14:59 |
| mtreinish | mriedem: it looks like the gate jobs run in ~28-37min | 14:59 |
| *** toure has quit IRC | 14:59 | |
| mtreinish | so a bit faster | 14:59 |
| mriedem | mtreinish: what's the normal avg today? | 14:59 |
| openstackgerrit | Chris Dent proposed a change to openstack/tempest: Add scenario test for swift-proxy ceilometer middleware https://review.openstack.org/115971 | 15:00 |
| mtreinish | like 40-45min | 15:00 |
| mtreinish | I think | 15:00 |
| mtreinish | I probably should make a graph to actually figure that out | 15:00 |
| mriedem | mtreinish: just ask jogo to do that :) | 15:00 |
| *** toure has joined #openstack-qa | 15:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/110110 | 15:01 |
| mriedem | well, jedi mind trick him, like "you know it'd be cool if we had a graph of this...." | 15:01 |
| jogo | mriedem: ask me to do what? | 15:01 |
| mriedem | jogo: nothing, | 15:01 |
| jogo | mriedem: oh run time | 15:01 |
| jogo | I think graphite has that | 15:02 |
| mriedem | jogo: these aren't the irc conversations you're looking for | 15:02 |
| *** salv-orlando has joined #openstack-qa | 15:02 | |
| *** tkammer has quit IRC | 15:07 | |
| *** marun has joined #openstack-qa | 15:08 | |
| jogo | mriedem: I think sdague had graphs at one point | 15:09 |
| *** yamahata has joined #openstack-qa | 15:10 | |
| *** coolsvap has joined #openstack-qa | 15:10 | |
| mtreinish | jogo: yeah I remember seeing something like them somewhere | 15:11 |
| *** samuelbercovici has quit IRC | 15:12 | |
| *** mkollaro has quit IRC | 15:19 | |
| *** rwsu has quit IRC | 15:23 | |
| *** jhenner has quit IRC | 15:25 | |
| *** arnaud__ has joined #openstack-qa | 15:26 | |
| *** afazekas has quit IRC | 15:29 | |
| *** toure has quit IRC | 15:31 | |
| *** toure has joined #openstack-qa | 15:31 | |
| *** chandankumar has quit IRC | 15:36 | |
| *** rwsu has joined #openstack-qa | 15:39 | |
| *** jaypipes has quit IRC | 15:41 | |
| *** markus_z has quit IRC | 15:43 | |
| *** jecarey has joined #openstack-qa | 15:45 | |
| *** jlanoux_ has quit IRC | 15:48 | |
| *** yamahata has quit IRC | 15:48 | |
| *** arnaud__ has quit IRC | 15:50 | |
| *** jaypipes has joined #openstack-qa | 15:54 | |
| *** ChanServ sets mode: +o jaypipes | 15:54 | |
| *** jhenner has joined #openstack-qa | 15:54 | |
| *** SridharG has quit IRC | 15:56 | |
| lifeless | sdague: whats the bug # for the testr workers disappearing issue? I'd like to fix that for us | 15:59 |
| *** e0ne has quit IRC | 15:59 | |
| *** SridharG has joined #openstack-qa | 15:59 | |
| mtreinish | lifeless: https://bugs.launchpad.net/testrepository/+bug/1316858 | 16:00 |
| *** krtaylor has quit IRC | 16:00 | |
| *** aix has quit IRC | 16:00 | |
| lifeless | mtreinish: thanks | 16:01 |
| *** toure_ has joined #openstack-qa | 16:02 | |
| *** toure_ has quit IRC | 16:02 | |
| *** kmartin has quit IRC | 16:10 | |
| *** aix has joined #openstack-qa | 16:13 | |
| *** ildikov has quit IRC | 16:15 | |
| *** aix has quit IRC | 16:21 | |
| *** andreaf has joined #openstack-qa | 16:24 | |
| *** andreaf has quit IRC | 16:24 | |
| *** andreaf has joined #openstack-qa | 16:25 | |
| *** andreaf has quit IRC | 16:25 | |
| *** andreaf has joined #openstack-qa | 16:25 | |
| *** mriedem has quit IRC | 16:27 | |
| *** mriedem has joined #openstack-qa | 16:31 | |
| *** jpich has quit IRC | 16:36 | |
| *** mriedem has quit IRC | 16:37 | |
| *** dims_ has joined #openstack-qa | 16:38 | |
| *** dims has quit IRC | 16:40 | |
| *** krtaylor has joined #openstack-qa | 16:45 | |
| *** aimon has quit IRC | 16:45 | |
| *** jaypipes has quit IRC | 16:49 | |
| *** kmartin has joined #openstack-qa | 16:49 | |
| *** jaypipes has joined #openstack-qa | 16:55 | |
| *** ChanServ sets mode: +o jaypipes | 16:55 | |
| *** cdent has quit IRC | 16:57 | |
| *** lucasagomes has quit IRC | 16:58 | |
| *** jlibosva has quit IRC | 16:59 | |
| *** suneelb has joined #openstack-qa | 16:59 | |
| *** gszasz has quit IRC | 17:03 | |
| *** mriedem has joined #openstack-qa | 17:06 | |
| *** atiwari has joined #openstack-qa | 17:10 | |
| *** sdmonov has joined #openstack-qa | 17:11 | |
| *** aimon has joined #openstack-qa | 17:15 | |
| *** KurtMartin has joined #openstack-qa | 17:17 | |
| *** kmartin has quit IRC | 17:20 | |
| *** kmartin has joined #openstack-qa | 17:21 | |
| *** mestery has joined #openstack-qa | 17:24 | |
| *** KurtMartin has quit IRC | 17:24 | |
| *** tosky has quit IRC | 17:25 | |
| *** harlowja_away is now known as harlowja_ | 17:25 | |
| andreaf | mtreinish, ping - re test-accounts | 17:30 |
| *** mestery has quit IRC | 17:31 | |
| *** mestery has joined #openstack-qa | 17:32 | |
| *** mestery has quit IRC | 17:32 | |
| *** jhenner has quit IRC | 17:32 | |
| *** mestery has joined #openstack-qa | 17:32 | |
| *** chandankumar has joined #openstack-qa | 17:34 | |
| *** dimtruck is now known as zz_dimtruck | 17:34 | |
| *** kmartin has quit IRC | 17:35 | |
| mtreinish | andreaf: pong | 17:35 |
| andreaf | mtreinish: so the last changes I did on accounts.py seem to be fine, and teardown is mostly working | 17:36 |
| andreaf | mtreinish: however there are still a lot of failures, due to the fact that tearDownClass is always invoked now - and in many cases it tries to delete resources which are not there, and it gets AttributeError or similar | 17:37 |
| andreaf | mtreinish: so my question would be if we should keep investing effort in https://review.openstack.org/#/c/115123/7 (the safe_setup patch) and try to make it proper | 17:37 |
| *** chandankumar has quit IRC | 17:37 | |
| andreaf | mtreinish, or if you have any better idea | 17:38 |
| *** markmcclain has joined #openstack-qa | 17:38 | |
| mtreinish | andreaf: well we could do a terrible thing like a dirty_teardown decorator which will catch the exception and ignore it | 17:38 |
| andreaf | mtreinish, there are other cases where setup is already using try except, and still it's decorated with safe_setup as I did a simple search and replace | 17:38 |
| andreaf | mtreinish, no not another one :D | 17:39 |
| mtreinish | andreaf: This really brings up the need to clean up all of this mess sooner rather than later | 17:39 |
| andreaf | mtreinish, what do you think about https://review.openstack.org/#/c/115353/ ? | 17:39 |
| andreaf | mtreinish: yes it would be good also for all the new test written by new projects to have a clear framework for doing things right in this respect | 17:40 |
| mtreinish | I need to investigate what a testresources/fixtures implementation would look like and then we compare them | 17:40 |
| mtreinish | I'll work on that today or tomorrow | 17:40 |
| andreaf | ok | 17:41 |
| mtreinish | because that looks fine to me, but I just don't want to dart off in a custom way of doing things | 17:41 |
| *** lordd_ has quit IRC | 17:42 | |
| openstackgerrit | Chris Dent proposed a change to openstack/tempest: Add scenario test for swift-proxy ceilometer middleware https://review.openstack.org/115971 | 17:42 |
| *** lordd_ has joined #openstack-qa | 17:42 | |
| andreaf | mtreinish, make sense | 17:42 |
| andreaf | mtreinish, btw I will be in the meeting later on but then I'll be mostly offline for one week - I'm moving home | 17:43 |
| *** Sukhdev has joined #openstack-qa | 17:43 | |
| mtreinish | andreaf: but I definitely think we might have just reached the point where need to bite the bullet and stop using setupClass | 17:43 |
| *** mestery has quit IRC | 17:43 | |
| mtreinish | andreaf: ok, I can try to take over this stuff while you're out | 17:43 |
| mtreinish | enjoy moving :) | 17:43 |
| andreaf | thanks | 17:44 |
| *** rfolco has quit IRC | 17:49 | |
| *** tosky has joined #openstack-qa | 17:52 | |
| mtreinish | andreaf: btw, did you see my comment here about https://review.openstack.org/#/c/114636/ from before? | 17:54 |
| mtreinish | you accidently rebased the hack into that patch | 17:54 |
| *** ArxCruz has quit IRC | 17:59 | |
| *** ArxCruz_ has joined #openstack-qa | 17:59 | |
| *** e0ne has joined #openstack-qa | 17:59 | |
| *** rfolco has joined #openstack-qa | 18:04 | |
| *** markmcclain has quit IRC | 18:06 | |
| *** zz_dimtruck is now known as dimtruck | 18:11 | |
| *** morganfainberg is now known as morganfainberg_Z | 18:13 | |
| *** lordd_ has quit IRC | 18:16 | |
| *** cnesa7 has quit IRC | 18:31 | |
| *** suneelb has quit IRC | 18:32 | |
| *** wuhg has quit IRC | 18:32 | |
| *** suneelb has joined #openstack-qa | 18:33 | |
| *** suneelb has quit IRC | 18:33 | |
| *** suneelb has joined #openstack-qa | 18:33 | |
| *** jhenner has joined #openstack-qa | 18:35 | |
| *** jhenner has quit IRC | 18:37 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 18:38 | |
| *** Longgeek has quit IRC | 18:44 | |
| *** tkammer has joined #openstack-qa | 18:44 | |
| *** morganfainberg_Z is now known as morganfainberg | 18:47 | |
| *** suneelb has quit IRC | 18:55 | |
| *** sabeen has joined #openstack-qa | 19:03 | |
| *** sabeen1 has quit IRC | 19:05 | |
| *** gfidente has quit IRC | 19:06 | |
| *** sabeen1 has joined #openstack-qa | 19:06 | |
| *** sabeen has quit IRC | 19:07 | |
| *** harlowja_ has quit IRC | 19:08 | |
| *** suneelb has joined #openstack-qa | 19:08 | |
| *** jhenner has joined #openstack-qa | 19:12 | |
| *** harlowja has joined #openstack-qa | 19:15 | |
| *** harlowja has quit IRC | 19:19 | |
| *** morganfainberg has quit IRC | 19:25 | |
| *** morganfainberg has joined #openstack-qa | 19:27 | |
| *** harlowja has joined #openstack-qa | 19:28 | |
| *** arnaud has joined #openstack-qa | 19:29 | |
| *** toure has quit IRC | 19:31 | |
| *** e0ne has quit IRC | 19:33 | |
| *** harlowja has quit IRC | 19:34 | |
| *** harlowja has joined #openstack-qa | 19:40 | |
| *** Sukhdev has quit IRC | 19:42 | |
| *** SridharG has quit IRC | 19:53 | |
| *** sabeen has joined #openstack-qa | 19:54 | |
| *** sabeen1 has quit IRC | 19:57 | |
| *** jecarey has quit IRC | 19:57 | |
| *** ArxCruz_ has quit IRC | 20:03 | |
| *** sabeen has left #openstack-qa | 20:03 | |
| *** sabeen has joined #openstack-qa | 20:04 | |
| *** atiwari has quit IRC | 20:06 | |
| *** dims has joined #openstack-qa | 20:15 | |
| *** kmartin has joined #openstack-qa | 20:18 | |
| *** dims_ has quit IRC | 20:18 | |
| *** dims has quit IRC | 20:21 | |
| *** dims has joined #openstack-qa | 20:22 | |
| *** SridharG has joined #openstack-qa | 20:29 | |
| *** marun has quit IRC | 20:29 | |
| *** ArxCruz has joined #openstack-qa | 20:31 | |
| *** morganfainberg is now known as morganfainberg_Z | 20:31 | |
| adam_g | hmm | 20:32 |
| adam_g | something looks borked with XML API tests http://logs.openstack.org/71/115871/5/check/check-tempest-dsvm-virtual-ironic-nv/9c9dfb7/logs/testr_results.html.gz | 20:33 |
| adam_g | for some reason (non-isolated creds?), virtual-ironic jobs are the only ones showing it AFAICS | 20:33 |
| mriedem | dtroyer: so just remove the env var and setting here right? https://review.openstack.org/#/c/115999/1 | 20:34 |
| *** atiwari has joined #openstack-qa | 20:34 | |
| mriedem | dtroyer: i'm totally cool with that | 20:34 |
| dtroyer | mriedem: yeah, the default setting plus the bit in create_nova_conf() that sets the value | 20:36 |
| mriedem | dtroyer: will do | 20:36 |
| openstackgerrit | Matt Riedemann proposed a change to openstack-dev/devstack: Remove API_RATE_LIMIT variable since it's wrong anyway https://review.openstack.org/115999 | 20:40 |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/devstack-gate: Remove API_RATE_LIMIT setting for Tempest runs https://review.openstack.org/116001 | 20:42 |
| *** SridharG has quit IRC | 20:47 | |
| *** morganfainberg_Z is now known as morganfainberg | 20:47 | |
| *** stevebaker has quit IRC | 20:49 | |
| *** stevebaker has joined #openstack-qa | 20:49 | |
| mtreinish | adam_g: hmm I've seen something like that recently. I think it was related to the random hashseed switch | 20:53 |
| mtreinish | but that was on stable havana it shouldn't matter on master | 20:53 |
| adam_g | mtreinish, its weird, it seems to still be passing on a small percentage of slaves. wonder if there is OS-level delta between slaves ATM | 20:54 |
| mtreinish | clarkb: ^^^ ? | 20:55 |
| * adam_g tries on a dist-upgrade'd trusty machine | 20:56 | |
| mtreinish | adam_g: yeah I'm not sure because I couldn't reproduce the fail on stable havana when I tested it either | 20:57 |
| clarkb | ? | 20:57 |
| clarkb | you set hashseed to 0 right? that preserves the old behaior | 20:57 |
| mtreinish | clarkb: yeah on master | 20:57 |
| clarkb | mtreinish: you only have master | 20:58 |
| mtreinish | well there is stable havana too | 20:58 |
| *** dkranz has quit IRC | 20:58 | |
| mtreinish | but I'm thinking there is a keystone bug | 20:58 |
| clarkb | there is? you should set it there too | 20:58 |
| clarkb | unless you think your tests are clean on havana | 20:58 |
| mtreinish | clarkb: I couldn't get it to fail on stable havana | 20:58 |
| mtreinish | there is a backport up for review | 20:58 |
| mtreinish | but https://review.openstack.org/#/c/116010/ passes | 20:58 |
| *** kmartin has quit IRC | 20:59 | |
| clarkb | wait adam_g's thing is against master though right? | 20:59 |
| clarkb | ironic doens't support havana | 20:59 |
| mtreinish | so I think this xml fail that adam_g saw and the stable bit rot jobs found is unrelated | 20:59 |
| mtreinish | clarkb: yeah, but the pattern was similar to the fail on stable havana | 21:00 |
| mtreinish | I'm probably just clouding things by drawing the comparison between the 2 fails... | 21:00 |
| adam_g | clarkb, we're seeing lotsa xml failures with master, but a small percentage of runs pass | 21:00 |
| clarkb | adam_g: mtreinish: ya I would be inclined to say its a bug that is unrelated | 21:00 |
| clarkb | because hashseed=0 on master means you are functionally equivalent to the thing before tox upgrade | 21:00 |
| *** stevebaker has quit IRC | 21:01 | |
| *** stevebaker has joined #openstack-qa | 21:02 | |
| *** sabeen1 has joined #openstack-qa | 21:03 | |
| *** sabeen has quit IRC | 21:03 | |
| *** sdmonov has quit IRC | 21:11 | |
| *** aysyd has quit IRC | 21:12 | |
| *** mestery has joined #openstack-qa | 21:19 | |
| *** ArxCruz_ has joined #openstack-qa | 21:20 | |
| *** flwang has joined #openstack-qa | 21:22 | |
| *** ArxCruz has quit IRC | 21:22 | |
| flwang | mtreinish: ping | 21:22 |
| mtreinish | flwang: pong | 21:23 |
| adam_g | is screen-key.txt searchable via logstash? | 21:23 |
| flwang | mtreinish: a question about SSL for tempest scenario test | 21:23 |
| mtreinish | adam_g: it should be | 21:23 |
| mtreinish | flwang: ok | 21:23 |
| mtreinish | adam_g: but it might not be :) | 21:24 |
| adam_g | its log format is much different than the other logs | 21:24 |
| mtreinish | adam_g: https://review.openstack.org/#/c/111160/ | 21:24 |
| flwang | mtreinish: seems the clients in scenarios test doesn't honour the config entry 'disable_ssl_certificate_validation', is there any other config entry I'm missing? | 21:24 |
| adam_g | and i cant seem to generate hits | 21:24 |
| adam_g | ah! :) | 21:24 |
| mtreinish | flwang: that's probably because the scenario tests currently use the official clients and we're not handling the flag properly there | 21:25 |
| flwang | mtreinish: oh no :) | 21:25 |
| mtreinish | flwang: but with the migration to the tempest clients for the scenario tests that should sort itself out | 21:25 |
| *** mestery has quit IRC | 21:25 | |
| *** ArxCruz_ has quit IRC | 21:26 | |
| mtreinish | flwang: that's also assuming the official clients support an option like that, which they may not | 21:26 |
| flwang | mtreinish: sorry, what's that mean 'migration to the tempest clients for the scenario tests'? | 21:26 |
| mtreinish | flwang: http://specs.openstack.org/openstack/qa-specs/specs/tempest-client-scenarios.html | 21:26 |
| *** ArxCruz has joined #openstack-qa | 21:26 | |
| flwang | mtreinish: so it's a limitation for now, right? | 21:27 |
| ianw | dtroyer: what are we going to do about this f20 firewall thing. it has been broken for too long | 21:27 |
| flwang | mtreinish: any workaround? | 21:27 |
| mtreinish | flwang: yeah, basically. I'm not sure there is a workaround, although honestly we should be finishing the bp fairly soon https://review.openstack.org/#/q/status:open+project:openstack/tempest+branch:master+topic:bp/tempest-client-scenarios,n,z | 21:28 |
| *** tkammer has quit IRC | 21:29 | |
| mtreinish | there's been a lot of movement on the bp | 21:29 |
| dtroyer | ianw: the current patchset has what I consider is the base workaround. I doubt dan will remove his -1 but so far nobody else has asked for anything like that last request, which doesn't belong in this review anyway… | 21:30 |
| *** dustins has quit IRC | 21:30 | |
| flwang | i see. good to know that. btw, would you mind revisiting this https://review.openstack.org/#/c/99514/10 ? I have addressed your comments. thanks | 21:30 |
| ianw | dtroyer: i would be sympathetic if some CI or something was explicitly testing firewalld or something. i'm quite sure no such thing exists | 21:31 |
| flwang | mtreinish: to avoid another rebase, thanks a lot | 21:32 |
| mtreinish | flwang: LGTM, but you might want to talk to dmorita about the conflict with: https://review.openstack.org/#/c/107722/3/tempest/scenario/test_swift_basic_ops.py | 21:32 |
| dtroyer | I understand he wants to enable something he is developing, but it isn't released and the number of others who want/need that is rather small… | 21:32 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Set PYTHONHASHSEED=0 for the 'all' tox suite https://review.openstack.org/116103 | 21:33 |
| flwang | mtreinish: so you mean I need to rebase my patch :) haha | 21:33 |
| adam_g | mtreinish, this is the issue: https://review.openstack.org/#/c/116103/ | 21:34 |
| mtreinish | adam_g: heh, yep that'll do it I guess I missed a setenv... | 21:34 |
| mtreinish | flwang: well, either you or he will need to | 21:35 |
| flwang | mtreinish: hehe, yep, I understand. I can do that. cheers | 21:35 |
| mtreinish | flwang: fwiw I gave it a +2 :) | 21:35 |
| dtroyer | ianw: I think I'd like sdague to weigh in on it before we just move forward anyway… | 21:35 |
| flwang | mtreinish: awesome | 21:36 |
| adam_g | mtreinish, still not clear why the other testenvs below that wouldn't also require it. not 100% literate on tox's config system | 21:36 |
| *** jhenner has quit IRC | 21:36 | |
| mtreinish | adam_g: the base testenv has it set | 21:36 |
| flwang | mtreinish: btw, may I get your opinion on the image(glance) scenario test? | 21:36 |
| mtreinish | adam_g: but when you add a setenv on a child job it overrides the base one | 21:36 |
| *** suneelb has quit IRC | 21:37 | |
| mtreinish | adam_g: so the actual fix is to drop the setenv there not to add the extra hashseed | 21:37 |
| flwang | mtreinish: somebody said it's useless (https://review.openstack.org/#/c/100140/), so I'm confused again for the policy what should be implemented in scenario test | 21:37 |
| adam_g | mtreinish, ah, right. | 21:37 |
| *** suneelb has joined #openstack-qa | 21:37 | |
| flwang | mtreinish: actually, my company is relaying on tempest as the gate if there is any change against our infrastructure | 21:38 |
| flwang | mtreinish: and we are using scenario test as an important gate | 21:38 |
| flwang | mtreinish: that's why we would like to see all the key components can be covered by scenarios test | 21:39 |
| *** kmartin has joined #openstack-qa | 21:39 | |
| flwang | mtreinish: besides, as a Glance core member, I do think those cases I implemented in this patch are reasonable. So I really need some advice about what's the thing I should do in scenario test | 21:40 |
| mtreinish | flwang: ok, yeah so the line between a scenario and an api test can get blurry sometimes | 21:40 |
| mtreinish | the rule of thumb is if you're testing a user workflow it makes a good scenario test | 21:40 |
| flwang | mtreinish: yep, that's what I'm following | 21:41 |
| mtreinish | what you're doing there is testing various aspects of the glance api and just putting them in a linear order | 21:41 |
| *** krtaylor has quit IRC | 21:41 | |
| flwang | mtreinish: wow :( | 21:41 |
| mtreinish | we have isolated test cases in the api tests for all of that already | 21:41 |
| mtreinish | which i think was __afazekas point | 21:41 |
| flwang | mtreinish: so you also agree it's not necessary, is it? | 21:42 |
| mtreinish | flwang: yeah that's my gut feeling on it by taking a glance at the code | 21:42 |
| flwang | mtreinish: ok, I really respect the opinion from Tempest PTL :) | 21:43 |
| mtreinish | flwang: honestly the swift and aggregates scenarios that are there are iffy being scenario (they're right up against the line) | 21:43 |
| mtreinish | but we added them because they used to be devstack exercises | 21:43 |
| mtreinish | so we ported them to scenario tests | 21:43 |
| mtreinish | flwang: I'm curious how you're company uses tempest though | 21:43 |
| *** masayukig has joined #openstack-qa | 21:43 | |
| mtreinish | do you run it manually after an upgrade | 21:43 |
| flwang | it's integrated in our CI system | 21:44 |
| mtreinish | or do you run it in a cron job, or use it to get a heartbeat with something like nagios? | 21:44 |
| mtreinish | ah, ok | 21:44 |
| mtreinish | and you're just running the scenario tests? | 21:44 |
| mtreinish | because honestly the smoke flag was originally intended for things like your use case (but it's purpose kinda got lost over time) | 21:45 |
| flwang | no, as you mentioned, we also run the others, but you know, scenarios test is like a workflow | 21:45 |
| flwang | got it | 21:45 |
| mtreinish | flwang: yep, scenario are good tests for cloud validation, especially because most of them test integration points between projects | 21:45 |
| mtreinish | all the scenario tests are by default part of the smoke job in tox | 21:46 |
| mtreinish | flwang: oh, yeah that's another criteria, a good scenario test involves an integration point between projects like using a volume attach | 21:46 |
| mtreinish | but it's not a strict requirement | 21:47 |
| mtreinish | it get's very confusing :) | 21:47 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Ensure PYTHONHASHSEED=0 for the 'all' tox env https://review.openstack.org/116103 | 21:47 |
| mtreinish | adam_g: how did you figure out it was the hashseed causing the bug? | 21:48 |
| mtreinish | adam_g: I mean it's not set for that job but that in and of itself isn't necessarily enough to know that caused the issue | 21:49 |
| andreaf | mtreinish: I didn't get your comment about merging the hack into the patch | 21:49 |
| adam_g | mtreinish, i was running tests manually using testr/nose unable to reproduce. decided to just simulate how DSG calls tox in the ironic case, albeit with a more strict regex to target the XML tests | 21:50 |
| andreaf | mtreinish: uhm no I see what you mean | 21:50 |
| mtreinish | andreaf: https://review.openstack.org/#/c/114636/9/tempest/common/accounts.py L39 | 21:50 |
| mtreinish | adam_g: ok, did it fail with a random hashseed? | 21:50 |
| adam_g | mtreinish, yup. | 21:51 |
| mtreinish | I'm trying to understand what the problem really is because someone will have to fix it someday... | 21:51 |
| mtreinish | adam_g: ok, thanks | 21:51 |
| mtreinish | adam_g: because https://review.openstack.org/#/c/116010/ passed | 21:51 |
| *** mriedem has quit IRC | 21:51 | |
| andreaf | mtreinish, let me fix that | 21:51 |
| mtreinish | but the nightly bit rot job failed with a similar error | 21:51 |
| mtreinish | andreaf: ok cool, thanks I was going to get around to it in a bit | 21:52 |
| clarkb | adam_g: mtreinish: tox will print the hashseed it uses when you run tox. you shoudl be able to use that to check if a random hashseed is being used | 21:52 |
| adam_g | mtreinish, full disclosure: i have NFI what the random hashseed would be tickling to cause the auth errors but it certainly fixed it in my case | 21:52 |
| adam_g | clarkb, right | 21:53 |
| mtreinish | clarkb: yeah on both passing jobs on my test patch it was fairly random: PYTHONHASHSEED='1448292531' and PYTHONHASHSEED='3357279388' | 21:53 |
| clarkb | mtreinish: so why isn't it 0 like set in tox.ini? | 21:53 |
| clarkb | I feel like I am missing a lot of context | 21:53 |
| mtreinish | clarkb: the commit msg here: https://review.openstack.org/#/c/116010/ | 21:54 |
| *** ArxCruz_ has joined #openstack-qa | 21:54 | |
| mtreinish | but alan left a comment saying it failed the nightlies | 21:54 |
| clarkb | oh I see, so havana needs it and adam_g needed to stop overriding in the all target | 21:54 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Fix use of nonexistent class variable in accounts https://review.openstack.org/114636 | 21:55 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Make class setup safe in compute base admin class https://review.openstack.org/115123 | 21:55 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 21:55 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 21:55 |
| *** ArxCruz has quit IRC | 21:55 | |
| andreaf | mtreinish: done ^^^ | 21:55 |
| mtreinish | clarkb: yeah, but my test patch passed with a random hashseed and I'm trying to figure out what is getting tickled sometimes | 21:55 |
| mtreinish | andreaf: cool thanks | 21:55 |
| clarkb | mtreinish: gotcha. fwiw the failure rate could be really low if the order only slight matters | 21:56 |
| clarkb | dict ordering is assumed in lots of places :/ | 21:57 |
| adam_g | mtreinish, i think the failures with random occasionally pass | 21:57 |
| *** Sukhdev has joined #openstack-qa | 21:57 | |
| adam_g | at least thats what the ironic gate seems to show: http://no-carrier.net/~adam/openstack/ironic_gate_status.html | 21:57 |
| mtreinish | yeah these errors are a bit more severe, like sending a request with a missing xml body | 21:57 |
| clarkb | adam_g: yup its possible to pass with random seed because the resulting error may come out as expected | 21:57 |
| mtreinish | at least based on what the debug logs indicated | 21:58 |
| clarkb | adam_g: mtreinish https://review.openstack.org/#/c/116106/ is one example of a tricky error caused by it | 21:58 |
| mtreinish | clarkb: http://logs.openstack.org/47/115947/1/check/check-tempest-dsvm-postgres-full/97db330/logs/tempest.txt.gz?#_2014-08-21_12_02_14_349 | 21:58 |
| mtreinish | I'm wondering lxml is just fubar with a random hashseed | 21:59 |
| mtreinish | because from what my understanding is that shouldn't be caused by a random hashseed... | 21:59 |
| *** dkranz has joined #openstack-qa | 22:00 | |
| *** jordanP has quit IRC | 22:00 | |
| *** ArxCruz_ has quit IRC | 22:07 | |
| *** ArxCruz has joined #openstack-qa | 22:07 | |
| *** jamielennox|away is now known as jamielennox | 22:10 | |
| *** ArxCruz_ has joined #openstack-qa | 22:11 | |
| *** ArxCruz has quit IRC | 22:11 | |
| *** suneelb has quit IRC | 22:12 | |
| *** suneelb has joined #openstack-qa | 22:13 | |
| *** ArxCruz_ has quit IRC | 22:13 | |
| *** ArxCruz has joined #openstack-qa | 22:14 | |
| *** jaypipes has quit IRC | 22:16 | |
| *** marun has joined #openstack-qa | 22:17 | |
| *** jgrimm has quit IRC | 22:22 | |
| *** lordd_ has joined #openstack-qa | 22:28 | |
| *** suneelb1 has joined #openstack-qa | 22:34 | |
| *** suneelb has quit IRC | 22:34 | |
| *** suneelb1 has quit IRC | 22:39 | |
| *** suneelb has joined #openstack-qa | 22:39 | |
| openstackgerrit | Salvatore Orlando proposed a change to openstack/tempest: Allows to specify disabled extension https://review.openstack.org/116121 | 22:41 |
| openstackgerrit | Salvatore Orlando proposed a change to openstack-dev/devstack: Configure enabled and disabled extensions in tempest https://review.openstack.org/116123 | 22:44 |
| *** suneelb has quit IRC | 23:01 | |
| mtreinish | salv-orlando: so the only suggestion I had was that http://git.openstack.org/cgit/openstack/tempest/tree/tempest/cmd/verify_tempest_config.py might be useful | 23:01 |
| *** tosky has quit IRC | 23:01 | |
| mtreinish | to get a full set of extensions | 23:01 |
| mtreinish | because something I'm wary of is a bug where we introduce a regression in the extension list api and we end up skipping a bunch of tests | 23:02 |
| mtreinish | we've had something similar happen before | 23:02 |
| adam_g | any other tempest cores able to get this quicky a +A to unbreak ironic gate? https://review.openstack.org/#/c/116103/ | 23:03 |
| *** suneelb has joined #openstack-qa | 23:03 | |
| *** rbak has quit IRC | 23:03 | |
| mtreinish | masayukig, dkranz, andreaf: ^^^ | 23:03 |
| *** dimtruck is now known as zz_dimtruck | 23:03 | |
| salv-orlando | mtreinish: thanks for the input. Indeed muy probably at the moment was to find a way for listing all extensions. | 23:04 |
| salv-orlando | I came up with a different solution, but I might reconsider in light of this. | 23:04 |
| mtreinish | salv-orlando: although looking at your patch you got around my concern by adding a disabled ext option | 23:05 |
| salv-orlando | mtreinish: yup but this required a slightly convoluted devstack-gate patch (which I forgot to push) | 23:05 |
| mtreinish | salv-orlando: yeah that script get's overlooked all the time which is why I thought I'd mention it to you | 23:05 |
| *** dkranz has quit IRC | 23:08 | |
| *** masayukig has quit IRC | 23:09 | |
| *** dkranz has joined #openstack-qa | 23:10 | |
| *** atiwari has quit IRC | 23:11 | |
| *** masayukig has joined #openstack-qa | 23:12 | |
| openstackgerrit | Salvatore Orlando proposed a change to openstack-infra/devstack-gate: Add support for enabling/disabling service extensions https://review.openstack.org/116129 | 23:12 |
| adam_g | mtreinish, looks like stable/havana hits the same XML auth issues: http://logs.openstack.org/03/116103/2/check/check-grenade-dsvm-icehouse/f729de3/logs/old/testr_results.html.gz | 23:14 |
| adam_g | well, similar | 23:14 |
| *** masayukig has quit IRC | 23:14 | |
| mtreinish | adam_g: yeah that's why I'm trying to figure out why a random hashseed is causing it with: https://review.openstack.org/#/c/116010/ | 23:14 |
| mtreinish | adam_g: the backport was already proposed here https://review.openstack.org/#/c/115963/ | 23:15 |
| mtreinish | I guess I should just push it through | 23:15 |
| adam_g | mtreinish, ah, sorry. i thought 116010 was demo'ing it not being needed | 23:15 |
| *** masayukig has joined #openstack-qa | 23:15 | |
| mtreinish | adam_g: because that test patch passes and locally I couldn't reproduce | 23:15 |
| mtreinish | adam_g: yeah | 23:15 |
| mtreinish | adam_g: yeah I'm trying to figure out why it fails for everyone else but not me :) | 23:16 |
| mtreinish | but I guess that's no reason not to push it through | 23:16 |
| adam_g | mtreinish, have you run it a number of times? i noticed i was hitting it only a percentage of the time locally at one point | 23:16 |
| *** Sukhdev has quit IRC | 23:17 | |
| mtreinish | yeah I ran 3 times locally on a guest I spun up this morning | 23:17 |
| mtreinish | I got random fails, but not the auth one | 23:17 |
| mtreinish | I should have snapshot it | 23:18 |
| *** aimon has quit IRC | 23:19 | |
| mtreinish | adam_g: whatever I'll just push the backport through | 23:19 |
| adam_g | mtreinish, thanks | 23:20 |
| mtreinish | adam_g: I just hate not knowing why it fixes it | 23:21 |
| *** catohornet1 has left #openstack-qa | 23:22 | |
| adam_g | mtreinish, yeah.. | 23:22 |
| *** Sukhdev has joined #openstack-qa | 23:23 | |
| adam_g | it certainly smells like lxml problems. master shows one lxml error on the server side, havana a different one on the client side | 23:23 |
| openstackgerrit | Brant Knudson proposed a change to openstack-dev/devstack: Change httpd Keystone log level to warn https://review.openstack.org/116132 | 23:29 |
| openstackgerrit | Brant Knudson proposed a change to openstack-dev/devstack: Change httpd Keystone access log to keystone_access.log https://review.openstack.org/116133 | 23:29 |
| openstackgerrit | Brant Knudson proposed a change to openstack-dev/devstack: Include Keystone access log in key screen https://review.openstack.org/116134 | 23:29 |
| openstackgerrit | Brant Knudson proposed a change to openstack-dev/devstack: Change httpd Keystone log to keystone.log https://review.openstack.org/116135 | 23:29 |
| mtreinish | adam_g: I pushed through your tox.ini fix | 23:29 |
| * mtreinish calls it a night (at least for now) | 23:30 | |
| adam_g | mtreinish, awesome, thanks! ill keep an eye on it. goodnight | 23:35 |
| *** oomichi has joined #openstack-qa | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!