Monday, 2013-08-19

« Sunday, 2013-08-18 Index Tuesday, 2013-08-20 »
*** LiangZheng has joined #openstack-qa00:00
*** LiangZheng has quit IRC00:05
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create security rule" methods  https://review.openstack.org/4230300:09
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create security rule" method  https://review.openstack.org/4230300:31
*** LiangZheng has joined #openstack-qa01:25
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create security rule" method  https://review.openstack.org/4230301:26
*** LiangZheng has quit IRC01:29
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create security rule" method  https://review.openstack.org/4230301:33
*** LiangZheng has joined #openstack-qa02:15
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create keypair" method  https://review.openstack.org/4255502:36
*** mriedem has quit IRC03:00
openstackgerritZhi Kun Liu proposed a change to openstack/tempest: test_servers_whitebox - using official client  https://review.openstack.org/4039603:01
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create keypair" method  https://review.openstack.org/4255503:21
openstackgerritJoseph Wang proposed a change to openstack-dev/devstack: add mageia support  https://review.openstack.org/4256103:36
openstackgerritJoseph Wang proposed a change to openstack-dev/devstack: turn out usb_tablet for libvirt  https://review.openstack.org/4256204:04
*** mkoderer has quit IRC04:19
*** Anju has joined #openstack-qa04:21
*** mberwanger has joined #openstack-qa04:41
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create keypair" method  https://review.openstack.org/4255504:44
*** mberwanger has quit IRC05:10
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "ssh-login server" method  https://review.openstack.org/4256905:11
*** thomasbiege1 has joined #openstack-qa05:15
*** thomasbiege1 has quit IRC05:17
*** thomasbiege has joined #openstack-qa05:21
*** thomasbiege has quit IRC05:24
*** SergeyLukjanov has joined #openstack-qa05:41
*** kashyap has left #openstack-qa05:45
*** kashyap has joined #openstack-qa05:45
*** thomasbiege1 has joined #openstack-qa06:06
*** thomasbiege1 has quit IRC06:08
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "ssh-login server" method  https://review.openstack.org/4256906:11
*** mkoderer has joined #openstack-qa06:21
*** SergeyLukjanov has quit IRC06:25
*** SergeyLukjanov has joined #openstack-qa06:30
*** sthaha has quit IRC06:35
*** SergeyLukjanov has quit IRC06:38
*** SergeyLukjanov has joined #openstack-qa06:42
*** SergeyLukjanov has quit IRC06:48
openstackgerritZhang Hua proposed a change to openstack/tempest: Fixes the issues of tempest due to async network allocation  https://review.openstack.org/4247207:04
*** Anju has quit IRC07:05
*** fbo_away is now known as fbo07:16
*** yfried has joined #openstack-qa07:21
*** nsaje1 has joined #openstack-qa07:23
*** thomasbiege1 has joined #openstack-qa07:42
*** thomasbiege1 has quit IRC07:43
*** yfried has quit IRC07:54
*** yfried has joined #openstack-qa07:56
*** jhenner has joined #openstack-qa07:59
*** tkammer has joined #openstack-qa08:00
*** yfried has quit IRC08:01
openstackgerritIvan-Zhu proposed a change to openstack-dev/grenade: add nova v3 api support in upgrade  https://review.openstack.org/4013408:11
*** sthaha has joined #openstack-qa08:13
*** sthaha has joined #openstack-qa08:13
*** giulivo has joined #openstack-qa08:15
*** yfried has joined #openstack-qa08:15
openstackgerritIvan-Zhu proposed a change to openstack-dev/grenade: add nova v3 api support in upgrade  https://review.openstack.org/4013408:15
*** VinodGupta has joined #openstack-qa08:17
openstackgerritIvan-Zhu proposed a change to openstack-dev/grenade: add nova v3 api support in upgrade  https://review.openstack.org/4013408:19
*** psedlak has joined #openstack-qa08:21
*** tkammer has quit IRC08:55
*** yfried has quit IRC08:57
*** yfried has joined #openstack-qa08:58
*** LiangZheng has quit IRC09:06
*** LiangZheng has joined #openstack-qa09:06
*** Guest has quit IRC09:19
*** tkammer has joined #openstack-qa09:25
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create volume" method  https://review.openstack.org/4259309:32
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create volume" method  https://review.openstack.org/4259309:42
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "ssh-login server" method  https://review.openstack.org/4256909:48
openstackgerritnayna-patel proposed a change to openstack/tempest: Added test case to check floating IP API operations  https://review.openstack.org/4103809:49
*** Anju has joined #openstack-qa09:50
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create volume" method  https://review.openstack.org/4259309:51
*** tkammer has quit IRC09:58
*** sthaha has quit IRC10:01
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create keypair" method  https://review.openstack.org/4255510:04
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create security rule" method  https://review.openstack.org/4230310:09
*** weshay has joined #openstack-qa10:14
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create security rule" method  https://review.openstack.org/4230310:19
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create keypair" method  https://review.openstack.org/4255510:21
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create volume" method  https://review.openstack.org/4259310:22
*** fbo is now known as fbo_away10:34
*** Anju has quit IRC10:45
*** Anju has joined #openstack-qa10:49
*** raies has joined #openstack-qa11:03
*** psedlak_ has joined #openstack-qa11:03
raiesafazekas: what is request body for bulk network creation with xml formatted body request11:04
raies?11:04
afazekasraies: I am still not OK :(, try to capture the traffic of the cli11:06
raiesi am trying cli but still there is problem11:06
raieswhen I tried "<networks> <name>sample_network_1</name> <admin_state_up>false</admin_state_up> </networks> <networks> <name>sample_network_2</name> <admin_state_up>false</admin_state_up> </networks>"11:07
raiesin request body then bad request11:07
*** psedlak has quit IRC11:07
raiesdisplayed11:07
raieswhen I tried "<networks><name>sample_network_1</name><name>sample_network_2</name></networks>"11:08
afazekasWild guess you have multiple <network> in the <networks>11:08
raiesthen server error 500 displayed11:08
afazekasthe 500 error should be reported11:08
raiesbut I am unable to find exact request body for xml request during bulk network creation11:10
afazekas<networks><network>   <name>sample_network_1</name>   <admin_state_up>false</admin_state_up></network>  </networks>   # just wild guess without a client11:13
raiesok11:13
raiesi will try it11:13
raiesafazekas :  I tried this with success   :)   yep    "<networks><network><name>sample_network_1</name><name>sample_network_2</name></network></networks>"11:15
afazekasif it working with multiple network it was good guess :)11:18
* afazekas go back to rest11:18
openstackgerritnayna-patel proposed a change to openstack/tempest: Added test case to check floating IP API operations  https://review.openstack.org/4103811:20
*** yfried has quit IRC11:21
openstackgerritafazekas proposed a change to openstack/tempest: Move the network api tests to smoke  https://review.openstack.org/4251711:36
*** fbo_away is now known as fbo11:37
*** thomasbiege1 has joined #openstack-qa11:40
*** yfried has joined #openstack-qa11:42
openstackgerritAditi Raveesh proposed a change to openstack/tempest: Fixing format_flavor to handle flavor extra_specs  https://review.openstack.org/4261611:42
*** weshay has quit IRC11:44
*** Anju has quit IRC11:44
*** thomasbiege1 has quit IRC11:44
*** thomasbiege1 has joined #openstack-qa11:45
*** weshay has joined #openstack-qa11:50
*** VinodGupta has quit IRC11:53
*** nsaje1 has quit IRC11:58
*** torandu has joined #openstack-qa11:58
*** weshay has quit IRC11:59
openstackgerritraiesmh08 proposed a change to openstack/tempest: Adding network api xml support  https://review.openstack.org/4167312:02
raiesafazekas: r u there ?12:03
*** torandu has quit IRC12:07
*** mkollaro has joined #openstack-qa12:07
openstackgerritKui Shi proposed a change to openstack-dev/devstack: libffi-dev is needed while install python package  https://review.openstack.org/4262012:09
*** psedlak__ has joined #openstack-qa12:10
*** yfried has quit IRC12:12
*** psedlak_ has quit IRC12:14
openstackgerritnayna-patel proposed a change to openstack/tempest: Adds verfication for Bootable Volume  https://review.openstack.org/4262112:16
*** woodspa has joined #openstack-qa12:21
*** mriedem has joined #openstack-qa12:36
*** thomasbiege1 has quit IRC12:42
*** jasondotstar has joined #openstack-qa12:47
*** thomasbiege1 has joined #openstack-qa12:51
*** thomasbiege1 has quit IRC12:52
*** anteaya has joined #openstack-qa12:53
*** adalbas has joined #openstack-qa12:55
*** tkammer has joined #openstack-qa13:03
*** LiangZheng has quit IRC13:11
*** xqueralt has joined #openstack-qa13:15
*** sthaha has joined #openstack-qa13:15
xqueralthey, I think tempest tests might be broken for grizzly13:15
xqueraltcould somebody take a look at: https://bugs.launchpad.net/tempest/+bug/1213912 ?13:15
openstackgerritJoe Hakim Rahme proposed a change to openstack/tempest: Add more tests for Swift Account Quota  https://review.openstack.org/4109913:16
xqueraltI've tried to debug it a bit more more but I don't know where the problem might come from13:16
*** timello_ has joined #openstack-qa13:20
*** stevebaker has quit IRC13:22
*** timello has quit IRC13:22
*** stevebaker has joined #openstack-qa13:23
*** burt has joined #openstack-qa13:26
openstackgerritJulien Danjou proposed a change to openstack-dev/devstack: Use 10gen repository to get MongoDB 2.4  https://review.openstack.org/4264213:28
*** psedlak has joined #openstack-qa13:36
*** psedlak__ has quit IRC13:37
*** LiangZheng has joined #openstack-qa13:41
*** LiangZheng has quit IRC13:49
*** sthaha has quit IRC13:57
*** psedlak_ has joined #openstack-qa14:05
*** psedlak has quit IRC14:09
*** annegentle has joined #openstack-qa14:10
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add tenant isolation to the swift tests  https://review.openstack.org/4216914:13
*** rustlebee is now known as russellb14:13
*** tkammer has quit IRC14:16
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Fix skip tracker regex for multi-line skips  https://review.openstack.org/4242414:16
openstackgerritJulien Danjou proposed a change to openstack-dev/devstack: Remove useless sources.list  https://review.openstack.org/4265214:17
*** tkammer has joined #openstack-qa14:18
*** tkammer has quit IRC14:23
*** xqueralt is now known as xqueralt-afk14:28
*** xqueralt-afk is now known as xqueralt14:28
*** malini has joined #openstack-qa14:31
*** fbo is now known as fbo_away14:33
*** dkranz has joined #openstack-qa14:36
*** malini is now known as malini_afk14:37
*** malini_afk is now known as malini14:37
*** fbo_away is now known as fbo14:40
*** mlavalle has joined #openstack-qa14:44
openstackgerritafazekas proposed a change to openstack/tempest: task_state must be consider before many action  https://review.openstack.org/3516514:45
*** LiangZheng has joined #openstack-qa14:46
*** LiangZheng has quit IRC14:50
openstackgerritafazekas proposed a change to openstack/tempest: Simplify xml/json client selection  https://review.openstack.org/3693715:00
*** openstackgerrit has quit IRC15:01
*** openstackgerrit has joined #openstack-qa15:02
openstackgerritJulien Danjou proposed a change to openstack-dev/devstack: Use 10gen repository to get MongoDB 2.4  https://review.openstack.org/4264215:06
openstackgerritraiesmh08 proposed a change to openstack/tempest: Adding network api xml support  https://review.openstack.org/4167315:07
*** psedlak_ has quit IRC15:11
openstackgerritafazekas proposed a change to openstack/tempest: Dump basic network info  in the test_network_basic_ops  https://review.openstack.org/4043515:12
*** raies_ has joined #openstack-qa15:12
raies_afazeks: hi15:13
raies_afazekas: hi15:13
openstackgerritafazekas proposed a change to openstack/tempest: Dump basic network info  in the test_network_basic_ops  https://review.openstack.org/4043515:16
*** thomasbiege1 has joined #openstack-qa15:16
*** SergeyLukjanov has joined #openstack-qa15:19
raies_afazekas: please check https://review.openstack.org/#/c/41673/ and https://review.openstack.org/#/c/42624/15:21
afazekashttps://review.openstack.org/#/c/41673/5/tempest/api/network/test_networks.py  my first inline note not addressed  (you removed a space without a reason)15:24
raies_oh silly mistake15:25
raies_i will amend15:26
afazekaswhy did you changed the test case ?15:26
raies_which one ?15:26
afazekashttps://review.openstack.org/#/c/41673/7/tempest/api/network/test_networks.py15:26
afazekastest_list_networkks .. all of them15:27
*** rhsu has joined #openstack-qa15:27
*** rhsu has quit IRC15:27
raies_to handle JSON as well s XML i used this technique15:27
raies_ultimately our motive ws to check self.id15:28
raies_so i checked server list methods15:28
raies_and implemented here lso15:28
raies_*also15:28
raies_Thus using help of server list methods I was able to handle xml as well as json response bodies :)15:30
afazekasAt first look the new version looks better,  but I am not sure it should be in the same change..15:32
raies_i didn't understand please elaborate ?15:33
raies_I checked, everything is working good15:33
raies_and your comments of keyword arguments also handled15:34
raies_and list cases are handled in an alternative and in good way15:34
afazekasusually we do not change/add test cases at the same time when we add xml support15:37
afazekasmtreinish: https://review.openstack.org/#/c/41673/7 can  we review this change once ?15:38
raies_Actually when I checked the return value in case of json i was "{{"ports": [      {"status": "DOWN",        "name": "",         "admin_state_up": true,         "network_id": "52a5bed2-3d7e-488d-8cca-b8a9a7883ef7",         "tenant_id": "40b7ab229c3b47dba7991e857e082216",         "device_owner": "network:dhcp",         "mac_address": "fa:16:3e:ba:fd:bf",         "fixed_ips": [            {"subnet_id": "34ddc9b5-4152-4d43-9a59-468ffd15:39
openstackgerritZhang Hua proposed a change to openstack/tempest: Fixes the issues of tempest due to async network allocation  https://review.openstack.org/4247215:39
raies_and when I checked the return response of xml the it was -- "{'ports':      {'port':         {'status': 'DOWN',          'name': 'port-1577847275',          'admin_state_up': 'True',          'network_id': '544147e2-c454-4434-98d2-0057b45f979f',          'tenant_id': '40b7ab229c3b47dba7991e857e082216',          'device_owner': {},          'mac_address': 'fa:16:3e:00:21:f4',          'fixed_ips':               {'fixed_ip':15:40
raies_2nd is just an example15:40
raies_So in case when I changed xml response in list15:40
raies_It was unable to parse as per previous test trend15:41
raies_so I have to made some changes15:41
afazekashttps://wiki.openstack.org/wiki/GitCommitMessages#Structural_split_of_changes15:41
raies_afazekas: Is the wiki page is suggested for me ?15:42
mtreinishafazekas: yeah I was going to leave a -1 comment about that. It probably should be split up per test file. Also, any xml client code written needs tests that use it.15:43
afazekaslooks there was some anomaly with the json client15:44
*** tkammer has joined #openstack-qa15:45
*** LiangZheng has joined #openstack-qa15:47
*** xqueralt has left #openstack-qa15:47
raies_afazekas:15:48
*** LiangZheng has quit IRC15:51
afazekasmtreinish: the change does not looks too big, do you think do we need an additional tempest specific change grouping guideline ?15:52
mtreinishafazekas: actually I was just giving it a first glance when I made the above remark I'm not sure that it really applie. My bigger issue is with all the changes in one of the tests files. That should be split out (and made a dependency if it is one)15:54
*** SergeyLu_ has joined #openstack-qa15:54
*** SergeyLukjanov has quit IRC15:57
afazekasmtreinish: Probably the second patch would be under 50 line, close to the limit where not really worth to split16:00
mtreinishafazekas: I disagree, its not just about making it a small enough patch to be easy to review. But also about making a logical split in patches to make bisecting easier and the logs descriptive.16:01
*** Anju has joined #openstack-qa16:03
*** thomasbiege1 has quit IRC16:06
*** tkammer has quit IRC16:07
raies_afazekas: https://review.openstack.org/#/c/41673/7/tempest/api/network/test_networks.py line 176 I did not understand ?16:12
afazekasraies_: As remember last time I recommended to add xml support in way when the test cases are not modified16:12
raies_afazekas: does it will affect our process ?16:13
afazekasVery frequently: - the style changing, renaming commits are individual16:13
afazekasraies_: can you submit the patch in way, when just the things what really necessary for the xml support included ?16:14
*** SergeyLu_ has quit IRC16:14
raies_afazekas: mtrennish has made several comments , out of which 2 comments are still in confusion16:15
raies_1. what i told you16:15
raies_2. https://review.openstack.org/#/c/41673/7/tempest/api/network/base.py line 5216:16
afazekas2. I think it can be added together with the xml16:17
afazekasmtreinish: do you agree ?16:17
afazekasIt was required for providing better common interface, based on a previous IRC discussion16:17
raies_sorry i gave wrong16:18
raies_https://review.openstack.org/#/c/41673/7/tempest/services/network/json/network_client.py line 11816:18
raies_2 . as mentioned just above16:18
afazekasstrange I read the correct one :)16:20
raies_afazekas: case 1. As xml and json responses are are quite different to handle in same test case trend, I have to choose redundant way16:21
afazekashttps://review.openstack.org/#/c/41673/7/tempest/api/network/base.py  line 51 should be deleted16:21
raies_afazekas: When my amendment is giving better for case 1 is giving good and a step more validation and also valid for xml as well as json then why there re being rejected ?16:23
raies_afazekas: last comment what u written should be for https://review.openstack.org/#/c/41673/7/tempest/services/network/json/network_client.py line 118 I think16:24
afazekasraies_: Normally a change  does does on "type" of change16:25
afazekasnow the patch grow to big, and does 3 type of change16:26
afazekashttps://review.openstack.org/#/c/42472/ probably this change also will be asked for separation16:27
afazekasthe skips are coming in a single change16:28
raies_afazekas: as per previous test trend xml does not work for list cases as I have told you before16:28
raies_then what could be the solution again16:28
raies_afazekas: r u there ?16:31
afazekasraies_: is the code you submitted recently working with XML correctly?16:33
* afazekas is not OK now, I try to look it closer tomorrow 16:34
afazekasmtreinish: when you returned can you discuss with raies_, how to split ?16:41
*** marun has joined #openstack-qa16:46
*** LiangZheng has joined #openstack-qa16:47
raies_afazekas: I m in now :)16:48
raies_Yes my code was working good for xml s well as json16:48
raies_all 24 testcases (xml+json) was pass OK16:48
*** afazekas_ has joined #openstack-qa16:51
afazekasraies_: I guess it would be able to work with unmodified test cases16:52
*** LiangZheng has quit IRC16:52
raies_No it is working for all I had checked specially for modified one16:52
*** fbo is now known as fbo_away16:53
afazekasInteresting, the json client return 'structure' looks unchanged just based on the code,  and I do not see condition in the test cases16:56
raies_afazekas: will you please run after applying this patch ?16:58
afazekasTomorrow I will17:00
raies_afazekas :17:00
raies_If jenkins will give  +1 , then it will be verified I think ?17:00
afazekasIf you say it does not working with the original one, something else can be wrong,  I want to investigate it17:02
raies_afazekas: Yes please :) and give your inputs to improve it soon as so many times it had been discaded :(17:03
mtreinishafazekas: when you get a chance can I get some eyes on: https://review.openstack.org/#/c/42169/17:07
mtreinishit's a fix for one of the common failures on the parallel runs17:07
*** jerryz has joined #openstack-qa17:07
afazekasmtreinish: the  tempest.conf.sample should mention the alt user should have operator rule, when the isolation is turned off17:11
afazekashttps://review.openstack.org/#/c/42169/3/etc/tempest.conf.sample L42 and L3417:12
mtreinishafazekas: if isolation is turned off I think its a safe assumption the users provided would have the operator role configured if they're being used for swift tests.17:13
mtreinishit'd be weird to try to run the swift tests with users that didn't have the right role17:14
afazekasmtreinish: yes, bit it should be documented somewhere, now the config sample is one possible place17:14
afazekasmtreinish: operator role is higher role in swift, the several old tests worked without operator role AFAIK17:15
mtreinishafazekas: well the devstack case is misleading because all the users it creates get an operator role.17:15
mtreinishwhen I tested it without the role set I think all the tests failed17:15
afazekasI really go to rest otherwise I will be sick tomorrow as well...17:16
mtreinishafazekas: ok go, don't work yourself too hard...17:16
afazekasoperator_roles "Member, admin"17:17
afazekasso 'all' regular devstack role considered as operator17:17
*** burt has quit IRC17:18
mtreinishafazekas: yeah that's what I saw17:18
mtreinishbut making users with the tempest client doesn't necessarily have that set17:18
afazekasLooks like I changed that time last :)17:18
afazekasit was refactoring ..17:19
afazekasOk, we should not forget to mention this operator role thing when document what is the expected system config for tempest17:20
mtreinishafazekas: ok, I'm just not sure that sample conf around the users is the right place for it. But I'm not sure where a good place would be.17:22
afazekashttps://review.openstack.org/#/c/42169/3/tempest/api/object_storage/base.py strange next usage17:24
afazekasfrequently we just define the id in the config17:25
afazekasthe role lookup should be outside of the loop17:25
afazekasIt only needs to happen once in timpest lifetyme or once in a process lifetime17:26
*** mlavalle has quit IRC17:26
mtreinishafazekas: that was just lifted from what I do for adding the admin role in tenant isolation. But, I agree I'll move it outside of the loop17:26
afazekassimpler to use a list expression with [0]17:27
*** afazekas is now known as afazekas_zz17:28
*** yfried has joined #openstack-qa17:34
*** raies_ has quit IRC17:37
*** Anju has quit IRC17:37
*** thomasbiege1 has joined #openstack-qa17:38
*** mlavalle has joined #openstack-qa17:44
*** thomasbiege2 has joined #openstack-qa17:46
*** LiangZheng has joined #openstack-qa17:48
openstackgerritJames E. Blair proposed a change to openstack-infra/devstack-gate: Use git.openstack.org as origin  https://review.openstack.org/4269317:49
*** thomasbiege1 has quit IRC17:50
*** afazekas_ has quit IRC17:51
*** LiangZheng has quit IRC17:53
*** thomasbiege2 has quit IRC17:56
*** thomasbiege has joined #openstack-qa18:04
*** giulivo has quit IRC18:06
*** thomasbiege has quit IRC18:15
*** thomasbiege has joined #openstack-qa18:23
*** thomasbiege has quit IRC18:24
*** SergeyLukjanov has joined #openstack-qa18:28
*** burt has joined #openstack-qa18:37
openstackgerritMaru Newby proposed a change to openstack/tempest: Remove skip of neutron connectivity check  https://review.openstack.org/4201818:46
*** LiangZheng has joined #openstack-qa18:48
*** LiangZheng has quit IRC18:53
*** mriedem has quit IRC18:55
*** mriedem has joined #openstack-qa18:56
*** fbo_away is now known as fbo19:27
*** xqueralt has joined #openstack-qa19:31
*** xqueralt is now known as xqueralt-afk19:37
*** malini is now known as malini_afk19:41
*** LiangZheng has joined #openstack-qa19:49
*** LiangZheng has quit IRC19:54
*** thomasbiege has joined #openstack-qa20:04
marunanyone have eyes on the glance cli failure I'm seeing at the end of the grenade gate?20:24
marunhttp://logs.openstack.org/18/42018/5/check/gate-grenade-devstack-vm/e5876b2/console.html20:25
*** harlowja_ has quit IRC20:31
*** harlowja has joined #openstack-qa20:31
dkranzmarun: This is a bug recently introduced, discussed in #openstack-infra in the last hour or two. A fix is on the way.20:37
mlavallemriedem: ping20:37
dkranzmarun: https://review.openstack.org/#/c/42720/20:38
mriedemmlavalle: pong20:38
marundkranz: thank you20:38
marundranz: while I have your attention: https://review.openstack.org/#/c/41202/20:38
mlavallemriedem: so, in regards to https://bugs.launchpad.net/tempest/+bug/1182384, what is the way forward? Are we going to implement uuid's for sec groups in nova?20:38
marundkranz: would like to get this merged and then pull it into the grizzly branch in for the rdo github repo20:39
dkranzmarun: I'll take a look.20:39
mriedemmlavalle: my notes on where that was at was in my tempest review to skip the tests: https://review.openstack.org/#/c/40015/20:40
mriedemi think the mailing list post is where the ideas were20:40
mriedemmlavalle: when i talked with sdague about it, he pointed out another nova patch to change how the security group api validation would be done in nova...trying to find that20:42
dkranzmarun: What does "Conflicts:" mean?20:42
mriedemdkranz: cherry pick merge conflicts20:42
mriedemadded by git20:42
marundkranz: it's output from the git cherry-pick20:42
dkranzmarun: Ah, OK.20:43
mriedemmarun: you cherry picked that from master right?20:43
marunmriedem: correct20:43
mriedemthe change-id doesn't match what's in master20:43
mriedemit should20:43
mlavallemriedem: in the mailing list, it seems the last to reply was Vish, back in July 10th: http://lists.openstack.org/pipermail/openstack-dev/2013-July/011666.html20:43
marunreally?20:43
mriedemyou need to use the -x option when you cherry pick20:43
marunmreidem: can you please point me at the relevant documentation?20:44
mriedemmarun: looking20:44
mlavallemriedem: are you aware of any further discussions / progress?20:44
marunmreidem: I'm not sure I understand why the change ids would match.20:44
mriedemmarun: i think the wiki might have a gap in not tellin you to use -x20:44
* mriedem getting blown up by qa :)20:44
mriedemmarun: https://wiki.openstack.org/wiki/StableBranch#Proposing_Fixes20:45
mriedemmarun: you should use the -x option so the change id is the same between reviews, it works like a topic branch20:45
mriedemin gerrit, you click the change id and it shows up the reviews on each branch20:45
marunmreidem: Hmmm, so the change id is 1 to many?20:46
mriedemmarun: yeah20:46
mriedemtrying to find an example20:46
mriedemmarun: example: https://review.openstack.org/#/q/I6c08b1b8040a1fd0db8e2b3b1fc798060733001f,n,z20:47
marunmriedem: no, that's fine.  i wasn't clear that it was safe to reuse a change id across branches.20:47
mriedemmarun: preferred even :)20:47
marunI haven't done any backports up till now.20:47
mriedemmlavalle: ok, back to your question, i'm not aware of further discussion on the mailing list, i had talked with sdague about it in -nova awhile back when i was looking at fixing it,20:47
mriedemmlavalle: and he said we should do the validation in nova but in a different way for nova-network vs neutron APIs, i'm trying to find that now20:48
marunmriedem: does that imply that including the cherry-picked rev hash in a commit is unnecessary?20:49
mriedemmlavalle: because i think the issue is the tempest nova tests pass id to nova and it pukes in the neutron API because it's looking for a uuid20:49
mriedemmarun: that should say in there20:49
marunmriedem: ok20:49
mlavallemriedem: Yep, that's the problem...20:49
*** LiangZheng has joined #openstack-qa20:50
mlavallemriedem: I want to move this forward. Any advice as to what the next step might be?20:50
marunmriedem: it looks like it created a new review.  Can I abandon the old one?20:50
mriedemmarun: yeah, i think you have to since the change id would change20:52
mriedemmlavalle: i'd say we should talk to sdague but that lazy bum us on vacation :)20:52
mriedemfor another week or two i think20:52
mriedemmlavalle: still looking up some notes i have on this20:52
*** thomasbiege has quit IRC20:53
*** thomasbiege has joined #openstack-qa20:53
mlavallemriedem: ok, i'll keep an eye on his return and start nagging him as soon as he is back20:53
marunhttps://review.openstack.org/#/c/42735/20:53
mlavallemriedem: if you find the notes, would you add them to the bug?20:54
marundkranz: ^20:54
mriedemmlavalle: yeah20:54
mlavalle;-)20:54
*** LiangZheng has quit IRC20:54
dkranzmarun: Saw that. Need to wait for the +1 from jenkins.20:55
marundkranz: i know it's procedure, but a part of me is not impressed with having to rerun the gate simply because the new change id required a new review20:56
marun(since the gate has already passed)20:57
*** thomasbiege has quit IRC20:57
*** thomasbiege has joined #openstack-qa20:57
dkranzmarun: I'm not sure why it is necessary. Ask sdague :)20:58
marundkranz: the robots do the work (or make it, depending :) )20:59
*** jhenner has quit IRC21:02
*** malini_afk is now known as malini21:03
*** woodspa has quit IRC21:08
mriedemmlavalle: bug reported updated with my notes21:11
*** SergeyLukjanov has quit IRC21:11
mriedemsdague == /dev/null right now :)21:11
mlavallemriedem: thanks. I appreciate it. This bug is causing 4 test failures in the neutron full gate test. So we need to find a solution21:12
*** thomasbiege has quit IRC21:12
mriedemmlavalle: yeah, that's why i proposed the skip test patch...it's a sticky one - definitely thanks for looking at it thought21:12
mriedem*though21:12
*** malini is now known as malini_afk21:15
*** jasondotstar has quit IRC21:28
*** dkranz has quit IRC21:36
*** yfried has quit IRC21:42
openstackgerritRavi Chunduru proposed a change to openstack-dev/devstack: blueprint quantum-fwaas-devstack Adds support for Openstack Networking FWaaS (Firewall)  https://review.openstack.org/3714721:48
*** LiangZheng has joined #openstack-qa21:50
*** LiangZheng has quit IRC21:58
*** burt has quit IRC22:09
*** mkollaro has quit IRC22:11
*** mriedem has quit IRC22:17
openstackgerritBob Ball proposed a change to openstack-dev/devstack: Use a global virtualenv  https://review.openstack.org/4053422:23
*** jerryz has quit IRC22:30
openstackgerritJames E. Blair proposed a change to openstack-infra/devstack-gate: Use git.openstack.org as origin  https://review.openstack.org/4269322:49
*** LiangZheng has joined #openstack-qa22:55
*** weshay has joined #openstack-qa22:57
*** LiangZheng has quit IRC23:00
*** malini_afk is now known as malini23:01
*** mlavalle has quit IRC23:09
*** sthaha has joined #openstack-qa23:13
*** sthaha has joined #openstack-qa23:13
*** weshay has quit IRC23:14
openstackgerritZhang Hua proposed a change to openstack/tempest: Fixes the issues of tempest due to async network allocation  https://review.openstack.org/4247223:17
*** LiangZheng has joined #openstack-qa23:56
« Sunday, 2013-08-18 Index Tuesday, 2013-08-20 »

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