Friday, 2014-10-17

openstackgerritTushar Kalra proposed a change to openstack/tempest: Quota tests don't take into account a limit of -1  https://review.openstack.org/10914800:01
*** ramishra has quit IRC00:02
*** dmorita has joined #openstack-qa00:30
*** suneelb has quit IRC00:34
openstackgerritAdam Gandelman proposed a change to openstack-dev/devstack: Move generation of baremetal VMs to lib/ironic  https://review.openstack.org/12909700:43
openstackgerritAdam Gandelman proposed a change to openstack-dev/devstack: Make screen_service() useful for more than services  https://review.openstack.org/12909800:43
openstackgerritAdam Gandelman proposed a change to openstack-dev/devstack: Spawn console log monitors for Ironic VM nodes  https://review.openstack.org/12909900:43
*** marun has quit IRC00:52
achandaI'm trying to find out how nova sample config is generated during check-tempest-dsvm-full00:57
achandaI'd expect a tox -e genconfig call somewhere00:57
achandawhich I did not see anywhere00:58
*** ramishra has joined #openstack-qa00:58
achandaso how is that file generated?00:59
*** ramishra has quit IRC01:02
*** salv-orlando has quit IRC01:03
*** X019 has quit IRC01:06
*** topshare has joined #openstack-qa01:17
*** yamamoto_ has joined #openstack-qa01:18
openstackgerritYAMAMOTO Takashi proposed a change to openstack-dev/devstack: ofagent: Support physical_interface_mappings  https://review.openstack.org/12118901:18
*** X019 has joined #openstack-qa01:19
*** mriedem has joined #openstack-qa01:24
*** topshare has quit IRC01:30
*** X019 has quit IRC01:30
*** nosnos has joined #openstack-qa01:32
*** Longgeek has joined #openstack-qa01:37
openstackgerritKyle Mestery proposed a change to openstack-dev/devstack: Update the OpenDaylight support to the Helium release  https://review.openstack.org/12893701:40
*** Longgeek has quit IRC01:42
*** gmann has quit IRC01:43
*** gmann has joined #openstack-qa01:44
*** Yogi2 has joined #openstack-qa01:49
*** wpf has joined #openstack-qa01:57
*** ramishra has joined #openstack-qa01:59
*** weshay has quit IRC02:00
*** chandankumar has joined #openstack-qa02:02
*** ramishra has quit IRC02:03
*** Daisy has joined #openstack-qa02:03
*** aimon has quit IRC02:19
*** aimon has joined #openstack-qa02:21
*** Yogi2 has quit IRC02:23
*** achanda_ has joined #openstack-qa02:43
*** achanda__ has joined #openstack-qa02:43
*** achanda has quit IRC02:46
*** achanda_ has quit IRC02:47
*** harlowja is now known as harlowja_away02:47
*** ramishra has joined #openstack-qa02:52
*** ramishra_ has joined #openstack-qa02:58
*** ramishra has quit IRC03:01
*** ramishra_ has quit IRC03:02
*** ramishra has joined #openstack-qa03:02
*** ramishra has quit IRC03:07
openstackgerritChristopher Yeoh proposed a change to openstack/tempest: Fix and re-enable test_disassociate_not_associated_floating_ip  https://review.openstack.org/12432303:15
*** aimon has quit IRC03:17
*** yamahata has joined #openstack-qa03:18
*** Longgeek has joined #openstack-qa03:22
*** ramishra has joined #openstack-qa03:22
*** gmann has quit IRC03:45
*** marun has joined #openstack-qa03:49
*** Daisy has quit IRC03:51
*** mmedvede has quit IRC03:51
*** aimon has joined #openstack-qa03:52
*** vaidy has joined #openstack-qa03:55
*** gmann has joined #openstack-qa03:56
*** vaidy has quit IRC03:57
*** mriedem has quit IRC04:03
*** adu has quit IRC04:04
*** chandankumar has quit IRC04:05
*** chandankumar has joined #openstack-qa04:10
*** swamireddy has joined #openstack-qa04:11
*** Madan has joined #openstack-qa04:15
*** adu has joined #openstack-qa04:16
*** ramishra has quit IRC04:17
*** ramishra has joined #openstack-qa04:20
*** ramishra has quit IRC04:20
*** neeti has joined #openstack-qa04:30
*** Poornima has joined #openstack-qa04:31
*** swamireddy has joined #openstack-qa04:32
*** pcrews has quit IRC04:42
*** dims_ has quit IRC04:48
*** dims_ has joined #openstack-qa04:48
*** ramishra has joined #openstack-qa04:50
*** dims_ has quit IRC04:53
*** achanda has joined #openstack-qa04:59
*** achanda__ has quit IRC05:02
*** chandan_kumar has joined #openstack-qa05:02
*** chandankumar has quit IRC05:06
openstackgerritprabhu murthy proposed a change to openstack/tempest: Added neutron cli test case for vpnaas  https://review.openstack.org/12341105:09
*** dimtruck is now known as zz_dimtruck05:14
*** zz_dimtruck is now known as dimtruck05:18
openstackgerritprabhu murthy proposed a change to openstack/tempest: Added neutron cli test case for vpnaas  https://review.openstack.org/12341105:18
*** ramishra has quit IRC05:19
openstackgerritprabhu murthy proposed a change to openstack/tempest: Added neutron cli test case for fwass  https://review.openstack.org/12363305:22
*** Daisy has joined #openstack-qa05:24
*** chandan_kumar has quit IRC05:28
*** chandankumar has joined #openstack-qa05:28
*** ramishra has joined #openstack-qa05:31
*** chandan_kumar has joined #openstack-qa05:41
*** chandankumar has quit IRC05:42
*** chandan_kumar has quit IRC05:47
*** marun has quit IRC05:48
*** afazekas has joined #openstack-qa05:50
openstackgerritAbhishek G M proposed a change to openstack/tempest: Test to verify  shared attribute of network  https://review.openstack.org/12499806:01
openstackgerritAbhishek G M proposed a change to openstack/tempest: Test update shared attribute  https://review.openstack.org/12913406:01
*** k4n0 has joined #openstack-qa06:09
*** SridharG has joined #openstack-qa06:13
*** achanda has quit IRC06:15
*** swamireddy has quit IRC06:15
*** achanda has joined #openstack-qa06:16
*** achanda has quit IRC06:16
*** marun has joined #openstack-qa06:26
*** mpaolino has joined #openstack-qa06:27
*** marun has quit IRC06:31
*** marun has joined #openstack-qa06:33
*** ramishra has quit IRC06:33
*** adu has quit IRC06:34
*** e0ne has joined #openstack-qa06:38
*** ramishra has joined #openstack-qa06:39
*** e0ne has quit IRC06:43
*** chandankumar has joined #openstack-qa06:44
*** dimtruck is now known as zz_dimtruck06:44
*** e0ne has joined #openstack-qa06:47
*** e0ne has quit IRC06:48
*** e0ne has joined #openstack-qa06:52
*** e0ne has quit IRC06:54
*** qba73 has joined #openstack-qa07:14
*** gfidente has joined #openstack-qa07:23
*** gfidente^2nd has joined #openstack-qa07:23
*** yamamoto_ has quit IRC07:23
*** luqas has joined #openstack-qa07:33
*** jlanoux has joined #openstack-qa07:34
*** salv-orlando has joined #openstack-qa07:36
*** markus_z has joined #openstack-qa07:38
*** ramishra has quit IRC07:41
*** gfidente^2nd has quit IRC07:46
*** jordanP has joined #openstack-qa07:51
*** e0ne has joined #openstack-qa07:52
*** jpich has joined #openstack-qa07:55
*** wuhg has joined #openstack-qa08:04
*** yamamoto_ has joined #openstack-qa08:04
*** Daisy has quit IRC08:07
*** ramishra has joined #openstack-qa08:18
*** jlibosva has joined #openstack-qa08:19
*** aix has joined #openstack-qa08:19
*** ramishra_ has joined #openstack-qa08:19
*** ramishra has quit IRC08:20
*** yamamoto_ has quit IRC08:26
*** dmellado_ has joined #openstack-qa08:26
*** vrovachev has joined #openstack-qa08:26
*** cdent has joined #openstack-qa08:27
*** gszasz has joined #openstack-qa08:30
*** yamamoto_ has joined #openstack-qa08:31
*** jvrbanac has quit IRC08:35
*** yamamoto_ has quit IRC08:36
*** wendar_ has joined #openstack-qa08:38
*** jlibosva has quit IRC08:38
*** jlibosva has joined #openstack-qa08:38
*** Longgeek has quit IRC08:40
*** mpaolino has quit IRC08:40
*** wendar has quit IRC08:40
*** sergsh has joined #openstack-qa08:40
*** sileht has quit IRC08:41
*** tkammer has joined #openstack-qa08:41
*** jvrbanac has joined #openstack-qa08:41
*** Longgeek has joined #openstack-qa08:41
*** sileht has joined #openstack-qa08:42
*** ramishra_ has quit IRC08:42
*** jlibosva has quit IRC08:42
*** jlibosva has joined #openstack-qa08:42
*** jlibosva has quit IRC08:43
*** jlibosva has joined #openstack-qa08:43
*** ramishra_ has joined #openstack-qa08:43
*** jlibosva has quit IRC08:45
*** jlibosva has joined #openstack-qa08:45
openstackgerritShang Yong proposed a change to openstack/tempest: Use "ip" first instead of "ifconfig" to get MAC  https://review.openstack.org/12864108:51
*** zz_johnthetubagu is now known as johnthetubaguy08:53
*** gmann has quit IRC09:02
salv-orlandoI hate to ask this but can you guys have a quick look at https://review.openstack.org/#/c/126908 and see if it can be approve?09:05
salv-orlandoBug 1357055 has become the #1 gate offender and this logging might enable us to root cause it09:05
salv-orlandotia!09:06
*** dmellado_ has quit IRC09:08
*** mpaolino has joined #openstack-qa09:10
openstackgerritSalvatore Orlando proposed a change to openstack-dev/devstack: Remove no_wait ovs flag from vmw_nsx 3rd party lib  https://review.openstack.org/12862609:12
*** ramishra_ has quit IRC09:14
*** flaper87|afk is now known as flaper8709:16
*** tkammer has quit IRC09:17
*** yamahata has quit IRC09:18
*** ramishra_ has joined #openstack-qa09:18
chandankumarjordanP, hello09:21
jordanPchandankumar, hi09:22
chandankumarjordanP, http://fpaste.org/142759/53779114/09:23
chandankumarjordanP, some where the client name is volume_client and some where is volumes_client09:23
chandankumarit creates a confusion,09:24
jordanPyup, I think i saw a review to fix this already09:24
jordanPlemme see09:24
chandankumarjordanP, gmann has sent for volume types09:24
jordanPah09:24
chandankumarjordanP, let me send a seperate patch for this09:24
jordanPokay, great09:25
jordanPwill review09:25
chandankumarjordanP, Thanks :)09:25
*** Daisy has joined #openstack-qa09:26
chandankumarjordanP, do i file a bug for the same?09:26
*** tosky has joined #openstack-qa09:26
jordanPbah, imo there's no need, it's not a "real" bug09:26
chandankumarok09:27
jordanPthough  very much welcomed09:27
*** dims_ has joined #openstack-qa09:27
*** dims_ has quit IRC09:32
*** mkollaro has joined #openstack-qa09:33
*** mpaolino has quit IRC09:38
*** ramishra_ has quit IRC09:39
*** flaper87 is now known as flaper87|afk09:50
*** Daisy has quit IRC09:50
*** dims_ has joined #openstack-qa10:02
openstackgerritAbhishek G M proposed a change to openstack/tempest: Test to verify  shared attribute of network  https://review.openstack.org/12499810:09
*** qba73 has quit IRC10:10
*** qba73 has joined #openstack-qa10:13
openstackgerritAlistair Coles proposed a change to openstack-dev/devstack: Add swift user and project in non-default domain  https://review.openstack.org/12888810:16
openstackgerritMate Lakat proposed a change to openstack-dev/devstack: XenAPI: fix for unsupp. associative arrays  https://review.openstack.org/12760310:27
*** mkollaro has quit IRC10:31
openstackgerritchandan kumar proposed a change to openstack/tempest: enable cinder v2 api for volumetype test  https://review.openstack.org/12621310:32
chandankumarjordanP, Updated a new patchset https://review.openstack.org/#/c/126213/ Please have a look10:34
*** Albert_13 has joined #openstack-qa10:35
*** Albert__13 has joined #openstack-qa10:35
*** johnthetubaguy is now known as zz_johnthetubagu10:43
*** dmellado_ has joined #openstack-qa10:50
*** Albert__13 has quit IRC10:53
Albert_13Hi guys, could someone take a few minutes to review this patch: https://review.openstack.org/#/c/126849/10:53
Albert_13thanks10:53
openstackgerritShang Yong proposed a change to openstack/tempest: Use "ip" first instead of "ifconfig" to get MAC  https://review.openstack.org/12864111:00
openstackgerritA change was merged to openstack-dev/devstack: XenAPI: Always update apt sources  https://review.openstack.org/12871811:02
openstackgerritA change was merged to openstack-dev/devstack: Use $() instead of ``  https://review.openstack.org/12843311:03
*** X019 has joined #openstack-qa11:10
*** dims_ has quit IRC11:14
*** dims_ has joined #openstack-qa11:15
openstackgerritMate Lakat proposed a change to openstack-dev/devstack: XenAPI: workaround for unsupp. associative arrays  https://review.openstack.org/12760311:17
openstackgerritMate Lakat proposed a change to openstack-dev/devstack: XenAPI: move Neutron plugin install to domU  https://review.openstack.org/12920811:17
openstackgerritMate Lakat proposed a change to openstack-dev/devstack: XenAPI: Remove non-used functions  https://review.openstack.org/12920911:17
*** dims_ has quit IRC11:18
*** dims_ has joined #openstack-qa11:19
*** dims_ has quit IRC11:19
*** dimsum_ has joined #openstack-qa11:19
openstackgerritMate Lakat proposed a change to openstack-dev/devstack: XenAPI: Always update proxy settings  https://review.openstack.org/12921111:21
*** yamamoto has joined #openstack-qa11:38
*** zz_johnthetubagu is now known as johnthetubaguy11:42
*** cdent has quit IRC11:44
chandankumarjordanP, hello11:47
jordanPre11:47
chandankumarjordanP, i have one confusion11:47
chandankumarhttps://github.com/openstack/tempest/blob/master/tempest/api/volume/admin/test_volume_quotas.py#L8311:48
chandankumarhttps://github.com/openstack/tempest/blob/master/tempest/api/volume/admin/test_volume_quotas.py#L9011:48
chandankumarin both place , we are getting quota usage11:49
chandankumarbut at line no: 9211:49
*** mwagner_lap has quit IRC11:49
chandankumarquota_usage['volumes']['in_use'] is incremented by 111:49
*** rfolco has joined #openstack-qa11:49
chandankumarwhy?11:50
chandankumari have modified the code as per cinder v2 api, but at this line , it is failing11:50
jordanPsec11:50
jordanPchandankumar, I don't know. You should print the value of quota_usage and new_quota_usage to see what is going on11:52
chandankumarjordanP, both is giving 011:53
chandankumarjordanP, i am doing that11:53
jordanPhumm as if quota were disabled...11:54
jordanPthere might be a bug in cinder11:54
chandankumarjordanP, checking that11:55
*** jordan__ has joined #openstack-qa11:55
jordanPchandankumar, in the v1 case, quota_usage is 0 but new_quota_usage is *not* zero ?11:55
*** jordan__ has quit IRC11:56
chandankumaryes11:56
chandankumarjordanP, let me try things through cli11:57
chandankumarand by using python client11:57
jordanPokay11:57
chandankumarjordanP, https://review.openstack.org/126213 Please review it when you are free.11:57
*** Madan has quit IRC12:00
*** mkollaro has joined #openstack-qa12:04
*** mmedvede has joined #openstack-qa12:06
*** eglynn-office is now known as eglynn-lunch12:07
*** weshay has joined #openstack-qa12:13
*** yamamoto has quit IRC12:19
*** aysyd has joined #openstack-qa12:20
*** dmorita has quit IRC12:25
*** dimsum_ has quit IRC12:28
*** dimsum_ has joined #openstack-qa12:28
*** Poornima has quit IRC12:29
*** mestery_ has quit IRC12:30
*** chandankumar has quit IRC12:32
*** eglynn-lunch is now known as eglynn-office12:37
*** salv-orlando has quit IRC12:38
*** andreaf has quit IRC12:38
*** andreaf has joined #openstack-qa12:38
*** salv-orlando has joined #openstack-qa12:39
*** nosnos has quit IRC12:41
*** nosnos has joined #openstack-qa12:42
*** KurtMartin has quit IRC12:44
*** KurtMartin has joined #openstack-qa12:45
*** nosnos has quit IRC12:46
*** cdent has joined #openstack-qa12:46
*** mestery has joined #openstack-qa12:49
*** aysyd has quit IRC12:50
*** ajo_ has joined #openstack-qa12:54
*** k4n0 has quit IRC12:55
*** ajo_ has quit IRC12:55
*** gszasz has quit IRC12:57
*** ajo_ has joined #openstack-qa12:57
*** Yogi has joined #openstack-qa12:57
*** flaper87|afk is now known as flaper8712:57
*** ramishra has joined #openstack-qa12:59
*** mpaolino has joined #openstack-qa12:59
*** mpaolino has quit IRC13:04
*** openstackgerrit has quit IRC13:19
*** ajo_ has quit IRC13:19
*** openstackgerrit has joined #openstack-qa13:19
*** openstackrecheck has quit IRC13:28
*** openstackrecheck has joined #openstack-qa13:29
*** flaper87 is now known as flaper87|afk13:30
*** ramishra has quit IRC13:32
*** marun has quit IRC13:34
*** chandankumar has joined #openstack-qa13:37
*** neeti has quit IRC13:39
*** mriedem has joined #openstack-qa13:40
*** jecarey_ has joined #openstack-qa13:41
*** zz_dimtruck is now known as dimtruck13:42
*** ramishra has joined #openstack-qa13:42
*** rfolco has quit IRC13:44
*** chandankumar has quit IRC13:44
*** adu has joined #openstack-qa13:46
*** dpaterson has joined #openstack-qa13:47
mtreinishandreaf: around?13:48
andreafmtreinish: yes - hi13:49
mtreinishdid you see my pings yesterday about the test accounts stuff?13:49
mtreinishandreaf: it looks like we broke the periodic non-isolated neutron job...13:52
mtreinishandreaf: http://logs.openstack.org/periodic-qa/periodic-tempest-dsvm-neutron-full-non-isolated-master/cac5129/13:52
*** adu has quit IRC13:53
andreafmtreinish: some network cleanup missing?13:54
andreafmtreinish: was it passing until the test-accounts code landed?13:54
*** adu has joined #openstack-qa13:54
mtreinishandreaf: yeah I think so, it looks like it started failing on the 15th13:55
*** mdbooth has quit IRC13:56
andreafmtreinish: it was passing... so it might be a combination of that and something that merged in the meanwhile13:56
andreafmtreinish: after the last check experimental was done13:56
*** mdbooth has joined #openstack-qa13:56
mtreinishok, I thought we checked all of that beforehand, sigh it's always something13:57
*** dimtruck is now known as zz_dimtruck13:57
andreafyes I thought we checked everything as well13:57
andreafbut something may have merged in between, it's not a gate job13:58
andreafI can look at it, only not right now, I'll try later today13:58
jlanouxmtreinish: andreaf actually I think it's coming from my test13:58
jlanouxit's not cleaning up behind itself13:58
andreafjlanoux: could you propose a fix and run check experimental on it, so we can see if it fixes the issue?13:59
jlanouxso in isolation, that's working fine, but not in non-isolated env13:59
jlanouxandreaf: yes, I'll do that13:59
andreafjlanoux: great thanks14:00
mtreinishjlanoux: oh, cool thanks for handling it14:00
*** flaper87|afk is now known as flaper8714:04
andreafmtreinish: non-isolated (i.e. pre-provisioned accounts) can run in parallel as well now, but it will skip all admin tests in that case, so we cannot use it in check yet14:04
mtreinishandreaf: and not with neutron either14:05
andreafmtreinish: if we could have pre-provisioned admin accounts we could keep the code tested in the gate as well - it's a bit against the reason for which the preprovisioned accounts are needed, but it would allows us to avoid the code bit-rots14:05
mtreinishandreaf: yeah, it's something I think we should migrate to using on at least one job once we start working on part 214:06
*** adu has quit IRC14:07
*** mpaolino has joined #openstack-qa14:08
andreafmtreinish: yep - and I'd like to do a similar thing for testing with keystone v3 once the bp is complete714:08
mtreinishheh, well I'm sure morganfainberg would say that we should move all the jobs to keystone v3 :)14:09
mtreinishbut yeah I agree we should run that in a config too14:09
*** bnemec is now known as beekneemech14:12
*** dansmith is now known as superdan14:12
*** ramishra has quit IRC14:12
*** wuhg has quit IRC14:13
*** Yogi has quit IRC14:26
*** zz_dimtruck is now known as dimtruck14:27
*** yfried|afk has quit IRC14:28
openstackgerritA change was merged to openstack/tempest: Log identifiers of resources being cleaned up  https://review.openstack.org/12690814:29
mtreinishsalv-orlando: ^^^ so you think that bug is tempest missing a cleanup somewhere which is interfering with a test down the road?14:30
*** rwsu has joined #openstack-qa14:31
*** morganfainberg has quit IRC14:32
*** morganfainberg has joined #openstack-qa14:33
*** mkollaro has quit IRC14:38
*** dustins has joined #openstack-qa14:39
*** Yogi has joined #openstack-qa14:39
*** dustins has quit IRC14:40
*** mkollaro has joined #openstack-qa14:40
*** dustins has joined #openstack-qa14:40
mriedemmtreinish: hey i think this is the thing you were referring to yesterday about possibly blowing up the allow_tenant_isolation=False test runs? https://review.openstack.org/#/c/107685/14:42
mriedemmtreinish: i think we've confirmed that internally, we had a lot of test failures with allow_tenant_isolation=False this week,14:42
mriedemwas a bug opened for the periodic jobs failing? if not i can create one14:43
mtreinishmriedem: I don't think a bug was opened yet14:44
mriedemmtreinish: ok, i'll get one opened14:44
mtreinishyeah that was the patch I was talking about14:44
*** nmagnezi has joined #openstack-qa14:44
*** jaypipes is now known as leakypipes14:44
mriedemfound it via this https://bugs.launchpad.net/tempest/+bug/132906514:44
*** nmagnezi has quit IRC14:44
*** cdent has quit IRC14:45
mtreinishmriedem: we were actually just talking about this issue, jlanoux thinks the cause is a test not cleaning up properly14:46
mtreinishmriedem: oh, I didn't realize there was a bug for that. I would have just figured that as a follow up cleanup :)14:46
mriedemmtreinish: that bug is old14:47
mriedembut i saw the update last night14:47
mriedemwhen looking at the plethora of internal failures and trying to figure out what we're doing differently internally and what's changed recently upstream, and remembered our chat14:48
*** achanda has joined #openstack-qa14:49
*** achanda_ has joined #openstack-qa14:50
mtreinishmriedem: yeah, I feel it's a big enough issue to hold up the tempest-2 tag until we have a fix14:52
*** achanda has quit IRC14:54
openstackgerritMatthew Treinish proposed a change to openstack-infra/subunit2sql: Release subunit2sql 0.2  https://review.openstack.org/12929014:54
*** cdent has joined #openstack-qa14:54
*** achanda_ has quit IRC14:55
*** pcrews has joined #openstack-qa14:57
*** achanda has joined #openstack-qa14:57
openstackgerritJoseph Lanoux proposed a change to openstack/tempest: Add cleanup to test_verify_multiple_nics_order  https://review.openstack.org/12929115:00
mtreinishjlanoux: ^^^ is that the fix for the non-isolated job?15:01
jlanouxmtreinish: yes15:01
mtreinishok cool15:02
mtreinishmriedem: ^^^15:02
mriedemmtreinish: jlanoux: yeah that's definitely the job that wasn't cleaning up we were seeing internally15:03
mriedemreviewed15:05
mriedemjlanoux: ^15:05
*** achanda has quit IRC15:05
* mriedem is on the fast path to tempest core now!15:05
openstackgerritA change was merged to openstack-infra/subunit2sql: Release subunit2sql 0.2  https://review.openstack.org/12929015:07
*** jlibosva has quit IRC15:07
*** sergsh has left #openstack-qa15:09
*** jlibosva has joined #openstack-qa15:10
*** jlibosva has quit IRC15:13
*** flaper87 is now known as flaper87|afk15:13
*** yamahata has joined #openstack-qa15:18
*** e0ne has quit IRC15:25
*** aimon has quit IRC15:29
*** SridharG has left #openstack-qa15:30
openstackgerritJoseph Lanoux proposed a change to openstack/tempest: Add cleanup to test_verify_multiple_nics_order  https://review.openstack.org/12929115:31
jlanouxmriedem: Thanks! Done.15:33
*** mlavalle has joined #openstack-qa15:35
mriedemjlanoux: it's still possibly a bit leaky, so would like to know what mtreinish thinks15:36
*** mkollaro has quit IRC15:37
mtreinishmriedem: hmm, yeah it should probably be a single add cleanup after each create call15:38
mriedemok, commented15:39
mriedemjlanoux: should be an easy update15:39
mtreinishandreaf: btw, did you want to close out the test-accounts bp?15:42
mtreinishwow, gerrit formatting failed on my review comment there...15:49
*** qba73 has quit IRC15:52
*** aimon has joined #openstack-qa15:52
*** aimon has quit IRC15:56
openstackgerritJoseph Lanoux proposed a change to openstack/tempest: Add cleanup to test_verify_multiple_nics_order  https://review.openstack.org/12929115:57
*** mpaolino has quit IRC15:58
jlanouxmriedem: mtreinish done!15:59
*** e0ne has joined #openstack-qa15:59
mriedemyup, looking15:59
*** aimon has joined #openstack-qa15:59
*** e0ne has quit IRC15:59
mriedemjlanoux: looks good, thanks16:00
mriedemandreaf: dkranz: ^ simple fix to get tests working w/o tenant isolation again, then the '2' tag can be created16:01
*** afazekas has quit IRC16:01
jlanouxmriedem: np - thanks for the review16:02
*** e0ne has joined #openstack-qa16:02
*** chandankumar has joined #openstack-qa16:02
*** bknudson has joined #openstack-qa16:03
mtreinishmriedem, jlanoux: I'm going to hold off on the +2 until I see the experimental job gets back, but it LGTM16:07
*** e0ne has quit IRC16:07
*** jlanoux has quit IRC16:07
*** safchain has joined #openstack-qa16:09
openstackgerritA change was merged to openstack-dev/devstack: Update the OpenDaylight support to the Helium release  https://review.openstack.org/12893716:09
openstackgerritA change was merged to openstack-dev/devstack: ofagent: Support physical_interface_mappings  https://review.openstack.org/12118916:09
*** markus_z has quit IRC16:12
*** e0ne has joined #openstack-qa16:13
*** e0ne has quit IRC16:14
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Remove internal bash8  https://review.openstack.org/12931316:14
*** marun has joined #openstack-qa16:16
*** dmellado_ has quit IRC16:18
*** Yogi has quit IRC16:19
*** jpich has quit IRC16:22
*** Yogi has joined #openstack-qa16:26
*** Albert_13 has quit IRC16:28
*** leakypipes is now known as leaky-afk16:29
*** jhenner has quit IRC16:33
*** chandankumar has quit IRC16:39
*** Longgeek has quit IRC16:44
*** johnthetubaguy is now known as zz_johnthetubagu16:48
*** luqas has quit IRC16:50
*** eglynn-office is now known as eglynn-officeafk16:50
openstackgerritA change was merged to openstack/tempest: Remove OfficialClient dependency from HACKING.rst  https://review.openstack.org/12621416:51
*** chandankumar has joined #openstack-qa16:54
openstackgerritJohn Davidge proposed a change to openstack-dev/devstack: Add IPv6 support for tenant data network  https://review.openstack.org/8798716:57
*** suneelb has joined #openstack-qa17:03
*** jordanP has quit IRC17:04
*** e0ne has joined #openstack-qa17:10
*** jogo is now known as flashgordon17:17
*** comstud is now known as bearhands17:23
*** achanda has joined #openstack-qa17:26
*** dustins has quit IRC17:27
*** pcrews has quit IRC17:27
*** morganfainberg is now known as captainmorgan17:29
*** dustins has joined #openstack-qa17:29
*** e0ne has quit IRC17:39
*** mriedem has quit IRC17:41
*** safchain has quit IRC17:43
openstackgerritAdam Gandelman proposed a change to openstack-infra/devstack-gate: Set SWIFT_TEMPURL_KEY for ironic agent_ssh job  https://review.openstack.org/12934117:47
*** harlowja_away is now known as harlowja17:49
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Remove internal bash8  https://review.openstack.org/12931317:50
*** yamahata has quit IRC17:51
*** Yogi has quit IRC17:52
*** hemnass is now known as hemna17:54
*** mriedem has joined #openstack-qa17:55
openstackgerritAdam Gandelman proposed a change to openstack-dev/devstack: Make screen_service() useful for more than services  https://review.openstack.org/12909817:56
openstackgerritAdam Gandelman proposed a change to openstack-dev/devstack: Spawn console log monitors for Ironic VM nodes  https://review.openstack.org/12909917:56
openstackgerritAdam Gandelman proposed a change to openstack-dev/devstack: Move generation of baremetal VM names to lib/ironic  https://review.openstack.org/12909717:56
*** mriedem has quit IRC17:56
*** achanda has quit IRC17:57
*** Madan has joined #openstack-qa17:57
*** mriedem has joined #openstack-qa17:58
flashgordondtroyer: is https://review.openstack.org/#/c/128464/ ready ?18:02
dtroyerflashgordon: I think it'll need https://review.openstack.org/129334 to land first18:03
dtroyerthat resets the devstack defaults to stable/juno repos18:04
dtroyerwait, no wrong one18:04
dtroyerthat should be good now18:05
*** mwagner_lap has joined #openstack-qa18:06
flashgordondtroyer: cool, approved18:09
*** achanda has joined #openstack-qa18:10
flashgordondtroyer: another grenade patch series  https://review.openstack.org/#/c/128734/118:12
*** cdent has quit IRC18:13
dtroyerflashgordon: yeah, I started on those last night, prelimiary is good but I want to look at a couple of things yet18:13
flashgordondtroyer: cool18:14
*** dustins has quit IRC18:17
openstackgerritA change was merged to openstack-infra/elastic-recheck: restore query for InterpreterNotFound  https://review.openstack.org/12898418:19
openstackgerritpatrick-crews proposed a change to openstack-infra/elastic-recheck: Adding query namespace to elastic-recheck  https://review.openstack.org/10195818:19
*** dustins has joined #openstack-qa18:21
*** Yogi has joined #openstack-qa18:22
*** dustins has quit IRC18:23
*** pcrews has joined #openstack-qa18:25
*** openstackrecheck has quit IRC18:27
*** openstackrecheck has joined #openstack-qa18:27
*** mpaolino has joined #openstack-qa18:29
*** sabeen1 has quit IRC18:30
adam_gdtroyer, flashgordon re the devstack sync, the only two parts that initially looked divergent were changes to install_package and screen_it, but both should be backward compatible with grenade18:36
mriedemmtreinish: fyi https://jenkins05.openstack.org/job/check-tempest-dsvm-full-non-isolated/4/18:41
mriedemlooks like https://review.openstack.org/#/c/129291/ is working18:42
mriedemthe neutron non-isolated one is still running but so far so good18:42
mriedemthe other failures in the exp queue are on non-voting jobs like ceph and cells, things we know don't work yet18:42
mriedemmtreinish: dkranz: i'm going to update that patch based on dkranz's comment since joseph appears to be gone now18:42
*** Madan has quit IRC18:43
*** leaky-afk is now known as leakypipes18:44
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Add cleanup to test_verify_multiple_nics_order  https://review.openstack.org/12929118:50
*** dustins has joined #openstack-qa18:53
mtreinishmriedem: yeah it looks like that was the issue18:55
mtreinishmriedem: +2 on the patch18:59
mriedem\o.19:00
leakypipesOpenStack Bootstrapping Hour currently on air!19:04
leakypipeshttps://etherpad.openstack.org/p/obh-nova-rpc-layer19:04
leakypipesLearn about Nova RPC Layer19:04
*** hogepodge has quit IRC19:10
openstackgerritA change was merged to openstack-dev/grenade: Add within-kilo and update within-master  https://review.openstack.org/12846419:16
*** tosky has quit IRC19:17
*** mpaolino has quit IRC19:21
dtroyermtreinish: devstack stable/juno review is up:  https://review.openstack.org/#/c/129334/19:29
*** pcrews has quit IRC19:31
mtreinishdtroyer: cool, lgtm19:34
mtreinishdtroyer: should I fast approve it?19:36
dtroyersure, nothing depends on it (yet)19:38
mtreinishok, approved19:39
mtreinishwell it'll be good to get it landed so we can start doing the grenade switchover19:39
mtreinishflashgordon: ^^^19:40
dtroyeryeah…I'm thinking we should wait on pushing adam_g's functions reviews in until after stable/juno is cut until they have some time on them, then backport to stable19:41
mtreinishyeah that's a good call19:42
*** sabeen has joined #openstack-qa19:44
*** e0ne has joined #openstack-qa19:44
dtroyerfor Grenade, I think we want 800788d5d71f18bc6c62a1764ced4cb6ddc00288 for stable/juno?  It's the last commit before within-kilo was added19:47
mtreinishyeah, that looks right19:48
*** dustins has quit IRC19:57
mtreinishmriedem: hmm it looks like it's failing neutron: http://logs.openstack.org/91/129291/4/check/check-tempest-dsvm-neutron-full/363c7c3/console.html20:07
mtreinishoh, because the server is still using the ip when we go to delete the subnet20:08
mtreinishit worked before because lifo20:08
mriedemmtreinish: oh i was thinking bug 135705520:09
mriedemdamn it....20:09
mriedemmtreinish: so i guess i'll restore the previous patchset and add a note why we have to explicitly cleanup the server in the test case20:10
mriedemsound ok?20:10
*** ajo_ has joined #openstack-qa20:10
mtreinishyeah that works for me, but we'll have a double delete on the teardownclass20:10
mriedemis that avoidable?20:10
mriedemshouldn't it just ignore the 404?20:11
*** pcrews has joined #openstack-qa20:11
mtreinishyeah it'll just ignore so it's fine20:11
mtreinishi was just thinking out loud20:11
*** dimtruck is now known as zz_dimtruck20:12
mriedemyup20:12
mriedemok i'll restore20:12
mriedemand comment20:12
*** zz_dimtruck is now known as dimtruck20:13
openstackgerritA change was merged to openstack-dev/grenade: Run dstat during the upgrade process  https://review.openstack.org/11959120:14
mtreinishdtroyer: ^^^ oops I forgot I +A'd that earlier20:14
mtreinishI guess that doesn't change the commit we branch on20:14
dtroyerno, we'll just need to backport it if it's desirable in stable20:15
*** chandankumar has quit IRC20:16
*** Yogi has quit IRC20:16
*** JayJ has joined #openstack-qa20:20
*** ajo_ has quit IRC20:28
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Add cleanup to test_verify_multiple_nics_order  https://review.openstack.org/12929120:29
mriedemmtreinish: there we go ^20:29
*** ajo_ has joined #openstack-qa20:29
-openstackstatus- NOTICE: Gerrit will be offline from 2100-2130 for project renames20:34
*** ChanServ changes topic to "Gerrit will be offline from 2100-2130 for project renames"20:34
mtreinishmriedem: +220:35
*** sabeen has quit IRC20:38
flashgordonmtreinish: why is jenkins angry at you? https://review.openstack.org/#/c/117649/20:50
*** e0ne has quit IRC20:50
mtreinishflashgordon: at first is was a mirror sync issue, one of the mirrors wasn't being updated20:52
mtreinishnow it's stuck on g-r backports which we need for the grenade and stable jobs20:52
mtreinishit never ends20:52
mtreinishflashgordon: https://review.openstack.org/129248 and https://review.openstack.org/12904520:52
flashgordonmtreinish: ohh branchless tempest FWT20:52
mtreinishbut the icehouse one is stuck because disk image builder broke icehouse requirements20:52
flashgordonFTW20:52
mtreinishoh, the dib icehouse fix landed not too long ago I guess that's unblocked now20:54
*** rwsu has quit IRC20:56
flashgordonsounds fun :)21:01
-openstackstatus- NOTICE: Gerrit is offline from 2100-2130 for project renames21:03
*** ChanServ changes topic to "Gerrit is offline from 2100-2130 for project renames"21:03
flashgordonguh no gerrit21:04
*** dimtruck is now known as zz_dimtruck21:04
mtreinishit's just a pain in the ass, this should have been finished a week ago when the requirements change landed21:04
*** zz_dimtruck is now known as dimtruck21:05
*** achanda has quit IRC21:06
*** weshay has quit IRC21:07
*** achanda has joined #openstack-qa21:12
*** suneelb has quit IRC21:16
*** suneelb has joined #openstack-qa21:16
*** hogepodge has joined #openstack-qa21:22
*** ChanServ changes topic to "All Things QA..."21:25
-openstackstatus- NOTICE: Gerrit is back online21:25
*** suneelb has quit IRC21:27
*** suneelb has joined #openstack-qa21:27
*** captainmorgan is now known as needscoffee21:29
*** needscoffee is now known as captainmorgan21:29
*** ajo_ has quit IRC21:35
openstackgerritAkihiro Motoki proposed a change to openstack-dev/devstack: Compile Horizon message catalogs during stack.sh  https://review.openstack.org/12860121:40
openstackgerritJoe Gordon proposed a change to openstack/tempest: Make test_list_servers_filter_exist_host use create_test_server  https://review.openstack.org/12356221:41
openstackgerritJoe Gordon proposed a change to openstack/tempest: Add admin_client support to create_test_server  https://review.openstack.org/12939221:41
mtreinishflashgordon: ^^^ you do realize there is a separate base class for tests that require admin...21:43
flashgordonmtreinish: nope21:44
flashgordonmtreinish: ahh I see, it adds admin_clients21:46
flashgordoninstead of overwritting the default client21:46
flashgordonwhich makes sense I guess21:46
flashgordonnot sure what the right solution is here now21:47
mtreinishflashgordon: what exactly are you trying to do?21:48
flashgordonmtreinish: see the dependent patch21:48
mtreinishdrop the admin true from the dependent patch and shouldn't that work, because an admin role can manipulate any tenant's servers21:50
mtreinishotherwise your option is to add the admin kwarg and check to see if the admin server client exists (raise an exception if doesn't) then change the client you use for all the operations in create_test_server to the admin one21:51
flashgordonmtreinish: it didn't work, see the inline comments that someone left21:52
*** JayJ has quit IRC21:53
mtreinishflashgordon: oh yeah, I missed the list. Then you just need to have the list command pass in params='all_tenants' as an arg to the list command and it should work I think21:58
mtreinishthat should be safe because you're only checking for a specific instance in the list so the race between the other threads shouldn't matter21:58
*** andreaf has quit IRC21:59
flashgordonmtreinish: so I don't need an admin server?22:01
*** gfidente has quit IRC22:01
mtreinishflashgordon: no you shouldn't because an admin user can see any server. Although, I'm not sure if that needs to be an admin test (unless admin is required to get access to the host param)22:02
*** openstackgerrit has quit IRC22:03
mtreinishalthough using one tenant does make things simpler, creating the server with one set of creds and then using another might confuse people22:04
*** openstackgerrit has joined #openstack-qa22:04
flashgordonmtreinish: yeah doing things with multiple tenants doesn't sound like a win here22:07
flashgordonso should I update my create_test_server patch to better support the admin flag instead?22:08
mtreinishflashgordon: yeah that would work fine too22:08
mtreinishI think that's probably the best path forward22:09
*** dimsum_ has quit IRC22:09
mtreinishalthough I'm surprised this hasn't come up before22:09
*** dimsum_ has joined #openstack-qa22:09
mtreinishflashgordon: oh, looking at the other admin tests, they just do cls.client = cls.os_adm.servers_client in the resource_setup22:10
mtreinishwhich makes sense22:10
mtreinishand if they need a non_admin user they alias that to a different name like cls.non_admin_client22:10
mtreinishI think maybe doing that would be better22:10
*** e0ne has joined #openstack-qa22:11
*** e0ne has quit IRC22:11
flashgordonmtreinish: I'll give that a shot instead22:13
*** dimsum_ has quit IRC22:14
*** dpaterson has quit IRC22:15
openstackgerritJoe Gordon proposed a change to openstack/tempest: Make test_list_servers_filter_exist_host use create_test_server  https://review.openstack.org/12356222:21
flashgordonmtreinish: lets see what happens ^22:22
*** e0ne has joined #openstack-qa22:26
*** gfidente has joined #openstack-qa22:30
*** gfidente has quit IRC22:30
*** e0ne has quit IRC22:36
*** dimtruck is now known as zz_dimtruck22:39
*** andreaf has joined #openstack-qa22:41
*** jecarey_ has quit IRC22:45
*** rwsu has joined #openstack-qa22:52
*** jecarey_ has joined #openstack-qa22:54
*** JayJ has joined #openstack-qa22:58
*** JayJ has quit IRC23:05
*** JayJ has joined #openstack-qa23:05
*** JayJ has quit IRC23:05
*** marun has quit IRC23:09
*** zz_dimtruck is now known as dimtruck23:19
*** achanda has quit IRC23:21
*** mlavalle has quit IRC23:30
*** dimsum_ has joined #openstack-qa23:44
*** dimsum_ has quit IRC23:46
*** dimsum_ has joined #openstack-qa23:46
*** dimsum_ has quit IRC23:47
*** dimsum_ has joined #openstack-qa23:48
*** dimsum_ has quit IRC23:52
*** dimsum_ has joined #openstack-qa23:55
*** dimsum_ has quit IRC23:59

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