| *** dmellado has joined #openstack-qa | 00:03 | |
| openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack-dev/devstack: Fix Typos found with misspell https://review.openstack.org/126208 | 00:06 |
|---|---|---|
| *** andreaf has quit IRC | 00:15 | |
| *** andreaf has joined #openstack-qa | 00:16 | |
| *** JayJ has quit IRC | 00:24 | |
| *** JayJ has joined #openstack-qa | 00:24 | |
| *** dmorita has joined #openstack-qa | 00:25 | |
| *** mriedem has quit IRC | 00:47 | |
| *** eezhova has quit IRC | 01:01 | |
| *** eezhova has joined #openstack-qa | 01:02 | |
| *** JayJ has quit IRC | 01:17 | |
| *** yamahata has joined #openstack-qa | 01:17 | |
| *** andreaf has quit IRC | 01:19 | |
| *** andreaf has joined #openstack-qa | 01:20 | |
| *** yamamoto_ has joined #openstack-qa | 01:27 | |
| *** andreaf has quit IRC | 01:30 | |
| *** andreaf has joined #openstack-qa | 01:30 | |
| *** andreaf has quit IRC | 01:44 | |
| *** andreaf has joined #openstack-qa | 01:45 | |
| *** dimsum_ has quit IRC | 01:47 | |
| *** dimsum_ has joined #openstack-qa | 01:58 | |
| *** JayJ has joined #openstack-qa | 01:59 | |
| *** dimsum_ has quit IRC | 02:00 | |
| *** dimsum_ has joined #openstack-qa | 02:10 | |
| *** JayJ has quit IRC | 02:10 | |
| *** mestery has quit IRC | 02:12 | |
| *** mestery has joined #openstack-qa | 02:14 | |
| *** ramishra has joined #openstack-qa | 02:19 | |
| *** andreaf has quit IRC | 02:34 | |
| *** andreaf has joined #openstack-qa | 02:34 | |
| *** dimsum_ has quit IRC | 02:38 | |
| *** nikhil_k|afk has quit IRC | 02:42 | |
| *** arnaud has joined #openstack-qa | 02:59 | |
| *** gmann has quit IRC | 03:00 | |
| *** gmann has joined #openstack-qa | 03:00 | |
| *** dkranz has quit IRC | 03:15 | |
| *** dimsum_ has joined #openstack-qa | 03:16 | |
| *** dimsum_ has quit IRC | 03:30 | |
| *** dimtruck is now known as zz_dimtruck | 03:33 | |
| *** zz_dimtruck is now known as dimtruck | 03:39 | |
| *** jecarey_ has joined #openstack-qa | 03:43 | |
| *** yamahata has quit IRC | 03:43 | |
| *** yamahata has joined #openstack-qa | 03:44 | |
| *** jecarey has quit IRC | 03:45 | |
| *** oomichi has joined #openstack-qa | 03:59 | |
| *** Poornima has joined #openstack-qa | 04:04 | |
| *** andreaf has quit IRC | 04:12 | |
| *** andreaf has joined #openstack-qa | 04:12 | |
| *** Madan has joined #openstack-qa | 04:12 | |
| *** dimtruck is now known as zz_dimtruck | 04:15 | |
| *** chkumar has joined #openstack-qa | 04:15 | |
| *** maroon has joined #openstack-qa | 04:16 | |
| openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Add Ironic driver API tests https://review.openstack.org/124313 | 04:18 |
| *** nosnos has joined #openstack-qa | 04:18 | |
| *** rushiagr_away is now known as rushiagr | 04:21 | |
| openstackgerrit | chandan kumar proposed a change to openstack/tempest: enable cinder v2 api for volumetype test https://review.openstack.org/126213 | 04:25 |
| *** amotoki has joined #openstack-qa | 04:27 | |
| *** andreaf has quit IRC | 04:28 | |
| *** andreaf has joined #openstack-qa | 04:29 | |
| openstackgerrit | chandan kumar proposed a change to openstack/tempest: enable cinder v2 api for test_multi_backend https://review.openstack.org/121055 | 04:34 |
| *** rushiagr is now known as rushiagr_away | 04:47 | |
| *** oomichi has quit IRC | 04:56 | |
| openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Remove OfficialClient dependency from HACKING.rst https://review.openstack.org/126214 | 05:01 |
| *** yamahata has quit IRC | 05:09 | |
| *** yamahata has joined #openstack-qa | 05:14 | |
| *** beisner has quit IRC | 05:15 | |
| ramishra | can someone from the core pl review this tempest patch https://review.openstack.org/#/c/104473/? Requires one more +2. | 05:22 |
| gmann | nikesh: hi | 05:28 |
| *** mdbooth has quit IRC | 05:30 | |
| *** mdbooth has joined #openstack-qa | 05:34 | |
| *** chandankumar has joined #openstack-qa | 05:35 | |
| *** coolsvap|afk is now known as coolsvap | 05:35 | |
| nikesh | gmann: hi | 05:39 |
| *** e0ne has joined #openstack-qa | 05:39 | |
| *** bkopilov has quit IRC | 05:40 | |
| *** e0ne has quit IRC | 05:40 | |
| *** beisner has joined #openstack-qa | 05:42 | |
| *** k4n0 has joined #openstack-qa | 05:44 | |
| *** e0ne has joined #openstack-qa | 05:45 | |
| gmann | nikesh: regarding attribute error you face | 05:46 |
| *** e0ne has quit IRC | 05:47 | |
| gmann | nikesh: those are known issue as resource cleanup work is going on and will be fixed properly on completion of resource cleanup BP | 05:47 |
| nikesh | kkk thanks | 05:48 |
| *** rushiagr_away is now known as rushiagr | 05:48 | |
| gmann | nikesh: but till then I or Andrea can put a work around to skip those error so that tests can be executed successfully. | 05:48 |
| gmann | nikesh: ll let you know on that | 05:49 |
| *** e0ne has joined #openstack-qa | 05:51 | |
| *** arnaud has quit IRC | 05:51 | |
| *** e0ne has quit IRC | 05:55 | |
| *** bkopilov has joined #openstack-qa | 05:56 | |
| *** evgenyf has joined #openstack-qa | 05:57 | |
| *** andreaf has quit IRC | 05:57 | |
| *** andreaf has joined #openstack-qa | 05:58 | |
| *** e0ne has joined #openstack-qa | 05:59 | |
| *** e0ne has quit IRC | 06:08 | |
| *** e0ne has joined #openstack-qa | 06:13 | |
| *** e0ne has quit IRC | 06:17 | |
| *** e0ne has joined #openstack-qa | 06:22 | |
| *** e0ne has quit IRC | 06:23 | |
| openstackgerrit | Steve Martinelli proposed a change to openstack-dev/devstack: Correct swift service name in stackrc docs https://review.openstack.org/126220 | 06:25 |
| *** e0ne has joined #openstack-qa | 06:27 | |
| *** jlibosva has joined #openstack-qa | 06:32 | |
| *** SridharG has joined #openstack-qa | 06:36 | |
| *** e0ne has quit IRC | 06:42 | |
| *** e0ne has joined #openstack-qa | 06:46 | |
| *** afazekas has joined #openstack-qa | 06:53 | |
| *** nmagnezi has joined #openstack-qa | 06:54 | |
| *** e0ne has quit IRC | 06:58 | |
| nikesh | gmann:thanks | 07:00 |
| *** andreaf has quit IRC | 07:02 | |
| openstackgerrit | A change was merged to openstack/tempest: Fix arguments for method expected_success https://review.openstack.org/112828 | 07:08 |
| *** markus_z has joined #openstack-qa | 07:09 | |
| *** tkammer has joined #openstack-qa | 07:12 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Log AttributeError as 'info' instead of 'exception' https://review.openstack.org/126225 | 07:15 |
| gmann | nikesh: np! FYI - https://review.openstack.org/#/c/126225/ | 07:15 |
| *** flaper87|afk is now known as flaper87 | 07:16 | |
| nikesh | gmann: so in juno release,this will be fixed | 07:20 |
| nikesh | gmann: i am new to this gerrit | 07:20 |
| gmann | nikesh: tempest is branchless means not dependent on release. All in tempest master branch is available for all release | 07:21 |
| gmann | nikesh: once that is merged you can take latest master branch and run your tests | 07:21 |
| *** pkoniszewski has joined #openstack-qa | 07:22 | |
| *** gfidente has joined #openstack-qa | 07:25 | |
| *** swamireddy has joined #openstack-qa | 07:26 | |
| *** yamamoto_ has quit IRC | 07:29 | |
| nikesh | gmann:kkk | 07:30 |
| *** gszasz has joined #openstack-qa | 07:34 | |
| openstackgerrit | Steve Martinelli proposed a change to openstack-dev/devstack: Move swift CLI commands to OSC commands https://review.openstack.org/126226 | 07:36 |
| *** jordanP has joined #openstack-qa | 07:45 | |
| *** e0ne has joined #openstack-qa | 07:46 | |
| *** jlanoux has joined #openstack-qa | 07:48 | |
| *** e0ne has quit IRC | 07:56 | |
| *** e0ne has joined #openstack-qa | 07:57 | |
| *** nikesh has quit IRC | 08:00 | |
| *** qba73 has joined #openstack-qa | 08:00 | |
| *** safchain has joined #openstack-qa | 08:02 | |
| *** jlibosva has quit IRC | 08:02 | |
| *** jpich has joined #openstack-qa | 08:02 | |
| *** jlibosva has joined #openstack-qa | 08:04 | |
| *** jlibosva has quit IRC | 08:14 | |
| *** jlibosva has joined #openstack-qa | 08:14 | |
| *** gszasz has quit IRC | 08:18 | |
| *** ramishra has quit IRC | 08:20 | |
| *** Dafna has joined #openstack-qa | 08:22 | |
| openstackgerrit | varun kumar yadav proposed a change to openstack/tempest: Test to update port with CIDR value https://review.openstack.org/113861 | 08:28 |
| *** oomichi has joined #openstack-qa | 08:37 | |
| *** coolsvap is now known as coolsvap|afk | 08:40 | |
| *** ramishra has joined #openstack-qa | 08:41 | |
| *** Poornima has quit IRC | 08:44 | |
| *** k4n0 has quit IRC | 08:46 | |
| *** Poornima has joined #openstack-qa | 08:46 | |
| *** coolsvap|afk is now known as coolsvap | 08:51 | |
| *** salv-orlando has joined #openstack-qa | 08:52 | |
| *** gszasz has joined #openstack-qa | 08:53 | |
| *** mkollaro has joined #openstack-qa | 08:55 | |
| *** k4n0 has joined #openstack-qa | 09:00 | |
| *** cdent has joined #openstack-qa | 09:03 | |
| *** aix has joined #openstack-qa | 09:07 | |
| gmann | mtreinish, andreaf: some more instance of reporting about AttributeError today on IRC. I put a temp fix for those. Please have a look - https://review.openstack.org/#/c/126225/ | 09:09 |
| *** zz_johnthetubagu is now known as johnthetubaguy | 09:13 | |
| *** swamireddy has quit IRC | 09:16 | |
| *** e0ne has quit IRC | 09:18 | |
| *** swamireddy has joined #openstack-qa | 09:24 | |
| *** Poornima_ has joined #openstack-qa | 09:41 | |
| *** k4n0 has quit IRC | 09:41 | |
| *** Poornima has quit IRC | 09:42 | |
| *** Longgeek has joined #openstack-qa | 09:44 | |
| *** Longgeek has quit IRC | 09:46 | |
| *** Longgeek has joined #openstack-qa | 09:47 | |
| *** qba73 has quit IRC | 09:50 | |
| *** qba73 has joined #openstack-qa | 09:51 | |
| *** k4n0 has joined #openstack-qa | 09:56 | |
| *** chandankumar has quit IRC | 10:00 | |
| *** chkumar has quit IRC | 10:01 | |
| openstackgerrit | Sébastien Han proposed a change to openstack-dev/devstack: Add RAW image support for Ceph https://review.openstack.org/126250 | 10:10 |
| *** dmorita has quit IRC | 10:16 | |
| *** vrovachev has joined #openstack-qa | 10:20 | |
| openstackgerrit | Sébastien Han proposed a change to openstack-dev/devstack: Add the ability to activate Nova ceph without Cinder https://review.openstack.org/126204 | 10:21 |
| cdent | It looks like some of the variables in the grenaderc used in the gate are not getting overridden appropriately. I'm not familiar enough with all the pieces to know how to fix: https://bugs.launchpad.net/grenade/+bug/1355899 | 10:25 |
| cdent | sdague, dtroyer, jogo: ^ is yet another "masks the real issue" issue in grenade | 10:26 |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/123350 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1370496 | 10:27 |
| *** andreaf_ is now known as andreaf | 10:32 | |
| *** jlanoux has quit IRC | 10:33 | |
| *** yamahata has quit IRC | 10:33 | |
| *** yamahata has joined #openstack-qa | 10:34 | |
| *** tosky has joined #openstack-qa | 10:36 | |
| *** qba73 has quit IRC | 10:40 | |
| *** qba73 has joined #openstack-qa | 10:41 | |
| *** qba73 has quit IRC | 10:42 | |
| *** harlowja_away has quit IRC | 10:47 | |
| *** amotoki has quit IRC | 10:48 | |
| openstackgerrit | Benny Kopilov proposed a change to openstack/tempest: Test volume snapshots with multi backend setup https://review.openstack.org/126256 | 10:54 |
| *** e0ne has joined #openstack-qa | 10:58 | |
| *** vigneshvar has joined #openstack-qa | 11:01 | |
| *** qba73 has joined #openstack-qa | 11:01 | |
| *** bkopilov has quit IRC | 11:06 | |
| *** JayJ has joined #openstack-qa | 11:06 | |
| *** JayJ has quit IRC | 11:10 | |
| *** mwagner_lap has quit IRC | 11:15 | |
| *** pkoniszewski has quit IRC | 11:16 | |
| *** bkopilov has joined #openstack-qa | 11:18 | |
| openstackgerrit | A change was merged to openstack/tempest: Improve IPV6 parity in Security Group testcases https://review.openstack.org/94130 | 11:18 |
| *** dimsum_ has joined #openstack-qa | 11:19 | |
| *** chkumar has joined #openstack-qa | 11:21 | |
| *** coolsvap is now known as coolsvap|afk | 11:23 | |
| *** chandankumar has joined #openstack-qa | 11:25 | |
| *** qba73 has quit IRC | 11:29 | |
| *** qba73 has joined #openstack-qa | 11:31 | |
| *** jlanoux has joined #openstack-qa | 11:31 | |
| *** qba73 has quit IRC | 11:32 | |
| *** nosnos has quit IRC | 11:34 | |
| *** kashyap has joined #openstack-qa | 11:36 | |
| *** chandankumar has quit IRC | 11:38 | |
| *** chkumar has quit IRC | 11:38 | |
| *** qba73 has joined #openstack-qa | 11:39 | |
| *** chkumar has joined #openstack-qa | 11:40 | |
| *** krtaylor_away is now known as krtaylor | 11:40 | |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: Enforce UTF-8 encoding when creating databases https://review.openstack.org/126267 | 11:41 |
| mkoderer | sdague: ping | 11:42 |
| *** weshay has joined #openstack-qa | 11:43 | |
| *** pkoniszewski has joined #openstack-qa | 11:47 | |
| *** cmyster is now known as cmyster_away | 11:52 | |
| *** nmagnezi has quit IRC | 11:54 | |
| *** Poornima_ has quit IRC | 11:59 | |
| *** pkoniszewski has quit IRC | 11:59 | |
| *** JayJ has joined #openstack-qa | 11:59 | |
| *** qba73 has quit IRC | 12:06 | |
| *** qba73 has joined #openstack-qa | 12:07 | |
| *** chandankumar has joined #openstack-qa | 12:07 | |
| *** mmedvede has joined #openstack-qa | 12:16 | |
| *** dkranz has joined #openstack-qa | 12:19 | |
| *** qba73 has quit IRC | 12:20 | |
| openstackgerrit | A change was merged to openstack/tempest: Fix use of code name in services decorator https://review.openstack.org/122538 | 12:20 |
| *** JayJ has quit IRC | 12:21 | |
| *** qba73 has joined #openstack-qa | 12:21 | |
| *** JayJ has joined #openstack-qa | 12:22 | |
| *** nmagnezi has joined #openstack-qa | 12:29 | |
| *** mkollaro has quit IRC | 12:35 | |
| *** salv-orlando has quit IRC | 12:37 | |
| *** jaypipes has joined #openstack-qa | 12:38 | |
| *** ChanServ sets mode: +o jaypipes | 12:38 | |
| openstackgerrit | Kirill Shileev proposed a change to openstack-dev/devstack: Use ALT_INSTANCE_USER for image_alt_ssh_user https://review.openstack.org/126278 | 12:39 |
| *** rfolco has joined #openstack-qa | 12:41 | |
| *** yfujioka has quit IRC | 12:42 | |
| *** ramishra has quit IRC | 12:43 | |
| openstackgerrit | Benny Kopilov proposed a change to openstack/tempest: Glance Negative - chanage image owner https://review.openstack.org/119611 | 12:43 |
| *** dimsum_ has quit IRC | 12:45 | |
| *** dimsum_ has joined #openstack-qa | 12:45 | |
| *** chkumar has quit IRC | 12:48 | |
| *** chandankumar has quit IRC | 12:48 | |
| *** gszasz has quit IRC | 12:50 | |
| *** gszasz has joined #openstack-qa | 12:51 | |
| *** mwagner_lap has joined #openstack-qa | 12:53 | |
| *** jecarey_ has quit IRC | 12:54 | |
| *** Madan has quit IRC | 12:54 | |
| *** dimsum_ is now known as dims | 12:58 | |
| *** aix has quit IRC | 12:58 | |
| *** aix has joined #openstack-qa | 12:59 | |
| *** dkranz has quit IRC | 13:03 | |
| *** nmagnezi has quit IRC | 13:08 | |
| *** dustins has joined #openstack-qa | 13:08 | |
| *** nmagnezi has joined #openstack-qa | 13:11 | |
| *** nmagnezi_ has joined #openstack-qa | 13:13 | |
| openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack-infra/devstack-gate: pypi.openstack.org is no longer used https://review.openstack.org/126290 | 13:15 |
| *** nmagnezi has quit IRC | 13:16 | |
| *** dkranz has joined #openstack-qa | 13:16 | |
| *** nmagnezi_ has quit IRC | 13:16 | |
| *** nmagnezi has joined #openstack-qa | 13:17 | |
| EmilienM | jogo: if you have time today, the javelin2 destroy patch in grenade is waiting for +A: https://review.openstack.org/#/c/120295/ | 13:17 |
| *** aysyd has joined #openstack-qa | 13:18 | |
| *** gszasz has quit IRC | 13:20 | |
| *** gszasz has joined #openstack-qa | 13:22 | |
| *** mriedem has joined #openstack-qa | 13:29 | |
| *** chkumar has joined #openstack-qa | 13:34 | |
| *** chandankumar has joined #openstack-qa | 13:34 | |
| *** oomichi has quit IRC | 13:34 | |
| *** markmcclain has joined #openstack-qa | 13:37 | |
| *** rbak has quit IRC | 13:37 | |
| *** dhellmann is now known as dhellmann_ | 13:43 | |
| *** dhellmann_ is now known as dhellmann | 13:44 | |
| *** andreaf is now known as andreaf_ | 13:45 | |
| *** gszasz has quit IRC | 13:48 | |
| *** gszasz has joined #openstack-qa | 13:48 | |
| *** jecarey has joined #openstack-qa | 13:51 | |
| *** ramishra has joined #openstack-qa | 13:53 | |
| *** jecarey has quit IRC | 13:54 | |
| *** jecarey has joined #openstack-qa | 13:55 | |
| *** adalbas has joined #openstack-qa | 14:06 | |
| *** markmcclain has quit IRC | 14:06 | |
| *** gszasz has quit IRC | 14:06 | |
| mriedem | mtreinish: do you know if there is an easy way to tell if the project_id in this message is the admin tenant in the tempest run? | 14:11 |
| mriedem | http://logs.openstack.org/98/125398/11/check/check-tempest-dsvm-postgres-full/f0d9495/logs/screen-n-net.txt.gz#_2014-10-06_08_00_23_874 | 14:11 |
| *** kashyap has quit IRC | 14:11 | |
| *** swamireddy has quit IRC | 14:11 | |
| mriedem | not seeing it in tempest.conf so i guess it wouldn't be | 14:11 |
| *** rbak has joined #openstack-qa | 14:11 | |
| *** rbak has quit IRC | 14:12 | |
| *** rbak has joined #openstack-qa | 14:12 | |
| *** markmcclain has joined #openstack-qa | 14:14 | |
| mtreinish | mriedem: so with tenant_isolation enabled it will also make a new project and add an admin user for admin tests (or at least most of them) | 14:18 |
| mriedem | mtreinish: hmm | 14:18 |
| mriedem | ok, checking out that nova-network fixed ip over-quota bug | 14:18 |
| mriedem | going to add some debug logging to nova since i think some of what's being logged in there right now could be misleading | 14:18 |
| mriedem | for the tenant | 14:18 |
| *** gszasz has joined #openstack-qa | 14:20 | |
| mtreinish | mriedem: yeah I looked at the code and the tempest log file that create server is being done with a tenant_isolated created user/project with an admin role | 14:23 |
| mriedem | mtreinish: ah.... | 14:24 |
| mtreinish | mriedem: http://logs.openstack.org/98/125398/11/check/check-tempest-dsvm-postgres-full/f0d9495/logs/tempest.txt.gz#_2014-10-06_08_00_23_308 | 14:24 |
| mriedem | ok, and fixed_ips are a per-project quota | 14:24 |
| mriedem | now to find out what the quota limit is per project in a tempest run | 14:24 |
| mtreinish | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/admin/test_servers.py#n29 | 14:24 |
| *** safchain has quit IRC | 14:25 | |
| *** safchain has joined #openstack-qa | 14:25 | |
| mriedem | oh duh it's in the name of the gd test | 14:25 |
| mtreinish | mriedem: unless another test is screwing with it, it should be printed in the nova log file | 14:25 |
| mriedem | ServersAdminTestJSON | 14:25 |
| mriedem | it is | 14:25 |
| *** kashyap has joined #openstack-qa | 14:27 | |
| *** gszasz has quit IRC | 14:29 | |
| *** gszasz has joined #openstack-qa | 14:30 | |
| *** zz_dimtruck is now known as dimtruck | 14:30 | |
| *** coolsvap|afk is now known as coolsvap | 14:41 | |
| *** pcrews has joined #openstack-qa | 14:45 | |
| *** maroon has quit IRC | 14:46 | |
| *** marun has joined #openstack-qa | 14:46 | |
| *** aix has quit IRC | 14:48 | |
| *** dkranz has quit IRC | 14:52 | |
| *** qba73 has quit IRC | 14:53 | |
| *** markmcclain has quit IRC | 14:53 | |
| *** qba73 has joined #openstack-qa | 14:57 | |
| *** coolsvap is now known as coolsvap|afk | 14:58 | |
| *** amotoki has joined #openstack-qa | 14:58 | |
| *** aix has joined #openstack-qa | 15:00 | |
| *** dpaterson has joined #openstack-qa | 15:03 | |
| *** dkranz has joined #openstack-qa | 15:05 | |
| mtreinish | andreaf_: for the test-accounts stuff, that one failure looks like it's caused by not using the default network from config | 15:12 |
| mtreinish | is that just because we broke the condition check for the test patch? or do we need to add more logic there | 15:12 |
| *** hemnass is now known as hemna | 15:13 | |
| *** yamahata has quit IRC | 15:14 | |
| *** yfried|afk has quit IRC | 15:15 | |
| *** markus_z has quit IRC | 15:16 | |
| *** toure has joined #openstack-qa | 15:17 | |
| *** nmagnezi has quit IRC | 15:18 | |
| openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack-infra/devstack-gate: Better check for pypi mirror host connectivity https://review.openstack.org/126290 | 15:21 |
| *** tkammer has quit IRC | 15:22 | |
| *** dkranz has quit IRC | 15:24 | |
| *** tkammer has joined #openstack-qa | 15:28 | |
| openstackgerrit | Zu Qiang proposed a change to openstack-dev/devstack: Reset no_proxy when the node is installed behind a proxy https://review.openstack.org/94988 | 15:34 |
| openstackgerrit | Zu Qiang proposed a change to openstack-dev/devstack: Closes-Bug: 1291111 Closes-Bug: 1322200 Problem: ======= When devstack is installed behind a proxy with proxy variables http_proxy and https_proxy being exported and without exporting no_proxy, the installation will fail. The reason is that any REST reque https://review.openstack.org/126324 | 15:34 |
| openstackgerrit | Zu Qiang proposed a change to openstack-dev/devstack: Reset no_proxy when the node is installed behind a proxy https://review.openstack.org/126325 | 15:34 |
| *** dkranz has joined #openstack-qa | 15:38 | |
| *** mlavalle has joined #openstack-qa | 15:42 | |
| openstackgerrit | David Kranz proposed a change to openstack/qa-specs: Add spec for clients returning one value https://review.openstack.org/126004 | 15:43 |
| *** mwagner_lap has quit IRC | 15:44 | |
| *** evgenyf has quit IRC | 15:45 | |
| *** arnaud has joined #openstack-qa | 15:47 | |
| *** qba73 has quit IRC | 15:47 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Fix Typos found with misspell https://review.openstack.org/126208 | 15:49 |
| *** aimon has quit IRC | 15:49 | |
| mriedem | mtreinish: notice the quota admin negative tests that are running right before we fail with over-quota here: | 15:49 |
| mriedem | http://logs.openstack.org/98/125398/11/check/check-tempest-dsvm-postgres-full/f0d9495/console.html#_2014-10-06_08_00_28_464 | 15:49 |
| openstackgerrit | A change was merged to openstack/qa-specs: Move implemented specs into implemented dir https://review.openstack.org/125867 | 15:49 |
| mriedem | mtreinish: the quota admin tests are supposed to be running in tenant isolation though | 15:49 |
| mriedem | but that's very suspicious | 15:50 |
| openstackgerrit | Zu Qiang proposed a change to openstack-dev/devstack: Reset no_proxy when the node is installed behind a proxy https://review.openstack.org/94988 | 15:51 |
| *** amotoki has quit IRC | 15:51 | |
| *** arnaud has quit IRC | 15:52 | |
| openstackgerrit | Zu Qiang proposed a change to openstack-dev/devstack: Reset no_proxy when the node is installed behind a proxy https://review.openstack.org/94988 | 15:53 |
| *** SridharG has left #openstack-qa | 15:53 | |
| *** ramishra has quit IRC | 15:53 | |
| *** ramishra has joined #openstack-qa | 15:54 | |
| *** mwagner_lap has joined #openstack-qa | 15:54 | |
| mtreinish | mriedem: yeah, those tests should always run with an isolated tenant | 15:54 |
| mtreinish | but there could definitely be some overlap between the 2 test classes | 15:54 |
| mriedem | mtreinish: yeah, so it creates 2 intsances in setup, so that's 4 between json and xml, should be well under a default fixed_ips quota of 10 | 15:55 |
| mriedem | so i'm looking for other compute admin tests creating lots of servers in their setup | 15:55 |
| mriedem | since with 4 workers we could have 2 other tests running | 15:55 |
| *** ramishra_ has joined #openstack-qa | 15:56 | |
| mtreinish | mriedem: well, json and xml should be in separate projects, because tenant_isolation is at the class level | 15:56 |
| mtreinish | or is that 10 fips total | 15:56 |
| mriedem | 10 per project for fips | 15:57 |
| *** ramishra has quit IRC | 15:57 | |
| mriedem | PER_PROJECT_QUOTAS = ['fixed_ips', 'floating_ips', 'networks'] | 15:57 |
| mriedem | that's in the nova db api | 15:57 |
| mriedem | mtreinish: i see there are 2 test cases in ServersAdminTestJSON that also create their own server | 15:58 |
| mriedem | rather than re-use the ones in setup | 15:58 |
| mtreinish | I could believe that, there might be some issue with re-using them based on the operations being tested | 15:59 |
| mriedem | test_create_server_with_scheduling_hint looks like it needs to since it creates the instance with a scheduling hint | 15:59 |
| mtreinish | or they might have missed the migration to a shared server | 15:59 |
| mriedem | test_reset_network_inject_network_info seems like it could re-use one of the resource_setup instances | 15:59 |
| mriedem | so if ServersAdminTest json and xml are running at the same time, sharing the same tenant, you could have 8 instances | 16:00 |
| mriedem | throw in 2 other compute admin tests creating at least more than 2 instances and kaboom | 16:00 |
| openstackgerrit | Steve Martinelli proposed a change to openstack-dev/devstack: Correct swift service name in stackrc docs https://review.openstack.org/126220 | 16:00 |
| mriedem | *at least more than one each | 16:00 |
| mtreinish | mriedem: but they don't share the same tenant | 16:01 |
| mtreinish | or they shouldn't be, because it gets done during setupclass | 16:01 |
| *** gszasz has quit IRC | 16:02 | |
| mriedem | ok so in the tenant isolation case the os_adm attribute is created with admin creds from tempest.conf right? | 16:02 |
| *** ramishra_ has quit IRC | 16:02 | |
| *** ramishra has joined #openstack-qa | 16:03 | |
| mtreinish | mriedem: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/base.py#n391 and http://git.openstack.org/cgit/openstack/tempest/tree/tempest/common/isolated_creds.py#n251 | 16:04 |
| mtreinish | each setup class should create a new project, user/password pair and assign the admin role to the user | 16:05 |
| mriedem | mtreinish: isn't that only called via http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/base.py#n395 though? | 16:06 |
| mriedem | creds = cls.isolated_creds.get_admin_creds() | 16:06 |
| *** coolsvap|afk is now known as coolsvap | 16:06 | |
| mriedem | i'm trying to figure out where this goes: | 16:06 |
| mriedem | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/base.py#n400 | 16:06 |
| mtreinish | L400 is if tenant_isolation is disabled | 16:06 |
| mriedem | right | 16:07 |
| mriedem | which it is in gate runs right? | 16:07 |
| mtreinish | no, the gate runs with tenant_isolation enabled | 16:07 |
| mtreinish | otherwise we wouldn't be able to run things in parallel | 16:07 |
| mriedem | oh nvm, backwargs | 16:07 |
| mriedem | *backwards | 16:07 |
| mriedem | right, ok, so tenant isolation is disabled unless your test class has the force_tenant_isolation flag, which the quota admin tests do | 16:07 |
| *** ramishra has quit IRC | 16:08 | |
| mriedem | so from here http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/base.py#n400 - don't i get here? http://git.openstack.org/cgit/openstack/tempest/tree/tempest/clients.py#n484 | 16:08 |
| mriedem | and to me, it looks like it's getting shared creds for compute admin | 16:08 |
| mtreinish | mriedem: tenant_isolation is a config flag which by default is False, but in the gate is set to True | 16:08 |
| mtreinish | L400 should not get executed in the gate, it should be L395 | 16:09 |
| mtreinish | you are correct though that it will use shared creds from the config file if tenant isolation is disabled | 16:09 |
| *** arnaud has joined #openstack-qa | 16:10 | |
| mriedem | oh i see http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/base.py#n393 | 16:10 |
| mriedem | allow_tenant_isolation = True | 16:10 |
| mriedem | yeah | 16:10 |
| mriedem | damn | 16:10 |
| mriedem | well gfdi | 16:10 |
| mriedem | :) | 16:10 |
| mtreinish | fwiw, we're working on getting project reuse working soon with bp test-accounts. We're quite close (only 1 test fails) to getting that landed | 16:12 |
| *** e0ne has quit IRC | 16:12 | |
| mtreinish | err, project reuse while running in parallel | 16:13 |
| mriedem | that seems rife for over-quota | 16:13 |
| *** aimon has joined #openstack-qa | 16:14 | |
| mtreinish | mriedem: yeah, it forces us to be good about cleanup. https://review.openstack.org/#/c/114578/ as a first pass it works pretty well though | 16:15 |
| mtreinish | mriedem: you also need to specify 2*N sets of creds where N is the number of workers being used. Otherwise you'll get resource contentions | 16:16 |
| *** arnaud has quit IRC | 16:17 | |
| *** vigneshvar has quit IRC | 16:17 | |
| mtreinish | heh, although this doesn't help get to the bottom of the overquota issue | 16:18 |
| *** salv-orlando has joined #openstack-qa | 16:18 | |
| mtreinish | mriedem: my first thought on those issues is always cross-interaction with default quota tests | 16:18 |
| mriedem | mtreinish: me too, and that test does run shortly before we fail | 16:19 |
| mriedem | but that test adds 100 to the default quota limit, so that shouldn't be an issue, especially when it's not a negative test | 16:19 |
| *** marun has quit IRC | 16:20 | |
| *** Longgeek has quit IRC | 16:21 | |
| mriedem | mtreinish: anyway, i've got some nova patches that should help with logging when we see it again, we weren't logging anything useful in there before - the OverQuota exception actually contains the numbers for the quota limit, the number used and how much room is left, etc, we should be logging all that | 16:24 |
| *** tkammer has quit IRC | 16:25 | |
| mriedem | mtreinish: i'd also like some kind of threshold warning when reserving quota on a resource, e.g. if i get to 90% quota use on fixed_ips for my admin tenant, i should start logging a warning or something | 16:25 |
| mriedem | we could walk that back to when we start tipping over | 16:25 |
| mtreinish | mriedem: yeah that would be really useful | 16:27 |
| *** jlanoux has quit IRC | 16:31 | |
| mriedem | mtreinish: http://lists.openstack.org/pipermail/openstack-dev/2014-October/047868.html | 16:33 |
| mtreinish | mriedem: cool | 16:35 |
| *** jpich has quit IRC | 16:35 | |
| *** jlibosva has quit IRC | 16:37 | |
| *** safchain has quit IRC | 16:39 | |
| *** rushiagr is now known as rushiagr_away | 16:39 | |
| *** rwsu has joined #openstack-qa | 16:44 | |
| *** mpaolino has joined #openstack-qa | 16:47 | |
| *** afazekas has quit IRC | 16:48 | |
| *** aix has quit IRC | 16:49 | |
| *** arnaud has joined #openstack-qa | 16:50 | |
| *** chkumar has quit IRC | 16:57 | |
| *** chandankumar has quit IRC | 16:58 | |
| *** mdbooth has quit IRC | 17:00 | |
| *** e0ne has joined #openstack-qa | 17:01 | |
| *** bknudson has joined #openstack-qa | 17:01 | |
| *** mkollaro has joined #openstack-qa | 17:02 | |
| *** suneelb has joined #openstack-qa | 17:02 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-dev/devstack: DO NOT MERGE: see what happens with tempest if we don't set quotas https://review.openstack.org/126358 | 17:04 |
| *** mdbooth has joined #openstack-qa | 17:06 | |
| *** jaypipes has quit IRC | 17:11 | |
| *** chkumar has joined #openstack-qa | 17:11 | |
| *** chandankumar has joined #openstack-qa | 17:12 | |
| *** jgrimm is now known as zz_jgrimm | 17:14 | |
| *** e0ne has quit IRC | 17:15 | |
| *** rushiagr_away is now known as rushiagr | 17:17 | |
| *** e0ne has joined #openstack-qa | 17:17 | |
| mkoderer | mtreinish: which load_test part did you mean? | 17:20 |
| mkoderer | I found a old message from you in my buffer ;) what it about https://review.openstack.org/121910 | 17:20 |
| *** cnesa16 has quit IRC | 17:21 | |
| *** akuznetsova has quit IRC | 17:22 | |
| *** coolsvap is now known as coolsvap|afk | 17:27 | |
| *** akuznetsova has joined #openstack-qa | 17:27 | |
| *** jaypipes has joined #openstack-qa | 17:27 | |
| *** ChanServ sets mode: +o jaypipes | 17:27 | |
| *** dustins has quit IRC | 17:33 | |
| *** harlowja has joined #openstack-qa | 17:34 | |
| *** tosky has quit IRC | 17:34 | |
| *** e0ne has quit IRC | 17:35 | |
| mtreinish | mkoderer: the modifications to scenario.utils | 17:43 |
| mtreinish | if there was no config file the scenario tests that use that were causing an issue with the test loader | 17:44 |
| mtreinish | I think what I did there to workaround it is safe, just wanted a 2nd opinion | 17:44 |
| *** jordanP has quit IRC | 17:45 | |
| *** chkumar has quit IRC | 17:49 | |
| *** chandankumar has quit IRC | 17:50 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 17:51 | |
| *** mpaolino has quit IRC | 17:51 | |
| mkoderer | mtreinish: I need to have a closer look tomorrow | 17:54 |
| *** kashyap has quit IRC | 17:54 | |
| mtreinish | mkoderer: cool, thanks | 17:55 |
| mkoderer | mtreinish: and I need some reviews on https://review.openstack.org/#/c/115273/ and https://review.openstack.org/#/c/120033/ | 17:55 |
| *** melwitt has joined #openstack-qa | 17:58 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for gate-osc-dsvm-functional bug 1378040 https://review.openstack.org/126374 | 18:03 |
| *** gordc has joined #openstack-qa | 18:06 | |
| *** akuznetsova has quit IRC | 18:08 | |
| *** zz_jgrimm is now known as jgrimm | 18:11 | |
| *** akuznetsova has joined #openstack-qa | 18:12 | |
| *** rushiagr is now known as rushiagr_away | 18:17 | |
| *** rushiagr_away is now known as rushiagr | 18:21 | |
| gordc | does anyone here know of a way/library to unittest an elasticsearch backend? | 18:22 |
| *** mkollaro has quit IRC | 18:27 | |
| *** rushiagr is now known as rushiagr_away | 18:31 | |
| *** e0ne has joined #openstack-qa | 18:41 | |
| mtreinish | jaypipes: thanks a bunch for the subunit2sql reviews | 18:50 |
| mtreinish | I didn't even think about a timestamp, I was so busy trying to figure out how to get autoincrementing to work, I didn't think about any alternatives | 18:51 |
| jaypipes | mtreinish: sorry for the delay on those! | 18:51 |
| *** arnaud__ has joined #openstack-qa | 18:52 | |
| mtreinish | jaypipes: what delay? I think I only added you as a reviewer on friday? That's plenty quick :) | 18:53 |
| jaypipes | mtreinish: well, yeah... that's still 4 days :) | 18:55 |
| *** rockyg has joined #openstack-qa | 18:55 | |
| *** jordanP has joined #openstack-qa | 18:58 | |
| *** mriedem1 has joined #openstack-qa | 19:00 | |
| openstackgerrit | A change was merged to openstack/tempest: Reduce complexity during import phase https://review.openstack.org/115273 | 19:00 |
| openstackgerrit | A change was merged to openstack/tempest: Fix schema definition admin_flavor_create https://review.openstack.org/124988 | 19:00 |
| *** mriedem has quit IRC | 19:01 | |
| *** beekneemech is now known as bnemec | 19:12 | |
| *** andreaf has joined #openstack-qa | 19:19 | |
| *** cdent has quit IRC | 19:19 | |
| *** atiwari has joined #openstack-qa | 19:22 | |
| *** atiwari has quit IRC | 19:24 | |
| *** atiwari has joined #openstack-qa | 19:24 | |
| *** atiwari has quit IRC | 19:26 | |
| *** andreaf has quit IRC | 19:27 | |
| *** atiwari has joined #openstack-qa | 19:27 | |
| *** atiwari has quit IRC | 19:28 | |
| *** atiwari has joined #openstack-qa | 19:28 | |
| *** atiwari has quit IRC | 19:28 | |
| *** atiwari has joined #openstack-qa | 19:28 | |
| *** mkollaro has joined #openstack-qa | 19:29 | |
| *** dustins_ has joined #openstack-qa | 19:33 | |
| *** salv-orlando_ has joined #openstack-qa | 19:39 | |
| *** salv-orlando has quit IRC | 19:39 | |
| *** salv-orlando_ is now known as salv-orlando | 19:39 | |
| *** andreaf has joined #openstack-qa | 19:41 | |
| andreaf | sdague, mtreinish, cyeoh, masayukig: got disconnected, to sure if the link was sent: https://review.openstack.org/#/q/topic:+bp/resource-cleanup+status:open,n,z | 19:42 |
| *** salv-orlando_ has joined #openstack-qa | 19:44 | |
| *** jordanP has quit IRC | 19:46 | |
| *** salv-orlando has quit IRC | 19:46 | |
| *** salv-orlando_ is now known as salv-orlando | 19:46 | |
| mtreinish | andreaf: reviewed, I'm thinking we should just repurpose the unit test no setupclass hacking rule | 19:48 |
| mtreinish | otherwise we'll have 2 checks that do the same basic thing | 19:48 |
| andreaf | mtreinish: well we would be enforcing no tearDownClass in unit tests as well which we are not now | 19:50 |
| andreaf | mtreinish: but I guess that would be ok... the two checks are doing the same thing but for slightly different reasons I think | 19:50 |
| mtreinish | andreaf: yeah the reasons are different, but the check is the same | 19:51 |
| mtreinish | you could adjust the msg based on the path if you want to highlight that, but I don't think it's needed | 19:51 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add order column to runs table https://review.openstack.org/117861 | 19:51 |
| mtreinish | jaypipes: ^^^ that makes things much simpler. The timestamp was a good call | 19:52 |
| jaypipes | mtreinish: coolio.. will re-review shortly. | 19:52 |
| *** cnesa16 has joined #openstack-qa | 19:52 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add db api function to get the latest run https://review.openstack.org/117882 | 19:53 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Fix get_duration db api method https://review.openstack.org/126113 | 19:53 |
| andreaf | mtreinish: uhm ok so you want to keep T105 only and extend it to tearDownClass as well | 19:53 |
| andreaf | mtreinish: I'll post an update | 19:53 |
| *** cnesa16 has quit IRC | 19:55 | |
| mtreinish | andreaf: yeah, I think we only need one rule for it. My faulty assumption with T105 was that if there was no setupClass there would be no teardownclass | 19:55 |
| mtreinish | so it only checked one of them | 19:55 |
| *** cnesa15 has joined #openstack-qa | 19:57 | |
| *** torandu has quit IRC | 19:57 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add migration to populate run_time where needed https://review.openstack.org/126114 | 19:59 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for gate-osc-dsvm-functional bug 1378040 https://review.openstack.org/126374 | 19:59 |
| mtreinish | andreaf: btw, did you have any comments on the sample config file stuff? otherwise I'll push it through | 20:00 |
| andreaf | mtreinish: it looks good - I guess nothing prevents one from using a single config file for all commands, or? | 20:00 |
| mtreinish | andreaf: yeah nothing stops that. The only reason I did separate samples was because of the cli options for things like run metadata | 20:01 |
| *** jordanP has joined #openstack-qa | 20:01 | |
| mtreinish | but you would normally only put that on the cli I think | 20:01 |
| andreaf | mtreinish: yes most likely | 20:02 |
| mtreinish | the only thing that I think would be useful in a config file is the db settings and those are the same for all the commands | 20:02 |
| andreaf | yes exactly - if you deploy with puppet in infra you can have the DB settings in hiera and put them in the config file, the rest can be CLI options | 20:03 |
| mtreinish | andreaf: heh, yep: https://review.openstack.org/108003 | 20:04 |
| *** openstackrecheck has quit IRC | 20:04 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add migration to populate run_time where needed https://review.openstack.org/126114 | 20:04 |
| *** openstackrecheck has joined #openstack-qa | 20:05 | |
| *** dustins_ is now known as dustins | 20:05 | |
| andreaf | mtreinish: ok you're doing that already :) | 20:06 |
| *** suneelb has quit IRC | 20:08 | |
| *** suneelb has joined #openstack-qa | 20:08 | |
| *** melwitt has quit IRC | 20:08 | |
| *** melwitt has joined #openstack-qa | 20:09 | |
| *** mkollaro has quit IRC | 20:10 | |
| openstackgerrit | A change was merged to openstack-infra/subunit2sql: Remove unneccesary state_path config option https://review.openstack.org/125822 | 20:13 |
| *** rockyg has quit IRC | 20:13 | |
| jaypipes | mtreinish: re-reviewed :) | 20:14 |
| mtreinish | jaypipes: cool, thanks. I'll respin it with your suggestions | 20:14 |
| andreaf | mtreinish: are you ok to keep the two unit tests for the hacking rule or do you want them to be merged as well | 20:16 |
| mtreinish | andreaf: well if you're adding a check for tearDownclass I think tests for that will be nice | 20:19 |
| mtreinish | and you're changing the ignored path to test.py | 20:19 |
| mtreinish | but other then that I think what is there should be sufficient | 20:19 |
| *** JayJ has quit IRC | 20:21 | |
| *** JayJ has joined #openstack-qa | 20:22 | |
| andreaf | mtreinish: ok | 20:22 |
| *** mriedem1 has quit IRC | 20:22 | |
| *** mriedem has joined #openstack-qa | 20:23 | |
| *** JayJ has quit IRC | 20:28 | |
| *** JayJ has joined #openstack-qa | 20:28 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add sample config files for commands https://review.openstack.org/124832 | 20:30 |
| *** JayJ has quit IRC | 20:33 | |
| *** JayJ has joined #openstack-qa | 20:33 | |
| mriedem | mtreinish: well, disabling quotas in nova makes the quota tests unhappy https://review.openstack.org/#/c/126358/ - that's shocking | 20:35 |
| *** JayJ has quit IRC | 20:38 | |
| *** JayJ has joined #openstack-qa | 20:38 | |
| *** adalbas has quit IRC | 20:40 | |
| *** toure has quit IRC | 20:42 | |
| *** e0ne has quit IRC | 20:42 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/126084 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 20:42 |
| *** JayJ has quit IRC | 20:43 | |
| openstackgerrit | A change was merged to openstack-infra/subunit2sql: Add sample config files for commands https://review.openstack.org/124832 | 20:43 |
| *** JayJ has joined #openstack-qa | 20:43 | |
| *** cnesa15 has quit IRC | 20:45 | |
| *** dustins has quit IRC | 20:47 | |
| *** JayJ has quit IRC | 20:48 | |
| *** JayJ has joined #openstack-qa | 20:48 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Hacking rule to forbid resource unsafe fixtures https://review.openstack.org/121554 | 20:49 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Drop unused safe_setup decorator https://review.openstack.org/122531 | 20:49 |
| andreaf | mtreinish: ^^^ | 20:49 |
| andreaf | mtreinish: they have to go through check... https://review.openstack.org/#/c/126134/ is ready though | 20:50 |
| *** JayJ has quit IRC | 20:53 | |
| *** JayJ has joined #openstack-qa | 20:53 | |
| *** JayJ has quit IRC | 20:58 | |
| *** JayJ has joined #openstack-qa | 20:58 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack-infra/subunit2sql: Cleanup some extra spaces https://review.openstack.org/126416 | 21:02 |
| *** suneelb has quit IRC | 21:02 | |
| *** melwitt has quit IRC | 21:02 | |
| *** melwitt has joined #openstack-qa | 21:03 | |
| *** suneelb has joined #openstack-qa | 21:03 | |
| *** suneelb has quit IRC | 21:03 | |
| *** JayJ has quit IRC | 21:03 | |
| *** suneelb has joined #openstack-qa | 21:03 | |
| *** melwitt has quit IRC | 21:04 | |
| *** JayJ has joined #openstack-qa | 21:04 | |
| *** melwitt has joined #openstack-qa | 21:04 | |
| andreaf | mtreinish: first ~contribution to subunit2sql https://review.openstack.org/126416 | 21:05 |
| mtreinish | andreaf: awesome | 21:06 |
| andreaf | mtreinish: the 2nd will be better I hope :P | 21:06 |
| openstackgerrit | A change was merged to openstack-infra/subunit2sql: Cleanup some extra spaces https://review.openstack.org/126416 | 21:07 |
| openstackgerrit | David Kranz proposed a change to openstack/tempest: Fix use of kwargs instead of **kwargs https://review.openstack.org/126417 | 21:08 |
| *** JayJ has quit IRC | 21:08 | |
| *** JayJ has joined #openstack-qa | 21:09 | |
| mtreinish | andreaf: +A/+2 | 21:10 |
| andreaf | mtreinish, yay | 21:10 |
| mtreinish | andreaf: well, except for the last patch in the resource-cleanup branch, just needs something in the commit msg besides removing the decorator :) | 21:13 |
| *** hogepodge has joined #openstack-qa | 21:13 | |
| *** JayJ has quit IRC | 21:13 | |
| mtreinish | hogepodge: hey | 21:14 |
| *** JayJ has joined #openstack-qa | 21:14 | |
| mtreinish | hogepodge: so the tempest docs are here: http://docs.openstack.org/developer/tempest/ | 21:14 |
| mtreinish | with a description of the types of tests here: http://docs.openstack.org/developer/tempest/field_guide/index.html | 21:15 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Drop unused safe_setup decorator https://review.openstack.org/122531 | 21:15 |
| *** JayJ has quit IRC | 21:15 | |
| mtreinish | mriedem: yeah that's not really surprising, do you have a link to your test results? | 21:15 |
| andreaf | mtreinish, ^^^ | 21:16 |
| mtreinish | I'm wondering if that's something we need to fix in the quota tests | 21:16 |
| mtreinish | andreaf: +2 | 21:16 |
| mriedem | mtreinish: http://logs.openstack.org/58/126358/1/check/check-tempest-dsvm-full/9580a75/console.html | 21:17 |
| mriedem | it's all quota tests | 21:17 |
| hogepodge | thanks mtreinish. I'm looking at getting more Keystone test coverage in Tempest that captures basic and essential functionality. Part of the goal being having a standard to carry over to the defcore process. | 21:17 |
| hogepodge | So this is the list of things we're looking at writing or porting over tests for | 21:18 |
| *** JayJ has joined #openstack-qa | 21:18 | |
| hogepodge | https://etherpad.openstack.org/p/keystone-defcore | 21:18 |
| mtreinish | hogepodge: so I think defcore is only using api tests. So for keystone it would be this dir: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/identity | 21:19 |
| mtreinish | right now it's pretty scarce and they're all admin tests | 21:19 |
| *** zz_johnthetubagu is now known as johnthetubaguy | 21:19 | |
| mtreinish | which I don't think refstack uses | 21:20 |
| mtreinish | davidlenwell: ^^^ is that right? | 21:20 |
| *** dkranz has quit IRC | 21:22 | |
| *** yjiang5 has joined #openstack-qa | 21:22 | |
| hogepodge | mtreinish: this is a good start. Thanks. | 21:23 |
| *** JayJ has quit IRC | 21:24 | |
| *** JayJ has joined #openstack-qa | 21:25 | |
| *** atiwari has quit IRC | 21:26 | |
| mtreinish | hogepodge: sure, np. I was talking to lbragstad about this stuff recently too. So I'd also check with him to see if he has any suggestions | 21:27 |
| lbragstad | mtreinish: o/ | 21:27 |
| * lbragstad reading back | 21:27 | |
| *** mmedvede has quit IRC | 21:27 | |
| hogepodge | hi lbragstad | 21:28 |
| *** flaper87 is now known as flaper87|afk | 21:28 | |
| lbragstad | hogepodge: hey | 21:28 |
| *** mwagner_lap has quit IRC | 21:28 | |
| lbragstad | functional test refactor? | 21:28 |
| hogepodge | so I'm working for the Foundation now as the Interop Engineer, which includes work on DefCore. Trying to figure out how to get better coverage of keystone into it. | 21:29 |
| lbragstad | cool! | 21:29 |
| hogepodge | So key APIs that we would want to validate basic and essential functionality with | 21:29 |
| * morganfainberg throws something across the table at hogepodge | 21:29 | |
| morganfainberg | lbragstad, yeah. | 21:30 |
| hogepodge | (once you have one keystone guy somewhere, you know there are at least a dozen more in the shadows) | 21:30 |
| hogepodge | So this is a bit of an introduction of myself. Also part of what I want to do is make sure I'm integrated in the qa and infra communities, as well as the project communities in a meaningful and helpful way, especially in regards to interop testing (both upstream and downstream) | 21:31 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/120172 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1352498 https://bugs.launchpad.net/bugs/1374458, gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1352498 https://bugs.launchpad.net/bugs/1374458 | 21:31 |
| *** dpaterson has quit IRC | 21:33 | |
| *** JayJ has quit IRC | 21:33 | |
| *** JayJ has joined #openstack-qa | 21:33 | |
| *** JayJ has quit IRC | 21:37 | |
| *** JayJ has joined #openstack-qa | 21:38 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 21:39 | |
| mtreinish | andreaf: if you have time do you want to review and push through (if they're good) https://review.openstack.org/#/c/124938/ and https://review.openstack.org/#/c/124939/ ? | 21:41 |
| mtreinish | I was just going to fiat them today, and fix bugs if they came up | 21:41 |
| mtreinish | but now you can do it :) | 21:41 |
| andreaf | mtreinish: looking... | 21:42 |
| *** jgrimm is now known as zz_jgrimm | 21:42 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/devstack: Stop n-cpu by correct process name /w fake https://review.openstack.org/126420 | 21:43 |
| *** gordc has quit IRC | 21:43 | |
| *** mriedem has quit IRC | 21:45 | |
| andreaf | mtreinish: where does test_migrations.py come from? so I can compare changes | 21:46 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add order column to runs table https://review.openstack.org/117861 | 21:47 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add db api function to get the latest run https://review.openstack.org/117882 | 21:47 |
| mtreinish | andreaf: glance and nova is what I based it on | 21:47 |
| mtreinish | andreaf: http://git.openstack.org/cgit/openstack/glance/tree/glance/tests/unit/test_migrations.py and http://git.openstack.org/cgit/openstack/nova/tree/nova/tests/db/test_migrations.py | 21:48 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Fix get_duration db api method https://review.openstack.org/126113 | 21:50 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add migration to populate run_time where needed https://review.openstack.org/126114 | 21:51 |
| *** JayJ has quit IRC | 21:52 | |
| *** JayJ has joined #openstack-qa | 21:53 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/devstack: Stop n-cpu by correct process name /w fake https://review.openstack.org/126420 | 21:55 |
| *** mmedvede has joined #openstack-qa | 22:04 | |
| openstackgerrit | Salvatore Orlando proposed a change to openstack-dev/devstack: Tempest: configure exact set of extensions to test https://review.openstack.org/126422 | 22:06 |
| openstackgerrit | Salvatore Orlando proposed a change to openstack-infra/devstack-gate: Add support for enabling/disabling service extensions https://review.openstack.org/116129 | 22:07 |
| *** edwa6643 has joined #openstack-qa | 22:08 | |
| andreaf | mtreinish: so things like _walk_versions you didn't consume then from oslo.db - any special reason? | 22:11 |
| mtreinish | andreaf: because there isn't a version in oslo.db with alembic yet | 22:12 |
| mtreinish | I could have used some of the fixtures from oslo.db with test scenarios for the opportunistic testing, but I prefer how it looks being explicit like I did it there. | 22:13 |
| mtreinish | I guess that could be a TODO refactor later though | 22:13 |
| andreaf | mtreinish: yes that's something that can be refactored also if alembic gets supported in oslo.db | 22:16 |
| andreaf | mtreinish, at high level it looks good | 22:16 |
| andreaf | mtreinish: I can't say I appreciate all the details of the change though, but it looks quite reasonable, and it's passing tests | 22:16 |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/devstack: Stop n-cpu by correct process name /w fake https://review.openstack.org/126420 | 22:17 |
| mtreinish | andreaf: heh, that was basically going to be my self approve comment. Something like: "It works so lets push it through we can fix issues if they come up later" | 22:17 |
| mtreinish | because I'm sure I missed things | 22:18 |
| *** bknudson has quit IRC | 22:20 | |
| *** jordanP has quit IRC | 22:21 | |
| andreaf | +2 to both | 22:22 |
| mtreinish | andreaf: ok, you could have +Ad them too :) | 22:22 |
| andreaf | done - oh double now :D | 22:23 |
| openstackgerrit | A change was merged to openstack-infra/subunit2sql: Add migration tests with data https://review.openstack.org/124938 | 22:26 |
| openstackgerrit | A change was merged to openstack-infra/subunit2sql: Preseed data for migration tests until migration 163fd5aa1380 https://review.openstack.org/124939 | 22:26 |
| *** JayJ has quit IRC | 22:27 | |
| *** JayJ has joined #openstack-qa | 22:27 | |
| openstackrecheck | openstack/trove change: https://review.openstack.org/124777 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1353962 | 22:28 |
| *** edwa66431 has joined #openstack-qa | 22:28 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add order column to runs table https://review.openstack.org/117861 | 22:30 |
| andreaf | mtreinish: it looks like another setUpClass has sneaked through :( https://review.openstack.org/#/c/121554/14 | 22:32 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add db api function to get the latest run https://review.openstack.org/117882 | 22:32 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Fix get_duration db api method https://review.openstack.org/126113 | 22:32 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add migration to populate run_time where needed https://review.openstack.org/126114 | 22:32 |
| *** edwa6643 has quit IRC | 22:33 | |
| mtreinish | andreaf: sigh, we need to land that branch soon | 22:33 |
| andreaf | mtreinish: so once https://review.openstack.org/#/c/126134/2 merges I will propose a new patch | 22:33 |
| mtreinish | otherwise things will just keep slipping in | 22:33 |
| *** dimtruck is now known as zz_dimtruck | 22:33 | |
| andreaf | yes I wonder if it should be a separate patch of if I should put the cleanup together with enforcing the rule | 22:34 |
| *** ianw_pto is now known as ianw | 22:34 | |
| andreaf | mtreinish: well 1h to go at least before the gate is done, so it will be tomorrow, it's late enough for me today | 22:36 |
| *** dims has quit IRC | 22:36 | |
| mtreinish | andreaf: if it's just the one slip I'd just combine them and change the commit msg summary to mention fixing it too | 22:36 |
| mtreinish | andreaf: sure, it is kinda late for you | 22:37 |
| *** dims has joined #openstack-qa | 22:37 | |
| andreaf | ok I will combine them | 22:37 |
| *** jecarey has quit IRC | 22:40 | |
| *** dims has quit IRC | 22:41 | |
| *** skata_ has joined #openstack-qa | 22:42 | |
| *** Dafna has quit IRC | 22:49 | |
| *** rwsu has quit IRC | 22:51 | |
| *** JayJ has quit IRC | 22:51 | |
| *** dims has joined #openstack-qa | 22:56 | |
| *** dims has quit IRC | 22:56 | |
| *** dims has joined #openstack-qa | 22:57 | |
| *** markmcclain has joined #openstack-qa | 22:58 | |
| *** gfidente has quit IRC | 22:58 | |
| *** hemna is now known as hemnass | 23:02 | |
| *** rbak has quit IRC | 23:03 | |
| *** rwsu has joined #openstack-qa | 23:04 | |
| *** aimon has quit IRC | 23:18 | |
| *** dims has quit IRC | 23:31 | |
| openstackgerrit | A change was merged to openstack/tempest: Remove Resp status code checks in stress tests https://review.openstack.org/125570 | 23:31 |
| *** dims has joined #openstack-qa | 23:31 | |
| *** markmcclain has quit IRC | 23:32 | |
| *** markmcclain has joined #openstack-qa | 23:35 | |
| *** markmcclain1 has joined #openstack-qa | 23:35 | |
| *** dims has quit IRC | 23:36 | |
| openstackgerrit | A change was merged to openstack/tempest: Remove setUpClass added after cleanup https://review.openstack.org/126134 | 23:36 |
| *** bknudson has joined #openstack-qa | 23:37 | |
| *** markmcclain has quit IRC | 23:39 | |
| *** dims has joined #openstack-qa | 23:40 | |
| *** bknudson has quit IRC | 23:41 | |
| *** arnaud has quit IRC | 23:46 | |
| *** arnaud__ has quit IRC | 23:46 | |
| *** melwitt has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!