Monday, 2013-09-30

*** matsuhashi has joined #openstack-qa00:05
*** sthaha has joined #openstack-qa00:16
*** jasondot_ has quit IRC00:23
kongAnju: ping00:35
kongtime difference...00:37
openstackgerritLingxian Kong proposed a change to openstack/tempest: add server reset_state tests  https://review.openstack.org/4879400:50
konganyone can help to review https://review.openstack.org/#/c/48794/, https://review.openstack.org/#/c/48796/, https://review.openstack.org/#/c/48814/. It's very appreciated!00:56
*** nosnos has joined #openstack-qa01:05
*** ekarlso has quit IRC01:33
openstackgerritJun Jie Nan proposed a change to openstack-dev/devstack: Using no proxy option to skip wget and curl proxy settings in config  https://review.openstack.org/4644701:34
openstackgerritYang Yu proposed a change to openstack/tempest: Adding negative tests for glance API update /v1/images/<image_id>  https://review.openstack.org/4857802:13
openstackgerritwanghao proposed a change to openstack/tempest: add volumes negative tests:  https://review.openstack.org/4887702:14
*** matsuhashi has quit IRC02:16
*** matsuhashi has joined #openstack-qa02:21
openstackgerritA change was merged to openstack-infra/elastic-recheck: Make unit tests pass again with new code layout  https://review.openstack.org/4875802:21
openstackgerritMatthew Treinish proposed a change to openstack-infra/elastic-recheck: Update elasticRecheck imports in bot.py  https://review.openstack.org/4887802:22
*** cyeoh_ is now known as cyeoh02:29
*** matsuhashi has quit IRC02:29
*** matsuhashi has joined #openstack-qa02:30
openstackgerritJinhe Fang proposed a change to openstack/tempest: Add test for admin deleting servers of others  https://review.openstack.org/4888102:52
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Switch neutron smoke job over to parallel  https://review.openstack.org/4855102:55
*** RecheckWatchBotI has quit IRC03:24
*** dims has quit IRC03:58
*** Madan has joined #openstack-qa04:23
*** Madan has quit IRC04:27
*** salv-orlando_ has joined #openstack-qa04:40
*** Madan has joined #openstack-qa04:42
*** zigo_ has joined #openstack-qa04:43
*** SergeyLukjanov has joined #openstack-qa04:43
*** rahmu_ has joined #openstack-qa04:43
*** jpipes has joined #openstack-qa04:44
*** salv-orlando has quit IRC04:49
*** jaypipes has quit IRC04:49
*** afazekas has quit IRC04:49
*** zigo has quit IRC04:49
*** rahmu has quit IRC04:49
*** salv-orlando_ is now known as salv-orlando04:49
*** rahmu_ is now known as rahmu04:49
*** prekarat has joined #openstack-qa04:54
*** salv-orlando has quit IRC04:59
*** jasondot_ has joined #openstack-qa05:16
openstackgerritYang Yu proposed a change to openstack/tempest: Adding negative tests for glance API update /v1/images/<image_id>  https://review.openstack.org/4857805:20
openstackgerritMh Raies proposed a change to openstack/tempest: VPNaas vpnservice test cases  https://review.openstack.org/4733905:32
openstackgerritMh Raies proposed a change to openstack/tempest: VPNaas IKE policies tests  https://review.openstack.org/4734805:34
openstackgerritMh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests  https://review.openstack.org/4781605:34
openstackgerritMh Raies proposed a change to openstack/tempest: VPNaas IPSec Site Connection API tests  https://review.openstack.org/4822005:34
openstackgerritMh Raies proposed a change to openstack/tempest: Adding testcase for dhcp agent scheduler  https://review.openstack.org/4863205:40
*** shilpa_ has joined #openstack-qa05:43
openstackgerritYang Yu proposed a change to openstack/tempest: Adding negative tests for glance API update /v1/images/<image_id>  https://review.openstack.org/4857805:45
openstackgerritYang Yu proposed a change to openstack/tempest: Adding glance API DELETE /images/<imageid> tests  https://review.openstack.org/4798305:48
*** matsuhas_ has joined #openstack-qa05:57
*** matsuhashi has quit IRC05:58
*** nosnos_ has joined #openstack-qa05:58
*** nosnos has quit IRC05:59
*** torandu has quit IRC06:11
*** jasondot_ has quit IRC06:12
*** SergeyLukjanov has quit IRC06:14
*** afazekas has joined #openstack-qa06:16
* afazekas re06:17
MadanCan anybody Help me out to get  test coverage for Openstack components (all/ most of them),,06:18
*** ekarlso has joined #openstack-qa06:21
Madanafazekas: Ping06:22
*** ekarlso has quit IRC06:26
*** ekarlso has joined #openstack-qa06:26
*** ken1ohmichi has joined #openstack-qa06:38
*** jlibosva has joined #openstack-qa06:44
afazekasMadan: pong06:48
afazekasMadan: you need to enable the 'backdoor service' in the nova.conf06:48
Madanafazekas: Hi,, I recently started working openstack tempest,, so can you please help me out to find the test coverage for the openstack components??06:49
afazekashttps://wiki.openstack.org/wiki/Nova/CoverageExtension06:51
MadanThis looks Nice to me,, Thank you,, :)06:54
*** dkorn has joined #openstack-qa06:56
*** matsuhas_ has quit IRC06:57
*** nosnos has joined #openstack-qa07:02
*** nosnos_ has quit IRC07:02
*** jhenner has joined #openstack-qa07:04
*** matsuhashi has joined #openstack-qa07:05
openstackgerritQing Xin Meng proposed a change to openstack/tempest: Add test for creating a server by using scheduler hints.  https://review.openstack.org/4889907:07
*** bashok has joined #openstack-qa07:08
*** tkammer has joined #openstack-qa07:14
*** jhenner has quit IRC07:15
openstackgerritwanghao proposed a change to openstack/tempest: add some negative tests for volume updating:  https://review.openstack.org/4884107:31
*** matsuhashi has quit IRC07:40
*** matsuhashi has joined #openstack-qa07:42
*** matsuhashi has quit IRC07:42
*** ken1ohmichi has quit IRC07:43
*** SergeyLukjanov has joined #openstack-qa07:44
*** matsuhashi has joined #openstack-qa07:44
*** matsuhashi has quit IRC07:45
*** matsuhashi has joined #openstack-qa07:48
*** matsuhashi has quit IRC07:48
openstackgerritYang Yu proposed a change to openstack/tempest: Adding active tests for updating/replacing image members  https://review.openstack.org/4890207:55
*** yassine has joined #openstack-qa07:59
Anjuafazekas:  ping07:59
afazekasAnju: pong07:59
openstackgerritYang Yu proposed a change to openstack/tempest: Adding glance API DELETE /images/<imageid> tests  https://review.openstack.org/4798307:59
Anjuafazekas:   a tests for invalid_volume_id and  nonexistant_volume_id08:00
Anjuare similar ?08:00
*** jlibosva has quit IRC08:01
*** jlibosva has joined #openstack-qa08:01
openstackgerritYang Yu proposed a change to openstack/tempest: Adding glance API DELETE /images/<imageid> tests  https://review.openstack.org/4798308:02
Anjuafazekas:  ^^^08:02
afazekasAnju: if any byte array is allowed to be volume id it is similar08:03
Anjuhttps://review.openstack.org/#/c/48841/3/tempest/api/volume/test_volumes_negative.py08:04
Anjuafazekas : i want to know about this08:04
openstackgerritQing Xin Meng proposed a change to openstack/tempest: Add test for creating a server by using scheduler hints.  https://review.openstack.org/4889908:04
Anjui think these two tests are same08:04
Anjuafazekas:  if I am wrong please clear me08:05
openstackgerritJun Jie Nan proposed a change to openstack-dev/devstack: Using no proxy option to skip wget and curl proxy settings in config  https://review.openstack.org/4644708:05
afazekascan we add a '\0' to the invalid version ?08:06
Anjuafazekas:  not done by me08:06
openstackgerritYang Yu proposed a change to openstack/tempest: Adding negative tests for glance API update /v1/images/<image_id>  https://review.openstack.org/4857808:06
Anjuafazekas:  https://review.openstack.org/#/c/48841/08:07
afazekasid                  | varchar(36)08:07
* afazekas I would deny any non valid utf-8 sequence as input string with BadRequest08:08
afazekasboth string does not looks like a valid uuid, probbaly both utilize the same code path08:09
Anjuokk afazekas08:09
Anjuafazekas:  are you agree that both tests are same08:10
afazekasa very long id can be used for negative and I have doubts it does anything differently with '\0'  , but it is usually good for testing08:11
Anjuokk afazekas thanks08:12
afazekasAnju: ~ 79% sure :)08:12
Anjuif you are saying then it must be 10008:12
Anju:)08:12
*** matsuhashi has joined #openstack-qa08:13
afazekas:)08:13
afazekas'\n' also a tricky character if the string passed to a shell script.. :)08:14
Anjuyes08:15
*** dkorn has quit IRC08:17
*** jlibosva has quit IRC08:21
*** jhenner has joined #openstack-qa08:21
*** jhenner has quit IRC08:24
*** jhenner has joined #openstack-qa08:24
*** fbo_away is now known as fbo08:25
*** prekarat has quit IRC08:25
openstackgerritIvan-Zhu proposed a change to openstack/tempest: port test_images and test_server_actions into v3 part2  https://review.openstack.org/3960908:32
*** dkorn has joined #openstack-qa08:32
*** jlibosva has joined #openstack-qa08:33
*** giulivo has joined #openstack-qa08:35
openstackgerritIvan-Zhu proposed a change to openstack/tempest: meta should be metadata in rebuild server  https://review.openstack.org/4890608:43
openstackgerritYang Yu proposed a change to openstack/tempest: Adding active tests for updating/replacing image members  https://review.openstack.org/4890208:49
openstackgerritQing Xin Meng proposed a change to openstack/tempest: Add test for creating a server by using scheduler hints.  https://review.openstack.org/4889908:55
openstackgerritwanghao proposed a change to openstack/tempest: add some negative tests for volume updating:  https://review.openstack.org/4884108:57
*** jasondot_ has joined #openstack-qa09:00
Anjuafazekas:  ping09:14
Anju:)09:14
Anjuafazekas:  a tag is introduced in v2 api .do you have any idea about this09:14
*** bashok has quit IRC09:17
*** tkammer has quit IRC09:26
*** thomasbiege has joined #openstack-qa09:41
kongcould anyone help to review https://review.openstack.org/#/c/48794/, https://review.openstack.org/#/c/48796/, https://review.openstack.org/#/c/48814/. They both go through Jenkins, and little changes are made.09:47
*** thomasbiege has quit IRC09:50
*** jasondot_ has quit IRC10:12
*** prekarat has joined #openstack-qa10:15
*** prekarat has quit IRC10:46
*** HenryG has joined #openstack-qa10:47
openstackgerrithuangtianhua proposed a change to openstack/tempest: add a negative test for tenant updating:  https://review.openstack.org/4891710:55
*** prekarat has joined #openstack-qa10:59
*** prekarat has quit IRC11:07
mkoderergiulivo: ping11:09
openstackgerritMatthieu Huin proposed a change to openstack-dev/devstack: admin user gets ResellerAdmin role if swift is up  https://review.openstack.org/4861011:14
*** thomasbiege has joined #openstack-qa11:22
mkodererafazekas: could you tell me, how do I get the tempest.log out of this job?11:33
mkodererhttps://jenkins01.openstack.org/job/periodic-tempest-devstack-vm-stress/32/11:33
afazekascheck the file uploads at the end of the console output: https://jenkins01.openstack.org/job/periodic-tempest-devstack-vm-stress/32/console11:36
afazekashttp://logs.openstack.org/periodic/periodic-tempest-devstack-vm-stress/devstack-precise-hpcloud-az3-418188/32/logs/11:36
afazekas'/srv/static/logs/' == http://logs.openstack.org/11:38
Anjuafazekas:  ^^^11:39
mkodererafazekas: ok thanks11:39
mkodererafazekas: stress test worked the last days but this night it failed11:39
*** kong has quit IRC11:41
afazekasis the number or time limited stop working now properly ?11:45
*** rfolco has joined #openstack-qa11:46
*** dims has joined #openstack-qa11:48
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Allow openrc to loadable in zsh  https://review.openstack.org/4892211:51
*** mkollaro has joined #openstack-qa11:52
mkodererafazekas: yes works now11:56
*** dkranz has joined #openstack-qa11:56
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Allow openrc to be loaded in zsh  https://review.openstack.org/4892211:57
giulivopong mkoderer11:59
mkoderergiulivo: https://review.openstack.org/#/c/48147/11:59
mkoderergiulivo: what is your opinion with the newest patchset12:00
mkoderergiulivo: ... I disagree.......12:00
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Allow openrc to be loaded in zsh  https://review.openstack.org/4892212:04
mkodererafazekas: in the nightly before we got 6006 single action executed. That's cool.12:04
giulivomkoderer, well, to me the idea looks okay12:04
giulivowhat is that you don't agree with?12:04
giulivothe stop_on_error default?12:05
mkoderergiulivo: he intorduced a new argument12:05
mkodererbut where is he using it?12:05
giulivoso isn't that the stop_on_error ?12:06
mkodereryep12:06
mkodererso it's always False?12:07
mkodererso means the user should change it manually?12:07
mkoderergiulivo: ^12:07
giulivoyeah it's not passing it12:08
giulivoI noticed that but I mean, the idea looks okay12:08
mkoderergiulivo: yes the idea is ok12:08
mkoderergiulivo: maybe we just add a config parameter.. somehing like "copy_all_logs=False"12:09
giulivowell, stop_on_error looks okay to me12:09
giulivoI mean, I'd expect the errors to be all logged if not stopped and not if stopped12:09
giulivoonly, that is a parameter one should pass to the decorator12:10
giulivoand I agree with you maybe it could more a sort of global 'config' parameter12:10
mkoderergiulivo: how is this related to the decorator?12:10
mkodererwith a configuration parameter it's ok for me as well :)12:11
*** dims has quit IRC12:12
*** Madan has quit IRC12:12
*** dims has joined #openstack-qa12:12
giulivosec I'm trying to figure what is the caller :)12:14
giulivooh that's "stop" from the json file, correct?12:14
*** hemnafk has quit IRC12:15
mkoderergiulivo: mhh there is a parameter in run_stress.py12:15
mkoderercalled "-S" / "--stop"12:15
*** hemnafk has joined #openstack-qa12:16
mkodereron the first error the whole stress test framework should stop12:16
mkodererafazekas: did you experienced something similar before? http://logs.openstack.org/periodic/periodic-tempest-devstack-vm-stress/devstack-precise-hpcloud-az3-418188/32/logs/tempest.txt.gz12:17
mkodererafazekas: it seems that the whole process didn't start at all12:17
giulivomkoderer, I see so given we call stress_openstack with both "tests" and "ns" maybe we can put the stop_on_error in "tests" , the json file?12:18
giulivomkoderer, nah it's global I see12:23
mkoderergiulivo: mhhh for me it was a general setting12:23
mkoderer:)12:24
*** adalbas has joined #openstack-qa12:24
* mkoderer is in a meeting12:31
afazekas2013-09-30 06:08:14.029 32146 CRITICAL tempest [-] local variable 'attrs' referenced before assignment12:31
*** adalbas has quit IRC12:31
afazekasis it possible the iterate_tests(list) is empty ?12:32
afazekasor is it raise an exception in all iteration ?12:33
afazekasexception logging is missing!!12:34
openstackgerritThomas Maddox proposed a change to openstack-dev/devstack: Add back rpc_notifier when ceilometer is enabled  https://review.openstack.org/4835112:35
*** weshay has joined #openstack-qa12:37
*** adalbas has joined #openstack-qa12:43
*** jasondot_ has joined #openstack-qa12:45
*** dkranz has quit IRC13:03
*** mriedem has joined #openstack-qa13:08
*** nosnos has quit IRC13:13
*** dkranz has joined #openstack-qa13:16
openstackgerritDirk Mueller proposed a change to openstack/tempest: Make smoketests working again with Python 2.6/nose  https://review.openstack.org/4893013:19
*** jhenner has quit IRC13:21
*** jhenner has joined #openstack-qa13:22
*** dkorn has quit IRC13:22
*** openstackgerrit has quit IRC13:22
*** openstackgerrit has joined #openstack-qa13:22
*** dkorn has joined #openstack-qa13:27
openstackgerritnayna-patel proposed a change to openstack/tempest: Adds "list hosts" test case - Cinder  https://review.openstack.org/4893313:28
openstackgerritafazekas proposed a change to openstack-infra/devstack-gate: Using the root user for tempest run  https://review.openstack.org/4210113:30
openstackgerritQing Xin Meng proposed a change to openstack/tempest: Add test for creating a server by using scheduler hints.  https://review.openstack.org/4889913:34
*** adalbas has quit IRC13:36
*** shilpa_ has quit IRC13:38
*** adalbas has joined #openstack-qa13:53
*** jhenner has quit IRC13:57
*** jhenner has joined #openstack-qa13:58
*** dkranz has quit IRC14:02
*** adalbas has quit IRC14:06
*** prekarat has joined #openstack-qa14:13
*** kong has joined #openstack-qa14:13
*** prekarat has quit IRC14:14
*** DennyZhang has joined #openstack-qa14:16
dimsjgriffith, around?14:18
jgriffithdims: yo14:18
jgriffithdims: looks like you found all my little notes etc :)14:19
jgriffithdims: you understand what the problem is (or what I think it is)?14:19
*** dkranz has joined #openstack-qa14:19
*** adalbas has joined #openstack-qa14:19
afazekasgiulivo: did you have time to figure out how to split the syslog / messages ,  (swift log)14:20
dimsjgriffith, have been following along :) i now spotted 2 "Creating iscsi_target for: volume-a109df91-d5b4-47b6-9c3e-17e514a08fe4", after the first call, i see the lun1 and after the 2nd lun1 is awol - http://logs.openstack.org/76/48876/4/check/check-tempest-devstack-vm-postgres-full/2ce7533/logs/screen-c-vol.txt.gz14:20
kongAnju: ping14:20
*** torandu has joined #openstack-qa14:20
giulivono honestly I haven't checked that, got a close friend held in prison in russia14:21
giulivohttp://www.greenpeace.org/international/en/news/features/From-peaceful-action-to-dramatic-seizure-a-timeline-of-events-since-the-Arctic-Sunrise-took-action-September-18-CET/14:21
dimsjgriffith, "2013-09-30 13:08:29.566" and "2013-09-30 13:08:31.439" are the timestamps for the creating iscsi_target line14:21
giulivoand we don't get updates from a few days now14:21
*** anteaya has joined #openstack-qa14:21
giulivoa bit off topic though :)14:23
dimsgiulivo, am speechless!14:23
jgriffithdims: so what's interesting is you notice it comes back later :)14:24
jgriffithdims: I'll need to track down the sequence there14:24
giulivoyeah I know it is OT but they like boarded the boat and are now helding people for 2 months14:24
dimsjgriffith, yep14:24
*** tkammer has joined #openstack-qa14:24
giulivodims, ^^ have no idea how to get in touch with the guys cause we never thought things could get so wrong14:25
openstackgerritMatthieu Huin proposed a change to openstack-dev/devstack: admin user gets ResellerAdmin role if swift is up  https://review.openstack.org/4861014:29
* jgriffith is still a big frightened after looking closer at n-cpu logs over the week-end14:29
jgriffiths/big/bit/14:29
dkranzafazekas: If a compute API with POST accepts an "optional" argument, is it OK to call http request with {"opt_arg_name":  None} ?14:31
dkranzafazekas: When the intent is to no provide a value for the optional parameter.14:32
jgriffithdims: check out http://logs.openstack.org/76/48876/4/check/check-tempest-devstack-vm-postgres-full/2ce7533/logs/screen-c-vol.txt.gz#_2013-09-30_13_08_30_87814:32
dkranz  not provide14:32
jgriffithdims: follow down to the next targets display14:32
jgriffithdims: WTF happened to Lun: 1 on Target 2 ?14:33
jgriffithdims: interesting...14:35
jgriffithdims: we created the target twice!14:35
dimsjgriffith, right14:37
jgriffithdims: think I found it14:39
jgriffithdims: stupid logic error14:40
jgriffithdims: cin.volume.drivers.lvm.py line 42614:40
jgriffithdims: that try block should set attempts = 0 if it doesn't raise14:41
jgriffithdims: patch is on it's way14:42
sdagueI'm going to be super excited if you guys nailed that bug14:43
dims:) touch wood14:43
*** DennyZhang has quit IRC14:44
jog0I concur with sdague14:46
sdaguejog0: oh, you're up early :)14:46
sdaguehey, so on elastic-recheck, I wanted to talk about unit tests in it14:46
jog0sdague: yup, morning14:46
jog0sdague: yeah  unit test is being generous14:46
sdaguecan we separate out the unit tests to be things that can be run without touching the network14:46
jog0sdague: ++ yeah14:47
sdaguethen have a separate tool for data validation14:47
jog0eventually we want to pull out the queries.json file (maybe as a different format) and put that in a seperate repo14:47
jog0that is gated by the queries test14:48
sdagueso I actually think leaving the brain (i.e. queries) in elastic-recheck is fine14:48
mtreinishjog0: the bot.py gerrit tests also talk to gerrit so its not just the queries tests14:48
*** dkranz has quit IRC14:48
jog0mtreinish: right14:48
jog0not sure how many tests don't touch the internet14:48
jog01? maybe?14:48
sdaguejog0: as far as I can tell, very few14:48
jgriffithdims: sdague https://review.openstack.org/#/c/48970/14:49
jog0sdague: anyway I am up finishing some slides for a talk in Bend, OR today. but the short of it is go for it14:49
sdagueok14:49
sdaguejog0: you doing the Ales conf?14:49
jog0sdague: yup14:50
sdaguecool14:50
sdaguejgriffith: cool, any way to buttress that up with unit tests, so someone else doesn't break it later by accident?14:50
jgriffithsdague: yeah, looking at that now...14:50
sdaguecoolio14:51
jgriffithsdague: I just want to get that through and see if it fixes us up (it should)14:51
sdagueyep14:51
sdaguealso should just keep running recheck on it until it merges, to collect data on it14:53
*** RecheckWatchBot has quit IRC14:53
giulivojgriffith, any idea why that wasn't causing issues in the normal circumstances?14:55
giulivolooks to me it would just create the target twice14:55
jgriffithgiulivo: depending on what the system is doing it may not care14:56
jgriffithgiulivo: ie if the backing lun isn't "busy" it just goes with it14:56
jgriffithand recreates it14:57
jgriffithbut if there's something actually going on with the system, or it's slow it pukes14:57
giulivoah but in normal conditions you can have the same backing attached to the multiple targets then14:57
giulivoeven outside of cinder I mean14:58
jgriffithgiulivo: oh... um, yes if I follow you correctly14:58
sdagueafazekas: you about?14:59
sdagueI was trying to get the debug info for neutron to actually work, as calling sudo in a tempest test isn't going to work in the gate (no tty)14:59
sdaguebut now everything seems to get lost in not logging it14:59
giulivojgriffith, sorry I was speculating on the meaning of "busy"; if the device is part of another target, that apparently doesn't make it busy14:59
jgriffithgiulivo: so in the Cinder case here it's not going to try and recreate the controller lun, there's nothing wrong with it so it reuses it15:00
mtreinishsdague: please don't pull in rootwrap :)15:00
jgriffithgiulivo: oh... now, what I'm getting at is that the backing file "lun" is busy15:00
sdaguemtreinish: nope, and I actually think we can get plenty of info in the no sudo case15:00
jgriffithmay or may not have anything to do with the device itself15:00
sdaguemtreinish: https://review.openstack.org/#/c/48792/ is what I did, but info seems to not print out15:00
sdagueany thoughts, I know you were tweaking logging at one point15:01
openstackgerritBob Ball proposed a change to openstack-dev/devstack: XenAPI: Get the management network dynamically  https://review.openstack.org/4883815:01
mtreinishsdague: ok, I never saw the sudo patch so I'm not sure what the goal was there. I've just seen the gate errors15:01
sdaguemtreinish: the goal was to dump a bunch of the network state info so we could get closer to figuring out root cause15:01
sdagueit had a lot of bugs15:01
*** dkranz has joined #openstack-qa15:01
jgriffithgiulivo: so if we hadn't tried to reuse the target id it would have worked the way you described15:02
mtreinishsdague: so for logging anything executed at the test method level (after super().setUp()) will be both logged in the tempest log file and printed on stdout during test failure15:02
mtreinishbut if the call path happens before the super call (setUpClass) it'll just go to the tempest log file and not stdout15:03
mtreinishsdague: I'm not sure where log_ip_ns() is being called from but that may be your issue15:03
giulivojgriffith, got it now, thanks, so it's the same target not a new one, but to which it was adding the same backing15:03
jgriffithgiulivo: yep15:04
jgriffithgiulivo: mostly the issue is using the same lun again15:04
sdaguemtreinish: so this is in tearDownClass15:05
jgriffithgiulivo: you could probably do something like this if you were to increment the iscsi_target number (which is a horrid var name btw)15:05
mtreinishsdague: https://git.openstack.org/cgit/openstack/tempest/tree/tempest/test.py#n199 that's where we set the logging capture so I think tearDownClass() is outside the fixture15:06
mtreinishbut it should be in the tempest log file15:06
sdagueoh, actually, it hit a different neutron fail15:09
sdaguethis is actually an awesome baseline for the state of things - https://review.openstack.org/#/c/48792/15:10
sdagueI've managed to get it to fail 60% of runs, but not yet on the fail I need to test things15:10
mtreinishsdague: hahaha, fun15:10
*** SergeyLukjanov has quit IRC15:15
openstackgerritJames E. Blair proposed a change to openstack-infra/elastic-recheck: Switch queries file to YAML  https://review.openstack.org/4898715:19
mtreinishjeblair: ^^^ you really didn't like the quotes :)15:22
jeblairmtreinish: i really didn't like deleting quotes!  we have computers that can do that for us! :)15:23
*** matsuhashi has quit IRC15:25
*** dkorn has quit IRC15:25
*** SergeyLukjanov has joined #openstack-qa15:26
*** prekarat has joined #openstack-qa15:28
*** torandu has quit IRC15:29
*** torandu has joined #openstack-qa15:31
*** torandu has joined #openstack-qa15:31
*** kong has quit IRC15:32
*** sthaha has quit IRC15:38
*** jasondot_ has quit IRC15:43
*** dkranz has quit IRC15:43
*** thomasbiege1 has joined #openstack-qa15:43
*** thomasbiege1 has quit IRC15:44
*** jasondot_ has joined #openstack-qa15:47
openstackgerritJames E. Blair proposed a change to openstack-infra/elastic-recheck: Switch queries file to YAML  https://review.openstack.org/4898715:47
afazekasre15:49
*** giulivo has quit IRC15:53
*** dkranz has joined #openstack-qa15:54
afazekasdkranz: It is not really a good practice15:54
dkranzafazekas: I didn't like it bug it seems to work.15:56
dkranzafazekas: I saw this in the aggregates patch you recently reviewed15:57
dkranzafazekas: Do you know where that None turns into "ignore this"?15:57
afazekasdkranz: It will translated to 'null' in the json15:59
dkranzafazekas: Sure. I guess the server interprets null as ignore this. Blah.16:00
afazekassdague:  the sudo is working in devstack and in the rootwrap , they are probbaly using the same tty settings16:04
afazekassdague: AFAIK ubuntu is not strict about sudo+tty ,16:06
afazekasThe stack user loses the ability to run arbitrary command here https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L24516:07
afazekasbut still has ta capability to use the *-rootwrap files16:07
*** hemnafk is now known as hemna16:08
afazekasdkranz: Do you think we should remove all default value for all optional parameter ?16:08
*** thomasbiege1 has joined #openstack-qa16:08
*** giulivo has joined #openstack-qa16:08
openstackgerritA change was merged to openstack-infra/elastic-recheck: Switch queries file to YAML  https://review.openstack.org/4898716:08
*** thomasbiege1 has quit IRC16:08
dkranzafazekas: No, because they really are optional in the API.16:09
dkranzafazekas: It's just that I would expect to handle a non-provided parameter by not including in the dict16:09
dkranzafazekas: But setting to None seems to work.16:09
dkranzafazekas: So I would be included to just accept this rather than putting in extra "ifs" to not set the not-supplied value in the dict16:10
dkranzafazekas:    "inclined to just"16:10
jog0is there a bug filed for http://logs.openstack.org/41/48341/4/check/check-tempest-devstack-vm-neutron/f4c6356/testr_results.html.gz16:10
jog0and logstash query16:10
*** yassine has quit IRC16:10
afazekasdkranz: I do not really like the setting to None style, but usually I do not give -1 score just for this16:11
dkranzafazekas: OK, then I won't either :)16:11
*** jlibosva has quit IRC16:12
afazekasjgriffith: thank you16:24
jgriffithafazekas: for?16:29
jgriffithafazekas: you're welcome of course ;)16:30
*** fbo is now known as fbo_away16:38
*** afazekas has quit IRC16:38
*** mikal has quit IRC16:40
*** mlavalle has joined #openstack-qa16:41
Anjuafazekas : ping16:45
*** kong has joined #openstack-qa16:45
*** jeblairtest has joined #openstack-qa16:46
*** jeblairtest has quit IRC16:46
*** SergeyLukjanov has quit IRC16:48
*** mikal has joined #openstack-qa16:49
konggiulivo: ping16:49
*** jeblairtest has joined #openstack-qa16:55
*** mkollaro has quit IRC16:58
openstackgerritTal Kammer proposed a change to openstack/tempest: Initial basic setup of openstack and tempest config file  https://review.openstack.org/4292016:58
Anjudkranz:  ping17:00
kongAnju: here?17:02
Anjukong hii17:02
*** jeblairtest has joined #openstack-qa17:02
kongAnju: :), may you please help to give a review?https://review.openstack.org/#/c/48794/17:03
kongAnju: I found you have reviewed...17:05
kongAnju: thanks for your time!17:05
openstackgerritSean Dague proposed a change to openstack/tempest: attempt to make the sudo call for debug gate friendly  https://review.openstack.org/4879217:06
Anjukong : :)17:06
*** jeblairtest has joined #openstack-qa17:08
*** jeblairtest has quit IRC17:08
kongAnju, I just want another +2, because in China, we will have National Day Holidays, and I will be off the office.17:09
kongfor 8 days.17:09
*** jeblairtest has joined #openstack-qa17:11
*** ekarlso has quit IRC17:13
AnjuKong : it will be merged soon17:13
kongAnju, I hope that.17:15
*** jeblairtest has joined #openstack-qa17:16
*** jeblairtest has quit IRC17:17
*** jeblairtest has joined #openstack-qa17:17
*** tkammer has quit IRC17:19
*** morganfainberg has quit IRC17:22
*** ekarlso has joined #openstack-qa17:22
kongSean, ping17:22
*** morganfainberg has joined #openstack-qa17:22
*** rockyg has joined #openstack-qa17:24
*** thomasbiege1 has joined #openstack-qa17:25
*** SergeyLukjanov has joined #openstack-qa17:26
*** thomasbiege1 has quit IRC17:28
openstackgerritDirk Mueller proposed a change to openstack/tempest: Do not check for id in the keystone output  https://review.openstack.org/4901717:28
openstackgerritJames E. Blair proposed a change to openstack-infra/elastic-recheck: Make pid file configurable  https://review.openstack.org/4901817:29
*** lynnkong has joined #openstack-qa17:29
*** kong has quit IRC17:31
*** thomasbiege1 has joined #openstack-qa17:31
*** morganfainberg has quit IRC17:33
*** mdrnstm has joined #openstack-qa17:33
*** mdrnstm is now known as morganfainberg17:34
*** thomasbiege1 has quit IRC17:35
*** ekarlso has quit IRC17:39
*** ekarlso has joined #openstack-qa17:39
*** mlavalle has quit IRC17:39
*** lynnkong has left #openstack-qa17:40
*** lynnkong has joined #openstack-qa17:40
*** lynnkong has quit IRC17:46
*** jasondot_ has quit IRC17:59
dkranzsdague: https://review.openstack.org/#/c/48792/5 has a "WIP: remove code before merge" or somesuch in the added code. Was it supposed to be ready for review?18:01
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: run tempest tests under tempest user  https://review.openstack.org/4902118:01
sdaguedkranz: no, not really18:02
dkranzsdague: OK, didn't see any indication.18:03
*** prekarat has quit IRC18:06
*** fbo_away is now known as fbo18:20
jog0do we have a bug for this: http://logs.openstack.org/03/47503/16/check/check-tempest-devstack-vm-neutron/0b35b85/testr_results.html.gz18:24
jog0sdague: ^18:24
sdaguejog0: no18:25
sdaguethough I know the query18:25
jog0sdague: cool lets file a bug and get a query in18:25
* jog0 files a bug, nut sure what title should be yet18:26
jog0not*18:26
sdagueoh, sorry, we have a bug18:26
sdaguewe don't have a recheck18:26
sdaguejog0: this is the bug - https://bugs.launchpad.net/swift/+bug/122400118:27
jog0sdague:  whats the root cause? or is that unknown?18:28
jog0don't know if this is related but this looks bad http://logs.openstack.org/03/47503/16/check/check-tempest-devstack-vm-neutron/0b35b85/logs/screen-q-dhcp.txt.gz#_2013-09-30_17_56_22_68218:30
sdaguejog0: we don't actually know the root cause18:30
sdaguewhich is why the half implemented sudo dump calls are there18:30
*** afazekas has joined #openstack-qa18:30
openstackgerritA change was merged to openstack-infra/elastic-recheck: Make pid file configurable  https://review.openstack.org/4901818:31
*** openstackgerrit has quit IRC18:47
*** openstackgerrit has joined #openstack-qa18:47
jog0sdague: this is a funny one18:48
jog0so many neutron stacktraces ... sigh18:56
jog0neutron has 10 critical bugs !18:57
dimsso, am staring at the @message:"Second simultaneous" issues19:00
dimscan't figure out where to start19:01
dimsmonkeypatching httplib2 did not seem to help - https://review.openstack.org/#/c/49011/1/neutronclient/client.py,unified19:01
openstackgerritJeremy Stanley proposed a change to openstack-infra/elastic-recheck: Install an executable for the elastic-recheck bot  https://review.openstack.org/4903319:04
openstackgerritMorgan Fainberg proposed a change to openstack/tempest: Unskip test_tokens and update expected status to 404 from 401  https://review.openstack.org/4903519:13
*** adalbas has quit IRC19:15
*** jhenner has quit IRC19:18
*** mlavalle has joined #openstack-qa19:29
openstackgerritMorgan Fainberg proposed a change to openstack/tempest: Unskip test_tokens and update expected status to 404 from 401  https://review.openstack.org/4903519:37
*** adalbas has joined #openstack-qa19:38
openstackgerritA change was merged to openstack-infra/elastic-recheck: Install an executable for the elastic-recheck bot  https://review.openstack.org/4903319:49
*** openstackrecheck has joined #openstack-qa20:18
*** adalbas has quit IRC20:23
jeblairopenstackrecheck: meet jog020:35
jeblairjog0: meet openstackrecheck20:36
*** jasondot_ has joined #openstack-qa20:36
mtreinishjeblair: cool20:36
mtreinishjeblair: lets see how long it stays up :)20:36
mtreinish(it would get stuck in loops pretty frequently on my laptop)20:37
jog0openstackrecheck: why so silent https://review.openstack.org/#/c/45134/10 just failed20:37
jog0was hit by https://bugs.launchpad.net/bugs/122947520:37
jog0mtreinish: I have elastirecheck.py running by me with no issues20:37
mtreinishjog0: ok, the log filtering probably improved stability because we don't wait a long time for ES anymore20:40
mtreinishI haven't run the bot since that change went through20:40
*** jhenner has joined #openstack-qa20:43
*** fbo is now known as fbo_away20:45
dkranzmtreinish: What is the recommended way to install tempest on a vanilla machine without devstack?20:45
dkranzmtreinish: I will update the README when I get it to work20:46
jog0mtreinish:  it did!20:46
dkranzmtreinish: I have only ever done this by looking at errors and then pip installing more stuff20:47
mtreinishdkranz: I don't think there is a recommended way. I mean if you look at the devstack steps installing it is pretty straight forward: it's pip install -r requirements.txt test-requirements.txt and make a conf file for your environment20:47
mtreinishthen optionally you can run setup.py if you're using tools that import tempest libs20:48
dkranzmtreinish: Ah. Last time I tried this it didn't work. I'll try again.20:48
*** jeblairtest_ has joined #openstack-qa20:48
*** jeblairtest_ is now known as jeblairtest__20:48
*** jeblairtest__ is now known as jeblairtest___20:48
*** jeblairtest___ is now known as jeblairtest____20:48
*** jeblairtest_ has joined #openstack-qa20:51
*** jeblairtest_ is now known as jeblairtest__20:51
*** jeblairtest_ has joined #openstack-qa20:51
*** jeblairtest_ is now known as jeblairtest__20:51
*** openstackrecheck has quit IRC20:51
*** openstackrecheck has joined #openstack-qa20:51
*** jeblairtest_ has joined #openstack-qa20:51
*** jeblairtest_ is now known as jeblairtest__20:51
*** jeblairtest__ is now known as jeblairtest___20:51
*** jeblairtest___ is now known as jeblairtest____20:51
*** jeblairtest____ is now known as jeblairtest_____20:51
*** jeblairtest has quit IRC20:52
*** jeblairtest has joined #openstack-qa20:55
*** mordred has quit IRC20:57
*** mordred has joined #openstack-qa20:57
*** rfolco has quit IRC21:04
*** jeblairtest has joined #openstack-qa21:14
mtreinishdkranz: on https://review.openstack.org/#/c/47720/ we actually shouldn't have a lock there sdague made a mistake21:16
mtreinishthe locking is only for aggregates with hosts21:16
dkranzmtreinish: Should I cancel it?21:17
mtreinishdkranz: I can just rip out the lock after it merges21:17
dkranzmtreinish: It won't hurt, right?21:17
mtreinishdkranz: no it shouldn't, it could just mask an issue21:18
mtreinishwhich is really unlikely21:18
dkranzmtreinish: OK, we can rip it out after then.21:18
*** jeblairtest has joined #openstack-qa21:19
*** jeblairtest has quit IRC21:19
mtreinishdkranz: actually I think some of the other az tests have duplicate locks I'll take care of those at the same time21:21
dkranzmtreinish: OK, thanks.21:21
*** hootjr has joined #openstack-qa21:24
*** weshay has quit IRC21:26
*** openstackrecheck has quit IRC21:26
*** openstackrecheck has joined #openstack-qa21:27
*** jasondot_ has quit IRC21:37
*** dkranz has quit IRC21:39
*** openstackrecheck has quit IRC21:39
*** openstackrecheck has joined #openstack-qa21:40
*** mriedem has quit IRC21:40
sdaguemtreinish: I thought when I looked at the fail results on that test it was az racing21:40
sdaguewhich is why I suggested the change21:40
sdaguebut I might have read the tests wrong21:41
mtreinishsdague: there shouldn't be a race in that test21:41
mtreinishit just makes a new aggregate with an az, then updates the name of the aggregate and az21:42
mtreinishthere should be a conflict with other aggregates tests I think21:42
mtreinishlet me double check21:42
sdaguesure21:45
*** openstackrecheck has quit IRC21:49
*** openstackrecheck has joined #openstack-qa21:49
mtreinishsdague: yeah I don't see any obvious races by inspection with any of the other az/aggregates tests21:49
mtreinishI'll propose the change after it merges and we'll see what happens21:49
sdagueyeh21:50
*** openstackrecheck has quit IRC21:59
jog0sdague: can you put a query into elastic-recheck for  https://bugs.launchpad.net/swift/+bug/122400121:59
jog0since you have a reasonable one21:59
sdaguejog0: yeh, in elastic recheck, are you doing something else that ensures @message="" isn't tokenized?22:00
sdaguebecause that was my only issue in putting a query in22:01
jog0sdague: @message:"tempest.scenario.test_network_basic_ops AssertionError: Timed out waiting for" AND @fields.filename:"console.html" AND @fields.build_status:"FAILURE"22:03
jog0with the yaml stuff it should be pretty easy22:04
jog0and running the test_query file will make sure its good22:04
*** openstackrecheck has joined #openstack-qa22:04
*** hootjr has quit IRC22:05
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: added bug #1224001 to skynet  https://review.openstack.org/4907222:06
*** openstackrecheck has quit IRC22:08
*** openstackrecheck has joined #openstack-qa22:11
openstackrecheckopenstack/requirements change: https://review.openstack.org/49064 failed tempest because of: https://bugs.launchpad.net/bugs/122947522:14
jeblairah, there it is22:14
clarkbwoot22:14
jog0\o/22:15
jeblairleft a gerrit comment too22:15
jog0\o/ \o/22:16
*** rockyg has quit IRC22:18
openstackrecheckopenstack/swift change: https://review.openstack.org/48990 failed tempest because of: https://bugs.launchpad.net/bugs/122633722:18
mtreinishyay22:18
openstackrecheckopenstack/neutron change: https://review.openstack.org/47432 failed tempest with an unrecognized error22:19
*** rockyg has joined #openstack-qa22:19
*** SergeyLukjanov has quit IRC22:24
*** rhsu1 has joined #openstack-qa22:25
*** DennyZhang has joined #openstack-qa22:26
openstackgerritJames E. Blair proposed a change to openstack-infra/elastic-recheck: Remove stray backslash  https://review.openstack.org/4907822:34
openstackgerritA change was merged to openstack-infra/elastic-recheck: added bug #1224001 to skynet  https://review.openstack.org/4907222:34
*** jhenner has quit IRC22:35
openstackrecheckopenstack/python-novaclient change: https://review.openstack.org/48893 failed tempest because of: https://bugs.launchpad.net/bugs/122947522:42
*** hemna has quit IRC22:42
*** hemna has joined #openstack-qa22:43
openstackrecheckopenstack/cinder change: https://review.openstack.org/48284 failed tempest because of: https://bugs.launchpad.net/bugs/122947522:46
*** mlavalle has quit IRC22:52
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Make test_queries save stdout in tox testr  https://review.openstack.org/4908022:53
openstackrecheckopenstack/heat change: https://review.openstack.org/49075 failed tempest because of: https://bugs.launchpad.net/bugs/122346922:57
openstackrecheckopenstack/heat change: https://review.openstack.org/49073 failed tempest because of: https://bugs.launchpad.net/bugs/122346922:58
openstackrecheckopenstack/horizon change: https://review.openstack.org/49019 failed tempest with an unrecognized error22:58
openstackrecheckopenstack/cinder change: https://review.openstack.org/48443 failed tempest with an unrecognized error23:01
openstackrecheckopenstack/cinder change: https://review.openstack.org/48460 failed tempest with an unrecognized error23:02
*** DennyZhang has quit IRC23:04
openstackrecheckopenstack/neutron change: https://review.openstack.org/47432 failed tempest with an unrecognized error23:06
openstackgerritA change was merged to openstack-infra/elastic-recheck: Remove stray backslash  https://review.openstack.org/4907823:08
openstackrecheckopenstack/horizon change: https://review.openstack.org/49054 failed tempest because of: https://bugs.launchpad.net/bugs/122633723:14
openstackrecheckopenstack/cinder change: https://review.openstack.org/48895 failed tempest because of: https://bugs.launchpad.net/bugs/122316123:23
openstackrecheckopenstack/swift change: https://review.openstack.org/45134 failed tempest with an unrecognized error23:28
openstackrecheckopenstack/nova change: https://review.openstack.org/49079 failed tempest because of: https://bugs.launchpad.net/bugs/122633723:36
*** hemna is now known as hemnafk23:37
openstackrecheckopenstack/horizon change: https://review.openstack.org/49085 failed tempest because of: https://bugs.launchpad.net/bugs/122947523:38
openstackrecheckopenstack/cinder change: https://review.openstack.org/49083 failed tempest because of: https://bugs.launchpad.net/bugs/122947523:38
*** dims has quit IRC23:44
openstackrecheckopenstack/cinder change: https://review.openstack.org/48443 failed tempest with an unrecognized error23:47
openstackrecheckopenstack/cinder change: https://review.openstack.org/48460 failed tempest with an unrecognized error23:48
openstackrecheckopenstack/neutron change: https://review.openstack.org/49023 failed tempest with an unrecognized error23:53
openstackrecheckopenstack/horizon change: https://review.openstack.org/49019 failed tempest with an unrecognized error23:55
*** matsuhashi has joined #openstack-qa23:58
openstackrecheckopenstack/nova change: https://review.openstack.org/48570 failed tempest with an unrecognized error23:58
*** dims has joined #openstack-qa23:59

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