| openstackrecheck | openstack/cinder change: https://review.openstack.org/53037 failed tempest with an unrecognized error | 00:00 |
|---|---|---|
| *** zhangyanzi has joined #openstack-qa | 00:11 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53244 failed tempest with an unrecognized error | 00:11 |
| openstackgerrit | wanghao proposed a change to openstack/tempest: add extend volume tests: https://review.openstack.org/53047 | 00:15 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Added Savanna Project https://review.openstack.org/50601 | 00:28 |
| *** nosnos has joined #openstack-qa | 00:43 | |
| *** zhangyanzi has quit IRC | 00:44 | |
| *** wanghao has joined #openstack-qa | 00:48 | |
| wanghao | giulivo, Anju, afazekas: ping hi guys, Cloud you help me review those tests : https://review.openstack.org/#/c/50217/ and https://review.openstack.org/#/c/50813/ | 00:55 |
| *** rhsu has quit IRC | 00:57 | |
| openstackgerrit | DennyZhang proposed a change to openstack-dev/devstack: grep may be fooled in aggregate.sh https://review.openstack.org/53233 | 01:03 |
| *** DennyZhang has joined #openstack-qa | 01:04 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53244 failed tempest with an unrecognized error | 01:16 |
| *** dims has quit IRC | 01:23 | |
| *** rhsu has joined #openstack-qa | 01:30 | |
| *** arnan has quit IRC | 01:30 | |
| *** openstack has joined #openstack-qa | 01:34 | |
| *** anteaya has quit IRC | 01:39 | |
| openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/53236 failed tempest with an unrecognized error | 01:41 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/51306 failed tempest with an unrecognized error | 01:43 |
| *** rhsu has quit IRC | 01:44 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53220 failed tempest because of: https://bugs.launchpad.net/bugs/1217734 | 01:47 |
| *** DennyZhang has quit IRC | 01:47 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/53258 failed tempest with an unrecognized error | 01:48 |
| *** wanghao has quit IRC | 01:49 | |
| *** SergeyLukjanov has quit IRC | 01:58 | |
| openstackgerrit | Daisuke Morita proposed a change to openstack/tempest: replaces assertLessEqual with assertThat and testtools matcher. https://review.openstack.org/53260 | 02:28 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52408 failed tempest with an unrecognized error | 02:28 |
| *** ivanzhu has joined #openstack-qa | 02:30 | |
| *** ivanzhu has left #openstack-qa | 02:31 | |
| *** ivanzhu has joined #openstack-qa | 02:31 | |
| openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for l3 agent scheduler API https://review.openstack.org/51143 | 02:34 |
| *** sourabh has left #openstack-qa | 02:42 | |
| *** mriedem has quit IRC | 02:42 | |
| openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for the neutron api with provider extension https://review.openstack.org/50542 | 02:48 |
| *** afazekas has joined #openstack-qa | 03:01 | |
| openstackgerrit | Yang Yu proposed a change to openstack/tempest: Adding active tests for updating/replacing image members https://review.openstack.org/48902 | 03:10 |
| *** openstackgerrit has quit IRC | 03:16 | |
| *** openstackgerrit has joined #openstack-qa | 03:17 | |
| openstackgerrit | Yang Yu proposed a change to openstack/tempest: Adding negative tests for glance API https://review.openstack.org/48578 | 03:23 |
| *** yfried has quit IRC | 03:24 | |
| openstackgerrit | Yang Yu proposed a change to openstack/tempest: Adding glance API DELETE /images/<imageid> active tests https://review.openstack.org/47983 | 03:27 |
| *** ivanzhu1 has joined #openstack-qa | 03:31 | |
| *** ivanzhu has quit IRC | 03:34 | |
| *** ivanzhu has joined #openstack-qa | 03:35 | |
| Anju | clarkb: ping | 03:35 |
| clarkb | Anju: pong | 03:35 |
| Anju | clarkb: https://review.openstack.org/#/c/50592/11/tempest/api/image/v2/test_images_tags_negative.py | 03:36 |
| *** ivanzhu1 has quit IRC | 03:36 | |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Rename to create_test_server in API tests https://review.openstack.org/53264 | 03:36 |
| Anju | can you please tell me should I remove the Copyright 2013 OpenStack Foundation here | 03:36 |
| Anju | clarkb: ^^^ | 03:37 |
| clarkb | Anju: did an employee of the foundation write the code? | 03:37 |
| Anju | I write | 03:38 |
| clarkb | if you wrote it then the copyright belongs to you or your employer | 03:38 |
| clarkb | not the foundation | 03:38 |
| Anju | clarkb: My I simply remove this ? | 03:39 |
| Anju | clarkb: May I simply remove this ? | 03:39 |
| clarkb | yes I think simply removing the line is fine | 03:39 |
| Anju | clarkb: thanks . :) | 03:40 |
| openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Added some test fo image tags https://review.openstack.org/50592 | 03:42 |
| *** DennyZhang has joined #openstack-qa | 03:48 | |
| *** Dharmit has joined #openstack-qa | 03:49 | |
| *** clarkb has quit IRC | 04:13 | |
| *** Madan has joined #openstack-qa | 04:20 | |
| *** clarkb has joined #openstack-qa | 04:23 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/50519 failed tempest with an unrecognized error | 04:25 |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Rename to create_test_server in API tests https://review.openstack.org/53264 | 04:28 |
| Anju | afazekas: ping | 04:29 |
| Anju | I have some question regarding Image update cli | 04:29 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/48902 failed tempest with an unrecognized error | 04:34 |
| *** yfried|mob has joined #openstack-qa | 04:37 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/47983 failed tempest with an unrecognized error | 04:38 |
| *** yfried|mob has quit IRC | 04:38 | |
| *** yfried|mob has joined #openstack-qa | 04:39 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/48578 failed tempest with an unrecognized error | 04:40 |
| *** yfried|mob has quit IRC | 04:40 | |
| *** yfried_mob has joined #openstack-qa | 04:41 | |
| *** thomasbiege has joined #openstack-qa | 04:44 | |
| *** yfried_mob has quit IRC | 04:45 | |
| *** yfried has joined #openstack-qa | 04:48 | |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Rename to create_test_server in API tests https://review.openstack.org/53264 | 04:53 |
| *** prekarat has joined #openstack-qa | 04:56 | |
| *** prekarat has quit IRC | 04:59 | |
| *** prekarat has joined #openstack-qa | 05:00 | |
| *** thomasbiege has quit IRC | 05:07 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: fix DeletableSubnet in api/network/common https://review.openstack.org/53074 | 05:08 |
| *** thomasbiege has joined #openstack-qa | 05:09 | |
| *** yfried_mob has joined #openstack-qa | 05:10 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: fix DeletableSubnet in api/network/common https://review.openstack.org/53074 | 05:12 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: edit inheritence tree api/network/security_groups https://review.openstack.org/51590 | 05:16 |
| *** DennyZhang has quit IRC | 05:17 | |
| yfried | mkoderer: what should i do regarding negative/smoke flags? what's the decision? | 05:19 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: edit inheritence tree api/network/security_groups https://review.openstack.org/51590 | 05:24 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/53264 failed tempest because of: https://bugs.launchpad.net/bugs/1217734 | 05:38 |
| *** thomasbiege has quit IRC | 05:43 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/50519 failed tempest with an unrecognized error | 05:46 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/53074 failed tempest with an unrecognized error | 05:53 |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Rename to create_test_server in API tests https://review.openstack.org/53264 | 05:58 |
| *** thomasbiege has joined #openstack-qa | 05:59 | |
| *** thomasbiege2 has joined #openstack-qa | 06:04 | |
| mkoderer | yfried: we will decide that on the summit | 06:05 |
| mkoderer | yfried: I guess we will skip the smoke flag completely | 06:05 |
| yfried | mkoderer: for now it's set to gate. we can change that later. care to approve? | 06:05 |
| mkoderer | yfried: ok | 06:05 |
| *** thomasbiege has quit IRC | 06:06 | |
| *** SergeyLukjanov has joined #openstack-qa | 06:06 | |
| *** yfried_mob has quit IRC | 06:07 | |
| *** yfried_mob has joined #openstack-qa | 06:07 | |
| mkoderer | yfried: jekins isn't gone through | 06:07 |
| mkoderer | I'll wait until its done | 06:08 |
| yfried | mkoderer: I know. unrelated question - why do some of the methods start with _ and some don't. I'm looking in scenario/manager and I see no strict difference | 06:09 |
| mkoderer | yfried: usually _ methods are "private" methods | 06:09 |
| mkoderer | yfried: I was already thinking about https://review.openstack.org/#/c/51590/9/tempest/api/network/base_security_groups.py | 06:10 |
| mkoderer | since all the methods are "_" but there are used in the child classes | 06:10 |
| yfried | mkoderer: is that a conventions are part of the python syntax? | 06:10 |
| *** thomasbiege2 has quit IRC | 06:10 | |
| mkoderer | yfried: yes let me search a link | 06:11 |
| *** jlibosva has joined #openstack-qa | 06:11 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/52883 failed tempest with an unrecognized error | 06:11 |
| yfried | mkoderer: looking at manager/OfficialClientTest most of the methods are named regularly but its "son" NetworkScenarioTest has all methods marked in _, so I don't understand to logic | 06:13 |
| mkoderer | yfried: http://www.python.org/dev/peps/pep-0008/#method-names-and-instance-variables | 06:13 |
| mkoderer | yfried: yes we don't handle this underscore rule really strict in tempest | 06:14 |
| openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some negative tests for flavor https://review.openstack.org/49588 | 06:14 |
| yfried | mkoderer: ok, just wanted to make sure. any guidelines for me? | 06:15 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/52189 failed tempest with an unrecognized error | 06:17 |
| Lingxian | mkoderer, ping | 06:17 |
| mkoderer | yfried: for me this _ methods thing is a little nit and I wouldn't reject a patch just because of this | 06:17 |
| mkoderer | Lingxian: pong | 06:17 |
| Lingxian | mkoderer, could you help to take a look at https://review.openstack.org/#/c/49157/15/tempest/api/compute/admin/test_aggregates.py, I have different opinion as Matthew Treinish, what do you thingk about? | 06:18 |
| yfried | mkoderer: tnx | 06:19 |
| Madan | mkoderer : Ping | 06:19 |
| mkoderer | Lingxian: ok I will have a look | 06:19 |
| mkoderer | Madan: shoot | 06:19 |
| Madan | mkorderer: I sent this patch https://review.openstack.org/#/c/52865/ | 06:21 |
| Madan | mkorderer: But jenkins is failing for a test case already merged | 06:21 |
| Madan | mkoderer:here is the console log http://logs.openstack.org/65/52865/4/check/check-tempest-devstack-vm-full/ad909ca/console.html | 06:22 |
| *** wanghao has joined #openstack-qa | 06:24 | |
| mkoderer | Lingxian: I am a fan of little logical patches. Sry but I have to agree to Matthew since the commit message is not really covering everything you did | 06:26 |
| mkoderer | Madan: ok let me have a look | 06:26 |
| Lingxian | mkoderer, greatly appreciated! | 06:27 |
| Lingxian | mkoderer, if so, I'll seperate it into two patches. | 06:28 |
| mkoderer | Lingxian: yes this would be great | 06:29 |
| mkoderer | Lingxian: thank you | 06:29 |
| Lingxian | mkoderer, I just can not comfirm the strategy. Because I saw some comments in other patches that ask the author do things right once for all. | 06:30 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_images and test_server_actions into v3 part1 https://review.openstack.org/39621 | 06:30 |
| Lingxian | mkoderer, so I am a little confused | 06:30 |
| mkoderer | Lingxian: yes I see, I think it depends a bit who is reviewing it. | 06:32 |
| *** wanghao has quit IRC | 06:32 | |
| mkoderer | Lingxian: if you want you could put this topic on the QA meeting on Thursday and we can talk about it | 06:33 |
| *** wanghao has joined #openstack-qa | 06:33 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_images and test_server_actions into v3 part2 https://review.openstack.org/39609 | 06:33 |
| Lingxian | mkoderer, yes, great! but how? | 06:33 |
| mkoderer | Lingxian: just add the topic there https://wiki.openstack.org/wiki/Meetings/QATeamMeeting | 06:34 |
| Lingxian | mkoderer, I'll do it, thanks! | 06:34 |
| *** yfried_mob has quit IRC | 06:35 | |
| Lingxian | mkoderer, but for this patch, I'll change it as you recommended. OK? | 06:36 |
| mkoderer | Lingxian: yes nobody can complain if you do it like this | 06:37 |
| mkoderer | Madan: I restarted the jenkins process. Please have a look to https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures | 06:37 |
| mkoderer | Madan: there is everything you need to know about jenkins problems | 06:37 |
| Madan | mkoderer: sure :) Thanx a lot :) :) | 06:38 |
| *** thomasbiege has joined #openstack-qa | 06:41 | |
| Anju | mkoderer: hello | 06:47 |
| openstackgerrit | Akihiro Motoki proposed a change to openstack-dev/devstack: Modification for Ubuntu 13.10 and minor fixes in Neutron NEC plugin https://review.openstack.org/53275 | 06:47 |
| *** thomasbiege2 has joined #openstack-qa | 06:47 | |
| Anju | mkoderer: https://review.openstack.org/#/c/50592/ please give some comments :) | 06:47 |
| mkoderer | Anju: ok, I'll do it. But takes a while since my meeting marathon begins right now | 06:50 |
| *** thomasbiege has quit IRC | 06:50 | |
| Anju | mkoderer: thanks , np :) | 06:51 |
| *** networkstatic has joined #openstack-qa | 06:52 | |
| *** dkorn has joined #openstack-qa | 06:52 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: Allow configure cinder with the nfs driver https://review.openstack.org/53276 | 06:57 |
| *** thomasbiege has joined #openstack-qa | 07:02 | |
| *** thomasbiege3 has joined #openstack-qa | 07:02 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53244 failed tempest with an unrecognized error | 07:04 |
| *** thomasbiege has quit IRC | 07:04 | |
| *** thomasbiege2 has quit IRC | 07:06 | |
| *** ivanzhu1 has joined #openstack-qa | 07:09 | |
| openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some tests for aggregates https://review.openstack.org/49157 | 07:10 |
| *** ivanzhu has quit IRC | 07:11 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52810 failed tempest with an unrecognized error | 07:18 |
| *** marun has quit IRC | 07:20 | |
| *** marun has joined #openstack-qa | 07:20 | |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Rename to create_test_server in API tests https://review.openstack.org/53264 | 07:27 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52128 failed tempest with an unrecognized error | 07:30 |
| openstackgerrit | Dong Liu proposed a change to openstack/tempest: add some negative tests for security group: https://review.openstack.org/50249 | 07:30 |
| *** yfried_mob has joined #openstack-qa | 07:33 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52716 failed tempest with an unrecognized error | 07:39 |
| Lingxian | mkoderer, ping | 07:56 |
| *** thomasbiege3 has quit IRC | 07:56 | |
| *** thomasbiege has joined #openstack-qa | 07:59 | |
| *** raies has quit IRC | 07:59 | |
| *** fbo_away is now known as fbo | 07:59 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: add tests for set_metadata in aggregate https://review.openstack.org/52585 | 08:00 |
| *** SergeyLukjanov has quit IRC | 08:02 | |
| mkoderer | Lingxian: yep | 08:05 |
| Lingxian | mkoderer, after fixing, please take a look again, https://review.openstack.org/#/c/49157/, if you have some time. | 08:06 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53112 failed tempest with an unrecognized error | 08:10 |
| *** yassine has joined #openstack-qa | 08:13 | |
| *** sthaha has quit IRC | 08:16 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/53264 failed tempest because of: https://bugs.launchpad.net/bugs/1235437 | 08:17 |
| *** networkstatic is now known as networkstatic_Zz | 08:19 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: added support for security_groups under neutron https://review.openstack.org/50892 | 08:20 |
| *** ivanzhu1 has quit IRC | 08:21 | |
| openstackgerrit | wingwj proposed a change to openstack/tempest: Add test cases for volume-transfer https://review.openstack.org/53070 | 08:22 |
| *** safchain has joined #openstack-qa | 08:23 | |
| *** ivanzhu has joined #openstack-qa | 08:23 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53244 failed tempest with an unrecognized error | 08:23 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: Change group for cm-cpu-agent only for libvirt https://review.openstack.org/53290 | 08:24 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: Allow to configuration the CM pipeline interval https://review.openstack.org/53291 | 08:24 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: [DONT REVIEW THIS] ceilometer debug https://review.openstack.org/53292 | 08:24 |
| *** gszasz has joined #openstack-qa | 08:25 | |
| *** yfried_mob has quit IRC | 08:25 | |
| *** safchain has quit IRC | 08:26 | |
| *** safchain has joined #openstack-qa | 08:26 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: [DONT REVIEW THIS] ceilometer debug https://review.openstack.org/53292 | 08:27 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/51877 failed tempest with an unrecognized error | 08:32 |
| marun | stupid question - how do I file a bug against stable/grizzly? | 08:33 |
| *** giulivo has joined #openstack-qa | 08:40 | |
| *** thomasbiege has quit IRC | 08:42 | |
| openstackgerrit | wanghao proposed a change to openstack/tempest: add extend volume tests: https://review.openstack.org/53047 | 08:54 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/50519 failed tempest with an unrecognized error | 08:59 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: Ensure cm-api is ready before start cm-alarm https://review.openstack.org/53297 | 09:03 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: [DONT REVIEW THIS] ceilometer debug https://review.openstack.org/53292 | 09:03 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: Allow to configuration the CM pipeline interval https://review.openstack.org/53291 | 09:03 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: Change group for cm-cpu-agent only for libvirt https://review.openstack.org/53290 | 09:04 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53112 failed tempest with an unrecognized error | 09:11 |
| *** nosnos has quit IRC | 09:12 | |
| *** nosnos has joined #openstack-qa | 09:13 | |
| *** matsuhashi has quit IRC | 09:13 | |
| *** matsuhashi has joined #openstack-qa | 09:14 | |
| *** nosnos has quit IRC | 09:17 | |
| *** matsuhashi has quit IRC | 09:19 | |
| openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for l3 agent scheduler API https://review.openstack.org/51143 | 09:22 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/51877 failed tempest because of: https://bugs.launchpad.net/bugs/1217734 | 09:29 |
| *** thomasbiege1 has joined #openstack-qa | 09:41 | |
| *** marun has quit IRC | 09:41 | |
| *** marun has joined #openstack-qa | 09:44 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: [DONT REVIEW THIS] ceilometer debug https://review.openstack.org/53292 | 09:47 |
| *** ivanzhu has quit IRC | 09:58 | |
| *** wanghao has quit IRC | 09:58 | |
| *** mkollaro has joined #openstack-qa | 09:59 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: add generic method create_secgroup & rule https://review.openstack.org/53305 | 10:00 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: added support for security_groups under neutron https://review.openstack.org/50892 | 10:07 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: [DONT REVIEW THIS] ceilometer debug https://review.openstack.org/53292 | 10:10 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: Allow to configuration the CM pipeline interval https://review.openstack.org/53291 | 10:10 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: cm-cpu-agent only support libvirt driver https://review.openstack.org/53290 | 10:10 |
| *** dims has joined #openstack-qa | 10:34 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/51877 failed tempest with an unrecognized error | 10:42 |
| *** yfried|mob has joined #openstack-qa | 10:54 | |
| *** yfried has quit IRC | 11:02 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/52993 failed tempest with an unrecognized error | 11:14 |
| *** weshay has joined #openstack-qa | 11:15 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: [DONT REVIEW THIS] ceilometer debug https://review.openstack.org/53292 | 11:17 |
| *** yfried has joined #openstack-qa | 11:21 | |
| *** jhenner has joined #openstack-qa | 11:24 | |
| maurosr | anyone else having this issue running grenade locally where I got several nova-api process running and not stoping during the upgrade? | 11:30 |
| sdague | maurosr: the havana devstack code might have changed and we don't do the stop right | 11:33 |
| sdague | I think nova_api was split from the rest of nova | 11:33 |
| sdague | so the havana grenade might need an extra stop in it | 11:33 |
| maurosr | sdague: ok.... I'll ensure that it's killing everything (although there is a kill agains this service).. thanks... | 11:35 |
| sdague | maurosr: yeh, we're probably just doing something wrong in grenade. My experience is that it needs a bit of work to freshen it for a new base release | 11:36 |
| *** yfried has quit IRC | 11:38 | |
| *** thomasbiege2 has joined #openstack-qa | 11:40 | |
| *** thomasbiege1 has quit IRC | 11:43 | |
| maurosr | the sad thing is that grenade didn't report any errors but wasn't able to put nova-api up again | 11:50 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/51877 failed tempest with an unrecognized error | 11:52 |
| *** yfried|mob has joined #openstack-qa | 11:52 | |
| *** adalbas has joined #openstack-qa | 11:57 | |
| openstackgerrit | A change was merged to openstack/tempest: fix DeletableSubnet in api/network/common https://review.openstack.org/53074 | 12:02 |
| *** dkranz has quit IRC | 12:03 | |
| *** yfried has joined #openstack-qa | 12:08 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53320 failed tempest with an unrecognized error | 12:11 |
| openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Additional ssh tests with tempest https://review.openstack.org/52870 | 12:14 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: add generic method create_secgroup & rule https://review.openstack.org/53305 | 12:18 |
| *** anteaya has joined #openstack-qa | 12:21 | |
| *** shilpa_ has joined #openstack-qa | 12:26 | |
| *** thomasbiege2 has quit IRC | 12:27 | |
| sdague | maurosr: yeh, I expect you'll need to throw some debug in there. Every release enough changes that grenade needs some tweaks | 12:28 |
| *** thomasbiege1 has joined #openstack-qa | 12:28 | |
| openstackgerrit | Hoisaleshwara Madan V S proposed a change to openstack/tempest: Adds api test to test_images https://review.openstack.org/53337 | 12:34 |
| maurosr | sure | 12:36 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52408 failed tempest with an unrecognized error | 12:37 |
| *** harlowja_ has quit IRC | 12:46 | |
| *** yfried|mob has quit IRC | 12:49 | |
| *** yfried|mob has joined #openstack-qa | 12:49 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Add the doc build tools https://review.openstack.org/52226 | 12:52 |
| *** dkranz has joined #openstack-qa | 12:52 | |
| *** weshay has quit IRC | 12:55 | |
| *** yfried has quit IRC | 12:56 | |
| *** yfried|mob has quit IRC | 12:59 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: RunTimeError on tearDownClass explained https://review.openstack.org/50643 | 12:59 |
| *** yfried_mob has joined #openstack-qa | 12:59 | |
| *** yfried|mob has joined #openstack-qa | 13:00 | |
| *** Madan has quit IRC | 13:01 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/51877 failed tempest with an unrecognized error | 13:04 |
| *** mriedem has joined #openstack-qa | 13:10 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53112 failed tempest because of: https://bugs.launchpad.net/bugs/1235486 | 13:17 |
| *** thomasbiege2 has joined #openstack-qa | 13:22 | |
| *** thomasbiege1 has quit IRC | 13:26 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53332 failed tempest with an unrecognized error | 13:31 |
| *** safchain_ has joined #openstack-qa | 13:33 | |
| *** safchain has quit IRC | 13:33 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/53343 failed tempest with an unrecognized error | 13:35 |
| *** weshay has joined #openstack-qa | 13:38 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Test image member is enforced https://review.openstack.org/50267 | 13:45 |
| *** yfried|mob has quit IRC | 14:03 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/51877 failed tempest with an unrecognized error | 14:05 |
| *** dkranz has quit IRC | 14:06 | |
| *** yfried_mob has joined #openstack-qa | 14:09 | |
| *** networkstatic_Zz has quit IRC | 14:11 | |
| *** networkstatic has joined #openstack-qa | 14:11 | |
| giulivo | so many negative tests submissions | 14:17 |
| *** dkranz has joined #openstack-qa | 14:20 | |
| *** safchain_ has quit IRC | 14:20 | |
| *** safchain_ has joined #openstack-qa | 14:21 | |
| sdague | yeh, I know | 14:22 |
| sdague | giulivo: honestly, right now I'd downgrade the priority of those and look at the non negative ones first | 14:22 |
| giulivo | hi sdague :) | 14:23 |
| giulivo | yeah will do | 14:23 |
| sdague | we're going to do a summit session on the negative tests, as we need a better approach. Manual review of those is just getting out of hand | 14:25 |
| giulivo | yeah I noticed the most common additions are create/list/get/delete with invalid ids | 14:28 |
| giulivo | so it looked 'natural' to add those tests as options in the base classes | 14:29 |
| giulivo | but I haven't figured yet how that could be implemented properly as notably I'm not any great coder, would like to look into it though | 14:29 |
| sdague | yeh, we need a way to auto generate those | 14:31 |
| sdague | I'm hoping we get dhellman engaged in that conversation with wsme | 14:31 |
| *** prekarat has quit IRC | 14:32 | |
| *** yfried_mob has quit IRC | 14:32 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Fix irc message for multiple bugs found https://review.openstack.org/53210 | 14:34 |
| *** prekarat has joined #openstack-qa | 14:34 | |
| *** cloudchimp has joined #openstack-qa | 14:34 | |
| openstackrecheck | openstack/requirements change: https://review.openstack.org/52137 failed tempest because of: https://bugs.launchpad.net/bugs/1225664 | 14:36 |
| *** cloudchimp has left #openstack-qa | 14:38 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/51778 failed tempest with an unrecognized error | 14:38 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/51778 failed tempest with an unrecognized error | 14:38 |
| *** openstackrecheck has quit IRC | 14:44 | |
| *** openstackrecheck has joined #openstack-qa | 14:44 | |
| *** dkranz has quit IRC | 14:48 | |
| *** yfried_mob has joined #openstack-qa | 14:55 | |
| *** marun has quit IRC | 14:56 | |
| *** shilpa_ has quit IRC | 14:57 | |
| *** marun has joined #openstack-qa | 15:00 | |
| *** dkranz has joined #openstack-qa | 15:01 | |
| dkranz | sdague: ping | 15:02 |
| sdague | pong | 15:03 |
| dkranz | sdague: The comments by jgriffith make me think we should send a message to the dev list explaining what we are doing in more detail. | 15:03 |
| *** yfried_mob has quit IRC | 15:03 | |
| dkranz | sdague: I really did not think this would be controversial | 15:03 |
| dkranz | sdague: Unless it is just a misunderstanding | 15:04 |
| sdague | yep, agreed, I put a response on there already | 15:04 |
| sdague | but I think I'll put something else out there as well | 15:04 |
| dkranz | sdague: ok, great. I'll take a look. | 15:04 |
| dkranz | sdague: It is about who "you" is in his message I think | 15:04 |
| dkranz | sdague: I'm not sure who he thinks "should" get an error message when a client presents bad data | 15:05 |
| sdague | yeh, I'm not that concerned. I think it's honestly just the beginning of a rigorous discussion about harmonizing our standards on logging | 15:05 |
| sdague | which I think is good and healthy, and something I'm hoping that we'd address in icehouse | 15:05 |
| dkranz | sdague: I hope so too. But I will be technically ready to start gating pretty soon. | 15:06 |
| dkranz | sdague: And new bugs keep appearing... | 15:06 |
| sdague | yep, the nature of the world | 15:06 |
| *** safchain_ has quit IRC | 15:06 | |
| *** safchain_ has joined #openstack-qa | 15:07 | |
| *** gszasz has quit IRC | 15:09 | |
| *** jhenner has quit IRC | 15:10 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/53369 failed tempest with an unrecognized error | 15:11 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53112 failed tempest with an unrecognized error | 15:11 |
| jgriffith | dkranz: sdague maybe it's just me... if so that's fine | 15:12 |
| jgriffith | dkranz: sdague I'm not going to make a stink about it | 15:12 |
| dkranz | jgriffith: Cool. I just wanted to understand your concern. | 15:13 |
| sdague | jgriffith: honestly, I think you raise a really good point that we should probably have a consistent view across the projects of what each of these logging levels mean | 15:13 |
| sdague | because, honestly, I don't think we have that today | 15:13 |
| jgriffith | dkranz: I think it's pretty clear in the bug as well as my post to the ML | 15:13 |
| jgriffith | sdague: agreed | 15:13 |
| openstackrecheck | openstack/requirements change: https://review.openstack.org/52137 failed tempest because of:https://bugs.launchpad.net/bugs/1225664 | 15:13 |
| jgriffith | sdague: frankly I don't necessarily care what it is, just that it's consistent | 15:14 |
| sdague | so I'm going to consider this a great openning to having that discussion | 15:14 |
| jgriffith | sdague: sounds good.. and thanks | 15:14 |
| sdague | I'll post a top level mailing list thread either today or tomorrow to try to get that rolling | 15:14 |
| dkranz | jgriffith: I am hoping we can agree that an operator should not be alerted when a user sends a bad value to an api | 15:14 |
| jgriffith | dkranz: ahh... hmm. | 15:15 |
| mtreinish | damn, I missed a space in my e-r irc message fix... | 15:15 |
| jgriffith | dkranz: I suppose I can see your point | 15:15 |
| jgriffith | dkranz: but I am a bit mixed on that | 15:15 |
| dkranz | jgriffith: I operated a system for a year and this issue was quite painful | 15:15 |
| jgriffith | dkranz: I would have to defer to folks such as yourself | 15:15 |
| jgriffith | dkranz: I can see your point | 15:15 |
| dkranz | jgriffith: You have no way of knowing whether something is going wrong! | 15:15 |
| jgriffith | dkranz: understood | 15:16 |
| jgriffith | dkranz: I'd almost like to introduce another log level in that case | 15:16 |
| dkranz | jgriffith: That said, and this should be part of the discussion, it would be useful to log such cases in a way that is distinguished so you can detect attacks of various sorts, but that is fine tuning. | 15:16 |
| jgriffith | dkranz: recoverable/expected errors | 15:16 |
| dkranz | jgriffith: Right. | 15:16 |
| jgriffith | dkranz: alright, sounds like we're on the same page | 15:16 |
| jgriffith | dkranz: I will go ahead and change the bug that you have for now | 15:17 |
| dkranz | jgriffith: Cool. The more general discussion is definitely worth having | 15:17 |
| jgriffith | dkranz: I'll clean up the trace obviously, and change the log level to warning for now if that fits better for you? | 15:17 |
| jgriffith | dkranz: or if you're convinced it should be just info that's fine too, I'll yield :) | 15:18 |
| dkranz | jgriffith: I think it would actually be better as INFO or DEBUG. I agree there should be another level if possible. | 15:18 |
| jgriffith | dkranz: I knew you were going to say that :) | 15:18 |
| jgriffith | dkranz: alright, I'll have a look. We can always revisit in the future | 15:18 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52128 failed tempest with an unrecognized error | 15:18 |
| dkranz | jgriffith: :). As an operator, I don't want to be warned about random user errors either. | 15:18 |
| jgriffith | dkranz: understood | 15:19 |
| jgriffith | dkranz: the tricky thing is though... we don't currently distinguish that | 15:20 |
| jgriffith | dkranz: in other words, we have no way of knowing it was a user error vs something trashed in the DB | 15:20 |
| dkranz | jgriffith: Well, it should be the difference between sending back 4xx and 5xx. | 15:20 |
| jgriffith | dkranz: yes, but how do *I* know inside Cinder that the user passed a bad ID as opposed to the system is hosed? | 15:21 |
| dkranz | jgriffith: What we really need is request validation at the wsgi entry but we may not be close to that. | 15:21 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/elastic-recheck: Add a missing space from irc messages https://review.openstack.org/53384 | 15:21 |
| *** yfried has joined #openstack-qa | 15:21 | |
| jgriffith | dkranz: reuest validation isn't the problem in this case | 15:21 |
| dkranz | jgriffith: I get it. | 15:21 |
| jgriffith | dkranz: we alr.. | 15:22 |
| jgriffith | ok :) | 15:22 |
| jgriffith | so that's my concern with this particular case | 15:22 |
| dkranz | jgriffith: I'm not sure how to generalize that. | 15:22 |
| jgriffith | dkranz: trouble is I don't think you can | 15:23 |
| dkranz | jgriffith: If the database is corrupted we are kind of losing anyway | 15:23 |
| jgriffith | dkranz: for sure | 15:23 |
| dkranz | jgriffith: I think as long as we approach it from the operator point of view we will get to the best solution | 15:24 |
| dkranz | jgriffith: It obviously requires more thought. | 15:24 |
| jgriffith | dkranz: agreed, and I appreciate the operator/admin perspective that you bring up | 15:25 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53364 failed tempest with an unrecognized error | 15:26 |
| giulivo | dkranz, sdague I won't be in HK but I'd propose as a topic for the negative tests a conversation on how to "auto generate" the negative tests (with invalid/nonexistent uuids) | 15:35 |
| giulivo | to me it looks that should start from the wrappers we have in the base class | 15:36 |
| mtreinish | giulivo: I think there is one scheduled for that | 15:36 |
| giulivo | I won't be there though :( | 15:36 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52801 failed tempest with an unrecognized error | 15:37 |
| giulivo | that still won't cover permission issues where we try to access resources a user shouldn't have rights to inspect though | 15:37 |
| giulivo | or the sharing facilities | 15:38 |
| *** bnemec has joined #openstack-qa | 15:38 | |
| *** Dharmit has quit IRC | 15:41 | |
| *** YorikSar has joined #openstack-qa | 15:42 | |
| YorikSar | Hello, everyone. We have a problem with lockutils: if you don't supply some known location as lock_path, you won't get cross-process locking that can be used in multiprocess test runs (with testr). | 15:44 |
| bnemec | Okay, so continuing our conversation from -dev, which went something like this: | 15:44 |
| bnemec | <YorikSar> bnemec: How about we make all lines that run test in tox.ini look like this: | 15:44 |
| bnemec | <YorikSar> python -m prj.openstack.common.lockutils python setup.py test... | 15:44 |
| bnemec | <YorikSar> bnemec: And so we need just add __main__ part to lockutils that will create temp dir, set some env var to its name, run a cmd line provided in arguments and remove that dir afterwards. | 15:44 |
| YorikSar | bnemec: Yeah. | 15:45 |
| *** thomasbiege2 has quit IRC | 15:45 | |
| YorikSar | So. I see one problem with this approach: the cmd line needed to run tests becomes ugly. | 15:46 |
| bnemec | This is related to the changes in https://review.openstack.org/#/c/42307/36/tests/utils.py and https://review.openstack.org/#/c/47557/10/neutron/tests/unit/__init__.py | 15:46 |
| bnemec | I'm not sure there's any way around that. | 15:47 |
| bnemec | This configuration has to be done at a level where it affects everything being run in test code. | 15:47 |
| YorikSar | bnemec: In my suggestion we do something like a TempDir fixture on process level. | 15:47 |
| bnemec | I generally just run tox anyway so it doesn't bother me. :-) | 15:48 |
| bnemec | YorikSar: Yeah, we would need something like that. | 15:48 |
| bnemec | It's possible we could put this all in a wrapper to encapsulate some of the ugliness. | 15:48 |
| bnemec | Though I'm not sure how the propogation into the test code would work. | 15:49 |
| YorikSar | bnemec: We'll set an env var and use it as a default for lock_path | 15:49 |
| bnemec | I wonder if we should add checking for an env var to the lockutils stuff so we could just create the directory, set the env var, run the tests, then clean it up afterward. | 15:49 |
| bnemec | Heh | 15:50 |
| bnemec | +1 to that. :-) | 15:50 |
| YorikSar | bnemec: We can add smth like tools/with_tempdir.sh to hold all the ugliness, but the need to add another shell script to every project sounds not right. | 15:51 |
| *** SergeyLukjanov has joined #openstack-qa | 15:51 | |
| bnemec | YorikSar: Well, it's a necessary part of testing with lockutils. | 15:52 |
| bnemec | The problem is that in the past we ran with improper locking that just didn't work. | 15:52 |
| bnemec | It silently failed to lock, which is nasty. | 15:53 |
| bnemec | We already have to make changes to all of the projects to use the new lockutils anyway because they were all relying on the old, broken behavior. | 15:53 |
| bnemec | Although now I'm wondering if some of the changes I made in projects like Nova are working right. | 15:54 |
| YorikSar | bnemec: I can think of another nasty hack: attribute setup.py so that it'll install something like with_tempdir to <venv>/bin that will point to lockutils. | 15:56 |
| *** yfried_mob has joined #openstack-qa | 15:57 | |
| bnemec | YorikSar: I think setup.py is centrally managed now, so that might be too intrusive. | 15:58 |
| bnemec | Although I don't know much about how that works, and I assume there's still a way to do project-specific stuff in setup. | 15:58 |
| YorikSar | bnemec: We can add something like this | 15:58 |
| YorikSar | if os.environ.get("WE_ARE_RUNNING_TEST"): import prj.openstack.common.lockutils_nasty_hack | 15:59 |
| YorikSar | bnemec: But yes, it looks too intrusive... | 16:00 |
| YorikSar | bnemec: Oh, and it won't work with py3k. | 16:00 |
| bnemec | YorikSar: So, I think we will definitely want to allow setting lock_path through an env var, and once we have that it's simply a question of the appropriate way to set the var. | 16:01 |
| bnemec | I'm thinking the thing to do now is to get the env var support into lockutils. | 16:02 |
| bnemec | Then maybe start a mailing list thread to discuss the proper way to use it. | 16:02 |
| *** gszasz has joined #openstack-qa | 16:03 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/52189 failed tempest with an unrecognized error | 16:04 |
| bnemec | YorikSar: Would you have time to work on that? I'm kind of swamped at the moment so I may not get to it for a few days. | 16:04 |
| *** krtaylor has quit IRC | 16:04 | |
| YorikSar | bnemec: Yes, I'll post two change requests today (maybe tomorrow): one with env var and one with main() for lockutils. | 16:05 |
| bnemec | YorikSar: Awesome, thanks! | 16:05 |
| bnemec | I will try to keep an eye out for them. | 16:05 |
| bnemec | Feel free to add me to the reviews too though. | 16:05 |
| *** DennyZhang has joined #openstack-qa | 16:07 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/52836 failed tempest with an unrecognized error | 16:10 |
| YorikSar | bnemec: Ok, sure :) | 16:12 |
| *** yassine has quit IRC | 16:16 | |
| *** afazekas has quit IRC | 16:19 | |
| *** jlibosva has quit IRC | 16:19 | |
| dkranz | sdague: Are you going to update the "Negative tests vs. unit tests" item on summit.openstack.org? | 16:27 |
| *** yfried_mob has quit IRC | 16:29 | |
| *** networkstatic has quit IRC | 16:37 | |
| *** harlowja has joined #openstack-qa | 16:43 | |
| *** safchain_ has quit IRC | 16:48 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/52836 failed tempest because of:https://bugs.launchpad.net/bugs/1239856 | 16:54 |
| *** dkorn has quit IRC | 16:56 | |
| *** shilpa_ has joined #openstack-qa | 16:58 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/45927 failed tempest because of:https://bugs.launchpad.net/bugs/1230407 | 17:02 |
| *** DennyZhang has quit IRC | 17:02 | |
| sdague | dkranz: yes, probably going to do that right now :) | 17:03 |
| sdague | actually... do you want to submit it as a new item, that way it will be under your name | 17:03 |
| sdague | and I'll just refuse marc's | 17:04 |
| *** aardvark has quit IRC | 17:05 | |
| *** wusui has joined #openstack-qa | 17:06 | |
| *** gszasz has quit IRC | 17:07 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/52189 failed tempest with an unrecognized error | 17:08 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/44942 failed tempest with an unrecognized error | 17:10 |
| *** yfried has quit IRC | 17:13 | |
| dkranz | sdague: ok, did you have any comments about the content? | 17:16 |
| sdague | I think the content is good | 17:17 |
| sdague | I'm ok with it as is | 17:17 |
| *** networkstatic has joined #openstack-qa | 17:20 | |
| *** SergeyLukjanov has quit IRC | 17:21 | |
| openstackgerrit | Hoisaleshwara Madan V S proposed a change to openstack/tempest: Adds api test to test_images https://review.openstack.org/53337 | 17:22 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add a missing space from irc messages https://review.openstack.org/53384 | 17:29 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/53404 failed tempest with an unrecognized error | 17:30 |
| *** openstackrecheck has quit IRC | 17:36 | |
| *** openstackrecheck has joined #openstack-qa | 17:37 | |
| *** SergeyLukjanov has joined #openstack-qa | 17:40 | |
| dkranz | sdague: Done | 17:51 |
| sdague | thanks | 17:51 |
| openstackrecheck | openstack/requirements change: https://review.openstack.org/52137 failed tempest because of: https://bugs.launchpad.net/bugs/1225664 | 17:52 |
| mtreinish | ^^^ alright the space is fixed! | 17:53 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53220 failed tempest because of: https://bugs.launchpad.net/bugs/1235486 | 18:02 |
| *** SergeyLukjanov has quit IRC | 18:07 | |
| *** jamielennox is now known as jamielennox|away | 18:10 | |
| *** SergeyLukjanov has joined #openstack-qa | 18:13 | |
| *** mriedem has quit IRC | 18:15 | |
| *** mriedem has joined #openstack-qa | 18:18 | |
| *** dims has quit IRC | 18:18 | |
| *** dims has joined #openstack-qa | 18:19 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/44942 failed tempest with an unrecognized error | 18:22 |
| sdague | jgriffith: http://lists.openstack.org/pipermail/openstack-dev/2013-October/017300.html there is the general conversation kick off | 18:26 |
| jgriffith | sdague: thanks, reading now | 18:26 |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/53409 failed tempest with an unrecognized error | 18:36 |
| *** harlowja has quit IRC | 18:40 | |
| *** harlowja has joined #openstack-qa | 18:50 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52983 failed tempest with an unrecognized error | 18:52 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Cleanup tempst.conf.sample https://review.openstack.org/53425 | 19:00 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add config feature verification script https://review.openstack.org/52988 | 19:00 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Stop auto-detecting glance API versions https://review.openstack.org/53181 | 19:00 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add api version detection to verify_tempest_config https://review.openstack.org/53182 | 19:00 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Reorganize project feature config options https://review.openstack.org/52524 | 19:00 |
| *** marun has quit IRC | 19:00 | |
| *** krtaylor has joined #openstack-qa | 19:05 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add volume extensions tests https://review.openstack.org/53011 | 19:07 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/52524 failed tempest with an unrecognized error | 19:19 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/53181 failed tempest with an unrecognized error | 19:19 |
| *** fbo is now known as fbo_away | 19:19 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/52988 failed tempest with an unrecognized error | 19:19 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/53425 failed tempest with an unrecognized error | 19:21 |
| openstackgerrit | David Kranz proposed a change to openstack/tempest: Add more stuff to the whitelist https://review.openstack.org/53428 | 19:21 |
| *** markmcclain has joined #openstack-qa | 19:22 | |
| markmcclain | sdague: ping | 19:22 |
| sdague | pong | 19:22 |
| sdague | I'm almost 100% certain you want to ping mtreinish though :) | 19:22 |
| sdague | but I'll wait for the question | 19:22 |
| markmcclain | actually this is a you question first | 19:23 |
| sdague | ok :) | 19:23 |
| markmcclain | so the tempest jobs don't run neutron isolation tests | 19:23 |
| sdague | well, there are 2 jobs on neutron that do | 19:23 |
| markmcclain | right except for something change yesterday in tempest | 19:24 |
| markmcclain | and now all of our checks fail | 19:24 |
| sdague | or more importantly, were supposed to, but weren't until mtreinish's patch landed yesterday | 19:24 |
| *** adalbas has quit IRC | 19:24 | |
| markmcclain | this one does not appear to running the same jobs as neutron | 19:25 |
| markmcclain | https://review.openstack.org/#/c/53425/ | 19:25 |
| sdague | that is true | 19:26 |
| sdague | right, we do have an asymetry right now | 19:26 |
| markmcclain | which makes it really hard to diagnose the current situation | 19:26 |
| sdague | yes, agreed | 19:27 |
| sdague | mtreinish: ping? | 19:27 |
| *** marun has joined #openstack-qa | 19:28 | |
| sdague | markmcclain: so the root issue looks like ip exhaution right? | 19:28 |
| markmcclain | right | 19:28 |
| markmcclain | but exhaustion implies a bad test scenario | 19:29 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/53182 failed tempest with an unrecognized error | 19:29 |
| mtreinish | sdague: pong | 19:30 |
| sdague | mtreinish: scrollback | 19:31 |
| *** shilpa_ has quit IRC | 19:31 | |
| * mtreinish reads | 19:31 | |
| sdague | trying to figure out if there is a proper fix here that we can get to quickly, or if we should just revert | 19:31 |
| mtreinish | markmcclain: so what I changed was the neutron api tests were not running with tenant isolation enabled before my patch | 19:32 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: Revert "Use isolation credentials for neutron api tests" https://review.openstack.org/53431 | 19:32 |
| sdague | so that's the revert, let's let it run through tests while we sort if there is a better way | 19:32 |
| mtreinish | markmcclain: https://review.openstack.org/#/c/53016/ | 19:32 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/50486 failed tempest with an unrecognized error | 19:33 |
| markmcclain | I think the revert is the way to go for now | 19:33 |
| markmcclain | I've got to step out for a bit | 19:33 |
| sdague | mtreinish: is it possible that by not deleting the tenants when we are done, we're leaving them with resources allocated? | 19:34 |
| mtreinish | sdague: there were issues with that in the early versions of the network isolation stuff | 19:34 |
| mtreinish | but I think we explicity clean things up at tearDown now to get around that | 19:35 |
| mtreinish | let me double check | 19:35 |
| mtreinish | sdague: https://git.openstack.org/cgit/openstack/tempest/tree/tempest/common/isolated_creds.py#n378 so it doesn't look like we wait for delete to finish | 19:36 |
| mtreinish | I was thinking of the port cleanup stuff which I don't think is the issue | 19:38 |
| *** networkstatic has quit IRC | 19:39 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/53423 failed tempest because of: https://bugs.launchpad.net/bugs/1240256 | 19:46 |
| sdague | dkranz: you about? | 19:50 |
| sdague | markmcclain: so the thing is, nova-network is running under all of these conditions today, so we really do need to get to the bottom of why neutron can't handle the same load as nova-network. | 19:51 |
| *** dkranz has quit IRC | 19:53 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/53428 failed tempest with an unrecognized error | 20:00 |
| sdague | mtreinish: so what about putting one of the tenant isolation jobs on tempest, then we could debug landing it? | 20:01 |
| sdague | because the real issue is we had an asymetric gate here | 20:01 |
| *** networkstatic has joined #openstack-qa | 20:01 | |
| mtreinish | sdague: sure I can do that one sec | 20:01 |
| mtreinish | sdague: voting? | 20:01 |
| sdague | sure | 20:01 |
| mtreinish | sdague: I'd say just make all the neutron jobs isolated actually | 20:02 |
| mtreinish | I think they've been stable for the most part | 20:03 |
| mtreinish | except for this blip | 20:03 |
| sdague | well, I still think it's important that there are at least 4 tempest jobs on neutron | 20:03 |
| sdague | just because it means it's harder to land a race condition | 20:04 |
| mtreinish | are there any other configurations worth running? | 20:04 |
| mtreinish | sdague: or we could make the isolated job what runs for all the projects instead | 20:05 |
| sdague | so right now, lets just add the mysql isolated job to tempest, minimum change | 20:05 |
| sdague | and we can think about adjustments to the rest of it post summit | 20:06 |
| mtreinish | sdague: ok, I actually just commited it with pg since we already had mysql | 20:06 |
| sdague | that will at least give us a way to test your patch | 20:06 |
| mtreinish | is that ok? | 20:06 |
| sdague | mtreinish: sure | 20:06 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 20:08 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53414 failed tempest with an unrecognized error | 20:08 |
| *** _SergeyLukjanov has quit IRC | 20:09 | |
| *** rdekel has joined #openstack-qa | 20:10 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53427 failed tempest with an unrecognized error | 20:11 |
| *** SergeyLukjanov has joined #openstack-qa | 20:12 | |
| jgriffith | sdague: coolio, patches submitted to on the two log files with Cinder errors | 20:12 |
| sdague | jgriffith: cool :) | 20:13 |
| *** mlavalle has joined #openstack-qa | 20:13 | |
| sdague | though, honestly, my point in the general conversation was to grow general concensus on things | 20:13 |
| sdague | but the fact that you are awesome fast on things, is appreciated :) | 20:14 |
| *** networkstatic has quit IRC | 20:17 | |
| *** networkstatic has joined #openstack-qa | 20:19 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add config feature verification script https://review.openstack.org/52988 | 20:26 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Stop auto-detecting glance API versions https://review.openstack.org/53181 | 20:26 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add api version detection to verify_tempest_config https://review.openstack.org/53182 | 20:26 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Cleanup tempst.conf.sample https://review.openstack.org/53425 | 20:26 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Reorganize project feature config options https://review.openstack.org/52524 | 20:26 |
| *** networkstatic has quit IRC | 20:26 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add config feature verification script https://review.openstack.org/52988 | 20:31 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Stop auto-detecting glance API versions https://review.openstack.org/53181 | 20:31 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add api version detection to verify_tempest_config https://review.openstack.org/53182 | 20:31 |
| jgriffith | sdague: haha!! | 20:38 |
| jgriffith | sdague: yeah, I just wanted to discuss it more broadly as well. No objection to fixing those two items up | 20:39 |
| jgriffith | sdague: anywho... onward and upward | 20:39 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53244 failed tempest with an unrecognized error | 20:52 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/44942 failed tempest with an unrecognized error | 20:54 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/53182 failed tempest with an unrecognized error | 21:00 |
| openstackgerrit | A change was merged to openstack/tempest: Revert "Use isolation credentials for neutron api tests" https://review.openstack.org/53431 | 21:00 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/53425 failed tempest with an unrecognized error | 21:06 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/53181 failed tempest with an unrecognized error | 21:06 |
| *** rdekel has quit IRC | 21:06 | |
| mtreinish | sdague: so all 5 of my config rework patches failed because of neutron ip exhaustion... | 21:09 |
| sdague | mtreinish: awesome | 21:11 |
| sdague | nachi just filed a bug for it | 21:11 |
| sdague | you should make sure to be on that bug and pile on whatever info you can | 21:11 |
| mtreinish | sdague: I though there was already one. I used this: https://bugs.launchpad.net/neutron/+bug/1243726 for the first recheck | 21:12 |
| sdague | ok, so natchi filed another bug then | 21:12 |
| sdague | oh, wait, same bug | 21:12 |
| sdague | the launchpad bug email confused me | 21:12 |
| mtreinish | sdague: https://etherpad.openstack.org/p/debug1243726 | 21:13 |
| sdague | cool | 21:13 |
| *** atiwari has joined #openstack-qa | 21:13 | |
| *** mkollaro has quit IRC | 21:19 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/52524 failed tempest with an unrecognized error | 21:20 |
| *** torandu has joined #openstack-qa | 21:21 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Reorganize project feature config options https://review.openstack.org/52524 | 21:21 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Cleanup tempst.conf.sample https://review.openstack.org/53425 | 21:22 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Stop auto-detecting glance API versions https://review.openstack.org/53181 | 21:22 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add config feature verification script https://review.openstack.org/52988 | 21:22 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add api version detection to verify_tempest_config https://review.openstack.org/53182 | 21:22 |
| *** mlavalle has quit IRC | 21:22 | |
| *** weshay has quit IRC | 21:22 | |
| mtreinish | sdague: you are really ok with: https://review.openstack.org/#/c/52785/2 it pulls out the version string from the setup.cfg | 21:24 |
| sdague | hmmm... right | 21:26 |
| sdague | which breaks the tarball generation? | 21:27 |
| mtreinish | sdague: I'm not sure of the full implications of it | 21:27 |
| mtreinish | but that would make sense | 21:27 |
| sdague | and mordred isn't around | 21:27 |
| sdague | ok, removing my +2, we'll ping him next week | 21:28 |
| mtreinish | sdague: I just based that patch on one he submitted earlier that did the same thing | 21:28 |
| *** jaypipes has quit IRC | 21:28 | |
| mtreinish | it's the only way that the docs jobs pass with the new pbr | 21:28 |
| sdague | ok, so docs jobs currently don't work? | 21:28 |
| mtreinish | sdague: with the old pbr (which is what we are using now) it works fine | 21:29 |
| mtreinish | but the upgrade fails in a funny way if a version is set | 21:29 |
| sdague | ok, so lets hold until mordred is back around | 21:29 |
| sdague | and figure it out | 21:29 |
| mtreinish | sdague: see: http://logs.openstack.org/41/51041/2/check/gate-tempest-docs/e729ff3/console.html.gz | 21:30 |
| sdague | I think his twitter said he'd be in rtp at this next - http://www.allthingsopen.org/ | 21:30 |
| sdague | that at least puts him back in the right TZ | 21:30 |
| mtreinish | "Where in the world is Monty Taylor?" | 21:31 |
| mtreinish | it doesn't have quite the same ring as Carmen Sandiago | 21:31 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/52988 failed tempest with an unrecognized error | 21:33 |
| annegentle | mtreinish: needs more syllables | 21:33 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53188 failed tempest with an unrecognized error | 21:33 |
| *** mriedem has quit IRC | 21:36 | |
| mtreinish | annegentle: heh, yeah that and it doesn't really sound exotic enough | 21:36 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Use isolation credentials for neutron api tests https://review.openstack.org/53459 | 21:38 |
| *** marun has quit IRC | 21:43 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Revert "Revert "Use isolation credentials for neutron api tests"" https://review.openstack.org/53460 | 21:43 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/53182 failed tempest because of: https://bugs.launchpad.net/bugs/1239637 | 21:54 |
| clarkb | nice ^ is still working. you'd never know I was rebooting servers behind the scenes :) | 21:54 |
| *** atiwari has quit IRC | 22:01 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 22:02 | |
| *** _SergeyLukjanov has quit IRC | 22:02 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/53440 failed tempest with an unrecognized error | 22:03 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/53454 failed tempest with an unrecognized error | 22:09 |
| *** fbo_away is now known as fbo | 22:17 | |
| mordred | sdague: what did I do? | 22:21 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/53459 failed tempest with an unrecognized error | 22:21 |
| mordred | mtreinish: that patch looks correct | 22:22 |
| mordred | mtreinish: pbr injects version into your doc/conf.py anyway - so if its causing problems, it's perfectly fine to remove it | 22:23 |
| openstackgerrit | Chris Behrens proposed a change to openstack-dev/devstack: Start nova-compute with child cell conf https://review.openstack.org/53468 | 22:31 |
| *** mlavalle has joined #openstack-qa | 22:33 | |
| *** fbo is now known as fbo_away | 22:33 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/52876 failed tempest with an unrecognized error | 22:34 |
| *** mkollaro has joined #openstack-qa | 22:35 | |
| sdague | mordred: where does it get that version from? | 22:36 |
| mordred | sdague: pbr | 22:36 |
| sdague | but we want the tempest version in the docs | 22:36 |
| mordred | yes | 22:36 |
| sdague | where does it get the tempest version from? | 22:36 |
| mordred | sdague: the version that pbr generates, it passes in to the constructors for the docs config | 22:37 |
| mordred | when you run build_sphinx | 22:37 |
| sdague | ok, how does it know the tempest version is 2014.1.0 ? | 22:37 |
| mordred | it will only tell you that's the version if there is a tag that's 2014.1.0 | 22:37 |
| mordred | if you have a tag that's 2014.1 two commits ago, it will tell you the version is 2014.1.dev2.g$gitshortsha | 22:38 |
| sdague | ok, so that's the thing we'd lose | 22:38 |
| mordred | what is the thing you want to achieve? | 22:38 |
| sdague | ok, so it will call it 2013.2.dev... in the docs? | 22:38 |
| * mordred has been travelling - please treat me like I'm dumb | 22:38 | |
| sdague | I guess that's ok | 22:38 |
| mordred | yes | 22:38 |
| sdague | so we'd been going the other way, and calling the docs the next version | 22:39 |
| mordred | same rules as versions for python-*client | 22:39 |
| sdague | instead of the last version | 22:39 |
| mordred | well, if you want to do that | 22:39 |
| sdague | but I can live with the change in convention | 22:39 |
| mordred | you want to put the next version in the version slot in setup.cfg | 22:39 |
| mordred | which is essentially just removing one of the lines of the patch | 22:39 |
| mordred | if you do that, it will do pre-release versioning like the server projects | 22:39 |
| sdague | so when mtreinish did that, the docs blew up | 22:39 |
| mordred | I thought it was just when he left the code in conf.py | 22:40 |
| sdague | but he just wandered out the door for the evening, so I'll assume he'll pick up the thread in the morning | 22:40 |
| sdague | hmmm... good question | 22:40 |
| mordred | btw... | 22:40 |
| mordred | for some reason, tempest is the only thing that has a problem here | 22:40 |
| mordred | I don't know why | 22:40 |
| mordred | I have not figured out root cause yet | 22:40 |
| sdague | ok, well at some point when you aren't on planes on the time, we should figure it out :) | 22:41 |
| *** SergeyLukjanov has joined #openstack-qa | 22:41 | |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/53462 failed tempest because of: https://bugs.launchpad.net/bugs/1239637 | 22:42 |
| *** openstack has joined #openstack-qa | 22:47 | |
| openstackgerrit | Chris Behrens proposed a change to openstack-dev/devstack: Start nova-compute with child cell conf https://review.openstack.org/53468 | 22:50 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/52546 failed tempest with an unrecognized error | 22:56 |
| *** krtaylor has quit IRC | 22:59 | |
| *** jamielennox|away is now known as jamielennox | 22:59 | |
| *** dims has quit IRC | 23:11 | |
| *** mkollaro has quit IRC | 23:14 | |
| *** mkollaro has joined #openstack-qa | 23:15 | |
| openstackgerrit | Aaron Rosen proposed a change to openstack-dev/devstack: Bump floatingip pool to /24 https://review.openstack.org/53474 | 23:15 |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/53467 failed tempest with an unrecognized error | 23:15 |
| *** mkollaro has quit IRC | 23:18 | |
| *** mkollaro has joined #openstack-qa | 23:18 | |
| *** mlavalle has left #openstack-qa | 23:19 | |
| *** dims has joined #openstack-qa | 23:25 | |
| *** mkollaro has quit IRC | 23:26 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/50519 failed tempest with an unrecognized error | 23:28 |
| *** sthaha has joined #openstack-qa | 23:36 | |
| *** sthaha has joined #openstack-qa | 23:36 | |
| *** rahmu has quit IRC | 23:43 | |
| *** rahmu has joined #openstack-qa | 23:45 | |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/53467 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 23:51 |
| *** krtaylor has joined #openstack-qa | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!