Monday, 2013-09-16

openstackgerritA change was merged to openstack/tempest: Adds verfication for Bootable Volume  https://review.openstack.org/4262100:00
*** hootjr has quit IRC00:25
openstackgerritSteve Baker proposed a change to openstack-dev/devstack: Install and use diskimage-builder, tripleo-image-elements  https://review.openstack.org/4596900:41
openstackgerritSteve Baker proposed a change to openstack-dev/devstack: Baremetal uses packaged diskimage-builder  https://review.openstack.org/4034300:41
*** sthaha has joined #openstack-qa00:53
*** sthaha has joined #openstack-qa00:53
*** marun has joined #openstack-qa01:49
*** prekarat has joined #openstack-qa03:56
*** yfried has quit IRC04:00
*** yfried has joined #openstack-qa04:00
*** SergeyLukjanov has joined #openstack-qa04:16
*** raies has joined #openstack-qa04:24
raiesmkoderer: hi04:26
*** madan has joined #openstack-qa04:48
*** SergeyLukjanov has quit IRC04:51
*** SergeyLukjanov has joined #openstack-qa04:59
*** bashok has joined #openstack-qa05:19
mkodererraies: https://review.openstack.org/#/c/45230/7/tempest/services/network/json/network_client.py line 43705:21
mkoderercould you delete this tab?05:21
mkodererIt's a nit I know05:21
raiesmkoderer: yes after deleting I will apply new patch05:22
*** SergeyLukjanov has quit IRC05:22
raiesshould I ?05:22
openstackgerritDavid Geng proposed a change to openstack/tempest: Enable the quota test cases  https://review.openstack.org/3315205:23
mkodererraies: I am not happy with a patch of a patch ;)05:25
mkoderercould you upload a new patch and you get my +2 immediately05:25
mkodererand I will ping afazekas to get it merged05:25
raiesok I will upload a new patch05:26
*** sungju has joined #openstack-qa05:28
openstackgerritMh Raies proposed a change to openstack/tempest: Network load balancer testing  https://review.openstack.org/4523005:39
raiesmkoderer: done05:39
mkodererafazekas: coudl you check https://review.openstack.org/4523005:42
mkodererafazekas: strange thing that I automatically gave a +A instead of a +205:42
*** sourabh has joined #openstack-qa05:44
afazekasmkoderer: ?05:46
mkodererafazekas: I think zuul process was started since I have a +A accidentally05:47
afazekasmkoderer: you can say -2 to prevent the merge05:47
*** bashok_ has joined #openstack-qa05:47
mkodererafazekas: will it stop the zuul process immediately?05:48
afazekasnot05:48
afazekasBTW the change looked OK05:49
mkodererafazekas: yes so could you give your +205:49
mkodererand I will give the +A05:49
mkodererafazekas: I just had a nit, so we can merge it05:50
afazekasI have one issue  someone might  want to skip the LB tests, so it should be configurable05:50
*** bashok has quit IRC05:51
mkodererafazekas: ok give my a ping when you gave your +205:52
afazekasmkoderer: ping05:53
mkodererafazekas: ok thanks05:54
* afazekas I will be back in 50 minutes05:54
*** afazekas is now known as __afazekas05:59
*** jhenner has joined #openstack-qa06:16
*** jhenner has quit IRC06:18
*** rdekel has joined #openstack-qa06:19
*** afazekas has joined #openstack-qa06:22
openstackgerritDavid Geng proposed a change to openstack/tempest: Enable the quota test cases  https://review.openstack.org/3315206:32
openstackgerritA change was merged to openstack/tempest: Network load balancer testing  https://review.openstack.org/4523006:37
*** __afazekas has quit IRC06:40
openstackgerritMh Raies proposed a change to openstack/tempest: Adding xml support for load balancer  https://review.openstack.org/4669606:41
afazekasraies: FYI: https://bugs.launchpad.net/tempest/+bug/122562306:44
raiesafazekas: thanks, looking06:45
*** rdekel has quit IRC06:49
openstackgerritMarc Koderer proposed a change to openstack/tempest: Fix problem with never stopping stress tests  https://review.openstack.org/4608706:50
*** rdekel has joined #openstack-qa06:52
*** SergeyLukjanov has joined #openstack-qa06:52
*** giulivo has joined #openstack-qa06:54
*** sungju has quit IRC06:55
*** tkammer has joined #openstack-qa06:55
raiesafazekas: L#64 creates a overlapping subnet ?06:59
raieshow u find it ?06:59
afazekasL64 creates a subnet in the same range as the bulk net creation07:00
afazekasIf the allow_overlapping_ips is False it causes an issue as you can can see in the bug report07:01
afazekasIf the bulk_ subnet creation would be in different class (where  the setUpClass does not creates subnets ), it would be ok07:03
raiesI looked there and satisfied with your opinion07:06
*** ekarlso- has quit IRC07:07
raiesI am thinking to create a separate class BulkSubnetCreateJSON and BulkSubnetCreateXML07:08
raieswhat do you think ?07:08
afazekasI am thinking about what if we create bulk test classes for all bulk tests07:10
raiesok yes it would be better enough07:11
afazekasit would be also a logical separation of test types07:11
raiesyes :)  BulkRequestJSON and  BulkRequestXML07:12
raies?07:12
*** giulivo has quit IRC07:12
*** giulivo has joined #openstack-qa07:13
*** thomasbiege has joined #openstack-qa07:18
afazekasok, or BulkNetworkOperationsJSON and BulkNetworkOperationsXML  or just Ops, does not really matters07:19
raiesok07:21
*** thomasbiege has quit IRC07:27
openstackgerritSahid Orentino Ferdjaoui proposed a change to openstack/tempest: Adds tests covering Swift's container quotas middleware  https://review.openstack.org/4623207:30
openstackgerritMh Raies proposed a change to openstack/tempest: Fix overlapping subnet creation  https://review.openstack.org/4669807:31
raiesafazekas: please look  https://review.openstack.org/4669807:32
openstackgerritDavid Geng proposed a change to openstack/tempest: Enable the quota test cases  https://review.openstack.org/3315207:32
raiesafazekas: are you there ?07:33
openstackgerritnayna-patel proposed a change to openstack/tempest: Added test to check list/show extensions-neutron  https://review.openstack.org/4503007:37
afazekasmkoderer: ^^07:45
openstackgerritSahid Orentino Ferdjaoui proposed a change to openstack/tempest: Adds tests covering Swift's container quotas middleware  https://review.openstack.org/4623207:50
*** mkollaro has joined #openstack-qa07:50
*** fbo_away is now known as fbo07:52
raiesafazekas: u there ?07:57
afazekasraies: yes, waiting for jenkins and thinking about is moving the bulks test to a new file good or bad idea07:58
afazekasmkoderer, giulivo: ^^   https://review.openstack.org/4669807:59
openstackgerritDavid Geng proposed a change to openstack/tempest: Enable the quota test cases  https://review.openstack.org/3315208:01
*** yassine has joined #openstack-qa08:04
*** jhenner has joined #openstack-qa08:06
openstackgerritMh Raies proposed a change to openstack/tempest: Adding clinet functions and test cases for pool  https://review.openstack.org/4560208:09
raiesafazekas: till now please check https://review.openstack.org/#/c/45602/08:10
*** SergeyLukjanov has quit IRC08:12
*** SergeyLukjanov has joined #openstack-qa08:13
*** mkollaro has quit IRC08:15
openstackgerritChangBo Guo proposed a change to openstack/tempest: Add keystone user-update test  https://review.openstack.org/4647708:20
*** mkollaro has joined #openstack-qa08:21
*** anteaya has quit IRC08:29
*** dosaboy has quit IRC09:03
*** dosaboy has joined #openstack-qa09:03
*** thomasbiege has joined #openstack-qa09:05
*** kashyap has joined #openstack-qa09:14
mkodererafazekas: I am ok with https://review.openstack.org/4669809:27
mkodererafazekas: but waiting for jenkins09:27
afazekasok09:29
raiesafazekas: please look https://review.openstack.org/46698 , all time jenkins fails it09:57
afazekaspsedlak: http://logs.openstack.org/98/46698/1/check/gate-tempest-devstack-vm-full/1273a96/console.html10:02
afazekashttp://logs.openstack.org/98/46698/1/check/gate-tempest-devstack-vm-full/1273a96/logs/screen-n-api.txt.gz#_2013-09-16_09_34_08_06110:04
afazekaspsedlak: do you think is it keystone bug ? Or one of the new keystone v3 test cases are incorrect ?10:08
afazekashttp://logs.openstack.org/98/46698/1/check/gate-tempest-devstack-vm-full/1273a96/logs/screen-key.txt.gz10:11
afazekas(access): 2013-09-16 09:34:08,047 INFO core __call__ 127.0.0.1 - - [16/Sep/2013:09:34:08 +0000] "GET http://127.0.0.1:35357/v2.0/tokens/revoked HTTP/1.0" 200 1361810:11
afazekasit queries the revoke list10:11
*** SergeyLukjanov has quit IRC10:13
psedlakafazekas: yes, and it says it was revoked ...10:14
*** SergeyLukjanov has joined #openstack-qa10:14
afazekasDo you see any explicit revoke in the keystone log before ?10:15
afazekashttp://logs.openstack.org/98/46698/1/check/gate-tempest-devstack-vm-full/1273a96/logs/tempest.txt.gz#_2013-09-16_09_34_08_06410:26
afazekassomething is wrong with the _log_request  and _log_response10:28
afazekasit should log the ``worker`` id as well10:33
afazekasSo who will create the bug ?10:36
psedlakafazekas: http://logs.openstack.org/98/46698/1/check/gate-tempest-devstack-vm-full/1273a96/logs/tempest.txt.gz#_2013-09-16_09_34_06_66610:49
psedlakafazekas: that fddcae75... should be our user10:49
psedlakafazekas: two seconds before we try to use it10:49
psedlakafazekas: tenant probably follows10:50
psedlakafazekas: http://logs.openstack.org/98/46698/1/check/gate-tempest-devstack-vm-full/1273a96/logs/tempest.txt.gz#_2013-09-16_09_33_47_67110:50
psedlakFixedIPsTestJson23961089710:50
psedlak*-user10:50
afazekaspsedlak, raises: https://bugs.launchpad.net/tempest/+bug/122600111:06
afazekasraies: you can recheck with 122600111:06
raiesafazekas: done11:12
openstackgerritA change was merged to openstack/tempest: VolumesListTest may hide configuration issues  https://review.openstack.org/4651411:14
openstackgerritMh Raies proposed a change to openstack/tempest: Fix overlapping subnet creation  https://review.openstack.org/4669811:20
openstackgerritnayna-patel proposed a change to openstack/tempest: Added test to check list/show extensions-neutron  https://review.openstack.org/4503011:22
openstackgerritMh Raies proposed a change to openstack/tempest: Adding clinet functions and test cases for pool  https://review.openstack.org/4560211:29
openstackgerritYassine Lamgarchal proposed a change to openstack/tempest: Adds client and CRUD tests for Ceilometer alarms  https://review.openstack.org/3923711:40
afazekasI have doubts it is working as described: https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_fixed_ips.py#L3411:42
openstackgerritYassine Lamgarchal proposed a change to openstack/tempest: Adds client and CRUD tests for Ceilometer alarms  https://review.openstack.org/3923711:44
afazekaspsedlak: similar issue: https://review.openstack.org/#/c/46696/11:45
*** madan has quit IRC11:45
*** HenryG has quit IRC11:49
*** sthaha has quit IRC11:53
*** weshay has joined #openstack-qa11:55
*** hootjr has joined #openstack-qa11:55
openstackgerritA change was merged to openstack/tempest: Fixes some typos in tempest  https://review.openstack.org/4660111:57
mkodererafazekas: regarding https://review.openstack.org/#/c/46619/12:01
mkodererafazekas: shouldn't be it in requirements.txt instead of test-requirements.txt12:01
mkoderer?12:01
openstackgerritMh Raies proposed a change to openstack/tempest: Adding member operation test cases  https://review.openstack.org/4562712:02
*** raies has quit IRC12:02
*** mkollaro has quit IRC12:09
*** mkollaro has joined #openstack-qa12:15
afazekasmkoderer: yes12:20
mkodererafazekas: ok :)12:20
afazekaspsedlak: Thank for the help, so  the 1226001 real tempest bug12:21
* afazekas set it to critical12:21
psedlakafazekas: np, i'm adding more info to the bug, so you can postpone the fix little bit (finish what you are doing now ;))12:21
*** mkollaro has quit IRC12:23
*** mkollaro1 has joined #openstack-qa12:23
*** mkollaro1 is now known as mkollaro12:23
openstackgerritA change was merged to openstack/tempest: add required python modules(six,iso8601,fixtures) to requirements.txt  https://review.openstack.org/4666512:25
openstackgerritafazekas proposed a change to openstack/tempest: test_fixed_ips unsafe xml/json resource share  https://review.openstack.org/4672212:27
rahmuafazekas: can I get a review on this: https://review.openstack.org/#/c/44287/? It's been inactive for a while12:28
afazekasrahmu: note:  the usual vim line is missing t the beginning of the files12:29
rahmuoh thanks, I didn't know it was required12:30
afazekasmkollaro: https://review.openstack.org/#/c/4428712:30
afazekasrahmu: As I remember the swift test should use just set match for success status codes : self.assertIn(int(resp['status']), HTTP_SUCCESS)12:34
afazekassee the git commit: 0d499bc6a14dfad88523e5b4807ad13d719268c2 , https://review.openstack.org/#/c/33689/12:35
rahmuafazekas: thanks for the pointer12:37
Anjuafazekas:12:40
Anju ping12:40
afazekasAnju: pong12:40
Anjuafazeaks : ragrding the lock_unlcok_test of a server12:40
Anjuafazeaks : regarding the lock_unlcok_test of a server12:40
Anjufirstly i tried to onlt lock a server12:41
Anjubut the status is something like this12:41
Anju File "/opt/stack/tempest/tempest/api/compute/servers/test_server_actions.py", line 278, in test_e_server12:41
Anju    self.client.wait_for_server_status(self.server_id, 'LOCKED')12:41
Anju  File "/opt/stack/tempest/tempest/services/compute/json/servers_client.py", line 156, in wait_for_server_status12:41
Anju    return waiters.wait_for_server_status(self, server_id, status)12:41
Anju  File "/opt/stack/tempest/tempest/common/waiters.py", line 80, in wait_for_server_status12:41
Anju    raise exceptions.TimeoutException(message)12:41
AnjuTimeoutException: Request timed out12:41
Anjuafazekas: ^^^12:41
*** bashok_ has quit IRC12:41
Anjuafazekas:  Server a3219b92-b523-4d33-b9a1-928c5c5ae4bb failed to reach LOCKED status within the required time (600 s). Current status: ACTIVE.12:42
*** bashok_ has joined #openstack-qa12:42
*** mkollaro1 has joined #openstack-qa12:44
*** mkollaro has quit IRC12:44
afazekasAnju: are you sure it is a server status12:45
Anjuyes12:45
Anjuthis test was not added12:45
Anjuu said last time that xml is not working12:46
*** sthaha has joined #openstack-qa12:46
Anjui run only with json12:46
*** jaypipes has joined #openstack-qa12:46
*** ChanServ sets mode: +o jaypipes12:46
*** mkollaro1 has quit IRC12:47
*** mkollaro has joined #openstack-qa12:47
afazekashttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L254512:48
Anjuafazekas:  yes12:49
Anjui knew this12:49
Anjubut i do not why this is not running in ttempest12:49
AnjuI told u this before alos12:49
Anjualso12:49
*** mkollaro has quit IRC12:52
*** mkollaro has joined #openstack-qa12:52
*** mkollaro has quit IRC12:53
*** mkollaro has joined #openstack-qa12:53
*** bashok__ has joined #openstack-qa13:00
afazekasAnju: so you can change the status to 'LOCKED' by CLI ?13:00
Anjuyes13:01
Anjuunlock alos13:01
Anjualso13:01
Anjumay i show u what i have done13:02
*** HenryG has joined #openstack-qa13:02
*** bashok_ has quit IRC13:03
afazekaspsedlak, raies, giulivo, cyeoh, mkoderer, mtreinish, sdague, jaypipes:  https://review.openstack.org/#/c/46722/13:09
openstackgerritYassine Lamgarchal proposed a change to openstack/tempest: Add list containers call to swift client.  https://review.openstack.org/4672713:09
*** jasondotstar has joined #openstack-qa13:10
*** dkranz has quit IRC13:10
afazekasAnju: how can you lock the server ?13:12
Anjunova lock server name13:13
Anju:)13:13
Anjuafazekas:  ^^^13:13
afazekaswith fresh devstack install it does not changes me the status as demo user13:13
Anjuu mean that this is the admin actions13:14
*** jasondotstar has quit IRC13:15
afazekaseven with admin13:17
Anjuyes13:18
Anju:13:18
Anjuafazekas:  but state should not changed13:18
*** mriedem has joined #openstack-qa13:24
openstackgerritYassine Lamgarchal proposed a change to openstack/tempest: Add list containers call to swift client.  https://review.openstack.org/4672713:25
Anjuafazekas:  yes afazekas . u are great ..now it is ok13:28
*** adalbas has joined #openstack-qa13:35
openstackgerritafazekas proposed a change to openstack/tempest: test_fixed_ips unsafe xml/json resource sharing  https://review.openstack.org/4672213:37
*** malini has joined #openstack-qa13:40
*** rdekel has quit IRC13:44
*** burt has joined #openstack-qa13:44
*** rdekel has joined #openstack-qa13:47
jaypipesafazekas: done13:51
*** SergeyLukjanov has quit IRC13:54
*** yfried has quit IRC13:55
afazekasjaypipes: thank you13:55
mkodererafazekas: done +A13:55
afazekasmkoderer: thank you13:55
* mkoderer didn't get the old logic that was there before13:56
*** dkranz has joined #openstack-qa13:56
*** sthaha has quit IRC13:56
*** SergeyLukjanov has joined #openstack-qa13:58
*** yfried has joined #openstack-qa13:58
openstackgerritChangBo Guo proposed a change to openstack/tempest: Add keystone user-update test  https://review.openstack.org/4647714:06
*** bashok__ has quit IRC14:16
*** jasondotstar has joined #openstack-qa14:17
*** bashok has joined #openstack-qa14:17
*** bashok has quit IRC14:18
*** adalbas has quit IRC14:19
*** bashok has joined #openstack-qa14:19
openstackgerritEdward Hope-Morley proposed a change to openstack/tempest: Add cinder-backup functional tests  https://review.openstack.org/4546514:21
*** jasondotstar has quit IRC14:23
*** jasondotstar has joined #openstack-qa14:25
*** adalbas has joined #openstack-qa14:35
*** jasondotstar has quit IRC14:41
openstackgerritMarc Koderer proposed a change to openstack/tempest: Fix problem with never stopping stress tests  https://review.openstack.org/4608714:41
*** raies has joined #openstack-qa14:44
raiesafazekas: http://logs.openstack.org/98/46698/2/check/gate-tempest-devstack-vm-full/38968eb/testr_results.html.gz14:44
*** jhenner has quit IRC14:45
raiesgetting failed14:45
raiesagain and again14:45
afazekasraise: wait until it is merged: https://review.openstack.org/#/c/46722/14:47
afazekasAdding a new class probably changes the test class distribution and ordering, and it triggers the bug14:47
*** jhenner has joined #openstack-qa14:49
raiesok I will wait14:49
*** ken1ohmichi has joined #openstack-qa14:49
*** ken1ohmichi has quit IRC14:50
openstackgerritA change was merged to openstack/tempest: Fixes typos in tempest/api  https://review.openstack.org/4659814:54
*** weshay has quit IRC14:57
openstackgerritYassine Lamgarchal proposed a change to openstack/tempest: Add tests for Ceilometer object storage metering  https://review.openstack.org/4674415:00
openstackgerritYassine Lamgarchal proposed a change to openstack/tempest: Add client and tests for Ceilometer object storage metering  https://review.openstack.org/4674515:00
openstackgerritYassine Lamgarchal proposed a change to openstack/tempest: Add client and tests for Ceilometer compute metering  https://review.openstack.org/4348115:00
*** thomasbiege1 has joined #openstack-qa15:05
*** bashok has quit IRC15:05
*** dkranz has quit IRC15:05
*** openstackgerrit has quit IRC15:13
*** cyeoh has quit IRC15:14
*** sdague has quit IRC15:14
*** zigo has joined #openstack-qa15:14
*** zigo_ has quit IRC15:15
*** thomasbiege1 has quit IRC15:18
*** sdague has joined #openstack-qa15:19
*** rdekel has quit IRC15:20
*** jasondotstar has joined #openstack-qa15:20
mkoderermtreinish: https://review.openstack.org/#/c/46087/15:24
mkodererI tried to enhance it15:25
mtreinishmkoderer: ok I'll take a look15:26
*** openstackgerrit has joined #openstack-qa15:30
*** openstackgerrit has quit IRC15:30
*** adam_g has quit IRC15:31
*** openstackgerrit has joined #openstack-qa15:31
openstackgerritA change was merged to openstack/tempest: test_fixed_ips unsafe xml/json resource sharing  https://review.openstack.org/4672215:32
*** thomasbiege1 has joined #openstack-qa15:33
*** jasondotstar has quit IRC15:33
*** thomasbiege1 has quit IRC15:33
*** weshay has joined #openstack-qa15:34
afazekasraies: ^^15:35
afazekasyou can push the rebase button15:35
raiesok15:35
raiesi will15:35
raiesrebase changes button on the UI ?15:36
afazekasyes15:36
raiesok15:36
openstackgerritMh Raies proposed a change to openstack/tempest: Fix overlapping subnet creation  https://review.openstack.org/4669815:36
afazekasit will rebase the change and runs the test again15:36
raiesok15:36
afazekasnote: $ git review -d <change-number> will download the new version15:37
raiesok15:37
raiesAnd clicking will rebase it existing version ?15:38
openstackgerritYassine Lamgarchal proposed a change to openstack/tempest: Add tests for Ceilometer object storage metering  https://review.openstack.org/4674415:45
afazekasyes15:48
*** thomasbiege1 has joined #openstack-qa15:54
openstackgerritA change was merged to openstack-dev/devstack: Disable ceilometer.compute.nova_notifier driver  https://review.openstack.org/4633616:02
openstackgerritA change was merged to openstack-dev/devstack: lib/tempest remove whitebox section  https://review.openstack.org/4668416:02
*** SergeyLukjanov has quit IRC16:04
*** atiwari has joined #openstack-qa16:06
*** thomasbiege1 has quit IRC16:09
*** dkranz has joined #openstack-qa16:12
*** adam_g has joined #openstack-qa16:15
*** adam_g has joined #openstack-qa16:15
*** bearhands is now known as comstud16:16
*** yfried has quit IRC16:17
*** dkranz has quit IRC16:18
*** dkranz has joined #openstack-qa16:19
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Add meta-config  https://review.openstack.org/4676816:24
*** yassine has quit IRC16:29
*** prekarat has quit IRC16:31
*** prekarat has joined #openstack-qa16:32
*** jasondotstar has joined #openstack-qa16:33
*** SergeyLukjanov has joined #openstack-qa16:34
openstackgerritafazekas proposed a change to openstack-dev/devstack: Increase default swift storage  https://review.openstack.org/4677016:38
*** SergeyLukjanov has quit IRC16:41
*** SergeyLukjanov has joined #openstack-qa16:43
*** cyeoh has joined #openstack-qa16:44
openstackgerritSahid Orentino Ferdjaoui proposed a change to openstack/tempest: Adds tests covering Swift's container quotas middleware  https://review.openstack.org/4623216:46
*** afazekas has quit IRC16:47
*** ekarlso has joined #openstack-qa16:49
raiesmkoderer: hi16:51
raiesplease review https://review.openstack.org/#/c/46698/16:52
raiesjaypipes: ^^16:54
jaypipesraies: looking now...16:54
*** fbo is now known as fbo_away17:00
*** tkammer has quit IRC17:00
raiesjaypipes: as per afazekas and mkoderer i was good, they were waiting for jenkins17:01
*** SergeyLukjanov has quit IRC17:09
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add parallel test execution section to HACKING.rst  https://review.openstack.org/4677417:09
*** morganfainberg has joined #openstack-qa17:10
*** prekarat has quit IRC17:11
*** prekarat has joined #openstack-qa17:11
*** hootjr has quit IRC17:12
*** SergeyLukjanov has joined #openstack-qa17:12
raiesJaypipes: u completed ?17:13
jaypipesraies: no, not yet. you will see when I am.17:15
jaypipesraies: ok, +2 from me.17:17
raies:) thanks :)17:18
raiesbtw who will approve it ? I know u are core approver17:18
raiesjaypipes: please also check https://review.openstack.org/#/c/45602/ and https://review.openstack.org/#/c/46696/17:21
*** morganfainberg has left #openstack-qa17:21
jaypipesraies: please be patient. another core reviewer will approve it in time.17:22
raiesok17:22
raiesbut please check links what I have shared17:23
*** thomasbiege1 has joined #openstack-qa17:24
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Sync latest versions of oslo incubator  https://review.openstack.org/4677717:25
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Update docs config  https://review.openstack.org/4677817:25
*** prekarat has quit IRC17:26
*** yfried has joined #openstack-qa17:26
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add parallel test execution section to HACKING.rst  https://review.openstack.org/4677417:26
*** wramthun has joined #openstack-qa17:27
*** harlowja has quit IRC17:33
*** harlowja has joined #openstack-qa17:33
*** thomasbiege2 has joined #openstack-qa17:33
*** thomasbiege1 has quit IRC17:35
openstackgerritSteve Baker proposed a change to openstack-dev/devstack: Install and use diskimage-builder, tripleo-image-elements  https://review.openstack.org/4596917:37
openstackgerritSteve Baker proposed a change to openstack-dev/devstack: Baremetal uses packaged diskimage-builder  https://review.openstack.org/4034317:37
*** raies has quit IRC17:44
*** burt has quit IRC17:46
*** jhenner has quit IRC17:49
*** jhenner has joined #openstack-qa17:50
*** thomasbiege2 has quit IRC17:53
*** thomasbiege1 has joined #openstack-qa17:56
*** SergeyLukjanov has quit IRC18:00
*** HenryG has quit IRC18:00
*** SergeyLukjanov has joined #openstack-qa18:04
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Update docs config  https://review.openstack.org/4677818:04
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Add meta-config  https://review.openstack.org/4676818:05
*** jasondotstar has quit IRC18:17
*** mkollaro has quit IRC18:21
jog0anyone see thsi bug before http://logs.openstack.org/14/46614/1/check/gate-tempest-devstack-vm-full/e3174ef/testr_results.html.gz18:23
*** jasondotstar has joined #openstack-qa18:23
mtreinishjog0: no I haven't but I've seen a few similar teardown failures with cinder lately18:26
mtreinishjog0: the c-vol log shows an error: http://logs.openstack.org/14/46614/1/check/gate-tempest-devstack-vm-full/e3174ef/logs/screen-c-vol.txt.gz#_2013-09-16_17_42_35_97918:26
jog0mtreinish: ahh cool18:28
jog0I will file a bug on that and add a logstash query to https://github.com/jogo/elasticRecheck18:28
jog0to detect it18:28
mtreinishjog0: look at you getting all fancy18:28
mtreinishI have to take a look at that18:28
mtreinishyeah, that's a lot better than me trying to keep it all in my head :)18:29
jog0mtreinish: heh yeah18:31
jog0its starting to work18:31
*** jasondotstar has quit IRC18:31
jog0going to work with clarkb to get it moved over to infra in the near future18:31
mtreinishjog0: ok cool18:32
*** afazekas has joined #openstack-qa18:32
*** jasondotstar has joined #openstack-qa18:32
jog0mtreinish: this looks like teh bug https://bugs.launchpad.net/cinder/+bug/119196018:33
jog0oh also for rechecks if you mark things as duplicates the recheck page will reflect that18:34
jog0mtreinish: https://review.openstack.org/#/c/45511/18:34
mtreinishjog0: yeah that looks like the same error. The patch to fix it was abandonded in Jul...18:35
mtreinishjog0: cool that cleans things up quite a bit18:35
jog0yeah18:37
jog0so you can go through and mark things are duplicates of this bug etc18:37
jog0logstash query: "Unable to deactivate logical volume" AND @fields.filename:"logs/screen-c-vol.txt"18:39
giulivodosaboy so this will never be merged as long as jenkins will fail and jenkins will fail until you get the backup test to run successfully18:40
giulivoso I think you need to pass the admin credentials to the backup client to make sure it will have the needed credentials to create a container in swift18:41
giulivobut you didn't do that in the last few changes, anything I didn't notice?18:41
giulivodosaboy, https://review.openstack.org/#/c/4546518:42
afazekashttp://logs.openstack.org/65/45465/8/check/gate-tempest-devstack-vm-full/e77ffc4/logs/screen-c-bak.txt.gz#_2013-09-16_14_39_48_44118:44
afazekascinder or config issue18:45
giulivoafazekas, I think it is just the PUT failing with 40318:45
giulivobecause the test is running with non-admin credentials so it can't create the container where to place the backup in18:46
afazekasgiulivo: the others with 404 because the put failed ?18:46
dkranzgiulivo: You gave a +1 to https://review.openstack.org/#/c/46696/ but I'm not sure what that meant.18:47
afazekasgiulivo: so the user needs to have an operator role ?18:47
dosaboygiulivo: agreed18:47
jog0mtreinish: I am looking for help with the logstash queries if your interested18:47
giulivoafazekas, to create the containers yes18:47
giulivobut honestly I'm not sure about the 404 I see there is a "None" in the url18:47
dosaboyi had not tested against swift the last few times so I'll try to get that fixed up18:47
mtreinishjog0: sure, I've got to learn how to use it at some point18:47
jog0mtreinish: to start with try running https://github.com/jogo/elasticRecheck18:48
giulivodkranz, because I think the code is actually okay only maybe those two methods could be pushed with the other change, where the tests using those are also pushed18:48
giulivodkranz, better to -1 ?18:49
jog0mtreinish: I added you as a 'collaborator' on github for that18:49
openstackgerritA change was merged to openstack/tempest: Fix overlapping subnet creation  https://review.openstack.org/4669818:49
jog0mtreinish: to run you just have to change the config file to have your username not mine18:49
dkranzgiulivo: If you want a change made before approving, give -118:50
jog0mtreinish: and queries go in https://github.com/jogo/elasticRecheck/blob/master/queries.json18:50
dkranzgiulivo: If it is not necessary for approval, give +2/+A with a comment.18:50
jog0http://logstash.openstack.org/18:50
dkranzgiulivo: If you are not sure, and just want to comment, then just comment with no +/-18:50
mtreinishjog0: ok cool I'll add one for the max retries atttempted neutron fail to start18:50
mtreinish(first one to pop up when I started running it)18:51
jog0mtreinish: I think I have that actually18:51
mtreinishjog0: yep looks like you do18:51
mtreinishbut it failed to find the failure18:51
afazekasgiulivo: can it be swift/object-store test-case ? the swift base class has method for assigning a role18:51
jog0mtreinish: hmm try running the query manually and see if it works still18:52
giulivoafazekas, yeah we went trough it but dosaboy is suggesting we should not put anything "swift" specific in the test because the backup api ideally doesn't depend on the actual backend and I think he's right18:52
jog0what gerrit patch hit that  bug?18:52
mtreinishjog0: https://review.openstack.org/#/c/46296/118:53
mtreinishjog0: the query doesn't seem to work18:53
giulivoafazekas, so we wanted to keep the test only stressing the cinder api but apparently facing the permissions issue18:53
*** jasondotstar has quit IRC18:53
afazekasIMHO if the credentials isolation is enabled  the test case can assign the configured operator roles, if not the predefined 'user' should have the role18:56
afazekasIt should not be harm to a different backend18:57
giulivoafazekas, let me see if I get it right, that is using the keystone admin client to assign the needed role to the user "currently" running the test?18:57
*** jhenner has quit IRC18:58
jog0mtreinish: looks like that run doesn't have stuff in logstash18:58
afazekasgiulivo: something like this, just with one user : https://github.com/openstack/tempest/blob/master/tempest/api/object_storage/base.py#L8218:59
afazekasjust only if the https://github.com/openstack/tempest/blob/master/tempest/api/object_storage/base.py#L35 is True19:00
jog0clarkb: ^19:00
*** mlavalle has joined #openstack-qa19:04
*** harlowja has quit IRC19:06
*** harlowja has joined #openstack-qa19:07
jog0mtreinish: clarkb said its because logstash is 30 min behind19:09
mtreinishjog0: ok that would do it19:10
openstackgerritSalvatore Orlando proposed a change to openstack/tempest: Check tenant connectivity when tenant networks are unreachable  https://review.openstack.org/4679119:10
mtreinishjog0: is the eventual plan for this to get it run recheck automatically?19:10
jog0mtreinish: perhaps one day, the next step for now is leave a comment saying hey I think you hit x19:11
jog0and adding an IRC bot in this room saying when a unclassified bug is hit19:11
*** jasondotstar has joined #openstack-qa19:13
giulivoafazekas, I see that19:13
giulivodosaboy, the one thing I don't get from the logs posted by afazekas is where does that "None" comes from in the URL19:14
giulivoit uses GET to see if the container exists already, later tries the PUT but why is there a "None" in the URL? the container should default to "cinderbackups"19:15
jog0anyone here a neutron bug triager19:17
jog0https://bugs.launchpad.net/neutron/+bug/1211915 should be in RC119:17
*** mlavalle has quit IRC19:19
*** mlavalle has joined #openstack-qa19:21
*** wramthun has quit IRC19:24
*** wramthun has joined #openstack-qa19:25
mtreinishjog0: good call on: https://bugs.launchpad.net/tempest/+bug/1218585 I had forgotten about that one. It's definitely a dup19:26
*** burt has joined #openstack-qa19:31
*** mlavalle has quit IRC19:32
*** mlavalle has joined #openstack-qa19:34
*** malini is now known as malini_afk19:35
*** thomasbiege1 has quit IRC19:45
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Add meta-config via local.conf  https://review.openstack.org/4676819:46
dkranzmtreinish: I can't see a sensible way to actually review diffs of synced files from oslo so I am just going to approve your change unless you have a different idea.19:47
dkranzmtreinish: I wish there were a better way to do this.19:47
mtreinishdkranz: yeah, it's annoying19:48
mtreinishall I did was run the oslo update script19:48
dkranzmtreinish: I know. I'm just going to approve it.19:48
*** morganfainberg has joined #openstack-qa19:52
*** mlavalle has quit IRC19:53
*** mlavalle has joined #openstack-qa19:55
*** mlavalle has quit IRC20:01
dkranzjog0: You said in a comment there was a problem with https://review.openstack.org/#/c/40435/ that is not visible from it's gate jobs. Is it OK now?20:19
*** atiwari has quit IRC20:20
*** hemnafk has quit IRC20:20
*** hemnafk has joined #openstack-qa20:22
jog0dkranz: logstash was just slow20:22
*** atiwari has joined #openstack-qa20:22
jog0ohh wooops sorry wrong thing20:22
openstackgerritafazekas proposed a change to openstack/tempest: test_images_oneserver.py ensure Active  https://review.openstack.org/4680320:24
*** hemnafk is now known as hemna20:25
jog0dkranz: http://logs.openstack.org/35/40435/9/silent/gate-tempest-devstack-vm-large-ops/06f0689/testr_results.html.gz20:26
jog0it works20:26
*** fbo_away is now known as fbo20:28
*** jhenner has joined #openstack-qa20:31
*** SergeyLukjanov has quit IRC20:34
jog0mtreinish: which bug is this http://logs.openstack.org/19/46219/5/check/gate-tempest-devstack-vm-postgres-full/9cc5b03/testr_results.html.gz20:38
dkranzjog0: Thanks.20:41
openstackgerritA change was merged to openstack/tempest: Adding xml support for load balancer  https://review.openstack.org/4669620:46
openstackgerritA change was merged to openstack/tempest: Add parallel test execution section to HACKING.rst  https://review.openstack.org/4677420:47
*** burt has quit IRC20:55
openstackgerritafazekas proposed a change to openstack/tempest: test_images_oneserver.py ensure Active  https://review.openstack.org/4680320:59
*** afazekas is now known as afazekas_zz20:59
*** thomasbiege1 has joined #openstack-qa21:00
*** HenryG has joined #openstack-qa21:01
openstackgerritAaron Rosen proposed a change to openstack-dev/devstack: use method pip_install over sudo pip install so proxy settings work  https://review.openstack.org/4681621:05
mtreinishjog0: I think it's https://bugs.launchpad.net/openstack-ci/+bug/122566421:06
jog0mtreinish: got it https://github.com/jogo/elasticRecheck/blob/master/queries.json21:08
mtreinishjog0: cool21:12
mtreinishis there still a delay on logstash? I've got one from ~30min ago that still isn't showing up in queries21:12
jog0yes21:15
jog0thats about right21:15
jog0its slow today21:15
jog0clarkb: can comment further21:15
*** jasondotstar has quit IRC21:17
jog0clarkb is working on  giving us a way to get notified when a log is in logstash21:21
mtreinishjog0: yeah that would be cool you could just queue up failures until it pops up in logstash then21:21
jog0mtreinish: I sort of try that21:23
jog0https://github.com/jogo/elasticRecheck/blob/master/elasticRecheck.py#L12921:24
jog0mtreinish: its pretty hacky but slightly works21:24
mtreinishjog0: yeah it would be better if you could just get a notification that it was ready to check21:26
jog0mtreinish: yup thats the plan21:29
*** mriedem has quit IRC21:30
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Add meta-config via local.conf  https://review.openstack.org/4676821:32
*** fbo is now known as fbo_away21:37
*** burt has joined #openstack-qa21:47
mtreinishjog0: so for adding a query just do a push?21:49
*** dkranz has quit IRC21:50
jog0mtreinish: yeah21:50
jog0make sure its proper json first though21:50
jog0but  Ithink push should work for you21:50
jog0mtreinish: if that doesn't work let me know21:51
jog0mtreinish: oh and I am adding the queries into a comment on each bug so others can use it21:55
mtreinishjog0: yeah I was going to do that too21:57
mtreinishright now I'm trying to figure out why the script says I'm missing a deliminator but it works fine manually21:57
mtreinish(loading the json from from a read() in the python interpreter)21:57
jog0pastebinit22:01
jog0you may be missing a comma at the end of the line22:01
jog0or have an extra22:01
*** malini_afk is now known as malini22:01
mtreinishjog0: I was being dumb and running it from the wrong dir22:02
mtreinishI pushed it22:02
jog0mtreinish: \o/22:02
jog0mtreinish: so we load in the file every time, so you don't need to restart anything to use the new query22:02
mtreinishjog0: yeah I got a traceback from when I was first running it so I was starting it up again22:03
*** HenryG has quit IRC22:05
*** thomasbiege1 has quit IRC22:09
*** burt has quit IRC22:13
mtreinishjog0: do you think that http://logs.openstack.org/22/46822/1/check/gate-tempest-devstack-vm-full/6f1ebf1/logs/screen-c-vol.txt.gz#_2013-09-16_21_57_14_834 should also be under: https://bugs.launchpad.net/cinder/+bug/119196022:15
mtreinishif so we'll have to change the query for that bug22:15
mtreinishjog0: I'm heading out for today I'll be back tomorrow22:17
openstackgerritA change was merged to openstack/tempest: Sync latest versions of oslo incubator  https://review.openstack.org/4677722:18
jog0mtreinish: hmm yes22:20
jog0I will change it22:20
jog0mtreinish: o/22:20
*** jhenner has quit IRC22:21
*** jhenner has joined #openstack-qa22:21
*** jhenner has quit IRC22:31
openstackgerritChangBo Guo proposed a change to openstack/tempest: Add keystone user-update test  https://review.openstack.org/4647722:33
*** weshay has quit IRC22:55
*** giulivo has quit IRC23:02
*** cyeoh has quit IRC23:27
*** atiwari has quit IRC23:39

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