| *** matsuhashi has joined #openstack-qa | 00:01 | |
| *** Guest41065 is now known as StevenK | 00:06 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Fix typos in HACKING.rst https://review.openstack.org/87114 | 00:25 |
|---|---|---|
| *** adalbas has quit IRC | 00:26 | |
| *** adalbas has joined #openstack-qa | 00:42 | |
| *** mestery has joined #openstack-qa | 00:48 | |
| *** mriedem1 has quit IRC | 01:01 | |
| *** mriedem has joined #openstack-qa | 01:05 | |
| *** nati_ueno has joined #openstack-qa | 01:12 | |
| *** ianw has quit IRC | 01:16 | |
| *** ianw has joined #openstack-qa | 01:16 | |
| *** mkollaro has quit IRC | 01:34 | |
| *** nosnos has joined #openstack-qa | 01:41 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Avoid pm-is-supported spam in logs https://review.openstack.org/84276 | 01:42 |
| *** mriedem has quit IRC | 02:04 | |
| *** nati_ueno has quit IRC | 02:21 | |
| *** GMann has joined #openstack-qa | 02:23 | |
| *** mlavalle has quit IRC | 02:36 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify create/get flavor attributes of Nova APIs https://review.openstack.org/80543 | 02:37 |
| *** amotoki has quit IRC | 02:38 | |
| *** adalbas has quit IRC | 02:51 | |
| *** Abhi has joined #openstack-qa | 03:06 | |
| *** GMann has quit IRC | 03:07 | |
| *** nati_ueno has joined #openstack-qa | 03:23 | |
| *** GMann has joined #openstack-qa | 03:24 | |
| *** matsuhashi has quit IRC | 03:31 | |
| *** nati_ueno has quit IRC | 03:35 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/75798 failed gate-heat-python26, gate-heat-python27, gate-grenade-dsvm-neutron in the gate queue with an unrecognized error | 03:47 |
| *** nosnos has quit IRC | 03:49 | |
| *** nati_ueno has joined #openstack-qa | 03:50 | |
| *** nati_ueno has quit IRC | 04:05 | |
| *** matsuhashi has joined #openstack-qa | 04:38 | |
| *** nosnos has joined #openstack-qa | 04:48 | |
| *** chandan_kumar has joined #openstack-qa | 04:52 | |
| openstackgerrit | Masayuki Igawa proposed a change to openstack-dev/devstack: Add fping utility path to nova.conf https://review.openstack.org/86806 | 05:26 |
| *** om has joined #openstack-qa | 05:35 | |
| *** nosnos_ has joined #openstack-qa | 05:36 | |
| *** nosnos has quit IRC | 05:36 | |
| *** nati_ueno has joined #openstack-qa | 05:42 | |
| *** zul has quit IRC | 05:46 | |
| *** nati_ueno has quit IRC | 05:56 | |
| *** zul has joined #openstack-qa | 06:00 | |
| *** matsuhas_ has joined #openstack-qa | 06:00 | |
| *** matsuhashi has quit IRC | 06:03 | |
| *** Abhi has quit IRC | 06:06 | |
| *** akamyshnikova has quit IRC | 06:11 | |
| *** afazekas has joined #openstack-qa | 06:12 | |
| *** psharma has joined #openstack-qa | 06:21 | |
| *** amotoki has joined #openstack-qa | 06:24 | |
| *** Abhi has joined #openstack-qa | 06:26 | |
| *** akamyshnikova has joined #openstack-qa | 06:26 | |
| *** nosnos has joined #openstack-qa | 06:32 | |
| *** nosnos_ has quit IRC | 06:32 | |
| *** matsuhas_ has quit IRC | 06:37 | |
| *** matsuhashi has joined #openstack-qa | 06:37 | |
| *** matsuhashi has quit IRC | 06:37 | |
| *** matsuhashi has joined #openstack-qa | 06:38 | |
| *** _sweston_ has joined #openstack-qa | 06:45 | |
| *** _sweston_ has quit IRC | 06:49 | |
| *** sweston has joined #openstack-qa | 06:49 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: fix import for default clients in clients.py https://review.openstack.org/87173 | 06:52 |
| openstackgerrit | xu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'create-aggregate' API https://review.openstack.org/83692 | 07:03 |
| *** andreaf has joined #openstack-qa | 07:08 | |
| *** vponomaryov has joined #openstack-qa | 07:13 | |
| *** jamielennox is now known as jamielennox|away | 07:18 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Implementing XML client for VPNaaS https://review.openstack.org/85255 | 07:21 |
| openstackgerrit | Christian Schwede proposed a change to openstack/tempest: Swift formpost cleanup https://review.openstack.org/86244 | 07:23 |
| *** jordanP has joined #openstack-qa | 07:39 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: fix import for default clients in clients.py https://review.openstack.org/87173 | 07:42 |
| *** qba73 has joined #openstack-qa | 07:47 | |
| openstackrecheck | Console logs not available after 13:31s for gate-ceilometer-pep8 77612,34,70eb9ed | 07:56 |
| *** morganfainberg is now known as morganfainberg_Z | 08:00 | |
| *** yassine has joined #openstack-qa | 08:04 | |
| *** giulivo has joined #openstack-qa | 08:08 | |
| *** jpich has joined #openstack-qa | 08:10 | |
| openstackrecheck | Console logs not available after 14:11s for gate-ceilometer-pep8 77845,30,4bd6a56 | 08:11 |
| *** jhenner has joined #openstack-qa | 08:20 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Adds test for volume attachments Nova V2 APIs https://review.openstack.org/87191 | 08:28 |
| *** jlibosva has joined #openstack-qa | 08:58 | |
| openstackrecheck | Console logs not available after 13:36s for gate-heat-pep8 86526,6,bd7b54a | 09:05 |
| *** fbo_away is now known as fbo | 09:18 | |
| *** mkollaro has joined #openstack-qa | 09:25 | |
| *** GMann has quit IRC | 09:30 | |
| *** StevenK has quit IRC | 09:44 | |
| openstackgerrit | Ala Rezmerita proposed a change to openstack/tempest: Cleanup neutron default security groups on tenant deletion https://review.openstack.org/79275 | 09:47 |
| *** yassine has quit IRC | 09:48 | |
| *** tosky has joined #openstack-qa | 10:00 | |
| akamyshnikova | afazekas, mkoderer, sdague Hello! I've got 3 changes with API test for lbaas and floating ips. They have already had +2s. Could you please find some time to have a look at them one more time? LBaaS - https://review.openstack.org/66541 Floating ips - https://review.openstack.org/63723 https://review.openstack.org/71251 | 10:05 |
| *** fbo is now known as fbo_away | 10:10 | |
| *** sahumada has quit IRC | 10:12 | |
| *** sahumada has joined #openstack-qa | 10:12 | |
| *** ianw has quit IRC | 10:17 | |
| *** mkollaro has quit IRC | 10:25 | |
| *** mkollaro has joined #openstack-qa | 10:32 | |
| openstackgerrit | Steven Hardy proposed a change to openstack-dev/grenade: Add initial support for Heat https://review.openstack.org/86978 | 10:40 |
| *** matsuhashi has quit IRC | 10:40 | |
| *** matsuhashi has joined #openstack-qa | 10:41 | |
| *** matsuhashi has quit IRC | 10:50 | |
| *** matsuhas_ has joined #openstack-qa | 10:52 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: normalize the services for a post grizzly world https://review.openstack.org/84287 | 10:54 |
| *** chandan_kumar has quit IRC | 10:55 | |
| *** dkorn has joined #openstack-qa | 11:00 | |
| *** matsuhas_ has quit IRC | 11:00 | |
| *** yassine has joined #openstack-qa | 11:01 | |
| *** matsuhas_ has joined #openstack-qa | 11:04 | |
| *** Abhi has quit IRC | 11:07 | |
| *** chandan_kumar has joined #openstack-qa | 11:08 | |
| *** matsuhas_ has quit IRC | 11:15 | |
| *** chandan_kumar has quit IRC | 11:16 | |
| *** matsuhashi has joined #openstack-qa | 11:16 | |
| *** fbo_away is now known as fbo | 11:26 | |
| *** amotoki has quit IRC | 11:28 | |
| *** weshay has joined #openstack-qa | 11:29 | |
| *** chandan_kumar has joined #openstack-qa | 11:29 | |
| *** weshay has quit IRC | 11:32 | |
| *** sthaha has joined #openstack-qa | 11:36 | |
| *** sthaha has quit IRC | 11:36 | |
| *** sthaha has joined #openstack-qa | 11:36 | |
| *** sthaha has quit IRC | 11:36 | |
| *** sthaha has joined #openstack-qa | 11:37 | |
| *** sthaha has quit IRC | 11:37 | |
| *** sthaha has joined #openstack-qa | 11:37 | |
| *** matsuhashi has quit IRC | 11:46 | |
| *** StevenK has joined #openstack-qa | 11:48 | |
| *** om has quit IRC | 11:50 | |
| *** om has joined #openstack-qa | 11:51 | |
| *** matsuhashi has joined #openstack-qa | 11:51 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/87053 failed gate-grenade-dsvm-neutron, gate-horizon-python27-django14 in the gate queue with an unrecognized error | 11:58 |
| *** nosnos has quit IRC | 12:01 | |
| openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds more testcases to test_telemetry_alarming_api https://review.openstack.org/82491 | 12:02 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Prevent the use of 'swift' in ENABLED_SERVICES https://review.openstack.org/87140 | 12:04 |
| *** enikanorov has quit IRC | 12:04 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Delete unused neutron port https://review.openstack.org/85013 | 12:04 |
| *** enikanorov has joined #openstack-qa | 12:04 | |
| shardy | Hi all, can anyone tell me what OS check-grenade-dsvm uses, or more specifically, what OS should I use for local VM testing of grenade? | 12:14 |
| shardy | I'm working on a new grenade test for heat, and a lot of stuff seems broken using a F20 image | 12:14 |
| *** dkranz has joined #openstack-qa | 12:15 | |
| *** rfolco has joined #openstack-qa | 12:16 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Remove keystone_ec2_uri flag from lib/heat https://review.openstack.org/83777 | 12:18 |
| *** aysyd has joined #openstack-qa | 12:20 | |
| maurosr | shardy: try it with ubuntu 12.04 | 12:25 |
| *** bada has quit IRC | 12:25 | |
| shardy | maurosr: Ok, will do, thanks | 12:26 |
| maurosr | yw | 12:26 |
| *** matsuhashi has quit IRC | 12:29 | |
| *** matsuhashi has joined #openstack-qa | 12:30 | |
| *** om has quit IRC | 12:31 | |
| *** weshay has joined #openstack-qa | 12:33 | |
| *** BobBall_ has quit IRC | 12:33 | |
| *** adalbas has joined #openstack-qa | 12:33 | |
| *** matsuhashi has quit IRC | 12:34 | |
| *** matsuhashi has joined #openstack-qa | 12:42 | |
| *** om has joined #openstack-qa | 12:45 | |
| *** zul has quit IRC | 12:48 | |
| sdague | shardy: it's running on 12.04 in the gate, fedora patches would be welcomed, though my guess is most of those would be patches against devstack | 12:48 |
| shardy | sdague: Ok, I'd like to debug the heat test issues first, then I'll try to work out the Fedora problems | 12:49 |
| sdague | cool | 12:49 |
| *** gszasz has joined #openstack-qa | 12:49 | |
| *** zul has joined #openstack-qa | 12:51 | |
| openstackgerrit | Yves-Gwenael Bourhis proposed a change to openstack-dev/devstack: Fixed pip with http(s)_proxy https://review.openstack.org/87261 | 12:52 |
| *** qba73_ has joined #openstack-qa | 12:55 | |
| *** qba73 has quit IRC | 12:57 | |
| *** devvesa has joined #openstack-qa | 13:00 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Use nova bindir of /usr/bin on SUSE https://review.openstack.org/86933 | 13:04 |
| *** matsuhashi has quit IRC | 13:06 | |
| *** matsuhas_ has joined #openstack-qa | 13:10 | |
| openstackrecheck | Console logs not available after 14:35s for check-tempest-dsvm-postgres-full 86966,4,392b95f | 13:17 |
| *** mriedem has joined #openstack-qa | 13:18 | |
| *** HenryG has quit IRC | 13:21 | |
| *** gszasz has quit IRC | 13:23 | |
| *** om has quit IRC | 13:28 | |
| *** om has joined #openstack-qa | 13:29 | |
| *** jgrimm has quit IRC | 13:30 | |
| *** kgriffs|afk is now known as kgriffs | 13:30 | |
| openstackrecheck | Console logs not available after 13:40s for check-tempest-dsvm-full 83371,6,f29cbaa | 13:35 |
| *** yfried__ has quit IRC | 13:39 | |
| *** mwagner_lap has quit IRC | 13:39 | |
| SergeyLukjanov | sdague, morning | 13:40 |
| SergeyLukjanov | sdague, I have a question re adding sahara client to tempest | 13:40 |
| SergeyLukjanov | https://review.openstack.org/#/c/83317/6/tempest/clients.py | 13:40 |
| sdague | SergeyLukjanov: sure | 13:41 |
| SergeyLukjanov | sdague, thx! | 13:41 |
| SergeyLukjanov | sdague, currently, our client searches for the endpoint in constructor and it means that it'll fail if no sahara available | 13:41 |
| SergeyLukjanov | sdague, we're initializing all stuff in client constructor | 13:42 |
| *** matsuhas_ has quit IRC | 13:42 | |
| SergeyLukjanov | sdague, is it incorrect? | 13:43 |
| sdague | no, that seems fine | 13:43 |
| *** salv-orlando has joined #openstack-qa | 13:43 | |
| SergeyLukjanov | sdague, in this case it requires check that sahara is available before creating client itself | 13:43 |
| sdague | sure | 13:44 |
| SergeyLukjanov | sdague, but it sounds fine for me that client could fail if service isn't available | 13:44 |
| sdague | yep, that seems fine. It will throw an exception right? And we'll just fail from there | 13:45 |
| SergeyLukjanov | sdague, yup | 13:46 |
| SergeyLukjanov | sdague, do you remember that I asked for your help several weeks ago re this patch? | 13:46 |
| SergeyLukjanov | sdague, it was the issue that error from sahara client was magically wrapped with ImportError and failed like error in running testr | 13:47 |
| openstackrecheck | Console logs not available after 13:27s for check-tempest-dsvm-postgres-full 83520,11,073ae86 | 13:49 |
| sdague | SergeyLukjanov: ah... | 13:50 |
| sdague | so can we wrap the import and throw a skip exception explictly? | 13:51 |
| *** matsuhashi has joined #openstack-qa | 13:52 | |
| *** yfried__ has joined #openstack-qa | 13:54 | |
| *** matsuhashi has quit IRC | 13:54 | |
| *** Aimon_ is now known as Aimon | 13:54 | |
| *** psharma has quit IRC | 13:56 | |
| *** enikanorov_ has joined #openstack-qa | 13:57 | |
| *** mestery_ has joined #openstack-qa | 13:58 | |
| SergeyLukjanov | sdague, it's an ImportError itself | 13:58 |
| *** mestery has quit IRC | 13:58 | |
| SergeyLukjanov | sdague, "service not found" exception from sahara client was wrapped somehow | 13:59 |
| *** enikanorov has quit IRC | 13:59 | |
| *** mlavalle has joined #openstack-qa | 13:59 | |
| *** mlavalle has quit IRC | 14:00 | |
| SergeyLukjanov | sdague, personally, I prefer to rely on service_available.sahara because if endpoint isn't available than something goes wrong | 14:00 |
| *** mlavalle has joined #openstack-qa | 14:01 | |
| openstackrecheck | Console logs not available after 13:47s for check-tempest-dsvm-postgres-full 83370,6,3af9578 | 14:03 |
| *** malini_afk is now known as malini | 14:03 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/87013 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1295876, gate-grenade-dsvm-neutron: unrecognized error | 14:03 |
| *** jgrimm has joined #openstack-qa | 14:06 | |
| *** rwsu has joined #openstack-qa | 14:09 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: added query for neutron migrations issue https://review.openstack.org/87274 | 14:12 |
| sdague | mriedem: got a quick sec for that? | 14:12 |
| mriedem | sdague: yeah if my system doesn't lock up again this morning for the 5th time | 14:14 |
| openstackgerrit | Sergey Lukjanov proposed a change to openstack/tempest: Add Sahara client for scenarios tests https://review.openstack.org/83317 | 14:14 |
| mriedem | and if these corporate IT security guys get off my back about updating openssl | 14:14 |
| SergeyLukjanov | sdague, could you please restore your +2? (it's just a rebase) | 14:14 |
| sdague | mriedem: are you using windows? | 14:14 |
| mriedem | sdague: yeah | 14:14 |
| mriedem | sdague: firefox decided to shit the bed | 14:15 |
| *** kgriffs is now known as kgriffs|afk | 14:15 | |
| openstackrecheck | Console logs not available after 13:23s for check-tempest-dsvm-full 84454,4,35a06cd | 14:16 |
| mriedem | sdague: looks good, only comment would be you could filter on the build_name also | 14:17 |
| *** om has quit IRC | 14:17 | |
| mriedem | AND (build_name:"check_grenade_dvsm_neutron" OR build_name:"gate_grenade_dvsm_neutron") | 14:18 |
| *** beekneemech is now known as bnemec | 14:18 | |
| mriedem | oops, s/_/-/ | 14:19 |
| sdague | sure | 14:20 |
| mriedem | s/dvsm/dsvm/ | 14:20 |
| mriedem | damn | 14:20 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: added query for neutron migrations issue https://review.openstack.org/87274 | 14:20 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: added query for neutron migrations issue https://review.openstack.org/87274 | 14:21 |
| sdague | yeh, I make that mistake all the time | 14:21 |
| mriedem | i need more coffee | 14:21 |
| mriedem | +A | 14:21 |
| sdague | I'm done on more coffee for this morning | 14:21 |
| *** otherwiseguy has joined #openstack-qa | 14:22 | |
| mriedem | now to just get these heartbleed guys off my back | 14:22 |
| mriedem | "you have to update!" "no i don't your scan tool sux" | 14:22 |
| mtreinish | mriedem: just hack the scan tool to say your in compliance | 14:23 |
| *** kgriffs|afk is now known as kgriffs | 14:23 | |
| mriedem | mtreinish: meh | 14:23 |
| mtreinish | mriedem: oh I missed the windows part sorry | 14:23 |
| mriedem | i already did the run around with 2 lab guys in POK | 14:23 |
| mriedem | mtreinish: unrelated | 14:23 |
| mriedem | my laptop is windows, my lab servers are RHEL 6.4 | 14:23 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: added query for neutron migrations issue https://review.openstack.org/87274 | 14:28 |
| *** openstackrecheck has quit IRC | 14:30 | |
| *** openstackrecheck has joined #openstack-qa | 14:30 | |
| *** mestery_ is now known as mestery | 14:32 | |
| *** dims has quit IRC | 14:35 | |
| *** kgriffs is now known as kgriffs_666 | 14:39 | |
| *** sthaha has quit IRC | 14:39 | |
| *** kgriffs_666 is now known as kgriffs | 14:40 | |
| *** dkorn has quit IRC | 14:41 | |
| *** dims has joined #openstack-qa | 14:42 | |
| *** otherwiseguy has quit IRC | 14:43 | |
| malini | dkranz: ping | 14:43 |
| dkranz | malini: Hi | 14:44 |
| malini | dkranz: how are you? | 14:44 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Fix typo in nova_admin_username https://review.openstack.org/85484 | 14:44 |
| dkranz | malini: pretty well. What's up? | 14:45 |
| malini | dkranz: I see you have a session suggestion in https://etherpad.openstack.org/p/Juno-QA-design-summit-topics | 14:45 |
| dkranz | malini: Yes | 14:45 |
| malini | I created a summit session a while back http://summit.openstack.org/cfp/details/134 ..Its along the same lines | 14:45 |
| dkranz | malini: Yes, that is the same though I did not see it. | 14:46 |
| malini | dkranz: I missed adding it to etherpad | 14:46 |
| malini | I was wondering if its ok to add this one in the session link to your etherpad entry | 14:47 |
| dkranz | malini: Sure | 14:47 |
| *** amotoki has joined #openstack-qa | 14:47 | |
| dkranz | malini: I think there was a little mixup between the etherpad and the "official" submission place | 14:47 |
| *** HenryG has joined #openstack-qa | 14:48 | |
| malini | dkranz: awesome.I just updated etherpad with the session link | 14:48 |
| malini | We can update the session details as our thoughts evolve | 14:48 |
| openstackgerrit | A change was merged to openstack/tempest: Add list roles api to identity v3 https://review.openstack.org/76143 | 14:51 |
| yfried__ | anyone knows how to get test logs for successful runs? I want to compare failed tests to succesfull ones running on the same setup, but tempest.log isn't devided by test, and test logs are only printed for failed tests | 14:53 |
| *** Guest38269 is now known as jeblair | 14:57 | |
| *** jeblair is now known as Guest34642 | 14:57 | |
| *** mwagner_lap has joined #openstack-qa | 14:58 | |
| *** Guest34642 is now known as corvus | 14:58 | |
| openstackrecheck | Console logs not available after 13:30s for gate-trove-python27 63789,22,be2ca75 | 15:02 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/85567 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1253896, gate-grenade-dsvm-neutron: unrecognized error | 15:03 |
| *** chandan_kumar has quit IRC | 15:11 | |
| jordanP | dtroyer, hi/ https://review.openstack.org/#/c/87140/ was commited recently "Prevent the use of 'swift' in ENABLED_SERVICES" | 15:12 |
| jordanP | dtroyer, but now I can't use swift3 middleware... | 15:12 |
| *** jpich has quit IRC | 15:12 | |
| *** dims has quit IRC | 15:13 | |
| *** otherwiseguy has joined #openstack-qa | 15:16 | |
| *** jpich has joined #openstack-qa | 15:19 | |
| *** om has joined #openstack-qa | 15:21 | |
| *** kmartin has joined #openstack-qa | 15:24 | |
| *** zul has quit IRC | 15:24 | |
| *** devvesa has quit IRC | 15:25 | |
| *** zul has joined #openstack-qa | 15:25 | |
| *** enikanorov has joined #openstack-qa | 15:27 | |
| *** chuck_ has joined #openstack-qa | 15:31 | |
| openstackrecheck | Console logs not available after 13:35s for gate-python-cinderclient-pypy 69898,3,286180b | 15:31 |
| *** zul has quit IRC | 15:31 | |
| *** chuck_ is now known as zul | 15:32 | |
| mriedem | sdague: bash8 doesn't support # noqa? | 15:33 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/85585 failed because of: gate-grenade-dsvm-neutron: unrecognized error, gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1302774 | 15:34 |
| *** atiwari has joined #openstack-qa | 15:36 | |
| *** dims has joined #openstack-qa | 15:39 | |
| *** enikanorov_ has quit IRC | 15:45 | |
| *** mmaglana has joined #openstack-qa | 15:48 | |
| *** asselin has joined #openstack-qa | 15:49 | |
| dkranz | mtreinish: In your last comment about non-admin with regard to configuration, are you talking about essentially having a service tag for admin? | 15:51 |
| *** dims has quit IRC | 15:52 | |
| mtreinish | dkranz: we already basically have it because the tests are named admin | 15:53 |
| mtreinish | but yes if that doesn't cover everything | 15:53 |
| mtreinish | we add an attr for it | 15:53 |
| dkranz | mtreinish: ok, it does not cover everything. I think it would be cleaner to just put the attr on the class even if there was Admin in the name. | 15:54 |
| dkranz | mtreinish: I will fix up that spec and resubmit. | 15:55 |
| *** zul has quit IRC | 16:02 | |
| *** zul has joined #openstack-qa | 16:04 | |
| *** zul has quit IRC | 16:04 | |
| sdague | mriedem: no | 16:04 |
| sdague | it's pretty basic | 16:04 |
| *** dims has joined #openstack-qa | 16:05 | |
| mriedem | yeah :) you provide something that's better than nothing and now i want it to do everything! | 16:05 |
| sdague | heh | 16:05 |
| mriedem | "patches welcome" | 16:05 |
| mtreinish | mriedem: nothing is stopping to from doing that | 16:05 |
| mriedem | yeah yeah | 16:05 |
| *** qba73_ has quit IRC | 16:06 | |
| sdague | yeh, honestly, if I get a few moments pre summit I'll finally split it off | 16:06 |
| mriedem | first i have to see if we have a regression in nova resizes | 16:06 |
| openstackgerrit | Mark Maglana proposed a change to openstack/tempest: Make sure to release the floating ip afterwards https://review.openstack.org/79153 | 16:06 |
| *** mmaglana has quit IRC | 16:07 | |
| *** mmaglana has joined #openstack-qa | 16:07 | |
| *** zul has joined #openstack-qa | 16:08 | |
| *** mmaglana has quit IRC | 16:12 | |
| *** dims has quit IRC | 16:13 | |
| *** jaypipes has joined #openstack-qa | 16:16 | |
| *** ChanServ sets mode: +o jaypipes | 16:16 | |
| *** Aimon has quit IRC | 16:16 | |
| mriedem | mtreinish: we don't have any api tests that test server resize from shutoff state do we? i'm not seeing any | 16:24 |
| *** jpich has quit IRC | 16:24 | |
| mriedem | nope, i need to add one | 16:24 |
| *** dims has joined #openstack-qa | 16:27 | |
| jaypipes | mkoderer: hey! want to chat about https://bugs.launchpad.net/nova/+bug/1306602 | 16:28 |
| jaypipes | mkoderer: I'm a bit confused as to why this is not expected behaviour... | 16:28 |
| mkoderer | jaypipes: hi | 16:29 |
| mkoderer | jaypipes: yep, so if you specify the default AZ in the cli it behaves different if you don't | 16:30 |
| jaypipes | mkoderer: that is correct behaviour, no? | 16:30 |
| jaypipes | AZ=None != AZ=default | 16:31 |
| mkoderer | jaypipes: is it? why.. ;) | 16:31 |
| mkoderer | jaypipes: so meaning AZ=None matches all nodes is correct? | 16:31 |
| jaypipes | mkoderer: well, I'm interpreting AZ=None as "any AZ", and interpreting AZ=default as "choose the AZ that is the default for the endpoint that the CLI is connecting to" | 16:32 |
| jaypipes | mkoderer: but this is one of those areas that is very poorly tested to date :( So, perhaps it is worth a ML post about it. | 16:32 |
| *** chandan_kumar has joined #openstack-qa | 16:33 | |
| mkoderer | jaypipes: I interpret it as "all hosts are in the default AZ as long there are not assigned to another one". | 16:33 |
| mkoderer | jaypipes: question if there is a case where you want to boot a vm and you don't care about the AZ | 16:34 |
| mkoderer | jaypipes: but you're right.. let's take it to the ML | 16:34 |
| jaypipes | mkoderer: yes, agree with you on the above. | 16:34 |
| jaypipes | re: host is in the default AZ (i.e. the availablity_zone option in nova.conf) if it is not specified to be in a host aggregate that is tagged with an availability zone | 16:35 |
| jaypipes | (this is, BTW, one of the reasons why I think hacking host aggregates with the concept of an availability zone was A Bad Idea ;) | 16:36 |
| jaypipes | mkoderer: regardless of result of ML discussion, looks like there's a couple unit test issues to fix on https://review.openstack.org/#/c/87247/ | 16:37 |
| mkoderer | jaypipes: yep I think the design is not really clear in that area | 16:37 |
| mkoderer | jaypipes: Yep.. I will fix them tomorrow... ;) | 16:37 |
| jaypipes | coolio. | 16:37 |
| jaypipes | mkoderer: I'll write a ML post, ccing you. | 16:38 |
| mkoderer | jaypipes: thx! | 16:38 |
| jaypipes | np | 16:38 |
| *** om has quit IRC | 16:38 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: WIP: Add API test for resizing a stopped server https://review.openstack.org/87312 | 16:39 |
| *** yassine has quit IRC | 16:40 | |
| *** Aimon_ has joined #openstack-qa | 16:42 | |
| *** mmaglana has joined #openstack-qa | 16:43 | |
| openstackrecheck | Console logs not available after 13:29s for check-grenade-dsvm 57715,13,dff8aaa | 16:43 |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/86505 failed gate-tempest-dsvm-neutron-heat-slow, gate-grenade-dsvm-neutron in the gate queue with an unrecognized error | 16:43 |
| *** jordanP has quit IRC | 16:44 | |
| *** akamyshnikova has quit IRC | 16:46 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: WIP: Add API test for resizing a stopped server https://review.openstack.org/87312 | 16:46 |
| *** harlowja_away is now known as harlowja | 16:54 | |
| *** afazekas has quit IRC | 16:55 | |
| *** fbo is now known as fbo_away | 16:56 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: normalize the services for a post grizzly world https://review.openstack.org/84287 | 16:57 |
| *** akamyshnikova has joined #openstack-qa | 16:59 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/76029 failed because of: gate-grenade-dsvm-neutron: unrecognized error, gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1295876 | 17:03 |
| *** nati_ueno has joined #openstack-qa | 17:04 | |
| mtreinish | mriedem: nope I don't think that we do | 17:10 |
| mriedem | mtreinish: we will if this passes https://review.openstack.org/#/c/87312/ | 17:10 |
| mriedem | then i'll add the same for the v3 api | 17:11 |
| *** corvus is now known as jeblair | 17:11 | |
| *** mrodden has quit IRC | 17:12 | |
| mkoderer | jaypipes: what came in my mind while driving home ;) Why should you need a "default AZ" if this is not default in the CLI | 17:20 |
| jaypipes | mkoderer: because the default AZ in the CLI is whatever Nova endpoint you are hitting, whereas the default AZ for a host is whatever is in nova.conf on that host, yes? | 17:22 |
| *** mrodden has joined #openstack-qa | 17:24 | |
| mkoderer | jaypipes: isn't the default AZ in nova.conf deprecated.. not sure about that, need to check the code | 17:26 |
| jaypipes | mkoderer: I didn't think so... | 17:26 |
| mkoderer | jaypipes: ah no this was cinder | 17:31 |
| mkoderer | jaypipes: anyway.. so yes the default AZ is the one defined in nova.conf in the API endpoint | 17:33 |
| jaypipes | yeah | 17:33 |
| mkoderer | jaypipes: for me if a default is set I don't have to specify it on the cli | 17:34 |
| jaypipes | mkoderer: right, but what if you specify availability-zone=None? :) | 17:36 |
| mkoderer | jaypipes: better the CLI wouldn't set availability-zone at all | 17:38 |
| jaypipes | agreed. | 17:38 |
| mkoderer | jaypipes: so then it should take "default" AZ right? ;) | 17:39 |
| jaypipes | yeah, i think so. | 17:39 |
| mkoderer | jaypipes: cool, let me fix it tomorrow in nova cli and nova | 17:40 |
| dkranz | chmouel: The swift client set up in OfficialClientManager needs admin creds but it doesn't seem like the swift scenario test does. Is it possible to run test_swift_basic_ops with admin creds? | 17:40 |
| dkranz | chmouel: without admin creds I mean | 17:40 |
| dkranz | notmyname: ^^^ | 17:41 |
| jaypipes | mkoderer: sounds good. | 17:41 |
| mkoderer | jaypipes: but anyway we can take it to the ML.. maybe there are some other thoughts around | 17:42 |
| jaypipes | mkoderer: frankly, I think the discussion may devolve into yet another debate about what is messed up with host aggregates and availability zones... not sure it would be all that useful at this point -- or add anything more than what has already been said about the topic :( | 17:44 |
| notmyname | dkranz: I have no idea (because I don't know what OfficialClientManager is or what admin means) | 17:44 |
| dkranz | notmyname: ok, who is the best person to ask about tempest/swift issues? | 17:44 |
| shardy | Can anyone advise the best way to run just one upgrade test in grenade? | 17:45 |
| mkoderer | jaypipes: lol all right ;) pls just help me to get it merged :D | 17:45 |
| shardy | Running grenade.sh sets up /opt/stack/havana, but then running upgrade-* individually looks for /opt/stack/icehouse which doesn't exist? | 17:45 |
| mtreinish | dkranz: it probably just needs permissions to make a container | 17:46 |
| mtreinish | which I believe we have a role config option for | 17:46 |
| jaypipes | mkoderer: will do my best :) | 17:46 |
| mtreinish | the admin perms are probably just for tenant isolation | 17:46 |
| dkranz | mtreinish: Yeah, but the role needs to be added which is why it is making a swiftclient admin | 17:46 |
| dkranz | mtreinish: I will just turn it off for now so I can see what fails with no admin creds | 17:47 |
| jaypipes | shardy: dtroyer may be able to assist you on that. | 17:48 |
| notmyname | dkranz: my understanding is that "admin" from a keystone perspective means that you can modify keystone. is that still correct? | 17:48 |
| dkranz | mtreinish: Unfortunately I have not found a way to blank the admin creds, fix this client issue, and turn off tenant isolation with a change to a single project | 17:48 |
| shardy | jaypipes: thanks, the docs I've found are a bit light on details, I'm probably missing something obvious ;) | 17:48 |
| dkranz | notmyname: Yes, and each service has some apis that are admin-only | 17:48 |
| dkranz | notmyname: In this case it is calling keystone to add an operator role | 17:49 |
| notmyname | dkranz: if my understanding is correct, I don't think you should need keystone admin rights to do anything in swift | 17:49 |
| dkranz | notmyname: I will find out soon :) | 17:50 |
| *** otherwiseguy has quit IRC | 17:50 | |
| notmyname | dkranz: "it is calling keystone" <-- what is "it" referring to? | 17:50 |
| dkranz | notmyname: The code that is creating the swiftclient connection | 17:50 |
| notmyname | dkranz: in swiftclient or tempest code? | 17:50 |
| dkranz | notmyname: tempest | 17:51 |
| notmyname | dkranz: what is the operator role? | 17:51 |
| dkranz | notmyname: It would be easiest to point you at the code. Just a sec | 17:51 |
| notmyname | dkranz: line 574 of clients.py? | 17:51 |
| dkranz | notmyname: yes | 17:52 |
| dkranz | notmyname: 547 | 17:52 |
| notmyname | well, that says it's being added, but not what it is :-) | 17:52 |
| *** mkollaro has quit IRC | 17:52 | |
| *** enikanorov_ has joined #openstack-qa | 17:52 | |
| dkranz | notmyname: Right, and the comment says it is necassry so the test user can operate swift. | 17:53 |
| dkranz | notmyname: But I'm not sure why that is. | 17:53 |
| dkranz | notmyname: In any event I am trying to run without it and will see what happens. That will give more concrete info. | 17:53 |
| mtreinish | notmyname: the operator role is just a user role in keystone that has permissions to create containers | 17:55 |
| mtreinish | notmyname: it's set via a config option and it should match the keystoneauth middleware setting | 17:55 |
| *** mkollaro has joined #openstack-qa | 17:57 | |
| notmyname | mtreinish: looking at sample configs in the respective codebases, tempest has operator_role set to "Member", and swift has keystone middleware configured as "operator_roles= = admin, swiftoperator" | 17:59 |
| openstackrecheck | Console logs not available after 13:41s for gate-nova-pep8 85804,6,3b38de9 | 18:00 |
| *** marun has joined #openstack-qa | 18:00 | |
| mtreinish | notmyname: well that just comes from the default in tempest.config, I think Member is what devstack sets so that's probably where the default value came from | 18:01 |
| mtreinish | but we can change it to be more consistent with the swift default | 18:01 |
| notmyname | mtreinish: is that a string match that has to be equal? | 18:02 |
| mtreinish | notmyname: is what a string match? the sample config with the value in the config.py file? | 18:02 |
| notmyname | mtreinish: sorry. is "Member" a string that is compared to the keystone middleware settings or is is a reference to something else? | 18:03 |
| *** catohornet has joined #openstack-qa | 18:04 | |
| notmyname | mtreinish: I'm slowly understanding how the tempest/keystone parts are put together with swift | 18:04 |
| mtreinish | Member is just the default value we set for the tempest config option. The burden is really on the user to verify that it matches what's in the keystone middleware setting | 18:05 |
| mtreinish | if it's incorrect the swift tests that use it will just fail | 18:05 |
| mtreinish | I think it's only really used in the tenant isolation case (where each test class makes a keystone tenant and user) so we add the role to the new user | 18:06 |
| mtreinish | to ensure that the user for the swift tests have permissions to do what's needed | 18:06 |
| notmyname | mtreinish: what you are calling "operator role" is, in swift, the "owner" of the swift account. an identity (or rather, an auth token) with that role has the ability to anything they want inside of a swift account | 18:06 |
| notmyname | (including creating containers) | 18:07 |
| *** nati_ueno has quit IRC | 18:08 | |
| mtreinish | notmyname: ok, that makes sense. Do you think that there is a better way we could be handling things here then? | 18:08 |
| notmyname | mtreinish: in which case, yes that operator role will be required for swift tests. | 18:08 |
| mtreinish | because I added the operator stuff in when I was trying to make the tests run with tenant isolation enabled | 18:09 |
| *** nati_ueno has joined #openstack-qa | 18:09 | |
| mtreinish | and I didn't dive to deeply into the swift-keystone user permission model | 18:09 |
| notmyname | mtreinish: from my (limited) understanding of keystone and tempest, it seems like the right thing is being done, but the wording or description isn't all that clear. also, I don't know if it's the best or right way to accomplish that goal in swift | 18:10 |
| notmyname | mtreinish: I too don't fully grok the swift-keystone permission model. it's actually set completely on the keystone side of things (swift has an auth interface usable by many auth systems, but doesn't provide semantic meaning to the values stored in the designated fields--thus my lack of understanding of keystone's particular semantics) | 18:12 |
| notmyname | oops..earlier statement should have said "...I don't know if it's the best or right way to accomplish that goal in keystone" | 18:12 |
| openstackgerrit | Steven Hardy proposed a change to openstack-dev/grenade: Add initial support for Heat https://review.openstack.org/86978 | 18:13 |
| mtreinish | ok, well we can definitely improving the description of the config option to make it clearer what it is | 18:13 |
| mtreinish | other than that I guess we can just leave it as good enough for now, if we need to make further refinements down the road we can tackle that as the need arises | 18:14 |
| mtreinish | dkranz: for your non-admin work non-tenant isolated stuff, you might just want to check that the user has the role set up front and error out if it doesn't | 18:15 |
| *** morganfainberg_Z is now known as morganfainberg | 18:15 | |
| dkranz | mtreinish: Yes. By coincidence the offending code was just proposed as stable/havana backport. | 18:15 |
| dkranz | mtreinish: I am going to see what happens without it | 18:16 |
| mtreinish | heh, yeah I think that I just approved that through recently | 18:16 |
| dkranz | mtreinish: IMO, tempest should not be doing this | 18:16 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Consider state in firewall create/delete test https://review.openstack.org/85622 | 18:16 |
| mtreinish | doing what? | 18:16 |
| dkranz | mtreinish: Creating some role that is needed to run a set of tests. | 18:17 |
| dkranz | mtreinish: It should be part of tempest setup/config | 18:17 |
| mtreinish | it is, tempest doesn't create the role | 18:17 |
| mtreinish | it just assigns create users the role | 18:17 |
| mtreinish | so they have permissions to create containers | 18:17 |
| dkranz | mtreinish: I misspoke but it is really the same thing if we want to be able to run as non-admin | 18:17 |
| mtreinish | dkranz: it's basically the same except that since it's limited to just swift we should error out if swift is enabled | 18:18 |
| dkranz | mtreinish: tempest should not try to do any admin thing that is not directly controlled by a flag or explicitly part of some test that needs admin | 18:18 |
| mtreinish | and the user doesn't have the proper permissions | 18:18 |
| mtreinish | dkranz: assigning the role should be limited to the tenant isolation code | 18:19 |
| dkranz | mtreinish: That is the fall back way but I will try to find something better. | 18:19 |
| mtreinish | which requires admin | 18:19 |
| dkranz | mtreinish: That would be fine with me. | 18:19 |
| *** andreaf has quit IRC | 18:20 | |
| dkranz | mtreinish: So if I comment out the role assignment, the only test that uses it still passes... | 18:24 |
| *** lcostantino has joined #openstack-qa | 18:25 | |
| lcostantino | hi, i was trying to deploy a cinder-volume host only with devstack, but apts/cinder seems to be missing python-mysqldb, etc as nova has. is that expected? | 18:26 |
| *** jlibosva has quit IRC | 18:29 | |
| mtreinish | dkranz: that's probably because devstack assigns the role the demo user when swift is enabled | 18:40 |
| dkranz | mtreinish: Another reason that code should not be there | 18:41 |
| mtreinish | which code are you talking about? | 18:41 |
| dkranz | mtreinish: The code in tempest that adds the role | 18:41 |
| dkranz | mtreinish: The heat tests require an admin role in the demo user but tempest does not try to add it. | 18:42 |
| mtreinish | dkranz: link? | 18:42 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Add API test for resizing a stopped server https://review.openstack.org/87312 | 18:42 |
| *** aysyd has quit IRC | 18:42 | |
| dkranz | mtreinish: I will chase it down. | 18:43 |
| dkranz | mtreinish: from lib/keystone: openstack role add --project $DEMO_TENANT --user $ADMIN_USER $ADMIN_ROLE | 18:47 |
| mtreinish | yeah that's the devstack part of it | 18:47 |
| dkranz | mtreinish: But I am just agreeing with what you said which was that such code should either be outside of tempest or in the isolated creds code. | 18:48 |
| mtreinish | dkranz: which I thought it was, where do we do it elsewhere? | 18:48 |
| mtreinish | that's what I was asking | 18:48 |
| dkranz | mtreinish: In the OfficialClientManager code | 18:48 |
| dkranz | mtreinish: Sorry, you probably missed the initial start of this irc thread | 18:49 |
| dkranz | mtreinish: _get_object_storage_client | 18:49 |
| mtreinish | dkranz: yeah propose a patch to remove that try block it was just a hack because of neutron | 18:51 |
| mtreinish | we may still need it on stable/havanna though because of tenant isolation issues with neutron | 18:51 |
| dkranz | mtreinish: I was objecting to the code inside the try block too. | 18:52 |
| *** tosky has quit IRC | 18:52 | |
| dkranz | mtreinish: How do you propose that we can run the swift scenario test as non-admin with that code there? | 18:52 |
| dkranz | mtreinish: You should not have to be an admin to use swift, right? | 18:53 |
| mtreinish | dkranz: no, the user just needs to have that role assigned | 18:53 |
| mtreinish | which you could either explicitly check for in the client | 18:53 |
| mtreinish | and raise an exception if it doesnt | 18:53 |
| mtreinish | or just let it fail down the line | 18:53 |
| dkranz | mtreinish: Yes, and devstack already does that so we could just move that code to isolated tenant | 18:53 |
| dkranz | mtreinish: Why should it fail if you are non-isolated and non-admin? | 18:54 |
| mtreinish | dkranz: that code is already in tenant isolation | 18:54 |
| dkranz | mtreinish: Ah, I didn't realize that. So we can get rid of the whole thing then (role assignment in that method) | 18:54 |
| *** aysyd has joined #openstack-qa | 18:55 | |
| mtreinish | err, maybe. I think we actually do it in the swift baseclass if tenant isolation is enabled | 18:55 |
| mtreinish | because we didn't want it assigned everywhere | 18:55 |
| mtreinish | dkranz: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/object_storage/base.py#n39 | 18:56 |
| mtreinish | we need something like that for the scenario test | 18:56 |
| *** catohornet has quit IRC | 18:56 | |
| mtreinish | but only if tenant_isolation | 18:56 |
| dkranz | mtreinish: I see. But that is not used by scenario. | 18:56 |
| dkranz | mtreinish: So I guess we just need to expand the try to include the call to get keystone admin creds | 18:57 |
| mtreinish | no it's not, and that's the crux of the confusion here I think. We need an equiv method using python-keystoneclient for the swift scenario test | 18:58 |
| dkranz | mtreinish: I think we have it, just not conditioned on non-tenant-isolation | 18:58 |
| dkranz | mtreinish: So that is the right fix. | 18:58 |
| dkranz | mtreinish: Only call that method if no isolation | 18:59 |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/devstack: Remove sshd reconfig from lib/ironic https://review.openstack.org/87355 | 18:59 |
| mtreinish | dkranz: I think you have that backwards. We should only assign the role to the user if tenant isolation is enabled | 18:59 |
| dkranz | mtreinish: Yes, that is what I meant. | 19:00 |
| dkranz | mtreinish: I'll push a fix after I understand why the get_network_client is *not* failing :) | 19:00 |
| *** nati_ueno has quit IRC | 19:01 | |
| *** jhenner has quit IRC | 19:02 | |
| *** jhenner has joined #openstack-qa | 19:02 | |
| *** jhenner has quit IRC | 19:02 | |
| *** nati_uen_ has joined #openstack-qa | 19:02 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add support for updating the config file https://review.openstack.org/83585 | 19:05 |
| *** aysyd has quit IRC | 19:10 | |
| mtreinish | sdague: have you seen this before: http://logs.openstack.org/85/85085/2/gate/gate-tempest-dsvm-neutron/f828d37/console.html.gz | 19:12 |
| mtreinish | where devstack takes >20min to finish | 19:12 |
| mtreinish | I looked in the devstack log and I didn't see anything obvious | 19:12 |
| mtreinish | there were some errors but logstash showed them mostly on successful runs | 19:13 |
| *** aysyd has joined #openstack-qa | 19:16 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: increase heat guests https://review.openstack.org/85078 | 19:21 |
| *** chandan_kumar has quit IRC | 19:22 | |
| *** enikanorov_ has quit IRC | 19:26 | |
| *** enikanorov_ has joined #openstack-qa | 19:27 | |
| openstackrecheck | Console logs not available after 14:25s for gate-horizon-python27 76133,5,d24f61f | 19:27 |
| sdague | mtreinish: looks like we're building kernel modules - http://logs.openstack.org/85/85085/2/gate/gate-tempest-dsvm-neutron/f828d37/logs/devstacklog.txt.gz#_2014-04-11_23_09_16_107 | 19:31 |
| *** lcostantino has quit IRC | 19:31 | |
| sdague | also, there are just some super slow nodes in rax | 19:32 |
| sdague | we probably hit one | 19:32 |
| mtreinish | ahh ok just recheck no bug then? | 19:32 |
| mtreinish | or do we have a generic timeout one | 19:32 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/76029 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1273908, gate-grenade-dsvm-neutron: unrecognized error | 19:32 |
| mtreinish | the kernel module build only took a little over a min though | 19:33 |
| *** malini is now known as malini_afk | 19:38 | |
| dkranz | mtreinish: One more thing. I am going to change the proposal to say that admin-required tests should be tagged as such and attempts to do admin things without admin creds will just fail. | 19:39 |
| dkranz | mtreinish: That is, no skipping of admin tests if there are no admin creds. Do you have a problem with that? | 19:40 |
| dkranz | mtreinish: So you run non-admin tests by setting isolated to false, admin creds to none, and selecting the regexp to not include admin tests | 19:40 |
| dkranz | mtreinish: I don't see the value in implementing skipping everywhere | 19:41 |
| mtreinish | dkranz: honestly the best way to do it is probably the helper method/decorator approach | 19:41 |
| mtreinish | like what I did with the service tags | 19:41 |
| mtreinish | the helper will check that admin creds are set to None | 19:42 |
| openstackgerrit | Dirk Mueller proposed a change to openstack/tempest: Fix occurrences of F812 https://review.openstack.org/87366 | 19:42 |
| mtreinish | then you can skip based on that | 19:42 |
| *** lcostantino has joined #openstack-qa | 19:42 | |
| mtreinish | the decorator will also add the attr besides skipping | 19:42 |
| mtreinish | because if admin creds aren't set we really shouldn't try to run the admin tests | 19:42 |
| dkranz | mtreinish: ok | 19:43 |
| openstackgerrit | Rob Crittenden proposed a change to openstack/tempest: botoclient does not honor disable_ssl_certificate_validation https://review.openstack.org/87367 | 19:43 |
| mtreinish | dkranz: for the tenant isolation stuff we probably should raise an Invalid configuration exception if tenant isolation is set without admin creds | 19:44 |
| mtreinish | or do you think failing through is better there? I really don't have a preference | 19:44 |
| dkranz | mtreinish: I was going to fail it | 19:44 |
| dkranz | mtreinish: Because it is going to fail anyway just in a less obvious way | 19:45 |
| *** nati_uen_ has quit IRC | 19:47 | |
| *** otherwiseguy has joined #openstack-qa | 19:48 | |
| *** fbo_away is now known as fbo | 19:59 | |
| *** otherwiseguy has quit IRC | 19:59 | |
| *** lcostantino has left #openstack-qa | 20:03 | |
| openstackgerrit | David Kranz proposed a change to openstack/qa-specs: Spec for running tempest as non-admin https://review.openstack.org/86967 | 20:04 |
| sdague | mtreinish: so the real issue is that job is still not parallel, and it landed on a slow node | 20:19 |
| sdague | the ec2 test took 4 minutes | 20:19 |
| sdague | for instance | 20:19 |
| *** mwagner_lap has quit IRC | 20:22 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/devstack: Insert required iptables rules for Ironic https://review.openstack.org/87377 | 20:24 |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/devstack: Insert required iptables rules for Ironic https://review.openstack.org/87377 | 20:25 |
| *** otherwiseguy has joined #openstack-qa | 20:28 | |
| *** dkranz has quit IRC | 20:32 | |
| *** nati_ueno has joined #openstack-qa | 20:32 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/79618 failed because of: gate-grenade-dsvm-neutron: unrecognized error, gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1297560 | 20:33 |
| *** Aimon_ is now known as Aimon | 20:42 | |
| *** yamahata has joined #openstack-qa | 20:47 | |
| *** nati_ueno has quit IRC | 20:48 | |
| *** nati_uen_ has joined #openstack-qa | 20:52 | |
| *** nati_uen_ has quit IRC | 20:56 | |
| *** jlibosva has joined #openstack-qa | 20:56 | |
| *** Sukhdev has joined #openstack-qa | 20:57 | |
| mtreinish | sdague: ok I'll just no bug it then, since it was just a slow node | 20:59 |
| *** fbo is now known as fbo_away | 20:59 | |
| *** markmcclain has joined #openstack-qa | 21:00 | |
| *** nati_ueno has joined #openstack-qa | 21:00 | |
| *** nati_ueno has quit IRC | 21:01 | |
| *** nati_ueno has joined #openstack-qa | 21:02 | |
| sdague | mtreinish: honestly, we should probably open an issue to track it | 21:05 |
| sdague | so we can see how often we're getting hit by it | 21:05 |
| openstackrecheck | Console logs not available after 13:27s for gate-glance-pep8 82899,4,e899806 | 21:16 |
| openstackgerrit | Steven Hardy proposed a change to openstack-dev/grenade: Add initial support for Heat https://review.openstack.org/86978 | 21:19 |
| mtreinish | sdague: with what fingerprint just the timeout one, because that seems a bit too generic | 21:32 |
| sdague | timeout is fine | 21:33 |
| sdague | well, timeout + job | 21:33 |
| sdague | that will let us track jobs that are hitting a threshold regularly | 21:33 |
| mtreinish | ok I'll add a bug and fingerprint for that later tonight | 21:34 |
| sdague | cool, thanks | 21:36 |
| *** giulivo has quit IRC | 21:44 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/87173 failed because of: gate-grenade-dsvm-neutron: unrecognized error, gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1297560 | 21:45 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/87013 failed because of: gate-grenade-dsvm-neutron: unrecognized error, gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1297560 | 21:49 |
| *** chuck_ has joined #openstack-qa | 21:50 | |
| *** zul has quit IRC | 21:51 | |
| *** mriedem has quit IRC | 21:53 | |
| *** chuck_ has quit IRC | 21:53 | |
| *** zul has joined #openstack-qa | 21:53 | |
| *** Sukhdev_ has joined #openstack-qa | 21:55 | |
| *** zul has quit IRC | 21:57 | |
| *** Sukhdev__ has joined #openstack-qa | 21:57 | |
| *** Sukhdev_ has quit IRC | 21:57 | |
| *** dims has quit IRC | 21:57 | |
| *** Sukhdev has quit IRC | 21:58 | |
| *** markmcclain has quit IRC | 22:01 | |
| *** jlibosva has quit IRC | 22:02 | |
| *** om has joined #openstack-qa | 22:04 | |
| *** dims has joined #openstack-qa | 22:10 | |
| *** Sukhdev__ has quit IRC | 22:14 | |
| *** zul has joined #openstack-qa | 22:17 | |
| *** zul has quit IRC | 22:21 | |
| *** zul has joined #openstack-qa | 22:21 | |
| *** zul has quit IRC | 22:24 | |
| *** chuck_ has joined #openstack-qa | 22:25 | |
| *** om has quit IRC | 22:28 | |
| *** rbak has joined #openstack-qa | 22:28 | |
| *** mfisch has joined #openstack-qa | 22:30 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/79619 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1226412 https://bugs.launchpad.net/bugs/1254872, gate-grenade-dsvm-neutron: unrecognized error | 22:30 |
| *** marun has quit IRC | 22:31 | |
| *** chuck_ is now known as zul | 22:34 | |
| *** otherwiseguy has quit IRC | 22:38 | |
| *** adalbas has quit IRC | 22:42 | |
| *** mrodden has quit IRC | 22:45 | |
| openstackgerrit | Paul Van Eck proposed a change to openstack/tempest: Add tests for nova floating IP Heat resources https://review.openstack.org/82282 | 22:46 |
| *** nati_ueno has quit IRC | 22:49 | |
| *** nati_ueno has joined #openstack-qa | 22:49 | |
| *** dims has quit IRC | 22:57 | |
| *** adalbas has joined #openstack-qa | 22:57 | |
| *** rbak has quit IRC | 22:59 | |
| *** yamahata has quit IRC | 23:01 | |
| *** mmaglana has quit IRC | 23:03 | |
| *** jamielennox|away is now known as jamielennox | 23:06 | |
| *** mrodden1 has joined #openstack-qa | 23:06 | |
| *** mwagner_lap has joined #openstack-qa | 23:11 | |
| *** mkollaro has quit IRC | 23:12 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/devstack: Add sudoers.d entry for Ironic conductor https://review.openstack.org/87408 | 23:15 |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/devstack: Insert required iptables rules for Ironic https://review.openstack.org/87377 | 23:15 |
| *** nati_ueno has quit IRC | 23:21 | |
| *** dims has joined #openstack-qa | 23:22 | |
| *** nati_ueno has joined #openstack-qa | 23:22 | |
| *** mrodden has joined #openstack-qa | 23:24 | |
| *** atiwari has quit IRC | 23:26 | |
| *** mrodden1 has quit IRC | 23:27 | |
| *** nati_ueno has quit IRC | 23:40 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Adds test for volume attachments Nova V2 APIs https://review.openstack.org/87191 | 23:41 |
| *** nati_ueno has joined #openstack-qa | 23:42 | |
| *** ianw has joined #openstack-qa | 23:47 | |
| *** yamahata has joined #openstack-qa | 23:47 | |
| *** yamahata has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!