Tuesday, 2014-06-03

*** tosky has quit IRC00:09
*** masayukig has quit IRC00:11
*** matsuhashi has quit IRC00:13
*** rbak has joined #openstack-qa00:13
*** levi_b has joined #openstack-qa00:16
*** nati_uen_ has quit IRC00:16
*** nati_ueno has joined #openstack-qa00:17
*** masayukig has joined #openstack-qa00:17
*** TravT has quit IRC00:18
*** levi_b has quit IRC00:20
*** sthaha has quit IRC00:21
*** nati_ueno has quit IRC00:21
*** safchain_ has quit IRC00:22
*** leseb has joined #openstack-qa00:22
*** ekhugen has joined #openstack-qa00:25
*** leseb has quit IRC00:27
*** aimon has joined #openstack-qa00:29
*** sthaha has joined #openstack-qa00:31
*** sthaha has quit IRC00:31
*** sthaha has joined #openstack-qa00:31
*** nati_ueno has joined #openstack-qa00:32
*** dmorita has joined #openstack-qa00:37
*** rbak has quit IRC00:40
openstackgerritHuruifeng proposed a change to openstack/tempest: Add tests for neutron firewall resource  https://review.openstack.org/9459000:42
*** nati_ueno has quit IRC00:46
*** nati_ueno has joined #openstack-qa00:47
*** nati_ueno has quit IRC00:51
*** jaypipes has joined #openstack-qa01:01
*** ChanServ sets mode: +o jaypipes01:01
*** GMann has quit IRC01:03
*** oomichi has quit IRC01:04
*** nati_ueno has joined #openstack-qa01:07
*** rbak has joined #openstack-qa01:07
*** jaypipes has quit IRC01:09
*** nosnos has joined #openstack-qa01:09
*** rbak has quit IRC01:11
*** levi_b has joined #openstack-qa01:11
*** rbak has joined #openstack-qa01:11
*** levi_b has quit IRC01:15
*** torandu has quit IRC01:15
*** torandu has joined #openstack-qa01:16
*** davlaps has joined #openstack-qa01:16
*** arosen has quit IRC01:19
*** leseb has joined #openstack-qa01:21
*** arosen has joined #openstack-qa01:23
*** nati_ueno has quit IRC01:23
*** nati_ueno has joined #openstack-qa01:24
*** nati_ueno has quit IRC01:24
*** nati_ueno has joined #openstack-qa01:25
*** nati_uen_ has joined #openstack-qa01:26
*** leseb has quit IRC01:26
*** nati_uen_ has quit IRC01:26
*** nati_uen_ has joined #openstack-qa01:26
*** nati_uen_ has quit IRC01:26
*** nati_ueno has quit IRC01:29
*** rbak has quit IRC01:29
*** morganfainberg is now known as morganfainberg_Z01:41
openstackgerritHuruifeng proposed a change to openstack/tempest: Add tests for neutron firewall resource  https://review.openstack.org/9459001:48
*** Sukhdev has joined #openstack-qa01:51
*** dims has quit IRC01:59
*** yamahata has joined #openstack-qa02:02
*** Poornima has joined #openstack-qa02:09
*** oomichi has joined #openstack-qa02:11
*** malini_afk is now known as malini02:12
*** GMann has joined #openstack-qa02:14
*** davlaps has quit IRC02:16
openstackgerritGhanshyam Mann proposed a change to openstack/tempest: Verify list_instance_action attributes of Nova API  https://review.openstack.org/8566602:16
*** Sukhdev has quit IRC02:17
*** ekhugen has quit IRC02:21
*** leseb has joined #openstack-qa02:22
*** leseb has quit IRC02:27
openstackrecheckopenstack/nova change: https://review.openstack.org/94812 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1273908, gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1323658, gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/125423802:47
openstackrecheckopenstack/tempest change: https://review.openstack.org/92519 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1254890, gate-tempest-dsvm-neutron: unrecognized error, gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/125423802:48
openstackrecheckopenstack/nova change: https://review.openstack.org/97164 failed because of: gate-tempest-dsvm-neutron: unrecognized error, gate-nova-docs: https://bugs.launchpad.net/bugs/127038202:48
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/96781 failed gate-ceilometer-python27 in the gate queue with an unrecognized error02:48
openstackrecheckopenstack/nova change: https://review.openstack.org/94713 failed gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error02:49
openstackgerritGhanshyam Mann proposed a change to openstack/tempest: Adds test for list/get volume attachments V2 APIs  https://review.openstack.org/8719102:52
*** GMann has quit IRC03:02
*** GMann has joined #openstack-qa03:03
*** Longgeek has joined #openstack-qa03:13
*** coolsvap|afk is now known as coolsvap03:13
openstackgerritA change was merged to openstack-dev/devstack: Fix the way Apache site configuration files are used, to improve OS portability  https://review.openstack.org/9597703:16
*** Longgeek has quit IRC03:17
*** nosnos has quit IRC03:20
openstackrecheckopenstack/nova change: https://review.openstack.org/96293 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1254238, gate-tempest-dsvm-full: unrecognized error03:22
*** leseb has joined #openstack-qa03:23
openstackrecheckopenstack/nova change: https://review.openstack.org/76829 failed because of: gate-nova-python27: https://bugs.launchpad.net/bugs/126661103:23
*** sthaha has quit IRC03:23
*** leseb has quit IRC03:24
*** leseb has joined #openstack-qa03:25
*** leseb has quit IRC03:29
openstackgerritSridhar Gaddam proposed a change to openstack-dev/devstack: Set default value for IPv6 subnet attribute flag  https://review.openstack.org/9738303:35
*** sthaha has joined #openstack-qa03:38
*** levi_b has joined #openstack-qa03:38
*** levi_b has quit IRC03:43
openstackgerritGhanshyam Mann proposed a change to openstack/tempest: Cleanup for local variable in test_attach_volume  https://review.openstack.org/9738503:50
*** nosnos has joined #openstack-qa04:00
*** k4n0 has joined #openstack-qa04:11
*** arosen has quit IRC04:12
*** torandu has quit IRC04:12
*** Longgeek has joined #openstack-qa04:14
*** Longgeek has quit IRC04:18
*** raies has joined #openstack-qa04:21
raiesylobankov: hi04:22
*** leseb has joined #openstack-qa04:26
*** leseb has quit IRC04:30
*** psharma has joined #openstack-qa04:32
*** levi_b has joined #openstack-qa04:38
raiesGMann: hi04:39
openstackgerritA change was merged to openstack-infra/devstack-gate: Allow to configure git base URL  https://review.openstack.org/9590104:39
*** raies has quit IRC04:41
*** raies has joined #openstack-qa04:42
*** levi_b has quit IRC04:42
*** harlowja is now known as harlowja_away04:48
*** Abhijeet has joined #openstack-qa04:52
raiesylobankov: hi04:56
openstackgerritajay proposed a change to openstack/tempest: Added cinder cli tests  https://review.openstack.org/9607204:56
*** Madan has joined #openstack-qa05:02
*** malini is now known as malini_afk05:07
*** Longgeek has joined #openstack-qa05:14
*** Longgeek has quit IRC05:19
*** leseb has joined #openstack-qa05:26
*** rdekel has joined #openstack-qa05:27
*** arosen has joined #openstack-qa05:28
*** arosen has quit IRC05:29
*** leseb has quit IRC05:30
*** jhenner has quit IRC05:35
*** Madan has quit IRC05:35
*** Madan has joined #openstack-qa05:39
*** Madan has quit IRC05:43
*** Madan has joined #openstack-qa05:46
*** rdekel has quit IRC05:53
*** coolsvap is now known as coolsvap|afk05:56
*** coolsvap|afk is now known as coolsvap06:01
*** Madan has quit IRC06:15
*** Longgeek has joined #openstack-qa06:15
*** ilyashakhat_ has quit IRC06:17
openstackgerritlifeless proposed a change to openstack-infra/elastic-recheck: Add rule for bug 1325815  https://review.openstack.org/9740906:19
*** Longgeek has quit IRC06:20
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/83369 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/132365806:24
*** levi_b has joined #openstack-qa06:26
*** Madan has joined #openstack-qa06:27
*** Longgeek has joined #openstack-qa06:30
*** levi_b has quit IRC06:31
*** Longgeek has quit IRC06:31
*** rdekel has joined #openstack-qa06:32
*** luqas has joined #openstack-qa06:34
*** nati_ueno has joined #openstack-qa06:38
openstackgerritJoshua Hesketh proposed a change to openstack-infra/elastic-recheck: Ignore non-voting jobs in gerrit  https://review.openstack.org/9736906:49
*** coolsvap is now known as coolsvap|afk06:57
openstackgerritMh Raies proposed a change to openstack/tempest: Compute "cells" basic API test  https://review.openstack.org/9033706:59
*** tkammer has joined #openstack-qa07:00
*** Longgeek has joined #openstack-qa07:01
*** coolsvap|afk is now known as coolsvap07:02
*** ildikov has joined #openstack-qa07:03
*** Longgeek has quit IRC07:07
*** Longgeek has joined #openstack-qa07:08
openstackgerritLonggeek proposed a change to openstack-dev/devstack: Add missing config option os_auth_url  https://review.openstack.org/9710707:12
ylobankovraies: hi07:12
*** afazekas has joined #openstack-qa07:14
openstackgerritIan Wienand proposed a change to openstack-dev/devstack: Add basic echo_* functions for grenade  https://review.openstack.org/9741807:17
raiesylobankov: please check http://pastebin.com/iKRkYBuc regarding https://review.openstack.org/#/c/47816/ I think it will do well07:17
*** dkorn has joined #openstack-qa07:18
*** Albert__13 has joined #openstack-qa07:18
ylobankovraies: OK. I will take a look at it right now07:18
eezhova@afazekas: Hi! Could you please take a look at my comment regarding https://review.openstack.org/#/c/88579/10 ?07:19
raiesylobankov: LOG.exception('Exception raised deleting policy %s', ipsec_policy_id)07:20
raiescell was written by mistake07:21
*** dkranz has quit IRC07:21
afazekaseezhova: no, sorry I was blind07:21
*** andreaf has joined #openstack-qa07:22
eezhovaafazekas: So is it ready to merge now or it needs some additional testing?07:23
raiesylobankov: have u checked ??07:23
ylobankovraies: which cell?07:23
eezhovaafazekas: thanks07:24
raiesin http://pastebin.com/iKRkYBuc in line 607:24
raiesylobankov: ^^07:24
*** salv-orlando has joined #openstack-qa07:26
*** andreaf has quit IRC07:26
ylobankovraies: ah, got it07:26
raiesylobankov: I think this code will be fine against your comment in https://review.openstack.org/#/c/47816/ what you say ??07:27
ylobankovraies: I suspect that you don't try/except block07:27
ylobankovraies: I suspect that you don't need try/except block07:28
ylobankovraies: You create policy and then you just add policy to self.addCleanup method. Why do you need try/except block?07:30
raiesas per Jordan Pittiersuggestion07:31
raiesylobankov: try delete, if exception is not found make it pass and if any other reason is there then log that thats seem to be good for me what you say07:32
raiesfirstly I had used list/filter methos07:32
raies**method07:32
raiesylobankov: what you say ??07:36
ylobankovraies: You create policy and immediately add policy to  self.addCleanup method to delete. Why do you need to catch NotFound exception if you created policy a moment go?07:38
ylobankovraies: *ago07:38
ylobankovraies: I will write comment to your patch07:39
raiesactually test is like this create --> update --->> delete07:40
raiesylobankov: I have removed delete but that should not be removed from test07:40
raiesylobankov: so if policy is delete in the test phase then there is no need to delete from addCleanup and it will throw nowfound exception07:41
raiesso that is handled07:41
ylobankovraies: Yes, but you don't delete policy in the test phase :) You delete policy trough  addCleanup07:43
raiesylobankov: as delete is part of test so it is being deleted in test07:44
raiesylobankov: I think same flow is follwed by all other07:44
raiesylobankov: cleanup does not indicate any test coverage, that why delete should be put in test.07:46
raiesylobankov: if we are talking that this test is create_update_delete then I think delete should also go in with test method07:49
ylobankovraies: Yes, I think deleting should be in test. Could you please submit new patch set?07:49
raiesylobankov: thanks :)07:50
*** ildikov has quit IRC07:51
openstackgerritMh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests  https://review.openstack.org/4781607:51
openstackgerritMh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests  https://review.openstack.org/4781607:53
*** jpich has joined #openstack-qa07:53
*** jordanP has joined #openstack-qa07:53
*** jlibosva has joined #openstack-qa07:56
*** salv-orlando has quit IRC08:02
*** safchain has joined #openstack-qa08:05
*** jlibosva has quit IRC08:08
*** jlibosva has joined #openstack-qa08:10
*** levi_b has joined #openstack-qa08:14
*** jlibosva has quit IRC08:15
*** jlibosva has joined #openstack-qa08:15
*** yrabl has joined #openstack-qa08:17
*** levi_b has quit IRC08:19
*** nati_ueno has quit IRC08:21
*** cdent has joined #openstack-qa08:22
*** SridharG has joined #openstack-qa08:24
*** coolsvap is now known as coolsvap|afk08:24
*** fbo_away is now known as fbo08:25
*** luqas has quit IRC08:26
*** andreaf has joined #openstack-qa08:28
*** oomichi has quit IRC08:29
*** mpavlase has joined #openstack-qa08:32
*** ildikov has joined #openstack-qa08:33
*** oomichi has joined #openstack-qa08:35
*** cdent_ has joined #openstack-qa08:42
*** cdent has quit IRC08:43
*** cdent_ is now known as cdent08:43
*** coolsvap|afk is now known as coolsvap08:46
raiesafazekas: hi can you please have a look on https://review.openstack.org/#/c/89472/08:47
*** tteggel has joined #openstack-qa08:54
*** luqas has joined #openstack-qa08:56
*** salv-orlando has joined #openstack-qa08:58
*** vrovachev has joined #openstack-qa09:03
*** cdent has quit IRC09:03
*** cdent has joined #openstack-qa09:05
*** cdent has quit IRC09:10
*** cdent has joined #openstack-qa09:11
*** cdent has quit IRC09:11
*** cdent has joined #openstack-qa09:12
*** cdent_ has joined #openstack-qa09:13
afazekasraies: can you ping dwalleck about this ?09:15
*** cdent has quit IRC09:17
*** cdent_ is now known as cdent09:17
raiesafazekas: I just wanted your view on his comment09:17
afazekasraies: both way works for me.09:18
raiesSo if you will give your opinion there then it will make an impact09:18
raiesplease give you opinion there on the patch so that he could evaluate his mark09:21
*** salv-orlando has quit IRC09:22
*** jhenner has joined #openstack-qa09:30
*** cdent_ has joined #openstack-qa09:32
*** gszasz has joined #openstack-qa09:33
*** cdent has quit IRC09:35
*** cdent_ is now known as cdent09:35
*** gszasz is now known as gszasz_lunch09:35
openstackrecheckopenstack/heat change: https://review.openstack.org/92498 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1262566 https://bugs.launchpad.net/bugs/125489009:41
openstackgerritVadim Rovachev proposed a change to openstack-dev/devstack: Add flag for ceilometer tests  https://review.openstack.org/9648409:41
GMannhi cyeoh09:43
GManncyeoh: regarding https://review.openstack.org/#/c/95409/1//COMMIT_MSG09:43
GManncyeoh: Please check my reply in the review. and please let me kow your opinion09:44
openstackgerritVadim Rovachev proposed a change to openstack-dev/devstack: Add flag for ceilometer tests  https://review.openstack.org/9648409:44
*** sthaha has quit IRC09:46
openstackgerritSean Dague proposed a change to openstack-dev/grenade: first pass of integrating javelin2 to grenade  https://review.openstack.org/9731709:47
*** giulivo has joined #openstack-qa09:49
openstackgerritVadim Rovachev proposed a change to openstack/tempest: Add compute notifications tests for ceilometer  https://review.openstack.org/6413609:50
openstackgerritMh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests  https://review.openstack.org/4781609:52
*** amotoki has joined #openstack-qa09:57
openstackgerritSean Dague proposed a change to openstack-dev/devstack: make mysql run in strict mode  https://review.openstack.org/9744209:58
*** levi_b has joined #openstack-qa10:02
*** gabriel-bezerra has quit IRC10:07
*** levi_b has quit IRC10:07
*** samuelmz has quit IRC10:07
*** rodrigods has quit IRC10:07
*** gabriel-bezerra has joined #openstack-qa10:09
*** samuelmz has joined #openstack-qa10:09
*** evgenyf has joined #openstack-qa10:10
*** rodrigods has joined #openstack-qa10:10
*** tosky has joined #openstack-qa10:20
*** coolsvap is now known as coolsvap|afk10:24
*** Longgeek_ has joined #openstack-qa10:27
*** Longgeek has quit IRC10:30
raiesjordanP: hi10:32
jordanPraies, hi10:32
raiesregarding your comment in https://review.openstack.org/#/c/47816/28/tempest/api/network/test_vpnaas_extensions.py #L8410:32
raiesplease do it a bit clear10:33
jordanPraies, in Tempest and specifically in cleanup functions we don't catch exceptions like this10:33
jordanPlet just the exception propagates10:34
jordanPjust catch the exception you "expect" and the exceptions that are not harmful10:34
raiesjordanP: ok10:35
*** rdekel has quit IRC10:35
raiesjordanP: but in my case if cleanup is failed10:35
raiesi.e. delete is failed to some problem them how to show the exception ?10:36
raiesthen should not be use except Exception:10:36
raiesThe link you gave I read f you really need to use a try block, please ensure the original exception at least logged. When the exception is logged you usually need to raise the same or a different exception anyway.10:36
jordanPif delete fails (and the exception is not exceptions.NotFound) then something wrong happen and it would need further investigation, so it's good to let the exception propagates so that the caller is informed10:37
raiesok10:38
raiesso in my case http://pastebin.com/VS8gS3Dd this piece of code is sufficient right ?10:40
raiesjordanP: ^^10:40
jordanPexactly10:40
raiesjordanP: ok thanks :)10:40
jordanPnp10:40
*** evgenyf has quit IRC10:44
openstackgerritMh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests  https://review.openstack.org/4781610:45
*** yamahata has quit IRC10:52
*** gszasz_lunch is now known as gszasz10:54
*** tkammer has quit IRC10:57
*** salv-orlando has joined #openstack-qa10:58
*** k4n0 has quit IRC11:02
*** mkollaro has joined #openstack-qa11:09
*** mkollaro has quit IRC11:10
*** mkollaro has joined #openstack-qa11:11
raiessmurashov: hi11:11
smurashovraies: hi11:11
raiessmurashov: as per your comment in https://review.openstack.org/#/c/86002/8/tempest/api/baremetal/test_nodestates.py11:11
raiesbreak should be replaced with return11:12
smurashovraies: yes, can i help?11:12
raiesin this case if raise_error: message will go in testcase itself11:12
raieswhat you say I wthink that will not be good11:12
raiessmurashov: ye ofcource11:13
raies**yes of course11:13
*** dims_ has joined #openstack-qa11:14
*** Madan has quit IRC11:14
raiessmurashov: u there ??11:14
smurashovraies: yes, why do you think it not be good?11:15
raiessmurashov: got your point please check it in next patch :)11:16
raiesthanks11:16
smurashovraies: ok11:16
smurashov:)11:16
raiessmurashov: still please check http://pastebin.com/AfmT271d for validation11:17
smurashovraies: it's seems good:)11:18
smurashovraies: but why do you return False?11:18
smurashovraies: may be would be better return None? or you expect False?11:18
raiessmurashov: yes this was good approach thanks :)11:18
smurashovraies: :)11:18
*** Madan has joined #openstack-qa11:19
openstackgerritMh Raies proposed a change to openstack/tempest: Nodestate API test for power state set  https://review.openstack.org/8600211:19
openstackrecheckopenstack/oslo.db change: https://review.openstack.org/94552 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error11:20
*** sthaha has joined #openstack-qa11:21
*** dmorita has quit IRC11:21
*** luqas has quit IRC11:22
openstackgerritMh Raies proposed a change to openstack/tempest: Nodestate API test for power state set  https://review.openstack.org/8600211:23
openstackgerritMh Raies proposed a change to openstack/tempest: Baremetal nodestate APIs  https://review.openstack.org/8947211:23
openstackgerritMh Raies proposed a change to openstack/tempest: Nodestate API test for power state set  https://review.openstack.org/8600211:31
*** Abhijeet has quit IRC11:32
*** Madan has quit IRC11:33
*** matsuhashi has joined #openstack-qa11:34
openstackgerritMh Raies proposed a change to openstack/tempest: Baremetal nodestate APIs  https://review.openstack.org/8947211:34
*** Poornima has quit IRC11:37
*** safchain has quit IRC11:37
*** yamahata has joined #openstack-qa11:38
*** jordanP has quit IRC11:45
*** rbak has joined #openstack-qa11:47
*** weshay has joined #openstack-qa11:49
*** gszasz has quit IRC11:50
*** rbak has quit IRC11:50
*** levi_b has joined #openstack-qa11:50
*** mwagner_lap has quit IRC11:53
*** safchain has joined #openstack-qa11:54
*** levi_b has quit IRC11:55
*** jordanP has joined #openstack-qa12:01
*** jhenner1 has joined #openstack-qa12:01
*** mpavlase1 has joined #openstack-qa12:01
*** dustins has joined #openstack-qa12:02
*** mkollaro has quit IRC12:02
*** jhenner has quit IRC12:02
*** mpavlase has quit IRC12:03
*** mkollaro has joined #openstack-qa12:03
*** ildikov_ has joined #openstack-qa12:05
*** amotoki has quit IRC12:05
*** ildikov has quit IRC12:05
openstackgerritA change was merged to openstack-infra/elastic-recheck: Ignore non-voting jobs in gerrit  https://review.openstack.org/9736912:13
*** evgenyf has joined #openstack-qa12:14
*** SridharG has quit IRC12:15
*** levi_b has joined #openstack-qa12:16
*** SridharG has joined #openstack-qa12:17
*** adalbas has joined #openstack-qa12:19
*** openstackrecheck has quit IRC12:23
*** openstackrecheck has joined #openstack-qa12:23
openstackgerritA change was merged to openstack/tempest: Upload public image requires admin role by default  https://review.openstack.org/9443512:25
*** psharma has quit IRC12:30
*** dims_ has quit IRC12:31
*** dims_ has joined #openstack-qa12:32
*** oomichi has quit IRC12:32
*** weshay has quit IRC12:36
*** weshay has joined #openstack-qa12:37
*** aysyd has joined #openstack-qa12:38
*** bauzas has quit IRC12:38
*** mpavlase1 has quit IRC12:39
*** mkollaro has quit IRC12:41
*** mkollaro1 has joined #openstack-qa12:41
*** jhenner1 has quit IRC12:43
*** mpavlase has joined #openstack-qa12:43
*** salv-orlando has quit IRC12:43
*** cdent has quit IRC12:44
*** krtaylor has quit IRC12:45
*** Longgeek_ has quit IRC12:48
*** luqas has joined #openstack-qa12:49
*** salv-orlando has joined #openstack-qa12:50
*** ekhugen has joined #openstack-qa12:51
openstackgerritOpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements  https://review.openstack.org/9682912:52
*** gszasz has joined #openstack-qa12:52
*** salv-orlando has quit IRC12:53
*** salv-orlando has joined #openstack-qa12:55
*** bauzas has joined #openstack-qa12:55
*** GMann has quit IRC12:58
*** jchai has joined #openstack-qa12:59
*** jhenner has joined #openstack-qa12:59
*** mriedem has joined #openstack-qa13:03
*** jaypipes has joined #openstack-qa13:03
*** ChanServ sets mode: +o jaypipes13:03
*** ildikov_ has quit IRC13:05
*** openstackgerrit has quit IRC13:06
*** openstackgerrit has joined #openstack-qa13:07
*** ildikov has joined #openstack-qa13:08
*** rbak has joined #openstack-qa13:08
*** matsuhashi has quit IRC13:12
*** bauzas has quit IRC13:12
*** nosnos has quit IRC13:12
*** jchai is now known as jchai_afk13:14
openstackgerritRavikumar Venkatesan proposed a change to openstack/tempest: Test cases for database limits  https://review.openstack.org/8532013:15
*** jchai_afk is now known as jchai13:16
*** bauzas has joined #openstack-qa13:25
*** asvechnikov has joined #openstack-qa13:26
*** malini_afk is now known as malini13:26
*** nati_ueno has joined #openstack-qa13:29
*** malini is now known as malini_afk13:29
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Wait for server activations for the updated servers  https://review.openstack.org/9749613:30
openstackgerritajay proposed a change to openstack/tempest: Added nova cli test  https://review.openstack.org/9749713:31
*** malini_afk is now known as malini13:31
*** Obulpathi has joined #openstack-qa13:32
*** Obulpathi has joined #openstack-qa13:32
*** jgrimm has joined #openstack-qa13:33
andreafmtreinish: ping13:35
mtreinishandreaf: pong13:35
andreafre https://blueprints.launchpad.net/tempest/+spec/cross-domain-testing - any reason for it being superseded rather than review?13:36
mtreinishoops, clicked the wrong one13:36
mtreinishlet me fix that13:36
mtreinishsry13:36
andreafmtreinish: np13:37
mtreinishit was muscle memory after doing so many superseeded in a row13:37
andreafmtreinish: with storyboard hopefully we'll be able to automate more this kind of tasks ^_^13:38
mtreinishandreaf: yeah hopefully, the lp api around bps kind of sucks, otherwise I would have written a script to do this13:39
*** pafuent has joined #openstack-qa13:39
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Wait for server activations for the updated servers  https://review.openstack.org/9749613:39
mtreinishSlickNik, GMann: for https://review.openstack.org/#/c/95639/ which bp should I leave open for it? https://blueprints.launchpad.net/tempest/+spec/tempest-guest-tests or https://blueprints.launchpad.net/tempest/+spec/trove-api-test-coverage-tempest ?13:39
mtreinishboth list the spec review13:40
pafuentHi. I'm from Blazar, and I'm interested in running our Tempest test in our gate job. I will like to know which is the recommended way to achieve this goal.13:40
*** oomichi has joined #openstack-qa13:44
*** oomichi has quit IRC13:47
*** rbak has quit IRC13:53
mkodererpafuent: do you mean something like this: http://www.joinfu.com/2014/02/setting-up-an-external-openstack-testing-system/ ?14:01
pafuentmkoderer: No. Just to run the Tempest test in our contrib directory inside a devstack jenkins gate job14:02
pafuentpafuent: We do locally copying our files inside tempest and adding an entry to tox.ini14:02
pafuentmkoderer: The idea is to get this running for each commit14:03
mkodererpafuent: triggered by gerrit?14:03
pafuentmkoderer: Yes}14:03
pafuentmkoderer: For devstack we copy some files of our contrib dir and devstack starts blazar as a service14:04
pafuentmkoderer: There is something similar for Tempest? In other words which is the recommended way to do this.14:05
*** GMann has joined #openstack-qa14:05
mkodererpafuent: is it review.openstack.org or an in-house gerrit installation?14:06
pafuentmkoderer: Is review.openstack.org14:06
mkodererpafuent: so IMHO jay's blog post can be the starting point14:07
pafuentmkoderer: OK. I'll check that blog post deeper. Thanks.14:07
mkodererI never heard about blazar btw14:07
pafuentmkoderer: Is Climate14:08
mkodererk14:08
pafuentmkoderer: We change the name a few weeks ago (during the gerrit downtime)14:08
*** adalbas has quit IRC14:08
jordanPmtreinish, sdague hi. The backport for nova/havana was merged yesterday (related to instance shelving). I'd like to retrigger jenkins for my tempest patch here : https://review.openstack.org/#/c/96165/ "recheck no bug" doesn't work anymore, what's the best way, for my information, to retrigger jenkins ?14:09
mtreinishjordanP: just use the bug number for the shelving bug14:10
mtreinishbecause that was the reason you need to recheck14:10
jordanPcool thanks14:10
jordanPI forgot the bug number didn't have to be a bug in the tempest project14:10
*** adalbas has joined #openstack-qa14:10
*** kgriffs|afk is now known as kgriffs14:11
mkodererpafuent: ahh so it's about a stackforge project?14:11
*** levi_b has quit IRC14:11
pafuentmkoderer: Yes14:12
*** GMann has quit IRC14:12
mkodererpafuent: ok forget what I said :) I misread it14:12
pafuentmkoderer: No problem14:13
mkodererpafuent: I think the infra ppl can help you with that. AFAIK you need to define the needed jobs in the jobs builder14:15
mkodererpafuent: http://ci.openstack.org/jjb.html#jjb14:15
mkodererbut maybe I still didn't get the question :)14:15
pafuentmkoderer: I think you got the idea. I'll chat with the infra team. Thanks.14:16
*** mwagner_lap has joined #openstack-qa14:21
openstackgerritA change was merged to openstack/tempest: Relax security group rules cleanup  https://review.openstack.org/8831714:25
openstackrecheckopenstack/trove change: https://review.openstack.org/97087 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error14:25
*** dkorn has quit IRC14:26
*** andreaf has quit IRC14:29
*** nati_ueno has quit IRC14:29
*** TravT has joined #openstack-qa14:39
*** rodrigods has quit IRC14:47
*** dims_ has quit IRC14:49
openstackgerritafazekas proposed a change to openstack-dev/devstack: Enable horizon on Fedoras  https://review.openstack.org/9753215:03
afazekassdague: https://review.openstack.org/9753215:03
*** safchain has quit IRC15:03
*** safchain has joined #openstack-qa15:04
afazekaschmouel: ^^15:05
*** mlavalle has joined #openstack-qa15:09
*** rbak has joined #openstack-qa15:12
*** jchai is now known as jchai_afk15:13
* afazekas grr.., I would't be surprised if another team request a gerrit service account soon..15:14
sdagueafazekas: +215:15
afazekassdague: thx15:15
*** afazekas has quit IRC15:20
*** jchai_afk is now known as jchai15:21
*** vrovachev has quit IRC15:22
*** davlaps has joined #openstack-qa15:23
*** malini is now known as malini_afk15:30
openstackgerritA change was merged to openstack/tempest: Switch back to nc in test_load_balancer_basic  https://review.openstack.org/8857915:31
chmoueli don't think "recheck bug 1" is a good thing to do in review https://review.openstack.org/9753215:33
*** tmcpeak has joined #openstack-qa15:35
*** morganfainberg_Z is now known as morganfainberg15:43
mtreinishmlavalle: which bp is attached to the spec in https://review.openstack.org/95600 ? https://blueprints.launchpad.net/tempest/+spec/neutron-advanced-scenarios or https://blueprints.launchpad.net/tempest/+spec/neutron-scenario-tests ?15:45
*** andreaf has joined #openstack-qa15:46
*** nati_ueno has joined #openstack-qa15:47
mlavallemtreinish: the good one is https://blueprints.launchpad.net/tempest/+spec/neutron-advanced-scenarios15:50
mtreinishmlavalle: ok then I'll close out neutron-scenario tests15:50
mlavalleYes, please15:50
*** evgenyf has quit IRC15:50
mlavallethanks :-)15:51
*** jlibosva has quit IRC15:52
*** nati_uen_ has joined #openstack-qa15:52
*** nati_ue__ has joined #openstack-qa15:53
sdagueianw: what's up with the check experimental from Red Hat CI?15:53
*** nati_ue__ has quit IRC15:53
*** nati_ue__ has joined #openstack-qa15:54
*** nati_ueno has quit IRC15:55
*** nati_uen_ has quit IRC15:57
*** gszasz has quit IRC15:57
*** amotoki has joined #openstack-qa16:00
*** torandu has joined #openstack-qa16:01
*** jhenner1 has joined #openstack-qa16:01
*** afazekas has joined #openstack-qa16:04
*** jhenner has quit IRC16:04
*** torandu has quit IRC16:05
*** torandu has joined #openstack-qa16:06
SlickNikmtreinish / gmann: The one at https://blueprints.launchpad.net/tempest/+spec/tempest-guest-tests seems to have more information in it. Let's use that one.16:08
asselin_mtreinish, what does supeseded mean here? https://blueprints.launchpad.net/tempest/+spec/stress-api-tracking16:09
mtreinishasselin_: see: http://lists.openstack.org/pipermail/openstack-dev/2014-May/035714.html16:09
mtreinishasselin_: it's just a status to mark it as closed it was that or obsolete to get it off the list16:10
*** aimon has quit IRC16:10
asselin_mtreinish, ok thanks will follow the new process16:10
mtreinishSlickNik: that works for me I'll close the other one16:11
SlickNikmtreinish: Sounds good. Thanks!16:11
*** jordanP has quit IRC16:12
*** mkollaro1 has quit IRC16:13
*** jhenner has joined #openstack-qa16:13
*** yfried has quit IRC16:15
*** dims_ has joined #openstack-qa16:16
*** yamahata has quit IRC16:16
*** jhenner1 has quit IRC16:17
tmcpeakhey, what's the best way to run a certain subset of tests16:18
tmcpeakI know I can use the regex16:18
tmcpeakis there anything more selective than that?16:18
tmcpeakfor example, if I want to run some API tests, but exclude a few16:19
*** dims_ has quit IRC16:21
mtreinishtmcpeak: with testr its only a regex right now16:21
mtreinishyou can do that with a regex it's just not that pretty16:22
mtreinishlook at the tox.ini for examples16:22
*** markmcclain has joined #openstack-qa16:24
*** SridharG has quit IRC16:25
*** cdent has joined #openstack-qa16:31
Shrewssdague: re your comment on 94439... i'm pretty new to tempest. care to give me a pointer to the regex i need to change?16:33
sdagueShrews: sure, so the ironic tempest job doesn't use a normal tempest target16:33
tmcpeakmtreinish: back16:34
tmcpeakmtreinish: ok cool16:34
tmcpeakmtreinish: thank you16:35
sdagueShrews: https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/jenkins_job_builder/config/devstack-gate.yaml#L587-L60216:35
sdaguewhich means adding tests doesn't mean they run16:35
sdagueso that regex is going to need to be expanded16:36
Shrewssdague: ah, now i see. thanks for the review and learnin' me good   :)16:36
sdagueit would also be interesting to understand if that's still needed16:37
sdagueit seemed like a lot more of this is working now16:37
Shrewssdague: hrm, yeah. looks like that needs some review now. it skips the scenario tests, but then later adds a scenario test. weird16:39
*** aimon has joined #openstack-qa16:44
*** harlowja_away is now known as harlowja16:45
*** ylobankov has quit IRC16:45
*** nati_ue__ has quit IRC16:46
*** Albert__13 has quit IRC17:00
*** fbo is now known as fbo_away17:02
*** jpich has quit IRC17:04
*** ashishsnigam has joined #openstack-qa17:13
*** luqas has quit IRC17:14
ashishsnigamNeed some core reviews on https://review.openstack.org/#/c/94599/17:14
*** kgriffs is now known as kgriffs|afk17:15
*** Obulpathi has quit IRC17:15
*** Obulpathi has joined #openstack-qa17:17
*** dims_ has joined #openstack-qa17:17
anteayasdague: are you going to be available for the -infra meeting today?17:20
anteayaI am noticing there appears to be a conflict with that and picking up your vegetable basket17:20
*** Obulpathi has quit IRC17:20
*** cdent_ has joined #openstack-qa17:21
anteayaI have an item on the agenda to discuss consistency when reviewing acl file changes17:21
*** Obulpathi has joined #openstack-qa17:21
*** markmcclain has quit IRC17:21
*** cdent has quit IRC17:21
*** cdent_ is now known as cdent17:21
*** giulivo has quit IRC17:26
*** levi_b has joined #openstack-qa17:28
*** masayukig has quit IRC17:29
jogowhoa the gate queue is 49 deep and less then two hours17:35
jogooh never mind17:35
*** masayukig has joined #openstack-qa17:36
*** nati_ueno has joined #openstack-qa17:36
*** mmaglana has joined #openstack-qa17:41
*** kgriffs|afk is now known as kgriffs17:45
*** safchain has quit IRC17:46
anteayajogo: :D17:46
anteayajogo: do you want to know the history of why that is?17:47
jogoanteaya: I do17:47
anteayalet's start off with a setuptools release sunday night that broke the world17:48
anteayaI don't have the link17:48
anteayathat was fixed about 10 or 12 hours later, but long enough that when the rechecks happened it was a fair pileon17:48
anteayaadd in hpcloud decommisioning 1.017:49
anteayaand 1.1 having serious outage issues last week17:49
anteayavarious reasons17:49
anteayanow we have 2 hpcloud techs in -infra with us and we are figuring out what is going on17:49
jogofun17:49
anteayaright now it takes about 40 minutes for nodepool to see a node is built17:50
anteayaand by that time the time is run out and it is scheduled for deletion17:50
anteayaso any tests actually running now are all on rax since hpcloud is just building and deleting17:50
anteayathe cores are figuring out what to do on our end to have actual tests running17:51
anteayaand neutron is working on an ssh bug, which I don't know the details17:51
anteayawhich is playing a role, but I don't know how large17:51
anteayathat is what I know17:51
jogoanteaya: I noticed the ssh bug on  http://status.openstack.org/elastic-recheck/gate.html17:52
anteayayup17:52
jogoanteaya: so that explains it then17:52
jogowow17:52
anteayawhich is affecting our gate and may also be affecting hpcloud17:52
anteayaI don't know if it is or not17:52
anteayaso yeah17:53
anteayaand as far as I can tell check isn't moving17:53
anteayathat horizon job at the top of the gate is 9 hours old17:54
anteayawaiting on a free python26 node17:54
jogowow17:54
anteayawoooo17:54
jogo I don't think I have ever seen so many jobs backed up17:55
anteayayup17:55
*** kgriffs is now known as kgriffs|afk17:55
anteayamy role is to not review anything and not make things worse than they are17:55
jogoanteaya: I'll make sure to not review that much either heh17:57
anteayano point really17:57
*** jhenner has quit IRC17:57
jogoyeah, good news is I have plenty of patches to work on17:58
jogotrying to think on what I can do to help, but not really sure17:58
*** jhenner has joined #openstack-qa17:58
*** nati_ueno has quit IRC18:02
*** nati_uen_ has joined #openstack-qa18:02
*** SridharG has joined #openstack-qa18:06
sdagueanteaya: I will probably not be at the infra meeting, or at least not for most of it. Because of TC and Project meetings, that's the only timeblock that I can go do CSA pickup.18:06
clarkbsdague: you need delivery18:07
sdagueclarkb: going to the farm is part of the whole experience :)18:07
sdagueand that's just the way it works around here18:07
jogosdague: have a moment? https://review.openstack.org/#/c/95350/218:08
anteayasdague: I was hoping we could come to an agreement around the acl reviewing18:08
sdaguejogo: shoot18:08
jogosdague: I responded in the patch, but I am not sure if the direction I am going is the right one18:09
jogoEvery so often a valid pep8 failure happens in the gate, so my thinking was we should count those failures as classified for the sake of http://status.openstack.org/elastic-recheck/data/uncategorized.html18:09
jogoex: http://logs.openstack.org/13/94813/6/gate/gate-nova-pep8/96604c4/console.html18:09
sdaguejogo: that a bad merge?18:09
jogoyes18:09
sdagueare you sure you want to supress it?18:10
sdagueI agree we should fingerprint18:10
jogosdague: and for some reason those are fairly common (weekly)18:10
jogosdague: well supress from gerrit side18:10
jogoso folks don't just run recheck18:10
jogomaybe we don't want to supress from the graph side18:11
openstackgerritafazekas proposed a change to openstack-dev/devstack: [Fedora] Apache double enable/disable apache site  https://review.openstack.org/9758518:11
sdagueI think I'd be ok if we suppress it from the bot side18:11
sdaguemaybe we should add 2 vars to the yaml18:11
sdaguereport: true/false18:11
sdaguegraph: true/false18:11
sdaguedefaulting to true18:12
jogosdague: thats what I meant to do somehow that patch didn't get pushed18:12
sdaguethere is a patch under it, but it's a little odd - https://review.openstack.org/#/c/97354/1/elastic_recheck/elasticRecheck.py18:13
sdaguewould it be better to just filter those out at load time/18:13
sdague?18:13
jogowe use the loader in other places though18:13
sdagueI think 97354 does it in all cases right18:13
jogoclassify is only used by the bot18:14
*** rbak has quit IRC18:14
sdagueok, so then loader.load(queries, drop_no_report=false, drop_no_graph=false)18:16
sdagueit just seems odd to constantly hit the if there instead of adding a couple flags to the loader to be able to skip things when it loads18:16
jogosdague: yeah, makes sense to me. new patch comming soon18:17
sdaguecoolio18:17
afazekassdague: can you stop this: https://review.openstack.org/#/c/97532/18:19
afazekassomething is still wrong18:19
sdagueafazekas: sure, but why?18:19
sdagueafazekas: you can also WorkFlow -1 it18:20
sdagueand it won't merge18:20
*** ildikov has quit IRC18:20
*** Sukhdev has joined #openstack-qa18:22
openstackgerritRamy Asselin proposed a change to openstack/qa-specs: Add qa spec for bp:stress-api-tracking  https://review.openstack.org/9758918:26
*** ashishsnigam has quit IRC18:27
tmcpeakwhen I run Tempest tests for components which I don't have (like tempest.api.baremetal) it knows to skip18:33
tmcpeakhow does that work?18:33
maurosrtmcpeak: it happens during class setup https://github.com/openstack/tempest/blob/master/tempest/api/baremetal/base.py#L4818:34
*** kgriffs|afk is now known as kgriffs18:34
tmcpeakmaurosr: awesome, I'll check this out, thanks!18:34
maurosrit checks CONF.service_available.<component>18:34
maurosryw18:34
maurosrmtreinish: you around?18:35
openstackgerritafazekas proposed a change to openstack-dev/devstack: Enable horizon on Fedoras  https://review.openstack.org/9753218:35
*** ajayaa has joined #openstack-qa18:38
maurosrqq, v3 and v2 will converge into v2.1, so patches to attribute tests are also in hold? specially something that is only implemented on v318:38
maurosrcause I think it will endup being useful into v2.1 since we want the new functionalities18:38
*** nati_uen_ has quit IRC18:39
*** nati_ueno has joined #openstack-qa18:39
maurosrso would be welcome anyway18:40
*** ajo|mac has joined #openstack-qa18:42
*** kgriffs is now known as kgriffs|afk18:44
*** SridharG has quit IRC18:47
*** tmcpeak has quit IRC18:48
*** SridharG has joined #openstack-qa18:49
ajayaaHi. Is it advisable to write tempest tests for custom policies?18:50
sdagueajayaa: an example?18:53
*** nati_ueno has quit IRC18:53
*** kgriffs|afk is now known as kgriffs18:54
*** nati_uen_ has joined #openstack-qa18:55
*** cdent has quit IRC18:58
*** mlavalle has quit IRC19:00
ajayaasdague: For e.g. I could define a new role called project_admin and give him privilege just to add users in his project.19:00
*** mlavalle has joined #openstack-qa19:00
*** bauzas has quit IRC19:01
ianwsdague: that's running the f20 job for devstack changes19:01
ajayaasdauge: He won't be able to add a role for an user in different project other than his own.19:01
sdagueajayaa: this is with keystone v3?19:01
ajayaasdague: yes.19:01
sdagueianw: but why the 2nd check experimental immediately after?19:01
*** ildikov has joined #openstack-qa19:02
sdagueajayaa: so I believe keystone v3 tests like that are in process. andreaf was working on something for that19:02
*** Obulpathi has quit IRC19:04
ajayaasdauge: thanks.  andreaf, ping!19:04
ajayaasdague: sorry! different keyboard.19:05
ianwsdague: which change did that happen on?19:05
*** Obulpathi has joined #openstack-qa19:05
sdagueianw: https://review.openstack.org/#/c/97442/19:07
sdaguemaybe it happens first, and gerrit changes order?19:07
ianwsdague: i think that'19:09
andreafajayaa: I'm going to have dinner now I may be online later or tomorrow - if you write your question I'll answer later19:09
ianwsdague: i think that's working as intended; it posts the results of our check (all failing, unfortunately due to known issues) and then asks for the experimental run19:10
sdagueah19:10
ajayaaandreaf, I will send an email to openstack-qa.19:10
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Support suppressing bugs in bot  https://review.openstack.org/9735419:14
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Add suppressed fingerprint for pep8 failures in gate  https://review.openstack.org/9535019:14
*** Obulpathi has quit IRC19:14
mtreinishmaurosr: yeah, what's up19:14
*** Obulpathi has joined #openstack-qa19:15
maurosrqq, v3 and v2 will converge into v2.1, so patches to attribute tests are also in hold? specially something that is only implemented on v319:15
maurosrcause I think it will endup being useful into v2.1 since we want the new functionalities19:15
maurosrso would be welcome anyway19:15
maurosrmtreinish: ^19:15
mtreinishmaurosr: yeah I interpreted it as everything v3 specific in tempest was on hold19:16
mtreinishbut things that also apply to v2.1 are still valide19:16
maurosrok nice =)19:17
*** bauzas has joined #openstack-qa19:17
maurosrtks19:17
*** masayukig has quit IRC19:22
*** adalbas has quit IRC19:25
*** masayukig has joined #openstack-qa19:29
*** Obulpathi has quit IRC19:30
*** Obulpathi has joined #openstack-qa19:31
*** Obulpathi has quit IRC19:33
*** Obulpathi has joined #openstack-qa19:33
*** adalbas has joined #openstack-qa19:38
*** mkoderer has quit IRC19:42
*** SridharG has quit IRC19:47
EmilienMHi there, I would appreciate a review on ceilometer support in Grenade: https://review.openstack.org/#/c/94468/ - Thank you!19:53
EmilienMsdague: since the patch in devstack-gate has been merged, I think the one in grenade is ready for review. jogo already +119:54
mtreinishlifeless: so I was looking at the unittest api and the tb is already stored in the returned testSuite class. So I think the only unittest api change is to add a public method to testSuite that either returns the methodname or None19:56
mtreinishsomething like http://paste.openstack.org/show/82671/19:56
mtreinishthen on the testtools side we do something like https://github.com/mtreinish/testtools/commit/430cec9321f0a37cca801797ffdb205f503c911f and just add a call to that new public method when it gets added to unittest19:57
mtreinishdoes that sound sane, or am I missing something (which is probably the case)19:57
lifelessmtreinish: I'd quibble about the name I think, we don't want this to look like 'and it ran tests'19:58
lifelessmtreinish: in fact, it might be better as a new loader return value kind of things - separate errors and successes19:58
mtreinishlifeless: sure, I'm not attached to that name19:58
*** pbrooko has joined #openstack-qa19:58
lifelessbut would that work? not as a it stands - needs to return an interable, because suites might not be leaf objects19:59
lifelessthere may be many errors19:59
lifelessand you'd then need a composite method (like run() or debug()) to query for instantiation errors across the whole composite structure19:59
mtreinishoh, I see what your saying the Suite isn't necessarily the suite that failed on import it could be the parent suite too20:00
mtreinishok I'll look into that, I was thinking of it strictly from the testtools use case and in that case it's already the leaf object20:01
*** davlaps has quit IRC20:03
mtreinishlifeless: in the meantime what about that testtools patch I referenced? do you think we could merge something like that in the short term. It probably won't really need to change much for python3.5 (assuming the new method is added for 3.5)20:03
lifelessI've found previously that merging things before upstream work is done generally leads to upstream work not being done20:04
lifelessso I'm hesitant about that20:04
*** tmcpeak has joined #openstack-qa20:04
*** ajayaa has quit IRC20:06
mtreinishyeah but that patch is for supporting < python 3.5. Unless we can add the new api to previous releases we're still going to have to guess the failed module name (or use a private attr), and the method of getting the error won't change.20:06
*** weshay has quit IRC20:10
lifelessmtreinish: right but we'll know that there is a path to upstream support, and can make our local code look like that20:14
openstackgerritJamie Lennox proposed a change to openstack-dev/devstack: Enable Horizon apache file on all platforms  https://review.openstack.org/9761420:16
*** mriedem has quit IRC20:19
*** mriedem has joined #openstack-qa20:21
*** Sukhdev has quit IRC20:22
*** weshay has joined #openstack-qa20:25
openstackgerritGrishkin proposed a change to openstack/tempest: Reworked scenario tests to use their own non-default security groups  https://review.openstack.org/8604920:27
*** nati_uen_ has quit IRC20:28
*** mriedem has quit IRC20:33
*** mriedem has joined #openstack-qa20:34
*** mpavlase has quit IRC20:42
*** ekhugen has quit IRC20:47
*** nati_ueno has joined #openstack-qa20:47
*** aysyd has quit IRC20:52
*** nati_uen_ has joined #openstack-qa21:00
*** nati_ueno has quit IRC21:00
*** kgriffs is now known as kgriffs|afk21:00
*** kgriffs|afk is now known as kgriffs21:05
*** dhellmann has quit IRC21:11
*** dims__ has joined #openstack-qa21:13
*** dhellmann has joined #openstack-qa21:13
*** dims_ has quit IRC21:15
*** adalbas has quit IRC21:22
*** safchain has joined #openstack-qa21:25
*** jchai has quit IRC21:26
*** Obulpathi has quit IRC21:30
*** eglynn has joined #openstack-qa21:30
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add missing compute service tags to volume API tests  https://review.openstack.org/9763021:30
*** pafuent has left #openstack-qa21:31
eglynnmtreinish: so just continuing on that skipping-tests-in-branchless-tempest theme from the project/release meeting, if you got a minute?21:34
*** weshay has quit IRC21:34
mtreinisheglynn: sure21:35
eglynnmtreinish: ... so I was a bit confused by the discussion with DinaBelova yesterday evening21:35
eglynn(... we re-hashed it on the openstack-ceilo channel this morning)21:36
*** levi_b has quit IRC21:36
eglynnspecifically this notion ... "the thing is if there is an api bug we could end up skipping a whole bunch of things because the api is misbehaving"21:36
*** levi_b has joined #openstack-qa21:36
eglynni.e. that we need config option as well as a discoverable API ... because the API may be buggy21:36
openstackgerritJamie Lennox proposed a change to openstack-dev/devstack: An example of Certificate Generation with devstack  https://review.openstack.org/4752621:37
openstackgerritJamie Lennox proposed a change to openstack-dev/devstack: SSL protected cinder endpoints  https://review.openstack.org/9763221:37
eglynnmtreinish: ... but surely the API call would generally be used to set the config option, in the case where branchless tempest is run against public cloud X?21:37
mtreinisheglynn: yes but it's an out of band tool21:38
mtreinisheglynn: see http://git.openstack.org/cgit/openstack/tempest/tree/tempest/cmd/verify_tempest_config.py21:38
eglynnmtreinish: out-of-band? ... in what sense?21:38
mtreinisheglynn: it's not integrated into running the tests you have to do it separately21:38
eglynnmtreinish: sure ... so I wanted to run the test against an unknown public cloud, I need to follow two distinct steps21:39
eglynn1. create config file with the correct settings for that cloud21:39
eglynn2. run actual tempest tests21:40
mtreinishyeah that's the basic idea21:40
eglynnmtreinish: in order to accomplish step #1 for a public cloud, I'd need to manually call out to the discoverable API, no?21:40
*** marun is now known as marun_afk21:40
*** levi_b has quit IRC21:41
mtreinishor use a tool to help write the config file, either that script I mentioned before and there is a spec on writing a tool to help generate one from scratch21:41
eglynn... but we appear to be saying that we don't fully "trust" that API21:41
mtreinisheglynn: from the tempest perspective we don't because it's all about verifying it21:41
mtreinishwe did autodiscovery of features and services before21:41
mtreinishand we started skipping broken things in the gate21:42
mtreinishbecause of the number of tests it was hard to notice a few new skips21:42
*** dims__ has quit IRC21:42
mtreinishso that's why we have everything like this be explict in the config file (or at least try to)21:42
eglynnmtreinish: would it be fair to say that we're still just as likely to skip broken things in the tempest-against-public-cloud-X case?21:43
eglynnmtreinish: (... seeing as the only way to populate the config file in that case IIUC is to rely on the API)21:43
mtreinisheglynn: well if you're using the api to discover what features are available then probably. but if you don't know what your running against there's not much you can do about that21:43
eglynnmtreinish: OK, that question was kind of the root of my confusion21:44
eglynnmtreinish: ... as I thought from the original discussion last week that the tempest-against-public-cloud-X case was the major concern21:44
eglynnmtreinish: ... but seems like we can live with some fuzziness in that case21:45
eglynnmtreinish: ... as long as the gate is locked down with static config for each branch21:45
mtreinishyeah, having the api is important for the end user case (which is greater than just tempest's concern) but the config option is a neccesity from tempest perspective21:45
mtreinisheglynn: yeah exactly, which we get but using devstack to generate them21:46
mtreinishs/but/by/21:46
eglynnmtreinish: yep ... static config for each branch in the (still branch-ful) *devstack* as opposed to the (newly branch-less) tempest21:46
mtreinishwhich is honestly the best way to generate a tempest config by getting whatever deployment tooling to generate it21:46
mtreinisheglynn: yeah I'll add something about having feature flags to the main readme for tempest at some point today or tomorrow21:48
eglynnmtreinish: so to summarize ...21:48
eglynnmtreinish: we need *both* a discoverable API and config-driven test exclusion21:49
eglynnmtreinish: ... with the former used (indirectly) for public cloud testing, but *only* the latter relied upon in branchless tempest runs21:49
mtreinishyeah, that's the best way to do it. Although the api is also useful for everyone using ceilo to understand what they can expect to work21:50
*** kgriffs is now known as kgriffs|afk21:51
mtreinishbecause running tempest against an unknown deployment (like a public cloud) is kind of like being an end user of the api21:51
mtreinisheglynn: btw, did you see?: http://lists.openstack.org/pipermail/openstack-dev/2014-May/036262.html21:51
* eglynn looks21:51
mtreinishI'm trying to figure out what to do for the qa day, and I'm wondering if some combined effort on ceilo and tempest might be a good idea21:52
*** masayukig has quit IRC21:52
*** Mike77 has joined #openstack-qa21:52
*** mriedem has quit IRC21:53
eglynnmtreinish: I personally would love to attend something like that21:54
eglynnmtreinish: ... BUT we're planning a ceilo meetup in Paris from July 2nd thru' 4th21:54
*** nati_uen_ has quit IRC21:54
eglynnmtreinish: ... and my boss is likely to shoot me if come back seeking more travel budget :(21:55
eglynnmtreinish: ... remote participation not a realistic option, I assume?21:55
mtreinisheglynn: ok, I can fully understand that. But, you can give it a try anyway right? :)21:55
mtreinishyeah remote participation probably won't work too well21:56
eglynnmtreinish: yep, it never does for those kind of events21:56
*** masayukig has joined #openstack-qa21:57
eglynnmtreinish: ... so is that proposed July 18th date anticipated as a full day?21:58
eglynnmtreinish: (or seeing as its a Friday, projected to finish at lunchtime ... to allow folks who've been there all week start travelling home?)21:58
* eglynn just wondering about possible travel logistics before going begging for budget ...21:58
mtreinishyeah the intent was for the full day. But we haven't locked down the agenda yet.21:59
mtreinishI hadn't even considered the half day because of travel21:59
mtreinishmaybe we'll need to do some shuffling21:59
eglynnmtreinish: cool enough, I look at the travel options and see if I can get my hands on some budget22:00
mtreinisheglynn: ok cool22:00
eglynnmtreinish: ... thanks for the heads-up!22:00
*** mwagner_lap has quit IRC22:02
*** nati_ueno has joined #openstack-qa22:02
*** dims has joined #openstack-qa22:12
*** dustins has quit IRC22:13
*** nati_ueno has quit IRC22:24
*** bauzas has quit IRC22:25
*** eglynn has quit IRC22:28
*** nati_ueno has joined #openstack-qa22:32
*** tosky has quit IRC22:38
*** aimon has quit IRC22:43
openstackgerritAndrea Frittoli  proposed a change to openstack/qa-specs: Spec for running tempest as non-admin  https://review.openstack.org/8696722:44
*** jgrimm has quit IRC22:44
*** vchernomor has quit IRC22:45
*** danfinch has quit IRC22:46
*** danfinch has joined #openstack-qa22:47
*** nati_uen_ has joined #openstack-qa22:47
*** aimon has joined #openstack-qa22:48
*** aimon has quit IRC22:49
*** nati_ueno has quit IRC22:50
*** pbrooko has quit IRC22:51
mtreinishandreaf: you missed one advantage of tenant isolation in that comment. It doesn't constrain the number of workers that we can use when running in parallel22:58
mtreinishbut was the spec actually proposing we remove it, I thought the proposal was to add the option to use a list of creds if admin wasn't provided22:59
andreafmtreinish: my comment was meant as a reply to sdague's comment on the same patch22:59
*** TravT has quit IRC22:59
*** levi_b has joined #openstack-qa23:01
mtreinishoh, I didn't read that as saying the current model goes away, just that we have an alternative which removes the dependence on admin23:01
mtreinishbecause without that what's the point of running tempest without admin if it has to be serial23:02
openstackgerritA change was merged to openstack-infra/devstack-gate: Copy devstacklog.txt  https://review.openstack.org/9725123:02
andreafmtreinish: ok - well it seems that we agree tenant isolation should not go away23:05
*** levi_b has quit IRC23:06
andreafmtreinish: it would nice though to refactor things a bit (doh, not again :P) and have something like a credentials provider, which serves credentials to client managers / tests, and worries about the logic around tenant isolation or not... now we have too many if spread around checking if tenant isolation is on or not23:07
mtreinishandreaf: that's kinda of what tenant isolation does now, you're just talking about bringing the logic up one level out of the individual test base classes.23:09
mtreinishI don't think it would be too bad of a refactor, you can bring that logic into test.py as a first stab at it23:09
mtreinishjust to unify the credential selection logic23:09
mtreinishand store the cred object in a canonical location in the test class which can be referenced for whatever the subclasses need23:13
mtreinishthat way we don't have to break it off into another external thing which I think would be a bit more involved23:13
*** eglynn has joined #openstack-qa23:15
*** mmaglana has quit IRC23:19
*** safchain has quit IRC23:19
*** EmilienM_ has joined #openstack-qa23:20
*** bdperkin_ has joined #openstack-qa23:20
*** DinaBelova2 has joined #openstack-qa23:20
*** salv-orlando has quit IRC23:20
*** bdperkin has quit IRC23:20
*** andreaf has quit IRC23:20
*** ruhe has quit IRC23:20
*** DinaBelova has quit IRC23:20
*** EmilienM has quit IRC23:20
*** DinaBelova2 is now known as DinaBelova23:20
*** EmilienM_ is now known as EmilienM23:20
*** rossella_s has quit IRC23:20
*** ruhe2 has joined #openstack-qa23:20
*** rossella has joined #openstack-qa23:20
*** andreaf has joined #openstack-qa23:20
*** rossella is now known as rossella_s23:20
*** ruhe2 is now known as ruhe23:20
*** bdperkin_ is now known as bdperkin23:21
*** bdperkin has quit IRC23:21
*** bdperkin has joined #openstack-qa23:21
andreafmtreinish: I lost a bit of what you wrote - got disconnected - but I think I understand what you mean23:22
mtreinishandreaf: ok cool. Because I'm not sure I would understand just from what I wrote, even complete :)23:24
*** eglynn-regus has quit IRC23:24
andreafmtreinish: :D23:24
openstackrecheckopenstack/heat change: https://review.openstack.org/94755 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1262566, gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/132365823:25
openstackgerritMiguel Lavalle proposed a change to openstack/qa-specs: Add spec for bp:/neutron-advanced-scenarios  https://review.openstack.org/9560023:39
openstackgerritMiguel Lavalle proposed a change to openstack/qa-specs: Add spec for bp:/neutron-advanced-scenarios  https://review.openstack.org/9560023:41
*** andreaf has quit IRC23:42
*** GMann has joined #openstack-qa23:43
*** oomichi has joined #openstack-qa23:48
oomichihi cyeoh23:49
*** tmcpeak has quit IRC23:51
*** tmcpeak has joined #openstack-qa23:52
*** GMann has quit IRC23:52
*** tmcpeak has quit IRC23:56
*** aimon has joined #openstack-qa23:58

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