Monday, 2015-04-20

*** salv-orlando has quit IRC00:04
*** yamamoto has quit IRC00:05
*** yamahata has quit IRC00:06
openstackgerritMerged openstack-dev/devstack: Append command to screenrc after we update it  https://review.openstack.org/17414700:09
*** amotoki has joined #openstack-qa00:14
*** yamahata has joined #openstack-qa00:18
*** dboik has quit IRC00:22
*** mahito has joined #openstack-qa00:23
*** sputnik13 has quit IRC00:31
*** sputnik13 has joined #openstack-qa00:36
*** woodster_ has quit IRC01:00
*** dboik has joined #openstack-qa01:04
*** salv-orlando has joined #openstack-qa01:05
*** salv-orlando has quit IRC01:11
*** sputnik13 has quit IRC01:12
*** luqas has joined #openstack-qa01:17
*** alex_xu has quit IRC01:18
*** luqas has quit IRC01:21
*** alex_xu has joined #openstack-qa01:23
*** mriedem has quit IRC01:26
*** yamamoto has joined #openstack-qa01:29
*** salv-orlando has joined #openstack-qa01:32
*** salv-orlando has quit IRC01:35
*** mahito_ has joined #openstack-qa01:36
*** mahito has quit IRC01:40
*** dboik has quit IRC01:40
*** salv-orlando has joined #openstack-qa01:41
*** salv-orlando has quit IRC01:51
*** achanda has joined #openstack-qa01:53
*** mahito_ has quit IRC01:56
*** mahito has joined #openstack-qa01:58
*** zz_dimtruck is now known as dimtruck02:00
*** cody-somerville has joined #openstack-qa02:07
*** annashen has joined #openstack-qa02:16
*** woodster_ has joined #openstack-qa02:28
*** sputnik13 has joined #openstack-qa02:28
*** annashen has quit IRC02:38
openstackgerritEli Qiao proposed openstack/tempest: fixed_ip unreserve should use an ip liked address  https://review.openstack.org/17002702:47
*** david-lyle has quit IRC02:55
*** annashen has joined #openstack-qa02:57
*** annashen has quit IRC02:57
*** jamielennox is now known as jamielennox|away02:58
*** achanda has quit IRC03:00
*** jamielennox|away is now known as jamielennox03:03
*** cody-somerville has quit IRC03:09
*** armax has joined #openstack-qa03:14
*** annashen has joined #openstack-qa03:15
*** annashen has quit IRC03:20
*** annashen has joined #openstack-qa03:21
*** annashen has quit IRC03:22
*** annashen has joined #openstack-qa03:23
*** annashen has quit IRC03:27
*** dboik has joined #openstack-qa03:30
*** armax has quit IRC03:37
*** coolsvap|afk is now known as coolsvap03:39
*** salv-orlando has joined #openstack-qa03:45
*** salv-orlando has quit IRC03:51
*** dpaterson has quit IRC03:58
*** dboik has quit IRC04:03
openstackrecheckopenstack/horizon change: https://review.openstack.org/167526 failed gate-grenade-dsvm in the gate queue with an unrecognized error04:05
*** yfried|afk is now known as yfried__04:18
*** rushiagr_away is now known as rushiagr04:20
*** subscope has joined #openstack-qa04:25
*** sabeen has joined #openstack-qa04:28
*** annashen has joined #openstack-qa04:31
*** Madan has joined #openstack-qa04:31
*** Poornima has joined #openstack-qa04:32
*** ajmiller has quit IRC04:34
*** Longgeek has joined #openstack-qa04:35
*** Longgeek has quit IRC04:35
*** Longgeek has joined #openstack-qa04:36
*** BharatK has joined #openstack-qa04:43
*** subscope has quit IRC04:47
*** sks has joined #openstack-qa04:48
*** ibiris_away is now known as ibiris04:48
*** yfried__ is now known as yfried|afk04:49
*** annashen has quit IRC04:50
*** k4n0 has joined #openstack-qa04:51
*** yfried|afk is now known as yfried__04:51
*** annashen has joined #openstack-qa04:51
*** sputnik13 has quit IRC04:52
*** luqas has joined #openstack-qa04:55
*** sputnik13 has joined #openstack-qa04:56
*** annashen has quit IRC04:56
*** boris-42 has quit IRC04:58
*** ibiris is now known as ibiris_away04:58
*** annashen has joined #openstack-qa04:59
*** luqas has quit IRC04:59
*** dimtruck is now known as zz_dimtruck05:00
*** yfried__ has quit IRC05:02
*** ibiris_away is now known as ibiris05:03
*** sputnik13 has quit IRC05:07
*** pal has joined #openstack-qa05:13
*** rdekel has joined #openstack-qa05:15
*** sabeen has quit IRC05:16
*** pal has quit IRC05:19
*** sputnik13 has joined #openstack-qa05:31
*** sputnik13 has quit IRC05:35
*** achanda has joined #openstack-qa05:37
*** sputnik13 has joined #openstack-qa05:38
*** gamado has joined #openstack-qa05:49
*** salv-orlando has joined #openstack-qa05:57
*** yamahata has quit IRC05:58
*** yamahata has joined #openstack-qa05:59
*** annashen has quit IRC06:01
*** annashen has joined #openstack-qa06:03
*** salv-orlando has quit IRC06:06
*** afazekas_ has joined #openstack-qa06:10
*** annashen has quit IRC06:12
*** anshul has joined #openstack-qa06:14
*** annashen has joined #openstack-qa06:14
*** yamahata has quit IRC06:15
*** SridharG has joined #openstack-qa06:15
*** annashen has quit IRC06:16
*** annashen has joined #openstack-qa06:16
*** zz_jgrimm is now known as jgrimm06:19
*** yfried__ has joined #openstack-qa06:19
*** annashen has quit IRC06:20
*** anshul has quit IRC06:27
*** yfried__ is now known as yfried|afk06:30
*** woodster_ has quit IRC06:30
*** neeti has joined #openstack-qa06:33
*** yfried|afk is now known as yfried__06:34
*** sergsh has joined #openstack-qa06:37
*** annashen has joined #openstack-qa06:39
*** anshul has joined #openstack-qa06:40
*** mmedvede has quit IRC06:44
*** flaper87 has quit IRC06:45
*** flaper87 has joined #openstack-qa06:45
*** gillllllliard is now known as gilliard06:48
*** anshul has quit IRC06:50
*** jamielennox is now known as jamielennox|away06:52
*** jyuso1 has joined #openstack-qa06:54
*** mmedvede has joined #openstack-qa07:00
*** itzikb has joined #openstack-qa07:01
*** annashen has quit IRC07:02
*** anshul has joined #openstack-qa07:04
*** sergsh has quit IRC07:06
*** SridharG has left #openstack-qa07:12
*** nmagnezi_ has joined #openstack-qa07:12
*** mpaolino has joined #openstack-qa07:15
*** apevec has joined #openstack-qa07:18
*** apevec has quit IRC07:18
*** apevec has joined #openstack-qa07:18
*** jhenner has joined #openstack-qa07:25
*** tochi has quit IRC07:28
*** dratushnyy has joined #openstack-qa07:29
*** ihrachyshka has joined #openstack-qa07:31
*** e0ne has joined #openstack-qa07:31
*** bkopilov has quit IRC07:32
*** bkopilov has joined #openstack-qa07:32
*** e0ne has quit IRC07:34
*** SridharG has joined #openstack-qa07:36
*** jlanoux has joined #openstack-qa07:37
*** luqas has joined #openstack-qa07:38
*** e0ne has joined #openstack-qa07:38
*** e0ne has quit IRC07:40
*** e0ne has joined #openstack-qa07:45
*** achanda has quit IRC07:47
*** weshay has joined #openstack-qa07:52
*** sputnik13 has quit IRC07:52
*** sputnik13 has joined #openstack-qa07:54
*** apevec has quit IRC07:57
*** safchain has joined #openstack-qa08:01
*** annashen has joined #openstack-qa08:03
*** jordanP has joined #openstack-qa08:04
*** e0ne has quit IRC08:04
*** sergsh has joined #openstack-qa08:05
*** salv-orlando has joined #openstack-qa08:07
*** mmedvede has quit IRC08:07
*** e0ne has joined #openstack-qa08:08
*** annashen has quit IRC08:08
*** e0ne is now known as e0ne_08:09
*** salv-orlando has quit IRC08:11
*** e0ne_ is now known as e0ne08:11
*** sergsh has quit IRC08:12
*** tkammer has joined #openstack-qa08:14
*** jlibosva has joined #openstack-qa08:15
*** jlibosva has quit IRC08:15
*** salv-orlando has joined #openstack-qa08:17
*** mmedvede has joined #openstack-qa08:17
*** e0ne has quit IRC08:17
*** vponomaryov has joined #openstack-qa08:19
*** shardy has joined #openstack-qa08:20
*** salv-orlando has quit IRC08:21
*** salv-orlando has joined #openstack-qa08:23
*** sergsh has joined #openstack-qa08:25
*** salv-orlando has quit IRC08:28
*** zz_johnthetubagu is now known as johnthetubaguy08:29
*** gfidente has joined #openstack-qa08:30
*** dmellado has joined #openstack-qa08:30
*** anshul has quit IRC08:32
*** mahito has quit IRC08:34
*** tkammer has quit IRC08:39
*** andreykurilin__ has joined #openstack-qa08:39
*** yamamoto has quit IRC08:45
*** anshul has joined #openstack-qa08:46
*** salv-orlando has joined #openstack-qa08:51
*** tkammer has joined #openstack-qa08:52
*** neeti has quit IRC08:52
*** gamado has quit IRC08:55
openstackgerritFilip Hubík proposed openstack/tempest: Validate keystone tokens  https://review.openstack.org/17513008:58
*** e0ne has joined #openstack-qa08:59
*** ihrachyshka has quit IRC09:00
*** rdekel has quit IRC09:03
*** apevec has joined #openstack-qa09:04
*** annashen has joined #openstack-qa09:04
*** jhenner has quit IRC09:05
*** gamado has joined #openstack-qa09:08
*** annashen has quit IRC09:09
*** ihrachyshka has joined #openstack-qa09:11
*** e0ne is now known as e0ne_09:13
*** gamado has quit IRC09:13
*** e0ne_ is now known as e0ne09:13
*** jlanoux_ has joined #openstack-qa09:17
*** jlanoux has quit IRC09:20
*** hichihara has quit IRC09:23
openstackgerritYaroslav Lobankov proposed openstack/tempest: Wait for complete deletion of volume snapshot before deleting volume  https://review.openstack.org/17156909:24
*** pcaruana has quit IRC09:27
*** Abhishek has joined #openstack-qa09:29
*** pcaruana has joined #openstack-qa09:30
*** sergsh has quit IRC09:32
*** amotoki has quit IRC09:35
*** sergsh has joined #openstack-qa09:44
*** yfried__ is now known as yfried|afk09:48
*** coolsvap is now known as coolsvap|afk09:51
*** boris-42 has joined #openstack-qa09:51
*** yfried|afk is now known as yfried__09:53
*** jhenner has joined #openstack-qa09:56
*** andreykurilin__ has quit IRC10:00
*** aix has joined #openstack-qa10:01
*** neeti has joined #openstack-qa10:01
*** pc_m has joined #openstack-qa10:09
*** safchain has quit IRC10:12
*** dratushnyy has quit IRC10:17
*** cdent has joined #openstack-qa10:22
*** BobBall has joined #openstack-qa10:25
*** yfried__ is now known as yfried|afk10:25
BobBallmtreinish: 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
BobBallmtreinish: 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_away10:38
*** anshul has quit IRC10:42
boris-42sdague: hi there10:44
*** rdekel has joined #openstack-qa10:49
*** anshul has joined #openstack-qa10:55
*** e0ne is now known as e0ne_10:56
sdagueboris-42: hey, what's up?10:56
boris-42sdague: 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-qa10:57
*** e0ne_ is now known as e0ne10:58
sdagueyeh, 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
sdaguebut that's just personal experience out of the nova project, where raw coverage numbers are not out problem10:59
sdaguebut the fact that tests don't always reflect reality, so you are testing the wrong things10:59
boris-42sdague: agree with you on this11:00
boris-42sdague: but it helps in next situation11:00
boris-42sdague: when new developer or lazy just publish patch with bad coverage11:00
boris-42sdague: so reviewers should spend each time to explain that he should write unit tests11:00
boris-42sdague: it's better to have CI that says that11:01
sdaguehonestly, 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 them11:02
sdagueotherwise they'll just iterate on some random use of mock until they make that number exceed the passing threshold11:02
sdagueit will pass the number, it doesn't mean it's good11:02
boris-42sdague: I am not saying that this job will allow core team NOT to review unit tests11:03
*** jhenner1 has joined #openstack-qa11:03
boris-42sdague: 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 cases11:04
*** jhenner has quit IRC11:05
*** jhenner1 has quit IRC11:07
*** anshul has quit IRC11:14
*** e0ne is now known as e0ne_11:16
*** afazekas_ has quit IRC11:24
*** jhenner has joined #openstack-qa11:24
*** yfried__ is now known as yfried|afk11:25
*** e0ne_ has quit IRC11:26
*** yfried|afk is now known as yfried__11:26
*** anshul has joined #openstack-qa11:27
*** afazekas_ has joined #openstack-qa11:28
*** BobBall has quit IRC11:29
HenryGAnyone here familiar with the workflow for adding stuff to the official cirros images and getting it released?11:29
HenryGafazekas: ping11:32
openstackgerritMerged openstack-dev/devstack: run dstat with run_process  https://review.openstack.org/17198811:34
*** safchain has joined #openstack-qa11:36
*** tosky has joined #openstack-qa11:36
*** BobBall has joined #openstack-qa11:40
*** Poornima has quit IRC11:43
*** woodster_ has joined #openstack-qa11:46
*** Madan has quit IRC11:49
*** mmedvede has quit IRC11:56
*** afazekas_ has quit IRC11:58
*** ihrachyshka has quit IRC12:03
*** mmedvede has joined #openstack-qa12:04
*** rfolco has joined #openstack-qa12:07
*** ihrachyshka has joined #openstack-qa12:19
*** ihrachyshka has quit IRC12:19
*** ihrachyshka has joined #openstack-qa12:19
*** bknudson has quit IRC12:26
*** e0ne has joined #openstack-qa12:26
*** salv-orlando has quit IRC12:28
*** jamielennox|away is now known as jamielennox12:31
*** jamielennox is now known as jamielennox|away12:32
*** jaypipes has joined #openstack-qa12:33
*** ChanServ sets mode: +o jaypipes12:33
*** erlon has joined #openstack-qa12:34
*** jecarey has quit IRC12:36
*** BharatK has quit IRC12:36
openstackgerritFilip Hubík proposed openstack/tempest: Add keystone v2.0 and v3 api discovery checks  https://review.openstack.org/16271012:37
*** ihrachyshka has quit IRC12:46
*** zz_dimtruck is now known as dimtruck12:46
*** cdent has quit IRC12:54
*** bknudson has joined #openstack-qa12:55
*** salv-orlando has joined #openstack-qa12:56
*** dimtruck is now known as zz_dimtruck12:56
*** ihrachyshka has joined #openstack-qa12:58
*** sks has quit IRC13:02
*** neeti has quit IRC13:03
*** lifeless has quit IRC13:05
*** annashen has joined #openstack-qa13:08
openstackgerritSean Dague proposed openstack-infra/devstack-gate: redo devstack-gate grenade selection as a case statement  https://review.openstack.org/17539013:10
openstackgerritSean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo  https://review.openstack.org/17539113:10
openstackgerritSean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo  https://review.openstack.org/17539113:11
openstackgerritSean Dague proposed openstack-infra/devstack-gate: redo devstack-gate grenade selection as a case statement  https://review.openstack.org/17539013:12
*** annashen has quit IRC13:13
*** sks has joined #openstack-qa13:16
openstackgerritSean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo  https://review.openstack.org/17539113:18
*** jecarey has joined #openstack-qa13:21
*** rbak has quit IRC13:21
*** bkopilov has quit IRC13:27
*** zz_dimtruck is now known as dimtruck13:28
*** artom has joined #openstack-qa13:28
*** sslypushenko has quit IRC13:30
*** superdan is now known as dansmith13:31
*** bkopilov has joined #openstack-qa13:35
openstackgerritMerged openstack-dev/devstack: refactor ping_check  https://review.openstack.org/17437013:39
*** bkopilov has quit IRC13:40
*** bkopilov has joined #openstack-qa13:42
*** dboik has joined #openstack-qa13:44
*** dboik has quit IRC13:45
*** dboik has joined #openstack-qa13:45
*** mriedem has joined #openstack-qa13:46
openstackgerritSergey Slipushenko proposed openstack/tempest: Test UUID duplication fix  https://review.openstack.org/16592113:47
*** bkopilov has quit IRC13:49
openstackgerritnithya-ganesan proposed openstack/tempest: Define validation_resources function for ssh validation  https://review.openstack.org/14922913:52
*** e0ne is now known as e0ne_13:53
SergeyLukjanovsdague, we'd like to enable more testing jobs for sahara (for all repos)13:55
SergeyLukjanovsdague, for sahara itself we'd like to add a few custom jobs that will be running tempest-lib based tests13:55
*** Abhishek has quit IRC13:56
SergeyLukjanovto ensure that integration with OpenStack and especially with Heat works well13:56
mtreinishBobBall: with devstack gate there is a tempest-regex var you can pass in to tell testr which tests to run13:57
SergeyLukjanovmtreinish, hey!13:57
mtreinishSergeyLukjanov: hi13:58
sdagueSergeyLukjanov: ok, so what is the question :)13:58
SergeyLukjanovsdague, 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 topologies13:59
SergeyLukjanovheat / no-heat and nova-net / neutron13:59
SergeyLukjanov4 dsvm jobs13:59
SergeyLukjanovsdague, heh, I've missed the question :)13:59
SergeyLukjanovthe question is - is it ok? :)14:00
SergeyLukjanovI think that the best case is to run *data_processing* tempest tests in this jobs and additional tests from sahara repo14:00
mtreinishSergeyLukjanov: why wouldn't it be ok? :) you guys can run whatever testing you think is needed against sahara changes14:02
*** e0ne_ is now known as e0ne14:02
SergeyLukjanovmtreinish, sdague, I just want to be sure that you're aware and okay with it :)14:03
*** jgrimm is now known as zz_jgrimm14:04
*** bkopilov has joined #openstack-qa14:04
*** dboik_ has joined #openstack-qa14:06
*** ajmiller has joined #openstack-qa14:08
mtreinishmriedem: is that test any different than the prexisting ports test?14:08
*** dboik has quit IRC14:08
*** mtanino has joined #openstack-qa14:08
*** sks has quit IRC14:08
mriedemmtreinish: not really, except it doesn't test the delete validation - and it's not skipped on an option14:09
BobBallmtreinish: Is there a nice way to test the regexp in a dev environment? (i.e. list the tests that would match it)14:09
mriedemmtreinish: i'm guessing something is working in kilo for ironic/nova in that hypervisor macs scenario that isn't working on juno14:09
*** mwagner_lap has joined #openstack-qa14:09
mtreinishBobBall: not right now, unless you do it manually before running the tests14:09
*** apevec has quit IRC14:09
mtreinishBobBall: but after reqs freeze ends I'll be moving tempest over to use os-testr which I added some stuff to do that for14:10
mtreinishmriedem: hmm, ok. I know the ironic networking is kinda wonky so I'm not surprised it doesnt work14:10
mtreinishmriedem: I'm just trying to weigh a revert vs a skipif14:10
BobBallok14:10
mriedemmtreinish: i didn't dig into the history of the driver to see if there were changes in kilo that should be backported14:10
mtreinishmriedem: tbh I don't care that much14:11
mriedemmtreinish: i thought about skipping if baremetal=True, but the pxe job passes on kilo with that change14:11
mriedemso it must be a bug in juno14:11
mriedemmaybe adam_g or devananda might know about that, idk14:11
mriedemi 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
mriedembut we probably aren't going to backport that to juno, and i don't know if it fixes the problem anyway14:14
*** sabeen1 has joined #openstack-qa14:17
*** k4n0 has quit IRC14:17
*** afazekas_ has joined #openstack-qa14:18
mtreinishmriedem: +A'd the revert14:18
mriedemk14:18
mriedemi don't know enough about the ironic code to know if there was a bug/race there in juno that's fixed in kilo14:19
mtreinishthere were some other issues I spotted with the test just from a quick glance. Like it leaks floating ips too14:19
mtreinishmriedem: the majority of the neutron tests have to be skipped with ironic. irrc it's because everything is in one shared network14:20
mtreinishalthough I'm not sure why creating a port before server creation would affect that14:20
mriedemfrom 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 up14:21
*** NobodyCam has left #openstack-qa14:21
mriedemthat list of macs might have been empty? i'm not sure14:21
mriedemcould have been useful debugging when that fails, i.e. dump the list of macs that we do have14:22
mriedemi'll throw up a patch for that14:22
*** Party_Bear is now known as BadCub14:24
*** sputnik13 has quit IRC14:26
*** afazekas_ has quit IRC14:26
*** mvandijk_ has quit IRC14:26
*** dboik_ has quit IRC14:27
openstackgerritMerged openstack-dev/devstack: Catalog Index Service - glance devstack  https://review.openstack.org/16920014:27
*** dboik has joined #openstack-qa14:27
*** mvandijk has joined #openstack-qa14:28
openstackgerritMerged openstack-dev/devstack: Add logging config to cinder.conf  https://review.openstack.org/17515514:28
openstackgerritnithya-ganesan proposed openstack/tempest: Define validation_resources function for ssh validation  https://review.openstack.org/14922914:29
*** afazekas_ has joined #openstack-qa14:30
*** dpaterson has joined #openstack-qa14:32
*** lifeless has joined #openstack-qa14:32
*** achanda has joined #openstack-qa14:32
*** rbak has joined #openstack-qa14:33
sdaguemtreinish: 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
mriedemsdague: i think that's old14:35
mriedemhttp://status.openstack.org/elastic-recheck/index.html#142915114:35
sdaguemriedem: ok, I *just* hit it14:35
mriedembeen around since early march https://bugs.launchpad.net/cinder/+bug/142915114:35
openstackLaunchpad bug 1429151 in Cinder "SimpleReadOnlyCinderClientTest.test_cinder_encryption_type_list fails randomly" [High,Confirmed]14:35
sdagueok14:35
*** afazekas_ has quit IRC14:36
mtreinishsdague: I didn't even know there was a cinder encryption cli test14:37
mriedemmestery: 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_64914:37
mtreinishsdague: but jgriffith should be removing all of those from tempest any day now14:37
mriedemhttp://goo.gl/iw3Aah14:37
jgriffithmtreinish: :)  thanks for the Monday morning reminder :)14:37
mtreinishjgriffith: heh, I wasn't going to miss that opportunity :)14:38
*** achanda has quit IRC14:39
mtreinishnotmyname: 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
mtreinishthose get run against a running swift deploy14:42
*** sslypushenko has joined #openstack-qa14:46
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck: Add query for neutron functional test bug 1446261  https://review.openstack.org/17543214:47
openstackbug 1446261 in neutron "gate-neutron-dsvm-functional race fails HA/DVR tests with network namespace not found" [Undecided,New] https://launchpad.net/bugs/144626114:47
*** afazekas_ has joined #openstack-qa14:47
*** jlanoux_ has quit IRC14:47
*** hemnafk is now known as hemna14:50
mesterymtreinish: 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
mesterymriedem: ^^^14:50
mesterymriedem: But I see you just proposed an ER for it, thanks!14:50
mriedemyup14:51
mriedemthere was an eventlet timeout in that job last week which marun posted a longer timeout that seems to have staved that off a bit14:51
mriedemi thought they were maybe related but doesn't look like it14:51
mesterymriedem: Ack14:52
*** nmagnezi_ has quit IRC14:52
*** tosky has quit IRC14:53
*** cdent has joined #openstack-qa14:53
*** tosky has joined #openstack-qa14:53
mriedemmestery: looks like that started showing up around 4/1514:55
mesterymriedem: Cool, thanks for that!14:56
*** anshul has quit IRC14:56
sdaguemriedem: your tempest revert just bounced on uit tests14:56
*** itzikb has quit IRC14:56
mriedemfun14:57
mtreinishsdague: link? I hope we didn't introduce a new unit test race recently...14:58
mtreinishalthough if we did I have a feeling it's my fault14:58
mriedemmtreinish: https://jenkins03.openstack.org/job/gate-tempest-python27/3203/console14:58
mriedemlooks unrelated14:58
*** tsekiyama has joined #openstack-qa14:58
mriedemtempest.tests.common.test_accounts.TestAccount.test__get_creds_by_roles_no_admin14:59
mtreinishyeah that's my fault I think14:59
mriedemway2go14:59
mtreinishthe test is pulling the "wrong" account from the fake list, because I wrote it assuming it would always be in order15:00
mriedemjust racey or blocking?15:00
mtreinishracey15:01
devanandamriedem: g'morning15:01
mtreinishone of the valid fake accounts has a network. But that test doesn't mock the network api calls15:01
*** BharatK has joined #openstack-qa15:01
mriedemdevananda: 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-qa15:02
mtreinishwe 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 test15:02
mriedemb/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 driver15:02
mriedemdevananda: i wasn't sure if there was a fix in kilo for that, in ironic15:02
devanandareading backscroll, but I don't know what tests you're referring to, or what revert....15:02
devanandaoh. helps if I scroll *down* too :)15:02
mriedemor if i summarize :)15:03
*** marun has joined #openstack-qa15:03
notmynamemtreinish: 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
devanandareading the revert patch ...15:04
notmynamemtreinish: also, why do you ask? ;-)15:04
*** zz_jgrimm is now known as jgrimm15:04
*** achanda has joined #openstack-qa15:05
mtreinishnotmyname: yeah they just do read only operation cli calls and check the response is sane15:05
mtreinishnotmyname: well I've been trying to remove the cli tests from tempest and into the client repos all cycle15:05
mtreinishand swift, heat, and cinder are the last 315:05
mtreinishbut if you guys have equiv coverage I can just delete the swift tests15:06
notmynamemtreinish: 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 IRC15:07
mtreinishnotmyname: so you joke but the cli tests were added to tempest because things like that used to stack trace some of the clients15:07
mtreinishwhich is actually hysterical, or sad depending on your perspective I guess :)15:08
*** afazekas_ has quit IRC15:08
mtreinishbut, yeah test is a bit of a strectch for that one :)15:10
*** annashen has joined #openstack-qa15:10
*** beekneemech is now known as bnemec15:10
*** e0ne is now known as e0ne_15:10
notmynamemtreinish: I think we're good. looking through the pythn-swiftclient tests, we've got coverage for that15:11
notmynamemtreinish: it's not 100% identical, but the functionality looks to be covered15:12
mtreinishnotmyname: ok cool, do you want to push a patch to delete those tests from tempest then :)15:12
mtreinishnotmyname: also fwiw, if you wanted to move the tests over it's pretty simple: http://lists.openstack.org/pipermail/openstack-dev/2015-February/057458.html15:12
notmynamemtreinish: test_swift_region_list is probably the only one that isn't specifically covered.... let me check something else15:12
devanandamriedem: still trying to wrap my brain around how this test is different from those that call _setup_network_and_servers15:14
*** annashen has quit IRC15:15
notmynamemtreinish: ok, we're good there, too, I think15:15
notmynamemtreinish: no, I don't want to do that patch, but I'll +1 it if you give me a link to it15:15
mtreinishdevananda: apparently it assigns a different sec group, but yeah I didn't see any difference15:15
devanandamriedem: 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 tests15:15
openstackgerritgordon chung proposed openstack-dev/devstack: ceilometer: add tempest option to test events  https://review.openstack.org/16486015:15
mtreinishdevananda: arent the other tests skipped if ironic is enabled?15:15
devanandasome, not all15:15
devanandaeg, test_network_basic_ops is not skipped15:16
mriedemdevananda: that being the list of hypervisor macs that are coming back from ironic15:16
sdaguemtanino: I'll push the patch15:16
mtaninosdague: hi15:16
*** e0ne_ is now known as e0ne15:17
mriedemdoes 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
sdaguemtanino: sorry, wrong complete15:17
mtaninogocha!15:17
mtreinishnotmyname: ok, I can push it then :) Do you have a link to where the swift equiv tests are?15:17
mtreinishsdague: heh too many m's15:17
openstackgerritSean Dague proposed openstack/tempest: remove swift cli tests  https://review.openstack.org/17544715:17
devanandamriedem: possibly? also, creating the port from a different secgroup could be it15:18
openstackrecheckopenstack/nova change: https://review.openstack.org/163378 failed because of: gate-grenade-dsvm-ironic-sideways: https://bugs.launchpad.net/bugs/144591715:18
openstackLaunchpad 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
openstackrecheckopenstack/tempest change: https://review.openstack.org/175219 failed gate-tempest-python27 in the gate queue with an unrecognized error15:18
devanandaspeaking of ...15:19
notmynamemtreinish: 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.py15:19
*** afazekas_ has joined #openstack-qa15:21
mtreinishnotmyname: cool, thanks15:21
*** armax_ has joined #openstack-qa15:22
*** weshay has quit IRC15:23
*** weshay has joined #openstack-qa15:24
*** rfolco has quit IRC15:24
*** rfolco has joined #openstack-qa15:25
*** rfolco has quit IRC15:26
*** afazekas_ has quit IRC15:26
*** ihrachyshka has quit IRC15:27
*** rfolco has joined #openstack-qa15:27
*** safchain has quit IRC15:28
*** dkranz has quit IRC15:30
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck: Add query for neutron functional test bug 1446261  https://review.openstack.org/17543215:30
openstackbug 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 amrith15:31
*** SridharG has left #openstack-qa15:33
*** dratushnyy has quit IRC15:35
*** dkranz has joined #openstack-qa15:37
*** afazekas_ has joined #openstack-qa15:37
mtreinishmriedem: did you file a bug for the tempest unit test race?15:39
mriedemmtreinish: nope15:39
mriedemi've hit my bug reporting quota for the morning15:39
mtreinishok15:39
mtreinishI've got the fix, I just wanted to make sure I didn't miss a bug number in the commit msg15:40
mriedemi would -1 that so hard15:40
mriedemdevananda: 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
mriedemmaybe that's what causes ironic to fail on juno15:41
*** safchain has joined #openstack-qa15:41
*** annashen has joined #openstack-qa15:41
*** Swami has joined #openstack-qa15:42
mriedemheh, 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.gz15:42
mriedemonly the baremetal tests are run there15:42
mriedemmtreinish: so i guess we could have just skipped if baremetal15:42
openstackgerritSean Dague proposed openstack/tempest: remove swift cli tests  https://review.openstack.org/17544715:43
*** amrith has left #openstack-qa15:44
mtreinishmriedem: sure, I figured that was the case15:44
mtreinishbut I was fine with reverting and bringing it back later15:44
mriedemyeah15:45
mriedemyou said it needed more work anyway15:45
*** yfried__ has quit IRC15:45
mtreinishyep15:46
sdaguedtroyer: so.... devstack branching?15:46
*** dboik_ has joined #openstack-qa15:47
*** afazekas_ has quit IRC15:47
*** dboik has quit IRC15:50
*** mlavalle has joined #openstack-qa15:50
*** rdekel has quit IRC15:52
*** jhenner has quit IRC15:57
*** ihrachyshka has joined #openstack-qa15:58
*** sergsh has quit IRC15:59
*** BharatK has quit IRC16:00
*** jhenner has joined #openstack-qa16:00
*** Longgeek has quit IRC16:00
*** sabeen1 has quit IRC16:02
dtroyersdague: 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-qa16:04
sdagueheh16:04
*** weshay has quit IRC16:05
*** jhenner has quit IRC16:05
*** tkammer has quit IRC16:06
openstackgerritMatthew Treinish proposed openstack/tempest: Fix test accounts unit test race with networks  https://review.openstack.org/17547016:09
mtreinishmriedem: ^^^ that's the unit test fix16:09
*** jhenner has joined #openstack-qa16:09
dtroyersdague: 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
sdaguedtroyer: you mean just the enable_service bit?16:22
sdagueI can respin and we can merge that if you like16:22
openstackgerritMerged openstack-infra/elastic-recheck: Add query for neutron functional test bug 1446261  https://review.openstack.org/17543216:22
openstackbug 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
dtroyersdague: yes16:22
sdaguedtroyer: you want it on by default, or off by default?16:24
dtroyersdague: no opinion, I'm guesing ianw wants it on in the gate?  or did he intend this for debugging only?16:24
sdaguehe wants it on in the gate, which I think means default off is probably right, and turn it on in d-g16:25
sdaguehe had some concerns around memory exhaustion16:25
dtroyersounds good to me16:25
mtreinishsdague: yeah the centos7 jobs were constantly fighting the oomkiller16:26
mtreinishI think16:26
openstackgerritSean Dague proposed openstack-dev/devstack: Add a peak memory tracker to dstat  https://review.openstack.org/17191916:27
sdaguedtroyer: ok, if that looks good, lets land it16:27
*** annashen has quit IRC16:27
dtroyer+2, thanks16:29
sdagueyou want to just +A it in?16:29
sdaguealso, I never remember who can actually make devstack branches, or how we do it correctly16:30
dtroyerok16:30
*** dboik_ has quit IRC16:30
*** dboik has joined #openstack-qa16:30
dtroyerI've always had infra or maybe ttx do the branch16:31
dtroyerI think we can only push tags16:31
sdaguefungi: ?16:31
clarkbya looks like openstack release has perms from all-projects then any gerrit admin can also do it16:31
fungilooking at how it's configured in the acl16:31
fungiright, that16:31
clarkband 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 IRC16:33
dtroyerHEAD after https://review.openstack.org/171919 and https://review.openstack.org/168796 merge16:33
*** dboik has joined #openstack-qa16:33
sdagueso, we probably could tag grenade first then, right?16:37
sdaguebecause we have a commit there that's solid16:37
sdaguegrenade would be stable/kilo at 1a17e0c14d90c5df3a0237356871ab6aa73a374a16:38
*** safchain has quit IRC16:40
dtroyerhttps://review.openstack.org/#/c/172156/ is ~15 minutes from merging…16:40
dtroyerotherwise, yeah, that'll work16:41
*** Swami has quit IRC16:41
sdagueoh, yeh, typo16:41
sdagueyeh, lets let the typo fix land16:42
*** luqas has quit IRC16:43
sdagueclarkb: 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 support16:43
clarkbsdague: sure16:43
*** armax_ has left #openstack-qa16:46
*** armax_ has joined #openstack-qa16:47
*** Nic has joined #openstack-qa16:47
*** Swami has joined #openstack-qa16:48
*** nmagnezi_ has joined #openstack-qa16:51
*** annashen has joined #openstack-qa16:51
clarkbsdague: can you see my comment on 175390?16:53
clarkbI don't think its worth a -1 but wanted to make sure that we are doing that intentionally and if so consider simplifying it16:53
sdagueoh, hmmm16:54
sdagueyeh, I was trying to be clever, and clearly failed16:54
sdagueit actually does all the wrong things16:54
sdagueok, let me go look at that again16:55
sdaguethat's a better catch than I think you originally though16:55
clarkbcould be :)16:55
sdaguebecause it actually means partial cpu nova doesn't any more16:55
clarkboh right because the variable is overridden16:56
clarkbso it is worth a -1 :)16:56
clarkbwill update16:56
openstackgerritSean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo  https://review.openstack.org/17539116:57
openstackgerritSean Dague proposed openstack-infra/devstack-gate: redo devstack-gate grenade selection as a case statement  https://review.openstack.org/17539016:57
sdagueok, fixed16:58
sdagueclarkb: thanks for the find16:58
*** suneelb has joined #openstack-qa17:02
*** dboik has quit IRC17:02
*** dboik has joined #openstack-qa17:02
clarkbsdague: we should add a third change that updates features.yaml for liberty17:02
clarkbsdague: just to get that out of the way while we have it17:03
clarkb*have it on our minds17:03
*** dboik has quit IRC17:03
*** BobBall has left #openstack-qa17:03
openstackgerritMerged openstack-dev/grenade: Fix typos in PLUGINS  https://review.openstack.org/17215617:03
sdagueclarkb: sure, should that be in patch #2 as well?17:03
*** dboik has joined #openstack-qa17:03
*** armax_ has quit IRC17:03
clarkbsdague: I think it either go there or in a separate change, either way is fine17:03
*** armax_ has joined #openstack-qa17:04
sdagueoh, yeh, because we don't actually need that for a while, right17:04
sdagueas kilo is already called out17:04
clarkbyup17:04
clarkbbut 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 work17:05
clarkbso its a nice prep step17:05
sdagueok, I'll do it as patch #3 then17:05
*** annashen has quit IRC17:06
openstackgerritSean Dague proposed openstack-infra/devstack-gate: add liberty to feature grid  https://review.openstack.org/17549317:07
*** cody-somerville has joined #openstack-qa17:07
*** dboik has quit IRC17:07
*** dboik has joined #openstack-qa17:07
*** mpaolino has quit IRC17:08
*** jhenner has quit IRC17:08
*** patrickeast has joined #openstack-qa17:10
openstackgerritMatt Riedemann proposed openstack/tempest: Fix test accounts unit test race with networks  https://review.openstack.org/17547017:11
mriedemsdague: mtreinish: ^ added bug line17:11
sdaguemriedem: thanks17:12
*** annashen has joined #openstack-qa17:12
openstackgerritMerged openstack-dev/devstack: Clone external plugins before overriding defaults  https://review.openstack.org/16879617:12
clarkbthats one17:13
*** openstackrecheck has quit IRC17:13
*** openstackrecheck has joined #openstack-qa17:13
openstackgerritMatthew Treinish proposed openstack-infra/elastic-recheck: Add query for bug 1446309  https://review.openstack.org/17549617:13
openstackbug 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
mtreinishmriedem: ^^^17:13
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck: Add query for tempest unit test bug 1446309  https://review.openstack.org/17549717:14
mriedemmtreinish: ^^^17:14
mtreinishmriedem: hehe, I win :)17:14
mtreinishfinally17:14
mriedemwhat?!17:14
mriedemgdi17:14
mriedemoh you could totally add the build_name!17:15
mriedem-3!17:15
mriedemha, co-authored-by17:15
mriedemthat's cute17:15
*** jhenner has joined #openstack-qa17:15
mtreinishwell I stole your message line from the link in the review17:15
mtreinishfigured that was worth something17:15
mriedemyeah17:15
*** jhenner has quit IRC17:16
*** jhenner has joined #openstack-qa17:16
openstackgerritMatthew Treinish proposed openstack-infra/elastic-recheck: Add query for bug 1446309  https://review.openstack.org/17549617:17
openstackbug 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
mtreinishmriedem: ^^^ build name added17:17
mriedemsheesh, beat me twice17:17
mriedemi was just updating your change locally17:17
*** davideagnello has joined #openstack-qa17:20
*** achanda has joined #openstack-qa17:24
openstackgerritSean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo  https://review.openstack.org/17539117:25
openstackgerritSean Dague proposed openstack-infra/devstack-gate: redo devstack-gate grenade selection as a case statement  https://review.openstack.org/17539017:25
openstackgerritSean Dague proposed openstack-infra/devstack-gate: add liberty to feature grid  https://review.openstack.org/17549317:25
*** cody-somerville has quit IRC17:34
openstackgerritMatthew Treinish proposed openstack/tempest: Add configuration guide sections for more required options  https://review.openstack.org/17404717:34
openstackgerritMatthew Treinish proposed openstack/tempest: Add section to config guide for setting up services  https://review.openstack.org/17414917:34
openstackgerritMatthew Treinish proposed openstack/tempest: Start a tempest config guide networking section  https://review.openstack.org/17250717:34
openstackgerritMatthew Treinish proposed openstack/tempest: Add config guide section on service feature enabled sections  https://review.openstack.org/17408617:34
openstackgerritMatthew Treinish proposed openstack/tempest: Add a paragraph on the use of tempest_roles option  https://review.openstack.org/17467517:34
mriedemthat revert failed in the gate again17:37
mriedemright at the end17:37
mriedemmy old friend test_volume_boot_pattern17:37
*** dboik has quit IRC17:37
mtreinishmriedem: was it the generic ssh failure or a different thing this time?17:37
mriedemwaiting on logs17:38
*** dboik has joined #openstack-qa17:38
mriedemconsole log looks like generic ssh failure17:38
*** melwitt has joined #openstack-qa17:38
*** harlowja_away is now known as harlowja17:38
mtreinishmriedem: neutron or n-net?17:40
mriedemneutron17:40
clarkbsdague: I approved the first d-g change, let me know when we can start tagging devstack and grenade so we can approve the second change17:40
clarkber s/tagging/branching/17:40
*** dboik has quit IRC17:40
sdagueclarkb: will do17:40
mtreinishmriedem: heh, figures...17:40
sdaguethere are still some devstack patches in flight we want17:40
*** dboik has joined #openstack-qa17:41
*** dustins has joined #openstack-qa17:41
mtreinishsdague: if you're going through the branching motions can you make sure the steps are all documented on: https://wiki.openstack.org/wiki/QA/releases17:41
openstackgerritMorgan Fainberg proposed openstack-dev/devstack: Add response time to keystone access log  https://review.openstack.org/16404217:43
*** artom has quit IRC17:43
*** artom has joined #openstack-qa17:43
*** luqas has joined #openstack-qa17:44
*** artom has quit IRC17:44
*** artom has joined #openstack-qa17:44
*** yfried__ has joined #openstack-qa17:48
*** luqas has quit IRC17:48
*** Nic has quit IRC17:49
*** Nic1 has joined #openstack-qa17:49
*** jordanP has quit IRC17:50
*** e0ne has quit IRC17:51
openstackgerritMerged openstack-infra/elastic-recheck: Add query for bug 1446309  https://review.openstack.org/17549617:55
openstackbug 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 IRC17:59
*** openstackrecheck has joined #openstack-qa18:00
*** dboik_ has joined #openstack-qa18:01
*** shashankhegde has joined #openstack-qa18:03
*** tosky has quit IRC18:04
*** dboik has quit IRC18:05
*** artom_ has joined #openstack-qa18:08
*** artom has quit IRC18:08
*** artom_ has quit IRC18:10
*** artom has joined #openstack-qa18:10
mriedemsdague: mtreinish: fyi i'm going through and deleting old e-r queries for closed bugs18:13
mtreinishmriedem: I noticed, I was getting lots of emails about activity on a bunch of bugs18:14
mriedemyeah lots of stale gorp in LP18:14
mriedemdtroyer: does anything need to be done with this? https://bugs.launchpad.net/openstack-ci/+bug/128277218:17
openstackLaunchpad bug 1282772 in grenade "Grenade job fails when trying to add a window to screen" [Low,Confirmed]18:17
sdaguemriedem: no, that code should all be gone now18:19
*** dboik_ has quit IRC18:19
mriedemsdague: ok18:19
mriedembtw, 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 page18:20
*** ihrachyshka has quit IRC18:20
mriedemsince LP doesn't close/change status on duplicates when the main bug is fixed18:20
mtreinishmriedem: it can actually be tricky to figure it out from the api iirc18:21
mtreinishmriedem: http://git.openstack.org/cgit/openstack/tempest-lib/tree/tempest_lib/cmd/skip_tracker.py#n11018:21
mriedemwere heat tests disabled in stable/icehouse18:24
mriedem?18:24
mriedemor am i dreaming that?18:24
mtreinishmriedem: most were18:24
mtreinishthe dedicated heat slow job was just deleted on icehouse18:24
mtreinishthere are still a few api tests which weren't tagged as slow, so I think those might still run18:24
sdagueclarkb / fungi: so grenade should be good to go with stable/kilo at 67834d825aa0170c8801b55473fc4ef1392e612818:29
*** e0ne has joined #openstack-qa18:29
sdaguethe last devstack patch is in the gate18:29
sdagueso 37 minutes from us having that ready to tag18:30
sdagueto branch18:30
clarkbI am branching grenade now18:30
*** suneelb has quit IRC18:30
clarkbdone18:30
*** suneelb has joined #openstack-qa18:30
*** suneelb has quit IRC18:30
*** suneelb1 has joined #openstack-qa18:31
*** dboik has joined #openstack-qa18:33
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck: Remove old queries for "fixed" bugs  https://review.openstack.org/17552918:36
*** annashen has quit IRC18:37
openstackgerritSean Dague proposed openstack-infra/devstack-gate: only force off KEYSTONE_USE_WSGI up until kilo  https://review.openstack.org/17553118:43
*** nmagnezi_ has quit IRC18:43
openstackgerritSean Dague proposed openstack-dev/grenade: adjust branches for kilo  https://review.openstack.org/17553318:44
*** e0ne is now known as e0ne_18:46
sdagueok, 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 done18:47
*** suneelb has joined #openstack-qa18:47
*** suneelb1 has quit IRC18:48
*** dboik has quit IRC18:48
*** annashen has joined #openstack-qa18:50
openstackgerritDavid Kranz proposed openstack/tempest: Start a tempest config guide networking section  https://review.openstack.org/17250718:54
*** e0ne_ has quit IRC18:54
*** andreykurilin__ has joined #openstack-qa18:55
openstackgerritMatthew Treinish proposed openstack/tempest: Add configuration guide sections for more required options  https://review.openstack.org/17404718:55
*** dboik has joined #openstack-qa18:57
openstackgerritMatthew Treinish proposed openstack/tempest: Add config guide section on service feature enabled sections  https://review.openstack.org/17408618:58
*** e0ne has joined #openstack-qa18:58
openstackgerritMatthew Treinish proposed openstack/tempest: Add section to config guide for setting up services  https://review.openstack.org/17414918:58
openstackgerritMatthew Treinish proposed openstack/tempest: Add a paragraph on the use of tempest_roles option  https://review.openstack.org/17467518:58
openstackgerritMerged openstack/tempest: Revert "Test creation of server attached to created port"  https://review.openstack.org/17521918:58
mtreinishmriedem: ^^^18:58
mriedemfinally18:58
*** annashen has quit IRC19:03
*** annashen has joined #openstack-qa19:03
*** dboik_ has joined #openstack-qa19:13
*** dboik has quit IRC19:13
*** yfried__ is now known as yfried|afk19:14
openstackgerritMerged openstack-dev/devstack: Add a peak memory tracker to dstat  https://review.openstack.org/17191919:17
*** johnthetubaguy is now known as zz_johnthetubagu19:19
*** yfried|afk is now known as yfried__19:22
*** shashankhegde has quit IRC19:24
*** aix has quit IRC19:29
*** ajmiller_ has joined #openstack-qa19:29
*** ajmiller has quit IRC19:33
*** ajmiller_ is now known as ajmiller19:34
openstackgerritMatthew Treinish proposed openstack/tempest-lib: Migrate ssh.py from tempest  https://review.openstack.org/16651819:37
*** nmagnezi_ has joined #openstack-qa19:41
sdagueclarkb: for devstack, stable/kilo should be 14d1cf90ed9992f706c9149ba46e2eb984ff8b5819:42
clarkbsdague: ok making branch now19:43
clarkbsdague: done19:43
clarkbsdague: I think you can self approve the d-g changes now since they should have all the necessary +2s19:43
sdagueclarkb: ok, cool. Do we need project-config definitions ?19:44
clarkbsdague: not for grenade, but for general periodic testing, tempest/d-g tests on older branches, etc we do19:46
*** luqas has joined #openstack-qa19:46
sdaguehmmm... keystone-all is not shutting down correctly all the time19:49
*** suneelb has quit IRC19:49
*** suneelb1 has joined #openstack-qa19:50
*** luqas has quit IRC19:51
*** suneelb has joined #openstack-qa19:53
*** suneelb1 has quit IRC19:54
*** shashankhegde has joined #openstack-qa19:55
*** liusheng has quit IRC19:58
mtreinishsdague: light the morganfainberg signal?19:58
*** liusheng has joined #openstack-qa19:59
morganfainberghmm?19:59
*** andreykurilin__ has quit IRC19:59
*** Rockyg has joined #openstack-qa19:59
morganfainbergmtreinish, i should really stop lurking in channels i know people say my name in :P19:59
morganfainbergmtreinish, whats up?19:59
mtreinishmorganfainberg: heh, ~10mins ago sdague said: "hmmm... keystone-all is not shutting down correctly all the time"20:00
*** andreykurilin__ has joined #openstack-qa20:00
morganfainbergmtreinish, ah missed that in this window...20:00
morganfainberggot disconnected when replaying the backlog20:00
* morganfainberg can't wait for keystone-all to die...20:01
morganfainbergdie die die die die, dead dead, eventlet no more in keystone20:01
mtreinishmorganfainberg: is that this cycle? or +120:01
mtreinishthis cycle being L I guess :)20:02
morganfainbergmtreinish, M20:02
morganfainbergmtreinish, :(20:02
morganfainbergdeprecated in Kilo20:02
morganfainbergcan't remove it until M ... i want ti dead in liberty though20:02
morganfainbergbut.....20:02
morganfainbergalas20:02
mtreinishwell we can at least remove it from all the test jobs if you want20:02
*** annashen has quit IRC20:02
mtreinishthat's something worthwhile I think20:02
morganfainbergthere is only 1 job using it20:03
morganfainbergand until it dies20:03
mtreinishpostgres20:03
morganfainbergi don't want to risk breaking people20:03
morganfainbergbecause we suck at something and refactor things in a way that really make it not work.20:03
mtreinishmorganfainberg: sure add an eventlet job to the keystone queue, but we can drop it everywhere else20:03
morganfainbergwe did postgres because it was low cost to do so20:04
morganfainbergand didn't consume extra workers20:04
sdaguemorganfainberg: well, someone needs to actually work on wsgi upgrade20:04
morganfainbergsdague, yes we do.20:04
sdaguewhich, isn't a thing20:04
morganfainbergsdague, thankfully it really is pretty straight forward.20:04
sdaguean eventlet -> wsgi upgrade would be useful to do20:04
sdaguemorganfainberg: ok, please write the tests for it :)20:05
morganfainbergsdague, i can verbalize it, i just need to learn grenade-isms to do it20:05
mtreinishmorganfainberg: 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
sdaguewell, I just made the grenadisms hopefully a lot easier20:05
mtreinishmorganfainberg: and no one else needs an evenlet job20:05
morganfainbergsdague, yay!20:05
clarkbmtreinish: eh sort of20:05
clarkbmtreinish: we have had not keystone make keystone eventlet pretty terrible20:06
clarkbmtreinish: basically performance gets really bad and tests take twice as long20:06
morganfainbergmtreinish, 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
morganfainberginfra/test/etc20:06
clarkband there is zero cost in using it in the postgres job20:06
morganfainbergmeaning, i am staying out of sdague and jogo's ways20:06
morganfainbergclarkb, exactly20:07
morganfainbergclarkb, zero cost is the kind of cost i like20:07
mtreinishclarkb: well if we're killing keystone eventlet who cares about performance, and back then it wasn't even running on keystone changes iirc20:07
morganfainbergmtreinish, we care until it's dead.20:07
morganfainbergmtreinish, it is a valid deployment methodology20:07
sdagueclarkb: there is complexity cost in so few people knowning what's buried in that pg job20:08
mtreinishand if we want to keep it around in the postgres jobs maybe rename it postgres-keystone_eventlet-metadata20:08
mtreinishbecause that's really what it is20:08
morganfainbergbut in general, i'm fine with shuffling around the tests keystone deals with20:08
mtreinishand tbh postgres is the one thing no one actually cares about in that list20:08
sdagueso when stuff goes wrong, it's a whole lot of ugly stepchild configs20:08
clarkbmtreinish: sure I don't mind renaming it20:08
morganfainbergif it means pgsql becomes something else - and we grow an eventlet specific job, i'm cool with that20:08
clarkbsdague: yup that is an issue20:09
* morganfainberg again defers to smarter folks. 20:09
morganfainbergnotably i'll defer to infra types so we don't consume un-needed resources20:09
morganfainbergas 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-qa20:15
*** andreykurilin__ has quit IRC20:16
*** andreykurilin__ has joined #openstack-qa20:17
*** yfried__ is now known as yfried|afk20:20
*** achanda has quit IRC20:22
*** dpaterson has quit IRC20:22
*** dpaterson has joined #openstack-qa20:23
*** andreykurilin__ has quit IRC20:24
*** shardy is now known as shardy_z20:26
openstackrecheckopenstack/cinder change: https://review.openstack.org/175475 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error20:29
*** achanda has joined #openstack-qa20:35
*** kevinbenton has joined #openstack-qa20:43
*** kevinbenton has quit IRC20:44
*** kevinbenton has joined #openstack-qa20:44
*** dboik has joined #openstack-qa20:48
*** gfidente has quit IRC20:51
*** dboik_ has quit IRC20:51
*** e0ne has quit IRC20:55
*** paul_glass has joined #openstack-qa20:59
paul_glassare there any examples of tempest/tempest-lib tests using more than one tenant? like POST with tenant A, GET with tenant B?21:02
clarkbpaul_glass: I think there is a cross tenant vm connectivity test that would do operations with multiple tenants21:02
sdaguepaul_glass: git grep for alt_client inside tempest tree21:06
sdaguehttps://github.com/openstack/tempest/blob/master/tempest/api/compute/test_authorization.py#L175-L18021:07
paul_glasssdague: I see. so just instantiate a second client with different creds.21:08
openstackgerritMerged openstack/tempest: Fix test accounts unit test race with networks  https://review.openstack.org/17547021:20
*** cdent has quit IRC21:21
clarkbsdague: 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 gate21:25
*** jgrimm is now known as zz_jgrimm21:25
*** dboik has quit IRC21:26
*** jaypipes has quit IRC21:26
openstackrecheckopenstack-infra/devstack-gate change: https://review.openstack.org/175390 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/137694121:27
openstackLaunchpad 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-qa21:27
openstackgerritMerged openstack-infra/elastic-recheck: Remove old queries for "fixed" bugs  https://review.openstack.org/17552921:29
*** e0ne has joined #openstack-qa21:30
openstackgerritMatthew Treinish proposed openstack/tempest: Add logging of account allocation to accounts provider  https://review.openstack.org/17313621:30
openstackgerritMatthew Treinish proposed openstack/tempest: Rework get_network_from_name() function  https://review.openstack.org/17447321:30
openstackgerritMatthew Treinish proposed openstack-dev/devstack: Do Not Merge: Test test-accounts with roles  https://review.openstack.org/17311421:31
openstackgerritMatthew Treinish proposed openstack-dev/devstack: Do Not Merge: Test test-accounts with roles and networks  https://review.openstack.org/15851121:31
*** e0ne has quit IRC21:35
mtreinishsdague: btw, this was where I found /ips used in tempest21:37
mtreinishit looks like that method is called in a few places21:37
*** erlon has quit IRC21:41
dkranzmtreinish, 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 IRC21:46
*** dustins has quit IRC21:47
*** shashankhegde has quit IRC21:48
openstackgerritnithya-ganesan proposed openstack/tempest: Define validation_resources function for ssh validation  https://review.openstack.org/14922921:49
mtreinishdkranz: sure I can leave another comment on there21:52
mtreinishbut yeah having a yaml template doesn't make much sense there21:53
*** mriedem is now known as mriedem_away21:53
*** yamahata has joined #openstack-qa21:54
*** dpaterson has quit IRC21:54
*** yamamoto has joined #openstack-qa21:56
*** shashankhegde has joined #openstack-qa22:00
*** openstackrecheck has quit IRC22:00
*** openstackrecheck has joined #openstack-qa22:00
*** melwitt has quit IRC22:04
*** Swami has quit IRC22:07
*** dboik has quit IRC22:09
*** dboik has joined #openstack-qa22:11
*** nmagnezi_ has quit IRC22:17
openstackgerritMerged openstack/tempest: Adding port_admin_state_change option to config  https://review.openstack.org/16867922:24
openstackgerritMerged openstack/tempest: Apply a naming rule of GET to messaging client  https://review.openstack.org/17100922:25
*** bknudson has quit IRC22:26
*** jecarey has quit IRC22:28
*** achanda has quit IRC22:31
sdaguemtreinish: 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 here22:35
mordredaroo?22:35
mordredoh - I'm not sure it will help22:36
mordredI was just reading docs22:36
mordredand it seemed like a thing that would be worth _Trying_ to see what the difference was22:36
mordredbut then it turns out it's in the docs we publish but not the client lib22:36
mordredso I wasn't sure if that was a bug, intentional, or in which direction it was a bug if it was22:36
mordredi have no skin in the /ips game at the moment22:36
mtreinishsdague: oh did I just point you to a link and fail to paste it?22:37
*** jaypipes has joined #openstack-qa22:37
*** ChanServ sets mode: +o jaypipes22:37
openstackgerritMerged openstack/tempest-lib: Migrate ssh.py from tempest  https://review.openstack.org/16651822:37
sdaguemordred: so it just returns the addresses slice of the json struct you get with a GET on .../servers/ID22:38
sdaguewhich, actually, in the hpcloud case is going to be a little weird, because that's going to be a proxy call to neutron22:38
mtreinishsdague: 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
sdagueso double API calls22:38
sdaguewhich I guess you were getting with the show anyway22:39
mordredsdague: yah - so short story is ...22:39
mordredwe currently do /servers/detail to check status of server boots22:40
mordredbecause we sometimes have 100 servers in booting state22:40
mordredso just grabbing the whole thing in one go is better than 100 roundtrips to the API22:40
mordred(especially with API rate limiting) - but that's constructing WAY more data on the backend than we need22:40
mordredif we did a /servers to just get ID and boot status22:41
mordredthen we could detect if a server was in ready state22:41
sdagueand then the only other info you need is addresses?22:41
mordredand _then_ just do an /ips to get its ips which is the only thing we'd need22:41
mordredyeah22:41
sdagueok, makes sense22:41
*** Swami has joined #openstack-qa22:41
*** sabeen1 has joined #openstack-qa22:41
sdagueI can get that into novaclient this week if you think it would help22:42
mtreinishmordred: 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 already22:44
mordredsdague: neat! it's certainly worth a try22:45
mordredmtreinish: yes - we have the id - you're right22:45
mordredmtreinish: it's the status mapping we need from /servers22:45
*** wuhg has joined #openstack-qa22:47
*** luqas has joined #openstack-qa22:48
*** paul_glass has quit IRC22:52
*** luqas has quit IRC22:53
sdaguemordred: do you have a server or a server manager when you do this today?22:54
mordreduhm22:56
mordredsdague: server manager22:56
mordredsdague: client.nova_client.servers.list22:57
*** sabeen1 has quit IRC22:57
openstackgerritMatthew Treinish proposed openstack/tempest: Rework get_network_from_name() function  https://review.openstack.org/17447322:57
openstackgerritMatthew Treinish proposed openstack-dev/devstack: Do Not Merge: Test test-accounts with roles  https://review.openstack.org/17311422:58
openstackgerritMatthew Treinish proposed openstack-dev/devstack: Do Not Merge: Test test-accounts with roles and networks  https://review.openstack.org/15851122:58
*** rbak has quit IRC22:58
sdaguewell, I need to write tests, but the add is as simple as - https://review.openstack.org/17560323:02
*** annashen has joined #openstack-qa23:03
*** hemna is now known as hemnafk23:07
*** annashen has quit IRC23:08
*** achanda has joined #openstack-qa23:10
ianwsdague: 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 it23:10
sdagueianw: yep, dtroyer and I thought we should make sure that was in stable/kilo to avoid having to backport23:10
sdagueianw: cool, as soon as that is ready to go, I'd say we turn it on by default in devstack-gate for the relevant branches23:11
ianwsdague: 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, etc23:11
sdagueneat, sounds awesome23:12
*** dimtruck is now known as zz_dimtruck23:14
openstackgerritClark Boylan proposed openstack-infra/devstack-gate: DO NOT MERGE Try turning on ipv6  https://review.openstack.org/17560723:15
*** sabeen1 has joined #openstack-qa23:16
*** pc_m has quit IRC23:17
*** yamamoto has quit IRC23:18
*** achanda_ has joined #openstack-qa23:18
*** achanda_ has quit IRC23:18
*** achanda_ has joined #openstack-qa23:19
*** achanda has quit IRC23:19
openstackgerritIan Wienand proposed openstack-dev/devstack: Fix negated services with common prefix  https://review.openstack.org/17359323:23
openstackgerritIan Wienand proposed openstack-dev/devstack: Stop deleting token prefixes when processing ENABLED_SERVICES negations  https://review.openstack.org/17140223:25
openstackgerritIan Wienand proposed openstack-dev/devstack: Stop deleting token prefixes when processing ENABLED_SERVICES negations  https://review.openstack.org/17140223:26
*** dboik has quit IRC23:33
*** sabeen3 has joined #openstack-qa23:33
*** madhu_ak has joined #openstack-qa23:34
madhu_akhi 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.html23:35
madhu_akIf it is so, could you point me the link?, I would like to go through and understand the behavior23:36
*** sabeen1 has quit IRC23:37
*** shashankhegde has quit IRC23:37
*** Rockyg has quit IRC23:41
*** marun has quit IRC23:42
*** tochi has joined #openstack-qa23:42
*** david-ly_ has joined #openstack-qa23:44
*** jamielennox|away is now known as jamielennox23:45
*** david-ly_ is now known as david-lyle23:46
*** mtanino has quit IRC23:46
*** ajmiller has quit IRC23:51
*** yamamoto has joined #openstack-qa23:55
openstackgerritMatthew Treinish proposed openstack/tempest: Rework get_network_from_name() function  https://review.openstack.org/17447323:57
*** openstackgerrit has quit IRC23:58
*** openstackgerrit has joined #openstack-qa23:58

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!