Friday, 2013-08-16

« Thursday, 2013-08-15 Index Saturday, 2013-08-17 »
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create_server" method  https://review.openstack.org/4207100:08
openstackgerritlawrancejing proposed a change to openstack/tempest: Fixes files with wrong bitmode  https://review.openstack.org/4168600:40
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create_server" method  https://review.openstack.org/4207101:14
openstackgerritNachi Ueno proposed a change to openstack-dev/devstack: Support OpenSwan in Neturon VPNaaS  https://review.openstack.org/4226501:33
*** mriedem has joined #openstack-qa01:37
*** torandu has quit IRC01:50
*** anteaya has quit IRC01:51
*** riverrat has joined #openstack-qa01:52
*** riverrat has quit IRC02:26
*** LiangZheng has joined #openstack-qa02:29
*** jasondotstar has quit IRC02:38
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create_server" method  https://review.openstack.org/4207102:54
openstackgerritA change was merged to openstack-dev/devstack: Added functions for get a ip on an instance  https://review.openstack.org/4179902:55
*** mriedem has quit IRC02:58
*** prekarat has joined #openstack-qa03:04
*** prekarat has quit IRC03:05
*** prekarat has joined #openstack-qa03:05
*** SergeyLukjanov has joined #openstack-qa03:32
*** adalbas has quit IRC03:36
*** adalbas has joined #openstack-qa03:37
*** ivanzhu has joined #openstack-qa04:00
*** ivanzhu has quit IRC04:14
*** yfried has joined #openstack-qa04:23
afazekas_zzmkoderer: https://review.openstack.org/#/c/41635/504:23
*** afazekas_zz is now known as afazekas04:23
*** prekarat has quit IRC04:53
*** prekarat has joined #openstack-qa04:56
*** afazekas_ has joined #openstack-qa04:58
*** SergeyLukjanov has quit IRC05:05
*** jerryz has quit IRC05:26
*** yfried has quit IRC05:26
mkodererafazekas: pong06:07
mkoderernews about https://review.openstack.org/#/c/41635/5?06:07
*** thomasbiege has joined #openstack-qa06:07
mkodererok I am reading the comments ;)06:08
*** thomasbiege has quit IRC06:11
*** prekarat has quit IRC06:27
*** prekarat has joined #openstack-qa06:30
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create_server" method  https://review.openstack.org/4207106:40
openstackgerritSumanth Nagadavalli proposed a change to openstack/tempest: Fixed test for non-public flavor  https://review.openstack.org/4163506:43
*** jerryz has joined #openstack-qa06:44
*** jerryz has quit IRC06:57
*** jerryz has joined #openstack-qa06:58
*** afazekas has quit IRC07:06
*** afazekas_ is now known as afazekas07:06
afazekasmkoderer: assertFalse used instead of assertTrue07:06
mkodereroh didn't see that07:07
afazekasI just saw it is not add test case, but the commit makes sense07:07
*** jerryz has quit IRC07:08
mkodererafazekas: btw http://www.openstack.org/rate/Presentation/stress-testing-openstack-chuck-norris-style07:09
mkoderervote if you like ;)07:09
*** prekarat has quit IRC07:15
*** prekarat has joined #openstack-qa07:15
*** prekarat has quit IRC07:16
*** prekarat has joined #openstack-qa07:16
openstackgerritafazekas proposed a change to openstack/tempest: Updating HACKING.rst  https://review.openstack.org/4229607:23
afazekasmkoderer: I am thinking about adding setUpProcess and tearDownProcess method to the base stressaction07:24
afazekasand an etexit call for the tearDownProcess07:25
mkodererok so you want to rename the existing methods or just add two?07:26
afazekasadding two additional07:26
afazekasdid you tried the unittest as stress test change ?07:27
mkodererok and tearDown will call tearDownProcess?07:27
afazekasIf it working as I expect, more life-cycle phase can be added07:27
mkoderernot yet - hopefully next week07:27
afazekasif that is working we 3 lifecycle step07:28
mkodererso did you read what I wrote yesterday in the qa meeting?07:28
afazekasonce when the action read and initialized07:28
afazekasjust the stress tag might not be enough, we might add  some options as well07:29
afazekasSome additional tuning is still possible on the pending patch ..07:29
*** LiangZheng has quit IRC07:43
*** psedlak has quit IRC07:44
mkodererafazekas: yes I know - I have various parameters for the decorator currently07:57
*** SergeyLukjanov has joined #openstack-qa07:57
mkodererbut it need to be finialzed07:57
*** jerryz has joined #openstack-qa07:59
afazekasIMHO test cases which does not needs admin permission (excluding the tenant isolation), should get a 'user' tag08:00
afazekasand they should be the primary target for stress testing08:00
*** giulivo has joined #openstack-qa08:01
afazekaswhen we get keyston v3 and domain admins, they should be part of the main stress test subject08:01
afazekasThe uber king admin can have too many side effect, so we should select them more carefully08:02
afazekashttps://review.openstack.org/#/c/41026/08:04
afazekasmkoderer: ^^08:04
afazekaswhat do you do if the keysone contains multiple region entries for the same service ?08:05
mkoderermh I scanned the source and never saw such a case08:06
afazekasmkoderer: because one devstack node is one region :)08:06
mkoderermaybe I don't get the point in this patch08:07
*** psedlak has joined #openstack-qa08:07
mkodereryou have a dict and write an every service the same region08:08
mkodererok but the comment gives a hint08:09
mkodererafazekas: so it's fine08:09
afazekasit need to be in the sample conf imho08:10
afazekasthe authentication in the rest_client is bad08:10
*** jerryz has quit IRC08:11
afazekasSome  dirty code is in the code base because of this (I can't recall to the exact example)08:11
mkodererafazekas: the thing is the conf is unchange with this patch08:11
mkodererall services get the region assigned automatically08:12
mkodererafazekas: is there an example where the region is manipulated by a subclass of RestClient?08:26
mkodererafazekas: I really don't find any08:26
mkodererif there is no one using this dict for this purpose. we should delete it08:27
*** fbo_away is now known as fbo08:37
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create security rule" methods  https://review.openstack.org/4230308:40
*** LiangZheng has joined #openstack-qa08:43
*** LiangZheng has quit IRC08:48
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Cleanup: Add common "create security rule" methods  https://review.openstack.org/4230308:51
openstackgerritIvan-Zhu proposed a change to openstack/tempest: port test_images and test_server_actions into v3 part2  https://review.openstack.org/3960909:02
*** pasquier-s has joined #openstack-qa09:25
afazekasmkoderer: I guess the committer wants to use It, and it does not cause harm to the gate09:41
*** xBsd has joined #openstack-qa09:42
xBsdping09:43
xBsdwhat happened with zulu and jenkins? who know?09:43
*** LiangZheng has joined #openstack-qa09:44
mkodererafazekas: more unused complexity harms the code base...09:45
*** LiangZheng has quit IRC09:48
*** thomasbiege has joined #openstack-qa09:49
*** thomasbiege has quit IRC09:50
afazekasmkoderer: in modeling view point the regions are exists, so it should be part of the logic, and we should not cry when we need regions for a test case10:30
*** LiangZheng has joined #openstack-qa10:44
*** LiangZheng has quit IRC10:49
*** raies has joined #openstack-qa10:49
mkodererafazekas: for me it's ok to have this dict but then the configuration must be adapted accordingly10:58
mkodererit's just confusing IMHO10:58
mkodererif we want to allow different regions for each service we should be able to configure that11:00
openstackgerritafazekas proposed a change to openstack/tempest: File header changes and rules  https://review.openstack.org/4231711:03
mkodererafazekas: or we could introduce the regions things when needed11:06
*** giulivo has quit IRC11:11
*** giulivo has joined #openstack-qa11:17
*** mriedem has joined #openstack-qa11:17
openstackgerritafazekas proposed a change to openstack-dev/devstack: .conf suffix for the horizon config on Fedora  https://review.openstack.org/4035211:18
afazekasmkoderer: IMHO you can ask some more why and more about future plans on the review11:20
mkodererafazekas: ok it try to be more precise11:21
openstackgerritafazekas proposed a change to openstack-infra/devstack-gate: Using the jenkins user for tempest run  https://review.openstack.org/4210111:30
*** thomasbiege has joined #openstack-qa11:42
*** raies has quit IRC11:43
*** SnowDust has joined #openstack-qa11:44
*** LiangZheng has joined #openstack-qa11:45
*** SnowDust has quit IRC11:46
*** thomasbiege has quit IRC11:48
afazekasgiulivo: AFAIK the vim line is there in most cases and it is used by another projects as well11:48
afazekasthe oslo modules contains them, so I can not kill them11:49
*** LiangZheng has quit IRC11:50
*** weshay has joined #openstack-qa11:57
*** jhenner has joined #openstack-qa12:07
*** jhenner2 has quit IRC12:07
openstackgerritafazekas proposed a change to openstack/tempest: File header changes and rules  https://review.openstack.org/4231712:09
giulivoafazekas, yeah I was going to propose to ensure they are correctly set on all files, not to strip them out12:09
giulivooh you did that, I think it's great12:11
*** mberwanger has joined #openstack-qa12:18
giulivoafazekas, by using a script I spotted a few more files where the tabstop thing is missing12:19
giulivodo you want me to point you at those files so you can update them in a new patches or should I do that myself after your submission is merged?12:20
giulivoafazekas, maybe better if you do it, it is only one file ./tempest/api/compute/servers/test_attach_interfaces.py12:22
openstackgerritafazekas proposed a change to openstack/tempest: Protected matcher import  https://review.openstack.org/4232512:24
*** torandu has joined #openstack-qa12:25
afazekasgiulivo: my script also see it, just I am blind :)12:26
giulivonp I like these "standardization" things12:26
afazekas./doc/source/conf.py is WONTFIX12:27
*** torandu has quit IRC12:28
*** torandu has joined #openstack-qa12:28
*** torandu has quit IRC12:29
openstackgerritafazekas proposed a change to openstack/tempest: File header changes and rules  https://review.openstack.org/4231712:30
*** SergeyLukjanov has quit IRC12:30
afazekasgiulivo: https://review.openstack.org/#/c/42296/  do you need more info about this ?12:31
*** torandu has joined #openstack-qa12:31
*** woodspa has joined #openstack-qa12:31
*** xBsd has left #openstack-qa12:31
giulivoafazekas, so maybe we can add as pointer a link to the testtools assertions?12:33
*** mberwanger has quit IRC12:33
*** mberwanger has joined #openstack-qa12:34
*** mberwang_ has joined #openstack-qa12:35
*** mberwanger has quit IRC12:38
afazekashttp://testtools.readthedocs.org/en/latest/for-test-authors.html#writing-your-own-matchers12:39
*** thomasbiege has joined #openstack-qa12:41
*** jasondotstar has joined #openstack-qa12:42
*** LiangZheng has joined #openstack-qa12:46
*** thomasbiege2 has joined #openstack-qa12:48
*** thomasbiege has quit IRC12:49
openstackgerritafazekas proposed a change to openstack-infra/devstack-gate: Using the jenkins user for tempest run  https://review.openstack.org/4210112:50
*** LiangZheng has quit IRC12:50
giulivoafazekas, that would be great yes12:51
giulivogood link12:51
*** mberwang_ has quit IRC12:56
*** mkollaro has joined #openstack-qa12:59
*** SergeyLukjanov has joined #openstack-qa13:00
*** anteaya has joined #openstack-qa13:03
*** thomasbiege2 has quit IRC13:05
*** russellb is now known as rustlebee13:11
*** comstud is now known as bearhands13:11
*** mberwanger has joined #openstack-qa13:12
*** thomasbiege has joined #openstack-qa13:14
*** mberwanger has quit IRC13:21
*** thomasbiege has quit IRC13:24
*** thomasbiege has joined #openstack-qa13:25
openstackgerritAlessio Ababilov proposed a change to openstack-dev/devstack: Fix olso typo in lib/oslo  https://review.openstack.org/4234413:30
*** openstackgerrit has quit IRC13:36
*** openstackgerrit has joined #openstack-qa13:37
openstackgerritA change was merged to openstack-dev/devstack: Colorize heat engine log  https://review.openstack.org/4034213:38
*** VinodGupta has joined #openstack-qa13:42
*** thomasbiege has quit IRC13:43
*** thomasbiege has joined #openstack-qa13:45
*** SergeyLukjanov has quit IRC13:45
openstackgerritafazekas proposed a change to openstack/tempest: Updating HACKING.rst  https://review.openstack.org/4229613:46
*** LiangZheng has joined #openstack-qa13:46
*** burt has joined #openstack-qa13:48
*** SergeyLukjanov has joined #openstack-qa13:51
*** LiangZheng has quit IRC13:51
openstackgerritadalbas proposed a change to openstack/tempest: Remove identity race condition  https://review.openstack.org/4235113:57
mtreinishafazekas: have you seen this: https://jenkins01.openstack.org/job/gate-tempest-devstack-vm-testr-full/1926/console ?13:57
mtreinishit looks like it's failing on your setup/teardown class detector13:58
mtreinishafazekas: I've seen that fail a few times on the testr parallel runs14:03
*** HenryG has joined #openstack-qa14:05
afazekastheKeyError: <class 'tempest.api.compute.images.test_list_image_filters.ListImageFiltersTestJSON'> is secondary thing14:05
afazekasI will add forced super call decorator prototype to the tearDownClass  and forced tearDownClass call on error decorator to the setUpclasses14:06
* afazekas hmmm14:07
mtreinishafazekas: oh, I see now that test class has a catchall for exceptions an then manually calls tearDownClass14:07
mtreinishwhich is weird14:07
afazekasif the setup fails the tearDown very frequently fails as well14:08
*** krtaylor has quit IRC14:08
afazekasI guess you will see more infro from the detailed log14:09
afazekasDo you know how the setUpClass is handled by subunit ?14:10
afazekasCan we turn on some kind of log capturing for the setUpClasses ?14:10
mtreinishafazekas: shouldn't it go to the tempest.log file?14:11
afazekasyes it should be always in the tempest.log file14:11
*** torandu has quit IRC14:11
afazekasBut IMHO some additional info in the console never harms14:12
afazekashttps://review.openstack.org/#/c/42071/14:14
afazekasgiulivo: IMHO if you are asking something a review you should not -1 it.14:15
giulivoafazekas, well considering we're trying to rewrite that into a generic utility method, I think we should strip that unless we're sure about what it does14:17
giulivoalso I'm not sure about the hidden/non-hidden naming choice, I'd actually make the NetworkScenarioTest methods non-hidden14:18
afazekasgiulivo: the change fixes more thing14:18
giulivoso the question is, is it worth doing these changes in the single submission^14:18
afazekasgiulivo: it has dependent change for other issues14:18
afazekasthe change is big enough14:18
giulivook I get what you mean, to move by small stesp14:19
giulivo*steps14:19
*** VinodGupta has quit IRC14:28
openstackgerritA change was merged to openstack/tempest: Fix typos in tempest/api/README.rst  https://review.openstack.org/4206314:31
afazekasgiulivo: ~30 self.fail remaining, deleting patch is welcome14:31
giulivoafazekas, glad you noticed the blueprint :)14:32
afazekas~ 2014:33
afazekasit is in the HACKING.rst14:33
giulivonp it was just some sort of "reminder", yeah I started from there indeed14:33
openstackgerritA change was merged to openstack/tempest: skip test_list_servers_filtered_by_ip_regex on neutron gate  https://review.openstack.org/4212514:34
openstackgerritA change was merged to openstack/tempest: Fixed test for non-public flavor  https://review.openstack.org/4163514:39
openstackgerritA change was merged to openstack/tempest: Add nose to run_tests and tox for python 2.6  https://review.openstack.org/4221514:46
*** LiangZheng has joined #openstack-qa14:47
mtreinishadalbas: https://jenkins01.openstack.org/job/gate-tempest-devstack-vm-testr-full/1932/console & https://jenkins01.openstack.org/job/gate-tempest-devstack-vm-testr-full/1928/console14:49
*** krtaylor has joined #openstack-qa14:51
afazekasmtreinish: What is your recomended log capturing level ? INFO/DEBUG/ERROR ?14:51
*** LiangZheng has quit IRC14:52
mtreinishadalbas: https://jenkins01.openstack.org/job/gate-tempest-devstack-vm-testr-full/buildTimeTrend14:54
*** pasquier-s has quit IRC14:54
openstackgerritafazekas proposed a change to openstack-infra/devstack-gate: Using the root user for tempest run  https://review.openstack.org/4210114:56
afazekasmtreinish: the ERROR is good enough if just want to see the LOG.exceptions() and several additional minimal14:59
afazekasINFO gives minimal info about what happened15:00
afazekasDEBUG tells almost everything (it is already size limited)15:00
*** sthaha has quit IRC15:00
*** woodspa_ has joined #openstack-qa15:01
mtreinishafazekas: for general use or in the gate?15:01
afazekason the gate15:02
afazekastempest.log must be DEBUG15:02
afazekasI am asking about the console15:02
afazekason error or failure15:02
mtreinishideally I'd say we'd want error or warn for the console output15:04
mtreinishbut then we wouldn't get the req-ids...15:04
*** woodspa has quit IRC15:04
afazekaslogging the id on ERROR or WARN is not too good thing15:05
mtreinishyeah I agree15:05
afazekasself.LOG.info("Response Status: " can be on DEBUG level15:07
mtreinishyeah and maybe the Request log messages too15:08
afazekasthe request enough important  to be on info15:10
afazekasthe response code, usually reises an exception if it was not OK, so does not really need to be on info15:11
afazekasthe request body is logged only on DEBUG level15:12
afazekasdoes not matters, you can  move it to debug as well15:16
afazekasso is the INFO level ok , for the general capturing to the console usage ?15:17
afazekascyeoh, mtreinish, jaypipes:  https://review.openstack.org/#/c/42082/15:19
jaypipesafazekas: done15:21
openstackgerritraiesmh08 proposed a change to openstack/tempest: Adding network api xml support  https://review.openstack.org/4167315:25
mtreinishadalbas: ^^^ https://review.openstack.org/#/c/42082/ you said that you were going to look into the exercises vs smoke15:26
afazekasjaypipes: thank you15:27
mtreinishafazekas: reviewed15:31
*** krtaylor has quit IRC15:32
*** LiangZheng has joined #openstack-qa15:48
openstackgerritA change was merged to openstack/tempest: Remove positive tag  https://review.openstack.org/4145615:48
openstackgerritafazekas proposed a change to openstack-infra/devstack-gate: Skip devstack/exercises by default  https://review.openstack.org/4208215:50
*** LiangZheng has quit IRC15:52
afazekasmtreinish: thank you16:05
afazekasSo I will change the logging to to INFO16:05
afazekasI think sooner a letter , I or someone else will be find the way to capture the logging and stdout/stderr in the setUpClasses16:06
afazekasLooks like today I will not have time time to to the force caller decorators, but they will run in the time when they have a chance to do it16:07
*** woodspa__ has joined #openstack-qa16:08
*** fbo is now known as fbo_away16:11
*** woodspa_ has quit IRC16:12
openstackgerritafazekas proposed a change to openstack/tempest: per test_method logging  https://review.openstack.org/4206916:12
*** giulivo has quit IRC16:17
openstackgerritafazekas proposed a change to openstack-dev/devstack: Configure bash completion  https://review.openstack.org/4192816:19
*** krtaylor has joined #openstack-qa16:19
openstackgerritafazekas proposed a change to openstack-dev/devstack: Configure bash completion  https://review.openstack.org/4192816:24
openstackgerritafazekas proposed a change to openstack-dev/devstack: Configure bash completion  https://review.openstack.org/4192816:26
*** riverrat has joined #openstack-qa16:26
openstackgerritafazekas proposed a change to openstack-dev/devstack: .conf suffix for the horizon config on Fedora  https://review.openstack.org/4035216:27
*** afazekas has quit IRC16:37
*** mriedem1 has joined #openstack-qa16:40
*** mriedem has quit IRC16:40
openstackgerritA change was merged to openstack/tempest: Cleanup: Add common "create_server" method  https://review.openstack.org/4207116:47
*** LiangZheng has joined #openstack-qa16:48
*** LiangZheng has quit IRC16:53
*** krtaylor has quit IRC17:06
*** psedlak has quit IRC17:20
*** SergeyLukjanov has quit IRC17:22
*** thomasbiege has quit IRC17:36
*** burt has quit IRC17:41
*** afazekas has joined #openstack-qa17:41
*** LiangZheng has joined #openstack-qa17:50
*** LiangZheng has quit IRC17:55
*** dmakogon_ has joined #openstack-qa18:22
dmakogon_hey, guys, we seeing bug in devstack, this bug failing gerrit build of Trove dashboard18:23
dmakogon_log: http://logs.openstack.org/28/42228/2/check/gate-tempest-devstack-vm-cells-full/12d70e4/console.html18:23
*** mkollaro has quit IRC18:24
*** VinodGupta has joined #openstack-qa18:26
*** VinodGupta has left #openstack-qa18:32
*** SergeyLukjanov has joined #openstack-qa18:40
*** LiangZheng has joined #openstack-qa18:50
*** LiangZheng has quit IRC18:54
openstackgerritAlessio Ababilov proposed a change to openstack-dev/devstack: Fix 'olso' typo in lib/oslo  https://review.openstack.org/4234418:57
*** thomasbiege has joined #openstack-qa19:00
*** thomasbiege has quit IRC19:03
mtreinishadalbas: did you say you were looking at the resize bug or the rescue bug?19:16
mtreinishhave 2 bugs both starting with 're' is very confusing...19:16
adalbasmtreinish, i said rescue bug (that s the one i saw). But you can decide which. I haven't started there yet19:17
mtreinishadalbas: no it doesn't matter I'm just opening bugs for them and I wanted to make sure that I assigned you the right one.19:18
mtreinishI can hold off on doing that and just keep them open19:18
adalbasmtreinish, ok. go on and add me to the rescue one19:19
*** mriedem1 has quit IRC19:22
openstackgerritafazekas proposed a change to openstack-dev/devstack: Defining a bash completion function  https://review.openstack.org/4240219:27
*** mriedem has joined #openstack-qa19:39
*** LiangZheng has joined #openstack-qa19:51
*** psedlak has joined #openstack-qa19:52
*** weshay has quit IRC19:54
*** LiangZheng has quit IRC19:55
*** psedlak has quit IRC20:08
*** krtaylor has joined #openstack-qa20:19
*** woodspa__ is now known as woodspa20:34
*** LiangZheng has joined #openstack-qa20:52
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Add support for Docker as Nova hypervisor  https://review.openstack.org/4075920:56
*** LiangZheng has quit IRC20:56
*** jasondotstar has quit IRC21:04
*** adalbas has quit IRC21:14
*** thomasbiege has joined #openstack-qa21:18
*** adalbas has joined #openstack-qa21:29
*** adalbas has quit IRC21:38
*** LiangZheng has joined #openstack-qa21:52
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Fix skip tracker regex for multi-line skips.  https://review.openstack.org/4242421:54
*** LiangZheng has quit IRC21:56
openstackgerritClark Boylan proposed a change to openstack-dev/devstack: Echo service start failures.  https://review.openstack.org/4242721:59
*** mriedem has quit IRC22:09
*** woodspa has quit IRC22:12
*** SergeyLukjanov has quit IRC22:25
*** mriedem has joined #openstack-qa22:35
*** LiangZheng has joined #openstack-qa22:53
*** LiangZheng has quit IRC22:57
*** jasondotstar has joined #openstack-qa23:04
*** riverrat has left #openstack-qa23:22
openstackgerritA change was merged to openstack-dev/devstack: Fix 'olso' typo in lib/oslo  https://review.openstack.org/4234423:25
*** LiangZheng has joined #openstack-qa23:53
*** LiangZheng has quit IRC23:58
« Thursday, 2013-08-15 Index Saturday, 2013-08-17 »

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