Thursday, 2013-10-24

openstackrecheckopenstack/python-neutronclient change: https://review.openstack.org/53462 failed tempest with an unrecognized error00:03
*** markmcclain has quit IRC00:05
*** markmcclain has joined #openstack-qa00:05
*** networkstatic has joined #openstack-qa00:10
openstackrecheckopenstack/neutron change: https://review.openstack.org/53478 failed tempest with an unrecognized error00:29
*** matsuhashi has joined #openstack-qa00:31
*** networkstatic has quit IRC00:37
openstackrecheckopenstack/nova change: https://review.openstack.org/52286 failed tempest with an unrecognized error00:39
*** SergeyLukjanov has quit IRC00:42
*** nosnos has joined #openstack-qa00:43
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/53474 failed tempest with an unrecognized error00:45
*** wanghao has joined #openstack-qa01:02
*** openstackrecheck has quit IRC01:18
*** openstackrecheck has joined #openstack-qa01:25
openstackrecheckopenstack/neutron change: https://review.openstack.org/53478 failed tempest with an unrecognized error01:31
*** anteaya has quit IRC01:37
*** dkranz has joined #openstack-qa01:39
*** mriedem has joined #openstack-qa01:44
*** markmcclain has quit IRC01:50
openstackgerritClark Boylan proposed a change to openstack-infra/elastic-recheck: Better end of console log indexing detection.  https://review.openstack.org/5348701:58
clarkbsdague: mtreinish ^ is a bit of a hack, curious to see what you think, but it should make stuff more reliable right now01:59
*** harlowja has quit IRC02:14
*** harlowja has joined #openstack-qa02:16
openstackrecheckopenstack/nova change: https://review.openstack.org/35764 failed tempest with an unrecognized error02:34
*** mriedem has quit IRC02:36
openstackrecheckopenstack/heat change: https://review.openstack.org/52835 failed tempest with an unrecognized error02:42
openstackgerritChenZheng proposed a change to openstack/tempest: Test for the agent management extension API  https://review.openstack.org/4842103:02
openstackgerritChenZheng proposed a change to openstack/tempest: Test for the agent management extension API  https://review.openstack.org/4842103:05
*** yfried_mob has joined #openstack-qa03:05
openstackrecheckopenstack/neutron change: https://review.openstack.org/53244 failed tempest because of: https://bugs.launchpad.net/bugs/1217734 and https://bugs.launchpad.net/bugs/123963703:07
*** yfried_mob has quit IRC03:09
*** yfried has joined #openstack-qa03:13
*** mkoderer has quit IRC03:15
*** mkoderer has joined #openstack-qa03:16
*** Anju has quit IRC03:17
*** Anju has joined #openstack-qa03:18
*** mlavalle has joined #openstack-qa03:21
Lingxianclarkb, ping03:24
Lingxianclarkb, could you please tell me that what's the difference between 'gate' and 'smoke'?03:29
clarkbLingxian: I am not quite sure03:30
LingxianAnju, do you know that?03:30
clarkbI think the distinction has become less and less important as we only run only the smoke tests during the neutron test03:30
Anjuhii Lingxian  its a summit topic ,03:30
clarkbeverything else runs gate03:30
Anjuits gonna be decide which tests remark as smoke03:31
Lingxianafazekas told me that 'smoke' should be marked if neutron is required in the test.03:32
LingxianI want to know why03:32
Lingxianbut I found some test use 'smoke', but has nothing to do with neutron03:33
clarkbLingxian: probably because we run the smoke tests against neutron03:33
clarkbwe don't run the full set of gate tests against neutron03:33
Lingxianwhy 'test_get_default_quotas' in the compute/admin/test_quotas.py use 'smoke'?03:38
clarkbLingxian: the tests aren't only for neutron, but if the test tests neutron it needs to be marked smoke otherwise the test won't be used03:39
Lingxianclarkb, you mean if a test can be used to test neutron, it should be marked as 'smoke', right?03:39
clarkbyes03:40
clarkbbecause only smoke tests are run when neutron is enabled03:40
Lingxianthanks, clarkb, Anju, got that!03:42
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for quota set operation  https://review.openstack.org/5023403:45
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some negative tests for attach_interfaces  https://review.openstack.org/4923103:50
openstackrecheckopenstack/heat change: https://review.openstack.org/53369 failed tempest with an unrecognized error03:51
*** Dharmit has joined #openstack-qa03:52
*** dims has quit IRC03:53
openstackrecheckopenstack/tempest change: https://review.openstack.org/48421 failed tempest with an unrecognized error03:56
*** yfried has quit IRC04:02
openstackrecheckopenstack/neutron change: https://review.openstack.org/48574 failed tempest because of: https://bugs.launchpad.net/bugs/123548604:03
*** yfried|mob has joined #openstack-qa04:07
*** yfried|mob has joined #openstack-qa04:10
*** marun has joined #openstack-qa04:13
*** yfried|mob has quit IRC04:15
*** giulivo has quit IRC04:20
openstackrecheckopenstack/tempest change: https://review.openstack.org/50234 failed tempest because of: https://bugs.launchpad.net/bugs/123548604:21
openstackrecheckopenstack/tempest change: https://review.openstack.org/49231 failed tempest with an unrecognized error04:21
*** prekarat has quit IRC04:23
*** yfried_mob has joined #openstack-qa04:37
*** yfried|mob has joined #openstack-qa04:37
*** DennyZhang has joined #openstack-qa04:41
*** Madan has joined #openstack-qa04:46
*** wanghao has quit IRC04:52
*** thomasbiege1 has joined #openstack-qa04:53
*** SergeyLukjanov has joined #openstack-qa04:54
openstackgerritAaron Rosen proposed a change to openstack-dev/devstack: OVS still working?  https://review.openstack.org/5075605:02
*** SergeyLukjanov has quit IRC05:05
openstackrecheckopenstack/oslo.version change: https://review.openstack.org/53488 failed tempest with an unrecognized error05:09
*** yfried_mob has joined #openstack-qa05:11
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds api test to test_images  https://review.openstack.org/5333705:13
*** yfried_mob has quit IRC05:15
*** yfried has joined #openstack-qa05:17
*** rdekel has joined #openstack-qa05:26
*** yfried_mob has joined #openstack-qa05:41
openstackrecheckopenstack/tempest change: https://review.openstack.org/53337 failed tempest because of: https://bugs.launchpad.net/bugs/123548605:46
*** yfried|mob has joined #openstack-qa05:46
*** yfried_mob has quit IRC05:47
*** yfried_mob has joined #openstack-qa05:49
*** DennyZhang has quit IRC05:50
*** yfried|mob has quit IRC05:51
openstackgerritChenZheng proposed a change to openstack/tempest: Test for the agent management extension API  https://review.openstack.org/4842105:51
*** yfried_mob has quit IRC05:55
*** yfried|mob has joined #openstack-qa05:55
*** yfried|mob has quit IRC05:56
*** yfried_mob has joined #openstack-qa05:56
openstackrecheckopenstack/nova change: https://review.openstack.org/53502 failed tempest with an unrecognized error05:58
*** gszasz has joined #openstack-qa05:59
*** shilpa_ has joined #openstack-qa06:00
*** yfried_mob has quit IRC06:00
*** rdekel has quit IRC06:03
*** matsuhashi has quit IRC06:04
*** markmcclain has joined #openstack-qa06:04
*** rdekel has joined #openstack-qa06:05
*** wanghao has joined #openstack-qa06:08
*** matsuhashi has joined #openstack-qa06:09
*** jlibosva has joined #openstack-qa06:14
openstackrecheckopenstack/neutron change: https://review.openstack.org/51877 failed tempest because of: https://bugs.launchpad.net/bugs/123548606:15
*** shilpa_ has quit IRC06:17
*** SergeyLukjanov has joined #openstack-qa06:20
*** afazekas has joined #openstack-qa06:21
*** shilpa_ has joined #openstack-qa06:22
openstackrecheckopenstack/neutron change: https://review.openstack.org/53505 failed tempest because of: https://bugs.launchpad.net/bugs/123548606:29
*** prekarat has joined #openstack-qa06:34
*** thomasbiege1 has quit IRC06:35
*** shilpa_ has quit IRC06:40
openstackrecheckopenstack/tempest change: https://review.openstack.org/48421 failed tempest because of: https://bugs.launchpad.net/bugs/123548606:40
openstackrecheckopenstack/nova change: https://review.openstack.org/51780 failed tempest with an unrecognized error06:42
*** markmcclain has quit IRC06:51
*** thomasbiege1 has joined #openstack-qa06:51
*** yfried_mob has joined #openstack-qa06:54
*** matsuhashi has quit IRC06:54
*** matsuhashi has joined #openstack-qa06:55
*** SergeyLukjanov_ has joined #openstack-qa06:56
*** matsuhashi has quit IRC06:59
*** SergeyLukjanov has quit IRC06:59
*** matsuhashi has joined #openstack-qa07:01
openstackrecheckopenstack/nova change: https://review.openstack.org/53528 failed tempest with an unrecognized error07:02
openstackgerritYang Yu proposed a change to openstack/tempest: Adding active tests for updating/replacing image members  https://review.openstack.org/4890207:02
openstackgerritZhu Zhu proposed a change to openstack/tempest: Test for flavor-access-list with private/public flavor  https://review.openstack.org/5118807:04
*** SergeyLukjanov_ has quit IRC07:08
openstackgerritYang Yu proposed a change to openstack/tempest: Adding glance API DELETE /images/<imageid> active tests  https://review.openstack.org/4798307:11
openstackgerritYang Yu proposed a change to openstack/tempest: Adding negative tests for glance API  https://review.openstack.org/4857807:14
*** harlowja has quit IRC07:28
openstackrecheckopenstack/cinder change: https://review.openstack.org/52876 failed tempest with an unrecognized error07:29
*** yassine has joined #openstack-qa07:44
*** yassine has quit IRC07:50
*** yassine has joined #openstack-qa07:50
*** yfried_mob has quit IRC07:51
*** yfried_mob has joined #openstack-qa07:52
*** safchain has joined #openstack-qa07:54
*** yassine has quit IRC07:58
openstackrecheckopenstack/tempest change: https://review.openstack.org/48902 failed tempest with an unrecognized error08:03
openstackgerritafazekas proposed a change to openstack/tempest: Test image member is enforced  https://review.openstack.org/5026708:09
*** yassine has joined #openstack-qa08:13
*** yfried_mob has quit IRC08:14
*** giulivo has joined #openstack-qa08:19
*** fbo_away is now known as fbo08:19
openstackgerritJoe Hakim Rahme proposed a change to openstack-dev/devstack: Enables Swift crossdomain middleware by default  https://review.openstack.org/5311608:20
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds api test to test_images  https://review.openstack.org/5333708:27
openstackgerritafazekas proposed a change to openstack/tempest: Early die if on image gets killed  https://review.openstack.org/5353908:37
openstackgerritwingwj proposed a change to openstack/tempest: Add test cases for volume-transfer  https://review.openstack.org/5307008:39
*** mlavalle has quit IRC08:40
openstackgerritjun xie proposed a change to openstack/tempest: Test for service type management  https://review.openstack.org/5199708:46
*** ivanzhu has joined #openstack-qa08:52
openstackrecheckopenstack/tempest change: https://review.openstack.org/53337 failed tempest with an unrecognized error09:00
openstackgerritJoe Hakim Rahme proposed a change to openstack/tempest: Adds test to cover Swift healthcheck middleware  https://review.openstack.org/5179509:07
*** matsuhashi has quit IRC09:08
*** matsuhashi has joined #openstack-qa09:09
*** matsuhashi has quit IRC09:09
*** matsuhashi has joined #openstack-qa09:09
openstackrecheckopenstack/neutron change: https://review.openstack.org/52934 failed tempest because of: https://bugs.launchpad.net/bugs/122400109:10
openstackrecheckopenstack/tempest change: https://review.openstack.org/50267 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 and https://bugs.launchpad.net/bugs/1235435 and https://bugs.launchpad.net/bugs/1235437 and https://bugs.launchpad.net/bugs/123963709:10
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds api test to test_images  https://review.openstack.org/5333709:11
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some negative tests for attach_interfaces  https://review.openstack.org/4923109:12
Lingxianafazekas, ping09:16
openstackgerritIvan-Zhu proposed a change to openstack/tempest: add tests for server_password  https://review.openstack.org/5354809:22
openstackrecheckopenstack/tempest change: https://review.openstack.org/51997 failed tempest with an unrecognized error09:28
openstackrecheckopenstack/heat change: https://review.openstack.org/53508 failed tempest because of: https://bugs.launchpad.net/bugs/123985609:35
*** shilpa_ has joined #openstack-qa09:39
*** ivanzhu has left #openstack-qa09:44
openstackgerritafazekas proposed a change to openstack/tempest: TypeError on a neutron 500 error handling  https://review.openstack.org/5355409:46
afazekasLingxian: pong09:46
openstackrecheckopenstack/tempest change: https://review.openstack.org/49231 failed tempest with an unrecognized error09:56
*** giulivo has quit IRC09:57
openstackrecheckopenstack/tempest change: https://review.openstack.org/53548 failed tempest because of: https://bugs.launchpad.net/bugs/123548610:02
Madanafazekas: ping10:05
afazekasMadan: pong10:05
Madanafazekas: if you could review my patch https://review.openstack.org/#/c/53337/ would be a great help10:06
*** giulivo has joined #openstack-qa10:09
*** prekarat has quit IRC10:10
afazekasMadan: IMHO the get_image_metadata should do a HEAD method instead of GET10:11
Madanafazekas: sorry, i didnt understand !!10:13
*** jhenner has joined #openstack-qa10:15
afazekasMadan: forget it :)10:18
Madanafazekas: I Just joined from college,, sorry,, did u see my patch ??10:20
* afazekas NOTE the get_image_file method never used10:21
*** prekarat has joined #openstack-qa10:22
afazekasMadan: why not just add new assertions to the test_register_then_upload ?10:22
Madanafaxekas: did u mean like a new versions of assertion ??10:24
Madan**afazekas:10:24
afazekasMadan: I am not sure we really need new test case, of you compare the operations the new test case does the subset of the 'test_register_then_upload' , but with more assert method related to the get.10:26
Madanafazekas: I was like there is no dedicated test case for get api so i thought of writing one10:27
afazekasif we an 'self.assertEqual(200, resp.status)  and  'self.assertEqual(image_id, body['id'])'  test_register_then_upload we get almost the same thing, "for free"10:28
Anjuafazekas:  is this necessary to make the client for xml and json both : https://review.openstack.org/#/c/50880/10:31
MadanAfazekas: so shall i add those two assertion in test_register_then_upload only, or i keep it in the same way so that it improves readability !! Kind of confusion with me10:31
openstackgerritAnju Tiwari proposed a change to openstack/tempest: Added some test for image tags  https://review.openstack.org/5059210:34
afazekasMadan: I would prefer just add the asserts to the ' test_register_then_upload'.10:34
afazekasBTW: 'get_image_file' from the tempest/services/image/v2/json/image_client.py is never used, IMHO it is a serious miss10:35
MadanAfazekas:  Ok,, Thank you i'll make changes and will add a test case for get_image_file10:36
Madanafazekas : is that good enough with that patch ??10:36
afazekasok10:37
MadanAfazekas: Thank you :),, :)10:38
Anjuafazekas:  ^^^10:39
afazekasAnju: According to jenkins it is not, but it is strongly recommended10:39
Anjuafazekas:  in future we will follow the same10:40
*** matsuhashi has quit IRC10:46
Madanafazekas: self.assertIn('name', body) and self.assertEqual('New Name', body.get('name')) ***** shall i make it to self.assertEqual[name, body['name']            ??10:48
MadanAfazekas: like 2 lines into one line ??10:49
*** HenryG has joined #openstack-qa10:57
*** weshay has joined #openstack-qa11:00
afazekasMadan: no, it is little better with two assert. But I agree it is little bit too many code11:01
afazekasHowever the self.assertTrue  should be replaced with self.assertEqual  #visibility11:02
afazekasmkoderer, giulivo, sdague, dkarnz, mtreinish, cyeoh, jaypipes: https://review.openstack.org/#/c/53554/11:05
openstackrecheckopenstack/heat change: https://review.openstack.org/53562 failed tempest with an unrecognized error11:05
Madanafazekas: Okk,, Thank you :) :) :)11:05
Lingxianafazekas, https://review.openstack.org/#/c/50267/, I think too many changes in one patch, your idea?11:06
sdagueafazekas: where is the actual error you are trying to fix in the logs?11:07
giulivoafazekas, I wonder if that isn't an actual bug on the rest side?11:07
afazekasLingxian: I just fixed the things what the reviewers asked for :)11:08
openstackgerritwingwj proposed a change to openstack/tempest: Add some test_cases for glance_cli  https://review.openstack.org/5117011:08
Lingxianafazekas, is it appropriate to split them to different small patches?11:09
Lingxianafazekas, according to https://wiki.openstack.org/wiki/GitCommitMessages#Structural_split_of_changes11:10
afazekasLingxian: Would you prefer the previous version of the patch ?11:10
afazekassdague: I'll check it, but we should not do double fault even if the exception is malformed11:11
*** nosnos has quit IRC11:12
*** nosnos has joined #openstack-qa11:12
openstackgerritzhangyanzi proposed a change to openstack/tempest: Added some tests for reserve and unreserve volume  https://review.openstack.org/5357811:13
Lingxianafazekas, because I'm confused by the same problem. I do all right things in a new negative tests file. but somebody said that I should only seperate tests in one patch, and do other things(e.g. change import 'data_utils') in another patch11:14
Lingxianafazekas, I just don't know what's the right direction11:14
Lingxianafazekas, just can11:15
sdagueafazekas: I don't think that should be done with a try block, we should be more explicit about error parsing with different services11:15
Lingxianafazekas, just can't satify everyone11:15
*** adalbas has joined #openstack-qa11:16
*** ivanzhu has joined #openstack-qa11:16
*** wramthun has quit IRC11:17
*** nosnos has quit IRC11:17
*** dims has joined #openstack-qa11:18
*** cyeoh has quit IRC11:18
*** cyeoh_ has joined #openstack-qa11:18
*** gszasz has quit IRC11:18
afazekassdague: actually I am thinking about put every 'intelligence' into try expect Exception block , and falling back to str(resp_body) on any 500 error11:18
*** zigo_ has joined #openstack-qa11:18
*** morganfainberg has quit IRC11:19
*** sdague has quit IRC11:19
*** zigo has quit IRC11:19
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some negative tests for attach_interfaces  https://review.openstack.org/4923111:19
*** morganfainberg has joined #openstack-qa11:21
*** wramthun has joined #openstack-qa11:22
*** sdague has joined #openstack-qa11:22
openstackgerritwingwj proposed a change to openstack/tempest: Add test cases for volume-transfer  https://review.openstack.org/5307011:22
Lingxianafazekas, you there?11:25
afazekasLingxian: vote -1 if you think it should be spitted, and make recommendation about how to split it11:27
Lingxianafazekas, yes, I also think that we should do things right once for all about some small nits.11:28
giulivoafazekas, sdague, pardon the stupid question then but isn't the error response structure "standardized" across serivces?11:30
giulivoor at least shouldn't it be?11:31
sdaguegiulivo: apparently not :)11:31
giulivoyeah so I'd consider that more an actual bug on the server side11:31
afazekasdoes anyone has objection against adding tid or pid to the log messages ?11:31
openstackgerritzhangyanzi proposed a change to openstack/tempest: Added some tests for reserve and unreserve volume  https://review.openstack.org/5357811:31
sdagueafazekas: why? what do you think it will help?11:32
afazekasmultiple processes logging to the same tempest.log11:34
afazekasby grepping on tid/pid you would see the logs just from one process11:34
sdagueafazekas: ok, sure, I'm cool with that11:35
openstackgerritwingwj proposed a change to openstack/tempest: Add test cases for volume-transfer  https://review.openstack.org/5307011:35
dkranzsdague, afazekas : Can you please push https://review.openstack.org/#/c/53428/ so I can restart my watcher?11:43
sdaguesure11:44
*** safchain_ has joined #openstack-qa11:55
*** safchain has quit IRC11:55
afazekaslooks like the logging fixture changing the loggers log level instead of the handler..11:56
*** prekarat has quit IRC11:58
*** prekarat has joined #openstack-qa11:58
openstackrecheckopenstack/neutron change: https://review.openstack.org/50519 failed tempest because of: https://bugs.launchpad.net/bugs/123548612:03
*** dkorn has joined #openstack-qa12:04
*** anteaya has joined #openstack-qa12:09
openstackrecheckopenstack/tempest change: https://review.openstack.org/50267 failed tempest because of: https://bugs.launchpad.net/bugs/1217734 and https://bugs.launchpad.net/bugs/1235486 and https://bugs.launchpad.net/bugs/123963712:10
yfriedanyone knows what is the default behavior of secgroup? if I have NO rule in any secgroup, what is blocked? what is allowed?12:10
*** mkollaro has joined #openstack-qa12:11
yfriedin AIO - is cross tenant network allowed by default?12:11
openstackrecheckopenstack/tempest change: https://review.openstack.org/53428 failed tempest with an unrecognized error12:15
openstackrecheckopenstack/tempest change: https://review.openstack.org/53578 failed tempest because of: https://bugs.launchpad.net/bugs/123963712:20
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds api test to test_images  https://review.openstack.org/5333712:22
*** shilpa_ has quit IRC12:23
*** gszasz has joined #openstack-qa12:24
Madanafazekas : ping12:33
*** dkranz has quit IRC12:35
afazekasMadan: can you assert something related to the get_image_file' body ?12:35
Madanafazekas yes i've mada changes suggested by u12:36
Madanafazekas: link https://review.openstack.org/#/c/53337/12:36
afazekasyou forget to replace the assertTrue with assertEqual , but the arguments are changed12:36
openstackrecheckopenstack/tempest change: https://review.openstack.org/53070 failed tempest because of: https://bugs.launchpad.net/bugs/122400112:37
Madangive me 2 min i'll do that,, sorry for that12:37
*** dims has quit IRC12:38
*** dims has joined #openstack-qa12:40
afazekasself.assertEqual(body, '*' * 1024)12:44
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds api test to test_images  https://review.openstack.org/5333712:44
afazekasMadan: ^^12:44
Madanafazekas: Do i need to add it for imagefile assertion ??12:45
afazekasMadan: since this would be actual reading back of the created image file. It should be asserted.12:49
Madansure i'll add those lines just give me 2 more minutes my devstack is getting up,, seriously i'm happy to work with,, you were really nice :) ::)12:50
openstackrecheckopenstack/nova change: https://review.openstack.org/53596 failed tempest with an unrecognized error12:51
Anjuhe is Madan not were :)12:52
Anjuafazekas:   need review comments https://review.openstack.org/#/c/50592/12:52
openstackrecheckopenstack/nova change: https://review.openstack.org/52014 failed tempest with an unrecognized error12:55
openstackrecheckopenstack/nova change: https://review.openstack.org/53597 failed tempest with an unrecognized error12:59
*** Dharmit has quit IRC13:00
Anjuafazekas:  thanks  :)13:01
Anjuafazekas:  I think I need ti change the commit message :(13:02
Anjuspcae mistake13:02
Anjuspace mistake13:02
openstackgerritAnju Tiwari proposed a change to openstack/tempest: Added some test for image tags  https://review.openstack.org/5059213:04
* afazekas just very rarely reads the commit message..13:05
Anjuafazekas:  but please review it next time . thanks in advance13:07
openstackrecheckopenstack/nova change: https://review.openstack.org/53595 failed tempest with an unrecognized error13:09
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds api test to test_images  https://review.openstack.org/5333713:09
Madanafazekas: I guess i'm up with the changes :)13:10
YorikSarbnemec: Hello. Around?13:13
openstackgerritJoe Gordon proposed a change to openstack/tempest: Set max_template_size to heat's default value  https://review.openstack.org/5361313:14
*** mriedem has joined #openstack-qa13:14
openstackgerritJoe Gordon proposed a change to openstack-dev/devstack: Use heat's default value for max_template_size  https://review.openstack.org/5361413:14
*** wanghao has quit IRC13:15
YorikSarbnemec: Please take a look at https://review.openstack.org/53391 and https://review.openstack.org/5361213:16
Madanafazekas : afazekas are you around ??13:16
afazekasMadan: FYI: the dict.get() returns with None, the [] raises KeyError exception http://www.fpaste.org/49180/82620547/13:16
*** DennyZhang has joined #openstack-qa13:17
YorikSarbnemec: Svetlana is working on incorporating these into her tests in Ceilometer.13:17
afazekasMadan: +213:17
*** yfried_mob has joined #openstack-qa13:18
MadanAfazekas:Thank you for your guidance :) :) :)13:18
afazekasMadan: You are welcome13:19
openstackrecheckopenstack/python-neutronclient change: https://review.openstack.org/53602 failed tempest because of: https://bugs.launchpad.net/bugs/123548613:19
*** Madan has quit IRC13:22
openstackrecheckopenstack/nova change: https://review.openstack.org/53603 failed tempest with an unrecognized error13:23
*** jaypipes has joined #openstack-qa13:25
*** ChanServ sets mode: +o jaypipes13:25
*** shilpa_ has joined #openstack-qa13:35
bnemecYorikSar: Okay, thanks.  I'll take a look.13:36
YorikSarbnemec: Great.13:40
YorikSarbnemec: Here's an example from Svetlana: https://review.openstack.org/4480913:41
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/53614 failed tempest with an unrecognized error13:50
openstackgerritNejc Saje proposed a change to openstack/tempest: Adds client and CRUD tests for Ceilometer alarms  https://review.openstack.org/3923713:52
openstackrecheckopenstack/horizon change: https://review.openstack.org/53370 failed tempest with an unrecognized error13:58
openstackrecheckopenstack/neutron change: https://review.openstack.org/51877 failed tempest because of: https://bugs.launchpad.net/bugs/123548614:10
*** DennyZhang has quit IRC14:12
*** dkranz has joined #openstack-qa14:12
*** yfried_mob has joined #openstack-qa14:16
mtreinishmordred: I thought it failed with the version in setup.cfg that's why the first revision had it but the second didn't.14:17
mtreinishlet me double check though14:17
openstackrecheckopenstack/keystone change: https://review.openstack.org/53623 failed tempest with an unrecognized error14:19
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Update to latest pbr  https://review.openstack.org/5278514:22
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Updated from global requirements  https://review.openstack.org/5104114:23
sdagueok, QA DS schedule is pushed - http://icehousedesignsummit.sched.org/14:23
openstackrecheckopenstack/nova change: https://review.openstack.org/53633 failed tempest with an unrecognized error14:23
mtreinishsdague: cool14:24
openstackrecheckopenstack/nova change: https://review.openstack.org/53528 failed tempest with an unrecognized error14:24
mtreinishmordred: I guess I was wrong (that teaches me to push things out over the weekend) it works fine with the version set in setup.cfg: https://jenkins01.openstack.org/job/gate-tempest-docs/2738/console14:25
openstackrecheckopenstack/nova change: https://review.openstack.org/53372 failed tempest with an unrecognized error14:25
sdaguemtreinish: ok, can you push the version you like of that patch then?14:26
mtreinishsdague: it's up https://review.openstack.org/5278514:27
sdaguedkranz: https://review.openstack.org/#/c/52412/1 +2 on this one please :)14:27
sdaguesetting the version, is good14:28
sdaguethough, honestly, I'll just +A that through as PTL call14:28
dkranzsdague: Done14:28
*** prekarat1 has joined #openstack-qa14:28
*** prekarat has quit IRC14:31
*** adalbas has quit IRC14:33
*** adalbas has joined #openstack-qa14:35
openstackrecheckopenstack/requirements change: https://review.openstack.org/52137 failed tempest because of: https://bugs.launchpad.net/bugs/122566414:36
mordredmtreinish: w00t! at least it's LESS weird why it was breaking before14:38
mordredI STILL don't understand what broke. at. all.14:38
openstackrecheckopenstack/tempest change: https://review.openstack.org/39237 failed tempest with an unrecognized error14:42
*** jhenner has quit IRC14:43
*** SergeyLukjanov has joined #openstack-qa14:44
*** jhenner has joined #openstack-qa14:44
jog0are you looking into why havana gate is down?14:51
openstackgerritafazekas proposed a change to openstack/tempest: Preserve the configured log level  https://review.openstack.org/5365314:51
*** dkorn has quit IRC14:52
openstackrecheckopenstack/cinder change: https://review.openstack.org/53642 failed tempest with an unrecognized error14:53
*** adalbas has quit IRC14:55
openstackgerritZhu Zhu proposed a change to openstack/tempest: Enhance tests for flavor_extra_spec API  https://review.openstack.org/5052314:56
*** yfried|mob has joined #openstack-qa14:58
*** yfried has quit IRC14:59
*** jhenner has quit IRC15:00
jog0sdague: ^15:01
*** yfried_mob has quit IRC15:01
sdaguejog0: I'm not, I just saw that email15:01
*** bill_az_ has joined #openstack-qa15:02
*** yfried|mob has quit IRC15:03
jog0about to look at it myself first filing a e-r query15:03
openstackrecheckopenstack/tempest change: https://review.openstack.org/53428 failed tempest with an unrecognized error15:05
sdaguejog0: cool15:05
openstackrecheckopenstack/neutron change: https://review.openstack.org/53188 failed tempest with an unrecognized error15:06
*** sthaha has quit IRC15:07
openstackgerritEoghan Glynn proposed a change to openstack-dev/devstack: Revert default ceilometer storage driver to mongodb  https://review.openstack.org/5365715:09
*** prekarat1 has quit IRC15:10
*** adalbas has joined #openstack-qa15:12
silehthi, I have a question about which services is enabled in which check-tempest-devstack-XXXXXX test15:12
silehtit seems that ceilometer is not enabled in all tests15:12
mtreinishsileht: it should be in the devstack log15:12
silehtwe are currently writting the first ceilometer tests: https://review.openstack.org/#/c/39237/15:13
mtreinishsileht: the code that setups devstack for the gate is here: https://git.openstack.org/cgit/openstack-infra/devstack-gate/15:13
silehtthis one fail on two checks because ceilometer is disabled in this one.15:13
mtreinishsileht: oh, you mean grenade15:14
silehtmtreinish, yes15:14
mtreinishyeah I don't think we have ceilometer enabled for grenade15:14
mtreinishmaurosr: ^^^15:14
silehtmtreinish, I'm sure not :) we are just started the integration testing for now15:15
mtreinishgrenade is the upgrade testing and since ceilometer wasn't integrated until havana there wasn't anything to upgrade from15:15
mtreinishsileht: maurosr is working on getting grenade going for icehouse so we should make sure he knows to turn it on to test h->master upgrade15:16
mtreinishsileht: you can probably just get around that failure in the meantime by removing the smoke tag from the test though15:16
silehtmtreinish, does grenade use tempest tests to validate the upgade ?15:16
mtreinishgrenade only runs tempest tests with the smoke attr right now (that will probably change soon though)15:17
mtreinishsileht: yes it runs the smoke tests after an upgrade15:17
silehtok it seems I need to update devstack/lib/tempest to run ceilometer tempest tests only if ceilometer is enabled in devstack15:19
openstackgerritAdam Spiers proposed a change to openstack-dev/devstack: Fix shocco errors and warnings  https://review.openstack.org/4987215:20
mtreinishsileht: devstack doesn't set the service availble config option correctly ceilo?15:21
mtreinishI guess it wouldn't since you haven't added the option yet15:21
silehtmtreinish, no I will propose a review for this, this is normal because service_avilable is added by this review https://review.openstack.org/#/c/39237/ in tempest15:21
adalbassileht, ceilometer is listed as enabled services for grenade. look at the console log, for instance: http://logs.openstack.org/85/52885/4/check/check-tempest-devstack-vm-full/f6f9961/console.html15:22
mtreinishadalbas: that's not a grenade log15:22
openstackgerritAdam Spiers proposed a change to openstack-dev/devstack: Fix some Markdown formatting issues  https://review.openstack.org/4987415:23
afazekassdague, mtreinish: https://review.openstack.org/#/c/53653/115:24
mtreinishsileht: so I think in the short term on 39237 just remove the smoke attr from the tests you are adding and open a grenade bug about ceilometer15:24
adalbasmtreinish, right, my mistake. I remembered to have used that once before15:24
* afazekas https://bugs.launchpad.net/python-fixtures/+bug/124425615:24
adalbasbut it s seems it s still not enabled.15:24
silehtmtreinish, this is a devstack issue the tempest service aviliabily is not disabled for ceilometer if ceilometer services is disabled15:25
*** wramthun has quit IRC15:26
*** afazekas has quit IRC15:26
mtreinishsileht: ok yeah push out a review for that and we'll see if things blow up (because we're adding an option to the config that doesnt exist)15:27
mtreinishsileht: but we still want to open a grenade bug to enable ceilo for it15:27
silehtmtreinish, yes too15:27
*** wramthun has joined #openstack-qa15:32
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Handle the CM service availability in tempest  https://review.openstack.org/5366615:32
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Allow to configuration the CM pipeline interval  https://review.openstack.org/5329115:32
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: cm-cpu-agent only support libvirt driver  https://review.openstack.org/5329015:32
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Ensure cm-api is ready before start cm-alarm  https://review.openstack.org/5329715:32
openstackgerritAdam Spiers proposed a change to openstack-dev/devstack: git-ignore files generated by "./tools/build_docs.sh ."  https://review.openstack.org/5366715:33
silehtmtreinish, as you see I have some waiting fixes for ceilometer to have a working setup in gate15:33
*** shilpa_ has quit IRC15:33
silehtmtreinish, the topic is : https://review.openstack.org/#/q/status:open+project:openstack-dev/devstack+branch:master+topic:sileht/ceilometer-fix,n,z15:33
silehtmtreinish, thanks for your help15:33
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/53321 failed tempest with an unrecognized error15:34
openstackgerritA change was merged to openstack/tempest: Set tempest version for icehouse  https://review.openstack.org/5241215:36
openstackrecheckopenstack/tempest change: https://review.openstack.org/53653 failed tempest because of: https://bugs.launchpad.net/bugs/123548615:37
openstackrecheckopenstack/nova change: https://review.openstack.org/53528 failed tempest with an unrecognized error15:45
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Add query for bug 1244255  https://review.openstack.org/5367015:45
*** yassine has quit IRC15:46
maurosrmtreinish: I was lunching.. reading backlog15:46
jog0sdague mtreinish: ^15:47
sdaguejog0: lgtm15:47
sdaguenow you need to review my patches there :)15:47
openstackrecheckopenstack/tempest change: https://review.openstack.org/53428 failed tempest because of: https://bugs.launchpad.net/bugs/1217734 and https://bugs.launchpad.net/bugs/123963715:47
openstackgerritFabien Boucher proposed a change to openstack/tempest: Add functional tests for auth through s3_token  https://review.openstack.org/5367115:47
mtreinishjog0: +A15:48
mtreinishdkranz: ^^^ ooh a double hit on your patch15:49
mtreinishyou'll have good luck for the rest of the day15:49
dkranzmtreinish: Experience says otherwise :)15:49
dkranzmtreinish: And the second one led me to a ticket that was marked as closed :(15:50
jog0mtreinish: thanks15:50
jog0now to figure out why havana gate is down15:50
dkranzmtreinish: I see it is actually a triple hit15:50
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add query for bug 1244255  https://review.openstack.org/5367015:50
dkranzmtreinish: what is neutron-pg-isolated? That's new.15:51
*** mlavalle has joined #openstack-qa15:51
*** YorikSar is now known as YorikSar_AFK15:54
*** dansmith is now known as someoneelse15:56
*** someoneelse is now known as dansmith15:56
*** openstackrecheck has quit IRC15:57
*** openstackrecheck has joined #openstack-qa15:58
jog0dkranz: I think your rooting for trouble15:58
jog0we should prod the integrated projects to fix there bugs15:58
*** mlavalle has quit IRC15:59
*** ivanzhu has left #openstack-qa16:00
*** rdekel has quit IRC16:00
*** HenryG has quit IRC16:06
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/53657 failed tempest because of: https://bugs.launchpad.net/bugs/1217734 and https://bugs.launchpad.net/bugs/123543716:07
*** HenryG has joined #openstack-qa16:07
*** bill_az_ has quit IRC16:08
openstackgerritAdam Spiers proposed a change to openstack-dev/devstack: git-ignore files generated by "./tools/build_docs.sh ."  https://review.openstack.org/5366716:09
*** yfried|mob has joined #openstack-qa16:11
*** markmcclain has joined #openstack-qa16:13
openstackrecheckopenstack/neutron change: https://review.openstack.org/53660 failed tempest because of: https://bugs.launchpad.net/bugs/123548616:13
jog0gate fail16:15
jog0sdague: want to send an email to the ML with top bugs again?16:16
openstackrecheckopenstack/nova change: https://review.openstack.org/53420 failed tempest with an unrecognized error16:17
openstackrecheckopenstack/nova change: https://review.openstack.org/53404 failed tempest with an unrecognized error16:17
openstackrecheckopenstack/tempest change: https://review.openstack.org/53671 failed tempest with an unrecognized error16:18
*** thomasbiege1 has quit IRC16:22
*** fbo is now known as fbo_away16:23
openstackgerritChris Behrens proposed a change to openstack-dev/devstack: Clean up use of global conf vars in start_nova  https://review.openstack.org/5368116:26
openstackgerritChris Behrens proposed a change to openstack-dev/devstack: Start nova-compute with child cell conf  https://review.openstack.org/5346816:26
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/53291 failed tempest with an unrecognized error16:35
*** afazekas has joined #openstack-qa16:39
openstackgerritAdam Spiers proposed a change to openstack-dev/devstack: only clone docs from gh-pages branch if they aren't there  https://review.openstack.org/5368716:43
*** weshay has quit IRC16:43
*** weshay has joined #openstack-qa16:43
openstackgerritAdam Spiers proposed a change to openstack-dev/devstack: Fix some Markdown formatting issues  https://review.openstack.org/4987416:46
openstackgerritA change was merged to openstack-dev/devstack: Just reset the repo instead of trying to co files  https://review.openstack.org/4841116:48
openstackgerritA change was merged to openstack/tempest: Add more stuff to the whitelist  https://review.openstack.org/5342816:49
*** mlavalle has joined #openstack-qa16:53
*** harlowja has joined #openstack-qa16:53
openstackrecheckopenstack/nova change: https://review.openstack.org/53404 failed tempest with an unrecognized error16:54
*** hemnafk is now known as hemna16:58
mtreinishdkranz: that's the neutron job with tenant isolation enabled16:59
dkranzmtreinish: ok, so it should soon go away when everything is working?16:59
mtreinishwe realized that the asymetrical gate made it more likely to push things that exposed more neutron bugs (and broke the neutron gate)16:59
sdaguejog0: honestly, I was going to try to get some tooling around that first, because I think e-r is only catching a small number of the bugs at thsi point16:59
*** yfried|mob has quit IRC17:00
*** yfried|mob has joined #openstack-qa17:00
mtreinishdkranz: yeah once we can get neutron jobs running everything and in parallel it should go away17:00
dkranzMeeting time17:00
mtreinishit's just that everytime I add more tests it exposes more bugs17:00
jog0sdague: cool, i need to approve your patches first anyway17:01
clarkbmtreinish: thats a fun problem17:01
*** jhenner has joined #openstack-qa17:02
mtreinishclarkb: actually add more tests is a misnomer because the actual number of tests doesn't change17:02
openstackgerritJoe Gordon proposed a change to openstack/tempest: Bump six version to match os requirements  https://review.openstack.org/5369017:02
*** jhenner has quit IRC17:03
jog0clarkb: the gate will tell is if it works or not17:03
*** jhenner has joined #openstack-qa17:03
jog0err wrong room17:03
mtreinishI'm just turning on tenant isolation (which makes networks with neutron now)17:03
*** safchain_ has quit IRC17:03
openstackrecheckopenstack/nova change: https://review.openstack.org/53422 failed tempest with an unrecognized error17:21
dosaboymkoderer: ping17:21
mkodererdosaboy: I am at the qa meeting and after that I have to leave17:22
mkodererdosaboy: could we chat tomorrow?17:22
*** hemna has quit IRC17:22
dosaboymkoderer: sure was just gonna ask if you had a chance to peek at https://review.openstack.org/#/c/45465/17:24
dosaboybut it can wait17:24
dosaboyping me tomorrow17:24
mkodererdosaboy: ok great I will do it17:24
dosaboythanks!17:24
*** hemna has joined #openstack-qa17:25
*** hemna has quit IRC17:26
*** gszasz has quit IRC17:27
jog0sdague mtreinish: global requirements https://review.openstack.org/#/c/51041/17:29
jog0needs reviewing17:29
*** jlibosva has quit IRC17:31
*** SergeyLukjanov is now known as _SergeyLukjanov17:31
*** mlavalle has quit IRC17:31
*** hemna has joined #openstack-qa17:31
*** _SergeyLukjanov has quit IRC17:31
*** SergeyLukjanov has joined #openstack-qa17:32
*** jlibosva has joined #openstack-qa17:33
weshaydkranz, anything on smoke?17:37
*** fbo_away is now known as fbo17:37
mtreinishjog0: oops yeah I forgot that I had to rebase that17:38
mtreinishI'll push it through17:38
jog0mtreinish: you need someone to merge your patch that it deps on too17:39
openstackrecheckopenstack/nova change: https://review.openstack.org/53466 failed tempest with an unrecognized error17:39
dkranzweshay: In the meeting now17:41
weshaythanks!17:42
mtreinishdkranz: can you push this through when you get a chance: https://review.openstack.org/#/c/52785/17:42
openstackgerritA change was merged to openstack-infra/elastic-recheck: add SearchResultSet and Hit objects  https://review.openstack.org/5269817:45
*** openstackrecheck has quit IRC17:49
*** openstackrecheck has joined #openstack-qa17:49
*** markmcclain has quit IRC17:56
*** yfried|mob has quit IRC17:57
*** yfried|mob has joined #openstack-qa17:57
*** markmcclain has joined #openstack-qa17:59
jog0I think there is a issue with the stable branch tests18:00
sdaguejog0: ok, so why isn't global requirements addressing this?18:00
jog0sdague: because of something about tox18:01
jog0in general we have an issue though18:01
jog0that global reqs cannot fix18:01
*** yfried has joined #openstack-qa18:01
dkranzweshay: See discussion in #openstack-meeting18:01
weshaythanks dkranz18:01
jog0if stable/havana requires a specific version and trunk python*client needs a different version18:01
jog0then we need to versions installed18:01
jog0somewhow here running tempest in tox forced six<1.4.0 to be installed18:03
jog0which broke novaclient18:03
sdagueis this because we're running the tempest unit tests?18:03
sdaguehave to run to a meeting, back in 3018:03
mtreinishsdague: that's a separate job now18:04
mtreinishin stable/havana too18:04
sdagueyeh, I'm confused why tempest requirements aren't getting overwritten18:04
jog0it appears they are but this patch does fix things18:04
jog0unless somehow gate fixed itself18:04
jog0actually we will find out soon18:04
jog0if https://review.openstack.org/#/c/52413/1 merges then my patch isn't needed18:05
jog0if it doesn't then ...18:05
clarkbtempest requirements aren't getting overwritten in new tox I think18:05
mtreinishclarkb: oh great that probably means I screwed up the tox.ini when I upgraded it18:06
mtreinishsigh...18:06
clarkbold tox <= 1.4.2 would only install packages into the venv they weren't available globally and sitepackages were enabled18:06
*** afazekas has quit IRC18:06
clarkb*when they weren't18:06
clarkbthis as regardless of version. New tox seems to check the version and install packages to the venv if the required version isn't met by the sitepackages18:06
mtreinishjog0: is the global requirement for six in stable/havana out of sync for tempest in general? if so then we probably want your patch anyway18:07
jog0mtreinish: yes18:07
jog0but if I am write your patch won't merge18:08
jog0which means mine won't either18:08
mtreinishclarkb: did I miss an option in the tox.ini when I set the minversion to 1.6?18:08
clarkbmtreinish: I am not sure yet. Let me look at it a bit more18:08
clarkbmtreinish: it may just be a change to tox (honestly a good one in most situations)18:08
clarkbmtreinish: I wonder if the python setup.py develop that is run on tempest is doing it18:10
clarkbsince that should install dependencies18:10
jog0clarkb: when is pip freeze run?18:10
clarkbit will find the site packages but if they don't satisfy the requirements will isntall the version it wants?18:10
clarkbjog0: pip freeze is run globally18:11
jog0at the end?18:11
clarkbya18:11
clarkbjog0: we need to run it in the venv to see the version in the venv18:11
jog0clarkb: the right version of tox will fix this?18:14
clarkbjog0: no18:14
clarkbwe don't want old tox18:14
clarkbbut old tox would fix it I think18:15
clarkbmordred: if you are still around ^18:15
jog0clarkb: so whats the long term fix?18:15
jog0venvs?18:15
mordredaroo?18:16
jog0for clients (which float on trunk)18:16
jog0ohh the expert in deps is here18:16
clarkbjog0: long term fix is putting clients in their own venv I think18:16
clarkbtempest and novaclient shouldn't be smooshed together like that18:16
* mordred is about to get on a call with a reporter - will be back ina bit18:17
mordredclarkb: why not?18:17
clarkbmordred: because tempest has stable branches and novaclient doesn't18:17
mordredand?18:17
clarkbthey have common dependencies with different version requirements18:17
clarkbthat causes conflicts and peopel cry18:17
mordrednope. we have to fix that and prevent it18:17
mordredit's not ok18:17
clarkbright we are fixing it in this case the way you want us to fix it18:18
mordredyay!18:18
clarkbbut I am not sure how long term manageable that is18:18
mordredit's one of the things that gating trunk clients on stable servers will fix18:18
clarkb(I think this is the sort of stuff that works great in the global reqs summit session)18:18
mordredbecause it will not allow the clients to grow a conflicting requirements18:18
mordredbecause they need to be not allowed to grow a conflicting requirmemt18:18
mordredbecause they may be installed with stable servers in the real world18:19
jog0Mordred why not?18:19
jog0mordred: I would like to pretend that doesn't happen but your right18:19
mordredif you install stable/grizzly right now, and you install python-keystoneclient on that server, you should be installing the latest keystoneclient18:19
mordredbecause that's how we do the client libs18:19
mordredthis is why we have to start gating on the combo18:19
mordredalso, they're client libs, they can probably be more conservative with their deps18:19
* jog0 blames six18:20
jog0well the issue inthis case was tempest didn't sync to global reqs for havana18:21
clarkbit was the pinning of six, we should never do that with six18:21
jog0AFAIK we fixed that but tempest didn't sync up18:21
clarkbjog0: they did sync the pin but didn't sync the unpin after I made folks revert the pin18:21
jog0clarkb: right18:21
clarkbI probably should've done a better jobs of making sure a sync post pin happened18:21
clarkb* post unpin18:22
jog0clarkb: to get the clients gating on this, we just need to add stable/havana test to clients right?18:22
jog0aka a one line change for novaclient18:22
clarkbjog0: yes, mordred has been working on that18:22
clarkbno it isn't a one line change18:22
jog0no?18:22
clarkbit is semi complicated, mordred has been getting pieces to fall into place18:22
mordredhttps://review.openstack.org/#/c/41931/18:23
clarkbjog0: there are problems in the way we check out code for testing that need to be fixed before it will just work18:23
clarkbI think d-g got updated with the fix18:23
clarkbnow 41931 needs to go in18:23
jog0ohh I see, stable/havana is failing because its using tempests versions of reqs in novalclient but we want to actually test the stable/havana version of global reqs instead18:26
clarkbjog0: right because novaclient is running within the context of tempests venv18:27
* jog0 has a moment of clarity 18:28
jog0clarkb: so now https://review.openstack.org/#/c/52413/ is failing18:28
jog0I guess I need to not make that a dependancy for the fix18:29
clarkbit will need to be the other way around, fix first18:30
clarkbor squas htme18:30
clarkb* squash them18:30
*** markmcclain has quit IRC18:33
*** markmcclain has joined #openstack-qa18:33
jog0dropped the dep (forced to type 'git review stable/havana' instead) https://review.openstack.org/#/c/53699/18:39
mtreinishjog0: isn't more work to rebase the patch so you didn't have to type the branch name then to do it up front?18:40
jog0tempest-cores ^18:40
jog0mtreinish: sshh18:40
jog0it was about doing it once, which turned out to  be wrong18:41
*** jlibosva has quit IRC18:42
sdaguejog0: so I really want to figure out why six is maybe getting replaced18:42
jog0clarkb: had a theory18:42
sdagueclarkb: theorytime?18:43
sdaguejog0: because otherwise, this could break us in the future18:43
sdagueso I'd rather get the tox fix now if we can18:43
jog0sdague: see mordred ranting on that part18:43
clarkbtheorytime: six is installed globally so tox sees that as a sitepackage and would use it except tempest requires a version that conflicts with the global version so it installs the version tempest wants into the venv overriding the global version18:44
jog0(in scrollback)18:44
sdagueclarkb: but we aren't supposed to have a venv18:44
sdaguethat was the point of the way we were using tox18:44
sdagueso we need to either figure out how to avoid doing that, or kill our use of tox for running, and come up with a better way18:45
* sdague realizes his understanding of tox probably contributed to this problem18:45
clarkbsdague: you have a venv there is no way around that18:46
sdagueoh, we can't get rid of it completely?18:46
clarkbsdague: but sitepackages is enabled so as long as tempest requirements match globally installed requirements you don't install much to the venv18:46
sdagueok, right, but that wasn't really the intent :)18:46
jog0sdague: can you review https://review.openstack.org/#/c/53699/18:47
jog0one more time18:47
sdagueok18:47
sdaguejog0: done18:47
clarkbsdague: right it was a hack workaround because tox == virtualenvs18:48
jog0sdague: thanks18:48
sdagueclarkb: ok, I guess when we went down this path there was a lack of understanding here18:48
sdaguehmmm... actually, no that still seems weird18:49
sdaguebecause we should have g-r stomped the tempest requirements18:49
clarkbsdague: we did partially18:49
clarkbtempest picked up the six pin, but not the unpin after I went around and said no to the pin18:49
clarkbI should've been more proactive about getting projects to pick up the unpin18:49
*** markmcclain has quit IRC18:49
sdagueclarkb: right, but that shouldn't matter18:50
sdaguein the devstack gate we take g-r and blast away everyone's requirements18:50
sdagueand say, we don't care, g-r is right18:50
clarkbI thought tempest was special case there18:50
clarkbbut maybe it shouldn't be18:50
sdaguewell, it shouldn't be18:50
clarkbya so that is the bug then18:50
jog0bug: tempest isn't special18:50
sdaguewell devstack looks like it should be stomping on it18:51
*** markmcclain has joined #openstack-qa18:51
sdagueif you call setup_develop in devstack, it does the g-r update18:51
sdagueoh, crap18:52
sdaguesigh, we have an undo at the end18:52
jog0haha18:52
sdagueman, and dhellman isn't even in channel to yell at about that :)18:53
sdaguehttps://github.com/openstack-dev/devstack/blob/master/functions#L126118:54
sdagueclarkb: ok, what's a good variable for me to work off of that's "if we are in the gate" :)18:54
clarkbsdague: let me see18:54
*** markmcclain has quit IRC18:54
*** yfried|mob has quit IRC18:55
*** yfried|mob has joined #openstack-qa18:55
clarkbsdague: ZUUL_REF maybe?18:55
clarkbneed to make sure that is present in the periodic jobs18:57
sdagueactually, i'm going to make it an explicit var if dtroyer is cool with it18:57
clarkbsdague: ZUUL_REF isn't in the periodic jobs but ZUUL_PROJECT and ZUUL_UUID are18:57
jog0sdague: why not move the git reset --hard to unstack18:57
dtroyerjog0: to keep the source trees unmolested, unstack isn't always needed when mucking around in dev mode.18:59
jog0dtroyer: does it hurt to unstack?18:59
sdaguejog0: it kills the world18:59
jog0sdague: so?18:59
dtroyersdague: if we have something to reference that would be my preference, but some sort of specific setting would be ok.19:00
jog0also if we keep the deps up to date with global then this won'19:00
jog0t happen in the first place19:00
dtroyerjog0:  the workflow is to not need to re-stack when working on just one service…that's also why we use screen19:00
openstackgerritSean Dague proposed a change to openstack-dev/devstack: create an UNDO_REQUIREMENTS variable  https://review.openstack.org/5371219:00
sdaguedtroyer: how about that?19:01
*** hemna has quit IRC19:01
jog0dtroyer: right, but if reqs are synced up in repos properly this will almost never happen19:01
openstackgerritJeremy Stanley proposed a change to openstack-infra/elastic-recheck: Correct conditional branching in field search  https://review.openstack.org/5371319:01
jog0and would be incentive for devs to fix that19:01
sdaguejog0: yeh, but the whole point of g-r is that projects didn't have to keep in sync manually19:01
sdaguebecause that's how we wedged things before19:01
sdaguewe know it doesn't work19:01
sdagueand more importantly, it causes some very weird results19:02
jog0sdague: well now we have a bot that does that19:02
jog0so its not all manaual19:02
sdagueyou still need approvers19:02
sdaguehonestly, we actually have a system that works, except for the fact that we let rollback into devstack in the base case19:03
dtroyersdague: jog0: in theory you're right.  in practice...19:03
sdagueas soon as that's addressed, I think we're good19:03
jog0dtroyer  sdague : yeah19:04
sdaguejog0: I want to live in theory, everything works in theory! :)19:04
jog0as long as we make sure to sync global reqs before a release so downstream fols don't get confused19:04
jog0folks*19:04
sdaguejog0: sure19:04
jog0sdague: :)19:04
dtroyersdague: I can live with that…maybe a comment on what it is intended for, "normally only used by the gate tests" or something19:06
openstackgerritThomas Maddox proposed a change to openstack-dev/devstack: Add PostgreSQL option for Ceilometer backend  https://review.openstack.org/5371519:06
sdaguedtroyer: sure, coming right up19:06
*** hemna has joined #openstack-qa19:08
jog0sdague: just ran http://paste.openstack.org/show/49552/19:08
jog0elastic-recheck  check success19:09
*** hemna has quit IRC19:09
openstackgerritSean Dague proposed a change to openstack-dev/devstack: create an UNDO_REQUIREMENTS variable  https://review.openstack.org/5371219:09
sdaguedtroyer: I went heavy on the prose, but it should be super clear what we're doing :)19:10
sdaguejog0: cool19:10
jog0some of those failures are getting up there19:10
sdaguejog0: so one of the things I was having issues with was check-success result stability19:11
sdaguewhich was a reason for the refactor on the resultset work, to make sure I wasn't doing something else crazy19:11
sdagueI need to loop around to that19:11
dtroyersdague: +2, thanks19:11
jog0sdague: interesting running a few times locally to see19:12
*** hemna has joined #openstack-qa19:13
jog0sdague: 2 back to back runs produced the same results so thats good19:15
jog0anyway i'm out19:15
*** andreaf has quit IRC19:25
openstackgerritSam Alba proposed a change to openstack-dev/devstack: Removed docker version requirement  https://review.openstack.org/5303119:26
openstackgerritSam Alba proposed a change to openstack-dev/devstack: Removed docker version requirement  https://review.openstack.org/5303119:27
*** SergeyLukjanov is now known as _SergeyLukjanov19:28
*** _SergeyLukjanov has quit IRC19:28
openstackgerritA change was merged to openstack-infra/elastic-recheck: Correct conditional branching in field search  https://review.openstack.org/5371319:30
*** markmcclain has joined #openstack-qa19:30
*** openstackrecheck has quit IRC19:30
clarkbfungi: ^ how did local testing go?19:30
*** openstackrecheck has joined #openstack-qa19:31
mtreinishclarkb: was I not supposed to merge that?19:37
clarkbmtreinish: no its fine19:37
clarkbfungi mentioned he was going to monkey patch it in to test, but then I never heard back. Looking at the generated graphs it seems to be working though19:37
openstackrecheckopenstack/tempest change: https://review.openstack.org/52413 failed tempest with an unrecognized error19:43
*** SergeyLukjanov has joined #openstack-qa19:48
*** yfried|mob has quit IRC19:52
*** yfried|mob has joined #openstack-qa19:53
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Sync config file generator from oslo  https://review.openstack.org/5372419:53
openstackgerritA change was merged to openstack/tempest: Update to latest pbr  https://review.openstack.org/5278519:54
openstackgerritA change was merged to openstack/tempest: Updated from global requirements  https://review.openstack.org/5104119:56
fungiclarkb: mtreinish: seems to be working, and no new tracebacks in the mailspool for it19:59
fungii afk'd while i waited for its first run to complete19:59
clarkbfungi: did the repo on static update properly after the change merged?20:00
*** fbo is now known as fbo_away20:01
fungiclarkb: yep20:01
mtreinishfungi: ok good, I didn't know you were doing manual testing. It made sense to me when I reviewed it so I hit +A.20:01
fungimtreinish: yeah, all i did was prematurely hand-patch that trivial change into production to make sure there weren't other errors hiding behind that one20:02
fungiso we could get a head start on them if so20:02
clarkbaccording to the graphs bug 1217734 happens on a lot of sccessful tests which is a bit scary20:05
mtreinishclarkb: the query looks for: "CalledProcessError: Command 'openssl' returned non-zero exit status" I don't think that should be on a succesful run20:06
clarkbI agree20:07
mtreinishoh but it's across all files20:07
clarkboh20:07
clarkbdo we need to restrict it more?20:07
sdagueso if something happens in a success run, it's a bad query20:07
sdagueand needs to be trimmed20:08
sdaguenot by file, but by content20:08
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Sync fixtures from oslo and use LockFixture  https://review.openstack.org/5372520:09
mtreinishsdague: I think by file is fine. It could be part of normal operation in one service but not another.20:09
sdaguemtreinish: I think it's sort of dubious to assume that :)20:10
mtreinishsdague: probably so, but the query is really open ended: https://git.openstack.org/cgit/openstack-infra/elastic-recheck/tree/queries.yaml#n1320:10
mtreinishnot log level for the message, no file, just the string20:11
clarkbwow no kidding20:11
sdagueyep, agreed.20:11
openstackrecheckopenstack/nova change: https://review.openstack.org/53595 failed tempest with an unrecognized error20:11
openstackrecheckopenstack/nova change: https://review.openstack.org/53597 failed tempest with an unrecognized error20:12
openstackrecheckopenstack/nova change: https://review.openstack.org/51396 failed tempest because of: https://bugs.launchpad.net/bugs/124425520:13
sdagueyeh, it looks like keystoneclient poops itself on invalid data, and spews the stacktrace20:14
sdaguehttp://logs.openstack.org/29/52929/4/check/check-tempest-devstack-vm-full/127e089/logs/screen-n-api.txt.gz?level=TRACE20:14
sdagueeven when it's being done intentionally20:14
openstackrecheckopenstack/nova change: https://review.openstack.org/53596 failed tempest with an unrecognized error20:16
openstackrecheckopenstack/nova change: https://review.openstack.org/53603 failed tempest with an unrecognized error20:17
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Sync latest module versions from oslo-incubator  https://review.openstack.org/5372920:18
openstackrecheckopenstack/nova change: https://review.openstack.org/52286 failed tempest with an unrecognized error20:30
openstackrecheckopenstack/neutron change: https://review.openstack.org/53255 failed tempest with an unrecognized error20:32
openstackrecheckopenstack/neutron change: https://review.openstack.org/53244 failed tempest because of: https://bugs.launchpad.net/bugs/123963720:32
openstackrecheckopenstack/cinder change: https://review.openstack.org/53723 failed tempest with an unrecognized error20:37
*** yfried|mob has quit IRC20:50
*** yfried|mob has joined #openstack-qa20:50
openstackrecheckopenstack/requirements change: https://review.openstack.org/52137 failed tempest because of: https://bugs.launchpad.net/bugs/122566420:51
*** DennyZhang has joined #openstack-qa20:52
*** DennyZhang has quit IRC20:53
*** DennyZhang has joined #openstack-qa20:55
*** yfried has quit IRC20:57
openstackgerritA change was merged to openstack-dev/devstack: Start nova-compute with child cell conf  https://review.openstack.org/5346821:03
*** DennyZhang has quit IRC21:04
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: don't UNDO_REQUIREMENTS after we update from g-r  https://review.openstack.org/5373921:06
openstackgerritAaron Rosen proposed a change to openstack-dev/devstack: All neutron plugins should now use LibvirtGenericVIFDriver  https://review.openstack.org/5374021:08
openstackrecheckopenstack/nova change: https://review.openstack.org/51396 failed tempest because of: https://bugs.launchpad.net/bugs/123985621:09
openstackrecheckopenstack/tempest change: https://review.openstack.org/52413 failed tempest because of: https://bugs.launchpad.net/bugs/123548621:13
*** yfried has joined #openstack-qa21:18
openstackrecheckopenstack/horizon change: https://review.openstack.org/53728 failed tempest because of: https://bugs.launchpad.net/bugs/123985621:20
*** DennyZhang has joined #openstack-qa21:25
*** dkranz has quit IRC21:31
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/53712 failed tempest with an unrecognized error21:33
*** mriedem has quit IRC21:36
*** yfried|mob has quit IRC21:36
openstackrecheckopenstack/keystone change: https://review.openstack.org/53741 failed tempest with an unrecognized error21:43
*** mlavalle has joined #openstack-qa21:47
*** mlavalle has left #openstack-qa21:48
openstackgerritMate Lakat proposed a change to openstack/tempest: Fix bad classname  https://review.openstack.org/5374521:50
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/53740 failed tempest because of: https://bugs.launchpad.net/bugs/124025621:51
*** SergeyLukjanov is now known as _SergeyLukjanov21:51
*** _SergeyLukjanov has quit IRC21:51
*** SergeyLukjanov has joined #openstack-qa21:53
*** adalbas has quit IRC21:56
*** jaypipes has quit IRC21:56
openstackgerritMate Lakat proposed a change to openstack/tempest: Fix bad classname  https://review.openstack.org/5374521:57
*** DennyZhang has quit IRC22:12
*** morganfainberg is now known as _ttx22:22
openstackrecheckopenstack/neutron change: https://review.openstack.org/53244 failed tempest because of: https://bugs.launchpad.net/bugs/123548622:23
*** _ttx is now known as morganfainberg22:23
*** dkranz has joined #openstack-qa22:26
openstackrecheckopenstack/tempest change: https://review.openstack.org/52413 failed tempest because of: https://bugs.launchpad.net/bugs/1235437 and https://bugs.launchpad.net/bugs/124025622:28
*** dims has quit IRC22:29
openstackgerritPeter Feiner proposed a change to openstack-dev/devstack: Fix apache horizon permissions on Ubuntu 13.10  https://review.openstack.org/5375122:38
openstackrecheckopenstack/tempest change: https://review.openstack.org/53745 failed tempest because of: https://bugs.launchpad.net/bugs/123548622:40
openstackgerritA change was merged to openstack-dev/devstack: create an UNDO_REQUIREMENTS variable  https://review.openstack.org/5371222:41
clarkbsdague: I have rechecked https://review.openstack.org/#/c/53739/ so that it is run with ^ merged into devstack22:44
*** dims has joined #openstack-qa22:44
*** jhenner has quit IRC22:50
openstackgerritPeter Feiner proposed a change to openstack-dev/devstack: zsh openrc compatibility  https://review.openstack.org/5375522:53
*** SergeyLukjanov has quit IRC22:58
*** mkollaro has quit IRC23:23
*** hemna is now known as hemnafk23:26
*** yfried_mob has joined #openstack-qa23:30
*** sthaha has joined #openstack-qa23:42
*** sthaha has joined #openstack-qa23:42

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