| *** hemna is now known as hemnafk | 00:04 | |
| *** matsuhashi has joined #openstack-qa | 00:09 | |
| openstackgerrit | Edward Hope-Morley proposed a change to openstack/tempest: Add cinder-backup functional tests https://review.openstack.org/45465 | 00:19 |
|---|---|---|
| giulivo | dosaboy, ? | 00:20 |
| *** marun has quit IRC | 00:32 | |
| dosaboy | giulivo: hey | 00:35 |
| dosaboy | wassup? | 00:35 |
| giulivo | so I'm rerunning devstack but I think you're wrong | 00:35 |
| giulivo | there is no checks for ENABLED_SERVICES in lib/cinder | 00:35 |
| giulivo | it just starts all those services if c-api is enables | 00:35 |
| dosaboy | the check is in screen_it | 00:36 |
| dosaboy | which checks ENABLED_SERVICES | 00:36 |
| giulivo | well hopefully screen_it is only going to create the screen window | 00:36 |
| giulivo | let me check | 00:36 |
| dosaboy | if I do not add c-bak to that global, i do not have c-bak in my screen window | 00:36 |
| dosaboy | right and that is how the service gets started | 00:37 |
| dosaboy | no screen window == no service | 00:37 |
| giulivo | well, no that's not the case but the screen_it function has a BIG if is_service_enabled on top | 00:38 |
| giulivo | GOD, I have no idea why it is doing that | 00:39 |
| giulivo | but then yes, you've to put that in the ENABLED_SERVICES array to see that working | 00:39 |
| dosaboy | :) | 00:39 |
| giulivo | sorry about that | 00:39 |
| *** nosnos has joined #openstack-qa | 00:40 | |
| dosaboy | again, not everyone wants to have to configure backup/swift/ceph to run devstack so it is not enabled by default | 00:40 |
| giulivo | yes but I still think we should keep the backup tests enabled by default | 00:40 |
| giulivo | so we should add c-bak to the gate script | 00:40 |
| giulivo | give me a sec | 00:41 |
| dosaboy | giulivo: i have set the backup test to 'gate' and it now defaults to true | 00:43 |
| openstackgerrit | Giulio Fidente proposed a change to openstack-infra/devstack-gate: add c-bak to MY_ENABLED_SERVICES https://review.openstack.org/45797 | 00:43 |
| dosaboy | check latest patchset | 00:43 |
| giulivo | dosaboy, but the thing is, we shoul try to keep the default tempest config on par with the default devstack config | 00:44 |
| giulivo | so if it is disabled in devstack , we should keep the default to false in tempest | 00:44 |
| giulivo | and update that at gating | 00:44 |
| giulivo | I just pushed a first change, see ^^ | 00:45 |
| giulivo | let me add the second one to devstack | 00:45 |
| *** mriedem has joined #openstack-qa | 00:48 | |
| openstackgerrit | Giulio Fidente proposed a change to openstack-dev/devstack: enable volume backup tests if c-bak is enabled https://review.openstack.org/45800 | 00:49 |
| giulivo | dosaboy, could you please review those two? | 00:50 |
| dosaboy | giulivo: defaulting backup test to true cause gate tests to fail - https://review.openstack.org/#/c/45465/ | 00:54 |
| giulivo | indeed there is no backup service running, that's the reason for this https://review.openstack.org/45797 | 00:54 |
| dosaboy | ah so it follows devstack, was not aware of that | 00:55 |
| dosaboy | i'll do that review now then | 00:55 |
| giulivo | so you should set it back to false (se my latest comment on your submission) and we enable it only when c-bak is enabled, that is the reason for this: https://review.openstack.org/45800 | 00:55 |
| dosaboy | your last comment says "but I'd set it to True by default then" | 00:57 |
| giulivo | dosaboy, I just reviewed, I'm sorry to have messed up | 00:57 |
| giulivo | hope this clarifies the process though | 00:57 |
| dosaboy | hehe ok | 00:58 |
| dosaboy | i'll set to false | 00:58 |
| giulivo | so we keep a sane default which matches the default devstack config, where we do not have backup running | 00:58 |
| giulivo | but enable it at gating and update the tempest config accordingly when c-bak is enabled | 00:58 |
| giulivo | makes sense | 00:58 |
| giulivo | ? | 00:58 |
| openstackgerrit | Edward Hope-Morley proposed a change to openstack/tempest: Add cinder-backup functional tests https://review.openstack.org/45465 | 00:59 |
| giulivo | ping mtreinish regarding the regexp that is what I wanted | 01:10 |
| giulivo | nose is matching against functions having test as a word boundary | 01:12 |
| giulivo | didn't know testr isn't | 01:12 |
| *** jasondotstar has joined #openstack-qa | 01:30 | |
| *** raies has quit IRC | 01:42 | |
| *** adalbas has quit IRC | 01:58 | |
| *** yjiang5 is now known as yjiang5_away | 02:02 | |
| *** malini_afk is now known as malini | 02:02 | |
| *** ken1ohmichi has joined #openstack-qa | 02:14 | |
| *** malini is now known as malini_afk | 02:16 | |
| *** malini_afk is now known as malini | 02:17 | |
| *** mriedem has quit IRC | 02:29 | |
| *** anteaya has quit IRC | 02:47 | |
| *** jasondotstar has quit IRC | 02:50 | |
| openstackgerrit | minsel proposed a change to openstack/tempest: Handling network resources in tenant isolation https://review.openstack.org/45578 | 02:52 |
| *** malini is now known as malini_afk | 02:54 | |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Use common create_server method for advanced_ops https://review.openstack.org/44197 | 04:34 |
| *** SergeyLukjanov has joined #openstack-qa | 04:38 | |
| *** yfried has joined #openstack-qa | 04:51 | |
| *** HenryG has quit IRC | 04:53 | |
| *** nosnos has quit IRC | 05:35 | |
| *** nosnos has joined #openstack-qa | 05:36 | |
| *** SergeyLukjanov has quit IRC | 05:55 | |
| *** SergeyLukjanov has joined #openstack-qa | 06:04 | |
| *** Anju has quit IRC | 06:06 | |
| *** annegentle_ has quit IRC | 06:06 | |
| *** mordred has quit IRC | 06:08 | |
| *** stevebaker has quit IRC | 06:09 | |
| *** annegentle has joined #openstack-qa | 06:09 | |
| *** dkorn has joined #openstack-qa | 06:09 | |
| *** SergeyLukjanov has quit IRC | 06:10 | |
| *** comstud has quit IRC | 06:10 | |
| *** comstud has joined #openstack-qa | 06:11 | |
| *** timello has quit IRC | 06:11 | |
| *** russellb has quit IRC | 06:12 | |
| *** stevebaker has joined #openstack-qa | 06:13 | |
| *** timello has joined #openstack-qa | 06:14 | |
| *** yjiang5_away has quit IRC | 06:19 | |
| *** yjiang5_away has joined #openstack-qa | 06:19 | |
| *** russellb has joined #openstack-qa | 06:21 | |
| *** rdekel has joined #openstack-qa | 06:25 | |
| *** Anju has joined #openstack-qa | 06:27 | |
| *** jhenner has joined #openstack-qa | 06:30 | |
| *** tkammer has joined #openstack-qa | 06:34 | |
| tkammer | Hey guys, can you point me out to the version/flavor of pep8 you use? it seems jenkins failed on my submission though my code does pass the pep8 on my system | 06:47 |
| afazekas_zz | tkammer: tox -epep8 | 06:48 |
| tkammer | thanks afazekas_zz | 06:49 |
| *** afazekas_zz is now known as afazekas | 06:51 | |
| Anju | afazeaks : ping | 06:51 |
| afazekas | Anju: pong | 06:52 |
| Anju | hii .. lock tests is not working in serveraction.py | 06:52 |
| Anju | although lock unlock tests are running in serverbasicops.py | 06:53 |
| Anju | afazekas: ^^^ | 06:53 |
| afazekas | Where ? | 06:54 |
| Anju | afazekas: i just check it after adding in the server actions.py | 06:56 |
| tkammer | afazekas, are you familiar with dependencies of some sorts for the tox? I have it installed on the system, but I'm missing a toxini file, as I'm not familiar with tox, can you help me out? | 06:56 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add negative volume tests https://review.openstack.org/44610 | 07:00 |
| afazekas | tkammer: try it at the tempest directory | 07:02 |
| Anju | afazekas: can u suggest me something | 07:04 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add negative volume tests https://review.openstack.org/44610 | 07:05 |
| afazekas | Anju: I do not really understand what is the question / or issue | 07:05 |
| afazekas | Anju: now it is working on the gate, at least I do not see it as recheck issue http://status.openstack.org/rechecks/ | 07:06 |
| Anju | afazekas like start /stop test the lock /unlock test is not runnig | 07:06 |
| Anju | mean to say that lock action is not performing | 07:07 |
| afazekas | what is the name of the not working test case ? is it already merged ? | 07:08 |
| Anju | nooo | 07:11 |
| Anju | i was trying to make it test_lock_unlock | 07:11 |
| afazekas | Anju: where is you patch, what is the exception ? | 07:19 |
| Anju | i did not add yet | 07:19 |
| Anju | :P | 07:19 |
| Anju | telling u but | 07:20 |
| afazekas | Too few info to find out why it not working | 07:24 |
| afazekas | I will be back in 25 min | 07:24 |
| *** afazekas has quit IRC | 07:24 | |
| *** fbo_away is now known as fbo | 07:33 | |
| *** yassine has joined #openstack-qa | 07:49 | |
| *** nosnos has quit IRC | 07:52 | |
| * tkammer not believing afazekas lies, it's been almost 40 minutes now.. ;) | 08:01 | |
| *** afazekas has joined #openstack-qa | 08:01 | |
| tkammer | lol, what timing afazekas | 08:02 |
| tkammer | [11:01] * tkammer not believing afazekas lies, it's been almost 40 minutes now.. ;) | 08:02 |
| afazekas | tkammer: sorry about that, I got an irq :) | 08:04 |
| tkammer | afazekas, no worries, just having "tox" problems.. for some reason I get "ERROR: could not install deps" :-\ | 08:05 |
| *** mordred has joined #openstack-qa | 08:05 | |
| tkammer | afazekas, and the thing is.. I don't need any deps.. I already have everything on the system.. | 08:06 |
| afazekas | tkammer: it will install it to virtual env | 08:09 |
| afazekas | you just need the libxslt-devel | 08:09 |
| afazekas | actually it uses pep8 1.4.4 with flake8 | 08:10 |
| afazekas | flake8 is a combination of pep8 an pyflakes checks with some addition | 08:11 |
| afazekas | the tox job also uses some tempest specific extension | 08:11 |
| afazekas | tempest/hacking/checks.py | 08:12 |
| tkammer | hmmm.. I have the libxslt-devel | 08:14 |
| tkammer | and I have flake8 | 08:14 |
| tkammer | but for some reason it fails :-\ | 08:15 |
| afazekas | python-devel ? Can you paste the error message to a pastebin? | 08:16 |
| tkammer | afazekas, give me a sec | 08:19 |
| Anju | afazekas : | 08:21 |
| Anju | resp, server = self.client.lock_server(self.server_id) | 08:22 |
| Anju | AttributeError: 'ServersClientXML' object has no attribute 'lock_server | 08:22 |
| Anju | this is the exception | 08:22 |
| afazekas | the xml version is missing | 08:23 |
| afazekas | it just have a json version | 08:23 |
| Anju | not running for json also | 08:23 |
| *** SergeyLukjanov has joined #openstack-qa | 08:37 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Bulk Subnets and Ports creation https://review.openstack.org/44870 | 08:43 |
| *** raies has joined #openstack-qa | 08:46 | |
| raies | afazekas: hi | 08:46 |
| afazekas | raies: hi | 08:48 |
| raies | please check https://review.openstack.org/#/c/44870/ I have added patch3 | 08:49 |
| raies | afazekas: u there ? | 08:53 |
| afazekas | yes, does not seams good | 08:53 |
| afazekas | rand_name should not be used in service | 08:53 |
| raies | any reason ? | 08:54 |
| afazekas | the count is not a real argument | 08:54 |
| raies | count is just to ensure iteration | 08:55 |
| raies | how many subnets/ports needed to create | 08:55 |
| afazekas | subnet_list should be the only parameter | 08:56 |
| raies | thats why taken default as 2 so that it can create at least 2 default for the case bulk | 08:56 |
| afazekas | len(cidrs) ?? | 08:56 |
| afazekas | FIXME: I do not think the api is enforcing the same network_id usage in all bulk subnet creation | 08:57 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add negative volume tests https://review.openstack.org/44610 | 08:57 |
| raies | len(cidrs) what is this ? I did not write len | 08:57 |
| raies | please ensure lines in comments also | 08:57 |
| raies | your first comment: rand_name should not use is service ,, any reason? | 08:59 |
| afazekas | I will add comments to the review | 08:59 |
| raies | the count is not a real argument .,,, what doest it mean and where ? | 08:59 |
| raies | ok | 08:59 |
| afazekas | raies: because the test cases responsibility to give name for everything | 09:00 |
| afazekas | and it should read it back as well | 09:00 |
| raies | afzekas: the count is not a real argument .,,, what doest it mean and where ? | 09:01 |
| afazekas | count = len(cidrs) ; but even not this is the recommended | 09:01 |
| raies | no len(cidrs) will give a huge number | 09:01 |
| raies | this wil lead to quata exceed | 09:02 |
| raies | "subnet_list should be the only parameter" I am not getting it. | 09:02 |
| *** yjiang5_ has joined #openstack-qa | 09:04 | |
| raies | afazekas: u there ? | 09:07 |
| *** afazekas_ has joined #openstack-qa | 09:07 | |
| raies | afazekas_: u there ? | 09:08 |
| *** afazekas has quit IRC | 09:11 | |
| *** yjiang5_away has quit IRC | 09:11 | |
| *** mtreinish has quit IRC | 09:11 | |
| *** comstud has quit IRC | 09:11 | |
| *** soren has quit IRC | 09:11 | |
| *** ianw has quit IRC | 09:11 | |
| *** toanster_ has quit IRC | 09:11 | |
| *** pleia2 has quit IRC | 09:11 | |
| *** tkammer has quit IRC | 09:11 | |
| *** jhenner has quit IRC | 09:11 | |
| *** matsuhashi has quit IRC | 09:11 | |
| *** sdague has quit IRC | 09:11 | |
| *** samalba has quit IRC | 09:11 | |
| *** ken1ohmichi has quit IRC | 09:15 | |
| *** afazekas_ is now known as afazekas | 09:15 | |
| afazekas | raies: yes | 09:15 |
| raies | afazekas: len(cidrs) will give a huge number and this wil lead to quata exceed | 09:16 |
| *** sdague has joined #openstack-qa | 09:17 | |
| *** samalba has joined #openstack-qa | 09:17 | |
| raies | afazekas: please elaborate some queries | 09:19 |
| raies | 1. len(cidrs) will give a huge number and this wil lead to quata exceed | 09:19 |
| *** mtreinish has joined #openstack-qa | 09:20 | |
| *** tkammer has joined #openstack-qa | 09:20 | |
| *** jhenner has joined #openstack-qa | 09:20 | |
| *** comstud has joined #openstack-qa | 09:20 | |
| *** matsuhashi has joined #openstack-qa | 09:20 | |
| *** pleia2 has joined #openstack-qa | 09:20 | |
| *** toanster_ has joined #openstack-qa | 09:20 | |
| *** ianw has joined #openstack-qa | 09:20 | |
| *** soren has joined #openstack-qa | 09:20 | |
| afazekas | you could pass juts only 2 cidrs[0:2] | 09:20 |
| raies | afazekas: 2. As per u "subnet_list should the create_bulk_subnet(self, sub_net_list) 's argument." is it nessesary ? | 09:21 |
| afazekas | It seams to be the most reasonable choose | 09:21 |
| afazekas | It makes the service method more generally useful | 09:22 |
| afazekas | you can use different network ids, and you can specify the name on the test case side | 09:23 |
| raies | u mean to say I should pass an empty list using parameter ? | 09:23 |
| raies | reate_bulk_subnet(self, sub_net_list) | 09:23 |
| raies | where sub_net_list is an empty list | 09:23 |
| raies | ? | 09:23 |
| afazekas | It can be allowed by the method definition | 09:24 |
| afazekas | it may try to send the empty list to the service.. | 09:25 |
| raies | ok fine | 09:26 |
| raies | as for as I am getting at the point "create_bulk_subnet(self, sub_net_list)" | 09:26 |
| raies | I should pass empty list | 09:27 |
| raies | afazekas: one more thingg is that, to use different net_id I have to merge networks/subnets/ports test case in single test case | 09:28 |
| raies | which will lead to disturb someone's test case | 09:29 |
| afazekas | raies: IMHO if you would like to use more networks, you can create more in the setUpClass | 09:33 |
| raies | afazekas: ok :) | 09:34 |
| raies | afazekas: will be back with next patch :) | 09:34 |
| *** matsuhashi has quit IRC | 09:34 | |
| *** matsuhashi has joined #openstack-qa | 09:36 | |
| raies | afazekas: ping | 09:44 |
| *** SergeyLukjanov has quit IRC | 09:45 | |
| *** lifeless has quit IRC | 09:56 | |
| *** afazekas is now known as afazekas_food | 10:01 | |
| *** matsuhashi has quit IRC | 10:03 | |
| yfried | any idea how to unblock ARP requests in security groups? | 10:15 |
| raies | afazekas_food | 10:29 |
| raies | http://pastebin.com/XnYzn2hJ | 10:29 |
| *** raies has quit IRC | 10:29 | |
| *** raies has joined #openstack-qa | 10:30 | |
| *** SergeyLukjanov has joined #openstack-qa | 10:48 | |
| *** lifeless has joined #openstack-qa | 11:15 | |
| *** adalbas has joined #openstack-qa | 11:20 | |
| *** weshay has joined #openstack-qa | 11:45 | |
| *** afazekas has joined #openstack-qa | 11:48 | |
| *** sthaha has joined #openstack-qa | 12:02 | |
| *** sthaha has joined #openstack-qa | 12:02 | |
| raies | afazekas: please check http://pastebin.com/XnYzn2hJ | 12:11 |
| raies | afazekas: r u there ? | 12:18 |
| openstackgerrit | garyk proposed a change to openstack-dev/devstack: VMware: remove invalid configuration variable https://review.openstack.org/45859 | 12:21 |
| raies | afazekas: will u please suggest on this http://pastebin.com/QVe7ui1a | 12:23 |
| *** weshay has quit IRC | 12:40 | |
| afazekas | list of dicts | 12:44 |
| raies | afazekas: I did not know its solution | 12:48 |
| raies | afazekas: have u checked this : http://pastebin.com/QVe7ui1a | 12:49 |
| raies | afazekas: r u there ? | 12:52 |
| *** jasondotstar has joined #openstack-qa | 12:53 | |
| *** weshay has joined #openstack-qa | 12:55 | |
| *** jhenner has quit IRC | 12:56 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Bulk Subnets and Ports creation https://review.openstack.org/44870 | 12:59 |
| afazekas | raies: IMHO for bulk request the 'list of dicts' arguments are good | 12:59 |
| raies | afazekas: I have added https://review.openstack.org/44870 please check | 13:00 |
| *** jhenner has joined #openstack-qa | 13:01 | |
| openstackgerrit | A change was merged to openstack/tempest: Fix import grouping in scenario tests https://review.openstack.org/45779 | 13:02 |
| raies | afazekas: no i am leaving please check https://review.openstack.org/44870 | 13:03 |
| raies | *now | 13:03 |
| *** whayutin_ has joined #openstack-qa | 13:06 | |
| *** weshay has quit IRC | 13:08 | |
| *** mriedem has joined #openstack-qa | 13:13 | |
| *** malini_afk is now known as malini | 13:19 | |
| afazekas | I will | 13:28 |
| *** burt has joined #openstack-qa | 13:43 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Swift: configure Ceilometer when it is enabled https://review.openstack.org/45201 | 13:55 |
| *** rdekel has quit IRC | 13:59 | |
| *** raies_ has joined #openstack-qa | 14:08 | |
| raies_ | afazekas: hi | 14:08 |
| afazekas | Do you know what is 'list of dicts' ? | 14:10 |
| raies_ | [{}, {}] | 14:11 |
| raies_ | does'nt it ? | 14:11 |
| afazekas | yes | 14:12 |
| raies_ | have u checked https://review.openstack.org/#/c/44870/ | 14:13 |
| raies_ | it seems jenkins is sick | 14:13 |
| raies_ | please also review my codes | 14:13 |
| raies_ | afazekas: u there ? | 14:16 |
| afazekas | yes, but I got offline interrupts .. | 14:17 |
| raies_ | are looking https://review.openstack.org/#/c/44870/ | 14:17 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Remove order dependence from network_basic_ops https://review.openstack.org/45780 | 14:18 |
| afazekas | raies_: can you change the bluk request function yo just accept list of dicts ? | 14:18 |
| raies_ | afazekas: u mean to say that i should delete list() function ? | 14:19 |
| raies_ | i.e subnet_list = [] | 14:20 |
| afazekas | def create_bulk_port(self, port_list) | 14:21 |
| afazekas | def create_bulk_subnet(self, sub_net_list) | 14:21 |
| afazekas | the dick should contain all parameter for the particular port or subnet | 14:21 |
| raies_ | u mean i should pass an empty list "port_list" | 14:22 |
| afazekas | raies_: it was in comment on the patche set 3 | 14:22 |
| afazekas | empty ? | 14:23 |
| afazekas | https://review.openstack.org/#/c/44870/4/tempest/services/network/json/network_client.py L338 contains a subnet_list | 14:23 |
| afazekas | that should be used as argument, and the other (above) logic should move to the test case | 14:24 |
| raies_ | oh | 14:25 |
| raies_ | but tell me one thing, is it good to pass kwargs in test cases ? | 14:25 |
| afazekas | in this case *posargs might work | 14:26 |
| raies_ | don't know use of *posargs | 14:28 |
| raies_ | :( | 14:28 |
| afazekas | I did not see any argument in the bulk request which is common for each subnet or port | 14:28 |
| afazekas | so, you just need a list of dicts argument IMHO | 14:29 |
| raies_ | as u suggested that move subnet_list part in test case | 14:30 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add services tags to scenario tests https://review.openstack.org/45783 | 14:30 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add service tags to api.volume https://review.openstack.org/45782 | 14:30 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add @services decorator https://review.openstack.org/45781 | 14:30 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add hacking check for service tags in scenario https://review.openstack.org/45784 | 14:30 |
| raies_ | how I would deal with keyword arguments i am not getting | 14:30 |
| raies_ | "I did not see any argument in the bulk request which is common for each subnet or port" , please elaborate ? | 14:31 |
| raies_ | to move request body logic in test case, what would be benefit ? | 14:33 |
| afazekas | looks like the bulk request individually fully specifies all port and subnets, so I do not see why to use **kwargs here | 14:34 |
| *** mkollaro has joined #openstack-qa | 14:35 | |
| openstackgerrit | Dirk Mueller proposed a change to openstack-dev/grenade: Install the new schema-image.json https://review.openstack.org/45884 | 14:35 |
| raies_ | afazekas: if i am not using ip_version then it will get fail | 14:36 |
| raies_ | for subnet | 14:36 |
| afazekas | **kwargs would be useful for additional global arguments , but I do not see any | 14:36 |
| raies_ | I have passed admin_state_up for port | 14:37 |
| raies_ | and ip_version for subnet | 14:37 |
| raies_ | I could add some more | 14:37 |
| raies_ | I don't think it is affecting our code | 14:39 |
| *** marun has joined #openstack-qa | 14:39 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Remove order dependence from network_basic_ops https://review.openstack.org/45780 | 14:41 |
| raies_ | afazekas: ?? | 14:42 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add @services decorator https://review.openstack.org/45781 | 14:43 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add service tags to api.volume https://review.openstack.org/45782 | 14:43 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add services tags to scenario tests https://review.openstack.org/45783 | 14:43 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add hacking check for service tags in scenario https://review.openstack.org/45784 | 14:43 |
| *** mriedem has quit IRC | 14:45 | |
| raies_ | afazekas: r u there / | 14:46 |
| raies_ | ? | 14:46 |
| *** mriedem has joined #openstack-qa | 14:48 | |
| afazekas | raies_: can you add it to every port and subnet on the test case side? | 14:49 |
| raies_ | what request-body logic ? | 14:50 |
| *** dkranz has joined #openstack-qa | 14:51 | |
| raies_ | afazekas; u mean to say i have to move request body logic from service to test case file | 14:52 |
| raies_ | fiine | 14:52 |
| afazekas | yes | 14:53 |
| *** rdekel has joined #openstack-qa | 14:53 | |
| raies_ | but please assist on, how I will add kwargs value | 14:53 |
| raies_ | there | 14:54 |
| afazekas | Do you need kwargs at all ? | 14:54 |
| raies_ | yes | 14:54 |
| raies_ | in subnet case , ip_version is must | 14:54 |
| raies_ | without it test case will fail | 14:54 |
| afazekas | you can add to all dicts in the list on the test case side | 14:55 |
| raies_ | means I should no use kwargs ? | 14:56 |
| afazekas | yes | 14:57 |
| raies_ | ok | 14:57 |
| raies_ | i can try it also | 14:57 |
| *** dkranz has quit IRC | 14:58 | |
| raies_ | ok np it will be done | 14:58 |
| raies_ | but tell me one thing that what would be it's benefit | 14:59 |
| raies_ | ? | 14:59 |
| *** yfried has quit IRC | 15:00 | |
| afazekas | you can use one bulk request method for creating v4 and v6 subnets (when the v6 will be fully supported) | 15:01 |
| afazekas | or create ports with different admin_state_up | 15:01 |
| raies_ | ok you mean if I am creating 2 subnet then one should be v4 and other should be v6 | 15:03 |
| raies_ | ? | 15:03 |
| afazekas | if it is working yes | 15:04 |
| raies_ | ok I will try to implement | 15:04 |
| raies_ | if I am moving logic from service to test case then what would be benefit, I am not getting | 15:05 |
| raies_ | same logic will be in test case file with subnet-_list = list () function | 15:06 |
| raies_ | then what would be benefit in only moving ? | 15:07 |
| afazekas | another test case can reuse the service function in different way | 15:07 |
| *** dkorn has quit IRC | 15:08 | |
| afazekas | the service functions supposed to be generic | 15:08 |
| *** raies_ has quit IRC | 15:11 | |
| *** dkranz has joined #openstack-qa | 15:15 | |
| *** rdekel has quit IRC | 15:18 | |
| *** marun has quit IRC | 15:20 | |
| *** marun has joined #openstack-qa | 15:25 | |
| openstackgerrit | Doug Hellmann proposed a change to openstack-dev/devstack: Only run chmod/chown for local files https://review.openstack.org/45248 | 15:27 |
| *** wramthun has quit IRC | 15:28 | |
| *** sthaha has quit IRC | 15:29 | |
| *** prekarat has joined #openstack-qa | 15:31 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Default to linuxbridge and openvswitch drivers for ML2. https://review.openstack.org/45493 | 15:36 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Augment instead of override extra ML2 options. https://review.openstack.org/45697 | 15:36 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Modify midonet plugin to support the latest MidoNet https://review.openstack.org/45560 | 15:38 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Dump basic network info in the test_network_basic_ops https://review.openstack.org/40435 | 15:39 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Configure VPNaaS Horizon panel if q-vpn is enabled https://review.openstack.org/45751 | 15:40 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Server+User leak detector https://review.openstack.org/35516 | 15:42 |
| *** yassine has quit IRC | 15:43 | |
| *** dkorn has joined #openstack-qa | 15:54 | |
| *** tkammer has quit IRC | 16:01 | |
| *** dkorn has quit IRC | 16:03 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: enable volume backup tests if c-bak is enabled https://review.openstack.org/45800 | 16:06 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Lower Heat max_template_size for Tempest https://review.openstack.org/45730 | 16:10 |
| *** prekarat has quit IRC | 16:11 | |
| *** SergeyLukjanov has quit IRC | 16:15 | |
| *** dkranz has quit IRC | 16:15 | |
| *** mkollaro has quit IRC | 16:20 | |
| *** mlavalle has joined #openstack-qa | 16:20 | |
| *** anteaya has joined #openstack-qa | 16:23 | |
| openstackgerrit | Dirk Mueller proposed a change to openstack-dev/grenade: Install the new schema-image.json https://review.openstack.org/45884 | 16:24 |
| *** dkranz has joined #openstack-qa | 16:28 | |
| *** fbo is now known as fbo_away | 16:41 | |
| *** SergeyLukjanov has joined #openstack-qa | 16:43 | |
| openstackgerrit | Dirk Mueller proposed a change to openstack-dev/grenade: Install the new schema-image.json https://review.openstack.org/45884 | 16:49 |
| *** raies_ has joined #openstack-qa | 17:08 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Translate server extension attributes to json https://review.openstack.org/45910 | 17:09 |
| afazekas | abalbas: https://review.openstack.org/45910 | 17:09 |
| sdague | raies_: please stop private pinging on irc, much better to talk on a channel | 17:17 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: File header changes and rules https://review.openstack.org/42317 | 17:18 |
| *** mlavalle has quit IRC | 17:25 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Simplify xml/json client selection https://review.openstack.org/36937 | 17:25 |
| *** yfried has joined #openstack-qa | 17:28 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Dump basic network info in the test_network_basic_ops https://review.openstack.org/40435 | 17:33 |
| *** mlavalle has joined #openstack-qa | 17:54 | |
| afazekas | mlavalle: hi | 17:54 |
| mlavalle | afazekas: hi | 17:54 |
| *** raies__ has joined #openstack-qa | 17:55 | |
| afazekas | Do you think the per tenant router/net/subnet is doable ? | 17:55 |
| mtreinish | afazekas: https://review.openstack.org/#/c/45578/ | 17:55 |
| * afazekas blind | 17:56 | |
| mlavalle | afazekas: I obviuosly do…. ;-) | 17:56 |
| *** raies_ has quit IRC | 17:57 | |
| *** jhenner has quit IRC | 17:57 | |
| *** jhenner has joined #openstack-qa | 17:58 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Changes to make Devstack work with Neutron L3 plugin patch. https://review.openstack.org/20909 | 18:02 |
| afazekas | What to do whit this bug https://bugs.launchpad.net/tempest/+bug/1092009 ? mark it as duplicate or add it to the commit message ? | 18:02 |
| mtreinish | afazekas: I'd mark it as duplicate | 18:03 |
| mtreinish | the solution for both should be the same | 18:03 |
| mlavalle | I agree | 18:04 |
| afazekas | Do we need exception handling to the _create_network_resources method ? I mean if the 2th step fails undo the 1. one , if the 3th fails undo the 1th and 2th .. ? | 18:09 |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/devstack-gate: Add a pip freeze invocation to save with the logs https://review.openstack.org/45919 | 18:11 |
| afazekas | Do we need config option for using shared network, and skipping the net isolation steps ? | 18:12 |
| afazekas | mtreinish, mlavalle ? ^^ | 18:12 |
| mlavalle | afazekas: as far as skipping the net isolation steps, mtreinish made a comment is patch set 2. Pretty much, the idea is that is a user doesn't want net isolation, he / she should create his /her isolated creds | 18:13 |
| mtreinish | afazekas: if tenant isolation is enabled with neutron this is a requirement for it to work | 18:14 |
| mtreinish | so to turn this off you turn off tenant isolation | 18:14 |
| mlavalle | yeah, I agree | 18:15 |
| afazekas | NOTE: AFAIK using a shared network, with credential isolation can be a use case | 18:15 |
| afazekas | how many total routers can we have at the same time on the gate with this isolation code ? | 18:16 |
| afazekas | AFAIK the routers eats one external ip address and we have just very small range | 18:16 |
| *** VinodGupta has joined #openstack-qa | 18:16 | |
| mlavalle | afazekas: if that becomes an issue I think we can expand the range | 18:17 |
| mlavalle | right now we have a /28 cidd. We could change it to 27 if that becomes an issue | 18:17 |
| afazekas | mtreinish: can we merge the code as it is and try to enable the full gate ? | 18:17 |
| afazekas | mlavalle: I do not see why not to use /24 , the bigger range has almost zero additional cost | 18:18 |
| mtreinish | afazekas: yeah that's what I'm thinking after review approval we have to push the devstack change to re-enable tenant isolation with neutron | 18:18 |
| mtreinish | and go from there | 18:18 |
| mlavalle | afazekas: I am fine changing it to /24….not a big deal | 18:19 |
| afazekas | +2 | 18:19 |
| *** raies__ has quit IRC | 18:19 | |
| mlavalle | afazekas: i'll push a new patch set tonight. I might add exception handling to clean up, as you pointed out above | 18:20 |
| *** VinodGupta has quit IRC | 18:21 | |
| mlavalle | that way we don't leave spurios | 18:21 |
| mlavalle | nets, subnets or routers | 18:21 |
| afazekas | cool | 18:23 |
| afazekas | 'Filter ComputeFilter returned 0 host(s) get_filtered_objects' is anybody working on this type of issues ? | 18:30 |
| *** yfried has quit IRC | 18:32 | |
| openstackgerrit | Clint Byrum proposed a change to openstack-dev/devstack: Fix section on iniset for max_template_size https://review.openstack.org/45925 | 18:39 |
| mlavalle | mtreinish: the official Neutron client is just a wrapper of the Neutron API. It just returns the dictionary it gets from the Neutron API. No objects with attributes like keystone or nova…. Anyway, I tested the code also with the official client, so I know it works with it | 18:41 |
| mtreinish | mlavalle: ok I just wanted to confirm | 18:42 |
| mlavalle | ;-) | 18:42 |
| mtreinish | I like that a lot more than the other clients | 18:42 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add service tag section to HACKING.rst https://review.openstack.org/45926 | 18:42 |
| *** adalbas has quit IRC | 18:44 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: VMware: remove invalid configuration variable https://review.openstack.org/45859 | 18:48 |
| openstackgerrit | Doug Hellmann proposed a change to openstack-dev/devstack: Only run chmod/chown for local files https://review.openstack.org/45248 | 18:50 |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Add a pip freeze invocation to save with the logs https://review.openstack.org/45919 | 18:53 |
| *** wramthun has joined #openstack-qa | 18:55 | |
| afazekas | jog0, sdague: https://bugs.launchpad.net/nova/+bug/1221987 I think this bug has many duplicate (reported against test cases), it's priority should be increased to high or critical | 18:58 |
| jog0 | afazekas: agreed | 18:59 |
| jog0 | trying to reproduce right now | 18:59 |
| *** adalbas has joined #openstack-qa | 19:00 | |
| *** fbo_away is now known as fbo | 19:00 | |
| afazekas | sdague, mtreinish, dkranz: can you review it https://review.openstack.org/#/c/45910/ (I need it for another change) | 19:01 |
| * afazekas I need to go soon | 19:02 | |
| afazekas | jog0: yes, it is an interesting issue | 19:04 |
| jog0 | afazekas: we hava a logstash query to look for failed jobs btw | 19:06 |
| afazekas | looks like the logstash does not have the DEBUG messages | 19:09 |
| afazekas | just the INFO and above | 19:10 |
| openstackgerrit | Doug Hellmann proposed a change to openstack-dev/devstack: Only run chmod/chown for local files https://review.openstack.org/45248 | 19:16 |
| *** fbo is now known as fbo_away | 19:21 | |
| *** HenryG has joined #openstack-qa | 19:24 | |
| *** fbo_away is now known as fbo | 19:30 | |
| *** wramthun has quit IRC | 19:43 | |
| *** mdenny has quit IRC | 19:47 | |
| *** mdenny has joined #openstack-qa | 19:48 | |
| *** malini is now known as malini_afk | 19:49 | |
| *** annegentle has quit IRC | 19:53 | |
| *** annegentle has joined #openstack-qa | 19:53 | |
| *** afazekas has quit IRC | 19:55 | |
| openstackgerrit | SlickNik proposed a change to openstack-dev/devstack: Added Trove (Database as a Service). https://review.openstack.org/38169 | 20:03 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add service tag section to HACKING.rst https://review.openstack.org/45926 | 20:19 |
| *** mlavalle has quit IRC | 20:26 | |
| openstackgerrit | SlickNik proposed a change to openstack-dev/devstack: Added Trove (Database as a Service). https://review.openstack.org/38169 | 20:32 |
| mtreinish | dkranz: that's a recurring failure that we've hit a few times: https://bugs.launchpad.net/nova/+bug/1221987 | 20:33 |
| dkranz | mtreinish: Thanks. | 20:33 |
| *** wramthun has joined #openstack-qa | 20:33 | |
| *** mlavalle has joined #openstack-qa | 20:47 | |
| *** adalbas has quit IRC | 20:57 | |
| *** rockyg has joined #openstack-qa | 21:09 | |
| *** adalbas has joined #openstack-qa | 21:11 | |
| *** malini_afk is now known as malini | 21:16 | |
| *** jasondotstar has quit IRC | 21:17 | |
| *** adalbas has quit IRC | 21:19 | |
| *** HenryG has quit IRC | 21:32 | |
| *** malini is now known as malini_afk | 21:32 | |
| *** mriedem has quit IRC | 21:41 | |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/devstack-gate: Log pre and post test hooks to file and stdout. https://review.openstack.org/45956 | 21:42 |
| *** fbo is now known as fbo_away | 21:47 | |
| *** rockyg has quit IRC | 21:55 | |
| *** krtaylor has joined #openstack-qa | 22:02 | |
| *** burt has quit IRC | 22:02 | |
| *** whayutin_ has quit IRC | 22:04 | |
| *** dkranz has quit IRC | 22:06 | |
| *** SergeyLukjanov has quit IRC | 22:06 | |
| *** SergeyLukjanov has joined #openstack-qa | 22:19 | |
| *** SergeyLukjanov has quit IRC | 22:20 | |
| *** harlowja has quit IRC | 22:42 | |
| *** mlavalle has quit IRC | 22:51 | |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Install and use diskimage-builder, tripleo-image-elements https://review.openstack.org/45969 | 22:52 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Baremetal uses packaged diskimage-builder https://review.openstack.org/40343 | 22:52 |
| openstackgerrit | A change was merged to openstack/tempest: Remove order dependence from network_basic_ops https://review.openstack.org/45780 | 23:28 |
| openstackgerrit | A change was merged to openstack/tempest: Add @services decorator https://review.openstack.org/45781 | 23:28 |
| *** HenryG has joined #openstack-qa | 23:33 | |
| openstackgerrit | Clark Boylan proposed a change to openstack-dev/devstack: Revert "Swift: configure Ceilometer when it is enabled" https://review.openstack.org/45977 | 23:41 |
| openstackgerrit | SlickNik proposed a change to openstack-dev/devstack: Added Trove (Database as a Service). https://review.openstack.org/38169 | 23:41 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!