Monday, 2013-10-14

openstackrecheckopenstack/heat change: https://review.openstack.org/50828 failed tempest because of: https://bugs.launchpad.net/bugs/123543700:01
*** rhsu has joined #openstack-qa00:04
*** sthaha has quit IRC00:12
openstackgerritJamie Lennox proposed a change to openstack-dev/devstack: Allow deploying keystone with SSL certificates  https://review.openstack.org/4707600:14
*** mriedem has joined #openstack-qa00:36
*** sthaha has joined #openstack-qa00:37
*** sthaha has quit IRC00:37
*** sthaha has joined #openstack-qa00:44
openstackrecheckopenstack/keystone change: https://review.openstack.org/49460 failed tempest with an unrecognized error01:21
*** tiantian has joined #openstack-qa01:23
openstackrecheckopenstack/keystone change: https://review.openstack.org/51481 failed tempest with an unrecognized error01:25
*** wanghao has joined #openstack-qa01:48
*** sileht has quit IRC02:05
*** sileht has joined #openstack-qa02:11
*** dims has joined #openstack-qa02:25
*** sthaha has quit IRC02:33
*** sthaha has joined #openstack-qa02:33
*** sthaha has joined #openstack-qa02:33
*** sthaha has quit IRC02:34
*** sthaha has joined #openstack-qa02:34
*** dims has quit IRC03:03
openstackgerritZhu Zhu proposed a change to openstack/tempest: Test for flavor-access-list with private/public flavor  https://review.openstack.org/5118803:05
*** ivanzhu has joined #openstack-qa03:26
*** mriedem has quit IRC03:40
*** ivanzhu has quit IRC04:23
*** ivanzhu has joined #openstack-qa04:37
openstackrecheckopenstack/neutron change: https://review.openstack.org/51515 failed tempest with an unrecognized error04:41
openstackgerritSunil Thaha proposed a change to openstack/tempest: Restores passing additional parameters to nosetest  https://review.openstack.org/5081904:44
*** wanghao has quit IRC04:48
openstackgerritYang Yu proposed a change to openstack/tempest: Adding glance v2 tests for image data  https://review.openstack.org/5117405:14
*** __afazekas is now known as afazekas05:15
*** tiantian has quit IRC05:18
*** ivanzhu has quit IRC05:20
*** yfried has joined #openstack-qa05:32
yfriedwhat is the policy regarding adding stuff to tempest.conf?05:34
yfriedafazekas: ping05:34
openstackrecheckopenstack/neutron change: https://review.openstack.org/51515 failed tempest because of: https://bugs.launchpad.net/bugs/123548605:36
*** ivanzhu has joined #openstack-qa05:37
*** SergeyLukjanov has joined #openstack-qa05:48
openstackgerritDennyZhang proposed a change to openstack-dev/devstack: remove useless step in cleanup_rpc_backend  https://review.openstack.org/5151905:51
openstackgerritDennyZhang proposed a change to openstack-dev/devstack: fix typo in functions  https://review.openstack.org/5152005:53
*** Dharmit has joined #openstack-qa05:54
*** rhsu has quit IRC06:04
*** jlibosva has joined #openstack-qa06:14
*** jlibosva1 has joined #openstack-qa06:14
*** jlibosva has quit IRC06:18
openstackgerritafazekas proposed a change to openstack/tempest: glance v2 image sharing tests  https://review.openstack.org/5012206:24
afazekasyfried: you need to add the new value to the sample as well06:25
*** tiantian has joined #openstack-qa06:26
afazekastry to avoid creating test cases (and options) which does not do anything else than reading back a configuration variable from the server side06:27
*** SergeyLukjanov is now known as _SergeyLukjanov06:29
*** SergeyLukjanov has joined #openstack-qa06:30
openstackgerritJinhe Fang proposed a change to openstack/tempest: Clean up keypair for test case test_create_specify_keypair  https://review.openstack.org/5152806:40
openstackrecheckopenstack/neutron change: https://review.openstack.org/51515 failed tempest because of: https://bugs.launchpad.net/bugs/123548606:42
Anjuafazekas:  ping06:42
Anjuafazekas:  is there any limitation that how many vm can be created in nova06:43
*** wanghao has joined #openstack-qa06:44
afazekasThe default quota is 10 /tenant06:45
yfriedafazekas: is there any support for 'ip netns' currently?06:45
afazekasotherwise it is usually limited by your memory06:45
Anjuok afazekas  thanku so much06:45
afazekasyfried: neutron using it, and in some cases tempest issues ip netns06:46
Anjuafazekas:  means we can not increase the limit06:46
afazekasAnju: nova help quota-update06:47
Anjuand afazekas  will we check it in the multinode tempest06:47
yfriedafazekas: I know neutron does, but I'm asking if tempest supports ssh and ping VMs using the internal ip, when manually you would need to use 'ip netns exec qrouter-<netid> <ssh/ping> <ip>' instead of simply '<ssh/ping> <ip>'06:48
*** jamielennox is now known as jamielennox|away06:49
afazekasyfried: If the ping/ssh working via floating ip,  why would you check internal stuff ?06:52
afazekaswith multiple l3/dhcp agent host it will be even more difficult06:53
openstackgerritYa Hong Du proposed a change to openstack/tempest: Add API test case for firewall  https://review.openstack.org/5152907:00
yfriedI need to check different connections, tenant isolation, vlan isolation, secgroup filtering (one way, 2 way), FWaaS. It's really our basic building block for all our required tests07:06
yfriedafazekas ^07:07
*** yeylon_ has joined #openstack-qa07:09
afazekasyfried: What cannot be done by http://en.wikipedia.org/wiki/Black-box_testing, without relaying in particular  implementation and deployment ?07:15
openstackgerritLijun Jiang proposed a change to openstack/tempest: Add a flavor test for getting a specified key value API  https://review.openstack.org/5153207:15
afazekasyfried: you can have any vm to make connection to another one07:15
yfriedafazekas if neutron is using netns by default, is that really a particular implementation?07:20
afazekasif you expect you have netns on the same machine as tempest running it is deployment specific assumption07:21
yfriednot on the tempest machine, on the neutron-L3-agent machine07:23
afazekasso in this case you would need to ssh to the l3 machine for doing something a network namespace.07:24
yfriedyes07:24
afazekasWhat  you cannot see as multiple user and admin related to the secgroups or tenant isolation ?07:25
yfriedafazekas, didn't understand your question07:25
afazekaswhy you need to login to the l3 machine, to see the security group rules are in use, and to see the different tenant networks does not see each other ?07:26
*** yfried has quit IRC07:34
openstackgerritLijun Jiang proposed a change to openstack/tempest: Add a flavor test for getting a specified key value API  https://review.openstack.org/5153207:35
*** afazekas is now known as __afazekas07:35
kong__afazekas, thanks!07:37
*** jlibosva1 has quit IRC07:39
*** jhenner has joined #openstack-qa07:40
*** wanghao has quit IRC07:46
*** wanghao has joined #openstack-qa07:47
*** thomasbiege has joined #openstack-qa07:53
*** afazekas has joined #openstack-qa07:57
*** giulivo has joined #openstack-qa08:09
*** yfried has joined #openstack-qa08:09
*** yfried|mob has joined #openstack-qa08:13
*** yassine has joined #openstack-qa08:14
*** SergeyLukjanov has quit IRC08:20
*** SergeyLukjanov has joined #openstack-qa08:21
openstackrecheckopenstack/tempest change: https://review.openstack.org/51532 failed tempest with an unrecognized error08:22
*** SergeyLukjanov has quit IRC08:25
*** yfried has quit IRC08:26
openstackgerritYa Hong Du proposed a change to openstack/tempest: Add API test case for firewall  https://review.openstack.org/5152908:32
*** gszasz has joined #openstack-qa08:32
*** gszasz has quit IRC08:36
*** yfried|mob has quit IRC08:42
*** yfried has joined #openstack-qa08:42
*** matsuhashi has joined #openstack-qa08:43
*** ivanzhu has quit IRC08:47
*** fbo_away is now known as fbo08:48
*** yfried1 has joined #openstack-qa08:53
*** yfried has quit IRC08:53
*** mkollaro has joined #openstack-qa08:53
*** mkollaro has quit IRC08:59
*** mkollaro has joined #openstack-qa08:59
openstackrecheckopenstack/nova change: https://review.openstack.org/44238 failed tempest because of: https://bugs.launchpad.net/bugs/121839108:59
*** jlibosva has joined #openstack-qa09:01
*** mkollaro has quit IRC09:03
*** mkollaro has joined #openstack-qa09:03
*** kong has quit IRC09:04
*** kong has joined #openstack-qa09:04
openstackgerritLijun Jiang proposed a change to openstack/tempest: Add a flavor test for getting a specified key value API  https://review.openstack.org/5153209:07
openstackrecheckopenstack/tempest change: https://review.openstack.org/51529 failed tempest with an unrecognized error09:09
openstackgerritYair Fried proposed a change to openstack/tempest: add more test cases to api/network/security_groups  https://review.openstack.org/5112309:10
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for host and seperate negative ones  https://review.openstack.org/4977809:11
*** mkollaro has quit IRC09:12
*** mkollaro has joined #openstack-qa09:17
openstackgerritYair Fried proposed a change to openstack/tempest: api/network/security_groups_negative add testcases  https://review.openstack.org/5145909:18
openstackgerritYang Yu proposed a change to openstack/tempest: Adding tests for image tags API  https://review.openstack.org/5155309:19
openstackgerritYang Yu proposed a change to openstack/tempest: Adding tests for image tags API  https://review.openstack.org/5155309:20
openstackgerritYair Fried proposed a change to openstack/tempest: Separate negative tests  api/network/security_groups  https://review.openstack.org/5145709:21
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for hypervisor operation  https://review.openstack.org/4987109:22
*** mkollaro has quit IRC09:23
*** mkollaro has joined #openstack-qa09:26
*** wanghao has quit IRC09:29
openstackgerritChenZheng proposed a change to openstack/tempest: Test for the neutron api with provider extension  https://review.openstack.org/5054209:33
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for host and seperate negative ones  https://review.openstack.org/4977809:38
*** salv-orlando has quit IRC09:39
*** gszasz has joined #openstack-qa09:39
*** tianma has joined #openstack-qa09:40
*** tiantian has quit IRC09:40
*** salv-orlando has joined #openstack-qa09:41
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for hypervisor operation  https://review.openstack.org/4987109:43
openstackgerritYang Yu proposed a change to openstack/tempest: Adding tests for image tags API  https://review.openstack.org/5155309:45
*** salv-orlando has quit IRC09:56
*** salv-orlando has joined #openstack-qa10:05
openstackrecheckopenstack/tempest change: https://review.openstack.org/50542 failed tempest because of: https://bugs.launchpad.net/bugs/123040710:07
openstackgerritSteven Hardy proposed a change to openstack/tempest: Add initial keystoneclient v3 auth tests  https://review.openstack.org/5155810:07
openstackgerritSteven Hardy proposed a change to openstack/tempest: Add initial keystoneclient trusts tests  https://review.openstack.org/5155910:07
openstackgerritSteven Hardy proposed a change to openstack/tempest: keystone : Add coverage of v2 client trust scoping  https://review.openstack.org/5156010:08
openstackgerritSteven Hardy proposed a change to openstack/tempest: keystone : Add coverage of v2 client trust scoping  https://review.openstack.org/5156010:10
openstackgerritSteven Hardy proposed a change to openstack/tempest: Add initial keystoneclient v3 auth tests  https://review.openstack.org/5155810:10
openstackgerritSteven Hardy proposed a change to openstack/tempest: Add initial keystoneclient trusts tests  https://review.openstack.org/5155910:10
*** prekarat has joined #openstack-qa10:17
*** salv-orlando has quit IRC10:19
*** prekarat has quit IRC10:19
*** prekarat has joined #openstack-qa10:19
*** salv-orlando has joined #openstack-qa10:21
*** salv-orlando_ has joined #openstack-qa10:28
*** salv-orlando has quit IRC10:29
*** salv-orlando_ is now known as salv-orlando10:29
openstackgerritCyril Roelandt proposed a change to openstack/tempest: Add missing CLI Neutron tests  https://review.openstack.org/5073910:31
*** yfried1 has quit IRC10:32
*** salv-orlando_ has joined #openstack-qa10:32
*** salv-orlando has quit IRC10:34
*** salv-orlando_ is now known as salv-orlando10:34
*** yfried has joined #openstack-qa10:39
yfriedafazekas, if I write this feature allowing support for netns (ie - ssh to L3-agent and execute commands via netns) will I be able to merge it, or will it be a total waste of time10:40
afazekasyfried: If you can prove there is behavior which needs to be tested and cannot do in any other way I am ok ip netns exec, other I would like to avoid it on the normal code path10:42
afazekasBTW: this https://github.com/openstack/tempest/blob/master/tempest/common/debug.py can be extended to do more debug op an ssh connection FAILURE, but those netns commands does not runs on a successful test10:43
openstackgerritSteven Hardy proposed a change to openstack/tempest: keystone : Add coverage of v2 client trust scoping  https://review.openstack.org/5156010:43
openstackgerritSteven Hardy proposed a change to openstack/tempest: Add initial keystoneclient trusts tests  https://review.openstack.org/5155910:43
afazekass/other/otherwise/10:44
*** tianma has quit IRC10:45
yfriedafazekas, so when does it run?10:49
afazekasNow the 'debug' commands are only running when the ssh connection fails on the neutron basic scenario test, but it can be made as more generic thing10:50
afazekasthe 'debug.py'  is not able to work in all mulitnode env10:51
openstackrecheckopenstack/neutron change: https://review.openstack.org/51522 failed tempest with an unrecognized error10:53
*** salv-orlando has quit IRC10:53
yfriedafazekas, it works only in AIO, right?10:56
yfriedsupposed you locate somehow the L3-agent host and ssh into it before running it, it will work for all env10:56
*** salv-orlando has joined #openstack-qa10:56
afazekasyes11:00
afazekasthe neutron can list the agents via an rest api call, by this way  you can find the hosts11:01
afazekashttp://www.fpaste.org/46591/48404138/11:01
afazekasthe there is trick to have the vm's to ran an arbitrary script even without ssh access11:02
afazekasnova boot --config-drive true may be able to work  without a connection to the metadata service11:04
*** dims has joined #openstack-qa11:08
*** prekarat has left #openstack-qa11:14
openstackrecheckopenstack/tempest change: https://review.openstack.org/50739 failed tempest because of: https://bugs.launchpad.net/bugs/122400111:15
*** yfried_mob has joined #openstack-qa11:20
yfried_mobafazekas, tempest.conf is looking for external network uuid. why not name?11:20
afazekasthe name are not unique, and most api call expects uuid.11:25
afazekasResolving the uuid would be an additional tempest runtime step11:25
*** tianma has joined #openstack-qa11:32
*** tianma has quit IRC11:40
*** yfried_mob has quit IRC11:45
*** yfried_mob has joined #openstack-qa11:45
openstackgerritafazekas proposed a change to openstack-infra/elastic-recheck: Add query for 1239637  https://review.openstack.org/5157911:47
yfriedafazekas, if I'm inheriting from an existing test, the "son" is running both his tests and the "father"s. how do I avoid that, if I want the son to have the same setup and tear down, and helper methods, but no to execute the father's test cases?11:49
*** adalbas has joined #openstack-qa11:49
afazekasyfried: the 'father' should not have TestCase (or derivation) base class11:51
*** yeylon_ has quit IRC11:52
yfriedafazekas, I'm talking about negative tests for the same setup (as I was told to separate negative test case to a new file).11:53
afazekasyfried: The simpler way Is to create new father without any test case, and inherit from the new father in both child11:53
yfriedafazekas, so, change the father also?11:54
afazekasIMHO creating BaseFather with the the common methods and inheriting from the BaseFather in both file can be one way11:56
afazekasBut if the common setUp and setUpClass is simple enough it can be copy-pasted11:57
afazekasIf you can create helper methods in one of the existing base.py which could make smaller the common fixture methods  also can work (and it can be the best option)11:59
afazekasBoth the code duplication and too complex inheritance model is bad, try to do the least harmful thing.12:00
*** matsuhashi has quit IRC12:18
*** matsuhashi has joined #openstack-qa12:18
*** matsuhashi has quit IRC12:22
*** prekarat has joined #openstack-qa12:24
openstackrecheckopenstack/nova change: https://review.openstack.org/50719 failed tempest with an unrecognized error12:32
*** weshay has joined #openstack-qa12:36
openstackgerritSalvatore Orlando proposed a change to openstack/tempest: API tests for neutron router gateway  https://review.openstack.org/4755412:39
openstackgerritYair Fried proposed a change to openstack/tempest: edit inheritence tree api/network/security_groups  https://review.openstack.org/5159012:39
*** yeylon_ has joined #openstack-qa12:39
*** jaypipes has joined #openstack-qa12:40
*** ChanServ sets mode: +o jaypipes12:40
*** yfried_mob has quit IRC12:42
*** yfried_mob has joined #openstack-qa12:42
*** krtaylor has joined #openstack-qa12:45
*** dims has quit IRC12:48
*** dims has joined #openstack-qa12:51
*** yfried has quit IRC12:57
*** yfried_mob has quit IRC12:59
*** jhenner has quit IRC13:00
*** yfried_mob has joined #openstack-qa13:01
*** afazekas has quit IRC13:03
*** jhenner has joined #openstack-qa13:03
openstackrecheckopenstack/python-ceilometerclient change: https://review.openstack.org/51586 failed tempest because of: https://bugs.launchpad.net/bugs/122400113:08
*** rustlebee is now known as russellb13:10
*** mriedem has joined #openstack-qa13:20
*** afazekas has joined #openstack-qa13:25
*** Guest34503 is now known as larsks13:28
*** larsks is now known as Guest5215813:28
*** Guest52158 has quit IRC13:29
silehtHi, I'm waiting for review for this https://review.openstack.org/#/c/49394/ that block ceilometer QA, someone can take a look ? sdague ?13:32
sdaguesileht: so dtroyer has a -2 on that, you aren't going to make progress in that direction. You should do the alternative and make the invocations like the nova / swift daemons are started13:34
silehtsdague, have read my argue ?13:34
sdagueyes, but it doesn't matter, dtroyer has a -2 on the patch13:35
silehtsdague, this fix a bug , and I need to use a work around ...13:35
sdaguesileht: go read dean's response, he doesn't agree13:35
*** jd__ has joined #openstack-qa13:36
silehtsdague, yep but it says the opposite of the function comment13:39
silehtsdague, have you this the impact of this bug on sar ? https://bugs.launchpad.net/devstack/+bug/123848213:40
silehtthis/see13:41
*** safchain has joined #openstack-qa13:41
*** yfried_mob has joined #openstack-qa13:43
sdagueyou are much better off making your point making the patch that makes everything work without the function change. And it if seems gross enough, maybe the function change is warrented13:43
sdaguebut you do need to realize that some times people look at things different ways, so now is really the time to give up on your other approach, because it's not going to make it through. And just insisting that it has to be done that way, means you don't get the behavior you need landed.13:44
*** yfried_mob has joined #openstack-qa13:47
*** yfried_mob has quit IRC13:47
*** yfried_mob has joined #openstack-qa13:48
yfried_mobadalbas, regarding your comment,  see Anju's comment on patch 6 here: https://review.openstack.org/#/c/51123/613:51
*** SergeyLukjanov has joined #openstack-qa13:51
yfried_mobadalbas, i'm sorry, I meant mkoderer comment13:54
adalbasyfried_mob, i had seen Anju, but not the one from mkoderer. I thought it would make sense to move to a different file if this one was too large and had too many negative tests.13:56
adalbasyfried_mob, but i'm ok with it.13:56
yfried_mobadalbas, tnx13:58
yfried_mobyf13:59
adalbasyfried_mob, tks13:59
yfried_mobadalbas, there's a third patch with more network14:06
yfried_mobmore negative tests https://review.openstack.org/#/c/51459/14:07
adalbasyfried_mob, ok. i ll take a look.14:07
silehtsdague, I just want to point something that look wrong for me, but I understand that they are many solutions, point of views, and we need to choice the better one. I will send a patch to update to comment of run_process, so other people can understand why we need to call a shell cmd before starting a daemon even this shell cmd is completly unusefull for the daemon and propose 3 three others patches to14:07
silehtfix, nova-baremetal, ceilometer and sar in gate14:07
*** afazekas has quit IRC14:08
*** jhenner has quit IRC14:09
*** yfried_mob has quit IRC14:11
*** yfried_mob has joined #openstack-qa14:12
*** SergeyLukjanov is now known as _SergeyLukjanov14:13
*** bill_az_ has joined #openstack-qa14:16
kongreview request for https://review.openstack.org/#/c/50557/,  just a very small change.14:17
*** SergeyLukjanov has joined #openstack-qa14:22
*** jhenner has joined #openstack-qa14:23
openstackgerritSteven Hardy proposed a change to openstack/tempest: keystone : Add coverage of v2 client trust scoping  https://review.openstack.org/5156014:25
openstackgerritSteven Hardy proposed a change to openstack/tempest: Add initial keystoneclient v3 auth tests  https://review.openstack.org/5155814:25
openstackgerritSteven Hardy proposed a change to openstack/tempest: Add initial keystoneclient trusts tests  https://review.openstack.org/5155914:25
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Add a comment to a hidden behavior of _run_process  https://review.openstack.org/5162114:27
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Don't kill sar prematurally in devstack-gate  https://review.openstack.org/5162214:27
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Don't kill ceilometer prematurally in devstackgate  https://review.openstack.org/5162314:27
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Don't kill nova-bm-deploy-helper prematurally  https://review.openstack.org/5162414:27
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Don't kill ceilometer prematurally in devstackgate  https://review.openstack.org/5162314:32
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Don't kill nova-bm-deploy-helper prematurally  https://review.openstack.org/5162414:32
*** liudong has joined #openstack-qa14:34
silehtdtroyer, sdague I have sent a new topic to solve the ceilometer issue that doesn't touch the run_process function http://bit.ly/1cO9k2614:38
sdaguesileht: cool, thanks!14:38
*** yfried_mob has quit IRC14:40
*** prekarat has quit IRC14:40
*** yfried_mob has joined #openstack-qa14:40
*** bill_az_ has quit IRC14:41
openstackgerritDennyZhang proposed a change to openstack-dev/devstack: remove code duplication in clean.sh  https://review.openstack.org/5163014:43
*** prekarat has joined #openstack-qa14:45
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Do not merge test of working tempest tests with neutron  https://review.openstack.org/5127514:47
*** yfried|mob has joined #openstack-qa14:48
openstackgerritDennyZhang proposed a change to openstack-dev/devstack: refine rabbit cleanup  https://review.openstack.org/5163114:50
*** yfried_mob has quit IRC14:51
openstackgerritLingxian Kong proposed a change to openstack/tempest: remove tearDownClass in test_quotas.py  https://review.openstack.org/5163214:53
*** sthaha has quit IRC14:54
*** yfried_mob has joined #openstack-qa14:59
*** yfried_mob has joined #openstack-qa15:03
*** yassine has quit IRC15:03
*** yfried|mob has quit IRC15:03
*** safchain has quit IRC15:05
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/51622 failed tempest with an unrecognized error15:08
*** yfried has joined #openstack-qa15:14
*** safchain has joined #openstack-qa15:14
*** yassine has joined #openstack-qa15:16
*** yassine has quit IRC15:19
*** yassine has joined #openstack-qa15:19
*** jlibosva has quit IRC15:25
*** yfried_mob has quit IRC15:25
openstackgerritMin Li proposed a change to openstack-dev/devstack: Fix bugs for installing docker, bug #1237581.   -The change in install_docker.sh corrects a typo ('=' should be '-').    This typo resutls in 'unable to locate the packet' error when executing apt-get.  https://review.openstack.org/5073115:26
*** yfried_mob has joined #openstack-qa15:31
openstackgerritDennyZhang proposed a change to openstack-dev/devstack: ./stack.sh complain no /etc/nova/nova.conf  https://review.openstack.org/5134615:32
*** yfried_mob has quit IRC15:35
*** yeylon_ has quit IRC15:40
*** rfolco has joined #openstack-qa15:42
*** rhsu has joined #openstack-qa15:46
openstackgerritFlorian Haas proposed a change to openstack-dev/devstack: fixup_stuff.sh: un-break site package detection for Python 2.6  https://review.openstack.org/5165115:48
openstackrecheckopenstack/tempest change: https://review.openstack.org/51275 failed tempest with an unrecognized error15:50
*** mlavalle has joined #openstack-qa15:52
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Do not merge test of working tempest tests with neutron  https://review.openstack.org/5127515:53
*** rhsu has quit IRC15:53
*** dkorn has joined #openstack-qa15:54
openstackgerritA change was merged to openstack/tempest: delete the duplicated variable definition  https://review.openstack.org/5055715:55
*** yfried_mob has joined #openstack-qa15:58
*** liudong has quit IRC15:59
openstackgerritFlorian Haas proposed a change to openstack-dev/devstack: fixup_stuff.sh: un-break site package detection for Python 2.6  https://review.openstack.org/5165116:00
*** gszasz has quit IRC16:01
*** mkollaro has quit IRC16:03
*** jhenner has quit IRC16:04
*** rhsu has joined #openstack-qa16:11
*** rhsu has quit IRC16:12
openstackgerritSteven Hardy proposed a change to openstack/tempest: keystone : Add coverage of v2 client trust scoping  https://review.openstack.org/5156016:12
openstackgerritSteven Hardy proposed a change to openstack/tempest: Add initial keystoneclient v3 auth tests  https://review.openstack.org/5155816:12
openstackgerritSteven Hardy proposed a change to openstack/tempest: Add initial keystoneclient trusts tests  https://review.openstack.org/5155916:12
*** safchain has quit IRC16:12
*** yassine has quit IRC16:13
openstackgerritBob Ball proposed a change to openstack-dev/devstack: XenAPI: Get pool default SR rather than "Local storage"  https://review.openstack.org/4886616:18
openstackgerritDennyZhang proposed a change to openstack-dev/devstack: Fix false negative, when HEAT_CREATE_TEST_IMAGE is unset  https://review.openstack.org/5133616:19
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Add a comment about how to daemonize process  https://review.openstack.org/5162116:25
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Don't kill ceilometer prematurally in devstackgate  https://review.openstack.org/5162316:25
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Don't kill sar prematurally in devstack-gate  https://review.openstack.org/5162216:25
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: Don't kill nova-bm-deploy-helper prematurally  https://review.openstack.org/5162416:25
*** fbo is now known as fbo_away16:26
openstackgerritFlorian Haas proposed a change to openstack-dev/devstack: fixup_stuff.sh: Drop pip 1.4 permissions hack  https://review.openstack.org/5165916:26
*** SergeyLukjanov has quit IRC16:31
*** SergeyLukjanov has joined #openstack-qa16:32
*** SergeyLukjanov has quit IRC16:32
*** mriedem has quit IRC16:39
*** yfried_mob has quit IRC16:44
*** rfolco has quit IRC16:44
*** yfried_mob has joined #openstack-qa16:45
openstackrecheckopenstack/tempest change: https://review.openstack.org/51559 failed tempest with an unrecognized error16:48
*** yfried|mob has joined #openstack-qa16:50
openstackrecheckopenstack/tempest change: https://review.openstack.org/51560 failed tempest with an unrecognized error16:51
*** yfried|mob has quit IRC16:54
*** yfried|mob has joined #openstack-qa16:54
*** SergeyLukjanov has joined #openstack-qa16:56
*** yfried has quit IRC16:58
*** SergeyLukjanov is now known as _SergeyLukjanov17:00
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/51651 failed tempest because of: https://bugs.launchpad.net/bugs/123543717:04
*** mlavalle has quit IRC17:05
*** SergeyLukjanov has joined #openstack-qa17:07
*** Dharmit has quit IRC17:09
*** giulivo has quit IRC17:11
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/51623 failed tempest because of: https://bugs.launchpad.net/bugs/121839117:15
openstackgerritDennyZhang proposed a change to openstack-dev/devstack: refine rabbit cleanup  https://review.openstack.org/5163117:21
*** thomasbiege has joined #openstack-qa17:22
*** thomasbiege has quit IRC17:24
*** mlavalle has joined #openstack-qa17:24
*** rockyg has joined #openstack-qa17:27
openstackgerritA change was merged to openstack-dev/devstack: Add the creation of /var/run/openstack when zeromq is chosen  https://review.openstack.org/4861617:29
openstackgerritA change was merged to openstack-dev/devstack: fix typo in functions  https://review.openstack.org/5152017:29
*** yfried has joined #openstack-qa17:33
*** yfried|mob has quit IRC17:37
*** SergeyLukjanov has quit IRC17:37
*** yfried|mob has joined #openstack-qa17:37
*** DennyZhang has joined #openstack-qa17:37
*** yeylon_ has joined #openstack-qa17:42
openstackgerritA change was merged to openstack-dev/devstack: Update diskimage-builder's URL  https://review.openstack.org/5086717:49
openstackgerritA change was merged to openstack-dev/devstack: Add meta-config via local.conf  https://review.openstack.org/4676817:49
openstackgerritA change was merged to openstack-dev/devstack: Specify ip address for nova metadata server for nicira plugin  https://review.openstack.org/4942817:49
openstackgerritA change was merged to openstack-dev/devstack: Correct logging_context_format_string for Heat  https://review.openstack.org/5057217:49
openstackgerritA change was merged to openstack-dev/devstack: Adding value for lock_path in configuration file  https://review.openstack.org/5092017:49
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/49237 failed tempest because of: https://bugs.launchpad.net/bugs/121839117:49
*** yfried has quit IRC17:50
*** mlavalle_ has joined #openstack-qa17:54
*** mlavalle has quit IRC17:55
*** mlavalle_ is now known as mlavalle17:55
*** yfried|mob has quit IRC17:56
*** DennyZhang has quit IRC17:57
*** yfried_mob has quit IRC17:57
*** mlavalle_ has joined #openstack-qa17:58
*** mlavalle has quit IRC17:59
*** mlavalle_ is now known as mlavalle17:59
*** yfried|mob has joined #openstack-qa18:08
*** yfried|mob has quit IRC18:09
*** mriedem has joined #openstack-qa18:09
openstackgerritSean Dague proposed a change to openstack-dev/devstack: add bash8 tool (like pep8, but way hackier)  https://review.openstack.org/5167618:10
sdaguedtroyer: you about?18:10
sdaguecurious about your thoughts on that - https://review.openstack.org/5167618:10
openstackgerritMatthew Treinish proposed a change to openstack-infra/elastic-recheck: Convert test_classifier to be a unit test  https://review.openstack.org/5073318:12
*** mlavalle has quit IRC18:14
openstackgerritA change was merged to openstack-dev/devstack: xenapi: max out VCPU count  https://review.openstack.org/4970318:14
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/51631 failed tempest with an unrecognized error18:14
openstackrecheckopenstack/nova change: https://review.openstack.org/51575 failed tempest because of: https://bugs.launchpad.net/bugs/121839118:14
sdaguemtreinish: don't forget to check out this for er - https://review.openstack.org/#/c/51579/18:15
sdaguewould be good to be tagging that18:15
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/49297 failed tempest with an unrecognized error18:16
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Remove query for bug 1227915  https://review.openstack.org/5168218:18
mtreinishsdague: approved18:18
sdaguemtreinish: cool18:19
*** mlavalle has joined #openstack-qa18:20
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add query for 1239637  https://review.openstack.org/5157918:21
openstackgerritA change was merged to openstack-infra/elastic-recheck: Remove query for bug 1227915  https://review.openstack.org/5168218:22
openstackrecheckopenstack/nova change: https://review.openstack.org/51670 failed tempest with an unrecognized error18:33
*** annegentle_ has quit IRC18:33
openstackrecheckopenstack/heat change: https://review.openstack.org/51671 failed tempest because of: https://bugs.launchpad.net/bugs/123040718:33
*** annegentle has joined #openstack-qa18:34
openstackrecheckopenstack/cinder change: https://review.openstack.org/51672 failed tempest with an unrecognized error18:36
mtreinishsdague: don't forget to check the tox update/changes for tempest: https://review.openstack.org/#/c/49559/ and https://review.openstack.org/#/c/49562/18:41
dtroyersdague: running between dr office and home, will have a look when I get back...18:43
openstackgerritA change was merged to openstack-dev/devstack: lib/swift variable changes and dd replaced by truncate  https://review.openstack.org/4666418:47
openstackgerritA change was merged to openstack-dev/devstack: Updates samples/localrc comment  https://review.openstack.org/5121318:47
openstackrecheckopenstack/python-keystoneclient change: https://review.openstack.org/51685 failed tempest with an unrecognized error18:58
*** SergeyLukjanov has joined #openstack-qa18:59
openstackrecheckopenstack/keystone change: https://review.openstack.org/50962 failed tempest because of: https://bugs.launchpad.net/bugs/121839119:10
*** DennyZhang has joined #openstack-qa19:25
openstackrecheckopenstack/horizon change: https://review.openstack.org/51157 failed tempest with an unrecognized error19:30
*** atiwari has joined #openstack-qa19:36
sdaguedtroyer: no worries, curious on your initial take on it. Mostly I found myself annoyed by -1ing people on whitespace stuff, and just want that automatic in devstack :)19:39
sdaguealso, I'd like to chat with you about your thoughts on the 3 devstack slots at summit, and what you wanted to get out of them?19:39
*** jaypipes has quit IRC19:41
openstackgerritDennyZhang proposed a change to openstack-dev/devstack: detect failure of 'keystone token-get'  https://review.openstack.org/5111819:44
dtroyersdague: re summit: I've got one earmarked for talking about further breakdowns of stack.sh to add hooks for calling out to vendor/user-supplied scripts in more places than we do now with the extras.d stuff.19:45
dtroyersdague: re bash8…for a second I felt _really_ out of the GNU release cycle loop… ;)  And settling down with it and a Dr Pepper now19:45
*** SergeyLukjanov is now known as _SergeyLukjanov19:46
*** _SergeyLukjanov has quit IRC19:46
*** yfried_mob has joined #openstack-qa19:47
dtroyerheh, it turns out to be shorter than I anticipated...19:47
*** yfried has joined #openstack-qa19:50
*** rockyg has quit IRC19:55
*** rockyg has joined #openstack-qa19:55
openstackgerritAaron Rosen proposed a change to openstack-dev/devstack: OVS still working?  https://review.openstack.org/5075619:55
*** yeylon_ has quit IRC19:57
*** yfried has quit IRC19:59
sdaguedtroyer: yeh, for just whitespace stuff, it's not bad19:59
sdaguethere were about 200 errors in devstack files found by it, mostly the 4 space indent20:00
dtroyerI do recall a bunch of 3-space creeping in once, I've only fixed it as I was in the area for other things...20:00
sdaguethere were some 11 space ones in big multiline blocks20:00
dtroyerfirst comments posted20:01
sdaguecoolio20:01
*** marun has joined #openstack-qa20:02
sdaguedtroyer: heh, the perl comment cracks me up :)20:02
sdaguehonestly, though I've written more perl than python, but that would just seem crazy pants to put another lang in the mix20:02
dtroyersdague: I think Perl would be the lesser of all the evils; the tools aspiers mentioned in another review needed either Haskell or Ruby runtimes.  At least Perl is already in the gate VM images20:03
sdaguedtroyer: true.20:04
sdaguethough building something new seemed to be start with python and go from there20:04
sdaguefor the error codes, a single comment list at the top good enough? or you want something code addressable?20:04
*** dkorn has quit IRC20:07
openstackgerritA change was merged to openstack-dev/devstack: Convert remaining hypervisors to plugin model  https://review.openstack.org/5074820:09
*** mlavalle has quit IRC20:10
openstackrecheckopenstack/python-keystoneclient change: https://review.openstack.org/51685 failed tempest because of: https://bugs.launchpad.net/bugs/123040720:12
*** SergeyLukjanov has joined #openstack-qa20:13
*** yfried_mob has quit IRC20:15
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/51118 failed tempest with an unrecognized error20:15
*** DennyZhang has quit IRC20:17
openstackgerritSean Dague proposed a change to openstack-dev/devstack: add bash8 tool (like pep8, but way hackier)  https://review.openstack.org/5167620:19
clarkb"but way hackier" <- wins commit message of the day award20:21
sdagueheh20:22
sdagueclarkb: hey, now that you popped up again, mind baby sitting through the os-loganalyze patch?20:23
clarkbsdague: doing it now :)20:23
sdaguecooli20:23
clarkbit was approved and merged. should be applied automagically shortly20:23
sdaguegreat, hopefully I don't blow up the whole log server in the process :)20:24
clarkbsdague: apache was restart ~2 minutes ago. I think for the vhost updates. Can you test it really quickly? I am going to poke at it too20:24
marunwould anyone like to discuss the logistics of running functional tests in the gate that require sudo?20:25
clarkbsdague: seems to work so far (I have checked syslog and keystone logs)20:25
clarkbmarun: sure, the single use hosts that we use for tempest can run tests that use sudo20:26
clarkbmarun: but the more static slaves that run unittests and the like cannot20:26
clarkbmarun: if needed you can run tests that are not tempest on the single use slaves. swift has done this with their functional tests and so has pbr20:28
sdagueclarkb: yep, looks like it's working20:28
sdaguewhen does console.html get gzipped?20:28
sdagueI thought I'd seen that some times20:28
clarkbsdague: it is on a 4 hour cron timer20:29
marunclarkb: how would I arrange to have that set up?20:29
clarkbmarun: just submit a change to openstack-infra/config that configures the job and runs it. Let me see if I can find an example sha20:29
marunclarkb: cool20:29
sdaguemarun: we already turned on iptables and ip for sudo for the devstack jobs for neutron to debug20:30
sdaguethe devstack jobs are now running tempest under a tempest user20:30
sdaguewhich has limitted sudo authority20:30
sdagueso that would provide a good model to replicate20:31
marunsdague: the question is how tests that get submitted and require sudo for a given command would update the job20:31
marunsdague: Note that the rootwrap would allow the commands in question, so would that be a good way to enable sudo usage?20:31
sdaguemarun: for right now we did that pretty manually, so that sudoing authority would be limitted20:31
clarkbmarun: actually it ended up going in in several pieces. but if you look in modules/openstack_project/files/jenkins_job_builder/config/swift-functional.yaml you will see the job. Then in modules/openstack_project/files/zuul/layout.yaml you can see where the job is run20:32
sdaguemarun: about to get on the phone, but an example of what you wanted to do might be helpful20:32
sdagueto help me think about the ways we'd do that20:32
marunsdague: I would say allowing password-less sudo for the rootwrap with the config that is in the neutron tree.20:33
marunsdague: This would allow functional testing of the commands as well as validation that the rootwrap config is correct.20:33
sdaguemarun: right, but provide a scenario about what you are doing with that authority?20:33
marunsdague: The change I'm looking to get merged has a functional test to validate management of an async instance of ovsdb-client20:34
marunsdague: https://review.openstack.org/#/c/45678/20:34
marunsdague: I currently require an environment variable to get set for the test to run (OS_SUDO_TESTING)20:34
marunsdague: The test in question is probably more useful than the review: https://review.openstack.org/#/c/45678/10/neutron/tests/functional/agent/linux/test_ovsdb_monitor.py20:35
sdagueok, have another thought, but need to run to meeting20:35
marunsdague: We don't do much functional testing in neutron right now, but I want to see that change.  We need to validate system interaction, not just at the tempest level20:35
sdagueback in a bit20:35
marunsdague: ok20:35
marunsdague: I'm in GMT+2 right now, so I might not be here when you return.  Please email to me or to the list with your thoughts if you can't find me.20:36
*** rockyg has quit IRC20:38
sdaguemarun: my thought is actually quick. What if neutron itself had a diagnotic api that would do the dump of that kind of thing to the logs20:40
marunsdague: that's not desirable in this case20:40
sdaguebecause of the unit tests?20:40
marunsdague: the intention is to isolate the interaction, not go through any api20:40
sdagueok, no prob20:40
marunsdague: +1 on having diagnostics via an api, frankly, but I think that fits better with an sdn backend20:41
marunsdague: the current agent model is problematic to extend to provide such functionality20:41
sdaguewell, I was hoping we'd get it in a general case, so we could do certain verification in tempest tests20:44
sdaguebut if it's problematic, such it is20:44
*** SergeyLukjanov has quit IRC20:47
openstackrecheckopenstack/neutron change: https://review.openstack.org/51696 failed tempest with an unrecognized error20:47
*** SergeyLukjanov has joined #openstack-qa20:48
*** mlavalle has joined #openstack-qa20:49
*** jamielennox|away is now known as jamielennox20:56
openstackrecheckopenstack/neutron change: https://review.openstack.org/45676 failed tempest with an unrecognized error20:59
openstackrecheckopenstack/keystone change: https://review.openstack.org/50966 failed tempest with an unrecognized error21:01
*** HenryG has joined #openstack-qa21:01
*** adalbas has quit IRC21:05
*** mriedem has quit IRC21:34
openstackgerritA change was merged to openstack-dev/devstack: Adds trove-conductor service to trove.  https://review.openstack.org/4923721:35
*** mlavalle has quit IRC21:40
openstackgerritSean Dague proposed a change to openstack-dev/devstack: add a simple run_tests.sh to use bash8  https://review.openstack.org/5171121:47
*** hemna is now known as hemnafk21:50
openstackgerritSean Dague proposed a change to openstack-dev/devstack: add a simple run_tests.sh to use bash8  https://review.openstack.org/5171121:55
openstackgerritSean Dague proposed a change to openstack-dev/devstack: add a simple run_tests.sh to use bash8  https://review.openstack.org/5171121:57
openstackrecheckopenstack/python-ceilometerclient change: https://review.openstack.org/51586 failed tempest with an unrecognized error22:06
*** syerrapragada has joined #openstack-qa22:12
*** psedlak has quit IRC22:19
*** psedlak has joined #openstack-qa22:26
*** syerrapragada has left #openstack-qa22:29
*** marun has quit IRC22:33
openstackrecheckopenstack/neutron change: https://review.openstack.org/51696 failed tempest with an unrecognized error22:35
*** SergeyLukjanov has quit IRC22:46
openstackrecheckopenstack/python-keystoneclient change: https://review.openstack.org/51685 failed tempest with an unrecognized error22:52
*** SergeyLukjanov has joined #openstack-qa22:53
openstackrecheckopenstack/nova change: https://review.openstack.org/50953 failed tempest with an unrecognized error22:54
openstackrecheckopenstack/nova change: https://review.openstack.org/50494 failed tempest because of: https://bugs.launchpad.net/bugs/121839122:56
openstackrecheckopenstack/nova change: https://review.openstack.org/50493 failed tempest with an unrecognized error23:02
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Add query for bug 1239856  https://review.openstack.org/5172823:11
*** jamielennox is now known as jamielennox|away23:12
openstackrecheckopenstack/python-ceilometerclient change: https://review.openstack.org/51584 failed tempest with an unrecognized error23:25
*** atiwari has quit IRC23:37
openstackgerritClint Byrum proposed a change to openstack/tempest: Change six to match global requirements  https://review.openstack.org/5173623:42
openstackrecheckopenstack/python-ceilometerclient change: https://review.openstack.org/51727 failed tempest with an unrecognized error23:43
*** SergeyLukjanov is now known as _SergeyLukjanov23:53
*** _SergeyLukjanov has quit IRC23:53
*** SergeyLukjanov has joined #openstack-qa23:54
*** jamielennox|away is now known as jamielennox23:59

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