*** salv-orlando has quit IRC | 00:04 | |
*** yamamoto has quit IRC | 00:05 | |
*** yamahata has quit IRC | 00:06 | |
openstackgerrit | Merged openstack-dev/devstack: Append command to screenrc after we update it https://review.openstack.org/174147 | 00:09 |
---|---|---|
*** amotoki has joined #openstack-qa | 00:14 | |
*** yamahata has joined #openstack-qa | 00:18 | |
*** dboik has quit IRC | 00:22 | |
*** mahito has joined #openstack-qa | 00:23 | |
*** sputnik13 has quit IRC | 00:31 | |
*** sputnik13 has joined #openstack-qa | 00:36 | |
*** woodster_ has quit IRC | 01:00 | |
*** dboik has joined #openstack-qa | 01:04 | |
*** salv-orlando has joined #openstack-qa | 01:05 | |
*** salv-orlando has quit IRC | 01:11 | |
*** sputnik13 has quit IRC | 01:12 | |
*** luqas has joined #openstack-qa | 01:17 | |
*** alex_xu has quit IRC | 01:18 | |
*** luqas has quit IRC | 01:21 | |
*** alex_xu has joined #openstack-qa | 01:23 | |
*** mriedem has quit IRC | 01:26 | |
*** yamamoto has joined #openstack-qa | 01:29 | |
*** salv-orlando has joined #openstack-qa | 01:32 | |
*** salv-orlando has quit IRC | 01:35 | |
*** mahito_ has joined #openstack-qa | 01:36 | |
*** mahito has quit IRC | 01:40 | |
*** dboik has quit IRC | 01:40 | |
*** salv-orlando has joined #openstack-qa | 01:41 | |
*** salv-orlando has quit IRC | 01:51 | |
*** achanda has joined #openstack-qa | 01:53 | |
*** mahito_ has quit IRC | 01:56 | |
*** mahito has joined #openstack-qa | 01:58 | |
*** zz_dimtruck is now known as dimtruck | 02:00 | |
*** cody-somerville has joined #openstack-qa | 02:07 | |
*** annashen has joined #openstack-qa | 02:16 | |
*** woodster_ has joined #openstack-qa | 02:28 | |
*** sputnik13 has joined #openstack-qa | 02:28 | |
*** annashen has quit IRC | 02:38 | |
openstackgerrit | Eli Qiao proposed openstack/tempest: fixed_ip unreserve should use an ip liked address https://review.openstack.org/170027 | 02:47 |
*** david-lyle has quit IRC | 02:55 | |
*** annashen has joined #openstack-qa | 02:57 | |
*** annashen has quit IRC | 02:57 | |
*** jamielennox is now known as jamielennox|away | 02:58 | |
*** achanda has quit IRC | 03:00 | |
*** jamielennox|away is now known as jamielennox | 03:03 | |
*** cody-somerville has quit IRC | 03:09 | |
*** armax has joined #openstack-qa | 03:14 | |
*** annashen has joined #openstack-qa | 03:15 | |
*** annashen has quit IRC | 03:20 | |
*** annashen has joined #openstack-qa | 03:21 | |
*** annashen has quit IRC | 03:22 | |
*** annashen has joined #openstack-qa | 03:23 | |
*** annashen has quit IRC | 03:27 | |
*** dboik has joined #openstack-qa | 03:30 | |
*** armax has quit IRC | 03:37 | |
*** coolsvap|afk is now known as coolsvap | 03:39 | |
*** salv-orlando has joined #openstack-qa | 03:45 | |
*** salv-orlando has quit IRC | 03:51 | |
*** dpaterson has quit IRC | 03:58 | |
*** dboik has quit IRC | 04:03 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/167526 failed gate-grenade-dsvm in the gate queue with an unrecognized error | 04:05 |
*** yfried|afk is now known as yfried__ | 04:18 | |
*** rushiagr_away is now known as rushiagr | 04:20 | |
*** subscope has joined #openstack-qa | 04:25 | |
*** sabeen has joined #openstack-qa | 04:28 | |
*** annashen has joined #openstack-qa | 04:31 | |
*** Madan has joined #openstack-qa | 04:31 | |
*** Poornima has joined #openstack-qa | 04:32 | |
*** ajmiller has quit IRC | 04:34 | |
*** Longgeek has joined #openstack-qa | 04:35 | |
*** Longgeek has quit IRC | 04:35 | |
*** Longgeek has joined #openstack-qa | 04:36 | |
*** BharatK has joined #openstack-qa | 04:43 | |
*** subscope has quit IRC | 04:47 | |
*** sks has joined #openstack-qa | 04:48 | |
*** ibiris_away is now known as ibiris | 04:48 | |
*** yfried__ is now known as yfried|afk | 04:49 | |
*** annashen has quit IRC | 04:50 | |
*** k4n0 has joined #openstack-qa | 04:51 | |
*** yfried|afk is now known as yfried__ | 04:51 | |
*** annashen has joined #openstack-qa | 04:51 | |
*** sputnik13 has quit IRC | 04:52 | |
*** luqas has joined #openstack-qa | 04:55 | |
*** sputnik13 has joined #openstack-qa | 04:56 | |
*** annashen has quit IRC | 04:56 | |
*** boris-42 has quit IRC | 04:58 | |
*** ibiris is now known as ibiris_away | 04:58 | |
*** annashen has joined #openstack-qa | 04:59 | |
*** luqas has quit IRC | 04:59 | |
*** dimtruck is now known as zz_dimtruck | 05:00 | |
*** yfried__ has quit IRC | 05:02 | |
*** ibiris_away is now known as ibiris | 05:03 | |
*** sputnik13 has quit IRC | 05:07 | |
*** pal has joined #openstack-qa | 05:13 | |
*** rdekel has joined #openstack-qa | 05:15 | |
*** sabeen has quit IRC | 05:16 | |
*** pal has quit IRC | 05:19 | |
*** sputnik13 has joined #openstack-qa | 05:31 | |
*** sputnik13 has quit IRC | 05:35 | |
*** achanda has joined #openstack-qa | 05:37 | |
*** sputnik13 has joined #openstack-qa | 05:38 | |
*** gamado has joined #openstack-qa | 05:49 | |
*** salv-orlando has joined #openstack-qa | 05:57 | |
*** yamahata has quit IRC | 05:58 | |
*** yamahata has joined #openstack-qa | 05:59 | |
*** annashen has quit IRC | 06:01 | |
*** annashen has joined #openstack-qa | 06:03 | |
*** salv-orlando has quit IRC | 06:06 | |
*** afazekas_ has joined #openstack-qa | 06:10 | |
*** annashen has quit IRC | 06:12 | |
*** anshul has joined #openstack-qa | 06:14 | |
*** annashen has joined #openstack-qa | 06:14 | |
*** yamahata has quit IRC | 06:15 | |
*** SridharG has joined #openstack-qa | 06:15 | |
*** annashen has quit IRC | 06:16 | |
*** annashen has joined #openstack-qa | 06:16 | |
*** zz_jgrimm is now known as jgrimm | 06:19 | |
*** yfried__ has joined #openstack-qa | 06:19 | |
*** annashen has quit IRC | 06:20 | |
*** anshul has quit IRC | 06:27 | |
*** yfried__ is now known as yfried|afk | 06:30 | |
*** woodster_ has quit IRC | 06:30 | |
*** neeti has joined #openstack-qa | 06:33 | |
*** yfried|afk is now known as yfried__ | 06:34 | |
*** sergsh has joined #openstack-qa | 06:37 | |
*** annashen has joined #openstack-qa | 06:39 | |
*** anshul has joined #openstack-qa | 06:40 | |
*** mmedvede has quit IRC | 06:44 | |
*** flaper87 has quit IRC | 06:45 | |
*** flaper87 has joined #openstack-qa | 06:45 | |
*** gillllllliard is now known as gilliard | 06:48 | |
*** anshul has quit IRC | 06:50 | |
*** jamielennox is now known as jamielennox|away | 06:52 | |
*** jyuso1 has joined #openstack-qa | 06:54 | |
*** mmedvede has joined #openstack-qa | 07:00 | |
*** itzikb has joined #openstack-qa | 07:01 | |
*** annashen has quit IRC | 07:02 | |
*** anshul has joined #openstack-qa | 07:04 | |
*** sergsh has quit IRC | 07:06 | |
*** SridharG has left #openstack-qa | 07:12 | |
*** nmagnezi_ has joined #openstack-qa | 07:12 | |
*** mpaolino has joined #openstack-qa | 07:15 | |
*** apevec has joined #openstack-qa | 07:18 | |
*** apevec has quit IRC | 07:18 | |
*** apevec has joined #openstack-qa | 07:18 | |
*** jhenner has joined #openstack-qa | 07:25 | |
*** tochi has quit IRC | 07:28 | |
*** dratushnyy has joined #openstack-qa | 07:29 | |
*** ihrachyshka has joined #openstack-qa | 07:31 | |
*** e0ne has joined #openstack-qa | 07:31 | |
*** bkopilov has quit IRC | 07:32 | |
*** bkopilov has joined #openstack-qa | 07:32 | |
*** e0ne has quit IRC | 07:34 | |
*** SridharG has joined #openstack-qa | 07:36 | |
*** jlanoux has joined #openstack-qa | 07:37 | |
*** luqas has joined #openstack-qa | 07:38 | |
*** e0ne has joined #openstack-qa | 07:38 | |
*** e0ne has quit IRC | 07:40 | |
*** e0ne has joined #openstack-qa | 07:45 | |
*** achanda has quit IRC | 07:47 | |
*** weshay has joined #openstack-qa | 07:52 | |
*** sputnik13 has quit IRC | 07:52 | |
*** sputnik13 has joined #openstack-qa | 07:54 | |
*** apevec has quit IRC | 07:57 | |
*** safchain has joined #openstack-qa | 08:01 | |
*** annashen has joined #openstack-qa | 08:03 | |
*** jordanP has joined #openstack-qa | 08:04 | |
*** e0ne has quit IRC | 08:04 | |
*** sergsh has joined #openstack-qa | 08:05 | |
*** salv-orlando has joined #openstack-qa | 08:07 | |
*** mmedvede has quit IRC | 08:07 | |
*** e0ne has joined #openstack-qa | 08:08 | |
*** annashen has quit IRC | 08:08 | |
*** e0ne is now known as e0ne_ | 08:09 | |
*** salv-orlando has quit IRC | 08:11 | |
*** e0ne_ is now known as e0ne | 08:11 | |
*** sergsh has quit IRC | 08:12 | |
*** tkammer has joined #openstack-qa | 08:14 | |
*** jlibosva has joined #openstack-qa | 08:15 | |
*** jlibosva has quit IRC | 08:15 | |
*** salv-orlando has joined #openstack-qa | 08:17 | |
*** mmedvede has joined #openstack-qa | 08:17 | |
*** e0ne has quit IRC | 08:17 | |
*** vponomaryov has joined #openstack-qa | 08:19 | |
*** shardy has joined #openstack-qa | 08:20 | |
*** salv-orlando has quit IRC | 08:21 | |
*** salv-orlando has joined #openstack-qa | 08:23 | |
*** sergsh has joined #openstack-qa | 08:25 | |
*** salv-orlando has quit IRC | 08:28 | |
*** zz_johnthetubagu is now known as johnthetubaguy | 08:29 | |
*** gfidente has joined #openstack-qa | 08:30 | |
*** dmellado has joined #openstack-qa | 08:30 | |
*** anshul has quit IRC | 08:32 | |
*** mahito has quit IRC | 08:34 | |
*** tkammer has quit IRC | 08:39 | |
*** andreykurilin__ has joined #openstack-qa | 08:39 | |
*** yamamoto has quit IRC | 08:45 | |
*** anshul has joined #openstack-qa | 08:46 | |
*** salv-orlando has joined #openstack-qa | 08:51 | |
*** tkammer has joined #openstack-qa | 08:52 | |
*** neeti has quit IRC | 08:52 | |
*** gamado has quit IRC | 08:55 | |
openstackgerrit | Filip Hubík proposed openstack/tempest: Validate keystone tokens https://review.openstack.org/175130 | 08:58 |
*** e0ne has joined #openstack-qa | 08:59 | |
*** ihrachyshka has quit IRC | 09:00 | |
*** rdekel has quit IRC | 09:03 | |
*** apevec has joined #openstack-qa | 09:04 | |
*** annashen has joined #openstack-qa | 09:04 | |
*** jhenner has quit IRC | 09:05 | |
*** gamado has joined #openstack-qa | 09:08 | |
*** annashen has quit IRC | 09:09 | |
*** ihrachyshka has joined #openstack-qa | 09:11 | |
*** e0ne is now known as e0ne_ | 09:13 | |
*** gamado has quit IRC | 09:13 | |
*** e0ne_ is now known as e0ne | 09:13 | |
*** jlanoux_ has joined #openstack-qa | 09:17 | |
*** jlanoux has quit IRC | 09:20 | |
*** hichihara has quit IRC | 09:23 | |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Wait for complete deletion of volume snapshot before deleting volume https://review.openstack.org/171569 | 09:24 |
*** pcaruana has quit IRC | 09:27 | |
*** Abhishek has joined #openstack-qa | 09:29 | |
*** pcaruana has joined #openstack-qa | 09:30 | |
*** sergsh has quit IRC | 09:32 | |
*** amotoki has quit IRC | 09:35 | |
*** sergsh has joined #openstack-qa | 09:44 | |
*** yfried__ is now known as yfried|afk | 09:48 | |
*** coolsvap is now known as coolsvap|afk | 09:51 | |
*** boris-42 has joined #openstack-qa | 09:51 | |
*** yfried|afk is now known as yfried__ | 09:53 | |
*** jhenner has joined #openstack-qa | 09:56 | |
*** andreykurilin__ has quit IRC | 10:00 | |
*** aix has joined #openstack-qa | 10:01 | |
*** neeti has joined #openstack-qa | 10:01 | |
*** pc_m has joined #openstack-qa | 10:09 | |
*** safchain has quit IRC | 10:12 | |
*** dratushnyy has quit IRC | 10:17 | |
*** cdent has joined #openstack-qa | 10:22 | |
*** BobBall has joined #openstack-qa | 10:25 | |
*** yfried__ is now known as yfried|afk | 10:25 | |
BobBall | mtreinish: Regarding https://review.openstack.org/#/c/174866/ ; You recommended telling the test running not to run the tests - is there a way to do this with devstack-gate? | 10:26 |
BobBall | mtreinish: I wasn't aware of one before, which is why we added https://review.openstack.org/#/c/161155/ for the Citrix CI; Is resurrecting that type of change an option, so we can give the test runner (possibly by an env-var rather than named file) a list of tests not to run? | 10:28 |
*** yfried|afk is now known as yfried__ | 10:31 | |
*** rushiagr is now known as rushiagr_away | 10:38 | |
*** anshul has quit IRC | 10:42 | |
boris-42 | sdague: hi there | 10:44 |
*** rdekel has joined #openstack-qa | 10:49 | |
*** anshul has joined #openstack-qa | 10:55 | |
*** e0ne is now known as e0ne_ | 10:56 | |
sdague | boris-42: hey, what's up? | 10:56 |
boris-42 | sdague: did you see this topic [openstack-dev][all][code quality] Voting coverage job (-1 if coverage get worse after patch) ? | 10:57 |
*** dratushnyy has joined #openstack-qa | 10:57 | |
*** e0ne_ is now known as e0ne | 10:58 | |
sdague | yeh, honestly, it's unclear to me that it's really worth while. It's really easy to up code coverage with mock, and still have all the code do the wrong things. | 10:58 |
sdague | but that's just personal experience out of the nova project, where raw coverage numbers are not out problem | 10:59 |
sdague | but the fact that tests don't always reflect reality, so you are testing the wrong things | 10:59 |
boris-42 | sdague: agree with you on this | 11:00 |
boris-42 | sdague: but it helps in next situation | 11:00 |
boris-42 | sdague: when new developer or lazy just publish patch with bad coverage | 11:00 |
boris-42 | sdague: so reviewers should spend each time to explain that he should write unit tests | 11:00 |
boris-42 | sdague: it's better to have CI that says that | 11:01 |
sdague | honestly, I disagree completely with that statement. The reason people don't write tests often is they don't know how to in our framework, and if you are reviewing the code at least you can open up a conversation with them | 11:02 |
sdague | otherwise they'll just iterate on some random use of mock until they make that number exceed the passing threshold | 11:02 |
sdague | it will pass the number, it doesn't mean it's good | 11:02 |
boris-42 | sdague: I am not saying that this job will allow core team NOT to review unit tests | 11:03 |
*** jhenner1 has joined #openstack-qa | 11:03 | |
boris-42 | sdague: it's the same like in case of functional testing, you can't say without reviewing code that the code works perfect and it is perfect in all cases | 11:04 |
*** jhenner has quit IRC | 11:05 | |
*** jhenner1 has quit IRC | 11:07 | |
*** anshul has quit IRC | 11:14 | |
*** e0ne is now known as e0ne_ | 11:16 | |
*** afazekas_ has quit IRC | 11:24 | |
*** jhenner has joined #openstack-qa | 11:24 | |
*** yfried__ is now known as yfried|afk | 11:25 | |
*** e0ne_ has quit IRC | 11:26 | |
*** yfried|afk is now known as yfried__ | 11:26 | |
*** anshul has joined #openstack-qa | 11:27 | |
*** afazekas_ has joined #openstack-qa | 11:28 | |
*** BobBall has quit IRC | 11:29 | |
HenryG | Anyone here familiar with the workflow for adding stuff to the official cirros images and getting it released? | 11:29 |
HenryG | afazekas: ping | 11:32 |
openstackgerrit | Merged openstack-dev/devstack: run dstat with run_process https://review.openstack.org/171988 | 11:34 |
*** safchain has joined #openstack-qa | 11:36 | |
*** tosky has joined #openstack-qa | 11:36 | |
*** BobBall has joined #openstack-qa | 11:40 | |
*** Poornima has quit IRC | 11:43 | |
*** woodster_ has joined #openstack-qa | 11:46 | |
*** Madan has quit IRC | 11:49 | |
*** mmedvede has quit IRC | 11:56 | |
*** afazekas_ has quit IRC | 11:58 | |
*** ihrachyshka has quit IRC | 12:03 | |
*** mmedvede has joined #openstack-qa | 12:04 | |
*** rfolco has joined #openstack-qa | 12:07 | |
*** ihrachyshka has joined #openstack-qa | 12:19 | |
*** ihrachyshka has quit IRC | 12:19 | |
*** ihrachyshka has joined #openstack-qa | 12:19 | |
*** bknudson has quit IRC | 12:26 | |
*** e0ne has joined #openstack-qa | 12:26 | |
*** salv-orlando has quit IRC | 12:28 | |
*** jamielennox|away is now known as jamielennox | 12:31 | |
*** jamielennox is now known as jamielennox|away | 12:32 | |
*** jaypipes has joined #openstack-qa | 12:33 | |
*** ChanServ sets mode: +o jaypipes | 12:33 | |
*** erlon has joined #openstack-qa | 12:34 | |
*** jecarey has quit IRC | 12:36 | |
*** BharatK has quit IRC | 12:36 | |
openstackgerrit | Filip Hubík proposed openstack/tempest: Add keystone v2.0 and v3 api discovery checks https://review.openstack.org/162710 | 12:37 |
*** ihrachyshka has quit IRC | 12:46 | |
*** zz_dimtruck is now known as dimtruck | 12:46 | |
*** cdent has quit IRC | 12:54 | |
*** bknudson has joined #openstack-qa | 12:55 | |
*** salv-orlando has joined #openstack-qa | 12:56 | |
*** dimtruck is now known as zz_dimtruck | 12:56 | |
*** ihrachyshka has joined #openstack-qa | 12:58 | |
*** sks has quit IRC | 13:02 | |
*** neeti has quit IRC | 13:03 | |
*** lifeless has quit IRC | 13:05 | |
*** annashen has joined #openstack-qa | 13:08 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: redo devstack-gate grenade selection as a case statement https://review.openstack.org/175390 | 13:10 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo https://review.openstack.org/175391 | 13:10 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo https://review.openstack.org/175391 | 13:11 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: redo devstack-gate grenade selection as a case statement https://review.openstack.org/175390 | 13:12 |
*** annashen has quit IRC | 13:13 | |
*** sks has joined #openstack-qa | 13:16 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo https://review.openstack.org/175391 | 13:18 |
*** jecarey has joined #openstack-qa | 13:21 | |
*** rbak has quit IRC | 13:21 | |
*** bkopilov has quit IRC | 13:27 | |
*** zz_dimtruck is now known as dimtruck | 13:28 | |
*** artom has joined #openstack-qa | 13:28 | |
*** sslypushenko has quit IRC | 13:30 | |
*** superdan is now known as dansmith | 13:31 | |
*** bkopilov has joined #openstack-qa | 13:35 | |
openstackgerrit | Merged openstack-dev/devstack: refactor ping_check https://review.openstack.org/174370 | 13:39 |
*** bkopilov has quit IRC | 13:40 | |
*** bkopilov has joined #openstack-qa | 13:42 | |
*** dboik has joined #openstack-qa | 13:44 | |
*** dboik has quit IRC | 13:45 | |
*** dboik has joined #openstack-qa | 13:45 | |
*** mriedem has joined #openstack-qa | 13:46 | |
openstackgerrit | Sergey Slipushenko proposed openstack/tempest: Test UUID duplication fix https://review.openstack.org/165921 | 13:47 |
*** bkopilov has quit IRC | 13:49 | |
openstackgerrit | nithya-ganesan proposed openstack/tempest: Define validation_resources function for ssh validation https://review.openstack.org/149229 | 13:52 |
*** e0ne is now known as e0ne_ | 13:53 | |
SergeyLukjanov | sdague, we'd like to enable more testing jobs for sahara (for all repos) | 13:55 |
SergeyLukjanov | sdague, for sahara itself we'd like to add a few custom jobs that will be running tempest-lib based tests | 13:55 |
*** Abhishek has quit IRC | 13:56 | |
SergeyLukjanov | to ensure that integration with OpenStack and especially with Heat works well | 13:56 |
mtreinish | BobBall: with devstack gate there is a tempest-regex var you can pass in to tell testr which tests to run | 13:57 |
SergeyLukjanov | mtreinish, hey! | 13:57 |
mtreinish | SergeyLukjanov: hi | 13:58 |
sdague | SergeyLukjanov: ok, so what is the question :) | 13:58 |
SergeyLukjanov | sdague, mtreinish, so, due to the current intention to move projects tests from tempest, we'd like to add some additional tests to it and run them in different openstack topologies | 13:59 |
SergeyLukjanov | heat / no-heat and nova-net / neutron | 13:59 |
SergeyLukjanov | 4 dsvm jobs | 13:59 |
SergeyLukjanov | sdague, heh, I've missed the question :) | 13:59 |
SergeyLukjanov | the question is - is it ok? :) | 14:00 |
SergeyLukjanov | I think that the best case is to run *data_processing* tempest tests in this jobs and additional tests from sahara repo | 14:00 |
mtreinish | SergeyLukjanov: why wouldn't it be ok? :) you guys can run whatever testing you think is needed against sahara changes | 14:02 |
*** e0ne_ is now known as e0ne | 14:02 | |
SergeyLukjanov | mtreinish, sdague, I just want to be sure that you're aware and okay with it :) | 14:03 |
*** jgrimm is now known as zz_jgrimm | 14:04 | |
*** bkopilov has joined #openstack-qa | 14:04 | |
*** dboik_ has joined #openstack-qa | 14:06 | |
*** ajmiller has joined #openstack-qa | 14:08 | |
mtreinish | mriedem: is that test any different than the prexisting ports test? | 14:08 |
*** dboik has quit IRC | 14:08 | |
*** mtanino has joined #openstack-qa | 14:08 | |
*** sks has quit IRC | 14:08 | |
mriedem | mtreinish: not really, except it doesn't test the delete validation - and it's not skipped on an option | 14:09 |
BobBall | mtreinish: Is there a nice way to test the regexp in a dev environment? (i.e. list the tests that would match it) | 14:09 |
mriedem | mtreinish: i'm guessing something is working in kilo for ironic/nova in that hypervisor macs scenario that isn't working on juno | 14:09 |
*** mwagner_lap has joined #openstack-qa | 14:09 | |
mtreinish | BobBall: not right now, unless you do it manually before running the tests | 14:09 |
*** apevec has quit IRC | 14:09 | |
mtreinish | BobBall: but after reqs freeze ends I'll be moving tempest over to use os-testr which I added some stuff to do that for | 14:10 |
mtreinish | mriedem: hmm, ok. I know the ironic networking is kinda wonky so I'm not surprised it doesnt work | 14:10 |
mtreinish | mriedem: I'm just trying to weigh a revert vs a skipif | 14:10 |
BobBall | ok | 14:10 |
mriedem | mtreinish: i didn't dig into the history of the driver to see if there were changes in kilo that should be backported | 14:10 |
mtreinish | mriedem: tbh I don't care that much | 14:11 |
mriedem | mtreinish: i thought about skipping if baremetal=True, but the pxe job passes on kilo with that change | 14:11 |
mriedem | so it must be a bug in juno | 14:11 |
mriedem | maybe adam_g or devananda might know about that, idk | 14:11 |
mriedem | i see this in kilo in the macs_for_instance method which returns the hypervisor macs used in the neutronv2 API allocate_for_instance call that's failing: https://review.openstack.org/#/c/141188/ | 14:14 |
mriedem | but we probably aren't going to backport that to juno, and i don't know if it fixes the problem anyway | 14:14 |
*** sabeen1 has joined #openstack-qa | 14:17 | |
*** k4n0 has quit IRC | 14:17 | |
*** afazekas_ has joined #openstack-qa | 14:18 | |
mtreinish | mriedem: +A'd the revert | 14:18 |
mriedem | k | 14:18 |
mriedem | i don't know enough about the ironic code to know if there was a bug/race there in juno that's fixed in kilo | 14:19 |
mtreinish | there were some other issues I spotted with the test just from a quick glance. Like it leaks floating ips too | 14:19 |
mtreinish | mriedem: the majority of the neutron tests have to be skipped with ironic. irrc it's because everything is in one shared network | 14:20 |
mtreinish | although I'm not sure why creating a port before server creation would affect that | 14:20 |
mriedem | from what's failing in juno, when we go to allocate the network for the instance we have the list of macs from ironic for the instance and the requested pre-created port's mac isn't in that list so it blows up | 14:21 |
*** NobodyCam has left #openstack-qa | 14:21 | |
mriedem | that list of macs might have been empty? i'm not sure | 14:21 |
mriedem | could have been useful debugging when that fails, i.e. dump the list of macs that we do have | 14:22 |
mriedem | i'll throw up a patch for that | 14:22 |
*** Party_Bear is now known as BadCub | 14:24 | |
*** sputnik13 has quit IRC | 14:26 | |
*** afazekas_ has quit IRC | 14:26 | |
*** mvandijk_ has quit IRC | 14:26 | |
*** dboik_ has quit IRC | 14:27 | |
openstackgerrit | Merged openstack-dev/devstack: Catalog Index Service - glance devstack https://review.openstack.org/169200 | 14:27 |
*** dboik has joined #openstack-qa | 14:27 | |
*** mvandijk has joined #openstack-qa | 14:28 | |
openstackgerrit | Merged openstack-dev/devstack: Add logging config to cinder.conf https://review.openstack.org/175155 | 14:28 |
openstackgerrit | nithya-ganesan proposed openstack/tempest: Define validation_resources function for ssh validation https://review.openstack.org/149229 | 14:29 |
*** afazekas_ has joined #openstack-qa | 14:30 | |
*** dpaterson has joined #openstack-qa | 14:32 | |
*** lifeless has joined #openstack-qa | 14:32 | |
*** achanda has joined #openstack-qa | 14:32 | |
*** rbak has joined #openstack-qa | 14:33 | |
sdague | mtreinish: cinder encryption cli test expected to be failing - http://logs.openstack.org/90/175390/2/check/check-dg-tempest-dsvm-full/ce3d4a3/console.html#_2015-04-20_14_06_45_271 ? | 14:34 |
mriedem | sdague: i think that's old | 14:35 |
mriedem | http://status.openstack.org/elastic-recheck/index.html#1429151 | 14:35 |
sdague | mriedem: ok, I *just* hit it | 14:35 |
mriedem | been around since early march https://bugs.launchpad.net/cinder/+bug/1429151 | 14:35 |
openstack | Launchpad bug 1429151 in Cinder "SimpleReadOnlyCinderClientTest.test_cinder_encryption_type_list fails randomly" [High,Confirmed] | 14:35 |
sdague | ok | 14:35 |
*** afazekas_ has quit IRC | 14:36 | |
mtreinish | sdague: I didn't even know there was a cinder encryption cli test | 14:37 |
mriedem | mestery: salv-orlando: sc68cal: do you know if there is a neutron bug for this? http://logs.openstack.org/21/174821/1/gate/gate-neutron-dsvm-functional/eb6b441/console.html#_2015-04-20_07_41_14_649 | 14:37 |
mtreinish | sdague: but jgriffith should be removing all of those from tempest any day now | 14:37 |
mriedem | http://goo.gl/iw3Aah | 14:37 |
jgriffith | mtreinish: :) thanks for the Monday morning reminder :) | 14:37 |
mtreinish | jgriffith: heh, I wasn't going to miss that opportunity :) | 14:38 |
*** achanda has quit IRC | 14:39 | |
mtreinish | notmyname: do you have equiv tests for: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/cli/simple_read_only/object_storage/test_swift.py in swiftclient? | 14:42 |
mtreinish | those get run against a running swift deploy | 14:42 |
*** sslypushenko has joined #openstack-qa | 14:46 | |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for neutron functional test bug 1446261 https://review.openstack.org/175432 | 14:47 |
openstack | bug 1446261 in neutron "gate-neutron-dsvm-functional race fails HA/DVR tests with network namespace not found" [Undecided,New] https://launchpad.net/bugs/1446261 | 14:47 |
*** afazekas_ has joined #openstack-qa | 14:47 | |
*** jlanoux_ has quit IRC | 14:47 | |
*** hemnafk is now known as hemna | 14:50 | |
mestery | mtreinish: Looking, but I don't think so. I'll see if I can get carl_baldwin to look at that as well or even marun. | 14:50 |
mestery | mriedem: ^^^ | 14:50 |
mestery | mriedem: But I see you just proposed an ER for it, thanks! | 14:50 |
mriedem | yup | 14:51 |
mriedem | there was an eventlet timeout in that job last week which marun posted a longer timeout that seems to have staved that off a bit | 14:51 |
mriedem | i thought they were maybe related but doesn't look like it | 14:51 |
mestery | mriedem: Ack | 14:52 |
*** nmagnezi_ has quit IRC | 14:52 | |
*** tosky has quit IRC | 14:53 | |
*** cdent has joined #openstack-qa | 14:53 | |
*** tosky has joined #openstack-qa | 14:53 | |
mriedem | mestery: looks like that started showing up around 4/15 | 14:55 |
mestery | mriedem: Cool, thanks for that! | 14:56 |
*** anshul has quit IRC | 14:56 | |
sdague | mriedem: your tempest revert just bounced on uit tests | 14:56 |
*** itzikb has quit IRC | 14:56 | |
mriedem | fun | 14:57 |
mtreinish | sdague: link? I hope we didn't introduce a new unit test race recently... | 14:58 |
mtreinish | although if we did I have a feeling it's my fault | 14:58 |
mriedem | mtreinish: https://jenkins03.openstack.org/job/gate-tempest-python27/3203/console | 14:58 |
mriedem | looks unrelated | 14:58 |
*** tsekiyama has joined #openstack-qa | 14:58 | |
mriedem | tempest.tests.common.test_accounts.TestAccount.test__get_creds_by_roles_no_admin | 14:59 |
mtreinish | yeah that's my fault I think | 14:59 |
mriedem | way2go | 14:59 |
mtreinish | the test is pulling the "wrong" account from the fake list, because I wrote it assuming it would always be in order | 15:00 |
mriedem | just racey or blocking? | 15:00 |
mtreinish | racey | 15:01 |
devananda | mriedem: g'morning | 15:01 |
mtreinish | one of the valid fake accounts has a network. But that test doesn't mock the network api calls | 15:01 |
*** BharatK has joined #openstack-qa | 15:01 | |
mriedem | devananda: we're reverting a tempest test which breaks stable/juno and the ironic sideways job https://review.openstack.org/#/c/175219/ | 15:02 |
*** rwsu has joined #openstack-qa | 15:02 | |
mtreinish | we can fix it by either adding the mock or just removing the networks from the class level list and just append it for the networks test | 15:02 |
mriedem | b/c when allocating the network for the instance, the pre-created port on the boot request has a mac which isn't in the list of available macs from the ironic driver | 15:02 |
mriedem | devananda: i wasn't sure if there was a fix in kilo for that, in ironic | 15:02 |
devananda | reading backscroll, but I don't know what tests you're referring to, or what revert.... | 15:02 |
devananda | oh. helps if I scroll *down* too :) | 15:02 |
mriedem | or if i summarize :) | 15:03 |
*** marun has joined #openstack-qa | 15:03 | |
notmyname | mtreinish: I'm not sure. I'm still looking at those to see what they do. looks like they run a command and look for keywords in the output? | 15:04 |
devananda | reading the revert patch ... | 15:04 |
notmyname | mtreinish: also, why do you ask? ;-) | 15:04 |
*** zz_jgrimm is now known as jgrimm | 15:04 | |
*** achanda has joined #openstack-qa | 15:05 | |
mtreinish | notmyname: yeah they just do read only operation cli calls and check the response is sane | 15:05 |
mtreinish | notmyname: well I've been trying to remove the cli tests from tempest and into the client repos all cycle | 15:05 |
mtreinish | and swift, heat, and cinder are the last 3 | 15:05 |
mtreinish | but if you guys have equiv coverage I can just delete the swift tests | 15:06 |
notmyname | mtreinish: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/cli/simple_read_only/object_storage/test_swift.py#n53 isn't really a "test" is it? ;-) | 15:06 |
*** achanda has quit IRC | 15:07 | |
mtreinish | notmyname: so you joke but the cli tests were added to tempest because things like that used to stack trace some of the clients | 15:07 |
mtreinish | which is actually hysterical, or sad depending on your perspective I guess :) | 15:08 |
*** afazekas_ has quit IRC | 15:08 | |
mtreinish | but, yeah test is a bit of a strectch for that one :) | 15:10 |
*** annashen has joined #openstack-qa | 15:10 | |
*** beekneemech is now known as bnemec | 15:10 | |
*** e0ne is now known as e0ne_ | 15:10 | |
notmyname | mtreinish: I think we're good. looking through the pythn-swiftclient tests, we've got coverage for that | 15:11 |
notmyname | mtreinish: it's not 100% identical, but the functionality looks to be covered | 15:12 |
mtreinish | notmyname: ok cool, do you want to push a patch to delete those tests from tempest then :) | 15:12 |
mtreinish | notmyname: also fwiw, if you wanted to move the tests over it's pretty simple: http://lists.openstack.org/pipermail/openstack-dev/2015-February/057458.html | 15:12 |
notmyname | mtreinish: test_swift_region_list is probably the only one that isn't specifically covered.... let me check something else | 15:12 |
devananda | mriedem: still trying to wrap my brain around how this test is different from those that call _setup_network_and_servers | 15:14 |
*** annashen has quit IRC | 15:15 | |
notmyname | mtreinish: ok, we're good there, too, I think | 15:15 |
notmyname | mtreinish: no, I don't want to do that patch, but I'll +1 it if you give me a link to it | 15:15 |
mtreinish | devananda: apparently it assigns a different sec group, but yeah I didn't see any difference | 15:15 |
devananda | mriedem: when you say "a fix for that" I'm not sure what "that" is -- but off hand, I dont know why this would be passing in kilo when it's not in juno. in part b/c I'm not sure how its different from other tests | 15:15 |
openstackgerrit | gordon chung proposed openstack-dev/devstack: ceilometer: add tempest option to test events https://review.openstack.org/164860 | 15:15 |
mtreinish | devananda: arent the other tests skipped if ironic is enabled? | 15:15 |
devananda | some, not all | 15:15 |
devananda | eg, test_network_basic_ops is not skipped | 15:16 |
mriedem | devananda: that being the list of hypervisor macs that are coming back from ironic | 15:16 |
sdague | mtanino: I'll push the patch | 15:16 |
mtanino | sdague: hi | 15:16 |
*** e0ne_ is now known as e0ne | 15:17 | |
mriedem | does neutron need to be configured differently somehow for ironic compute? if so, maybe there is a change in devstack on kilo that didn't make it to juno? | 15:17 |
sdague | mtanino: sorry, wrong complete | 15:17 |
mtanino | gocha! | 15:17 |
mtreinish | notmyname: ok, I can push it then :) Do you have a link to where the swift equiv tests are? | 15:17 |
mtreinish | sdague: heh too many m's | 15:17 |
openstackgerrit | Sean Dague proposed openstack/tempest: remove swift cli tests https://review.openstack.org/175447 | 15:17 |
devananda | mriedem: possibly? also, creating the port from a different secgroup could be it | 15:18 |
openstackrecheck | openstack/nova change: https://review.openstack.org/163378 failed because of: gate-grenade-dsvm-ironic-sideways: https://bugs.launchpad.net/bugs/1445917 | 15:18 |
openstack | Launchpad bug 1445917 in tempest "(juno) gate-grenade-dsvm-ironic-sideways failing with NoValidHost due to PortNotUsable" [High,In progress] - Assigned to Matt Riedemann (mriedem) | 15:18 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/175219 failed gate-tempest-python27 in the gate queue with an unrecognized error | 15:18 |
devananda | speaking of ... | 15:19 |
notmyname | mtreinish: a combination of http://git.openstack.org/cgit/openstack/python-swiftclient/tree/tests/unit/test_shell.py and http://git.openstack.org/cgit/openstack/python-swiftclient/tree/tests/functional/test_swiftclient.py | 15:19 |
*** afazekas_ has joined #openstack-qa | 15:21 | |
mtreinish | notmyname: cool, thanks | 15:21 |
*** armax_ has joined #openstack-qa | 15:22 | |
*** weshay has quit IRC | 15:23 | |
*** weshay has joined #openstack-qa | 15:24 | |
*** rfolco has quit IRC | 15:24 | |
*** rfolco has joined #openstack-qa | 15:25 | |
*** rfolco has quit IRC | 15:26 | |
*** afazekas_ has quit IRC | 15:26 | |
*** ihrachyshka has quit IRC | 15:27 | |
*** rfolco has joined #openstack-qa | 15:27 | |
*** safchain has quit IRC | 15:28 | |
*** dkranz has quit IRC | 15:30 | |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for neutron functional test bug 1446261 https://review.openstack.org/175432 | 15:30 |
openstack | bug 1446261 in neutron "gate-neutron-dsvm-functional race fails HA/DVR tests with network namespace not found" [High,Confirmed] https://launchpad.net/bugs/1446261 - Assigned to Maru Newby (maru) | 15:30 |
*** _amrith_ is now known as amrith | 15:31 | |
*** SridharG has left #openstack-qa | 15:33 | |
*** dratushnyy has quit IRC | 15:35 | |
*** dkranz has joined #openstack-qa | 15:37 | |
*** afazekas_ has joined #openstack-qa | 15:37 | |
mtreinish | mriedem: did you file a bug for the tempest unit test race? | 15:39 |
mriedem | mtreinish: nope | 15:39 |
mriedem | i've hit my bug reporting quota for the morning | 15:39 |
mtreinish | ok | 15:39 |
mtreinish | I've got the fix, I just wanted to make sure I didn't miss a bug number in the commit msg | 15:40 |
mriedem | i would -1 that so hard | 15:40 |
mriedem | devananda: the difference in that failing test is i guess the port is created with a specific security group https://review.openstack.org/#/c/162886/ | 15:41 |
mriedem | maybe that's what causes ironic to fail on juno | 15:41 |
*** safchain has joined #openstack-qa | 15:41 | |
*** annashen has joined #openstack-qa | 15:41 | |
*** Swami has joined #openstack-qa | 15:42 | |
mriedem | heh, the test isn't run in the ironic job http://logs.openstack.org/86/162886/8/check/check-tempest-dsvm-ironic-pxe_ssh/ffd43e7/console.html.gz | 15:42 |
mriedem | only the baremetal tests are run there | 15:42 |
mriedem | mtreinish: so i guess we could have just skipped if baremetal | 15:42 |
openstackgerrit | Sean Dague proposed openstack/tempest: remove swift cli tests https://review.openstack.org/175447 | 15:43 |
*** amrith has left #openstack-qa | 15:44 | |
mtreinish | mriedem: sure, I figured that was the case | 15:44 |
mtreinish | but I was fine with reverting and bringing it back later | 15:44 |
mriedem | yeah | 15:45 |
mriedem | you said it needed more work anyway | 15:45 |
*** yfried__ has quit IRC | 15:45 | |
mtreinish | yep | 15:46 |
sdague | dtroyer: so.... devstack branching? | 15:46 |
*** dboik_ has joined #openstack-qa | 15:47 | |
*** afazekas_ has quit IRC | 15:47 | |
*** dboik has quit IRC | 15:50 | |
*** mlavalle has joined #openstack-qa | 15:50 | |
*** rdekel has quit IRC | 15:52 | |
*** jhenner has quit IRC | 15:57 | |
*** ihrachyshka has joined #openstack-qa | 15:58 | |
*** sergsh has quit IRC | 15:59 | |
*** BharatK has quit IRC | 16:00 | |
*** jhenner has joined #openstack-qa | 16:00 | |
*** Longgeek has quit IRC | 16:00 | |
*** sabeen1 has quit IRC | 16:02 | |
dtroyer | sdague: catching up, was in a Go-like world all weekend making up for a two week lack of coding ;) back to Earth… | 16:03 |
*** sputnik13 has joined #openstack-qa | 16:04 | |
sdague | heh | 16:04 |
*** weshay has quit IRC | 16:05 | |
*** jhenner has quit IRC | 16:05 | |
*** tkammer has quit IRC | 16:06 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Fix test accounts unit test race with networks https://review.openstack.org/175470 | 16:09 |
mtreinish | mriedem: ^^^ that's the unit test fix | 16:09 |
*** jhenner has joined #openstack-qa | 16:09 | |
dtroyer | sdague: the only outstanding change that strike me as worth waiting to branch on is ianw's peak mem tracker (https://review.openstack.org/171919). It's easy enough to backport but its also a tweak away from being approved… | 16:17 |
sdague | dtroyer: you mean just the enable_service bit? | 16:22 |
sdague | I can respin and we can merge that if you like | 16:22 |
openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for neutron functional test bug 1446261 https://review.openstack.org/175432 | 16:22 |
openstack | bug 1446261 in neutron "gate-neutron-dsvm-functional race fails HA/DVR tests with network namespace not found" [High,Confirmed] https://launchpad.net/bugs/1446261 - Assigned to Maru Newby (maru) | 16:22 |
dtroyer | sdague: yes | 16:22 |
sdague | dtroyer: you want it on by default, or off by default? | 16:24 |
dtroyer | sdague: no opinion, I'm guesing ianw wants it on in the gate? or did he intend this for debugging only? | 16:24 |
sdague | he wants it on in the gate, which I think means default off is probably right, and turn it on in d-g | 16:25 |
sdague | he had some concerns around memory exhaustion | 16:25 |
dtroyer | sounds good to me | 16:25 |
mtreinish | sdague: yeah the centos7 jobs were constantly fighting the oomkiller | 16:26 |
mtreinish | I think | 16:26 |
openstackgerrit | Sean Dague proposed openstack-dev/devstack: Add a peak memory tracker to dstat https://review.openstack.org/171919 | 16:27 |
sdague | dtroyer: ok, if that looks good, lets land it | 16:27 |
*** annashen has quit IRC | 16:27 | |
dtroyer | +2, thanks | 16:29 |
sdague | you want to just +A it in? | 16:29 |
sdague | also, I never remember who can actually make devstack branches, or how we do it correctly | 16:30 |
dtroyer | ok | 16:30 |
*** dboik_ has quit IRC | 16:30 | |
*** dboik has joined #openstack-qa | 16:30 | |
dtroyer | I've always had infra or maybe ttx do the branch | 16:31 |
dtroyer | I think we can only push tags | 16:31 |
sdague | fungi: ? | 16:31 |
clarkb | ya looks like openstack release has perms from all-projects then any gerrit admin can also do it | 16:31 |
fungi | looking at how it's configured in the acl | 16:31 |
fungi | right, that | 16:31 |
clarkb | and the info needed are branch name and commit to base it on. I assume stable/kilo and whatever HEAD currently is? | 16:32 |
*** dboik has quit IRC | 16:33 | |
dtroyer | HEAD after https://review.openstack.org/171919 and https://review.openstack.org/168796 merge | 16:33 |
*** dboik has joined #openstack-qa | 16:33 | |
sdague | so, we probably could tag grenade first then, right? | 16:37 |
sdague | because we have a commit there that's solid | 16:37 |
sdague | grenade would be stable/kilo at 1a17e0c14d90c5df3a0237356871ab6aa73a374a | 16:38 |
*** safchain has quit IRC | 16:40 | |
dtroyer | https://review.openstack.org/#/c/172156/ is ~15 minutes from merging… | 16:40 |
dtroyer | otherwise, yeah, that'll work | 16:41 |
*** Swami has quit IRC | 16:41 | |
sdague | oh, yeh, typo | 16:41 |
sdague | yeh, lets let the typo fix land | 16:42 |
*** luqas has quit IRC | 16:43 | |
sdague | clarkb: you want to look at - https://review.openstack.org/#/c/175390/ in the mean time, which is a clean up on d-g before adding the kilo branch support | 16:43 |
clarkb | sdague: sure | 16:43 |
*** armax_ has left #openstack-qa | 16:46 | |
*** armax_ has joined #openstack-qa | 16:47 | |
*** Nic has joined #openstack-qa | 16:47 | |
*** Swami has joined #openstack-qa | 16:48 | |
*** nmagnezi_ has joined #openstack-qa | 16:51 | |
*** annashen has joined #openstack-qa | 16:51 | |
clarkb | sdague: can you see my comment on 175390? | 16:53 |
clarkb | I don't think its worth a -1 but wanted to make sure that we are doing that intentionally and if so consider simplifying it | 16:53 |
sdague | oh, hmmm | 16:54 |
sdague | yeh, I was trying to be clever, and clearly failed | 16:54 |
sdague | it actually does all the wrong things | 16:54 |
sdague | ok, let me go look at that again | 16:55 |
sdague | that's a better catch than I think you originally though | 16:55 |
clarkb | could be :) | 16:55 |
sdague | because it actually means partial cpu nova doesn't any more | 16:55 |
clarkb | oh right because the variable is overridden | 16:56 |
clarkb | so it is worth a -1 :) | 16:56 |
clarkb | will update | 16:56 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo https://review.openstack.org/175391 | 16:57 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: redo devstack-gate grenade selection as a case statement https://review.openstack.org/175390 | 16:57 |
sdague | ok, fixed | 16:58 |
sdague | clarkb: thanks for the find | 16:58 |
*** suneelb has joined #openstack-qa | 17:02 | |
*** dboik has quit IRC | 17:02 | |
*** dboik has joined #openstack-qa | 17:02 | |
clarkb | sdague: we should add a third change that updates features.yaml for liberty | 17:02 |
clarkb | sdague: just to get that out of the way while we have it | 17:03 |
clarkb | *have it on our minds | 17:03 |
*** dboik has quit IRC | 17:03 | |
*** BobBall has left #openstack-qa | 17:03 | |
openstackgerrit | Merged openstack-dev/grenade: Fix typos in PLUGINS https://review.openstack.org/172156 | 17:03 |
sdague | clarkb: sure, should that be in patch #2 as well? | 17:03 |
*** dboik has joined #openstack-qa | 17:03 | |
*** armax_ has quit IRC | 17:03 | |
clarkb | sdague: I think it either go there or in a separate change, either way is fine | 17:03 |
*** armax_ has joined #openstack-qa | 17:04 | |
sdague | oh, yeh, because we don't actually need that for a while, right | 17:04 |
sdague | as kilo is already called out | 17:04 |
clarkb | yup | 17:04 |
clarkb | but last time I did kilo stuff when making juno go and I think it has massively simplified the changes you ened to make today for kilo to work | 17:05 |
clarkb | so its a nice prep step | 17:05 |
sdague | ok, I'll do it as patch #3 then | 17:05 |
*** annashen has quit IRC | 17:06 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add liberty to feature grid https://review.openstack.org/175493 | 17:07 |
*** cody-somerville has joined #openstack-qa | 17:07 | |
*** dboik has quit IRC | 17:07 | |
*** dboik has joined #openstack-qa | 17:07 | |
*** mpaolino has quit IRC | 17:08 | |
*** jhenner has quit IRC | 17:08 | |
*** patrickeast has joined #openstack-qa | 17:10 | |
openstackgerrit | Matt Riedemann proposed openstack/tempest: Fix test accounts unit test race with networks https://review.openstack.org/175470 | 17:11 |
mriedem | sdague: mtreinish: ^ added bug line | 17:11 |
sdague | mriedem: thanks | 17:12 |
*** annashen has joined #openstack-qa | 17:12 | |
openstackgerrit | Merged openstack-dev/devstack: Clone external plugins before overriding defaults https://review.openstack.org/168796 | 17:12 |
clarkb | thats one | 17:13 |
*** openstackrecheck has quit IRC | 17:13 | |
*** openstackrecheck has joined #openstack-qa | 17:13 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/elastic-recheck: Add query for bug 1446309 https://review.openstack.org/175496 | 17:13 |
openstack | bug 1446309 in tempest "tempest.tests.common.test_accounts.TestAccount.test__get_creds_by_roles_no_admin race fails" [High,In progress] https://launchpad.net/bugs/1446309 - Assigned to Matthew Treinish (treinish) | 17:13 |
mtreinish | mriedem: ^^^ | 17:13 |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for tempest unit test bug 1446309 https://review.openstack.org/175497 | 17:14 |
mriedem | mtreinish: ^^^ | 17:14 |
mtreinish | mriedem: hehe, I win :) | 17:14 |
mtreinish | finally | 17:14 |
mriedem | what?! | 17:14 |
mriedem | gdi | 17:14 |
mriedem | oh you could totally add the build_name! | 17:15 |
mriedem | -3! | 17:15 |
mriedem | ha, co-authored-by | 17:15 |
mriedem | that's cute | 17:15 |
*** jhenner has joined #openstack-qa | 17:15 | |
mtreinish | well I stole your message line from the link in the review | 17:15 |
mtreinish | figured that was worth something | 17:15 |
mriedem | yeah | 17:15 |
*** jhenner has quit IRC | 17:16 | |
*** jhenner has joined #openstack-qa | 17:16 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/elastic-recheck: Add query for bug 1446309 https://review.openstack.org/175496 | 17:17 |
openstack | bug 1446309 in tempest "tempest.tests.common.test_accounts.TestAccount.test__get_creds_by_roles_no_admin race fails" [High,In progress] https://launchpad.net/bugs/1446309 - Assigned to Matthew Treinish (treinish) | 17:17 |
mtreinish | mriedem: ^^^ build name added | 17:17 |
mriedem | sheesh, beat me twice | 17:17 |
mriedem | i was just updating your change locally | 17:17 |
*** davideagnello has joined #openstack-qa | 17:20 | |
*** achanda has joined #openstack-qa | 17:24 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo https://review.openstack.org/175391 | 17:25 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: redo devstack-gate grenade selection as a case statement https://review.openstack.org/175390 | 17:25 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add liberty to feature grid https://review.openstack.org/175493 | 17:25 |
*** cody-somerville has quit IRC | 17:34 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add configuration guide sections for more required options https://review.openstack.org/174047 | 17:34 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add section to config guide for setting up services https://review.openstack.org/174149 | 17:34 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Start a tempest config guide networking section https://review.openstack.org/172507 | 17:34 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add config guide section on service feature enabled sections https://review.openstack.org/174086 | 17:34 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add a paragraph on the use of tempest_roles option https://review.openstack.org/174675 | 17:34 |
mriedem | that revert failed in the gate again | 17:37 |
mriedem | right at the end | 17:37 |
mriedem | my old friend test_volume_boot_pattern | 17:37 |
*** dboik has quit IRC | 17:37 | |
mtreinish | mriedem: was it the generic ssh failure or a different thing this time? | 17:37 |
mriedem | waiting on logs | 17:38 |
*** dboik has joined #openstack-qa | 17:38 | |
mriedem | console log looks like generic ssh failure | 17:38 |
*** melwitt has joined #openstack-qa | 17:38 | |
*** harlowja_away is now known as harlowja | 17:38 | |
mtreinish | mriedem: neutron or n-net? | 17:40 |
mriedem | neutron | 17:40 |
clarkb | sdague: I approved the first d-g change, let me know when we can start tagging devstack and grenade so we can approve the second change | 17:40 |
clarkb | er s/tagging/branching/ | 17:40 |
*** dboik has quit IRC | 17:40 | |
sdague | clarkb: will do | 17:40 |
mtreinish | mriedem: heh, figures... | 17:40 |
sdague | there are still some devstack patches in flight we want | 17:40 |
*** dboik has joined #openstack-qa | 17:41 | |
*** dustins has joined #openstack-qa | 17:41 | |
mtreinish | sdague: if you're going through the branching motions can you make sure the steps are all documented on: https://wiki.openstack.org/wiki/QA/releases | 17:41 |
openstackgerrit | Morgan Fainberg proposed openstack-dev/devstack: Add response time to keystone access log https://review.openstack.org/164042 | 17:43 |
*** artom has quit IRC | 17:43 | |
*** artom has joined #openstack-qa | 17:43 | |
*** luqas has joined #openstack-qa | 17:44 | |
*** artom has quit IRC | 17:44 | |
*** artom has joined #openstack-qa | 17:44 | |
*** yfried__ has joined #openstack-qa | 17:48 | |
*** luqas has quit IRC | 17:48 | |
*** Nic has quit IRC | 17:49 | |
*** Nic1 has joined #openstack-qa | 17:49 | |
*** jordanP has quit IRC | 17:50 | |
*** e0ne has quit IRC | 17:51 | |
openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for bug 1446309 https://review.openstack.org/175496 | 17:55 |
openstack | bug 1446309 in tempest "tempest.tests.common.test_accounts.TestAccount.test__get_creds_by_roles_no_admin race fails" [High,In progress] https://launchpad.net/bugs/1446309 - Assigned to Matthew Treinish (treinish) | 17:55 |
*** openstackrecheck has quit IRC | 17:59 | |
*** openstackrecheck has joined #openstack-qa | 18:00 | |
*** dboik_ has joined #openstack-qa | 18:01 | |
*** shashankhegde has joined #openstack-qa | 18:03 | |
*** tosky has quit IRC | 18:04 | |
*** dboik has quit IRC | 18:05 | |
*** artom_ has joined #openstack-qa | 18:08 | |
*** artom has quit IRC | 18:08 | |
*** artom_ has quit IRC | 18:10 | |
*** artom has joined #openstack-qa | 18:10 | |
mriedem | sdague: mtreinish: fyi i'm going through and deleting old e-r queries for closed bugs | 18:13 |
mtreinish | mriedem: I noticed, I was getting lots of emails about activity on a bunch of bugs | 18:14 |
mriedem | yeah lots of stale gorp in LP | 18:14 |
mriedem | dtroyer: does anything need to be done with this? https://bugs.launchpad.net/openstack-ci/+bug/1282772 | 18:17 |
openstack | Launchpad bug 1282772 in grenade "Grenade job fails when trying to add a window to screen" [Low,Confirmed] | 18:17 |
sdague | mriedem: no, that code should all be gone now | 18:19 |
*** dboik_ has quit IRC | 18:19 | |
mriedem | sdague: ok | 18:19 |
mriedem | btw, i'm assuming the LP API tells you duplicates for a given bug? might be nice to show duplicate bugs in the e-r index page | 18:20 |
*** ihrachyshka has quit IRC | 18:20 | |
mriedem | since LP doesn't close/change status on duplicates when the main bug is fixed | 18:20 |
mtreinish | mriedem: it can actually be tricky to figure it out from the api iirc | 18:21 |
mtreinish | mriedem: http://git.openstack.org/cgit/openstack/tempest-lib/tree/tempest_lib/cmd/skip_tracker.py#n110 | 18:21 |
mriedem | were heat tests disabled in stable/icehouse | 18:24 |
mriedem | ? | 18:24 |
mriedem | or am i dreaming that? | 18:24 |
mtreinish | mriedem: most were | 18:24 |
mtreinish | the dedicated heat slow job was just deleted on icehouse | 18:24 |
mtreinish | there are still a few api tests which weren't tagged as slow, so I think those might still run | 18:24 |
sdague | clarkb / fungi: so grenade should be good to go with stable/kilo at 67834d825aa0170c8801b55473fc4ef1392e6128 | 18:29 |
*** e0ne has joined #openstack-qa | 18:29 | |
sdague | the last devstack patch is in the gate | 18:29 |
sdague | so 37 minutes from us having that ready to tag | 18:30 |
sdague | to branch | 18:30 |
clarkb | I am branching grenade now | 18:30 |
*** suneelb has quit IRC | 18:30 | |
clarkb | done | 18:30 |
*** suneelb has joined #openstack-qa | 18:30 | |
*** suneelb has quit IRC | 18:30 | |
*** suneelb1 has joined #openstack-qa | 18:31 | |
*** dboik has joined #openstack-qa | 18:33 | |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Remove old queries for "fixed" bugs https://review.openstack.org/175529 | 18:36 |
*** annashen has quit IRC | 18:37 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: only force off KEYSTONE_USE_WSGI up until kilo https://review.openstack.org/175531 | 18:43 |
*** nmagnezi_ has quit IRC | 18:43 | |
openstackgerrit | Sean Dague proposed openstack-dev/grenade: adjust branches for kilo https://review.openstack.org/175533 | 18:44 |
*** e0ne is now known as e0ne_ | 18:46 | |
sdague | ok, so 20 minutes until the devstack patch lands, I'm going to step away from the desk for a bit. Back later to hopefully get all these branches done | 18:47 |
*** suneelb has joined #openstack-qa | 18:47 | |
*** suneelb1 has quit IRC | 18:48 | |
*** dboik has quit IRC | 18:48 | |
*** annashen has joined #openstack-qa | 18:50 | |
openstackgerrit | David Kranz proposed openstack/tempest: Start a tempest config guide networking section https://review.openstack.org/172507 | 18:54 |
*** e0ne_ has quit IRC | 18:54 | |
*** andreykurilin__ has joined #openstack-qa | 18:55 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add configuration guide sections for more required options https://review.openstack.org/174047 | 18:55 |
*** dboik has joined #openstack-qa | 18:57 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add config guide section on service feature enabled sections https://review.openstack.org/174086 | 18:58 |
*** e0ne has joined #openstack-qa | 18:58 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add section to config guide for setting up services https://review.openstack.org/174149 | 18:58 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add a paragraph on the use of tempest_roles option https://review.openstack.org/174675 | 18:58 |
openstackgerrit | Merged openstack/tempest: Revert "Test creation of server attached to created port" https://review.openstack.org/175219 | 18:58 |
mtreinish | mriedem: ^^^ | 18:58 |
mriedem | finally | 18:58 |
*** annashen has quit IRC | 19:03 | |
*** annashen has joined #openstack-qa | 19:03 | |
*** dboik_ has joined #openstack-qa | 19:13 | |
*** dboik has quit IRC | 19:13 | |
*** yfried__ is now known as yfried|afk | 19:14 | |
openstackgerrit | Merged openstack-dev/devstack: Add a peak memory tracker to dstat https://review.openstack.org/171919 | 19:17 |
*** johnthetubaguy is now known as zz_johnthetubagu | 19:19 | |
*** yfried|afk is now known as yfried__ | 19:22 | |
*** shashankhegde has quit IRC | 19:24 | |
*** aix has quit IRC | 19:29 | |
*** ajmiller_ has joined #openstack-qa | 19:29 | |
*** ajmiller has quit IRC | 19:33 | |
*** ajmiller_ is now known as ajmiller | 19:34 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest-lib: Migrate ssh.py from tempest https://review.openstack.org/166518 | 19:37 |
*** nmagnezi_ has joined #openstack-qa | 19:41 | |
sdague | clarkb: for devstack, stable/kilo should be 14d1cf90ed9992f706c9149ba46e2eb984ff8b58 | 19:42 |
clarkb | sdague: ok making branch now | 19:43 |
clarkb | sdague: done | 19:43 |
clarkb | sdague: I think you can self approve the d-g changes now since they should have all the necessary +2s | 19:43 |
sdague | clarkb: ok, cool. Do we need project-config definitions ? | 19:44 |
clarkb | sdague: not for grenade, but for general periodic testing, tempest/d-g tests on older branches, etc we do | 19:46 |
*** luqas has joined #openstack-qa | 19:46 | |
sdague | hmmm... keystone-all is not shutting down correctly all the time | 19:49 |
*** suneelb has quit IRC | 19:49 | |
*** suneelb1 has joined #openstack-qa | 19:50 | |
*** luqas has quit IRC | 19:51 | |
*** suneelb has joined #openstack-qa | 19:53 | |
*** suneelb1 has quit IRC | 19:54 | |
*** shashankhegde has joined #openstack-qa | 19:55 | |
*** liusheng has quit IRC | 19:58 | |
mtreinish | sdague: light the morganfainberg signal? | 19:58 |
*** liusheng has joined #openstack-qa | 19:59 | |
morganfainberg | hmm? | 19:59 |
*** andreykurilin__ has quit IRC | 19:59 | |
*** Rockyg has joined #openstack-qa | 19:59 | |
morganfainberg | mtreinish, i should really stop lurking in channels i know people say my name in :P | 19:59 |
morganfainberg | mtreinish, whats up? | 19:59 |
mtreinish | morganfainberg: heh, ~10mins ago sdague said: "hmmm... keystone-all is not shutting down correctly all the time" | 20:00 |
*** andreykurilin__ has joined #openstack-qa | 20:00 | |
morganfainberg | mtreinish, ah missed that in this window... | 20:00 |
morganfainberg | got disconnected when replaying the backlog | 20:00 |
* morganfainberg can't wait for keystone-all to die... | 20:01 | |
morganfainberg | die die die die die, dead dead, eventlet no more in keystone | 20:01 |
mtreinish | morganfainberg: is that this cycle? or +1 | 20:01 |
mtreinish | this cycle being L I guess :) | 20:02 |
morganfainberg | mtreinish, M | 20:02 |
morganfainberg | mtreinish, :( | 20:02 |
morganfainberg | deprecated in Kilo | 20:02 |
morganfainberg | can't remove it until M ... i want ti dead in liberty though | 20:02 |
morganfainberg | but..... | 20:02 |
morganfainberg | alas | 20:02 |
mtreinish | well we can at least remove it from all the test jobs if you want | 20:02 |
*** annashen has quit IRC | 20:02 | |
mtreinish | that's something worthwhile I think | 20:02 |
morganfainberg | there is only 1 job using it | 20:03 |
morganfainberg | and until it dies | 20:03 |
mtreinish | postgres | 20:03 |
morganfainberg | i don't want to risk breaking people | 20:03 |
morganfainberg | because we suck at something and refactor things in a way that really make it not work. | 20:03 |
mtreinish | morganfainberg: sure add an eventlet job to the keystone queue, but we can drop it everywhere else | 20:03 |
morganfainberg | we did postgres because it was low cost to do so | 20:04 |
morganfainberg | and didn't consume extra workers | 20:04 |
sdague | morganfainberg: well, someone needs to actually work on wsgi upgrade | 20:04 |
morganfainberg | sdague, yes we do. | 20:04 |
sdague | which, isn't a thing | 20:04 |
morganfainberg | sdague, thankfully it really is pretty straight forward. | 20:04 |
sdague | an eventlet -> wsgi upgrade would be useful to do | 20:04 |
sdague | morganfainberg: ok, please write the tests for it :) | 20:05 |
morganfainberg | sdague, i can verbalize it, i just need to learn grenade-isms to do it | 20:05 |
mtreinish | morganfainberg: yeah I realize that, but I'm just saying we could be a bit smarter about how we split things up. Because keystone doesn't need a neutron job, but you need an eventlet job. | 20:05 |
sdague | well, I just made the grenadisms hopefully a lot easier | 20:05 |
mtreinish | morganfainberg: and no one else needs an evenlet job | 20:05 |
morganfainberg | sdague, yay! | 20:05 |
clarkb | mtreinish: eh sort of | 20:05 |
clarkb | mtreinish: we have had not keystone make keystone eventlet pretty terrible | 20:06 |
clarkb | mtreinish: basically performance gets really bad and tests take twice as long | 20:06 |
morganfainberg | mtreinish, sure. i'm happy to drop tests that don't benefit people, but right now i'm letting the smart infra folks tell me when we should evaluate that stuff. | 20:06 |
morganfainberg | infra/test/etc | 20:06 |
clarkb | and there is zero cost in using it in the postgres job | 20:06 |
morganfainberg | meaning, i am staying out of sdague and jogo's ways | 20:06 |
morganfainberg | clarkb, exactly | 20:07 |
morganfainberg | clarkb, zero cost is the kind of cost i like | 20:07 |
mtreinish | clarkb: well if we're killing keystone eventlet who cares about performance, and back then it wasn't even running on keystone changes iirc | 20:07 |
morganfainberg | mtreinish, we care until it's dead. | 20:07 |
morganfainberg | mtreinish, it is a valid deployment methodology | 20:07 |
sdague | clarkb: there is complexity cost in so few people knowning what's buried in that pg job | 20:08 |
mtreinish | and if we want to keep it around in the postgres jobs maybe rename it postgres-keystone_eventlet-metadata | 20:08 |
mtreinish | because that's really what it is | 20:08 |
morganfainberg | but in general, i'm fine with shuffling around the tests keystone deals with | 20:08 |
mtreinish | and tbh postgres is the one thing no one actually cares about in that list | 20:08 |
sdague | so when stuff goes wrong, it's a whole lot of ugly stepchild configs | 20:08 |
clarkb | mtreinish: sure I don't mind renaming it | 20:08 |
morganfainberg | if it means pgsql becomes something else - and we grow an eventlet specific job, i'm cool with that | 20:08 |
clarkb | sdague: yup that is an issue | 20:09 |
* morganfainberg again defers to smarter folks. | 20:09 | |
morganfainberg | notably i'll defer to infra types so we don't consume un-needed resources | 20:09 |
morganfainberg | as long as the coverage is there, i'm content - how we get the coverage doesn't really matter to me. | 20:10 |
*** jogo has joined #openstack-qa | 20:15 | |
*** andreykurilin__ has quit IRC | 20:16 | |
*** andreykurilin__ has joined #openstack-qa | 20:17 | |
*** yfried__ is now known as yfried|afk | 20:20 | |
*** achanda has quit IRC | 20:22 | |
*** dpaterson has quit IRC | 20:22 | |
*** dpaterson has joined #openstack-qa | 20:23 | |
*** andreykurilin__ has quit IRC | 20:24 | |
*** shardy is now known as shardy_z | 20:26 | |
openstackrecheck | openstack/cinder change: https://review.openstack.org/175475 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error | 20:29 |
*** achanda has joined #openstack-qa | 20:35 | |
*** kevinbenton has joined #openstack-qa | 20:43 | |
*** kevinbenton has quit IRC | 20:44 | |
*** kevinbenton has joined #openstack-qa | 20:44 | |
*** dboik has joined #openstack-qa | 20:48 | |
*** gfidente has quit IRC | 20:51 | |
*** dboik_ has quit IRC | 20:51 | |
*** e0ne has quit IRC | 20:55 | |
*** paul_glass has joined #openstack-qa | 20:59 | |
paul_glass | are there any examples of tempest/tempest-lib tests using more than one tenant? like POST with tenant A, GET with tenant B? | 21:02 |
clarkb | paul_glass: I think there is a cross tenant vm connectivity test that would do operations with multiple tenants | 21:02 |
sdague | paul_glass: git grep for alt_client inside tempest tree | 21:06 |
sdague | https://github.com/openstack/tempest/blob/master/tempest/api/compute/test_authorization.py#L175-L180 | 21:07 |
paul_glass | sdague: I see. so just instantiate a second client with different creds. | 21:08 |
openstackgerrit | Merged openstack/tempest: Fix test accounts unit test race with networks https://review.openstack.org/175470 | 21:20 |
*** cdent has quit IRC | 21:21 | |
clarkb | sdague: fyi https://jenkins01.openstack.org/job/gate-tempest-dsvm-full/20906/console failed and will bump that d-g change for grenade refactoring out of the gate | 21:25 |
*** jgrimm is now known as zz_jgrimm | 21:25 | |
*** dboik has quit IRC | 21:26 | |
*** jaypipes has quit IRC | 21:26 | |
openstackrecheck | openstack-infra/devstack-gate change: https://review.openstack.org/175390 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1376941 | 21:27 |
openstack | Launchpad bug 1352498 in Cinder "duplicate for #1376941 Tempest fails spectacularly with "VolumeType c39f18bf-2408-4377-98f6-ef2839d6f70e deletion failed, VolumeType in use."" [Undecided,New] | 21:27 |
*** dboik has joined #openstack-qa | 21:27 | |
openstackgerrit | Merged openstack-infra/elastic-recheck: Remove old queries for "fixed" bugs https://review.openstack.org/175529 | 21:29 |
*** e0ne has joined #openstack-qa | 21:30 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add logging of account allocation to accounts provider https://review.openstack.org/173136 | 21:30 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Rework get_network_from_name() function https://review.openstack.org/174473 | 21:30 |
openstackgerrit | Matthew Treinish proposed openstack-dev/devstack: Do Not Merge: Test test-accounts with roles https://review.openstack.org/173114 | 21:31 |
openstackgerrit | Matthew Treinish proposed openstack-dev/devstack: Do Not Merge: Test test-accounts with roles and networks https://review.openstack.org/158511 | 21:31 |
*** e0ne has quit IRC | 21:35 | |
mtreinish | sdague: btw, this was where I found /ips used in tempest | 21:37 |
mtreinish | it looks like that method is called in a few places | 21:37 |
*** erlon has quit IRC | 21:41 | |
dkranz | mtreinish, andreaf : I think it might help if you put your 2c in https://review.openstack.org/#/c/172605/ | 21:42 |
dkranz | "Tool for generation user accounts from spec" | 21:43 |
*** mwagner_lap has quit IRC | 21:46 | |
*** dustins has quit IRC | 21:47 | |
*** shashankhegde has quit IRC | 21:48 | |
openstackgerrit | nithya-ganesan proposed openstack/tempest: Define validation_resources function for ssh validation https://review.openstack.org/149229 | 21:49 |
mtreinish | dkranz: sure I can leave another comment on there | 21:52 |
mtreinish | but yeah having a yaml template doesn't make much sense there | 21:53 |
*** mriedem is now known as mriedem_away | 21:53 | |
*** yamahata has joined #openstack-qa | 21:54 | |
*** dpaterson has quit IRC | 21:54 | |
*** yamamoto has joined #openstack-qa | 21:56 | |
*** shashankhegde has joined #openstack-qa | 22:00 | |
*** openstackrecheck has quit IRC | 22:00 | |
*** openstackrecheck has joined #openstack-qa | 22:00 | |
*** melwitt has quit IRC | 22:04 | |
*** Swami has quit IRC | 22:07 | |
*** dboik has quit IRC | 22:09 | |
*** dboik has joined #openstack-qa | 22:11 | |
*** nmagnezi_ has quit IRC | 22:17 | |
openstackgerrit | Merged openstack/tempest: Adding port_admin_state_change option to config https://review.openstack.org/168679 | 22:24 |
openstackgerrit | Merged openstack/tempest: Apply a naming rule of GET to messaging client https://review.openstack.org/171009 | 22:25 |
*** bknudson has quit IRC | 22:26 | |
*** jecarey has quit IRC | 22:28 | |
*** achanda has quit IRC | 22:31 | |
sdague | mtreinish: yeh, I poked a little, actually need to circle with mordred tomorrow to figure out why he believes exposing it in the client will help with hpcloud here | 22:35 |
mordred | aroo? | 22:35 |
mordred | oh - I'm not sure it will help | 22:36 |
mordred | I was just reading docs | 22:36 |
mordred | and it seemed like a thing that would be worth _Trying_ to see what the difference was | 22:36 |
mordred | but then it turns out it's in the docs we publish but not the client lib | 22:36 |
mordred | so I wasn't sure if that was a bug, intentional, or in which direction it was a bug if it was | 22:36 |
mordred | i have no skin in the /ips game at the moment | 22:36 |
mtreinish | sdague: oh did I just point you to a link and fail to paste it? | 22:37 |
*** jaypipes has joined #openstack-qa | 22:37 | |
*** ChanServ sets mode: +o jaypipes | 22:37 | |
openstackgerrit | Merged openstack/tempest-lib: Migrate ssh.py from tempest https://review.openstack.org/166518 | 22:37 |
sdague | mordred: so it just returns the addresses slice of the json struct you get with a GET on .../servers/ID | 22:38 |
sdague | which, actually, in the hpcloud case is going to be a little weird, because that's going to be a proxy call to neutron | 22:38 |
mtreinish | sdague: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/services/compute/json/servers_client.py#n203 that's the link (not that it really matters at this point) | 22:38 |
sdague | so double API calls | 22:38 |
sdague | which I guess you were getting with the show anyway | 22:39 |
mordred | sdague: yah - so short story is ... | 22:39 |
mordred | we currently do /servers/detail to check status of server boots | 22:40 |
mordred | because we sometimes have 100 servers in booting state | 22:40 |
mordred | so just grabbing the whole thing in one go is better than 100 roundtrips to the API | 22:40 |
mordred | (especially with API rate limiting) - but that's constructing WAY more data on the backend than we need | 22:40 |
mordred | if we did a /servers to just get ID and boot status | 22:41 |
mordred | then we could detect if a server was in ready state | 22:41 |
sdague | and then the only other info you need is addresses? | 22:41 |
mordred | and _then_ just do an /ips to get its ips which is the only thing we'd need | 22:41 |
mordred | yeah | 22:41 |
sdague | ok, makes sense | 22:41 |
*** Swami has joined #openstack-qa | 22:41 | |
*** sabeen1 has joined #openstack-qa | 22:41 | |
sdague | I can get that into novaclient this week if you think it would help | 22:42 |
mtreinish | mordred: wait doesn't create return the id? I mean you'll still have to poll loop on the status, but you should at least have the id already | 22:44 |
mordred | sdague: neat! it's certainly worth a try | 22:45 |
mordred | mtreinish: yes - we have the id - you're right | 22:45 |
mordred | mtreinish: it's the status mapping we need from /servers | 22:45 |
*** wuhg has joined #openstack-qa | 22:47 | |
*** luqas has joined #openstack-qa | 22:48 | |
*** paul_glass has quit IRC | 22:52 | |
*** luqas has quit IRC | 22:53 | |
sdague | mordred: do you have a server or a server manager when you do this today? | 22:54 |
mordred | uhm | 22:56 |
mordred | sdague: server manager | 22:56 |
mordred | sdague: client.nova_client.servers.list | 22:57 |
*** sabeen1 has quit IRC | 22:57 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Rework get_network_from_name() function https://review.openstack.org/174473 | 22:57 |
openstackgerrit | Matthew Treinish proposed openstack-dev/devstack: Do Not Merge: Test test-accounts with roles https://review.openstack.org/173114 | 22:58 |
openstackgerrit | Matthew Treinish proposed openstack-dev/devstack: Do Not Merge: Test test-accounts with roles and networks https://review.openstack.org/158511 | 22:58 |
*** rbak has quit IRC | 22:58 | |
sdague | well, I need to write tests, but the add is as simple as - https://review.openstack.org/175603 | 23:02 |
*** annashen has joined #openstack-qa | 23:03 | |
*** hemna is now known as hemnafk | 23:07 | |
*** annashen has quit IRC | 23:08 | |
*** achanda has joined #openstack-qa | 23:10 | |
ianw | sdague: thanks; re peakmem tracker -- working on the other side of getting the log processor to be able to send stats to graphite ... that seems like the best place to do it | 23:10 |
sdague | ianw: yep, dtroyer and I thought we should make sure that was in stable/kilo to avoid having to backport | 23:10 |
sdague | ianw: cool, as soon as that is ready to go, I'd say we turn it on by default in devstack-gate for the relevant branches | 23:11 |
ianw | sdague: thanks ... i'm hoping to have the log processor just pick up a "stats" file, so anything in key:value format can get in there. we could track things like timing breakdowns like you did, etc | 23:11 |
sdague | neat, sounds awesome | 23:12 |
*** dimtruck is now known as zz_dimtruck | 23:14 | |
openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: DO NOT MERGE Try turning on ipv6 https://review.openstack.org/175607 | 23:15 |
*** sabeen1 has joined #openstack-qa | 23:16 | |
*** pc_m has quit IRC | 23:17 | |
*** yamamoto has quit IRC | 23:18 | |
*** achanda_ has joined #openstack-qa | 23:18 | |
*** achanda_ has quit IRC | 23:18 | |
*** achanda_ has joined #openstack-qa | 23:19 | |
*** achanda has quit IRC | 23:19 | |
openstackgerrit | Ian Wienand proposed openstack-dev/devstack: Fix negated services with common prefix https://review.openstack.org/173593 | 23:23 |
openstackgerrit | Ian Wienand proposed openstack-dev/devstack: Stop deleting token prefixes when processing ENABLED_SERVICES negations https://review.openstack.org/171402 | 23:25 |
openstackgerrit | Ian Wienand proposed openstack-dev/devstack: Stop deleting token prefixes when processing ENABLED_SERVICES negations https://review.openstack.org/171402 | 23:26 |
*** dboik has quit IRC | 23:33 | |
*** sabeen3 has joined #openstack-qa | 23:33 | |
*** madhu_ak has joined #openstack-qa | 23:34 | |
madhu_ak | hi folks, I am just checking whether any project using ddt module for their tempest tests? like this one: http://ddt.readthedocs.org/en/latest/example.html | 23:35 |
madhu_ak | If it is so, could you point me the link?, I would like to go through and understand the behavior | 23:36 |
*** sabeen1 has quit IRC | 23:37 | |
*** shashankhegde has quit IRC | 23:37 | |
*** Rockyg has quit IRC | 23:41 | |
*** marun has quit IRC | 23:42 | |
*** tochi has joined #openstack-qa | 23:42 | |
*** david-ly_ has joined #openstack-qa | 23:44 | |
*** jamielennox|away is now known as jamielennox | 23:45 | |
*** david-ly_ is now known as david-lyle | 23:46 | |
*** mtanino has quit IRC | 23:46 | |
*** ajmiller has quit IRC | 23:51 | |
*** yamamoto has joined #openstack-qa | 23:55 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Rework get_network_from_name() function https://review.openstack.org/174473 | 23:57 |
*** openstackgerrit has quit IRC | 23:58 | |
*** openstackgerrit has joined #openstack-qa | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!