Thursday, 2013-08-29

« Wednesday, 2013-08-28 Index Friday, 2013-08-30 »
*** hemna is now known as hemnafk00:08
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add common "snapshot_server" method  https://review.openstack.org/4365800:19
*** krtaylor has quit IRC00:25
*** krtaylor has joined #openstack-qa00:33
*** sthaha has joined #openstack-qa00:57
*** sthaha has joined #openstack-qa00:57
*** sthaha has quit IRC00:58
openstackgerritYuuichi Fujioka proposed a change to openstack/tempest: Append some operations to boot from volume pattern  https://review.openstack.org/4104801:02
*** jasondotstar has joined #openstack-qa01:10
*** giulivo has quit IRC01:11
*** sthaha has joined #openstack-qa01:12
*** sthaha has joined #openstack-qa01:12
openstackgerritYuuichi Fujioka proposed a change to openstack/tempest: Append some operations to boot from volume pattern  https://review.openstack.org/4104801:13
openstackgerritA change was merged to openstack-dev/devstack: Adds new tag for testonly packages in package lists  https://review.openstack.org/3812701:17
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add a scenario test for verifying user scope  https://review.openstack.org/4399801:25
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add common "create_server_snapshot" method  https://review.openstack.org/4365801:27
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add a scenario test for verifying user scope  https://review.openstack.org/4399801:29
*** jaypipes has joined #openstack-qa01:34
*** ChanServ sets mode: +o jaypipes01:34
openstackgerritA change was merged to openstack/tempest: Switch run_tests.sh to run in parallel by default  https://review.openstack.org/4393501:47
*** sthaha has quit IRC02:03
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Use common create_server method for advanced_ops  https://review.openstack.org/4419702:03
*** anteaya has quit IRC02:05
openstackgerritAngus Salkeld proposed a change to openstack-dev/devstack: Move Heat to a single heat.conf  https://review.openstack.org/4419902:08
*** adalbas has joined #openstack-qa02:10
*** sthaha has joined #openstack-qa02:11
*** sthaha has joined #openstack-qa02:11
openstackgerritA change was merged to openstack-dev/devstack: xenapi: devstack support for raw tgz image upload  https://review.openstack.org/4217202:17
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Use common create_server method for advanced_ops  https://review.openstack.org/4419702:34
*** SergeyLukjanov has joined #openstack-qa02:50
*** sthaha has quit IRC02:55
openstackgerritZhi Kun Liu proposed a change to openstack/tempest: test_servers_whitebox - using official client  https://review.openstack.org/4039603:04
*** sthaha has joined #openstack-qa03:08
*** sthaha has quit IRC03:24
*** sthaha has joined #openstack-qa03:24
*** sthaha has joined #openstack-qa03:24
*** Anju has joined #openstack-qa03:31
*** SergeyLukjanov has quit IRC03:34
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Add support for Docker as Nova hypervisor  https://review.openstack.org/4075903:38
*** prekarat1 has joined #openstack-qa03:40
*** yfried has joined #openstack-qa03:56
*** HenryG has quit IRC04:07
*** sthaha has quit IRC04:50
*** sthaha has joined #openstack-qa04:50
*** sthaha has joined #openstack-qa04:50
openstackgerritA change was merged to openstack-dev/devstack: Fix is_package_installed() check with dpkg  https://review.openstack.org/4396604:51
*** prekarat has joined #openstack-qa04:59
*** prekarat1 has quit IRC05:00
*** SergeyLukjanov has joined #openstack-qa05:11
*** SergeyLukjanov has quit IRC05:13
*** SergeyLukjanov has joined #openstack-qa05:16
*** SergeyLukjanov has quit IRC05:21
*** SergeyLukjanov has joined #openstack-qa05:23
*** SergeyLukjanov has quit IRC05:32
*** SergeyLukjanov has joined #openstack-qa05:48
*** SergeyLukjanov has quit IRC05:53
*** SergeyLukjanov has joined #openstack-qa05:56
*** SergeyLukjanov has quit IRC06:02
openstackgerritIvan-Zhu proposed a change to openstack/tempest: port test_disk_config and test_server_rescue into v3 part1  https://review.openstack.org/4422006:44
openstackgerritIvan-Zhu proposed a change to openstack/tempest: port test_disk_config and test_server_rescue part2  https://review.openstack.org/4422106:44
*** thomasbiege has joined #openstack-qa07:00
*** tkammer has joined #openstack-qa07:05
openstackgerritafazekas proposed a change to openstack-infra/devstack-gate: Missing space at the end of the conditional expression  https://review.openstack.org/4422307:08
openstackgerritMarc Koderer proposed a change to openstack/tempest: Fix logging problem for stress test wrappers  https://review.openstack.org/4422407:11
openstackgerritafazekas proposed a change to openstack-infra/devstack-gate: Missing default value  https://review.openstack.org/4422607:20
*** afazekas has joined #openstack-qa07:21
afazekaspsedlak: https://bugs.launchpad.net/tempest/+bug/1217734  nova->cinder->token_validate  (memcached)07:23
*** SergeyLukjanov has joined #openstack-qa07:32
*** SergeyLukjanov has quit IRC07:33
*** SergeyLukjanov has joined #openstack-qa07:36
*** jgallard has joined #openstack-qa07:42
*** fbo_away is now known as fbo07:48
psedlakmtreinish: good finding and fix for that >1 tables in catalog ;)07:51
*** SergeyLukjanov has quit IRC07:51
*** giulivo has joined #openstack-qa07:53
*** SergeyLukjanov has joined #openstack-qa07:58
*** yassine has joined #openstack-qa07:59
*** SergeyLukjanov has quit IRC07:59
psedlakafazekas: https://review.openstack.org/#/c/44178/07:59
*** tkammer has quit IRC08:01
afazekascyeoh_: https://review.openstack.org/#/c/44178/08:06
cyeoh_afazekas: having a look now08:12
afazekascyeoh_: it is in the top 3 recheck issues08:17
cyeoh_afazekas: yea I've hit it myself a few times!08:17
cyeoh_afazekas: just making sure I understand the interaction with the keystone api tests08:18
afazekaspsedlak: ^^08:19
*** jhenner has joined #openstack-qa08:23
*** SergeyLukjanov has joined #openstack-qa08:27
cyeoh_afazekas: I've approved it. Looks like there is still a tempest bug in there though...08:29
*** prekarat has quit IRC08:48
openstackgerritLucas Alvares Gomes proposed a change to openstack-dev/devstack: Missing pxelinux.0 on RedHat systems  https://review.openstack.org/4402408:54
*** SergeyLukjanov has quit IRC08:54
afazekascyeoh_: https://review.openstack.org/#/c/44178/ we give more stress on keystone08:57
afazekasThe isolation probably makes safe almost all user only operation in parallel08:57
afazekasThe missing waits can be visible on single thread, but it they has higher chances to be visible   when the system has higher load08:58
afazekasmtresinish yesterday mentioned some nova-mange db related operation fail, in theory those should be parallel safe..08:59
Anjuafazekas : will all reviwers agrree to delete thse server_basic_ops.py09:09
Anjuafazekas : or we have to discuss in meeting09:09
afazekaspsedlak,mtreinish,cyeoh_:  IMHO it is not tempest bug : https://bugs.launchpad.net/tempest/+bug/121726509:10
afazekasAnju: you can submit the patch, and pop up it on the meeting IMHO09:11
Anjuokk afazeaks  :  as per my knowledge  this file is not creating any scenario09:12
*** prekarat has joined #openstack-qa09:18
openstackgerritA change was merged to openstack/tempest: Use common create_keypair method for autoscaling  https://review.openstack.org/4347209:21
*** tkammer has joined #openstack-qa09:37
Anjuafazekas : why lock_unlock test is faling in server_actions.py09:37
afazekasWhere is it failing ?09:42
Anjuit is not running simply like strt -stop09:42
Anjuwait09:42
*** prekarat has quit IRC09:43
Anjuafazekas : AttributeError: 'ServersClientJSON' object has no attribute 'lock'09:44
openstackgerritnayna-patel proposed a change to openstack/tempest: Adds verfication for Bootable Volume  https://review.openstack.org/4262109:55
openstackgerritIvan-Zhu proposed a change to openstack/tempest: port test_disk_config and test_server_rescue into v3 part1  https://review.openstack.org/4422009:56
openstackgerritIvan-Zhu proposed a change to openstack/tempest: port test_disk_config and test_server_rescue part2  https://review.openstack.org/4422109:56
openstackgerritIvan-Zhu proposed a change to openstack/tempest: test_server_actions into v3 part2  https://review.openstack.org/3960909:56
afazekasAnju: is it an unmodified tempest and everything is from the latest master branch ?09:58
Anjuyes09:58
afazekasSo you have a full stack trace ?09:59
Anjuu mean the full error ?10:00
afazekasyes10:00
afazekasI can't see 'lock' as an obvious attribute in the source10:00
afazekaslog.py has one10:00
*** prekarat has joined #openstack-qa10:04
openstackgerritnayna-patel proposed a change to openstack/tempest: Adds verfication for Bootable Volume  https://review.openstack.org/4262110:08
Anjuafazekas : http://paste.fedoraproject.org/35692/77711101/10:12
*** yassine has quit IRC10:14
*** yassine_ has joined #openstack-qa10:14
*** tkammer has quit IRC10:18
Anjuafazekas : ^^^10:36
openstackgerritafazekas proposed a change to openstack/tempest: KeyError when tearDownClass called from setUpClass  https://review.openstack.org/4426510:38
*** prekarat has quit IRC10:39
afazekasAnju: on the L293 I have : _interface = 'xml'10:40
afazekasclient.lock_server probbaly the correct10:42
Anjui ran it witj serner.client also10:42
Anjuand afazekas : ithe problem in not with interface10:43
Anjubecause it is failed for json also10:43
openstackgerritA change was merged to openstack/tempest: Fix test_admin_catalog_list  https://review.openstack.org/4417810:47
*** weshay has quit IRC10:53
*** weshay has joined #openstack-qa10:54
openstackgerritMarc Koderer proposed a change to openstack/tempest: Fix logging problem for stress test wrappers  https://review.openstack.org/4422410:58
openstackgerritBob Ball proposed a change to openstack-dev/devstack: Allow setting of EXTRA_OPTs in categories other than DEFAULT  https://review.openstack.org/4426611:03
afazekasself.__class__.__name__  do you know about existing helper function for the fully qualified name ?11:04
Anjuafazekas : no11:05
*** ken1ohmichi has joined #openstack-qa11:06
Anjuafazekas : i dont know much about this11:07
giulivoAnju, did you ever posted the stuff for the volumexml interface?11:07
openstackgerritMate Lakat proposed a change to openstack-dev/devstack: xenapi: enable block device access for stack user  https://review.openstack.org/4426811:07
openstackgerritnayna-patel proposed a change to openstack/tempest: Adds verfication for Bootable Volume  https://review.openstack.org/4262111:16
giulivomkoderer, now, honestly11:17
giulivoisn't this https://review.openstack.org/#/c/42964 just overcomplicating things?11:17
*** jasondotstar has quit IRC11:20
mkoderergiulivo: mhh I hate code dupliation11:22
mkoderergiulivo: do you mean my comment or the patch in general?11:22
giulivoagree but to me that looks like is just turning a string message into an unneeded passing around of variables and string concat11:23
giulivothe patch in general11:23
mkodereryes your right.. I needed some minutes to understand what happens there.. but it's just a print out....11:24
mkoderergiulivo: but I didn't know how the output was before11:25
giulivomkoderer, compare side by side11:25
giulivothe base and patched files11:25
giulivolet's try something in between and suggest he tries to sort that out at least using only _one_ external method?11:26
mkoderergiulivo: i think it wouldn't be easier to read if everything is in one method ;)11:27
giulivouhm, so we have a list of expected volumes and a list of volumes which were expected to be found there but were not11:28
mkoderergiulivo: ok and the fuction name is also misleading11:28
giulivolet me add a comment11:28
mkoderergiulivo: I have doubts that the message at the end looks really readable11:30
mkoderergiulivo: good that we discussed it... I was to fast with my "+1"11:30
giulivoindeed11:30
giulivomkoderer, I know afazekas prefers discussing the stuff on IRC too11:32
giulivowelcome ken1ohmichi11:32
openstackgerritGiulio Fidente proposed a change to openstack/tempest: moves addCleanup few lines upper to avoid potential leftovers  https://review.openstack.org/4427311:39
giulivoafazekas, ^^11:39
giulivothat is an oversight from a past change I submitted11:39
giulivoa few days back, I think can be merged easily11:40
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Remove unused LOG variable in scenario tests  https://review.openstack.org/4427411:41
*** Anju_ has joined #openstack-qa11:45
afazekasgiulivo: is this the only one from this type ?11:45
*** HenryG has joined #openstack-qa11:46
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Remove unused LOG variable in scenario tests  https://review.openstack.org/4427411:47
giulivoafazekas, I think so, I came across this one randomly11:48
*** Anju has quit IRC11:49
*** Anju_ is now known as Anju11:49
afazekasI think we have a few more ..11:49
*** sthaha has quit IRC11:56
*** sthaha has joined #openstack-qa11:57
*** sthaha has joined #openstack-qa11:57
*** weshay has quit IRC12:09
*** SergeyLukjanov has joined #openstack-qa12:15
*** SergeyLukjanov has quit IRC12:20
*** dkranz has quit IRC12:21
openstackgerritA change was merged to openstack/tempest: Add large-ops option to tox  https://review.openstack.org/4379112:29
mkodererafazekas: could you have a look https://review.openstack.org/#/c/4422412:34
*** jasondotstar has joined #openstack-qa12:52
afazekasself.__class__.__name__ question was related to this12:56
* afazekas I will be back12:56
*** afazekas has quit IRC12:57
*** weshay has joined #openstack-qa13:02
*** sthaha has quit IRC13:03
*** mriedem has joined #openstack-qa13:22
*** afazekas has joined #openstack-qa13:33
* afazekas re13:34
*** marun has joined #openstack-qa13:37
mkodererafazekas: sorry  didn't recognized that it was for me ;)13:40
mkodererso for me just the class name is enough13:41
mkodererand no - don't know if there is an built-in function13:41
afazekasfull_cname13:42
afazekasL2613:43
mkodererI know13:43
mkodererI used that to get the full class name13:44
mkodererbut I don't think it's needed for the patch13:44
afazekasAre you sure just the class name is better than the full_cname ?13:44
mkodererso the statistic at the end prints the action13:45
mkodererif it's too long it looks  a bit odd13:45
afazekas+13:46
*** jhenner has quit IRC13:48
giulivoafazekas, so what is your suggestion in relation to the addcleanup fix?13:50
openstackgerritJoe Hakim Rahme proposed a change to openstack/tempest: Adds more test to cover Swift tempURL middleware  https://review.openstack.org/4428713:50
afazekasgiulivo: as I remember near some heat test it also can be moved up13:51
afazekasapi/network/test_networks.py13:52
afazekastest_security_group_rules_create_with_invalid_ip_protocol13:53
afazekasgiulivo: IMHO you can add 3+ addCleanup move in one change13:54
*** dkranz has joined #openstack-qa13:58
mtreinishafazekas: yeah that is probably not a tempest bug14:00
*** jhenner has joined #openstack-qa14:01
rahmuafazekas: thanks for assigning someone to review the patch14:01
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Remove '--' from rand_name calls and enforce it  https://review.openstack.org/4416814:08
mtreinishjaypipes: you're the db guy, any ideas: http://logs.openstack.org/56/43056/9/check/gate-tempest-devstack-vm-full/375f6f0/logs/tempest.txt.gz#_2013-08-29_02_12_43_70914:10
jaypipesmtreinish: pretty simple.... traceback error says it all... This version of MySQL doesn't yet support 'LIMIT & IN/ALL/ANY/SOME subquery'") 'DELETE FROM security_group_instance_association WHERE security_group_instance_association.id in (SELECT security_group_instance_association.id \nFROM security_group_instance_association \nWHERE security_group_instance_association.deleted != %s ORDER BY security_group_instance_assoc14:11
jaypipesiation.id \n LIMIT %s)14:11
jaypipesmtreinish: whoever wrote a particular query in sqlalchemy used a limit expression on a subquery...14:12
jaypipesmtreinish: link to the original patch pls?14:12
mtreinishjaypipes: it's an intermitent failure we hit with the cli tests14:12
mtreinishwhen the test runs nova-manage db archive_deleted_rows 5014:13
jaypipesmtreinish: what change introduced that functionality? you know?14:13
mtreinishjaypipes: no I don't sorry, I can keep on digging though14:14
jaypipesmtreinish: k14:14
jaypipesmtreinish: bottom line is the existing code won't work with MySQL...14:15
dkranzafazekas: ping14:16
afazekasdkranz: pong14:19
dkranzafazekas: IN https://review.openstack.org/#/c/43301/2/tempest/whitebox/manager.py,unified why id tearDownClass needed?14:19
dkranzafazekas: And are you sure there is no issue with poking tokens like this with parallel execution?14:20
mtreinishjaypipes: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L5439 that's what is getting run by the cli command14:20
afazekasdkranz: the whitebox test are not part of the gate now14:20
dkranzafazekas: Yes, but you +2'ed this and I had the above questions/concerns14:21
afazekasIMHO we should have a dedicated serial testr tox entry for them, for a periodic job14:21
mtreinishafazekas: but they get run in parallel as part of a periodic job14:21
dkranzmtreinish: That was my concern with this new test14:21
dkranzmtreinish: I added to the agenda about the more general issue14:22
afazekasmtreinish: they will be fast without parallel, and I am not sure we want whitebox tests in parallel14:22
dkranzafazekas: I agree. It is too hard to review.14:23
mtreinishafazekas: ok, I was just pointing out the only time whitebox gets run is in all periodic job14:23
mtreinishwhich is run in parallel14:23
*** adalbas has quit IRC14:23
mtreinishwe can change things up if needed14:23
mtreinishor use locks to force them running serially14:23
dkranzmtreinish: It is a problem that any new test needs to know if it will introduce a flaky with regard to any existing test.14:23
dkranzmtreinish: That is a tough environment to code against.14:24
mtreinishdkranz: tenant isolation should fix that for most cases. The only real issue is for things that are above(or outside) the tenant level14:24
dkranzmtreinish: That's why I proposed "incubating" tests for a while before committing to the parallel gate. Just an idea.14:24
mtreinishwhich is where we've been hitting the bugs14:24
dkranzmtreinish: Yes. whitebox is a clear example of that.14:25
dkranzafazekas: Do you have a comment on my first question above, about tearDownCLass?14:25
mtreinishdkranz: I'm not sure that would necessarily catch things. We were running parallel for about a month and it still didn't catch all the bugs.14:26
mtreinishit's hard to find everything beforehand I agree, but flaky tests are going to happen14:26
dkranzmtreinish: True, but those were by definition rare. I am more concerned about introducing tests that fail say 25% the time.14:27
burtmtreinish: looks like https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L5420 is the culprit. git blame says it's part of the original commit.14:27
dkranzmtreinish: But we can see how it goes.14:27
afazekasdkranz: the whitebox test now in -1 state14:27
dkranzafazekas: OK, thanks.14:27
afazekastearDownClass is not necessary14:28
mtreinishdkranz: I think at this point the best thing to do is have a guide somewhere about writing tests in parallel14:28
afazekasdkranz: the other whitebox ..14:28
mtreinishand we have issues then we can come up with a more involved process for getting tests gating14:28
*** mlavalle has joined #openstack-qa14:29
dkranzmtreinish: Agreed.14:29
mtreinishburt: cool thanks14:29
dkranzmtreinish: We should probably send a warning to the list as well for those less in the thick of things who are still contributing tests.14:29
mtreinishdkranz: sure, do you want to do that?14:31
*** jhenner has quit IRC14:33
dkranzmtreinish: OK. We can touch this briefly at the meeting and then I'll send it.14:33
mtreinishdkranz: ok sounds good, thanks14:33
jaypipesmtreinish: so the code at the link you gave me above does not use a subquery in the delete statement... must be something else.14:38
jaypipesburt: nah, the limit there is on the outer query (which is fine)...14:39
*** ken1ohmichi has quit IRC14:39
burtjaypipes: oh, right..14:40
jaypipesburt: you'll note that the line numbers in the traceback don't match... :)14:40
jaypipesburt: so I gather somebody picked up on this problem and fixed it..14:41
*** adalbas has joined #openstack-qa14:44
jaypipesburt, mtreinish: the problem is in the proposed change: https://review.openstack.org/#/c/43056/10/nova/db/sqlalchemy/api.py14:46
openstackgerritJoe Hakim Rahme proposed a change to openstack/tempest: Adds more test to cover Swift tempURL middleware  https://review.openstack.org/4428714:49
mtreinishjaypipes: ok, that makes sense. I think that he was the guy put it on the recheck list :)14:49
jaypipesmtreinish: added a review comment to the patch.14:50
*** amscanne has quit IRC15:18
*** amscanne has joined #openstack-qa15:19
openstackgerritJoe Hakim Rahme proposed a change to openstack/tempest: Adds more test to cover Swift tempURL middleware  https://review.openstack.org/4428715:21
jog0anyone see this error  before http://logs.openstack.org/51/41551/6/check/gate-tempest-devstack-vm-full/c7b200b/testr_results.html.gz15:24
*** amscanne has quit IRC15:25
jog0mtreinish: ^15:29
openstackgerritDean Troyer proposed a change to openstack-dev/grenade: Add Cinder API v2 to Keystone catalog  https://review.openstack.org/4335215:30
mtreinishjog0: I think I've seen it on the recheck list15:31
mtreinishbut I haven't hit it on any of my patches15:31
openstackgerritSylvain Bauza proposed a change to openstack-dev/devstack: Fix Neutron issues related to Baremetal service  https://review.openstack.org/4430415:31
jog0mtreinish: looks like the bug in there hits another one http://logs.openstack.org/51/41551/6/check/gate-tempest-devstack-vm-full/c7b200b/console.html15:38
jog0scroll to the bottom of that15:38
jog0something about teardown15:38
marundoes anyone know if there is a way for testtools.run/subunit.run to output log output?15:39
clarkbmarun: log output generated by the tests?15:40
clarkbmarun: or logs for the test runners?15:40
marunclarkb: log output generated by code under test, so the former15:41
*** yfried has quit IRC15:41
marunat least in neutron, a logging fixture is used that captures all the output.15:41
clarkbmarun: fixtures comes with a log fixture that will capture all logging and attach it as a detail for each test in the subunit stream15:41
clarkbmarun: let me get you a link to what nova does15:42
marunclarkb: And how do I output if I'm running a single test?  It just eats it.15:42
clarkbmarun: http://git.openstack.org/cgit/openstack/nova/tree/nova/test.py#n22915:42
clarkbmarun: you look in your subunit stream15:42
marunand this is where?15:43
marunIs the key self.log_fixture - subunit knows what to do with that?15:43
marun(neutron just adds a fixture without adding an instance variable)15:43
clarkb.testrepository/$TEST_ID15:43
marun*sigh*15:44
marunreally?15:44
marunI can't just get it on stdout when I'm running a test15:44
marun?15:44
clarkbmarun: pass --subunit to testr15:44
clarkbit will stream the subunit to stdout15:44
marunWait, I'm not using testr15:44
marunI'm using 'python -m testtools.run'15:44
marunSince that's the only way to ensure I can get to a pdb prompt15:45
marunso dev use vs ci use of the tests15:45
clarkbfor subunit.run it will stream it too, but for testtools.run I am not sure let me see where the loggign fixture puts it15:45
*** yfried has joined #openstack-qa15:46
marunok, cool15:46
clarkbmarun: in that nova file you'll note that there are flags to enable stdout and stderr capture so that we don't do it by default and only when testr runs15:47
clarkbmarun: we might need to do something similar with log capturing15:47
marunclarkb: ah, fair enough.15:47
mtreinishjog0: that teardown error is thrown because teardown didn't finish. The timeout exception exited tear down class before it ran super()15:48
clarkbmarun: http://bazaar.launchpad.net/~python-fixtures/python-fixtures/trunk/view/head:/lib/fixtures/_fixtures/logger.py#L68  Ithink if we set nuke_handlers to False your logs will go where they default eg stderr15:48
mtreinishjog0: afazekas added a check to the base class to ensure that each test has the super call to make sure we run methods from the base class15:48
jog0mtreinish: ahh its really hard to find the issue in the logs without req-ids15:49
marunclarkb: ok, cool.15:49
jog0so I reran with no bug15:49
clarkbmarun: the fixtures output property should contain the logs too, you could dump that in pdb presumably15:49
clarkbmarun: but I think most straightforward thing is to make log capture conditional, don't do it by default only when testr runs15:49
mtreinishjog0: yeah I'll pick up the req-id bug today at some point15:49
marunclarkb: +1 to the second approach15:50
mtreinishjust using a print actually wasn't the best way todo it though because the fixture doesn't capture stdout from setupClass (it's run in setup)15:50
jog0mtreinish: ouch15:50
marunclarkb: I'll submit a patch for neutron.  Would you like me to include you or someone else from qa/infra to ensure the approach is valid and can then be replicated across projects?15:50
clarkbmarun: sure you can add me to the review I will take a look at it15:50
marunclarkb: ok, will do.  Thanks for your help!  :)15:51
*** weshay has quit IRC16:04
rahmuhello, can anyone help me understand the error I got on my tempest patch here? https://review.openstack.org/#/c/44287/16:09
rahmuI get "Duplicate: An object with that identifier already exists" on a test I haven't touched, only on the postgresql tests16:09
*** Anju has quit IRC16:12
*** Anju has joined #openstack-qa16:13
*** yfried has quit IRC16:18
*** weshay has joined #openstack-qa16:19
*** jgallard has quit IRC16:24
*** SergeyLukjanov has joined #openstack-qa16:26
openstackgerritA change was merged to openstack-dev/devstack: Adds support for Openstack Networking FWaaS (Firewall)  https://review.openstack.org/3714716:27
*** yfried has joined #openstack-qa16:31
*** yfried has quit IRC16:38
mtreinishmeeting time17:00
afazekashi17:00
*** tkammer has joined #openstack-qa17:01
*** fbo is now known as fbo_away17:01
*** weshay has quit IRC17:02
*** tkammer_ has joined #openstack-qa17:05
*** tkammer has quit IRC17:06
openstackgerritA change was merged to openstack-dev/devstack: Add support for heat resource templates  https://review.openstack.org/4363117:12
openstackgerritA change was merged to openstack-dev/grenade: Add Cinder API v2 to Keystone catalog  https://review.openstack.org/4335217:12
*** yassine_ has quit IRC17:14
*** weshay has joined #openstack-qa17:15
*** HenryG has quit IRC17:27
*** HenryG has joined #openstack-qa17:27
*** amscanne has joined #openstack-qa17:33
*** jasondotstar has quit IRC17:33
*** jasondotstar has joined #openstack-qa17:35
*** hemnafk is now known as hemna17:39
openstackgerritAnju Tiwari proposed a change to openstack/tempest: hh  https://review.openstack.org/4432717:44
*** yfried has joined #openstack-qa17:47
*** SergeyLukjanov has quit IRC17:52
*** SergeyLukjanov has joined #openstack-qa17:53
*** jasondotstar has quit IRC17:55
*** woodspa has joined #openstack-qa17:58
*** jasondotstar has joined #openstack-qa17:59
* afazekas ON vacation, returning sept 918:01
giulivoafazekas, today or tomorrow?18:01
mkodererafazekas: have a good time18:02
afazekasthx18:02
mtreinishafazekas: enjoy18:02
afazekasthx18:02
afazekasI need a lot of regeneration --> long vacation ..18:03
*** sfineberg has joined #openstack-qa18:06
*** tkammer_ has quit IRC18:06
openstackgerritAnju Tiwari proposed a change to openstack/tempest: Added missing xml tests of volume  https://review.openstack.org/4433018:08
*** thomasbiege has joined #openstack-qa18:09
*** sfineberg has quit IRC18:13
openstackgerritAnju Tiwari proposed a change to openstack/tempest: Added missing xml tests of volume  https://review.openstack.org/4433018:15
*** sfineberg has joined #openstack-qa18:18
*** sfineberg has quit IRC18:20
openstackgerritJoe Gordon proposed a change to openstack-infra/devstack-gate: Add support for large_ops tempest test  https://review.openstack.org/4377918:21
*** sfineberg has joined #openstack-qa18:21
*** thomasbiege has quit IRC18:38
openstackgerritJoe Gordon proposed a change to openstack-infra/devstack-gate: Add support for large_ops tempest test  https://review.openstack.org/4377918:40
*** jasondotstar has quit IRC19:03
openstackgerritJoe Gordon proposed a change to openstack-infra/devstack-gate: Add support for large_ops tempest test  https://review.openstack.org/4377919:03
*** jasondotstar has joined #openstack-qa19:06
*** ianw has quit IRC19:08
*** SergeyLukjanov has quit IRC19:32
openstackgerritJoe Gordon proposed a change to openstack-infra/devstack-gate: Add support for large_ops tempest test  https://review.openstack.org/4377919:36
*** SergeyLukjanov has joined #openstack-qa19:41
openstackgerritA change was merged to openstack/tempest: Add common "create_server_snapshot" method  https://review.openstack.org/4365819:45
openstackgerritA change was merged to openstack/tempest: Fix ssh timeout issue  https://review.openstack.org/4341519:45
openstackgerritJoe Gordon proposed a change to openstack-dev/devstack: Make nova use fatal_deprecations=true  https://review.openstack.org/4434919:47
openstackgerritDavanum Srinivas (dims) proposed a change to openstack-dev/devstack: Skip DHCP variables for Nova Network FlatManager  https://review.openstack.org/4346819:53
*** SergeyLukjanov has quit IRC19:59
*** fbo_away is now known as fbo20:00
*** HenryG has quit IRC20:02
*** yfried has quit IRC20:07
*** ianw has joined #openstack-qa20:08
openstackgerritA change was merged to openstack/tempest: Fix logging problem for stress test wrappers  https://review.openstack.org/4422420:16
*** afazekas has quit IRC20:16
*** yfried has joined #openstack-qa20:23
openstackgerritJoe Gordon proposed a change to openstack-infra/devstack-gate: Add support for large_ops tempest test  https://review.openstack.org/4377921:02
*** yfried has quit IRC21:08
*** thomasbiege has joined #openstack-qa21:14
*** thomasbiege has quit IRC21:16
*** jasondotstar has quit IRC21:20
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Log request ids from glance and nova  https://review.openstack.org/4193421:28
mtreinishjog0: ^^^ that should help with the parallel logs hopefully21:28
mtreinishI haven't tested it yet pip is fubar on my devstack box21:29
*** thomasbiege has joined #openstack-qa21:33
*** dkranz has quit IRC21:35
*** thomasbiege has quit IRC21:48
*** mriedem has quit IRC22:07
*** burt has quit IRC22:11
*** weshay has quit IRC22:14
*** adalbas has quit IRC22:24
*** fbo is now known as fbo_away22:25
*** adalbas has joined #openstack-qa22:36
*** mlavalle has quit IRC23:01
*** hemna is now known as hemnafk23:12
openstackgerritRyan Hsu proposed a change to openstack/tempest: Adds disk_format parameter to upload_volume method in volumes client  https://review.openstack.org/4437423:41
« Wednesday, 2013-08-28 Index Friday, 2013-08-30 »

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