| *** cyeoh has quit IRC | 00:04 | |
| *** cyeoh has joined #openstack-qa | 00:05 | |
| *** torandu has quit IRC | 00:05 | |
| *** hemna is now known as hemnafk | 00:06 | |
| *** krtaylor has quit IRC | 00:12 | |
| *** krtaylor has joined #openstack-qa | 00:13 | |
| *** LiangZheng has joined #openstack-qa | 00:17 | |
| *** openstack has joined #openstack-qa | 00:41 | |
| *** mberwanger has joined #openstack-qa | 00:45 | |
| *** jerryz has joined #openstack-qa | 00:49 | |
| *** marun has quit IRC | 00:55 | |
| *** mriedem has joined #openstack-qa | 00:57 | |
| *** mberwanger has quit IRC | 01:00 | |
| *** jerryz has quit IRC | 01:15 | |
| *** jerryz has joined #openstack-qa | 01:15 | |
| *** LiangZheng has joined #openstack-qa | 01:18 | |
| *** LiangZheng has quit IRC | 01:22 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: test_servers_whitebox - using official client https://review.openstack.org/40396 | 01:52 |
|---|---|---|
| *** adalbas has quit IRC | 02:08 | |
| *** mriedem has quit IRC | 02:12 | |
| *** marun has joined #openstack-qa | 02:14 | |
| *** LiangZheng has joined #openstack-qa | 02:19 | |
| *** marun has quit IRC | 02:20 | |
| *** marun has joined #openstack-qa | 02:21 | |
| *** LiangZheng has quit IRC | 02:23 | |
| *** marun has quit IRC | 02:29 | |
| *** marun has joined #openstack-qa | 02:31 | |
| *** LiangZheng has joined #openstack-qa | 02:48 | |
| *** LiangZheng has joined #openstack-qa | 02:49 | |
| *** LiangZhe_ has joined #openstack-qa | 02:49 | |
| *** LiangZheng has quit IRC | 02:49 | |
| *** LiangZhe_ has quit IRC | 02:49 | |
| *** LiangZheng has joined #openstack-qa | 02:50 | |
| *** LiangZhe_ has joined #openstack-qa | 02:51 | |
| *** LiangZheng has quit IRC | 02:51 | |
| *** LiangZheng has joined #openstack-qa | 02:52 | |
| *** LiangZhe_ has quit IRC | 02:52 | |
| *** jasondotstar has quit IRC | 02:57 | |
| *** SergeyLukjanov has joined #openstack-qa | 03:22 | |
| *** HenryG has quit IRC | 03:29 | |
| *** SergeyLukjanov has quit IRC | 04:13 | |
| *** jerryz has quit IRC | 04:31 | |
| *** marun has quit IRC | 04:34 | |
| *** kashyap` is now known as kashyap | 04:50 | |
| *** afazekas has joined #openstack-qa | 04:56 | |
| *** afazekas has quit IRC | 05:17 | |
| *** fungi has quit IRC | 05:31 | |
| *** thomasbiege has joined #openstack-qa | 05:43 | |
| *** fungi has joined #openstack-qa | 05:45 | |
| *** thomasbiege has quit IRC | 05:47 | |
| openstackgerrit | Christian Berendt proposed a change to openstack-dev/devstack: added new tag for testonly packages in lists https://review.openstack.org/38127 | 05:50 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: prot test_images and test_server_actions part1 https://review.openstack.org/39621 | 05:56 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_images and test_server_actions into v3 part2 https://review.openstack.org/39609 | 05:56 |
| openstackgerrit | gongysh proposed a change to openstack/tempest: Fix typos in tempest/api/README.rst https://review.openstack.org/42063 | 05:59 |
| *** afazekas has joined #openstack-qa | 06:07 | |
| *** marun has joined #openstack-qa | 06:15 | |
| openstackgerrit | Christian Berendt proposed a change to openstack-dev/devstack: added new tag for testonly packages in lists https://review.openstack.org/38127 | 06:18 |
| *** afazekas has quit IRC | 06:19 | |
| *** afazekas has joined #openstack-qa | 06:40 | |
| *** marun has quit IRC | 06:47 | |
| *** jerryz has joined #openstack-qa | 07:00 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: per test_method logging https://review.openstack.org/42069 | 07:02 |
| *** jhenner has joined #openstack-qa | 07:03 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: per test_method logging https://review.openstack.org/42069 | 07:03 |
| *** thomasbiege has joined #openstack-qa | 07:06 | |
| *** jerryz has quit IRC | 07:23 | |
| *** thomasbiege has quit IRC | 07:23 | |
| *** tkammer has joined #openstack-qa | 07:24 | |
| *** thomasbiege has joined #openstack-qa | 07:27 | |
| *** thomasbiege has quit IRC | 07:29 | |
| *** thomasbiege has joined #openstack-qa | 07:30 | |
| *** sungju_ has joined #openstack-qa | 07:35 | |
| *** sungju_ has quit IRC | 07:35 | |
| *** thomasbiege has quit IRC | 07:47 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_images and test_server_actions into v3 part2 https://review.openstack.org/39609 | 08:14 |
| *** thomasbiege has joined #openstack-qa | 08:36 | |
| *** thomasbiege has quit IRC | 08:41 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: test_servers_whitebox - using official client https://review.openstack.org/40396 | 08:41 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: test_servers_whitebox - using official client https://review.openstack.org/40396 | 08:46 |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Use a global virtualenv https://review.openstack.org/40534 | 08:47 |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Stop installing PyPI things via apt https://review.openstack.org/40019 | 08:47 |
| openstackgerrit | afazekas proposed a change to openstack-infra/devstack-gate: Skip devstack/exercises by default https://review.openstack.org/42082 | 08:51 |
| * afazekas https://bugs.launchpad.net/tempest/+bug/1212405 considering skipping this test | 09:01 | |
| *** SergeyLukjanov has joined #openstack-qa | 09:26 | |
| afazekas | mtreinish: https://review.openstack.org/#/c/42069/ | 09:38 |
| *** mkollaro has joined #openstack-qa | 09:46 | |
| *** sungju_ has joined #openstack-qa | 09:57 | |
| *** sungju_ has quit IRC | 09:58 | |
| *** psedlak has joined #openstack-qa | 10:02 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_images and test_server_actions into v3 part2 https://review.openstack.org/39609 | 10:06 |
| openstackgerrit | afazekas proposed a change to openstack-infra/devstack-gate: Skip devstack/exercises by default https://review.openstack.org/42082 | 10:11 |
| *** LiangZheng has quit IRC | 10:15 | |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Use a global virtualenv https://review.openstack.org/40534 | 10:20 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port attacte_interfaces and server_address test into v3 part1 https://review.openstack.org/42099 | 10:28 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port attach_interfaces and server_address tests into v3 part2 https://review.openstack.org/42100 | 10:28 |
| *** fbo_away is now known as fbo | 10:31 | |
| openstackgerrit | afazekas proposed a change to openstack-infra/devstack-gate: Preserve sudo permissions of the stack user https://review.openstack.org/42101 | 10:31 |
| *** weshay has joined #openstack-qa | 10:58 | |
| *** tkammer has quit IRC | 11:14 | |
| *** yfried has joined #openstack-qa | 11:15 | |
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/tempest: Add a test case for instance boot from volume https://review.openstack.org/41048 | 11:18 |
| *** fbo is now known as fbo_away | 11:23 | |
| openstackgerrit | Monty Taylor proposed a change to openstack-dev/devstack: Use a global virtualenv https://review.openstack.org/40534 | 11:23 |
| openstackgerrit | Monty Taylor proposed a change to openstack-dev/devstack: Stop installing PyPI things via apt https://review.openstack.org/40019 | 11:23 |
| *** SergeyLukjanov has quit IRC | 11:41 | |
| *** weshay has quit IRC | 11:42 | |
| *** ivanzhu has joined #openstack-qa | 11:55 | |
| openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Configure the ec2 credentials explicitly in tempest https://review.openstack.org/39525 | 11:59 |
| *** woodspa has joined #openstack-qa | 12:07 | |
| *** tkammer has joined #openstack-qa | 12:07 | |
| *** adalbas has joined #openstack-qa | 12:23 | |
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/tempest: Add a test case for instance boot from volume https://review.openstack.org/41048 | 12:25 |
| *** yfried has quit IRC | 12:42 | |
| *** yfried has joined #openstack-qa | 12:48 | |
| *** HenryG has joined #openstack-qa | 12:55 | |
| openstackgerrit | afazekas proposed a change to openstack-infra/devstack-gate: Enable Load Balancer as a Service https://review.openstack.org/42119 | 13:00 |
| *** weshay has joined #openstack-qa | 13:02 | |
| *** anteaya has joined #openstack-qa | 13:06 | |
| openstackgerrit | lawrancejing proposed a change to openstack/tempest: Fixes files with wrong bitmode https://review.openstack.org/41686 | 13:06 |
| *** burt has joined #openstack-qa | 13:17 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: skip test_list_servers_filtered_by_ip_regex on neutron gate https://review.openstack.org/42125 | 13:19 |
| *** SergeyLukjanov has joined #openstack-qa | 13:21 | |
| *** SergeyLukjanov has quit IRC | 13:23 | |
| openstackgerrit | A change was merged to openstack/tempest: setUpClass/tearDownClass full chain https://review.openstack.org/39346 | 13:25 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Faster old oslo.config detection https://review.openstack.org/41733 | 13:26 |
| *** SergeyLukjanov has joined #openstack-qa | 13:28 | |
| *** mriedem has joined #openstack-qa | 13:28 | |
| *** ivanzhu has quit IRC | 13:31 | |
| *** jhenner has quit IRC | 13:35 | |
| *** krtaylor has quit IRC | 13:48 | |
| *** ivanzhu has joined #openstack-qa | 13:50 | |
| openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Configure the ec2 credentials explicitly in tempest https://review.openstack.org/39525 | 13:54 |
| mtreinish | afazekas: I think it's too early to skip because of 42609, the horizon guys seem to be on top of it. | 13:57 |
| mtreinish | sorry wrong paste buffer because of: bug 1212405 | 13:57 |
| afazekas | mtreinish: the horizon revert is merged: https://review.openstack.org/#/c/42033/ | 13:59 |
| afazekas | but the real is is somewhere else.. | 13:59 |
| afazekas | but, hopefully at least it solves the recheck issue | 14:00 |
| *** jasondotstar has joined #openstack-qa | 14:02 | |
| *** ivanzhu has quit IRC | 14:06 | |
| adalbas | mtreinish, let me bring the subject here. it looks like the group_regex in .testr.conf is not conditioning a class to a single thread, when running in parallel | 14:07 |
| mtreinish | adalbas: yes it does | 14:09 |
| adalbas | in that specific case for tempest.api.object_storage.test_account_services, it creates different threads for the same class, what is causing the test failures | 14:09 |
| *** HenryG_ has joined #openstack-qa | 14:09 | |
| adalbas | mtreinish, i ran testr run --parallel tempest.api.object_storage.test_account_services.AccountTest | 14:09 |
| *** SergeyLukjanov has quit IRC | 14:10 | |
| mtreinish | but you need testrepository >=0.0.17 for the option to do anything | 14:10 |
| adalbas | and i have different worker-# tags for each test | 14:10 |
| adalbas | mtreinish, ok. im on 0.0.14 | 14:11 |
| adalbas | let me upgrade it. | 14:11 |
| *** HenryG has quit IRC | 14:12 | |
| mtreinish | adalbas: is that just on your local system? | 14:14 |
| adalbas | mtreinish, right. | 14:14 |
| adalbas | i upgrade it and it worked | 14:14 |
| adalbas | the tests passed | 14:14 |
| adalbas | but i think in the gate, we had it running in different threads. | 14:14 |
| adalbas | mtreinish, let me see if i find the version of testr it's using | 14:15 |
| *** torandu has joined #openstack-qa | 14:15 | |
| mtreinish | adalbas: looking at the logs it looks like it pulled in 0.0.17 | 14:20 |
| adalbas | mtreinish, yeah, i checkd here, the race condition is between two different classes; AccountTest and ObjectTest | 14:21 |
| adalbas | it may be using the same user | 14:21 |
| mtreinish | adalbas: ah ok | 14:21 |
| mtreinish | yeah there is no tenant isolation on the swift tests yet | 14:22 |
| mtreinish | when I turned it on the tests failed | 14:22 |
| adalbas | so, when getting the container info, there s a mismatch of expected values | 14:22 |
| *** SergeyLukjanov has joined #openstack-qa | 14:22 | |
| mtreinish | but I guess we need it so I'll start working on that | 14:22 |
| mtreinish | adalbas: in the short term we can just lock the offending tests that race | 14:22 |
| mtreinish | do you want a push out a patch for that while I work on the tenant isolation? | 14:23 |
| adalbas | mtreinish, ok. how can we do that? | 14:23 |
| mtreinish | adalbas: take a look at: test_aggregates.py | 14:23 |
| adalbas | mtreinish, sure | 14:23 |
| *** ivanzhu has joined #openstack-qa | 14:24 | |
| *** torandu has quit IRC | 14:26 | |
| *** sthaha has quit IRC | 14:27 | |
| *** adalbas has quit IRC | 14:30 | |
| *** torandu has joined #openstack-qa | 14:30 | |
| *** torandu has left #openstack-qa | 14:31 | |
| *** tkammer has quit IRC | 14:32 | |
| openstackgerrit | afazekas proposed a change to openstack-infra/devstack-gate: Using the jenkis user for tempest run https://review.openstack.org/42101 | 14:39 |
| *** torandu has joined #openstack-qa | 14:42 | |
| *** mkollaro has quit IRC | 14:44 | |
| *** tkammer has joined #openstack-qa | 14:45 | |
| *** adalbas has joined #openstack-qa | 14:46 | |
| *** torandu has left #openstack-qa | 14:47 | |
| *** krtaylor has joined #openstack-qa | 14:50 | |
| *** ivanzhu has quit IRC | 14:52 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: VMware: Add cinder support to devstack https://review.openstack.org/41612 | 14:53 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: prot test_images and test_server_actions part1 https://review.openstack.org/39621 | 14:54 |
| *** mlavalle has joined #openstack-qa | 14:57 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_images and test_server_actions into v3 part1 https://review.openstack.org/39621 | 14:57 |
| openstackgerrit | Monty Taylor proposed a change to openstack-dev/devstack: Use a global virtualenv https://review.openstack.org/40534 | 15:00 |
| *** HenryG_ has quit IRC | 15:00 | |
| *** marun has joined #openstack-qa | 15:00 | |
| *** mkollaro has joined #openstack-qa | 15:01 | |
| *** afazekas has quit IRC | 15:17 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_images and test_server_actions into v3 part2 https://review.openstack.org/39609 | 15:17 |
| *** SergeyLukjanov has quit IRC | 15:24 | |
| *** SergeyLukjanov has joined #openstack-qa | 15:29 | |
| *** burt has quit IRC | 15:54 | |
| *** afazekas has joined #openstack-qa | 15:56 | |
| *** yfried has quit IRC | 16:04 | |
| *** tkammer has quit IRC | 16:04 | |
| *** yfried has joined #openstack-qa | 16:09 | |
| openstackgerrit | afazekas proposed a change to openstack-infra/devstack-gate: Using the jenkis user for tempest run https://review.openstack.org/42101 | 16:12 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Add XenServer to GetDistro's logic https://review.openstack.org/41464 | 16:13 |
| *** krtaylor has quit IRC | 16:23 | |
| *** SergeyLukjanov has quit IRC | 16:27 | |
| *** HenryG has joined #openstack-qa | 16:28 | |
| *** krtaylor has joined #openstack-qa | 16:37 | |
| *** yfried has quit IRC | 16:39 | |
| *** jasondotstar has quit IRC | 16:39 | |
| *** thomasbiege has joined #openstack-qa | 16:42 | |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Add support for local XenAPI connection https://review.openstack.org/40613 | 16:44 |
| *** thomasbiege has quit IRC | 16:46 | |
| *** fungi has quit IRC | 16:49 | |
| *** fungi has joined #openstack-qa | 16:50 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add tenant isolation to the swift tests https://review.openstack.org/42169 | 16:50 |
| mkoderer | meeting? | 17:00 |
| mtreinish | mkoderer: yep | 17:00 |
| *** thomasbiege has joined #openstack-qa | 17:03 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Fix option for metadata access in nicira neutron plugin https://review.openstack.org/41714 | 17:11 |
| *** thomasbiege has quit IRC | 17:13 | |
| openstackgerrit | Mate Lakat proposed a change to openstack-dev/devstack: xenapi: devstack support for raw ovf image upload https://review.openstack.org/42172 | 17:16 |
| *** jhenner has joined #openstack-qa | 17:20 | |
| *** HenryG has quit IRC | 17:27 | |
| *** SnowDust has joined #openstack-qa | 17:32 | |
| *** krtaylor has quit IRC | 17:33 | |
| *** yfried has joined #openstack-qa | 17:34 | |
| *** jhenner1 has joined #openstack-qa | 17:37 | |
| *** jhenner has quit IRC | 17:37 | |
| *** jhenner1 has quit IRC | 17:37 | |
| *** jhenner2 has joined #openstack-qa | 17:37 | |
| *** SergeyLukjanov has joined #openstack-qa | 17:50 | |
| *** jasondotstar has joined #openstack-qa | 17:59 | |
| marun | afazekas, mtreinish: there may be an easier way forward | 18:01 |
| afazekas | I'll need to go soon | 18:01 |
| afazekas | marun: there is possible hack to call the setupclass and tearDown class on every test case | 18:02 |
| marun | as per mordred in #openstack-infra: extract the content of the setupClass in its current form, wrap it in a simple fixture unmodified, and then have setupClass call the fixture as a first pass | 18:02 |
| marun | [then, as a second pass, have testresources (which nose doesn't know how to handle) also instanatiate the fixture] | 18:02 |
| marun | This would allow for py26 compatibility and a way forward | 18:02 |
| afazekas | marun: the fixtures working in differnt order | 18:02 |
| marun | but it wouldn't matter | 18:03 |
| marun | would it? | 18:03 |
| mordred | afazekas: yeah - I just mean into fixtures classes | 18:03 |
| afazekas | ro can override the run method in the BaseTestCase inorder to call the setUpClass and tearDownCalss on every test case at the right time | 18:03 |
| mordred | not actually instantiated in setUp | 18:03 |
| afazekas | marun: the numbered tests cases should be fixed before trying it | 18:04 |
| afazekas | I do not know is it really matters or not, but probably it is | 18:04 |
| marun | afazekas: Ok. I'll work on fixing the neutron test so it's not numbered | 18:04 |
| afazekas | another possible blocker is the sanity check logic related to the mangers | 18:04 |
| marun | afazekas: not sure what you mean? | 18:05 |
| afazekas | if the hack working, with a good performance, we can consider using more fixtures or somthing like the fixtures in the future | 18:05 |
| marun | +1 | 18:05 |
| marun | moving towards a good fixture framework means tempest doesn't have to maintain that functionality | 18:06 |
| marun | afazekas: that's all, I won't keep you longer. Thank you for your input as always! | 18:06 |
| afazekas | marun: in tempest case the fixtures little bit overkill and muffin style | 18:07 |
| marun | muffin style? | 18:07 |
| afazekas | http://www.youtube.com/watch?v=o9pEzgHorH0 | 18:07 |
| afazekas | basically we just need create commands with register cleanUps, it does not requires a fixture | 18:08 |
| afazekas | but I like the other magics what are doable by the fixtures | 18:08 |
| marun | I think using testresources would be great for tempest. we could potentially reuse expensive resources like vm's if it were safe to do so. | 18:09 |
| afazekas | marun: agreed | 18:09 |
| afazekas | It is requres a per worker process tenant isolation | 18:09 |
| marun | afazekas: we just need some good examples of how to use testresources so everyone doesn't have to know all the possible complexity of its api | 18:09 |
| afazekas | https://review.openstack.org/#/c/38641/ I will reabse this | 18:10 |
| marun | afazekas: wait | 18:11 |
| afazekas | Do not forget you cannot mix resources from different tenant | 18:11 |
| marun | afazekas: didn't you just send me a video about how we shouldn't write classes?? | 18:11 |
| marun | afazekas: fixtures provide whatever isolation you need | 18:11 |
| afazekas | marun: they are classes because of the single ton usage | 18:12 |
| marun | afazekas: That's not a good reason | 18:12 |
| marun | singletons in the context of python are stupid | 18:12 |
| afazekas | It will make more sense, when I add the real logic | 18:12 |
| *** burt has joined #openstack-qa | 18:12 | |
| afazekas | Another option is a factory style function | 18:13 |
| marun | Uh, sure. But I suggest fixing those classes too. | 18:13 |
| marun | That's not pythonic | 18:13 |
| marun | factory? | 18:13 |
| marun | have you been reading a design patterns book :p | 18:13 |
| afazekas | marun: I try to forget those things :) | 18:13 |
| marun | There's a piece of magic i want to suggest you use instead. | 18:13 |
| marun | namedtuple | 18:13 |
| afazekas | just for dump analysis ? | 18:14 |
| marun | For construction of a trivial class | 18:14 |
| marun | http://docs.python.org/2/library/collections.html#collections.namedtuple | 18:14 |
| *** krtaylor has joined #openstack-qa | 18:15 | |
| marun | The other thing to note that a module is by nature a singleton | 18:15 |
| marun | Which makes an attribute of a module - wait for it - effectively a singleton. | 18:15 |
| afazekas | Do not forget we do not have real main function or anything like that | 18:15 |
| *** jerryz has joined #openstack-qa | 18:16 | |
| marun | Well, maybe I'm missing something and you need this complexity. | 18:16 |
| afazekas | some unusual thing just used because of this | 18:16 |
| marun | I'll add myself as reviewer and wait for you to submit something a bit more complete. | 18:16 |
| marun | But my gut reaction is - this is too complicated for what is being attempted. | 18:17 |
| afazekas | the singleton patter usage usually means you do not have full execution control | 18:17 |
| marun | which isn't really the case here | 18:17 |
| marun | the classes are just data in their current incarnation | 18:17 |
| marun | are you planning on adding methods? | 18:17 |
| marun | and what would those methods do? | 18:18 |
| *** SoFLy has joined #openstack-qa | 18:18 | |
| *** SoFLy has left #openstack-qa | 18:18 | |
| afazekas | marun: since I submitted that patch lot of thing changed, I will reconsider the things | 18:19 |
| afazekas | I need to go now | 18:19 |
| marun | afazekas: fair enough | 18:19 |
| *** toanster has joined #openstack-qa | 18:19 | |
| marun | take care | 18:19 |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Enable Load Balancer as a Service https://review.openstack.org/42119 | 18:20 |
| *** HenryG has joined #openstack-qa | 18:20 | |
| toanster | afazekas: thanks for approving https://review.openstack.org/#/c/40384/ | 18:20 |
| afazekas | toanster: you are welcome | 18:23 |
| adalbas | mtreinish, can we chat about the tenant isolation? | 18:27 |
| *** SnowDust has quit IRC | 18:29 | |
| mtreinish | adalbas: sure | 18:43 |
| mtreinish | sorry I had to run out for a bit | 18:43 |
| adalbas | mtreinish, :) . So, you mentioned to have that isolation patch working, it might need some change in devstack. | 18:43 |
| adalbas | what sort of change? | 18:43 |
| mtreinish | adalbas: it has to do with how swift handles keystone auth | 18:45 |
| mtreinish | when we add users they get assigned the role _member_ | 18:46 |
| adalbas | mtreinish, im asking that because there might be required changes on grenade as well, depending on what they are | 18:46 |
| mtreinish | adalbas: oh, I didn't even think about that | 18:47 |
| mtreinish | yeah it might | 18:47 |
| mtreinish | adalbas: if you look in proxy-server.conf under the filter:keystoneauth group you'll see a list operater_roles | 18:48 |
| mtreinish | the tempest tests assume the role for the user it is using is on that list | 18:49 |
| mtreinish | but '_member_' by default isn't | 18:50 |
| adalbas | mtreinish, so, currently, the users are under the Member role in tempest for swift, is that it? | 18:51 |
| mtreinish | adalbas: so the devstack created users are assigned the 'Member' role, but the ones created with tenant isolation are assigned '_member_' so they can't do the things we're testing in swift | 18:52 |
| mtreinish | I guess I can just assign the role to the user in setup class | 18:52 |
| adalbas | the Member role | 18:53 |
| mtreinish | yeah | 18:54 |
| adalbas | in that case, there would be no need to change anything in devstack | 18:54 |
| mtreinish | adalbas: yep | 18:54 |
| adalbas | mtreinish, ok. should i try the keystone tenant isolation, or do you want me to work with that? Other possibility is to lock the tests, but i think in this case, this would take as much time as get yours approved. | 18:58 |
| mtreinish | adalbas: I'd give keystone tenant isolation a shot | 19:01 |
| adalbas | mtreinish, ok | 19:01 |
| mtreinish | the locking shouldnt be necessary based on how you described the race | 19:02 |
| adalbas | yeah, i think so | 19:02 |
| openstackgerrit | A change was merged to openstack/tempest: Adding extra_specs to flavor format https://review.openstack.org/40384 | 19:10 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add tenant isolation to the swift tests https://review.openstack.org/42169 | 19:17 |
| mtreinish | adalbas: ^^^ I made the role needed for swift configurable in case grenade(or non devstack environment) needs to change it to something else | 19:18 |
| adalbas | mtreinish, great | 19:21 |
| adalbas | mtreinish, did you see any race condition for keystone in the gate jobs? | 19:38 |
| mtreinish | adalbas: I haven't seen one | 19:39 |
| mtreinish | the race I'm seeing most frequently right now is related to server rescue | 19:40 |
| mtreinish | for example: https://jenkins02.openstack.org/job/gate-tempest-devstack-vm-testr-full/1486/console | 19:40 |
| adalbas | mtreinish, ok. and in my env, i just saw a few when trying to add user with the same name. | 19:40 |
| mtreinish | actually resize not rescue thats a different one | 19:41 |
| mtreinish | adalbas: a keystone failure or something else? | 19:41 |
| *** yfried has quit IRC | 19:43 | |
| adalbas | mtreinish, Conflict occurred attempting to store user , when trying to run the api.identity.admin in parallel | 19:43 |
| mtreinish | adalbas: can you put up a pastebin of the failure I don't think I've seen that one before | 19:44 |
| mtreinish | but tenant isolation for keystone may fix it | 19:44 |
| adalbas | mtreinish, http://paste.openstack.org/show/44255/ | 19:45 |
| adalbas | mtreinish, double checking it does not occur in serial | 19:46 |
| adalbas | right, in serial : Ran 324 (+160) tests in 114.633s (+16.058s) -PASSED (id=18) | 19:46 |
| mtreinish | adalbas: actually that one is easier to solve | 19:47 |
| mtreinish | they're using a hard coded name for tests | 19:47 |
| mtreinish | just switch them to rand_names | 19:47 |
| mtreinish | if xml and json run at the same time they'll fail | 19:48 |
| mtreinish | adalbas: since you found it you get to fix it :) | 19:48 |
| adalbas | mtreinish, alright! | 19:48 |
| openstackgerrit | Shawn Hartsock proposed a change to openstack-dev/devstack: vmware - vsphere v5.0 - WSDL cleanup script. https://review.openstack.org/41824 | 19:58 |
| afazekas | mtreinish: https://review.openstack.org/#/c/42069/2 | 20:08 |
| mtreinish | afazekas: as opposed to the print? but wouldn't that screw up logging for the regular case? | 20:09 |
| afazekas | mtreinish: the nuke option is false,so it will not kills the existing handlers | 20:09 |
| mtreinish | afazekas: ok cool | 20:10 |
| *** mriedem has quit IRC | 20:10 | |
| mtreinish | but won't that make the stdout huge during a failure? | 20:11 |
| afazekas | it depends on the test case | 20:11 |
| afazekas | if the test case had wait at it fails after wait it will be huge | 20:11 |
| mtreinish | afazekas: well yeah, but I was looking for a way to just print the req-id to use for debugging | 20:12 |
| afazekas | But for suppressing the wait logs, I think the waiters could be enhanced | 20:12 |
| mtreinish | the rest of the logging isn't really needed in stdout | 20:12 |
| afazekas | http://logs.openstack.org/34/41934/1/check/gate-tempest-devstack-vm-full/5e1165f/console.html.gz | 20:12 |
| afazekas | As you can see it prints when it not wanted | 20:12 |
| mtreinish | afazekas: oh, I didn't even notice that | 20:12 |
| afazekas | usually the ID is not enough information, because the api services does not logs everything | 20:13 |
| mtreinish | I guess that the stdout capture doesn't work as I expected | 20:13 |
| mtreinish | afazekas: it was for locating test failures in the service logs | 20:13 |
| afazekas | mtreinish: they are from setUpClasses and tearDownClasses | 20:14 |
| mtreinish | the stray stdout? | 20:14 |
| afazekas | The stdout capturing turns on every test method and turned of after the test method | 20:15 |
| afazekas | somewhere at setUp and tearDown time | 20:15 |
| mtreinish | afazekas: I see, that makes sense since it's set in the base setUp() | 20:16 |
| mtreinish | afazekas: ok, so how about we set the loglevel for the fixture in 42069 to error then that should decrease the verbosity on stdout and also help with debugging | 20:22 |
| mtreinish | no that won't work for printing the req ids then because Id have to log at error level all the time | 20:22 |
| mtreinish | which is a bad practice... | 20:22 |
| afazekas | mtreinish: I was thinking about INFO | 20:22 |
| mtreinish | how verbose is that? | 20:23 |
| afazekas | most of the large data are on debug level, so it can be ok | 20:23 |
| afazekas | but it will not show the request id | 20:23 |
| afazekas | so you can add the id logging on info level | 20:24 |
| mtreinish | afazekas: I'm looking at: http://logs.openstack.org/93/40893/6/check/gate-tempest-devstack-vm-testr-full/58fc638/logs/tempest.txt.gz (just for an example) and there are still a lot of messages at INFO level | 20:24 |
| mtreinish | every request call and it's corresponding response gets a message | 20:25 |
| *** mriedem has joined #openstack-qa | 20:25 | |
| afazekas | 64% is debug log | 20:27 |
| afazekas | (byte) | 20:28 |
| afazekas | so it will 3 time less size in the output than it was in the old days if we switch to just info | 20:29 |
| afazekas | For compacting the wait logs we have several options, one of them is a custom filter | 20:33 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add nose to run_tests and tox for python 2.6 https://review.openstack.org/42215 | 21:00 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add nose to run_tests and tox for python 2.6 https://review.openstack.org/42215 | 21:01 |
| *** mlavalle has quit IRC | 21:02 | |
| *** jasondotstar has quit IRC | 21:03 | |
| mtreinish | mriedem: ^^^ I knew that would catch your attention... | 21:06 |
| mriedem | mtreinish: ha, yeah | 21:06 |
| mriedem | py26 tempest what?!?!!?! | 21:06 |
| *** thomasbiege has joined #openstack-qa | 21:06 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add nose to run_tests and tox for python 2.6 https://review.openstack.org/42215 | 21:07 |
| *** thomasbiege has quit IRC | 21:07 | |
| *** mlavalle has joined #openstack-qa | 21:11 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add nose to run_tests and tox for python 2.6 https://review.openstack.org/42215 | 21:23 |
| mriedem | mtreinish: what changed? ^ | 21:25 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add unittest framework + tests for wrapper scripts https://review.openstack.org/41770 | 21:31 |
| mtreinish | mriedem: I fixed the python2.6 detection logic for run_tests.sh | 21:31 |
| mtreinish | it helps if I actually test things before I push them for review :) | 21:31 |
| mtreinish | mriedem: could you double check that it works on a real py2.6 env. I was just hardcoding the string for testing. | 21:32 |
| mriedem | mtreinish: :( http://paste.openstack.org/show/44275/ | 21:34 |
| afazekas | marun: https://review.openstack.org/42215 | 21:37 |
| mriedem | mtreinish: gotta head out for a bit | 21:38 |
| *** mriedem has quit IRC | 21:38 | |
| marun | afazekas: danke | 21:39 |
| * afazekas sleep | 21:42 | |
| *** afazekas is now known as afazekas_zz | 21:42 | |
| *** HenryG has quit IRC | 21:42 | |
| mtreinish | marun: I knew that I forgot something I'll add posargs to both jobs that should take care of it | 21:43 |
| mtreinish | thanks for pointing that out | 21:43 |
| marun | mtreinish: awesome, thank you! | 21:44 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add nose to run_tests and tox for python 2.6 https://review.openstack.org/42215 | 21:45 |
| *** weshay has quit IRC | 21:48 | |
| *** jasondotstar has joined #openstack-qa | 21:55 | |
| *** burt has quit IRC | 22:01 | |
| *** torandu has joined #openstack-qa | 22:40 | |
| *** mlavalle has left #openstack-qa | 22:43 | |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/devstack-gate: Ignore node not found error https://review.openstack.org/42235 | 22:47 |
| *** torandu has quit IRC | 22:51 | |
| *** torandu has joined #openstack-qa | 22:51 | |
| *** mkollaro has quit IRC | 22:57 | |
| *** sthaha has joined #openstack-qa | 23:09 | |
| *** sthaha has joined #openstack-qa | 23:09 | |
| *** woodspa has quit IRC | 23:21 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Ignore node not found error https://review.openstack.org/42235 | 23:43 |
| *** SergeyLukjanov has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!