| *** lordd_ has joined #openstack-qa | 00:00 | |
| *** oomichi has joined #openstack-qa | 00:00 | |
| *** aimon has quit IRC | 00:05 | |
| *** lordd_ has quit IRC | 00:05 | |
| openstackgerrit | Alessandro Pilotti proposed a change to openstack/tempest: Fixes Hyper-V basic scenario volume partition name https://review.openstack.org/114820 | 00:12 |
|---|---|---|
| openstackgerrit | A change was merged to openstack-dev/devstack: Rename Marconi to Zaqar https://review.openstack.org/111399 | 00:12 |
| openstackgerrit | A change was merged to openstack/tempest: Deacrease the required image store size by 4 GiB https://review.openstack.org/114901 | 00:15 |
| openstackgerrit | A change was merged to openstack/tempest: Add os-networks JSON client for tempest https://review.openstack.org/107552 | 00:16 |
| *** yamahata has joined #openstack-qa | 00:18 | |
| *** aysyd has quit IRC | 00:26 | |
| *** dmorita has joined #openstack-qa | 00:27 | |
| *** aimon has joined #openstack-qa | 00:28 | |
| *** harlowja is now known as harlowja_away | 00:29 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Fix use of nonexistent class variable in accounts.py https://review.openstack.org/114636 | 00:36 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Make class setup safe in compute base admin class https://review.openstack.org/115123 | 00:36 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 00:36 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 00:36 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Supports DEFAULT group opts in TempestPrivateConfig https://review.openstack.org/114609 | 00:36 |
| mtreinish | andreaf: ^^^ we missed a couple of NotImplemented exceptions in the scenarios | 00:37 |
| *** rwsu has quit IRC | 00:37 | |
| mtreinish | although I needed to rebase because of the aggregates scenario conversion | 00:37 |
| *** gmann has quit IRC | 00:41 | |
| *** rwsu has joined #openstack-qa | 00:50 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 00:51 |
| *** suneelb has quit IRC | 00:53 | |
| *** andreaf_ has joined #openstack-qa | 00:54 | |
| *** dims has joined #openstack-qa | 00:54 | |
| *** harlowja_away is now known as harlowja | 00:56 | |
| *** andreaf has quit IRC | 00:56 | |
| *** catohornet1 has quit IRC | 01:06 | |
| *** gmann has joined #openstack-qa | 01:08 | |
| *** topshare has joined #openstack-qa | 01:11 | |
| *** nosnos has joined #openstack-qa | 01:47 | |
| *** oomichi has quit IRC | 01:48 | |
| *** mitz has quit IRC | 01:48 | |
| *** oomichi has joined #openstack-qa | 01:57 | |
| *** arnaud__ has quit IRC | 02:01 | |
| *** mwagner_lap has joined #openstack-qa | 02:10 | |
| *** morganfainberg is now known as morganfainberg_Z | 02:14 | |
| *** yjiang5 has quit IRC | 02:19 | |
| *** Poornima_ has joined #openstack-qa | 02:26 | |
| *** Krast_ has quit IRC | 02:26 | |
| *** Krast has joined #openstack-qa | 02:26 | |
| *** arnaud has quit IRC | 02:35 | |
| *** dims has quit IRC | 02:35 | |
| *** amotoki_ has quit IRC | 02:37 | |
| *** arnaud has joined #openstack-qa | 02:41 | |
| *** dims has joined #openstack-qa | 02:42 | |
| *** dims has quit IRC | 02:47 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Allow for grenade upgrades within a release https://review.openstack.org/111094 | 02:49 |
| *** adalbas has quit IRC | 03:01 | |
| *** dmorita has quit IRC | 03:03 | |
| *** dmorita has joined #openstack-qa | 03:04 | |
| *** jecarey has joined #openstack-qa | 03:06 | |
| *** cmyster has joined #openstack-qa | 03:11 | |
| *** cmyster has quit IRC | 03:11 | |
| *** cmyster has joined #openstack-qa | 03:11 | |
| *** dims has joined #openstack-qa | 03:12 | |
| *** wuhg has joined #openstack-qa | 03:16 | |
| *** dims has quit IRC | 03:17 | |
| *** markmcclain has quit IRC | 03:24 | |
| *** Sukhdev has joined #openstack-qa | 03:27 | |
| *** dmorita has quit IRC | 03:28 | |
| *** dmorita has joined #openstack-qa | 03:29 | |
| *** amotoki has joined #openstack-qa | 03:33 | |
| *** topshare has quit IRC | 03:34 | |
| *** topshare has joined #openstack-qa | 03:35 | |
| *** Poornima_ has quit IRC | 03:43 | |
| santosh_ | gmann: hi | 03:56 |
| *** xianghuihui has quit IRC | 04:03 | |
| *** xianghuihui has joined #openstack-qa | 04:03 | |
| *** harlowja is now known as harlowja_away | 04:08 | |
| gmann | santosh_: hi | 04:09 |
| *** Santosh has joined #openstack-qa | 04:11 | |
| Santosh | gmann: So thanks for the comments ..you shared.... | 04:12 |
| Santosh | gmann: i have incorporated the same.... | 04:12 |
| Santosh | gmann: now waiting for +1 .. from your side ... if everything is ok with you :) | 04:12 |
| *** santosh_ has quit IRC | 04:13 | |
| *** SridharG has joined #openstack-qa | 04:14 | |
| *** Poornima_ has joined #openstack-qa | 04:18 | |
| *** xianghuihui has quit IRC | 04:18 | |
| gmann | Santosh: just 1 comment for better commit msg | 04:19 |
| gmann | other than that it looks fine :) | 04:19 |
| *** xianghui has joined #openstack-qa | 04:21 | |
| Santosh | gmann: sure .. actually this will be my first openstack commit... | 04:23 |
| Santosh | gmann: so may not be aware about every procedure of these things... | 04:23 |
| Santosh | gmann: will learn and take care in furtue... :) | 04:23 |
| gmann | Santosh: np!. :) | 04:27 |
| Santosh | gmann: how can i take care of '-1' .... now :( ...pls guide | 04:28 |
| Puneet_ | afazekas: hi | 04:29 |
| Puneet_ | afazekas: Can you please review https://review.openstack.org/#/c/114885/ | 04:29 |
| Puneet_ | oomichi: Hi | 04:29 |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Update volume client check success https://review.openstack.org/114783 | 04:32 |
| gmann | Santosh: just change the commit msg with one in review comment | 04:32 |
| gmann | Santosh: do- git commit --amend | 04:33 |
| gmann | then change the commit msg and git review. | 04:33 |
| Santosh | ok will do it gmann | 04:33 |
| Santosh | thnx | 04:33 |
| gmann | Santosh: np | 04:33 |
| Santosh | gmann: :-) | 04:33 |
| openstackgerrit | Santosh Kumar proposed a change to openstack/tempest: Add a check after creation of network https://review.openstack.org/112831 | 04:34 |
| coolsvap | gmann, please have a look at https://review.openstack.org/#/c/90306/ | 04:34 |
| Santosh | gmann: i have updated the commit msg. | 04:35 |
| gmann | coolsvap: Hi, i will look into this today. | 04:36 |
| Puneet_ | gmann: please also have a look at https://review.openstack.org/#/c/114885/ and https://review.openstack.org/#/c/114897/ | 04:36 |
| gmann | Puneet_: ok | 04:39 |
| Puneet_ | gmann: thanks | 04:41 |
| *** saurabh has joined #openstack-qa | 04:42 | |
| *** saurabh is now known as Guest10897 | 04:42 | |
| *** Guest10897 has quit IRC | 04:45 | |
| openstackgerrit | Santosh Kumar proposed a change to openstack/tempest: Add network's status check after creation of network https://review.openstack.org/112831 | 04:51 |
| *** oomichi has left #openstack-qa | 04:52 | |
| Puneet_ | andreaf_: Hi | 04:53 |
| Puneet_ | andreaf_: can you please review https://review.openstack.org/#/c/114885/ and https://review.openstack.org/#/c/114897/ | 04:54 |
| *** arnaud has quit IRC | 05:00 | |
| *** dims has joined #openstack-qa | 05:01 | |
| *** topshare has quit IRC | 05:05 | |
| *** dims has quit IRC | 05:06 | |
| *** Krast has quit IRC | 05:08 | |
| *** pcrews has quit IRC | 05:08 | |
| *** topshare has joined #openstack-qa | 05:10 | |
| *** afazekas has quit IRC | 05:17 | |
| *** arnaud has joined #openstack-qa | 05:26 | |
| *** topshare has quit IRC | 05:26 | |
| *** Sukhdev has quit IRC | 05:27 | |
| *** arnaud has quit IRC | 05:31 | |
| *** topshare has joined #openstack-qa | 05:34 | |
| *** k4n0 has joined #openstack-qa | 05:36 | |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Update volume client check success https://review.openstack.org/114783 | 05:36 |
| *** morganfainberg_Z is now known as morganfainberg | 05:41 | |
| *** SridharG has quit IRC | 05:47 | |
| *** rwsu has quit IRC | 05:51 | |
| *** ildikov has quit IRC | 05:58 | |
| openstackgerrit | Emily Hugenbruch proposed a change to openstack/tempest: Add services tags to tempest/cli https://review.openstack.org/111068 | 05:59 |
| openstackgerrit | yunhong-jiang proposed a change to openstack/tempest: Test resource claim scenario https://review.openstack.org/115176 | 06:01 |
| Santosh | gmann: Jenkin is passed... can you pls have a look.. | 06:03 |
| *** arnaud has joined #openstack-qa | 06:04 | |
| *** rwsu has joined #openstack-qa | 06:07 | |
| *** saurabh_ has joined #openstack-qa | 06:08 | |
| *** saurabh_ has quit IRC | 06:09 | |
| *** saurabh_ has joined #openstack-qa | 06:09 | |
| *** saurabh_ has joined #openstack-qa | 06:09 | |
| openstackgerrit | Rohan Kanade proposed a change to openstack/tempest: Move API response success check to Neutron client https://review.openstack.org/114186 | 06:09 |
| *** Krast has joined #openstack-qa | 06:14 | |
| *** SridharG has joined #openstack-qa | 06:15 | |
| *** arnaud__ has joined #openstack-qa | 06:16 | |
| *** Krast has quit IRC | 06:28 | |
| raies | hi | 06:29 |
| *** rdekel1 has joined #openstack-qa | 06:29 | |
| *** lordd_ has joined #openstack-qa | 06:32 | |
| *** arnaud has quit IRC | 06:40 | |
| *** arnaud__ has quit IRC | 06:40 | |
| *** nmagnezi has joined #openstack-qa | 06:55 | |
| *** topshare has quit IRC | 06:56 | |
| *** ildikov has joined #openstack-qa | 06:58 | |
| openstackgerrit | Emily Hugenbruch proposed a change to openstack/tempest: Add service tags to tempest/api/orchestration/stacks https://review.openstack.org/110429 | 06:59 |
| *** topshare has joined #openstack-qa | 07:03 | |
| *** chandankumar has joined #openstack-qa | 07:04 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: test_hotplug_nic wait for the guest https://review.openstack.org/115195 | 07:11 |
| *** afazekas has joined #openstack-qa | 07:12 | |
| afazekas | http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOiBcInRlbXBlc3Quc2NlbmFyaW8udGVzdF9uZXR3b3JrX2Jhc2ljX29wcy5UZXN0TmV0d29ya0Jhc2ljT3BzLnRlc3RfaG90cGx1Z19uaWNcIiBBTkQgbWVzc2FnZTogXCJGQUlMRURcIiIsImZpZWxkcyI6W10sIm9mZnNldCI6MCwidGltZWZyYW1lIjoiNjA0ODAwIiwiZ3JhcGhtb2RlIjoiY291bnQiLCJ0aW1lIjp7InVzZXJfaW50ZXJ2YWwiOjB9LCJzdGFtcCI6MTQwODQzMTY4NjA2NH0= | 07:12 |
| afazekas | 38 failure in 7 day | 07:12 |
| *** mwagner_lap has quit IRC | 07:13 | |
| *** gmann_ has joined #openstack-qa | 07:13 | |
| *** ildikov has quit IRC | 07:15 | |
| *** gmann has quit IRC | 07:15 | |
| *** ildikov_ has joined #openstack-qa | 07:15 | |
| *** jlibosva has joined #openstack-qa | 07:18 | |
| *** Longgeek has joined #openstack-qa | 07:26 | |
| *** mitz has joined #openstack-qa | 07:49 | |
| *** jlanoux has joined #openstack-qa | 07:57 | |
| *** aix has joined #openstack-qa | 07:57 | |
| *** gszasz has joined #openstack-qa | 07:59 | |
| *** gszasz has quit IRC | 08:01 | |
| *** gszasz has joined #openstack-qa | 08:02 | |
| *** jpich has joined #openstack-qa | 08:02 | |
| *** Longgeek has quit IRC | 08:04 | |
| *** Longgeek has joined #openstack-qa | 08:05 | |
| *** Longgeek has quit IRC | 08:06 | |
| *** Longgeek has joined #openstack-qa | 08:06 | |
| *** gszasz has quit IRC | 08:06 | |
| *** markus_z has joined #openstack-qa | 08:17 | |
| *** bkopilov has joined #openstack-qa | 08:17 | |
| *** qba73 has joined #openstack-qa | 08:18 | |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: (DO NOT MERGE) test run using mysqlconnector https://review.openstack.org/114207 | 08:20 |
| *** jordanP has joined #openstack-qa | 08:21 | |
| *** Krast has joined #openstack-qa | 08:22 | |
| *** danfinch has quit IRC | 08:26 | |
| *** aix has quit IRC | 08:31 | |
| coolsvap | afazekas, please have a look at https://review.openstack.org/90306 and https://review.openstack.org/114783, I have updated according to comments | 08:36 |
| 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 | 08:37 |
| 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 | 08:38 |
| *** Longgeek has quit IRC | 08:42 | |
| *** Longgeek has joined #openstack-qa | 08:43 | |
| afazekas | s/BaseVolumeAdminTest/BaseVolumeV2AdminTest/ | 08:46 |
| *** Longgeek has quit IRC | 08:47 | |
| coolsvap | afazekas, but the BaseVolume is for version 2 | 08:49 |
| *** Longgeek has joined #openstack-qa | 08:49 | |
| afazekas | ok | 08:50 |
| *** yamahata has quit IRC | 08:57 | |
| *** aix has joined #openstack-qa | 08:59 | |
| afazekas | coolsvap: The 114783 readded a moved test | 08:59 |
| *** cdent has joined #openstack-qa | 08:59 | |
| *** gmann has joined #openstack-qa | 08:59 | |
| *** gmann_ has quit IRC | 09:00 | |
| coolsvap | afazekas, checking | 09:01 |
| coolsvap | afazekas, i think its very latest patch :(, this should have been blocked since it does not meet the client check success :-/ | 09:03 |
| coolsvap | afazekas, updating my patch to include the changes for extend | 09:04 |
| afazekas | ok, thx | 09:04 |
| *** rfolco has quit IRC | 09:07 | |
| *** nmagnezi has quit IRC | 09:07 | |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Update volume client check success https://review.openstack.org/114783 | 09:08 |
| *** andreaf_ has quit IRC | 09:10 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: test_hotplug_nic wait for the guest https://review.openstack.org/115195 | 09:10 |
| *** mkollaro has joined #openstack-qa | 09:13 | |
| afazekas | mkoderer, andreaf, mtreinish, dkanz, yfried, psedlak, cyeoh, salv-orlando: https://review.openstack.org/115195 | 09:14 |
| psedlak | afazekas: isn't there better place for check_new_nic instead of 'embedding' it like this (in the midle of code)? | 09:18 |
| openstackgerrit | prabhu murthy proposed a change to openstack/tempest: Added test for DVR device owner check using port https://review.openstack.org/113455 | 09:18 |
| openstackgerrit | prabhu murthy proposed a change to openstack/tempest: Add API tests for Neutron DVR extension https://review.openstack.org/98235 | 09:18 |
| *** nmagnezi has joined #openstack-qa | 09:20 | |
| *** bkopilov has quit IRC | 09:24 | |
| openstackgerrit | prabhu murthy proposed a change to openstack/tempest: Added test to verify dvr snat port creation https://review.openstack.org/115222 | 09:24 |
| *** nmagnezi has quit IRC | 09:24 | |
| *** cnesa7 has joined #openstack-qa | 09:27 | |
| *** kashyap is now known as kashyapc | 09:34 | |
| *** dims has joined #openstack-qa | 09:35 | |
| *** dims has quit IRC | 09:39 | |
| *** andreaf has joined #openstack-qa | 09:42 | |
| Santosh | afazekas: hi | 09:47 |
| Santosh | Poornima_: hi | 09:49 |
| *** tosky has joined #openstack-qa | 09:52 | |
| Santosh | afazekas: request you to please review - https://review.openstack.org/#/c/112831/ | 09:58 |
| *** sdmonov has joined #openstack-qa | 10:00 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: test_hotplug_nic wait for the guest https://review.openstack.org/115195 | 10:01 |
| afazekas | Santosh: I am unsure about is the network MUST be in ACTIVE status after creation immediately | 10:02 |
| afazekas | salv-orlando: ^ | 10:03 |
| Santosh | afazekas: I have observed that ... as soon as network is created it comes into active state... | 10:03 |
| Santosh | salv-orlando: ^^^ | 10:04 |
| salv-orlando | Santosh: that is the default behaviour implemented by the db classes, and by the ML2 plugin as well. | 10:05 |
| salv-orlando | with the OVS agent mech driver that works fine. | 10:05 |
| salv-orlando | but there might be mech driver which need to do network provisioning. | 10:05 |
| salv-orlando | if the driver is not able to handle network status in pre or post commit, I think it’s a mech driver problem | 10:05 |
| *** gszasz has joined #openstack-qa | 10:06 | |
| afazekas | salv-orlando: So probbaly in this change https://review.openstack.org/#/c/112831/ we should wait for 'ACTIVE' even it is not necessary with ovs/ml2 ? | 10:07 |
| Santosh | salv-orlando: thanks , so in that case use case is valid and test should fail..... correct ? | 10:07 |
| salv-orlando | Santosh: idk… I will review your patch. But in general I welcome this kind of changes. | 10:09 |
| salv-orlando | I would love if you do the same for ports - distinguishing between vifs and router interface, and floating ips | 10:09 |
| salv-orlando | If you have not done that already | 10:09 |
| Santosh | salv-orlando: thanks , please let land this patch.. i will do this for port in another patch. | 10:10 |
| salv-orlando | Santosh: I am not tempest core so it’s not me that you should ask for that | 10:14 |
| Puneet_ | Santosh: good job santosh | 10:15 |
| openstackgerrit | Kirill Shileev proposed a change to openstack/tempest: Allows choosing ip_version for scenario tests https://review.openstack.org/112336 | 10:16 |
| Santosh | salv-orlando: i agree , however i will really appreciate .. if you can give your review comments ( -/+1 ) | 10:17 |
| *** qba73 has quit IRC | 10:17 | |
| Puneet_ | santosh i have given +1 | 10:19 |
| *** qba73 has joined #openstack-qa | 10:19 | |
| *** Longgeek has quit IRC | 10:24 | |
| *** danfinch has joined #openstack-qa | 10:25 | |
| *** jlanoux has quit IRC | 10:28 | |
| *** andreaf has quit IRC | 10:33 | |
| *** zul has quit IRC | 10:36 | |
| *** sdmonov has quit IRC | 10:43 | |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Updated the baremetal client for bp client-checks-success https://review.openstack.org/115238 | 10:43 |
| gmann | afazekas: salv-orlando: Santosh: for https://review.openstack.org/#/c/112831/, as API return code is 201 IMO test should not have wait for that. If network does not come in active state then something wrong with drivers or status code. | 10:48 |
| gmann | and i think that's what test should test. | 10:48 |
| *** cdent has quit IRC | 10:51 | |
| *** mitz has quit IRC | 10:51 | |
| *** rdekel1 has quit IRC | 10:52 | |
| *** mitz has joined #openstack-qa | 10:53 | |
| *** Longgeek has joined #openstack-qa | 10:53 | |
| *** rdekel1 has joined #openstack-qa | 10:54 | |
| *** rdekel1 has left #openstack-qa | 10:54 | |
| *** rdekel1 has joined #openstack-qa | 10:54 | |
| *** gmann has quit IRC | 10:56 | |
| *** gmann has joined #openstack-qa | 10:57 | |
| afazekas | gmann: 201 means the resource created, but does not means it is created with foo bar attribute. after 201 the subsequent GET to the resource needs to succeed | 10:58 |
| *** topshare has quit IRC | 10:58 | |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Updated the baremetal client for bp client-checks-success https://review.openstack.org/115238 | 10:59 |
| afazekas | https://wiki.openstack.org/wiki/Neutron/APIv2-specification 'status String N/A R N/A N/A ' , from this we can know status is a string (if present) and read only, but nothing else | 11:01 |
| gmann | afazekas: I see..... Ya you are right. Agree we should have wait there. | 11:05 |
| gmann | afazekas: Thanks :) | 11:07 |
| Santosh | afazekas: salv-orlando : please suggest , what to do ... current patch is okay ..or need to do any modification | 11:09 |
| *** lordd_ has quit IRC | 11:16 | |
| *** dims has joined #openstack-qa | 11:17 | |
| afazekas | Santosh: I'll try to ask it on #openstack-neutron | 11:18 |
| Santosh | afazekas: thanks | 11:19 |
| openstackgerrit | prabhu murthy proposed a change to openstack/tempest: Added test to verify dvr snat port creation https://review.openstack.org/115222 | 11:19 |
| *** andreaf has joined #openstack-qa | 11:19 | |
| openstackgerrit | prabhu murthy proposed a change to openstack/tempest: Added test for DVR device owner check using port https://review.openstack.org/113455 | 11:21 |
| *** Longgeek has quit IRC | 11:26 | |
| *** Longgeek has joined #openstack-qa | 11:26 | |
| openstackgerrit | prabhu murthy proposed a change to openstack/tempest: Added test for DVR device owner check using port https://review.openstack.org/113455 | 11:28 |
| *** sdmonov has joined #openstack-qa | 11:29 | |
| *** lordd_ has joined #openstack-qa | 11:29 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: add network resources https://review.openstack.org/100169 | 11:39 |
| *** lordd_ has quit IRC | 11:40 | |
| *** salv-orlando_ has joined #openstack-qa | 11:43 | |
| *** salv-orlando has quit IRC | 11:43 | |
| *** salv-orlando_ is now known as salv-orlando | 11:43 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: add security groups support https://review.openstack.org/104984 | 11:49 |
| *** salv-orlando has quit IRC | 11:49 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: implement resource destruction https://review.openstack.org/104986 | 11:52 |
| *** salv-orlando has joined #openstack-qa | 11:55 | |
| *** dkranz has joined #openstack-qa | 11:56 | |
| *** dims has quit IRC | 11:56 | |
| *** weshay has joined #openstack-qa | 11:57 | |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Change default size for strings to 1 https://review.openstack.org/115258 | 11:58 |
| EmilienM | jlibosva: quick review :) | 11:59 |
| *** weshay is now known as weshay_mtg | 12:00 | |
| *** tkammer has quit IRC | 12:01 | |
| *** Krast has quit IRC | 12:02 | |
| *** rdekel1 has quit IRC | 12:04 | |
| *** rdekel1 has joined #openstack-qa | 12:04 | |
| *** sdague has joined #openstack-qa | 12:04 | |
| jlibosva | EmilienM: that was copied from patchset 7, I was about to send it and saw you submitted another patch :) | 12:05 |
| openstackgerrit | Kirill Shileev proposed a change to openstack/tempest: Allows choosing ip_version for scenario tests https://review.openstack.org/112336 | 12:08 |
| *** dims has joined #openstack-qa | 12:09 | |
| *** kashyapc is now known as kashyap | 12:10 | |
| openstackgerrit | Flavio Percoco proposed a change to openstack-dev/devstack: Install glance.store from git instead of pypi https://review.openstack.org/115265 | 12:12 |
| openstackgerrit | Flavio Percoco proposed a change to openstack-dev/devstack: Install glance.store from git instead of pypi https://review.openstack.org/115265 | 12:15 |
| *** kashyap is now known as kashyapc | 12:16 | |
| *** tkammer has joined #openstack-qa | 12:16 | |
| openstackgerrit | Flavio Percoco proposed a change to openstack-dev/devstack: Install glance.store from git instead of pypi https://review.openstack.org/115265 | 12:16 |
| *** dmorita has quit IRC | 12:17 | |
| *** nmagnezi has joined #openstack-qa | 12:20 | |
| openstackgerrit | Cedric Brandily proposed a change to openstack/tempest: Add provider extension positive test cases https://review.openstack.org/103498 | 12:20 |
| *** rdekel1 has quit IRC | 12:21 | |
| *** rfolco has joined #openstack-qa | 12:22 | |
| *** cmyster has quit IRC | 12:23 | |
| *** aysyd has joined #openstack-qa | 12:26 | |
| *** rdekel1 has joined #openstack-qa | 12:31 | |
| *** adalbas has joined #openstack-qa | 12:32 | |
| *** rdekel1 has quit IRC | 12:35 | |
| *** dims has quit IRC | 12:37 | |
| *** dims has joined #openstack-qa | 12:38 | |
| *** ylobankov has quit IRC | 12:38 | |
| openstackgerrit | A change was merged to openstack/tempest: Add MatchesDictExceptForKeys custom matcher https://review.openstack.org/112115 | 12:38 |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Change default size for strings to 1 https://review.openstack.org/115258 | 12:39 |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add functools.wraps on all function decorators https://review.openstack.org/113899 | 12:39 |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Reduce complexity during import phase https://review.openstack.org/115273 | 12:39 |
| openstackgerrit | Rohan Kanade proposed a change to openstack/tempest: Move API response success check to Neutron client https://review.openstack.org/114186 | 12:49 |
| *** dkranz has quit IRC | 12:54 | |
| *** nosnos has quit IRC | 12:56 | |
| *** dkranz has joined #openstack-qa | 12:57 | |
| *** jgrimm has joined #openstack-qa | 13:00 | |
| *** Poornima_ has quit IRC | 13:02 | |
| *** amotoki has quit IRC | 13:03 | |
| *** rdekel1 has joined #openstack-qa | 13:06 | |
| *** timello has joined #openstack-qa | 13:08 | |
| *** tkammer has quit IRC | 13:08 | |
| *** dustins has joined #openstack-qa | 13:11 | |
| openstackgerrit | Udi Kalifon proposed a change to openstack/tempest: keystone sanity test, creating a user in a domain and uses his credentials https://review.openstack.org/115279 | 13:14 |
| *** mriedem has joined #openstack-qa | 13:15 | |
| *** toure has joined #openstack-qa | 13:16 | |
| afazekas | mkoderer, andreaf, mtreinish, dkanz, yfried, psedlak, cyeoh, salv-orlando: https://review.openstack.org/115195 | 13:16 |
| openstackgerrit | Udi Kalifon proposed a change to openstack/tempest: keystone sanity test, creating a user in a domain and uses his credentials https://review.openstack.org/115279 | 13:18 |
| *** cdent has joined #openstack-qa | 13:20 | |
| *** ilyashakhat has joined #openstack-qa | 13:21 | |
| *** tkammer has joined #openstack-qa | 13:21 | |
| *** ildikov_ has quit IRC | 13:23 | |
| *** cdent has quit IRC | 13:27 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: add network resources https://review.openstack.org/100169 | 13:28 |
| *** jecarey has quit IRC | 13:28 | |
| *** jaypipes has joined #openstack-qa | 13:29 | |
| *** ChanServ sets mode: +o jaypipes | 13:29 | |
| *** malini has joined #openstack-qa | 13:29 | |
| Puneet_ | hi | 13:30 |
| Puneet_ | could anyone please confirm error handling in patch https://review.openstack.org/#/c/114940/1/tempest/cli/simple_read_only/test_neutron.py correct for this case :-) | 13:31 |
| malini | mtreinish, dkranz: Can you please review https://review.openstack.org/#/c/114980/ when you have a moment? We need this to unblock Zaqar (Marconi) gate. | 13:31 |
| *** pcrews has joined #openstack-qa | 13:33 | |
| *** pcrews_ has joined #openstack-qa | 13:34 | |
| *** pcrews__ has joined #openstack-qa | 13:35 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: implement resource destruction https://review.openstack.org/104986 | 13:36 |
| mtreinish | malini: it looks like that skips all the zaqar tests: http://logs.openstack.org/80/114980/1/experimental/check-tempest-dsvm-zaqar/c5d4dba/console.html#_2014-08-19_01_37_41_329 | 13:37 |
| mtreinish | was that the intent | 13:37 |
| malini | oops..no | 13:37 |
| malini | Let me chk that..tht was a good catch :) | 13:37 |
| *** pcrews has quit IRC | 13:38 | |
| *** ildikov_ has joined #openstack-qa | 13:38 | |
| *** pcrews_ has quit IRC | 13:39 | |
| *** timello has quit IRC | 13:40 | |
| *** pcrews__ has quit IRC | 13:41 | |
| *** pcrews has joined #openstack-qa | 13:43 | |
| openstackgerrit | Cedric Brandily proposed a change to openstack/tempest: Add multi-provider extension positive test cases https://review.openstack.org/104526 | 13:44 |
| openstackgerrit | Cedric Brandily proposed a change to openstack/tempest: Add partial specs (multi-)provider extension positive test cases https://review.openstack.org/105234 | 13:45 |
| *** HenryG_ has joined #openstack-qa | 13:46 | |
| *** zz_dimtruck is now known as dimtruck | 13:46 | |
| *** nmagnezi has quit IRC | 13:47 | |
| *** HenryG has quit IRC | 13:47 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: add network resources https://review.openstack.org/100169 | 13:51 |
| *** flaper87 has joined #openstack-qa | 13:57 | |
| flaper87 | hey guys, any hints how to debug this? http://logs.openstack.org/65/115265/3/check/gate-tempest-dsvm-large-ops/ba64713/logs/devstacklog.txt.gz Not sure why I'm hitting that RECLONE issue. Clone seems to succeeded | 13:58 |
| openstackgerrit | Alessandro Pilotti proposed a change to openstack/tempest: Fixes Hyper-V basic scenario volume partition name https://review.openstack.org/114820 | 13:59 |
| *** nmagnezi has joined #openstack-qa | 14:00 | |
| mtreinish | flaper87: it's probably something wrong with the intial image. Error on clone is set to true in the gate so it will fail if the repo isn't on the image already | 14:00 |
| *** cpallares has joined #openstack-qa | 14:00 | |
| flaper87 | mtreinish: ahhhh, that means I need to submit a patch to infra first, right? | 14:01 |
| flaper87 | or set RECLONE to False | 14:01 |
| flaper87 | mhh, I wonder if that hack would be accepted | 14:01 |
| mtreinish | sure I guess, I'm missing the context of what you're trying to do :) | 14:02 |
| flaper87 | mtreinish: I need to install glance.store as develop (just like glance is installed). I looked into how oslo packages do this and followed the same logic. I'm not sure why I would need the projects to be precloned, though. | 14:03 |
| *** sdmonov has quit IRC | 14:04 | |
| mtreinish | it's a networking thing for the gate. If we went out to the network for each run flaky fails go way up because of random outages | 14:04 |
| *** marun has joined #openstack-qa | 14:04 | |
| flaper87 | gotcha | 14:05 |
| *** marun is now known as marun_afk | 14:05 | |
| mtreinish | flaper87: I'd ask on infra, because I'm not sure of all the mechanics involved with adding a new repo to the nodepool image | 14:05 |
| flaper87 | mtreinish: thanks :) | 14:06 |
| *** HenryG_ is now known as HenryG | 14:07 | |
| *** markmcclain has joined #openstack-qa | 14:07 | |
| *** enikanorov__ has joined #openstack-qa | 14:09 | |
| openstackgerrit | Joseph Lanoux proposed a change to openstack/tempest: Migrate test_snapshot_pattern to tempest client https://review.openstack.org/110426 | 14:09 |
| openstackgerrit | Joseph Lanoux proposed a change to openstack/tempest: Migrate test_volume_boot_pattern to tempest client https://review.openstack.org/112535 | 14:09 |
| openstackgerrit | Joseph Lanoux proposed a change to openstack/tempest: Migrate test_minimun_basic to tempest client https://review.openstack.org/107562 | 14:09 |
| *** enikanorov has quit IRC | 14:10 | |
| *** jlanoux has joined #openstack-qa | 14:10 | |
| *** tnurlygayanov has quit IRC | 14:11 | |
| *** amotoki has joined #openstack-qa | 14:12 | |
| *** oomichi has joined #openstack-qa | 14:12 | |
| *** jhenner has quit IRC | 14:13 | |
| openstackgerrit | Flavio Percoco proposed a change to openstack-infra/devstack-gate: Add glance.store to the list of projects https://review.openstack.org/115299 | 14:14 |
| openstackgerrit | Flavio Percoco proposed a change to openstack-infra/devstack-gate: Add glance.store to the list of projects https://review.openstack.org/115299 | 14:14 |
| *** oomichi has quit IRC | 14:17 | |
| *** chandankumar has quit IRC | 14:17 | |
| openstackgerrit | Rohan Kanade proposed a change to openstack/tempest: Add subnet tests for extra attributes https://review.openstack.org/106326 | 14:17 |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: add security groups support https://review.openstack.org/104984 | 14:19 |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: add network resources https://review.openstack.org/100169 | 14:23 |
| *** jhenner has joined #openstack-qa | 14:24 | |
| *** marun_afk has quit IRC | 14:27 | |
| andreaf | mtreinish: ping - it seems that we have *a lot* of tests potentially leaking credentials | 14:29 |
| andreaf | mtreinish: everything a test or base class uses raise cls.skipException(msg) without safe_setup, creds are leaked if the skip takes place | 14:29 |
| mtreinish | oh, fun | 14:30 |
| andreaf | mtreinish: I have 129 hits of cls.skipException - I'm wondering whether this should be done in a better way entirely | 14:30 |
| andreaf | mtreinish: for now I guess I have to update the 129 cases of skip :( | 14:31 |
| andreaf | mtreinish: if would be good to make the setup fixture safe by default somehow | 14:32 |
| mtreinish | andreaf: well once https://blueprints.launchpad.net/tempest/+spec/post-run-cleanup lands we can use that to see what we leak during a gate run | 14:33 |
| mtreinish | but that doesn't help us now | 14:33 |
| *** sdmonov has joined #openstack-qa | 14:33 | |
| mtreinish | andreaf: yeah, if we put the decorator on the base class's setupUpClass I'm guessing it wouldn't work | 14:33 |
| andreaf | mtreinish: no that handles only exceptions in the base class fixture | 14:34 |
| andreaf | so every new test will always have to add safe_setup | 14:34 |
| andreaf | which is not so nice afterall | 14:34 |
| mtreinish | there has to be a way to do it by default | 14:34 |
| mtreinish | but that's pushing the limits of my python-fu | 14:35 |
| *** rbak has joined #openstack-qa | 14:35 | |
| andreaf | so the only way I could think of solving this is overwriting the test framework | 14:35 |
| andreaf | there'll be somewhere in testtools code or unittest code where it calls the fixtures | 14:35 |
| andreaf | so we should wrap the call to setupClass with try except and invoke tearDown on exception | 14:36 |
| mtreinish | btw, if you ever wondered why we have a hacking rule preventing setupClass in the unit tests.... :) | 14:36 |
| andreaf | eh eh | 14:36 |
| andreaf | maybe there is a way already I'll have a look through the code - or maybe lifeless knows a trick to fix this | 14:37 |
| *** ildikov_ has quit IRC | 14:38 | |
| *** mestery_afk is now known as mestery | 14:39 | |
| *** tkammer has quit IRC | 14:44 | |
| *** sdmonov has quit IRC | 14:44 | |
| *** kashyapc is now known as kashyap | 14:45 | |
| *** atiwari has joined #openstack-qa | 14:46 | |
| *** jecarey has joined #openstack-qa | 14:48 | |
| *** nmagnezi has quit IRC | 14:48 | |
| openstackgerrit | Felipe Reyes proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1358779 https://review.openstack.org/115306 | 14:48 |
| *** kashyap is now known as kashyapc | 14:48 | |
| *** kashyapc is now known as kashyap | 14:49 | |
| openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Allow installing tempest only with keystone https://review.openstack.org/115307 | 14:50 |
| *** markmcclain has quit IRC | 14:52 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add fingerprint for nova unit test bug 1357578 https://review.openstack.org/114680 | 14:53 |
| *** tnurlygayanov has joined #openstack-qa | 14:53 | |
| *** markmcclain has joined #openstack-qa | 14:53 | |
| *** rdekel1 has quit IRC | 14:54 | |
| *** sergeysh has joined #openstack-qa | 14:55 | |
| *** afazekas has quit IRC | 14:58 | |
| *** dkranz has quit IRC | 15:00 | |
| *** mestery is now known as mestery_afk | 15:01 | |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add functools.wraps on all function decorators https://review.openstack.org/113899 | 15:05 |
| *** marun has joined #openstack-qa | 15:08 | |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Change default size for strings to 1 https://review.openstack.org/115258 | 15:08 |
| *** marun has quit IRC | 15:09 | |
| *** marun has joined #openstack-qa | 15:09 | |
| *** marun has quit IRC | 15:09 | |
| *** marun has joined #openstack-qa | 15:10 | |
| *** marun has quit IRC | 15:10 | |
| *** markmcclain has quit IRC | 15:12 | |
| *** marun has joined #openstack-qa | 15:14 | |
| *** dkranz has joined #openstack-qa | 15:14 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: implement resource destruction https://review.openstack.org/104986 | 15:17 |
| *** lordd_ has joined #openstack-qa | 15:18 | |
| *** flaper87 is now known as flaper87|afk | 15:19 | |
| *** sergeysh has quit IRC | 15:19 | |
| *** marun has quit IRC | 15:22 | |
| *** wuhg has quit IRC | 15:22 | |
| *** jhenner has quit IRC | 15:23 | |
| *** jhenner1 has joined #openstack-qa | 15:23 | |
| *** cdent has joined #openstack-qa | 15:25 | |
| *** gszasz has quit IRC | 15:31 | |
| *** atiwari has quit IRC | 15:32 | |
| openstackgerrit | A change was merged to openstack/tempest: Disable hacking rule H305 https://review.openstack.org/114622 | 15:36 |
| *** KRK_ has joined #openstack-qa | 15:38 | |
| *** mestery_afk has quit IRC | 15:39 | |
| *** afazekas has joined #openstack-qa | 15:42 | |
| *** timello has joined #openstack-qa | 15:42 | |
| *** dimtruck is now known as zz_dimtruck | 15:44 | |
| afazekas | mtreinish: can you have a look on this change ? https://review.openstack.org/#/c/115195/ , it is related to 42 failure/week | 15:44 |
| *** timello has quit IRC | 15:49 | |
| *** mriedem has quit IRC | 15:55 | |
| *** sdague has quit IRC | 15:55 | |
| *** zz_dimtruck is now known as dimtruck | 15:57 | |
| *** chandankumar has joined #openstack-qa | 15:57 | |
| mtreinish | afazekas: ok reviewed | 15:59 |
| *** dims has quit IRC | 15:59 | |
| *** dims has joined #openstack-qa | 15:59 | |
| *** aimon has quit IRC | 16:00 | |
| *** SridharG has quit IRC | 16:00 | |
| openstackgerrit | Cedric Brandily proposed a change to openstack/tempest: Add provider extension positive test cases https://review.openstack.org/103498 | 16:02 |
| openstackgerrit | Cedric Brandily proposed a change to openstack/tempest: Add multi-provider extension positive test cases https://review.openstack.org/104526 | 16:02 |
| openstackgerrit | Cedric Brandily proposed a change to openstack/tempest: Add partial specs (multi-)provider extension positive test cases https://review.openstack.org/105234 | 16:02 |
| *** mriedem has joined #openstack-qa | 16:03 | |
| *** sdague has joined #openstack-qa | 16:03 | |
| *** dims has quit IRC | 16:04 | |
| *** ildikov has joined #openstack-qa | 16:05 | |
| *** arnaud has joined #openstack-qa | 16:05 | |
| *** dkranz has quit IRC | 16:08 | |
| *** atiwari has joined #openstack-qa | 16:09 | |
| dimtruck | Good morning everyone. I wanted to propose an open source tool to use in OpenStack qa/infra to collect and trend metric quality data called Sonar. It's a pretty popular open source tool and we added pylint and flake8 support to gather python quality metrics. It also collects code coverage from coverage.py and does some other cool things. You can check out the tool here: http://www.sonarqube.org/. I had a cou | 16:12 |
| dimtruck | ple questions: is there work currently to get something like this to be used for OpenStack and what i need to do to make a use case for this tool? | 16:12 |
| *** jlibosva has quit IRC | 16:14 | |
| *** afazekas_ has joined #openstack-qa | 16:14 | |
| *** yjiang5 has joined #openstack-qa | 16:16 | |
| andreaf | mtreinish: so I've been reading a bit about the setUpClass and tearDownClass - we won't be easily changing the unittest behaviour - see https://docs.python.org/2/library/unittest.html#class-and-module-fixtures | 16:21 |
| *** marun has joined #openstack-qa | 16:21 | |
| andreaf | mtreinish: so the option we have to prevent subclasses from overwriting the setUpClass | 16:21 |
| andreaf | mtreinish: and write the setUpClass in test.py base class so that it calls one or two other functions that can be overwritten by subclasses | 16:22 |
| andreaf | mtreinish: this way the safe_setup would have to be applied only once | 16:23 |
| andreaf | mtreinish: and we could enforce a structure e.g. check for skips first, then get credentials, then create resources | 16:23 |
| *** marun has quit IRC | 16:23 | |
| *** arnaud has quit IRC | 16:24 | |
| *** marun has joined #openstack-qa | 16:24 | |
| *** marun has quit IRC | 16:25 | |
| *** markus_z has quit IRC | 16:26 | |
| *** timello has joined #openstack-qa | 16:28 | |
| coolsvap | dkranz andreaf, mtreinish please have a look at https://review.openstack.org/#/c/90306/ , https://review.openstack.org/114783 , https://review.openstack.org/115238 | 16:29 |
| *** sdmonov has joined #openstack-qa | 16:31 | |
| *** qba73 has quit IRC | 16:31 | |
| *** marun has joined #openstack-qa | 16:32 | |
| *** aimon has joined #openstack-qa | 16:33 | |
| *** jlanoux has quit IRC | 16:33 | |
| *** markmcclain has joined #openstack-qa | 16:34 | |
| openstackgerrit | Felipe Reyes proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1358779 https://review.openstack.org/115306 | 16:35 |
| andreaf | coolsvap, +A on 114783 | 16:38 |
| *** jlibosva has joined #openstack-qa | 16:47 | |
| afazekas | mtreinish: So you would like to have an `if not <expression>; raise Timeout` instead of assertTrue or assertEqual? | 16:47 |
| *** markmcclain has quit IRC | 16:49 | |
| *** arnaud has joined #openstack-qa | 16:54 | |
| *** markmcclain has joined #openstack-qa | 16:55 | |
| *** jordanP has quit IRC | 16:55 | |
| *** amotoki has quit IRC | 16:56 | |
| *** amotoki has joined #openstack-qa | 16:56 | |
| *** harlowja_away is now known as harlowja | 16:56 | |
| *** cdent has quit IRC | 16:58 | |
| *** suneelb has joined #openstack-qa | 17:00 | |
| *** mriedem1 has joined #openstack-qa | 17:02 | |
| *** arnaud has quit IRC | 17:02 | |
| *** mriedem has quit IRC | 17:03 | |
| *** dkranz has joined #openstack-qa | 17:05 | |
| *** timello has quit IRC | 17:09 | |
| *** timello has joined #openstack-qa | 17:09 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 17:10 | |
| *** jpich has quit IRC | 17:15 | |
| mtreinish | afazekas: yeah I think that will make the code a bit easier to follow, especially if there is a failure | 17:18 |
| mtreinish | andreaf: hmm, I'm not sure how I feel about that. I guess you could poc it and then we can weight the approach after | 17:23 |
| andreaf | mtreinish: yes I'm writing a small PoC - I'm not 100% convinced either - let's see what people thinks about it | 17:24 |
| mtreinish | the other thing we've considered is converting to testresources instead of using setupclass | 17:25 |
| mtreinish | but that's a lot more work | 17:25 |
| *** markmcclain has quit IRC | 17:25 | |
| *** markmcclain has joined #openstack-qa | 17:25 | |
| *** SridharG has joined #openstack-qa | 17:31 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Fixed structure, safe, class setUp for all tests https://review.openstack.org/115353 | 17:31 |
| andreaf | mtreinish: ^^^ | 17:31 |
| andreaf | mtreinish: yes I remember that - I think the reason why the framework (setUpClass) does not work well for us is that we are not using it for what it was thought to be used | 17:32 |
| andreaf | or designed to do | 17:32 |
| *** adalbas has quit IRC | 17:33 | |
| mtreinish | andreaf: heh, yeah that's probably part of it. | 17:36 |
| *** chandankumar has quit IRC | 17:39 | |
| *** pcrews has quit IRC | 17:41 | |
| *** pcrews has joined #openstack-qa | 17:41 | |
| coolsvap | andreaf, any comments for https://review.openstack.org/90306 | 17:45 |
| *** timello has quit IRC | 17:45 | |
| *** timello has joined #openstack-qa | 17:46 | |
| *** adalbas has joined #openstack-qa | 17:51 | |
| *** dims has joined #openstack-qa | 17:53 | |
| *** atiwari has quit IRC | 17:55 | |
| mtreinish | dkranz: so I'm looking at https://review.openstack.org/#/c/86967/9/specs/run-without-admin.rst and I'm thinking it's too soon to +A it. | 18:00 |
| mtreinish | mostly because through the test-accounts stuff andreaf and I are having to change a lot about how cred allocation is being done | 18:01 |
| dkranz | mtreinish: right. That spec is really a joint-own between andreaf and myself for that reason | 18:01 |
| mtreinish | and then when a big chunk of what we want to do is dependent on hiearchical multi-tenancy I think holding off until kilo might be the best | 18:02 |
| dkranz | mtreinish: It is reasonable to wait until the dust settles on the cred stuff before proceeding | 18:02 |
| mtreinish | ok, how do you want to handle it in the review then? | 18:02 |
| dkranz | mtreinish: I mark it WIP | 18:02 |
| mtreinish | ok cool, thanks | 18:03 |
| dkranz | mtreinish: Having the multi-user way to run in parallel makes this less essential anyway | 18:03 |
| dkranz | mtreinish: more nice-to-have than must-have | 18:03 |
| dkranz | mtreinish: actually scratch that. But it is fine to postpone. | 18:04 |
| mtreinish | dkranz: yeah, and we're actually finding some bugs with the admin handling now because the test-accounts doesn't handle admin creds | 18:04 |
| mtreinish | dkranz: heh, ok :) | 18:04 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/110110 | 18:07 |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/devstack: Share the fixed network for Ironic https://review.openstack.org/110522 | 18:09 |
| *** timello_ has joined #openstack-qa | 18:10 | |
| *** timello has quit IRC | 18:11 | |
| *** aix has quit IRC | 18:15 | |
| mtreinish | jgrimm: I had a couple questions about the config generator update you were working on | 18:16 |
| *** cnesa7 has quit IRC | 18:16 | |
| jgrimm | mtreinish, fire away. btw, i gave up on the oslo-incubator pieces as someone else was already working on | 18:18 |
| *** weshay_mtg has quit IRC | 18:21 | |
| *** weshay has joined #openstack-qa | 18:21 | |
| *** kmartin has joined #openstack-qa | 18:21 | |
| *** mriedem has joined #openstack-qa | 18:23 | |
| *** mriedem1 has quit IRC | 18:24 | |
| mtreinish | jgrimm: heh, ok that covers the bigger question then | 18:25 |
| jgrimm | mtreinish, :) so yeah, i'll pick up again was all changes make it through oslo-incubator (and oslo.* apparently) | 18:26 |
| mtreinish | ok cool | 18:27 |
| mtreinish | because I noticed the changes to the incubator stuff and I was curious if it was a sync | 18:27 |
| mtreinish | and then the other one is I'm assuming it actually requires the alpha oslo release | 18:28 |
| *** chandankumar has joined #openstack-qa | 18:28 | |
| andreaf | coolsvap: I started looking at it but I won't be finished today I'm afraid - it's dinner time for me now | 18:28 |
| mtreinish | jgrimm: because that's something I've -2'd from the proposal bot: https://review.openstack.org/#/c/110110/ | 18:29 |
| mtreinish | but if we actually have a need for an alpha version then it'll be ok | 18:29 |
| coolsvap | andreaf, np I just wanted to check before i sleep :) | 18:30 |
| jgrimm | mtreinish, yeah, it would require the alpha version for that function | 18:30 |
| *** tosky has quit IRC | 18:30 | |
| andreaf | coolsvap btw do you have any good reference to the cinder qos api and use cases? | 18:31 |
| andreaf | coolsvap, I found bits and pieces | 18:31 |
| andreaf | mtreinish: let me know what you think about https://review.openstack.org/115353 | 18:31 |
| coolsvap | andreaf, for qos api i referred the api-ref only | 18:32 |
| andreaf | mtreinish, afazekas : and thanks for the reviews on the scenario migration bp - the chain of patches looks more reasonable now :) | 18:32 |
| andreaf | coolsvap ok I'll stick to that one then | 18:32 |
| *** suneelb has quit IRC | 18:33 | |
| *** suneelb has joined #openstack-qa | 18:33 | |
| *** suneelb has quit IRC | 18:33 | |
| *** suneelb has joined #openstack-qa | 18:34 | |
| *** catohornet1 has joined #openstack-qa | 18:34 | |
| mtreinish | andreaf: yeah, so at a quick glance that looks fine, but I'm wondering if we go through all the effort to do something like that, whether it's worth looking into using testresources or another library to handle it | 18:35 |
| mtreinish | because it's basically going to be another giant refactor that touches pretty much every test file | 18:35 |
| *** dims has quit IRC | 18:37 | |
| andreaf | mtreinish: fair enough - do we have some results / PoC already from looking into testresources? | 18:38 |
| andreaf | mtreinish: and the other thing is that I would not want to keep the test-account bp on hold until we do such a migration | 18:39 |
| mtreinish | andreaf: lifeless gave sdague a poc on how to do the conversion easily at HK, but I'm pretty sure it's been lost to the ether | 18:39 |
| mtreinish | andreaf: yeah that's fair, but the alternative is adding safe_setup everywhere right? | 18:40 |
| andreaf | mtreinish: uhm yes but it's straight forward to review | 18:40 |
| andreaf | mtreinish: so I may even manage to script it | 18:40 |
| mtreinish | sure that's true, I thought igawa actually had a patch to do that at some point | 18:41 |
| mtreinish | andreaf: and if every setupclass requires it it probably makes sense for a hacking check too | 18:41 |
| andreaf | mtreinish: it's not something nice to maintain though - we may need an hacking rule | 18:41 |
| andreaf | eh eh | 18:41 |
| andreaf | mtreinish: ok tomorrow I'll work on some smart sed script to add it to all classes - the tricky bit is that some classes do import tempest.test and others to from tempest import test | 18:43 |
| coolsvap | mtreinish, I am not able to understand how expected_status is not part of BaremetalClientJSON since it inherits from BaremetalClientV1 which inherits from BaremetalClient which inherits from RestClient | 18:43 |
| andreaf | mtreinish: well I'll sort it out | 18:43 |
| andreaf | mtreinish: this one is probably good for approval as well: https://review.openstack.org/#/c/114609/ | 18:44 |
| mtreinish | coolsvap: yeah I was surprised by that too, but the runs clearly show an issue | 18:44 |
| mtreinish | coolsvap: it might be something else, that comment was just my guess based on the stack trace | 18:46 |
| mtreinish | andreaf: doesn't that override the local getattr for that class to just use the one from the oslo cfg object instead? | 18:47 |
| mtreinish | so it'll never getattr from the proxy object | 18:47 |
| lifeless | andreaf: mtreinish: fixing setUpClass? its implemented by the python upstream TestSuite, and eeeesssh. | 18:48 |
| coolsvap | mtreinish, yup I will dig into it | 18:48 |
| *** Sukhdev has joined #openstack-qa | 18:48 | |
| openstackgerrit | Ashish Kumar Gupta proposed a change to openstack/tempest: Port API Tests Enhancements https://review.openstack.org/107277 | 18:48 |
| andreaf | mtreinish: but the proxy already overrides __getattr__ towards the internal config object | 18:49 |
| andreaf | mtreinish: so it should be fine | 18:49 |
| mtreinish | lifeless: yeah, at this point I think it's more about how we migrate away from using it :) | 18:49 |
| mtreinish | too many annoying issues to work through... | 18:50 |
| mtreinish | andreaf: hmm, ok I'll take a closer look | 18:50 |
| *** SridharG has quit IRC | 18:51 | |
| andreaf | lifeless: setUpClass still behaves the same in python 3 https://docs.python.org/3/library/unittest.html#class-and-module-fixtures - I think I didn't get what you mean | 18:52 |
| openstackgerrit | Jeremy Stanley proposed a change to openstack-dev/devstack: TESTING: Add secondary mirror for ubuntu https://review.openstack.org/107060 | 18:55 |
| *** sean-toner has joined #openstack-qa | 18:56 | |
| *** cpallares has quit IRC | 18:57 | |
| *** atiwari has joined #openstack-qa | 18:58 | |
| *** dims has joined #openstack-qa | 19:00 | |
| *** dims has quit IRC | 19:00 | |
| *** suneelb has quit IRC | 19:00 | |
| *** suneelb has joined #openstack-qa | 19:00 | |
| *** suneelb has quit IRC | 19:01 | |
| *** dims has joined #openstack-qa | 19:01 | |
| *** suneelb has joined #openstack-qa | 19:01 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: test_hotplug_nic wait for the guest https://review.openstack.org/115195 | 19:04 |
| *** dims has quit IRC | 19:05 | |
| *** Longgeek has quit IRC | 19:06 | |
| afazekas | mtreinish: switched to exceptions https://review.openstack.org/115195 | 19:06 |
| mtreinish | afazekas: ok cool, thanks | 19:06 |
| openstackgerrit | A change was merged to openstack/tempest: Update volume client check success https://review.openstack.org/114783 | 19:10 |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Remove firewalld on Fedora 20 by default https://review.openstack.org/113856 | 19:11 |
| *** harlowja has quit IRC | 19:13 | |
| *** harlowja has joined #openstack-qa | 19:13 | |
| *** markmcclain has quit IRC | 19:17 | |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Add some known contacts https://review.openstack.org/114318 | 19:18 |
| *** jlibosva has quit IRC | 19:20 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Pull flavor_ref from config for javelin servers https://review.openstack.org/114372 | 19:22 |
| *** ildikov has quit IRC | 19:24 | |
| openstackgerrit | Malini Kamalambal proposed a change to openstack-infra/devstack-gate: Update Marconi to Zaqar https://review.openstack.org/115384 | 19:26 |
| afazekas | jogo: https://review.openstack.org/#/c/114742/ | 19:32 |
| *** malini has quit IRC | 19:32 | |
| *** rockyg has joined #openstack-qa | 19:36 | |
| *** ildikov_ has joined #openstack-qa | 19:39 | |
| *** nmagnezi has joined #openstack-qa | 19:45 | |
| *** adalbas has quit IRC | 19:49 | |
| *** marun has quit IRC | 19:49 | |
| *** markmcclain has joined #openstack-qa | 19:50 | |
| *** chandankumar has quit IRC | 19:56 | |
| *** stevebaker has quit IRC | 20:00 | |
| *** stevebaker has joined #openstack-qa | 20:00 | |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: (DO NOT MERGE) test run using mysqlconnector https://review.openstack.org/114207 | 20:02 |
| *** nmagnezi has quit IRC | 20:08 | |
| *** mkollaro has quit IRC | 20:09 | |
| openstackgerrit | Felipe Reyes proposed a change to openstack-dev/devstack: Add pxebin location for SUSE https://review.openstack.org/114268 | 20:09 |
| *** andreaf_ has joined #openstack-qa | 20:11 | |
| *** andreaf_ has quit IRC | 20:12 | |
| *** andreaf_ has joined #openstack-qa | 20:12 | |
| *** andreaf_ has quit IRC | 20:12 | |
| *** andreaf_ has joined #openstack-qa | 20:13 | |
| *** andreaf has quit IRC | 20:14 | |
| *** atiwari has quit IRC | 20:28 | |
| *** lordd_ has quit IRC | 20:29 | |
| *** nmagnezi has joined #openstack-qa | 20:31 | |
| *** dims has joined #openstack-qa | 20:37 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Guard for worlddump in case LOGDIR is undefined https://review.openstack.org/115066 | 20:39 |
| *** stevebaker has quit IRC | 20:41 | |
| *** stevebaker has joined #openstack-qa | 20:41 | |
| *** dims has quit IRC | 20:45 | |
| afazekas | dkranz, andreaf: I need to leave now, can have look on this https://review.openstack.org/115195 after the gate finished. | 20:47 |
| *** Sukhdev has quit IRC | 20:48 | |
| *** mestery has joined #openstack-qa | 20:49 | |
| *** mestery has quit IRC | 20:49 | |
| *** mestery has joined #openstack-qa | 20:50 | |
| *** atiwari has joined #openstack-qa | 20:50 | |
| *** jgrimm has quit IRC | 20:53 | |
| *** andreaf_ has quit IRC | 20:54 | |
| *** andreaf_ has joined #openstack-qa | 20:54 | |
| *** andreaf_ has quit IRC | 20:54 | |
| *** andreaf_ has joined #openstack-qa | 20:55 | |
| *** andreaf_ has quit IRC | 20:55 | |
| *** nmagnezi has quit IRC | 20:55 | |
| *** andreaf_ has joined #openstack-qa | 20:55 | |
| *** toure has quit IRC | 21:00 | |
| *** weshay has quit IRC | 21:01 | |
| *** rfolco has quit IRC | 21:06 | |
| *** cbader has joined #openstack-qa | 21:09 | |
| *** cbader has quit IRC | 21:10 | |
| *** cbader has joined #openstack-qa | 21:10 | |
| openstackgerrit | Felipe Reyes proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1358779 https://review.openstack.org/115306 | 21:11 |
| *** Sukhdev has joined #openstack-qa | 21:11 | |
| *** sean-toner has quit IRC | 21:11 | |
| *** aysyd has quit IRC | 21:12 | |
| *** dustins has quit IRC | 21:25 | |
| *** dmellado has joined #openstack-qa | 21:26 | |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Ironic and baremetal install diskimage-builder with pip https://review.openstack.org/92257 | 21:27 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: A service to install diskimage-builder https://review.openstack.org/92256 | 21:27 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Install heat-cfntools and heat-templates repos https://review.openstack.org/92258 | 21:27 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: WIP enable building test image https://review.openstack.org/95617 | 21:27 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Utility functions for building test images https://review.openstack.org/95616 | 21:27 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Build an image for heat functional tests https://review.openstack.org/110819 | 21:27 |
| *** dkranz has quit IRC | 21:34 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Add ability to switch database driver https://review.openstack.org/105209 | 21:34 |
| *** mestery_ has joined #openstack-qa | 21:36 | |
| *** mestery has quit IRC | 21:37 | |
| *** malini has joined #openstack-qa | 21:38 | |
| *** mestery_ is now known as mestery | 21:40 | |
| *** tkammer has joined #openstack-qa | 21:41 | |
| *** mestery has quit IRC | 21:45 | |
| *** mestery has joined #openstack-qa | 21:46 | |
| yjiang5 | hi, all, I want to add a test case to make sure the resource allocation is correct in compute node (i.e. the resource is really released when drop_resize_claim() ), https://review.openstack.org/#/c/115176/ . I want to make sure when the case are executed, no other server to be scheduled to that compute node. Are there any method to achieve it considering currently the tempest is in parrellel by default? | 21:47 |
| openstackgerrit | A change was merged to openstack/tempest: Add new snapshot compute feature flag https://review.openstack.org/101381 | 21:53 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Fix network/secgroup scenario tests for ironic https://review.openstack.org/112660 | 21:53 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Fix load balancer scenario if no tenant network https://review.openstack.org/113707 | 21:55 |
| *** dimtruck is now known as zz_dimtruck | 21:57 | |
| *** mestery has quit IRC | 22:01 | |
| *** zz_dimtruck is now known as dimtruck | 22:02 | |
| *** KRK_ has quit IRC | 22:02 | |
| mtreinish | yjiang5: the only way I can think to ensure that would be to lock against all the server creates | 22:04 |
| mtreinish | but that would only work for the single node case (which is the gate) | 22:04 |
| mtreinish | I don't think there is a good way to isolate that | 22:04 |
| mtreinish | short of having a multinode env and removing the node from scheduling when it's empty then only use it for the test | 22:05 |
| *** malini has quit IRC | 22:06 | |
| *** malini has joined #openstack-qa | 22:06 | |
| *** rockyg has quit IRC | 22:06 | |
| mtreinish | but that's probably far too synthetic and concerned with the nova internals to be valid in tempest | 22:06 |
| *** malini1 has joined #openstack-qa | 22:08 | |
| *** malini has quit IRC | 22:08 | |
| yjiang5 | mtreinish: thanks for reply. Are there any method now to lock against all server creates? My patch target for tempest, so I'm ok if assuming gate is always single-node is acceptable. | 22:12 |
| mtreinish | yjiang5: no, and that's really a bad idea. It was just the only hypothetical way I could think of doing it | 22:13 |
| *** tkammer has quit IRC | 22:14 | |
| yjiang5 | mtreinish: :) | 22:14 |
| *** malini1 has quit IRC | 22:16 | |
| *** malini has joined #openstack-qa | 22:16 | |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: WIP enable building test image https://review.openstack.org/95617 | 22:17 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Utility functions for building test images https://review.openstack.org/95616 | 22:17 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Build an image for heat functional tests https://review.openstack.org/110819 | 22:17 |
| *** HenryG_ has joined #openstack-qa | 22:17 | |
| *** HenryG has quit IRC | 22:19 | |
| *** malini has quit IRC | 22:19 | |
| *** malini has joined #openstack-qa | 22:19 | |
| *** malini has quit IRC | 22:20 | |
| *** malini has joined #openstack-qa | 22:21 | |
| *** malini has quit IRC | 22:21 | |
| *** malini has joined #openstack-qa | 22:21 | |
| *** markmcclain has quit IRC | 22:26 | |
| *** kmartin has quit IRC | 22:27 | |
| *** jecarey has quit IRC | 22:29 | |
| *** kmartin has joined #openstack-qa | 22:38 | |
| *** harlowja has quit IRC | 22:39 | |
| *** harlowja has joined #openstack-qa | 22:39 | |
| ianw | stevebaker: what's the deal with the pip mirror in https://review.openstack.org/#/c/95616/ | 22:39 |
| stevebaker | ianw: its to allow diskimage-builder to install locally built pip archives into the image, which will eventually allow gating on agent projects like heat-cfntools and os-collect-config. This was initially implemented as a file based mirror but bugs in pip prevented that from working, hence the switch to an http based mirror | 22:42 |
| *** rbak has quit IRC | 22:43 | |
| stevebaker | ianw: its only configured if images are being built, so even though it configures the shared apache the risk and impact should be low | 22:43 |
| ianw | stevebaker: ok, i'm not super concerned, just trying to understand how it works. it's not really used as part of https://review.openstack.org/#/c/95616/ then? i know d-i-b, but not really familiar with the pip bits of it | 22:45 |
| ianw | (it does seem it could be split out, and more context in the commit message might help) | 22:46 |
| stevebaker | ianw: it is part of https://review.openstack.org/#/c/95616/ since the mirror is only useful when you're building an image. the tripleo folk are using pip more to manage versions of things installed on the image | 22:48 |
| *** mriedem has quit IRC | 22:49 | |
| stevebaker | ianw: disk_image_create_upload pypi_mirror_url refers to the mirror built by build_dib_pip_repo | 22:49 |
| *** sdmonov has quit IRC | 22:52 | |
| *** mestery has joined #openstack-qa | 22:53 | |
| *** cbader has quit IRC | 22:54 | |
| *** arnaud has joined #openstack-qa | 22:59 | |
| *** dimtruck is now known as zz_dimtruck | 23:00 | |
| *** mestery_ has joined #openstack-qa | 23:02 | |
| *** mestery has quit IRC | 23:02 | |
| *** mestery_ has quit IRC | 23:07 | |
| *** mestery has joined #openstack-qa | 23:07 | |
| *** aimon has quit IRC | 23:08 | |
| *** Sukhdev has quit IRC | 23:11 | |
| ianw | stevebaker: oh, yeah ok the PYPI_MIRROR_URL bit | 23:12 |
| *** rbak has joined #openstack-qa | 23:12 | |
| *** mestery has quit IRC | 23:12 | |
| stevebaker | ianw: yep | 23:12 |
| *** mestery has joined #openstack-qa | 23:13 | |
| *** rbak has quit IRC | 23:13 | |
| ianw | stevebaker: but nothing is calling build_dib_pip_repo right now? | 23:14 |
| ianw | oh, that's the next change | 23:14 |
| stevebaker | ianw: yep, and in the change after that it force-enables image building so we're actually running tempest tests against a custom built image | 23:15 |
| stevebaker | ianw: just for demonstration purposes | 23:15 |
| stevebaker | ianw: (actually it builds the image but no tests will run against it until the new heat functional tests job is fully working) | 23:16 |
| ianw | stevebaker: ok, cool, getting a better idea :) | 23:16 |
| ianw | stevebaker: do you have an opinion on where the divide between what happens in d-i-b and what happens on a system with scripts/puppet lies | 23:17 |
| ianw | stevebaker: i'm thinking about our current issues with firewalld on f20 | 23:17 |
| ianw | stevebaker: should that be disabled by puppet, or uninstalled by d-i-b, or turned off in a script, etc | 23:17 |
| ianw | (the script being devstack) | 23:18 |
| ianw | we seem spoilt for choice :) | 23:18 |
| stevebaker | ianw: general rule of thumb is the image gets all packages, software, templates for config files, pre/post configuration scripts etc. So the only config that heat needs to inject on boot is the specific details regarding the other newly created resources in the cloud stack | 23:19 |
| stevebaker | ianw: I would think d-i-b would uninstall or disable firewalld in that scenario | 23:20 |
| ianw | stevebaker: ok cool. how does puppet fit in that scenario? | 23:21 |
| stevebaker | ianw: you could use puppet during image building to install things, and you could also install puppet modules for running later by something like heat | 23:28 |
| stevebaker | ianw: but probably more like the latter | 23:29 |
| *** atiwari has quit IRC | 23:31 | |
| *** KurtMartin has joined #openstack-qa | 23:33 | |
| stevebaker | ianw: this is the start of the series though https://review.openstack.org/#/c/92257/ | 23:34 |
| *** kmartin has quit IRC | 23:36 | |
| *** KurtMartin has quit IRC | 23:39 | |
| *** mestery has quit IRC | 23:41 | |
| *** HenryG_ is now known as HenryG | 23:42 | |
| *** mestery has joined #openstack-qa | 23:44 | |
| *** mestery has quit IRC | 23:45 | |
| *** kmartin has joined #openstack-qa | 23:47 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Allow cinder types to be created without be file https://review.openstack.org/114693 | 23:48 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Remove code tagged as "can be removed after Icehouse" https://review.openstack.org/111703 | 23:48 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Set display name in apache processes https://review.openstack.org/114280 | 23:48 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Avoid using deprecated Q_AGENT_EXTRA_AGENT_OPTS in ML2 setup https://review.openstack.org/113086 | 23:48 |
| openstackgerrit | A change was merged to openstack-dev/devstack: enable multiple collector workers https://review.openstack.org/109058 | 23:50 |
| *** yamahata has joined #openstack-qa | 23:53 | |
| *** marun has joined #openstack-qa | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!