| clarkb | the way I see percolation possibly workign in logstash is as a post output action, so that you can do additional filtering and percolate on what falls through | 00:01 |
|---|---|---|
| clarkb | but logstash doesn't do anything like that today | 00:01 |
| clarkb | however logstash and elasticsearch are now best buddies so maybe we will see tighter coupling between the two | 00:01 |
| jog0 | yeah | 00:04 |
| giulivo | ping sdague mtreinish | 00:35 |
| openstackgerrit | Russell Sim proposed a change to openstack/tempest: Make the admin role configurable within tempest. https://review.openstack.org/46396 | 00:37 |
| *** sthaha has joined #openstack-qa | 00:38 | |
| *** sthaha has joined #openstack-qa | 00:38 | |
| *** malini_afk is now known as malini | 00:51 | |
| *** HenryG has joined #openstack-qa | 00:59 | |
| *** jog0 has quit IRC | 01:00 | |
| *** jog0 has joined #openstack-qa | 01:01 | |
| *** dkranz has joined #openstack-qa | 01:06 | |
| *** nosnos has joined #openstack-qa | 01:06 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-dev/devstack: Enable tenant isolation to tempest for neutron https://review.openstack.org/46314 | 01:27 |
| *** fandikurnia01 has joined #openstack-qa | 01:29 | |
| jog0 | clarkb: I expect to have a working first pass tomorrow | 01:32 |
| openstackgerrit | Russell Sim proposed a change to openstack/tempest: Make the admin role configurable within tempest https://review.openstack.org/46396 | 01:38 |
| openstackgerrit | Giulio Fidente proposed a change to openstack-infra/devstack-gate: extend the floating ips range for neutron tests https://review.openstack.org/46403 | 01:48 |
| *** fandikurnia01 has quit IRC | 01:59 | |
| openstackgerrit | A change was merged to openstack/tempest: added an api test for security_groups https://review.openstack.org/44607 | 02:00 |
| *** fandikurnia01 has joined #openstack-qa | 02:04 | |
| giulivo | ping clarkb | 02:05 |
| clarkb | giulivo: hi | 02:06 |
| giulivo | thanks for helping there | 02:06 |
| giulivo | so you're absolutely right with the allocation ips starting from 0 | 02:06 |
| clarkb | giulivo: technically the ip and mask you gave is correct | 02:06 |
| clarkb | it is just easier on my brain when we deal with boundaries :) | 02:07 |
| giulivo | but do I get it right you prefer the option not to be set only for the neutron vm? | 02:07 |
| clarkb | giulivo: yes, I think that if that is an option that nova network understands we should set it for nova network as well | 02:07 |
| giulivo | yeah I thought about it too | 02:07 |
| clarkb | giulivo: that way we are consistent and don't have to remember which things are different when debugging the various tests | 02:07 |
| giulivo | but apparently, because we have less tests for nova network | 02:08 |
| giulivo | this wasn't failing there | 02:08 |
| giulivo | but yeah I agree with it being consistent | 02:08 |
| clarkb | right, I don't expect it to fix a problem, I am more interested in consistency | 02:08 |
| giulivo | will do thanks | 02:08 |
| clarkb | no problem | 02:08 |
| giulivo | fingers crossed about it fixing the actual problem | 02:08 |
| giulivo | if you look at the logs http://logs.openstack.org/30/46330/1/check/gate-tempest-devstack-vm-neutron/87ca515/console.html | 02:09 |
| giulivo | the "no more ip addresses" all are from the same network id, which upper in the setup | 02:09 |
| giulivo | matched the external network id | 02:10 |
| giulivo | for every test class a new router will have a leg on that network and when run in parallel ... | 02:10 |
| openstackgerrit | Giulio Fidente proposed a change to openstack-infra/devstack-gate: extend the floating ips range (for neutron tests to pass) https://review.openstack.org/46403 | 02:14 |
| *** fandikurnia01 has quit IRC | 02:25 | |
| *** giulivo has quit IRC | 02:28 | |
| *** anteaya has quit IRC | 02:32 | |
| *** cyeoh has quit IRC | 02:36 | |
| openstackgerrit | Russell Sim proposed a change to openstack/tempest: Make the admin role configurable within tempest https://review.openstack.org/46396 | 03:08 |
| openstackgerrit | minsel proposed a change to openstack/tempest: Handling network resources in tenant isolation https://review.openstack.org/45578 | 03:19 |
| *** malini is now known as malini_afk | 03:28 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Do not merge test of neutron tenant isolation. https://review.openstack.org/46330 | 03:50 |
| *** fandikurnia01 has joined #openstack-qa | 03:58 | |
| *** fandikurnia01 has quit IRC | 04:15 | |
| *** fandikurnia01 has joined #openstack-qa | 04:16 | |
| *** stevebaker has quit IRC | 04:19 | |
| *** adam_g has quit IRC | 04:19 | |
| *** russellb has quit IRC | 04:19 | |
| *** zigo_ has joined #openstack-qa | 04:19 | |
| *** zigo has quit IRC | 04:19 | |
| *** fandikurnia01 has quit IRC | 04:21 | |
| *** russellb has joined #openstack-qa | 04:22 | |
| *** stevebaker has joined #openstack-qa | 04:22 | |
| *** cyeoh_ has joined #openstack-qa | 04:25 | |
| *** cyeoh_ is now known as cyeoh | 04:25 | |
| raies | any one from tempest ? | 04:26 |
| *** madan has joined #openstack-qa | 04:42 | |
| *** SergeyLukjanov has joined #openstack-qa | 04:49 | |
| *** prekarat has joined #openstack-qa | 04:55 | |
| *** prekarat has quit IRC | 05:00 | |
| *** SergeyLukjanov has quit IRC | 05:08 | |
| raies | jaypipes: please look https://review.openstack.org/#/c/45372/ and https://review.openstack.org/#/c/44870/ they are not getting merged | 05:12 |
| *** prekarat has joined #openstack-qa | 05:14 | |
| raies | sdague: please look https://review.openstack.org/#/c/45372/ and https://review.openstack.org/#/c/44870/ they are not getting merged | 05:15 |
| *** prekarat has quit IRC | 05:15 | |
| *** prekarat has joined #openstack-qa | 05:15 | |
| raies | mkoderer: please look https://review.openstack.org/#/c/45372/ and https://review.openstack.org/#/c/44870/ they are not getting merged | 05:17 |
| mkoderer | raies: it's strange | 05:26 |
| mkoderer | looks like a problem with zuul | 05:26 |
| mkoderer | raies: https://review.openstack.org/#/c/44870/ had a zuul run now | 05:27 |
| mkoderer | raies: u need to rebase | 05:27 |
| *** yfried has quit IRC | 05:27 | |
| raies | mkoderer: ok I will rebase it now | 05:28 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Bulk Subnets and Ports creation https://review.openstack.org/44870 | 05:33 |
| raies | mkoderer: please check again https://review.openstack.org/44870 | 05:35 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding negative test for port https://review.openstack.org/45372 | 05:39 |
| *** madan has quit IRC | 05:42 | |
| *** sthaha has quit IRC | 06:00 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Bulk Subnets and Ports creation https://review.openstack.org/44870 | 06:03 |
| raies | mkoderer: please check again https://review.openstack.org/44870 | 06:04 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding negative test for port https://review.openstack.org/45372 | 06:10 |
| *** raies has quit IRC | 06:10 | |
| *** raies has joined #openstack-qa | 06:11 | |
| raies | mkoderer: please check https://review.openstack.org/#/c/44870/ | 06:12 |
| *** prekarat1 has joined #openstack-qa | 06:30 | |
| *** prekarat has quit IRC | 06:30 | |
| *** prekarat1 has quit IRC | 06:48 | |
| *** giulivo has joined #openstack-qa | 07:04 | |
| *** prekarat has joined #openstack-qa | 07:07 | |
| raies | sdague ? | 07:11 |
| raies | sdague: please approve https://review.openstack.org/#/c/44870/ as ,jenkins merge was failed | 07:13 |
| *** bashok has joined #openstack-qa | 07:13 | |
| openstackgerrit | Arata Notsu proposed a change to openstack/tempest: Add "region" config for each service https://review.openstack.org/41026 | 07:23 |
| *** HenryG has quit IRC | 07:30 | |
| *** HenryG has joined #openstack-qa | 07:31 | |
| *** afazekas has joined #openstack-qa | 07:31 | |
| openstackgerrit | Arata Notsu proposed a change to openstack/tempest: Add "region" config for each service https://review.openstack.org/41026 | 07:32 |
| *** giulivo has quit IRC | 07:41 | |
| *** turul_ has joined #openstack-qa | 07:44 | |
| *** thomasbiege has joined #openstack-qa | 07:44 | |
| *** afazekas is now known as afazekas__ | 07:44 | |
| *** turul_ is now known as afazekas | 07:44 | |
| afazekas | bashok: http://fpaste.org/39289/79058333/ | 07:45 |
| afazekas | FYI: with neutron at the moment the allow_tenant_isolation = False is required in the tempest.conf | 07:46 |
| bashok | afazekas thanks for the information, thats really helpful | 07:46 |
| afazekas | the paste has an additional new line after nosetests :( | 07:48 |
| openstackgerrit | Jun Jie Nan proposed a change to openstack-dev/devstack: Pass no proxy to skip proxy using for wget and curl https://review.openstack.org/46447 | 07:51 |
| *** thomasbiege has quit IRC | 07:59 | |
| *** fbo_away is now known as fbo | 08:05 | |
| *** yassine has joined #openstack-qa | 08:06 | |
| *** thomasbiege has joined #openstack-qa | 08:09 | |
| *** thomasbiege has quit IRC | 08:23 | |
| raies | afazekas__ : hi https://review.openstack.org/#/c/44870/ was not being merged due to jenkins problem please have a look | 08:24 |
| afazekas__ | I will | 08:25 |
| raies | afazekas__ : please review all unreviewed in the list - https://review.openstack.org/#/q/owner:raiesmh08,n,z | 08:33 |
| *** Madan has joined #openstack-qa | 08:36 | |
| afazekas | raies: I had leave for while | 08:36 |
| afazekas | load blancer :)))) | 08:36 |
| raies | yes | 08:37 |
| afazekas | cool | 08:38 |
| *** afazekas has quit IRC | 08:38 | |
| *** afazekas__ has quit IRC | 08:42 | |
| *** giulivo has joined #openstack-qa | 08:43 | |
| *** ianw has quit IRC | 08:47 | |
| raies | mkoderer: hi are you there ? | 08:50 |
| *** prekarat1 has joined #openstack-qa | 08:53 | |
| *** prekarat has quit IRC | 08:56 | |
| *** madan_ has joined #openstack-qa | 09:05 | |
| mkoderer | raies: I am currently quite busy, I'll come back to you | 09:05 |
| raies | ok | 09:05 |
| *** yfried has joined #openstack-qa | 09:11 | |
| *** tkammer has joined #openstack-qa | 09:12 | |
| *** adam_g has joined #openstack-qa | 09:15 | |
| *** adam_g has joined #openstack-qa | 09:15 | |
| *** yfried has quit IRC | 09:18 | |
| *** prekarat1 has quit IRC | 09:25 | |
| *** prekarat has joined #openstack-qa | 09:39 | |
| *** SergeyLukjanov has joined #openstack-qa | 09:41 | |
| *** afazekas__ has joined #openstack-qa | 09:42 | |
| afazekas__ | raises: until the full gate not working you should always use type=smoke with the neutron tests | 09:52 |
| afazekas__ | raise: ^^ | 09:54 |
| afazekas__ | raies: ^^ | 09:54 |
| *** SergeyLukjanov has quit IRC | 09:54 | |
| *** prekarat has quit IRC | 09:54 | |
| afazekas__ | sorry | 09:54 |
| openstackgerrit | Dave Cahill proposed a change to openstack-dev/devstack: Allow setting necessary config options for MidoNet https://review.openstack.org/46465 | 09:56 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Network load balancer testing https://review.openstack.org/45230 | 10:02 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Network load balancer testing https://review.openstack.org/45230 | 10:04 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding clinet functions and test cases for pool https://review.openstack.org/45602 | 10:08 |
| *** SergeyLukjanov has joined #openstack-qa | 10:10 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding member operation test cases https://review.openstack.org/45627 | 10:11 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding health monitoring test case https://review.openstack.org/45675 | 10:16 |
| *** raies has quit IRC | 10:21 | |
| *** mkollaro has joined #openstack-qa | 10:23 | |
| *** nosnos has quit IRC | 10:26 | |
| *** nosnos has joined #openstack-qa | 10:27 | |
| *** nosnos has quit IRC | 10:27 | |
| *** nosnos has joined #openstack-qa | 10:28 | |
| *** nosnos has quit IRC | 10:35 | |
| *** thomasbiege has joined #openstack-qa | 10:35 | |
| *** nosnos has joined #openstack-qa | 10:36 | |
| *** nosnos has quit IRC | 10:40 | |
| *** afazekas__ is now known as afazekas_food | 11:16 | |
| *** jhenner has joined #openstack-qa | 11:19 | |
| *** thomasbiege2 has joined #openstack-qa | 11:21 | |
| *** thomasbiege has quit IRC | 11:24 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Stop doing special things with setuptools https://review.openstack.org/41281 | 11:27 |
| *** afazekas_food is now known as afazekas | 11:29 | |
| openstackgerrit | A change was merged to openstack/tempest: Add logging to the python-clients https://review.openstack.org/43868 | 11:31 |
| *** thomasbiege2 has quit IRC | 11:35 | |
| *** malini_afk is now known as malini | 11:48 | |
| *** tkammer has quit IRC | 11:51 | |
| *** adalbas has joined #openstack-qa | 11:52 | |
| *** bashok_ has joined #openstack-qa | 11:53 | |
| *** bashok has quit IRC | 11:56 | |
| openstackgerrit | ChangBo Guo proposed a change to openstack/tempest: Add keystone user-update test https://review.openstack.org/46477 | 11:56 |
| *** mkollaro has quit IRC | 11:59 | |
| openstackgerrit | ChangBo Guo proposed a change to openstack/tempest: Add keystone user-update test https://review.openstack.org/46477 | 11:59 |
| *** weshay has joined #openstack-qa | 12:03 | |
| *** madan_ has quit IRC | 12:17 | |
| *** Madan has quit IRC | 12:18 | |
| *** stevebaker has quit IRC | 12:24 | |
| *** stevebaker has joined #openstack-qa | 12:24 | |
| *** SergeyLukjanov has quit IRC | 12:29 | |
| *** SergeyLukjanov has joined #openstack-qa | 12:42 | |
| *** sthaha has joined #openstack-qa | 12:46 | |
| *** sthaha has quit IRC | 12:46 | |
| openstackgerrit | ChangBo Guo proposed a change to openstack/tempest: Add keystone user-update test https://review.openstack.org/46477 | 12:47 |
| *** sthaha has joined #openstack-qa | 12:47 | |
| *** sthaha has joined #openstack-qa | 12:47 | |
| *** sthaha has quit IRC | 12:48 | |
| *** sthaha has joined #openstack-qa | 12:54 | |
| *** sthaha has joined #openstack-qa | 12:54 | |
| *** sthaha has quit IRC | 12:54 | |
| *** sthaha has joined #openstack-qa | 12:54 | |
| *** malini is now known as malini_afk | 13:00 | |
| *** anteaya has joined #openstack-qa | 13:09 | |
| openstackgerrit | DennyZhang proposed a change to openstack/tempest: Add missing import of 'subunit' in test-requirements.txt https://review.openstack.org/46324 | 13:17 |
| *** mkollaro has joined #openstack-qa | 13:25 | |
| openstackgerrit | A change was merged to openstack/tempest: Bulk Subnets and Ports creation https://review.openstack.org/44870 | 13:28 |
| *** SergeyLukjanov has quit IRC | 13:32 | |
| openstackgerrit | ChangBo Guo proposed a change to openstack/tempest: Add keystone user-update test https://review.openstack.org/46477 | 13:38 |
| openstackgerrit | DennyZhang proposed a change to openstack/tempest: Add missing import of 'subunit' in test-requirements.txt https://review.openstack.org/46324 | 13:42 |
| *** yfried has joined #openstack-qa | 13:48 | |
| *** jasondotstar has joined #openstack-qa | 13:55 | |
| *** malini_afk is now known as malini | 13:58 | |
| *** yfried has quit IRC | 14:00 | |
| *** mriedem has joined #openstack-qa | 14:02 | |
| openstackgerrit | ChangBo Guo proposed a change to openstack/tempest: Add keystone user-update test https://review.openstack.org/46477 | 14:14 |
| *** bashok_ has quit IRC | 14:25 | |
| *** bashok_ has joined #openstack-qa | 14:25 | |
| *** adalbas has quit IRC | 14:27 | |
| *** mlavalle has joined #openstack-qa | 14:28 | |
| *** thomasbiege has joined #openstack-qa | 14:29 | |
| *** thomasbiege has quit IRC | 14:30 | |
| mlavalle | mtreinish: are you around? | 14:32 |
| *** SergeyLukjanov has joined #openstack-qa | 14:35 | |
| mtreinish | mlavalle: yep, what's up? | 14:36 |
| mlavalle | mtreinish: I went through the logs of yesterday's run. I counted the number of times a router was created and the number of times a router was deleted by isolated credentials. There is a huge mismatch | 14:37 |
| mlavalle | mtreinish: There are 53 creations and only 29 deletions. So my suspicion is that in some of the teardown methods that the tests are running, they are not cleaning up. That is why we are running out of addressing space | 14:39 |
| *** atiwari has joined #openstack-qa | 14:40 | |
| mlavalle | mtreinish: I added a LOG.info statement to the _clear_network_resources method to be able to identify who is the culprit. Would you run the job again and send me the pointer to the logs? I pushed free code to gerrit last night | 14:41 |
| afazekas | mtreinish: can you review it : https://review.openstack.org/#/c/35165/ | 14:41 |
| mtreinish | mlavalle: I already did: http://logs.openstack.org/30/46330/2/check/gate-tempest-devstack-vm-neutron/8dc1341/ | 14:42 |
| mtreinish | mlavalle: I could see that happening but I think we call tearDownClass everywhere. (afazekas add a check to ensure we call it) | 14:43 |
| mtreinish | afazekas: I've got that open in a tab I'm going to go through it shortly | 14:44 |
| afazekas | We should be able to see in the log if the teardown class not reached | 14:45 |
| *** adalbas has joined #openstack-qa | 14:45 | |
| mlavalle | mtreinish: thanks. Yeah, when I say the culprit, that potentially includes my code. With the log files I should be able to debug this. I'll keep you posted | 14:45 |
| mtreinish | mlavalle: ok | 14:46 |
| afazekas | is the class skips happens before or after the super call ? | 14:47 |
| mtreinish | afazekas: I think we normally put them after but I'm not sure | 14:49 |
| afazekas | tearDownClass call testing only happens if at least one test_ method ran happened | 14:51 |
| afazekas | tearDownClass does not happens after class skip | 14:51 |
| mlavalle | mtreinish, afazekas: I just went quickly through the latest log. I find 'Created isolated network resources for ' 36 times, while I find 'Cleared isolated network resources' only 25 times…. | 15:01 |
| mtreinish | mlavalle: ok, so you think that we're probably missing a few clear_isolated_creds() calls from tearDownClass? | 15:03 |
| mlavalle | mtreinsh, afazekas: the good news is that with this log statement I added to the clear_network_rersources method, I will be able to match the creations with the deletions and know what tests classes are missing the deletion invocation. I will do that slowly later today | 15:03 |
| mlavalle | mtreinish: yes, that is my current thinking. But I will be sure after I comb the log slowly later today | 15:04 |
| afazekas | I think it is possible on skipped classes, I need to check it | 15:04 |
| mlavalle | mtreinish, afazekas: an the numbers make sense. We have about 14 available ports in a /28 subnet. So if we miss ~11 tear downs, we exhausted the addressing space | 15:05 |
| afazekas | IMHO we should do the skip decision before super call | 15:05 |
| *** sthaha has quit IRC | 15:07 | |
| mtreinish | afazekas: that makes sense, but I'm not sure how many skips we have in the smoke runs | 15:07 |
| mlavalle | mtreinish, afazekas: I still think that when we start running with neutron in parallel, we will need a /28 cidr in the public net. But at this point I just don't want to mask a potential bug or bugs in tearDown methods by throwing ports at the problem ;-) | 15:10 |
| mlavalle | mtreinish, afazekas: I meant a /24 cidd when we run in parallel | 15:11 |
| afazekas | suspects http://www.fpaste.org/39408/90852101/ | 15:13 |
| afazekas | xml/json can duplicate | 15:14 |
| mtreinish | this is what I got by grepping the log: http://paste.openstack.org/show/47011/ | 15:14 |
| mtreinish | those classes didn't have a delete | 15:14 |
| mtreinish | mlavalle: yeah your right we want to plug the leak before we up the cidr | 15:15 |
| afazekas | looks swift does not have clear call on the tearDownclass chain | 15:21 |
| mlavalle | afazekas: yeah, that makes sense with what I saw last night. It was 11pm, though, and my wife was already yelling at me, so I didn't have to dig deeper | 15:22 |
| mtreinish | afazekas: why am I not surprised. If there's a random problem with something it's always either swift or keystone tests doing something dumb. | 15:23 |
| mlavalle | so I didn't want to point fingers with a really tired brain -;) | 15:24 |
| mtreinish | mlavalle: I'll push out a fix in one sec (it looks like a 2 liner) | 15:28 |
| mlavalle | mtreinish: cool! | 15:29 |
| *** jasondotstar has quit IRC | 15:41 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add tearDownClass to base swift test class https://review.openstack.org/46512 | 15:41 |
| mtreinish | mlavalle: ^^^ | 15:41 |
| mlavalle | mtreinish: thanks for the help! | 15:42 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: VolumesListTest may hide configuration issues https://review.openstack.org/46514 | 15:50 |
| afazekas | I need to go I would really appreciate if someone else could review this change https://review.openstack.org/#/c/35165/ | 15:51 |
| *** afazekas has quit IRC | 15:51 | |
| *** jasondotstar has joined #openstack-qa | 15:59 | |
| *** malini is now known as malini_afk | 16:02 | |
| *** malini_afk is now known as malini | 16:09 | |
| *** marun has quit IRC | 16:10 | |
| *** marun has joined #openstack-qa | 16:11 | |
| *** SergeyLukjanov has quit IRC | 16:26 | |
| *** afazekas has joined #openstack-qa | 16:31 | |
| *** comstud is now known as bearhands | 16:32 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: VolumesListTest may hide configuration issues https://review.openstack.org/46514 | 16:35 |
| *** hemnafk is now known as hemna | 17:04 | |
| openstackgerrit | Sean McCully proposed a change to openstack/tempest: Changes Test, port is not gauranteed unused https://review.openstack.org/45459 | 17:07 |
| *** bashok_ is now known as bashok_tired | 17:13 | |
| *** yassine has quit IRC | 17:15 | |
| *** mkollaro has quit IRC | 17:23 | |
| *** mlavalle has quit IRC | 17:24 | |
| *** malini is now known as malini_afk | 17:26 | |
| *** bashok_ has joined #openstack-qa | 17:26 | |
| *** bashok_tired has quit IRC | 17:30 | |
| openstackgerrit | Salvatore Orlando proposed a change to openstack-dev/devstack: Colorize Neutron log output and refactor log setup code https://review.openstack.org/43006 | 17:32 |
| *** jasondotstar has quit IRC | 17:33 | |
| *** jhenner has quit IRC | 17:45 | |
| openstackgerrit | A change was merged to openstack/tempest: Make the admin role configurable within tempest https://review.openstack.org/46396 | 17:58 |
| *** burt has quit IRC | 18:00 | |
| *** giulivo has quit IRC | 18:10 | |
| *** malini_afk is now known as malini | 18:17 | |
| *** bashok_ is now known as bashok_zzzz | 18:36 | |
| *** atiwari has quit IRC | 18:58 | |
| openstackgerrit | A change was merged to openstack/tempest: Adds disk_format parameter to upload_volume method in volumes client https://review.openstack.org/44374 | 19:19 |
| mtreinish | dkranz: If you get a chance I think this will fix a big recheck bug: https://review.openstack.org/#/c/35165/ | 19:20 |
| *** bashok_ has joined #openstack-qa | 19:22 | |
| *** bashok_zzzz has quit IRC | 19:26 | |
| openstackgerrit | A change was merged to openstack/tempest: Add tearDownClass to base swift test class https://review.openstack.org/46512 | 19:26 |
| dkranz | mtreinish: OK, done. I'm not too worried about verbosity on failure. Better to err with too much info in that case. | 19:30 |
| mtreinish | dkranz: it'll always logs to the log file. It was just the console output I was worried about. | 19:35 |
| dkranz | mtreinish: I know but it can't hurt to see relevant info in the log at the point of failure | 19:36 |
| dkranz | mtreinish: Not a big deal either way | 19:36 |
| mtreinish | dkranz: yeah that's what I thought which is why I changed it to a +2. | 19:36 |
| dkranz | mtreinish: Great. I hope it helps :) | 19:37 |
| *** mlavalle has joined #openstack-qa | 19:38 | |
| *** jhenner has joined #openstack-qa | 19:48 | |
| *** bashok_ has quit IRC | 19:54 | |
| openstackgerrit | A change was merged to openstack/tempest: task_state must be consider before many action https://review.openstack.org/35165 | 19:59 |
| *** atiwari has joined #openstack-qa | 20:06 | |
| *** jasondotstar has joined #openstack-qa | 20:07 | |
| mtreinish | mlavalle: can you rebase your patch so I can test if the leak fix was enough | 20:12 |
| mlavalle | mtreinish: I can't do it from this machine. I will do it later today | 20:17 |
| mlavalle | mtreinish: i'll ping you | 20:17 |
| mtreinish | mlavalle: ok | 20:22 |
| mtreinish | mlavalle: all you need is to push rebase on the gerrit page though | 20:22 |
| openstackgerrit | minsel proposed a change to openstack/tempest: Handling network resources in tenant isolation https://review.openstack.org/45578 | 20:23 |
| mlavalle | mtreinish: ahhh!!!…… Done | 20:24 |
| *** jasondotstar has quit IRC | 20:24 | |
| mtreinish | mlavalle: ok thanks | 20:24 |
| *** jasondotstar has joined #openstack-qa | 20:24 | |
| *** jasondotstar has quit IRC | 20:25 | |
| mlavalle | mtreinish: let me know if we hit another wall….. if we do, please put a pointer in the comments of the patch set, so i can debug it later. Thanks! | 20:25 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Do not merge test of neutron tenant isolation. https://review.openstack.org/46330 | 20:25 |
| openstackgerrit | Arvind Tiwari proposed a change to openstack/tempest: Disabling a test so that change 46123 can be reviewed https://review.openstack.org/46341 | 20:38 |
| mtreinish | mlavalle: a new failure: https://jenkins01.openstack.org/job/gate-tempest-devstack-vm-neutron/11132/console I'll add a comment again on the review | 20:45 |
| mlavalle | mtreinish: cool. i'll debug it later and try again. After taking a quick look, I think we made progress….. it's a different error….. and much more limited in the numbers of tests affected | 20:48 |
| mtreinish | mlavalle: yeah it looks like a cleanup problem on teardown it says ports are in use | 20:49 |
| mlavalle | mtreinish: yeah, it's a cleaning up problem….. have a nice weekend…. :-) | 20:50 |
| *** SergeyLukjanov has joined #openstack-qa | 20:53 | |
| *** marun has quit IRC | 20:56 | |
| *** EspressoLuv has quit IRC | 21:06 | |
| *** openstackgerrit has quit IRC | 21:14 | |
| *** mriedem has quit IRC | 21:20 | |
| *** mlavalle has quit IRC | 21:41 | |
| *** giulivo has joined #openstack-qa | 21:41 | |
| *** weshay has quit IRC | 21:51 | |
| atiwari | mtreinish, regarding https://review.openstack.org/#/c/46341/, I am not disabling the test just modified it so that it would not failed with new response code | 21:55 |
| atiwari | I think I should change the commit message. which is still saying "Disabling a test so that change 46123 can be reviewed" | 21:56 |
| atiwari | wd that be OK? | 21:56 |
| mtreinish | atiwari: that's not valid if you're making a change to a publish api we have a policy on getting that merged | 21:56 |
| mtreinish | the commit making the change has to have a +2 from a core on the project | 21:56 |
| mtreinish | then we merge the change to skip it with a bug to track the change | 21:56 |
| mtreinish | you merge the commit | 21:56 |
| mtreinish | and then unskip the test making the api change to the test | 21:57 |
| mtreinish | I forget where we have that published but here is a presentation that gives a basic outline for it: http://dague.net/presentations/tempest-101/#/10 | 21:58 |
| atiwari | I am totally lost, you want us to merge the https://review.openstack.org/#/c/46123 for which I am making the change in thsi test? | 21:59 |
| mtreinish | atiwari: no, that change has to have a +2 to show support for the change from keystone core | 22:00 |
| *** SergeyLukjanov has quit IRC | 22:00 | |
| mtreinish | then we'll approve a skip for the test that is expecting the old response code | 22:00 |
| *** mlavalle has joined #openstack-qa | 22:00 | |
| mtreinish | which will enable you to merge that keystone commit | 22:00 |
| atiwari | how come that will have +2, Jenkins will fail | 22:01 |
| atiwari | without test fix | 22:01 |
| mtreinish | the +2 from a core contributor is independent of jenkins passing | 22:01 |
| atiwari | ok, sir I will work with keystone then. | 22:02 |
| atiwari | thanks | 22:02 |
| *** mlavalle has quit IRC | 22:54 | |
| *** giulivo has quit IRC | 23:00 | |
| *** fbo is now known as fbo_away | 23:42 | |
| *** hemna is now known as hemnafk | 23:45 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!