Friday, 2013-10-25

*** SergeyLukjanov has joined #openstack-qa00:02
openstackrecheckopenstack/horizon change: https://review.openstack.org/53301 failed tempest with an unrecognized error00:10
openstackgerritSam Alba proposed a change to openstack-dev/devstack: Removed dependency from dotcloud repos for the Nova docker driver  https://review.openstack.org/5376300:11
*** ken1ohmichi has joined #openstack-qa00:13
openstackrecheckopenstack/cinder change: https://review.openstack.org/53760 failed tempest with an unrecognized error00:17
*** markmcclain has quit IRC00:19
*** yfried_mob has quit IRC00:27
*** yfried_mob has joined #openstack-qa00:28
*** markmcclain has joined #openstack-qa00:45
*** nosnos has joined #openstack-qa00:49
*** markmcclain has quit IRC00:53
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/53763 failed tempest because of: https://bugs.launchpad.net/bugs/124425500:56
*** SergeyLukjanov_ has joined #openstack-qa00:56
*** SergeyLukjanov has quit IRC00:57
*** mriedem has joined #openstack-qa01:07
*** weshay has quit IRC01:12
*** SergeyLukjanov has joined #openstack-qa01:15
*** SergeyLukjanov_ has quit IRC01:15
ken1ohmichihi01:17
ken1ohmichiAre there any features for creating a Tempest result file?01:17
clarkbyes testr outputs a subunit log file in .testrepository01:20
ken1ohmichiI want to get the result file with CSV format, and convert it to MS excel file for reporting it.01:20
clarkbyou'll need to parse it and spit out csv, there are already tools to convert to xunit and html and so on01:21
ken1ohmichiclarkb: thanks so much! I will check it :-)01:22
*** markmcclain has joined #openstack-qa01:22
*** yfried_mob has quit IRC01:25
*** yfried_mob has joined #openstack-qa01:25
*** ken1ohmichi has quit IRC01:26
*** mriedem has quit IRC01:32
openstackrecheckopenstack/nova change: https://review.openstack.org/52028 failed tempest with an unrecognized error01:42
*** ivanzhu has joined #openstack-qa01:45
*** wanghao has joined #openstack-qa01:46
openstackgerritAaron Rosen proposed a change to openstack-dev/devstack: OVS still working?  https://review.openstack.org/5075601:49
*** markmcclain has quit IRC01:53
*** dims has quit IRC02:03
openstackgerritjun xie proposed a change to openstack/tempest: Some tests for dhcp agent scheduler  https://review.openstack.org/5085602:04
wanghaogiulivo: hi~ Cloud you help me approve the test :https://review.openstack.org/#/c/50217/? Thanks.02:05
Lingxiandkranz, ping02:13
openstackgerritwingwj proposed a change to openstack/tempest: Add some test_cases for glance_cli  https://review.openstack.org/5117002:16
*** dims has joined #openstack-qa02:20
openstackgerritIvan-Zhu proposed a change to openstack/tempest: port test_images and test_server_actions into v3 part1  https://review.openstack.org/3962102:22
openstackgerritIvan-Zhu proposed a change to openstack/tempest: port test_images and test_server_actions into v3 part2  https://review.openstack.org/3960902:22
*** yfried_mob has quit IRC02:23
*** anteaya has quit IRC02:23
*** yfried_mob has joined #openstack-qa02:23
*** dims has quit IRC02:27
*** dims has joined #openstack-qa02:30
openstackrecheckopenstack/neutron change: https://review.openstack.org/53767 failed tempest because of: https://bugs.launchpad.net/bugs/123548602:41
*** Dharmit has joined #openstack-qa02:42
*** andreaf has joined #openstack-qa02:55
*** andreaf has quit IRC03:05
openstackgerritwingwj proposed a change to openstack/tempest: Add test cases for volume-transfer  https://review.openstack.org/5307003:05
openstackrecheckopenstack/heat change: https://review.openstack.org/53774 failed tempest with an unrecognized error03:05
*** ivanzhu has quit IRC03:17
*** mlavalle has joined #openstack-qa03:19
*** mlavalle has quit IRC03:19
*** yfried_mob has quit IRC03:20
*** mlavalle has joined #openstack-qa03:20
*** yfried_mob has joined #openstack-qa03:21
*** andreaf has joined #openstack-qa03:21
*** ivanzhu has joined #openstack-qa03:22
openstackrecheckopenstack/heat change: https://review.openstack.org/53769 failed tempest because of: https://bugs.launchpad.net/bugs/123985603:24
openstackrecheckopenstack/heat change: https://review.openstack.org/53770 failed tempest because of: https://bugs.launchpad.net/bugs/124425503:24
*** dims has quit IRC03:31
*** wanghao has quit IRC03:34
*** wanghao has joined #openstack-qa03:37
*** prekarat has joined #openstack-qa03:41
openstackrecheckopenstack/tempest change: https://review.openstack.org/53070 failed tempest because of: https://bugs.launchpad.net/bugs/123548603:45
openstackgerritDaisuke Morita proposed a change to openstack/tempest: replaces assertLessEqual with assertThat and testtools matcher  https://review.openstack.org/5377804:08
*** yfried_mob has quit IRC04:18
*** yfried_mob has joined #openstack-qa04:18
openstackrecheckopenstack/tempest change: https://review.openstack.org/53778 failed tempest because of: https://bugs.launchpad.net/bugs/123548604:41
*** wanghao has quit IRC04:51
openstackrecheckopenstack/neutron change: https://review.openstack.org/50519 failed tempest because of: https://bugs.launchpad.net/bugs/123963704:59
*** Madan has joined #openstack-qa05:01
*** yfried_mob has quit IRC05:01
*** jhenner has joined #openstack-qa05:06
*** thomasbiege has joined #openstack-qa05:14
*** yfried_mob has joined #openstack-qa05:14
*** yfried_mob has joined #openstack-qa05:16
*** shilpa_ has joined #openstack-qa05:33
*** ivanzhu1 has joined #openstack-qa05:37
*** ivanzhu has quit IRC05:39
*** ken1ohmichi has joined #openstack-qa05:39
*** yfried_mob has quit IRC05:40
openstackgerritjun xie proposed a change to openstack/tempest: Test for service type management  https://review.openstack.org/5199705:41
*** ken1ohmichi has quit IRC05:50
openstackgerritAaron Rosen proposed a change to openstack-dev/devstack: All neutron plugins should now use LibvirtGenericVIFDriver  https://review.openstack.org/5374005:58
openstackgerritAaron Rosen proposed a change to openstack-dev/devstack: All neutron plugins should now use LibvirtGenericVIFDriver  https://review.openstack.org/5374006:01
*** yfried_mob has joined #openstack-qa06:02
*** yfried_mob has quit IRC06:06
*** yfried_mob has joined #openstack-qa06:06
*** matsuhashi has joined #openstack-qa06:07
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds api test to test_images  https://review.openstack.org/5333706:09
Madandkranaz: ping06:10
*** yfried_mob has quit IRC06:11
*** yfried_mob has joined #openstack-qa06:11
*** thomasbiege has quit IRC06:14
*** yfried|mob has joined #openstack-qa06:15
*** yfried|mob has joined #openstack-qa06:17
*** yfried_mob has quit IRC06:18
Madanmkorderer: ping06:18
*** yfried|mob has quit IRC06:22
*** yfried|mob has joined #openstack-qa06:26
*** yfried|mob has quit IRC06:31
*** Madan has quit IRC06:40
*** ivanzhu has joined #openstack-qa06:44
*** ivanzhu1 has quit IRC06:44
openstackgerrithuangtianhua proposed a change to openstack/tempest: add migrate negative tests  https://review.openstack.org/5108906:45
*** thomasbiege has joined #openstack-qa06:45
openstackrecheckopenstack/heat change: https://review.openstack.org/53780 failed tempest with an unrecognized error06:46
Anjumkoderer:  hii, please review this. https://review.openstack.org/#/c/50592/06:47
*** ivanzhu has quit IRC06:49
*** jamielennox is now known as jamielennox|away06:51
*** thomasbiege1 has joined #openstack-qa06:51
*** yfried_mob has joined #openstack-qa06:53
*** yfried|mob has joined #openstack-qa06:54
*** Madan has joined #openstack-qa06:54
*** thomasbiege has quit IRC06:55
*** yfried_mob has quit IRC06:57
*** wanghao has joined #openstack-qa07:01
*** ivanzhu has joined #openstack-qa07:03
*** YorikSar_AFK has quit IRC07:03
*** yfried|mob has quit IRC07:09
*** jhenner has quit IRC07:14
*** matsuhashi has quit IRC07:18
*** matsuhashi has joined #openstack-qa07:18
openstackrecheckopenstack/tempest change: https://review.openstack.org/53778 failed tempest because of: https://bugs.launchpad.net/bugs/123548607:20
*** SergeyLukjanov has quit IRC07:24
*** yfried|mob has joined #openstack-qa07:24
openstackrecheckopenstack/cinder change: https://review.openstack.org/53783 failed tempest because of: https://bugs.launchpad.net/bugs/122566407:29
*** jlibosva has joined #openstack-qa07:37
openstackgerritAbhijeet Malawade proposed a change to openstack/tempest: Add testcases for security groups and images  https://review.openstack.org/4707907:48
*** safchain has joined #openstack-qa07:48
*** afazekas has joined #openstack-qa07:49
mkodererdosaboy: ping07:50
*** gszasz has joined #openstack-qa07:50
*** yassine has joined #openstack-qa07:57
*** yfried|mob has quit IRC08:02
*** yfried|mob has joined #openstack-qa08:02
Madanmkoderer: ping08:05
*** fbo_away is now known as fbo08:11
openstackgerritafazekas proposed a change to openstack/tempest: Preserve the configured log level  https://review.openstack.org/5365308:20
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/52539 failed tempest with an unrecognized error08:20
*** yfried|mob has quit IRC08:24
*** yfried|mob has joined #openstack-qa08:24
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds api test to test_images  https://review.openstack.org/5333708:34
openstackgerritA change was merged to openstack/tempest: Fix bad classname  https://review.openstack.org/5374508:37
*** wanghao has quit IRC08:42
*** matsuhashi has quit IRC08:46
*** matsuhashi has joined #openstack-qa08:46
mkodererMadan: pong08:46
Madanmkorderer: if u could review my patch https://review.openstack.org/#/c/53337/ would be a great help !!08:49
*** thomasbiege1 has quit IRC08:53
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/41869 failed tempest with an unrecognized error08:53
*** ivanzhu1 has joined #openstack-qa08:54
*** ivanzhu has quit IRC08:56
*** yfried|mob has joined #openstack-qa09:00
openstackrecheckopenstack/neutron change: https://review.openstack.org/53805 failed tempest because of: https://bugs.launchpad.net/bugs/124425509:02
openstackrecheckopenstack/tempest change: https://review.openstack.org/53653 failed tempest with an unrecognized error09:07
*** wanghao has joined #openstack-qa09:08
mkodererMadan: ok the patch lgtm, but you could enhance your commit message a bit09:08
mkoderersince you actually not adding tests, you are modify one to have more coverage09:08
mkodererMadan: and I need to wait for the jenkins run before voting09:09
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/35454 failed tempest with an unrecognized error09:14
openstackgerritwingwj proposed a change to openstack/tempest: Add test_cases for cinder cli in v2 version  https://review.openstack.org/5084809:17
*** yfried has quit IRC09:17
*** yfried_mob has joined #openstack-qa09:18
*** yfried|mob has quit IRC09:19
openstackrecheckopenstack/neutron change: https://review.openstack.org/53364 failed tempest because of: https://bugs.launchpad.net/bugs/123963709:26
*** ivanzhu1 has quit IRC09:32
*** yfried has joined #openstack-qa09:32
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/51845 failed tempest because of: https://bugs.launchpad.net/bugs/123985609:33
*** matsuhashi has quit IRC09:34
*** matsuhashi has joined #openstack-qa09:34
*** matsuhashi has quit IRC09:39
*** plomakin_ has quit IRC09:49
*** plomakin has joined #openstack-qa09:50
openstackgerrithuangtianhua proposed a change to openstack/tempest: add migrate negative tests  https://review.openstack.org/5108909:52
*** YorikSar has joined #openstack-qa09:53
*** sthaha has quit IRC09:53
*** sthaha has joined #openstack-qa09:56
openstackrecheckopenstack/python-novaclient change: https://review.openstack.org/50438 failed tempest because of: https://bugs.launchpad.net/bugs/124025609:59
Madanmkorderer: Thank you for reviewing,, what would u suggest me on commit message ??09:59
openstackrecheckopenstack/neutron change: https://review.openstack.org/53636 failed tempest because of: https://bugs.launchpad.net/bugs/122400110:04
*** prekarat has quit IRC10:06
*** thomasbiege1 has joined #openstack-qa10:08
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/44809 failed tempest with an unrecognized error10:10
openstackrecheckopenstack/neutron change: https://review.openstack.org/53364 failed tempest with an unrecognized error10:12
*** wanghao has quit IRC10:12
openstackgerritafazekas proposed a change to openstack/tempest: Basic connectivity between two node  https://review.openstack.org/5299410:13
openstackgerritA change was merged to openstack-dev/devstack: Enable keystone authentication in Ironic  https://review.openstack.org/5184510:15
*** ekarlso has quit IRC10:17
*** ekarlso has joined #openstack-qa10:17
*** matsuhashi has joined #openstack-qa10:27
*** nosnos has quit IRC10:31
*** mlavalle has left #openstack-qa10:40
*** matsuhashi has quit IRC10:46
*** matsuhashi has joined #openstack-qa10:47
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/44809 failed tempest with an unrecognized error10:49
*** matsuhashi has quit IRC10:51
openstackrecheckopenstack/horizon change: https://review.openstack.org/53301 failed tempest because of: https://bugs.launchpad.net/bugs/122502410:57
*** thomasbiege1 has quit IRC11:03
mkodererMadan: just have a look to https://wiki.openstack.org/wiki/GitCommitMessages11:06
*** sthaha has quit IRC11:07
*** safchain has quit IRC11:09
*** dims has joined #openstack-qa11:12
*** mriedem has joined #openstack-qa11:16
*** matsuhashi has joined #openstack-qa11:20
openstackrecheckopenstack/nova change: https://review.openstack.org/53789 failed tempest with an unrecognized error11:23
*** matsuhashi has quit IRC11:23
*** matsuhashi has joined #openstack-qa11:24
*** matsuhashi has quit IRC11:24
*** matsuhashi has joined #openstack-qa11:25
*** matsuhashi has quit IRC11:29
openstackgerritMarc Koderer proposed a change to openstack/tempest: Change negative smoke test to gate tests  https://review.openstack.org/5382511:31
*** adalbas has joined #openstack-qa11:33
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/52539 failed tempest because of: https://bugs.launchpad.net/bugs/123985611:33
*** safchain has joined #openstack-qa11:40
*** safchain has quit IRC11:41
openstackrecheckopenstack/neutron change: https://review.openstack.org/53364 failed tempest with an unrecognized error11:44
openstackrecheckopenstack/neutron change: https://review.openstack.org/53122 failed tempest with an unrecognized error11:47
*** safchain has joined #openstack-qa11:52
*** Madan has quit IRC11:54
*** thomasbiege1 has joined #openstack-qa11:54
openstackrecheckopenstack/tempest change: https://review.openstack.org/53825 failed tempest with an unrecognized error12:01
*** weshay has joined #openstack-qa12:03
*** annegentle has quit IRC12:04
*** annegentle has joined #openstack-qa12:07
*** annegentle_ has joined #openstack-qa12:11
*** shilpa_ has quit IRC12:12
*** annegentle has quit IRC12:12
*** mkollaro has joined #openstack-qa12:26
openstackgerritafazekas proposed a change to openstack/tempest: assertTrue to assertIn  https://review.openstack.org/5383512:29
openstackrecheckopenstack/nova change: https://review.openstack.org/49455 failed tempest because of: https://bugs.launchpad.net/bugs/122502412:31
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/50613 failed tempest with an unrecognized error12:33
*** yfried has quit IRC12:35
*** dims has quit IRC12:38
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/47145 failed tempest because of: https://bugs.launchpad.net/bugs/123985612:39
*** dims has joined #openstack-qa12:40
*** kong has joined #openstack-qa12:43
*** Lingxian has quit IRC12:45
*** kong has quit IRC12:51
*** jlibosva has quit IRC12:55
openstackrecheckopenstack/tempest change: https://review.openstack.org/53835 failed tempest with an unrecognized error12:58
*** anteaya has joined #openstack-qa13:08
*** YorikSar has quit IRC13:08
openstackgerritThomas Maddox proposed a change to openstack-dev/devstack: Add PostgreSQL option for Ceilometer backend  https://review.openstack.org/5371513:09
*** russellb is now known as rustlebee13:12
*** Dharmit has quit IRC13:18
*** dansmith is now known as Steely_Dan13:26
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/53133 failed tempest because of: https://bugs.launchpad.net/bugs/121773413:31
openstackrecheckopenstack/keystone change: https://review.openstack.org/51370 failed tempest because of: https://bugs.launchpad.net/bugs/122502413:33
*** shilpa_ has joined #openstack-qa13:46
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/50613 failed tempest because of: https://bugs.launchpad.net/bugs/122502413:59
*** thomasbiege1 has quit IRC13:59
openstackgerritA change was merged to openstack/tempest: add some tests for host and seperate negative ones  https://review.openstack.org/4977814:01
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Fix incorrect config option types  https://review.openstack.org/5386914:10
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add sample config check to tox pep8 job  https://review.openstack.org/5387014:10
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Sync config file generator from oslo  https://review.openstack.org/5372414:10
*** bnemec is now known as beekneemech14:16
mkoderermtreinish: do we have a problem with neutron-pg-isolated?14:17
mkoderermtreinish: second time that jekins fails in teardown14:18
mkodererhttps://review.openstack.org/#/c/53825/14:18
mtreinishmkoderer: probably, neutron has issues with tenant isolation enabled14:18
mtreinishwell the failure there is a swift bug right14:19
mtreinishyou probably could have hit that on a non neutron run14:20
*** dims is now known as blur14:21
mkoderermtreinish: mh next run failed in  tearDownClass (tempest.api.volume.test_volumes_actions.VolumesActionsTest)14:21
mtreinishmkoderer: do you have a link?14:23
mkoderermtreinish: https://jenkins01.openstack.org/job/check-tempest-devstack-vm-neutron-pg-isolated/489/console14:23
mtreinishmkoderer: when the job finishes I bet e-r will tag a bug for it14:24
mkoderermtreinish: all right we will see14:25
mtreinishmkoderer: I just checked the logs it'll be bug 123548614:27
openstackrecheckopenstack/tempest change: https://review.openstack.org/53825 failed tempest because of: https://bugs.launchpad.net/bugs/123548614:36
*** yfried_mob has joined #openstack-qa14:38
openstackgerritMauro S. M. Rodrigues proposed a change to openstack-dev/grenade: Cleanup for test upgrade from Havana to Icehouse  https://review.openstack.org/5387414:39
*** shilpa_ has quit IRC14:41
*** cyeoh_ is now known as cyeoh14:44
*** prekarat has joined #openstack-qa14:48
*** yfried|mob has joined #openstack-qa14:49
*** yfried|mob has quit IRC14:49
*** yfried|mob has joined #openstack-qa14:49
*** Lingxian has joined #openstack-qa14:51
*** SergeyLukjanov has joined #openstack-qa14:56
*** kong has joined #openstack-qa14:58
*** kong has quit IRC15:00
*** kong has joined #openstack-qa15:00
*** Lingxian has quit IRC15:01
*** Lingxian has joined #openstack-qa15:06
openstackrecheckopenstack/neutron change: https://review.openstack.org/44942 failed tempest because of: https://bugs.launchpad.net/bugs/124425515:06
*** kong has quit IRC15:06
mkollarohow was the syntax for rechecks? #recheck bug XX?15:10
openstackrecheckopenstack/requirements change: https://review.openstack.org/52137 failed tempest because of: https://bugs.launchpad.net/bugs/1217734 and https://bugs.launchpad.net/bugs/122566415:12
sdaguerecheck bug XXXXXX15:14
mkollarosdague: thanks15:15
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Fix default values so they work in a devstack run  https://review.openstack.org/5388115:16
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add sample config check to tox pep8 job  https://review.openstack.org/5387015:16
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Sync config file generator from oslo  https://review.openstack.org/5372415:16
*** blur is now known as dims15:18
dosaboymkoderer: pong15:21
openstackrecheckopenstack/neutron change: https://review.openstack.org/51778 failed tempest because of: https://bugs.launchpad.net/bugs/123548615:24
openstackgerritJoe Hakim Rahme proposed a change to openstack-dev/devstack: Enables Swift crossdomain middleware by default  https://review.openstack.org/5311615:27
openstackrecheckopenstack/neutron change: https://review.openstack.org/52697 failed tempest because of: https://bugs.launchpad.net/bugs/123548615:28
openstackrecheckopenstack/nova change: https://review.openstack.org/51751 failed tempest because of: https://bugs.launchpad.net/bugs/123985615:35
*** jaypipes has joined #openstack-qa15:35
*** ChanServ sets mode: +o jaypipes15:35
openstackgerritA change was merged to openstack-dev/devstack: add lib/stackforge to let us test wsme / pecan from git  https://review.openstack.org/5302615:41
openstackrecheckopenstack/tempest change: https://review.openstack.org/53881 failed tempest with an unrecognized error15:47
*** adalbas has quit IRC15:48
openstackrecheckopenstack/tempest change: https://review.openstack.org/49488 failed tempest with an unrecognized error15:50
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/49456 failed tempest because of: https://bugs.launchpad.net/bugs/121773416:02
openstackgerritPeter Feiner proposed a change to openstack-dev/devstack: zsh openrc compatibility  https://review.openstack.org/5375516:02
*** safchain has quit IRC16:03
openstackgerritRoman Prykhodchenko proposed a change to openstack-infra/devstack-gate: Support of Ironic  https://review.openstack.org/5389916:03
*** Lingxian has quit IRC16:07
*** Lingxian has joined #openstack-qa16:08
*** YorikSar has joined #openstack-qa16:20
*** SergeyLukjanov has quit IRC16:25
*** prekarat has quit IRC16:25
openstackgerritRoman Prykhodchenko proposed a change to openstack-infra/devstack-gate: Support of Ironic  https://review.openstack.org/5389916:26
openstackrecheckopenstack/requirements change: https://review.openstack.org/52137 failed tempest because of: https://bugs.launchpad.net/bugs/122566416:29
mkollarocould somebody please review this short change? https://review.openstack.org/#/c/53778/16:32
mkollaroit's the last think keeping me from getting tempest to run 100% :]16:33
*** markmcclain has joined #openstack-qa16:37
*** SergeyLukjanov has joined #openstack-qa16:52
*** torandu has quit IRC16:55
dkranzmkollaro: This is new to me. Can you say what is wrong with the original?16:57
mkollarodkranz: https://bugs.launchpad.net/tempest/+bug/123351316:57
*** thomasbiege1 has joined #openstack-qa16:58
dkranzmkollaro:  I see. Can you take over this patch and fix the commit message to refer to this bug please?16:59
mkollarodkranz: ok17:00
dkranzmkoderer: Thanks!17:00
*** yassine has quit IRC17:01
dkranzmkollaro: The commit message should also say it is a python 2.6 issue17:01
mkollarodkranz: do you maybe think that using assertTrue(x <= y) would be better in this case? I find the current way a bit too complicated17:03
dkranzmkollaro: I know we have shied away from AssertTrue because it gives not much info when it fails.17:04
openstackrecheckopenstack/neutron change: https://review.openstack.org/53660 failed tempest because of: https://bugs.launchpad.net/bugs/1239637 and https://bugs.launchpad.net/bugs/124425517:05
mkollarodkranz: ok, I'll keep it the way it is17:05
dkranzmkollaro: Unless the proposal is new to tempest I would keep it foro now.17:06
dkranzmkoderer: Probably should not make editorial changes when taking over some one else's patch :)17:06
mkollaro:)17:06
dkranzjd__: Do you still mean to have -1 on https://review.openstack.org/#/c/53396/ ?17:07
jd__dkranz: yes unless I missed something?17:08
dkranzjd__: There was a response to your comment.17:08
openstackgerritMartina Kollarova proposed a change to openstack/tempest: replace assertLessEqual - is not in py26 testtools  https://review.openstack.org/5377817:08
dkranzjd__: I was going to add this error to the whitelist but if it is fixed I won't bother17:09
mkollarodkranz: https://review.openstack.org/#/c/53778/17:09
mkollarodkranz: I was hoping it would keep the +1 if it recognized a trivial rebase :(17:09
YorikSarbeekneemech: Hi17:10
YorikSarbeekneemech: I found you :)17:10
YorikSarbeekneemech: I'm sorry for the mess with patchsets.17:11
beekneemechYorikSar: No problem. :-)17:12
beekneemechYorikSar: I really like the new test case.17:12
YorikSarbeekneemech: I'm not sure how update.py replaces strings. Will it catch OSLO_ in main() too?17:13
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Sync config file generator from oslo  https://review.openstack.org/5372417:13
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add sample config check to tox pep8 job  https://review.openstack.org/5387017:13
YorikSarbeekneemech: I found os.fork and a lot of try..finaly blocks. I think, this test module should be revised to make use of TempDir fixture and multiprocessing instead.17:14
beekneemechYorikSar: I think it should.  I need to double-check though - my test update seems to be missing main() entirely...17:15
YorikSarbeekneemech: It's in the next change request: https://review.openstack.org/5361217:15
YorikSarbeekneemech: I've added a test there too, btw17:16
beekneemechYorikSar: Cool, will take a look.17:16
beekneemechOh, that's right.  main was added in a separate patch set.  No wonder it's missing. :-)17:17
*** torandu has joined #openstack-qa17:19
mtreinishsdague: https://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/network/base.py#n70 sigh...17:19
sdaguewhy is line 70 a sigh?17:21
mtreinishthe whole teardown function there17:22
mtreinishand the exception handling17:22
*** mkollaro has quit IRC17:22
YorikSarbeekneemech: Oh my... update.py makes me sad.17:24
beekneemechYorikSar: How so?  I can't say I've ever looked into it much.17:26
openstackgerritSean Dague proposed a change to openstack-dev/devstack: put the stackforge library installation behind a conditional  https://review.openstack.org/5391517:27
*** markmcclain has quit IRC17:28
YorikSarbeekneemech: Every single file is being rewritten at least 5 times. But given the number of modules it makes up to 40.17:28
*** markmcclain has joined #openstack-qa17:28
beekneemechYorikSar: Ah, yeah, that sounds kind of nasty, but at least it doesn't have to get run very often. :-)17:29
YorikSarbeekneemech: Yeah... But still. Every time I see this code my WAT/sec counter goes throw the roof.17:30
beekneemechlol17:30
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Raise specific exceptions on tearDownClass failure  https://review.openstack.org/5391817:32
*** adalbas has joined #openstack-qa17:38
openstackrecheckopenstack/swift change: https://review.openstack.org/44787 failed tempest because of: https://bugs.launchpad.net/bugs/121773417:38
openstackrecheckopenstack/tempest change: https://review.openstack.org/53724 failed tempest with an unrecognized error17:41
openstackrecheckopenstack/heat change: https://review.openstack.org/53780 failed tempest because of: https://bugs.launchpad.net/bugs/123985617:43
YorikSarbeekneemech: I've stumbled upon your change in hacking regarding import grouping and order.17:45
YorikSarbeekneemech: Did you consider class-style extension for this? It looks too hacky currently.17:46
beekneemechYorikSar: I mostly started with the alphabetical order check and extended it.  This is the first time I've ever worked on hacking, but I got sick of -1'ing patches for import grouping problems. :-)17:48
beekneemechSo I guess the answer is no, I didn't even know that was an option.17:48
YorikSarbeekneemech: I totally understand you. I've fought with one guy for almost a week and after each change he kept comming back with messed up imports.17:49
YorikSarbeekneemech: I actually never wrote class extension for flake8, but as it looks in docs, it's way easier.17:50
YorikSarhttp://flake8.readthedocs.org/en/2.0/extensions.html17:50
YorikSarbeekneemech: You can take a look at all Git checks. They are written as classes.17:50
beekneemechYorikSar: Thanks, I'll check it out.17:51
YorikSarbeekneemech: The point is that class is instanciated for every file. So you don't need to guess if you got into a new file.17:51
beekneemechYorikSar: That would be nice. :-)17:51
YorikSarbeekneemech: And it looks like you can just analyze AST as a whole. So no need for external state machine.17:52
YorikSarbeekneemech: I remember how hard it was to write checks for PEP8... I think if we can convert old checks to the new style, it would make everything cleaner. And probably faster.17:57
openstackrecheckopenstack/python-keystoneclient change: https://review.openstack.org/50996 failed tempest with an unrecognized error17:58
openstackgerritafazekas proposed a change to openstack/tempest: assertTrue to assertIn  https://review.openstack.org/5383518:06
dkranzmtreinish: Do you have an opinion about https://review.openstack.org/#/c/53778/2 ?18:10
dkranzmtreinish: We could use assertTrue rather than introducing this matcher stuff18:11
dkranzmtreinish: I don't think it's wrong but seems annoying18:11
openstackrecheckopenstack/swift change: https://review.openstack.org/44787 failed tempest because of: https://bugs.launchpad.net/bugs/124425518:11
afazekasdkranz: FYI assertGreater also has issue,  IMHO we can implement the py2.7  assertFoo functions (GE,G,LE,L ) in the tempest/test.py18:21
dkranzafazekas: That might be better. But py2.6 will be gone in <6mo so perhaps it is not worth bothering since according to mkolloro this is the last bug18:22
dkranzafazekas: That is, the last change needed for this issue.18:23
dkranzafazekas: But if you want to go for it...18:24
openstackrecheckopenstack/tempest change: https://review.openstack.org/53653 failed tempest with an unrecognized error18:34
openstackrecheckopenstack/tempest change: https://review.openstack.org/53835 failed tempest because of: https://bugs.launchpad.net/bugs/123963718:36
mtreinishdkranz: wow yeah lets not do that. There are easier ways assertTrue being the obvious one18:40
mtreinishdkranz: although I'm not sure we want py2.6 anymore. I was going to propose in I we rip it out so we can get rid of these crazy hacks.18:41
mtreinishand finally drop nose from the requirments list18:41
dkranzmtreinish: I would like to do that but I believe it is still needed for centos, rhel, etc.18:42
dkranzmtreinish: I don't think we should drop 2.6 until the other projects do.18:43
dkranzmtreinish: We are one of the few that don't actually gate on 2.6 I think18:43
mtreinishdkranz: the argument I've always made is you don't need to run tempest on the host. Especially because you already need cloud creds for it why not just spin up a guest with 2.7 and use that to run the tests18:43
dkranzmtreinish: I'll think about that18:45
mtreinishof course if we did drop 2.6 during icehouse we'll probably have to start working on python 3 support18:47
mtreinishsdague: ^^^ any thoughts?18:47
*** syerrapragada has joined #openstack-qa18:48
*** syerrapragada has left #openstack-qa18:48
afazekasmtreinish: the cli tests would run on the py2.7 host, instead of the remote py2.618:51
afazekashttps://bugs.launchpad.net/tempest/+bug/115029618:51
sdaguehonestly, I'd love to drop 2.6, I think it's just kind of nutty to keep it there18:52
mtreinishafazekas: on that note there is also this https://testrepository.readthedocs.org/en/latest/MANUAL.html#remote-or-isolated-test-environments18:53
mtreinishalthough I've never tested it18:53
*** SergeyLukjanov is now known as _SergeyLukjanov18:54
*** _SergeyLukjanov has quit IRC18:54
*** SergeyLukjanov has joined #openstack-qa18:54
openstackgerritDavid Kranz proposed a change to openstack/tempest: Replace assertLessEqual - is not in py26 testtools  https://review.openstack.org/5393318:56
openstackgerritA change was merged to openstack-dev/devstack: put the stackforge library installation behind a conditional  https://review.openstack.org/5391518:56
openstackgerritA change was merged to openstack/tempest: Added some test for image tags  https://review.openstack.org/5059218:57
afazekasself.assertTrue(len(container_list) <= limit, str(container_list))18:57
afazekassdague: https://review.openstack.org/#/c/52870/18:59
openstackgerritDavid Kranz proposed a change to openstack/tempest: Replace assertLessEqual - is not in py26 testtools  https://review.openstack.org/5393319:00
openstackgerritA change was merged to openstack/tempest: add tests for set_metadata in aggregate  https://review.openstack.org/5258519:02
openstackgerritA change was merged to openstack/tempest: Add filenames to skip_tracker.py output  https://review.openstack.org/5223719:07
afazekasBTW: More than 50 tempest patches waiting for review19:09
mtreinishafazekas: yeah, but how many without a -1 or -2 from either a reviewer or jenkins?19:10
afazekas4019:11
afazekasmany :)19:12
mtreinishafazekas: heh, yeah I was just curious because I see a lot of red just looking at the bare review queue19:13
mtreinishof course the flaky fail rate on the isolated neutron job is really high19:13
openstackrecheckopenstack/tempest change: https://review.openstack.org/53835 failed tempest because of: https://bugs.launchpad.net/bugs/1235486 and https://bugs.launchpad.net/bugs/124025619:13
mtreinishdkranz, afazekas: https://review.openstack.org/#/c/39609/ lets get v3 testing started19:14
afazekasmtreinish: I wonder was the fail rate different with the mysql/maridb job ?19:14
* afazekas got a "boss" alert, bye19:15
dkranzmtreinish: Out of curiosity, did you see how much time is added when running with v3 as well?19:16
mtreinishdkranz: it looks pretty sane: check-tempest-devstack-vm-full SUCCESS in 30m 51s and check-tempest-devstack-vm-postgres-full SUCCESS in 31m 10s19:17
dkranzmtreinish: OK, I'll take a look but may not be able to finish today.19:18
mtreinishdkranz: sure np, thanks19:18
mtreinishdkranz: granted those numbers are only with a small number of v3 tests added19:19
dkranzmtreinish: Yeah. I would expect total run time for v3 to be > time for v219:19
dkranzmtreinish: minus possible image tests not in v319:20
dkranzmtreinish: But I have not looked at v3 in detail yet.19:20
sdagueit's probably worth taking in clarkb's 8-way patch one way or another19:20
mtreinishsdague: yeah it also might help to revist storing the test repository on a semiregular interval to try to optimize the parallel splitting19:21
sdaguemtreinish / lifeless: is there actually a way to analyze our subunit output to figure out how lopsided we are19:22
sdaguethat would actually be really interesting to dump out at the end the runtimes of each testr process19:23
lifelesssdague: of each worker?19:24
lifelesssdague: yeah, its fairly trivial19:24
lifelesssdague: testr last --subunit | subunit-stats19:25
lifelessthat will get you the worker tags19:25
lifelessthen for each worker19:26
lifelesstestr last --subunit | subunit-filter --with-tag=worker-N19:26
lifelessgets you that stream19:26
lifelessand we can either pull the first and last event out, or my suggestion is we tweak subunit-stats to report duration19:27
lifelessso you'll have19:27
lifelesstestr last --subunit | subunit-filter --with-tag=worker-N | subunit-stats19:27
lifelessobviously we can also write a single-pass integrated filter too19:27
lifelessjust giving you what low-hanging fruit19:27
dkranzmtreinish: Can you give a +A to https://review.openstack.org/#/c/53933/ ?19:32
mtreinishdkranz: done19:33
dkranzmtreinish: Thanks!19:33
*** marun has quit IRC19:36
sdaguelifeless: yeh, what I would imagine from a ui perspective is something like http://paste.openstack.org/show/49754/19:38
lifelessI want to add a simple map-reduce idiom into the subunit cli tools19:38
lifelessso you can express this as one query and have it work fairly efficiently19:38
lifelesswithout having a use-case-specific filter19:39
lifelesssdague: sure19:39
lifelesssdague: if you're interested in adding that I'll happily give more detailed pointers ;)19:39
sdaguebut then you'd make me work in bzr :)19:40
sdaguehonestly, I don't really have cycles for a while19:40
sdaguebut maybe later on towards the end of the year19:40
lifelesssdague: one teeeny patch to add times output to -stats, rest can all be done in 'userspace'19:40
dkranzmtreinish: This review and follow-ons are invalid based on the meeting yesterday, right? https://review.openstack.org/#/c/52524/519:41
dkranzmtreinish: You are going to redo the conf feature part?19:41
mtreinishdkranz: for now I think it's fine actually this is just a rework of what's there now.19:42
dkranzmtreinish: ok19:42
mtreinishdkranz: to do what we were talking about in the meeting I need to add checks for other extensions in the test code before adding an option makes sense19:43
*** thomasbiege1 has quit IRC19:44
openstackgerritMauro S. M. Rodrigues proposed a change to openstack-infra/devstack-gate: Enable grenade to test upgrades from H to I  https://review.openstack.org/5394019:48
openstackrecheckopenstack/neutron change: https://review.openstack.org/44942 failed tempest because of: https://bugs.launchpad.net/bugs/123548619:59
openstackrecheckopenstack/neutron change: https://review.openstack.org/53414 failed tempest because of: https://bugs.launchpad.net/bugs/123548620:01
*** SergeyLukjanov has quit IRC20:07
openstackrecheckopenstack/neutron change: https://review.openstack.org/50519 failed tempest with an unrecognized error20:11
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: Let us gate grenade forward  https://review.openstack.org/5394020:22
*** gszasz has quit IRC20:23
openstackgerritA change was merged to openstack/tempest: Replace assertLessEqual - is not in py26 testtools  https://review.openstack.org/5393320:27
*** SergeyLukjanov has joined #openstack-qa20:39
openstackrecheckopenstack/nova change: https://review.openstack.org/53943 failed tempest because of: https://bugs.launchpad.net/bugs/124425520:48
openstackrecheckopenstack/neutron change: https://review.openstack.org/51877 failed tempest because of: https://bugs.launchpad.net/bugs/123548620:49
openstackrecheckopenstack/nova change: https://review.openstack.org/51908 failed tempest because of: https://bugs.launchpad.net/bugs/122502420:51
*** YorikSar has quit IRC20:52
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: Let us gate grenade forward  https://review.openstack.org/5394020:53
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: Let us gate grenade forward  https://review.openstack.org/5394020:54
openstackrecheckopenstack/neutron change: https://review.openstack.org/52834 failed tempest because of: https://bugs.launchpad.net/bugs/123548620:54
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Seed testr with data from a successful gate run  https://review.openstack.org/5395620:56
mtreinishsdague: ^^^20:57
*** jaypipes has quit IRC20:58
sdagueman, 36k lines.... :)20:58
sdaguedo we actually need the 0 file?20:58
mtreinish\o/ I'm totally going to win the tempest LOC for icehouse20:58
sdagueor can we live with the times.db?20:58
clarkblifeless: ^20:59
mtreinishsdague: I don't know20:59
sdaguehmmm... which is binary only, so not reviewable.20:59
sdagueI wonder if we can get a text version of something we could convert into the right format pre run20:59
clarkbthe subunit log has all of the time data in it, so in theory you could go from that to times.db21:00
*** DennyZhang has joined #openstack-qa21:06
openstackgerritA change was merged to openstack/tempest: Early die if on image gets killed  https://review.openstack.org/5353921:14
*** mriedem has quit IRC21:23
openstackgerritDennyZhang proposed a change to openstack-dev/devstack: handle multiple directories in fixup_stuff.sh  https://review.openstack.org/5324621:27
clarkbmtreinish: the normal devstack vm full job took 30 minutes21:29
clarkbI think the randomish distribution is sufficiently good and balancing load across the workes21:29
*** weshay has quit IRC21:31
openstackgerritDennyZhang proposed a change to openstack-dev/devstack: grep may be fooled in aggregate.sh  https://review.openstack.org/5323321:33
openstackrecheckopenstack/keystone change: https://review.openstack.org/53046 failed tempest because of: https://bugs.launchpad.net/bugs/122502421:52
*** DennyZha` has joined #openstack-qa21:52
*** DennyZhang has quit IRC21:53
*** DennyZha` has quit IRC21:57
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/53233 failed tempest with an unrecognized error22:04
openstackrecheckopenstack/swift change: https://review.openstack.org/44090 failed tempest because of: https://bugs.launchpad.net/bugs/124425522:13
openstackgerritChuck Short proposed a change to openstack-dev/devstack: Enable trusty  https://review.openstack.org/5396522:14
*** DennyZhang has joined #openstack-qa22:16
*** mlavalle has joined #openstack-qa22:27
*** DennyZhang has quit IRC22:33
openstackrecheckopenstack-infra/devstack-gate change: https://review.openstack.org/53940 failed tempest because of: https://bugs.launchpad.net/bugs/123543722:37
*** mkollaro has joined #openstack-qa23:09
openstackrecheckopenstack/glance change: https://review.openstack.org/53935 failed tempest because of: https://bugs.launchpad.net/bugs/123985623:15
*** markmcclain has quit IRC23:18
*** kong has joined #openstack-qa23:21
*** Lingxian has quit IRC23:21
*** Lingxian has joined #openstack-qa23:30
*** kong has quit IRC23:30
*** mlavalle has quit IRC23:36
openstackrecheckopenstack/nova change: https://review.openstack.org/53971 failed tempest with an unrecognized error23:57

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