| *** hemna is now known as hemnafk | 00:06 | |
| *** mberwanger has quit IRC | 00:12 | |
| openstackgerrit | A change was merged to openstack/tempest: Use assertIn and assertNotIn instead of assertTrue/assertFalse https://review.openstack.org/39019 | 00:52 |
|---|---|---|
| *** dkranz has joined #openstack-qa | 00:53 | |
| *** dosaboy has quit IRC | 00:59 | |
| *** dkranz has quit IRC | 01:04 | |
| openstackgerrit | A change was merged to openstack/tempest: Add Neutron CLI tests to tempest https://review.openstack.org/38829 | 01:08 |
| openstackgerrit | A change was merged to openstack/tempest: Unskip test_register_get_deregister_ari_image https://review.openstack.org/38744 | 01:08 |
| openstackgerrit | A change was merged to openstack/tempest: Handle a possible volume attachment visibility wait/race issue https://review.openstack.org/38841 | 01:08 |
| openstackgerrit | A change was merged to openstack/tempest: Simplify whitebox/manager https://review.openstack.org/35302 | 01:09 |
| openstackgerrit | A change was merged to openstack/tempest: Ignore the default quota values https://review.openstack.org/36941 | 01:09 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Fix various things when n-cell is enabled https://review.openstack.org/38897 | 01:14 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Update OpenVZ driver image https://review.openstack.org/38936 | 01:14 |
| openstackgerrit | A change was merged to openstack-dev/devstack: xenapi: Uninstall instances https://review.openstack.org/39042 | 01:14 |
| openstackgerrit | A change was merged to openstack/tempest: Add tenant isolation for glance api tests https://review.openstack.org/38371 | 01:25 |
| *** anteaya has quit IRC | 01:30 | |
| openstackgerrit | Ravi Chunduru proposed a change to openstack-dev/devstack: blueprint quantum-fwaas-devstack Adds support for Openstack Networking FWaaS (Firewall) https://review.openstack.org/37147 | 01:33 |
| openstackgerrit | A change was merged to openstack/tempest: Allow to run swift and keystone api tests standalone https://review.openstack.org/39001 | 01:34 |
| openstackgerrit | Arata Notsu proposed a change to openstack-dev/devstack: Add option to serve PXE for baremetal from nova-network https://review.openstack.org/38794 | 01:49 |
| openstackgerrit | Alex Vasilyev proposed a change to openstack-dev/devstack: Add Midonet plugin support to devstack. https://review.openstack.org/36607 | 02:00 |
| openstackgerrit | John Griffith proposed a change to openstack/tempest: Add wait_for_volume_status in upload_to_image test https://review.openstack.org/39178 | 02:04 |
| *** woodspa_ has quit IRC | 02:23 | |
| *** mberwanger has joined #openstack-qa | 03:08 | |
| *** HenryG has quit IRC | 03:10 | |
| *** hrebicek_ has quit IRC | 03:24 | |
| ianw | lifeless: hi, i'm trying to debug a testr issue; hoping you have some insights | 03:37 |
| ianw | aggregate_name = rand_name(self.aggregate_name_prefix) | 03:38 |
| ianw | AttributeError: 'AggregatesAdminTestJSON' object has no attribute 'aggregate_name_prefix' | 03:38 |
| *** marun has quit IRC | 03:38 | |
| ianw | it seems like the class isn't being setup correctly, if that's possible | 03:38 |
| afazekas__ | ianw: subunit version ? | 03:41 |
| ianw | afazekas__: hi! ... python_subunit-0.0.13-py2.6.egg-info | 03:44 |
| ianw | this is tempest, first time i'm really trying running with testr | 03:44 |
| *** mberwang_ has joined #openstack-qa | 03:45 | |
| ianw | afazekas__: i can put completely invalid code in setUpClass and get no syntax error, so i'm guessing it's not being called | 03:46 |
| ianw | huh, and if i modify it | 03:47 |
| ianw | @attr(type=['negative', 'gate']) | 03:47 |
| ianw | def test_aggregate_add_host_as_user(self): | 03:47 |
| ianw | # Regular user is not allowed to add a host to an aggregate. | 03:47 |
| ianw | self.setUpClass() | 03:47 |
| ianw | aggregate_name = rand_name(self.aggregate_name_prefix) | 03:47 |
| ianw | that works; so yeah, setUpClass isn't being called i guess is the root issue | 03:48 |
| *** afazekas__ has quit IRC | 03:48 | |
| *** mberwanger has quit IRC | 03:48 | |
| clarkb | ianw py26 doesnt do setupClass I thought | 03:52 |
| clarkb | well unittest/testools doesnt under py26 | 03:53 |
| ianw | clarkb: yeah, i'm getting to that conclusion :) but i don't understand how it works under nose yet | 03:53 |
| clarkb | nose supports it | 03:54 |
| clarkb | as part of its runner stuff | 03:54 |
| ianw | oh, ok, that makes sense | 03:54 |
| ianw | clarkb: do you know off-hand if unittest2 supports it? | 03:56 |
| clarkb | it does | 03:56 |
| ianw | ok, that might be the solution then, thanks | 03:57 |
| clarkb | well no. unittest2 is meh | 03:57 |
| clarkb | preferably testools + testresources would happen but that is down the road aiui | 03:58 |
| ianw | clarkb: making testr support running it? | 03:58 |
| clarkb | testresources and fixtures would run under py26 | 03:58 |
| ianw | nose has at least two issues (incompat with unittest skip exception & ability to skip tests with --exclude) that make it annoying for CI testing with python2.6 | 03:59 |
| ianw | unfortunately upstream seems to have abandoned nose | 03:59 |
| Anju | ianw : ping | 03:59 |
| clarkb | nose does a lot of funky stuff too | 04:00 |
| ianw | Anju: hi | 04:00 |
| clarkb | setup class can fail in some cases and nose reports success | 04:00 |
| Anju | ianw : i want to add a test for resscue a vm in paused state | 04:01 |
| Anju | ianw :https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_server_rescue.py#L71 | 04:01 |
| Anju | ianw :want to ask that i should use the server_is for pausing a vm ? | 04:02 |
| *** afazekas has joined #openstack-qa | 04:03 | |
| afazekas | Did you tried it with 'tox -etestr-full' ? | 04:03 |
| * afazekas connection issue | 04:03 | |
| *** prekarat has joined #openstack-qa | 04:05 | |
| *** sthaha has joined #openstack-qa | 04:05 | |
| ianw | afazekas: clarkb helped me figure out that python2.6 unittest doesn't call setUpClass so it doesn't work, and nose has work-arounds | 04:06 |
| ianw | Anju: i'm not sure i have any special knowledge about this. do you mean server_id? | 04:07 |
| *** guest__ has joined #openstack-qa | 04:07 | |
| *** prekarat has quit IRC | 04:09 | |
| ianw | afazekas: also, no i was running in a venv created by run_tests.sh | 04:13 |
| Anju | afazekas:ping | 04:19 |
| *** krtaylor has joined #openstack-qa | 04:20 | |
| *** prekarat has joined #openstack-qa | 04:23 | |
| Anju | afazekas:i wrote a test for rescuing a paused vm | 04:26 |
| Anju | it is running | 04:26 |
| Anju | but when i am adding other tests "test_rescue_unrescue_instance" in the same file | 04:26 |
| Anju | then this tests is fail | 04:27 |
| afazekas | Anju: maybe you forget to clean up something , or creating something | 04:28 |
| Anju | what should i clean here? | 04:29 |
| Anju | Afazekas: i just use this method pause_server(self.server_id) | 04:31 |
| Anju | and after pausing this i am throwing an exception for rescue the vm | 04:31 |
| Anju | afazekas:after a pause call ther sholud be a clenup for unpausing? | 04:38 |
| afazekas | Without seeing the diff and the exception, I can't say anything. You can pestebin it to somewhere.. | 04:39 |
| *** SergeyLukjanov has joined #openstack-qa | 04:42 | |
| Anju | afazekas: http://pastebin.com/BgGdD2cQ | 04:43 |
| afazekas | Anju: what is the exception ? | 04:51 |
| Anju | dupliacte | 04:51 |
| afazekas | but in your test case it is the expected one | 04:52 |
| Anju | means? | 04:52 |
| afazekas | I see, you pauses a shared server | 04:53 |
| Anju | hmm ,,bcz of this ither server test are not working | 04:53 |
| afazekas | you should add an self.addCleanup for unpause before the pause call | 04:54 |
| Anju | the i can use the server_id? in that | 04:55 |
| afazekas | yes | 04:56 |
| Anju | afazekas: if i ma not pausing a server that means it is in pause state | 04:57 |
| Anju | then why nedd to unpause and then pause | 04:57 |
| afazekas | you pause the server in this test case, and it remains paused for the another test cases | 04:58 |
| afazekas | the addCleanUp ensures it will be unaused | 04:59 |
| afazekas | BTW, why not possible to rescue a paused server ? | 04:59 |
| Anju | it is giving an error | 05:02 |
| *** mberwang_ has quit IRC | 05:03 | |
| Anju | afazekas:i told you before | 05:16 |
| Anju | that resvuing a vm in paused state is giving an error | 05:16 |
| Anju | "rescuing" | 05:16 |
| Anju | afazekas:http://pastebin.com/QSVjAD1C | 05:16 |
| *** saurabh has joined #openstack-qa | 05:23 | |
| *** afazekas is now known as afazekas__ | 05:25 | |
| openstackgerrit | Edgar Magana proposed a change to openstack-dev/devstack: Set horizon configuration for security groups https://review.openstack.org/39181 | 05:29 |
| *** afazekas__ has quit IRC | 05:30 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add image tests v2 https://review.openstack.org/39212 | 05:31 |
| *** prekarat1 has joined #openstack-qa | 05:34 | |
| *** prekarat has quit IRC | 05:36 | |
| *** Anju has quit IRC | 05:36 | |
| *** afazekas has joined #openstack-qa | 05:49 | |
| *** jhenner has joined #openstack-qa | 06:10 | |
| *** SergeyLukjanov has quit IRC | 06:10 | |
| *** prekarat1 has quit IRC | 06:14 | |
| *** prekarat has joined #openstack-qa | 06:24 | |
| kashyap | afazekas, Any hints here - http://paste.openstack.org/show/42566/ | 06:56 |
| afazekas | Try to remove the Ipv6 localhost from the /etc/hosts file | 06:58 |
| *** jhenner has quit IRC | 07:00 | |
| afazekas | check the qpid security settings, you may have secure defaults and the other parts are not prepared/configured for secure communication | 07:01 |
| afazekas | Check the iptables rules, AMQP is disabled by default . | 07:01 |
| afazekas | kashyap: BTW as I remember the qpid is ipv6 ready, just some clients are confused by ipv6, can you check is it bug, or just configuration issue ? | 07:05 |
| *** SergeyLukjanov has joined #openstack-qa | 07:10 | |
| afazekas | sdague: will it be fixed by the nova V3 API ? https://bugs.launchpad.net/nova/+bug/1080278 | 07:11 |
| afazekas | psedlak: ^^ | 07:11 |
| kashyap | afazekas, Sorry, was on phone | 07:20 |
| kashyap | afazekas, I added this too $ lokkit -p 5672:tcp | 07:21 |
| kashyap | afazekas, This is with Grizzly bits | 07:21 |
| afazekas | Is it solved your issue ? | 07:22 |
| kashyap | afazekas, No, I just removed the ::1 IPv6 entry entry | 07:23 |
| kashyap | andreaf, what else qpid settings? I just have auth=no in it | 07:23 |
| kashyap | afazekas, It maybe iptables (my weakness area) | 07:23 |
| afazekas | try to restart everything after removing the ipv6 entry | 07:24 |
| kashyap | andreaf, what should be the iptables rules | 07:24 |
| kashyap | (whoops, meant afazekas ^) | 07:24 |
| * kashyap first goes for lunch | 07:25 | |
| afazekas | kashyap: As I remember the command you showed is working | 07:26 |
| *** Anju has joined #openstack-qa | 07:26 | |
| Anju | afazekas:ping | 07:29 |
| Anju | are you not aggree | 07:29 |
| Anju | with this neagtive test (raescuingb a vm in paused state) | 07:29 |
| Anju | (rescuing a vm in paused state) | 07:29 |
| afazekas | Anju: pong | 07:34 |
| afazekas | git diff ? | 07:34 |
| afazekas | git diff master ? | 07:35 |
| Anju | afazekas: | 07:39 |
| Anju | not understanding | 07:39 |
| Anju | afazekas: it is running now you can see it here | 07:39 |
| Anju | http://pastebin.com/STEJELRb | 07:40 |
| Anju | if agree with this test | 07:40 |
| afazekas | LGTM | 07:40 |
| Anju | afazekas:is this for me? | 07:41 |
| afazekas | Anju: looks good to me | 07:41 |
| Anju | ohh thanks :afazekas:) | 07:41 |
| afazekas | But usually is better if you show it as patch, for example as a 'git diff master' command output | 07:42 |
| Anju | ok next time i will try this | 07:43 |
| Anju | want to ask many things about xml/json | 07:43 |
| Anju | this is crawling in my mind day and night | 07:43 |
| *** mkollaro has joined #openstack-qa | 07:43 | |
| *** jhenner has joined #openstack-qa | 07:52 | |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Negative test added for rescuing a paused vm https://review.openstack.org/39230 | 07:58 |
| *** psedlak has joined #openstack-qa | 08:01 | |
| *** SergeyLukjanov has quit IRC | 08:02 | |
| *** SergeyLukjanov has joined #openstack-qa | 08:05 | |
| *** jgallard has joined #openstack-qa | 08:40 | |
| *** jhenner has quit IRC | 08:42 | |
| *** jhenner has joined #openstack-qa | 08:42 | |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Negative test added for rescuing a paused vm https://review.openstack.org/39230 | 08:44 |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Some bash versions don't support negative lengths https://review.openstack.org/39236 | 08:46 |
| openstackgerrit | Nejc Saje proposed a change to openstack/tempest: Adds client for Ceilometer alarms https://review.openstack.org/39237 | 08:50 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add image tests v2 https://review.openstack.org/39212 | 09:14 |
| *** Anju has left #openstack-qa | 09:15 | |
| *** jgallard has quit IRC | 09:37 | |
| *** jgallard has joined #openstack-qa | 09:37 | |
| *** Anju has joined #openstack-qa | 09:42 | |
| Anju | afazekas:ping | 09:42 |
| *** mkollaro has quit IRC | 10:15 | |
| afazekas | Anju: pong | 10:18 |
| afazekas | mtreinish: looks like the oslo.logging with little tempset/config.py modification can work, but It might be incompatible with the https://review.openstack.org/#/c/38980/ , but it is not +2 blocker now | 10:20 |
| Anju | afazekas: https://review.openstack.org/#/c/39230/ | 10:28 |
| Anju | please have a look | 10:28 |
| Anju | afazekas: i wnat to ask many question in xml/json | 10:28 |
| *** sthaha has quit IRC | 10:31 | |
| *** prekarat1 has joined #openstack-qa | 10:39 | |
| *** prekarat has quit IRC | 10:39 | |
| *** prekarat1 has quit IRC | 10:46 | |
| *** prekarat has joined #openstack-qa | 10:49 | |
| *** prekarat has joined #openstack-qa | 10:49 | |
| Anju | how to wnable swift in devstack? | 10:51 |
| afazekas | Anju: http://logs.openstack.org/80/38980/1/check/gate-tempest-devstack-vm-full/36908/logs/localrc.txt.gz | 10:53 |
| afazekas | "s-proxy,s-account,s-container,s-object" | 10:53 |
| Anju | what should be the value of SWIFT_HASH | 10:54 |
| afazekas | The same as on the gate :) | 10:54 |
| afazekas | you can pick any random string | 10:54 |
| Anju | ok afazekas thanks | 10:55 |
| Anju | but still i am puzzled in xml/json | 10:55 |
| *** saurabh has quit IRC | 10:57 | |
| *** prekarat has quit IRC | 10:59 | |
| afazekas | $(openssl rand -hex 10) for random vales can work | 10:59 |
| afazekas | Anju: which request/repose you would like to parse/craft in xml ? | 11:00 |
| Anju | afazekas:https://github.com/openstack/tempest/blob/master/tempest/services/volume/json/volumes_client.py | 11:02 |
| *** jgallard has quit IRC | 11:02 | |
| Anju | afzekas :no not thsi one ,wait a second | 11:03 |
| Anju | afazekas:https://github.com/openstack/tempest/blob/master/tempest/services/volume/json/volumes_client.py#L88 | 11:04 |
| Anju | function like upload ,attach ,dettach is missing in https://github.com/openstack/tempest/blob/master/tempest/services/volume/xml/volumes_client.py | 11:05 |
| *** sthaha has joined #openstack-qa | 11:08 | |
| *** sthaha has joined #openstack-qa | 11:08 | |
| *** prekarat has joined #openstack-qa | 11:08 | |
| afazekas | Anju: first you can report a bug it is undocumented: http://api.openstack.org/api-ref.html#volumes-api | 11:10 |
| afazekas | "attach ,dettach" might be just for internal usage but the upload should be a public one, this feature appeared in grizzly AFAIK | 11:11 |
| Anju | afazekas: ok i will | 11:12 |
| Anju | afazekas: a cli is available for creating arole keystone role-create | 11:14 |
| Anju | but i think this also not documented | 11:14 |
| Anju | afazekas: any idea so i can start work on this | 11:15 |
| Anju | i working in this from many days | 11:15 |
| Anju | but i am not getting any soluntion | 11:15 |
| afazekas | IMHO you can report all not documented api call | 11:16 |
| Anju | afazekas :is this a bug | 11:16 |
| Anju | ???? | 11:16 |
| afazekas | in is bug in the documentation | 11:17 |
| Anju | but i will report it | 11:17 |
| Anju | afazekas :but to start in this | 11:17 |
| *** dosaboy has joined #openstack-qa | 11:17 | |
| afazekas | I will try to figure out which repo/and project is responsible for the api.openstack.org | 11:17 |
| afazekas | psedlak: do you remember to how this pages are generated ? AS I remember you reported a similar bug .. | 11:20 |
| Anju | afazekas : suggest me something in this https://review.openstack.org/#/c/39230/ :) | 11:22 |
| *** prekarat has quit IRC | 11:22 | |
| *** Dmitry-I has quit IRC | 11:31 | |
| afazekas | can you move the self.addCleanup(self._unpause, self.server_id) before the ' self.assertEqual(202, resp.status)' ? | 11:32 |
| Anju | hmm | 11:33 |
| Anju | is this not correct? | 11:33 |
| afazekas | Se the addCleanUp method should be as close as possible to the operation, what it actually wants to undo | 11:34 |
| Anju | ok i doing | 11:35 |
| psedlak | afazekas, Anju: https://github.com/openstack/identity-api ... https://github.com/openstack/volume-api ... and others? | 11:35 |
| afazekas | psedlak: So that page is assembled from multiple repo , right ? | 11:35 |
| psedlak | afazekas: not sure now, it's some time ago already, but i think yes ... what i remember that it seemed messy for me how to change something there | 11:36 |
| psedlak | https://wiki.openstack.org/wiki/Documentation/HowTo | 11:37 |
| psedlak | just in the first https://wiki.openstack.org/wiki/Documentation/HowTo#Tools_and_Content_Overview ... DocBook and WADL source files for http://api.openstack.org, look for the openstack/api-site ... API spec documentation that builds to http://docs.openstack.org/api is housed in image-api, identity-api | 11:38 |
| afazekas | Are they belongs to the https://bugs.launchpad.net/openstack-manuals project or to the service project, in case of bug report ? | 11:38 |
| psedlak | afazekas, Anju: ... "Doc bugs are logged and triaged at http://bugs.launchpad.net/openstack-manuals." but there is also https://bugs.launchpad.net/openstack-api-site ... so clearly i'm not sure about this | 11:40 |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Negative test added for rescuing a paused vm https://review.openstack.org/39230 | 11:40 |
| psedlak | Anju: you better ask annegentle for where to put api/docs bugs etc | 11:41 |
| *** prekarat has joined #openstack-qa | 11:42 | |
| *** prekarat has quit IRC | 12:25 | |
| *** mkollaro has joined #openstack-qa | 12:33 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Switching to oslo importutils in stress tests https://review.openstack.org/39279 | 12:40 |
| *** dkranz has joined #openstack-qa | 12:42 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Use state transition checker wait function in the ec2 image tests https://review.openstack.org/38805 | 12:46 |
| *** meteorfox has quit IRC | 12:53 | |
| *** prekarat has joined #openstack-qa | 12:55 | |
| dkranz | afazekas: Are you ready with https://review.openstack.org/#/c/38980/1 or are you going to change it more? | 12:55 |
| afazekas | dkranz: I you have specific request I can add it now, but is working now | 12:59 |
| dkranz | afazekas: No, just checking. | 13:00 |
| afazekas | I am thinking about just using test_method config option, with full class path | 13:00 |
| *** mkollaro has quit IRC | 13:01 | |
| *** anteaya has joined #openstack-qa | 13:02 | |
| afazekas | dkranz: The other option is defining multiple test methods from the same class, but I think you would consider it miss-using of the stress engine :) | 13:05 |
| afazekas | BTW: I looking for the api doc gen macro, after that I'll check the setUpClass call chain correctness | 13:06 |
| openstackgerrit | A change was merged to openstack/tempest: Add wait_for_volume_status in upload_to_image test https://review.openstack.org/39178 | 13:12 |
| sdague | afazekas: re nova v3 API, I don't know. I'd ask on the list | 13:29 |
| afazekas | sdague: thank you | 13:33 |
| sdague | afazekas: your thoughts on this? - https://review.openstack.org/#/c/37838/ as you are the boto expert | 13:35 |
| *** krtaylor has quit IRC | 13:36 | |
| *** meteorfox has joined #openstack-qa | 13:40 | |
| afazekas | LGTM, the helper method could "_run_instance" could be moved outside and used by all test other test cases, but it would be a re-factor change type instead of add test case change type. | 13:42 |
| *** mkollaro has joined #openstack-qa | 13:57 | |
| openstackgerrit | Kyle Mestery proposed a change to openstack-dev/devstack: Update the ML2 configuration in devstack https://review.openstack.org/37963 | 14:00 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Generate temepst API doc from source https://review.openstack.org/39291 | 14:09 |
| afazekas | sdague: ^^ | 14:09 |
| afazekas | There is no macro for document all, I had an rst file per module ... | 14:18 |
| *** krtaylor has joined #openstack-qa | 14:18 | |
| *** dkranz has quit IRC | 14:19 | |
| *** mlavalle has joined #openstack-qa | 14:30 | |
| *** burt has joined #openstack-qa | 14:31 | |
| *** dkranz has joined #openstack-qa | 14:34 | |
| openstackgerrit | Eoghan Glynn proposed a change to openstack-dev/devstack: Launch ceilometer-alarm-* services. https://review.openstack.org/39300 | 14:39 |
| *** jgallard has joined #openstack-qa | 14:42 | |
| openstackgerrit | A change was merged to openstack/tempest: Add boto tests for idempotent RunInstances calls https://review.openstack.org/37838 | 14:47 |
| *** jgallard has quit IRC | 14:49 | |
| *** jgallard has joined #openstack-qa | 14:49 | |
| mtreinish | afazekas: on https://review.openstack.org/#/c/38879 in config.py I have to move the 3 logging lines right before cfg.CONF or right after? | 14:50 |
| mlavalle | sdague: I read your comment in the patch set I submitted for review in nova. Thanks. I have never written a unit test for nova. Is this guide valid / up to date? http://docs.openstack.org/developer/nova/devref/unit_tests.html | 14:52 |
| afazekas | mtreinish: yes | 14:52 |
| afazekas | mtreinish: would be nice If you would add the log option the tempest.conf.sample as well | 14:53 |
| *** dkranz has quit IRC | 14:55 | |
| *** prekarat has left #openstack-qa | 14:59 | |
| mtreinish | afazekas: which one before or after? | 14:59 |
| afazekas | 606,607,621 should be deleted | 15:01 |
| *** dkranz has joined #openstack-qa | 15:01 | |
| afazekas | The lines in 3th comment should be after the 629 line | 15:01 |
| afazekas | cfg.CONF will setup the cli config parameters, after that the logging.setup can use it | 15:02 |
| *** malini has joined #openstack-qa | 15:02 | |
| *** malini has quit IRC | 15:03 | |
| *** krtaylor has quit IRC | 15:05 | |
| mtreinish | afazekas: ok that's what I thought, but the comment was before | 15:09 |
| burt | sdague: ping | 15:10 |
| *** jgallard has quit IRC | 15:11 | |
| *** jgallard has joined #openstack-qa | 15:11 | |
| afazekas | mtreinish: Yes, If I click an exact line the comment will be above that, next I try to pick a line below. | 15:13 |
| *** sthaha has quit IRC | 15:16 | |
| *** jgallard has quit IRC | 15:36 | |
| *** jgallard has joined #openstack-qa | 15:37 | |
| mtreinish | afazekas: do you think that we should rename logging.conf.sample to something else especially if I'm going to put it in the config file? | 15:42 |
| mtreinish | maybe just logging.conf | 15:42 |
| afazekas | mtreinish: AFAIK to openstack custom us to add sample conf | 15:43 |
| afazekas | If we ever want to use it on the gate we should rename it | 15:44 |
| afazekas | psedlak: http://mainlydata.kubadev.com/python/sphinx-how-to-make-autodoc-really-automatic/ | 15:45 |
| *** dkranz has quit IRC | 15:47 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add locking to test_aggregates https://review.openstack.org/38857 | 15:56 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Switch to using Oslo logging https://review.openstack.org/38879 | 15:56 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add tenant isolation for scenario tests https://review.openstack.org/39099 | 15:56 |
| *** dkranz has joined #openstack-qa | 16:00 | |
| *** jgallard has quit IRC | 16:08 | |
| *** lexinator has joined #openstack-qa | 16:10 | |
| *** marun has joined #openstack-qa | 16:12 | |
| *** SergeyLukjanov has quit IRC | 16:23 | |
| openstackgerrit | Christian Berendt proposed a change to openstack-dev/devstack: renamed deprecated glanceclient parameter https://review.openstack.org/39323 | 16:28 |
| *** krtaylor has joined #openstack-qa | 16:30 | |
| *** sourabh has joined #openstack-qa | 16:42 | |
| *** ankit_ has joined #openstack-qa | 16:45 | |
| *** sourabh has left #openstack-qa | 16:47 | |
| *** Anju_ has joined #openstack-qa | 16:49 | |
| *** sourabh has joined #openstack-qa | 16:50 | |
| *** sourabh has left #openstack-qa | 16:51 | |
| *** sourabh has joined #openstack-qa | 16:51 | |
| *** SergeyLukjanov has joined #openstack-qa | 16:51 | |
| *** sourabh has quit IRC | 16:52 | |
| Anju_ | afazekas: where should i have to report that bug | 16:53 |
| Anju_ | ? | 16:53 |
| *** sdague has quit IRC | 16:56 | |
| *** lexinator has quit IRC | 16:58 | |
| afazekas | annegentle, psedlak: ^^^ | 16:58 |
| psedlak | Anju_: check that https://wiki.openstack.org/wiki/Documentation/HowTo | 16:59 |
| Anju_ | afaezkas :https://review.openstack.org/#/c/39230/ i commit code here.please check | 16:59 |
| psedlak | Anju_: for api probably in https://bugs.launchpad.net/openstack-api-site otherwise maybe in https://bugs.launchpad.net/openstack-manuals | 17:00 |
| psedlak | Anju_: check those two if there is similar bug report to what you want to report | 17:00 |
| Anju_ | psedlak:checking | 17:00 |
| *** lexinator has joined #openstack-qa | 17:02 | |
| Anju_ | psedlak:no it is not reported | 17:02 |
| *** jhenner has quit IRC | 17:04 | |
| Anju_ | psedlak:i just want to report that some api are not documented | 17:04 |
| kashyap | afazekas, Heya, do you have any hints for this stupid error - http://paste.openstack.org/show/42649/ . Why does it deny access | 17:05 |
| kashyap | Or psedlak ^ :-) | 17:06 |
| psedlak | kashyap: as it was already installed, is it really fresh install? who asks for that password ... ? | 17:08 |
| kashyap | psedlak, It's already installed, and all other services were init'ed on it | 17:08 |
| kashyap | psedlak, Quantum server seems to ask for it as you see there | 17:09 |
| psedlak | Anju_: yeah, it seems that https://launchpad.net/openstack-api-site is the right one for you | 17:10 |
| afazekas | kashyap: check that script is it really knows you have mariadb instead of mysql | 17:10 |
| *** psedlak has quit IRC | 17:10 | |
| afazekas | kashyap: which branch do you using ? | 17:11 |
| kashyap | afazekas, Heya, Grizzly this is. | 17:11 |
| kashyap | afazekas, Havana doesn't even have Neutron pkgs yet. | 17:11 |
| afazekas | I mean F19 stable or testing ? | 17:11 |
| afazekas | koji has | 17:11 |
| afazekas | f20 | 17:12 |
| kashyap | afazekas, F19 | 17:12 |
| afazekas | kashyap: as I remember that script has password argument | 17:13 |
| *** sdague has joined #openstack-qa | 17:13 | |
| * kashyap reads man page | 17:13 | |
| *** sourabh has joined #openstack-qa | 17:15 | |
| afazekas | you can read the script | 17:15 |
| burt | any advice on how to test nova instance shutdown/termination in tempest? It relies on sync_interval_power_state, which is periodic but only called every 600s by default. | 17:16 |
| afazekas | According to log it is confused .. | 17:16 |
| Anju_ | kashyap: yoc can try this just tellig you according to database basics http://www.howtoforge.com/setting-changing-resetting-mysql-root-passwords | 17:16 |
| *** sourabh has quit IRC | 17:18 | |
| afazekas | burt: in a tempest test ? | 17:20 |
| kashyap | Anju_, Yes, I could muck around with this. But was figuring why this piece of $@#^ is behaving this way. afazekas - Yep, reading the script. | 17:20 |
| afazekas | burt: Can it be configured to several seconds ? does it have significant performance impact ? | 17:20 |
| burt | afazekas: yes. I want to test this (once it's merged) https://review.openstack.org/#/c/38081/ | 17:21 |
| afazekas | kashyap: as I remember the scripts main task is creating a symbolic link.. :) | 17:21 |
| burt | afazekas: yes, it's a nova config parameter configurable via sync_power_state_interval in the nova.conf. I think for QA it shouldn't have a big performance impact, but I don't know if it should be the default for devstack | 17:21 |
| afazekas | sdague: ^^^ | 17:22 |
| sdague | burt: pong | 17:24 |
| burt | sdague: thanks for the reviews. I was wondering about what I was just discussing with afazekas | 17:25 |
| sdague | mtreinish: helps to test the logging code :) | 17:25 |
| mtreinish | sdague: yeah I know | 17:26 |
| mtreinish | its all that stuff merging in the background | 17:26 |
| mtreinish | I tested it before I rebased | 17:26 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add locking to test_aggregates https://review.openstack.org/38857 | 17:27 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Switch to using Oslo logging https://review.openstack.org/38879 | 17:27 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add tenant isolation for scenario tests https://review.openstack.org/39099 | 17:27 |
| Anju_ | adazekas :i reported that in a very few words https://bugs.launchpad.net/openstack-api-site/+bug/1206608 | 17:31 |
| Anju_ | afazekas:i reported that in a very few words https://bugs.launchpad.net/openstack-api-site/+bug/1206608 | 17:31 |
| *** sdague has quit IRC | 17:32 | |
| Anju_ | afazekas :stil waiting for your suggestions https://review.openstack.org/#/c/39230/ | 17:33 |
| *** sdague has joined #openstack-qa | 17:36 | |
| burt | sdague: this is the tempest test I'm thinking of: https://review.openstack.org/#/c/39340/ | 17:37 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Fix neutron cli tests skip for testr https://review.openstack.org/39342 | 17:40 |
| adalbas | mtreinish, ping | 17:41 |
| *** sdague has quit IRC | 17:41 | |
| mtreinish | adalbas: pong | 17:41 |
| *** sdague has joined #openstack-qa | 17:42 | |
| adalbas | mtreinish, from this list of failure, https://etherpad.openstack.org/debugging-testr-tempest , any suggestions for one i could look at (to avoid going over some you are already investigating) | 17:42 |
| adalbas | ? | 17:42 |
| mtreinish | adalbas: one sec let me take a look at the list. But, I think that I've marked the ones that I have started working on. | 17:43 |
| burt | sdague: think you got bounced before I responded. I was wondering about how to handle https://review.openstack.org/#/c/39340/, but the sync_power_state only runs every 600s by default. | 17:43 |
| mtreinish | adalbas: how about failure #2 that happened a few times earlier (although I don't think I've seen it recently) | 17:44 |
| *** Anju_ has quit IRC | 17:44 | |
| openstackgerrit | Eoghan Glynn proposed a change to openstack-dev/devstack: Launch ceilometer-alarm-* services. https://review.openstack.org/39300 | 17:44 |
| mtreinish | adalbas: actually that may have already been fixed | 17:44 |
| *** ankit_ has quit IRC | 17:44 | |
| adalbas | mtreinish, i started looking #6, what do you think? | 17:45 |
| mtreinish | adalbas: no nm I was thinking of https://review.openstack.org/#/c/39178/ which was a different test | 17:45 |
| mtreinish | adalbas: yeah that should be fine | 17:46 |
| mtreinish | adalbas: although I haven't seen a bunch of these failures in a while. Partially because there have been a couple of changes which broke the testr runs for a while. (I've got fixes pending for all of them though) | 17:48 |
| adalbas | mtreinish, in your devel env, is it free of failures (except by th the ones pointed out) | 17:48 |
| adalbas | ? | 17:48 |
| adalbas | im getting many failures running it now. | 17:48 |
| mtreinish | adalbas: no, not really I have failures that pop up with things related to cinder and other broken state from testing. So I can't really differentiate that from testr issues. | 17:49 |
| mtreinish | I need to clean things up | 17:49 |
| mtreinish | adalbas: what are you hitting? | 17:49 |
| adalbas | mtreinish, let me investigate if all is running fine in my env first. Image failures and other things. | 17:50 |
| mtreinish | if it's on the glance v2 with schemas there is a bug related to dependencies there | 17:50 |
| mtreinish | because you could have an old version lying around which won't work | 17:50 |
| adalbas | mtreinish, ok. i ll think i need to run devstack from the beginning again, to ensure i have the latests patches. | 17:51 |
| mtreinish | adalbas: I'm not sure that would fix it actually | 17:52 |
| mtreinish | sdague: ^^^ do you know since you had to revert me pinning the version? | 17:52 |
| adalbas | mtreinish, what is this old version? | 17:52 |
| sdague | burt: I thought you could forcably trigger it? | 17:53 |
| sdague | or is this really only on the periodic | 17:53 |
| burt | If you tell me how I could force it, that would be cool. | 17:53 |
| sdague | adalbas: right, the requirements issue isn't resolvable for a while | 17:53 |
| burt | but I didn't see anything obvious | 17:53 |
| mtreinish | adalbas: I believe you need jsonschema>=1.3.0 | 17:54 |
| sdague | burt: https://review.openstack.org/#/c/39340/ is not found for me | 17:55 |
| adalbas | mtreinish, alright, you are talking about the requirement that broke grenade, right? | 17:55 |
| burt | eh, try again. I published it as a draft. | 17:55 |
| mtreinish | adalbas: yeah its the same one. The patch I pushed out to set the right version broke grenade. But if you have the old one installed the tests will fail | 17:56 |
| adalbas | mtreinish, ok! | 17:56 |
| mtreinish | adalbas: you can just manually install it with 'pip install -U jsonschema==1.3.0' and then restack | 17:56 |
| mtreinish | I think | 17:56 |
| adalbas | i ll try that | 17:56 |
| *** prekarat has joined #openstack-qa | 17:57 | |
| openstackgerrit | minsel proposed a change to openstack/tempest: Add testing of Neutron per tenant quotas API https://review.openstack.org/35724 | 17:57 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: setUpClass/tearDownClass full chain https://review.openstack.org/39346 | 17:59 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: setUpClass/tearDownClass full chain https://review.openstack.org/39346 | 18:07 |
| *** marun has quit IRC | 18:12 | |
| kashyap | afazekas, I just replaced the RPM name in the quantum script | 18:22 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Switch to using Oslo logging https://review.openstack.org/38879 | 18:22 |
| afazekas | kashyap: I am not sure why you need that script, but If it not ok it should be fixed anyway :) | 18:22 |
| afazekas | bye | 18:22 |
| *** afazekas has quit IRC | 18:23 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add locking to test_aggregates https://review.openstack.org/38857 | 18:23 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add tenant isolation for scenario tests https://review.openstack.org/39099 | 18:23 |
| *** prekarat has quit IRC | 18:28 | |
| *** marun has joined #openstack-qa | 18:29 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Fix fail logic for server of another tenant test https://review.openstack.org/39357 | 18:35 |
| sdague | burt: that seems like what I was thinking, you don | 18:39 |
| sdague | 't think that will work without periodic? | 18:40 |
| mtreinish | sdague, dkranz, jaypipes: can you take a look at: https://review.openstack.org/39342 | 18:40 |
| mtreinish | it's breaking the testr runs right now | 18:40 |
| mtreinish | well, just marking them as failed they still run | 18:41 |
| sdague | lgtm | 18:41 |
| *** marun has quit IRC | 18:42 | |
| *** marun has joined #openstack-qa | 18:43 | |
| burt | sdague: no, I don't think so. AFAICT the only way nova knows that the VM has shut itself down is via the periodic call. | 18:45 |
| sdague | I thought we were reporting that back more timely now | 18:48 |
| sdague | you could be right though | 18:48 |
| sdague | hmmmm | 18:48 |
| openstackgerrit | Mate Lakat proposed a change to openstack-dev/devstack: xenapi: Use a jeos vm as a template https://review.openstack.org/39360 | 18:50 |
| openstackgerrit | Mate Lakat proposed a change to openstack-dev/devstack: xenapi: Use a jeos vm as a template https://review.openstack.org/39360 | 19:04 |
| dkranz | mtreinish: Sorry, at a meeting. Done. | 19:09 |
| mtreinish | dkranz, sdague: cool thanks | 19:11 |
| sdague | oh, hey, dkranz voted the same second I did on the log | 19:50 |
| dkranz | sdague: :) | 19:50 |
| mtreinish | sdague: sweet | 19:50 |
| mtreinish | sdague: can you look at https://review.openstack.org/#/c/38526/ I'd like to start getting some of these race fixes through | 19:52 |
| sdague | mtreinish: yep, good with that | 19:53 |
| sdague | just put it to merge queue | 19:53 |
| mtreinish | sdague: great | 19:53 |
| sdague | dkranz: https://review.openstack.org/#/c/38828/ - can you look at that? | 19:54 |
| dkranz | sdague: Just a sec... | 19:55 |
| *** afazekas has joined #openstack-qa | 19:55 | |
| dkranz | sdague: I don't see anything wrong with it but I am no python+fixtures expert. | 19:57 |
| dkranz | sdague: If you are confident though I will +1 it. | 19:58 |
| sdague | yeh, it comes out of nova equiv of this | 19:58 |
| sdague | so it's been working over there | 19:58 |
| dkranz | sdague: ok. "in the land of the blind, the one-eyed man is king" :) | 19:59 |
| dkranz | sdague: Done. | 20:00 |
| sdague | heh | 20:08 |
| dkranz | mtreinish: How close are we to being able to take nose out of the gate, using serial testr instead? | 20:10 |
| mtreinish | dkranz: hmm that's an interesting thought, I didn't even think about doing that. Do you have a clean devstack environment to test it? | 20:11 |
| dkranz | mtreinish: I am having trouble getting "ignore slow" to work in nose the way we are using tags. | 20:11 |
| dkranz | mtreinish: I hope too tomorrow. I have not had a real environment which is whiy I havne't made much progress. | 20:11 |
| mtreinish | dkranz: well if it works with testr, I don't see a reason why serially testr would introduce any problems | 20:11 |
| mtreinish | (the tags that is) | 20:12 |
| dkranz | mtreinish: WHere is the best doc about how to use testr? I've been avoiding it :) | 20:12 |
| mtreinish | 'testr help' is pretty useful, there is also: https://testrepository.readthedocs.org/en/latest/MANUAL.html and https://testrepository.readthedocs.org/en/latest/ | 20:13 |
| dkranz | mtreinish: OK, thanks. I'll check it out. | 20:13 |
| mtreinish | if you just run 'testr run' from your tempest root dir that will run everything serially | 20:13 |
| mtreinish | dkranz: if you want I can push a patch out that switches everything over to testr serially | 20:14 |
| mtreinish | just so we can get a feel for how it would work in the gate | 20:14 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: setUpClass/tearDownClass full chain https://review.openstack.org/39346 | 20:16 |
| sdague | mtreinish: add a new tox target for it | 20:16 |
| sdague | that would be better | 20:16 |
| mtreinish | sdague: but then it wouldn't run in the gate? | 20:17 |
| mtreinish | (why the question mark...) | 20:17 |
| *** lexinator has quit IRC | 20:17 | |
| sdague | mtreinish: but it would make it easy for dkranz to kick the tires on | 20:18 |
| dkranz | mtreinish: I think I should play with it first. | 20:19 |
| dkranz | mtreinish: I presume testr has an option like nose --collect-only where you can play with what will run quickly. At least I hope it does. | 20:20 |
| dkranz | mtreinish: Then use just testr in the gate, then skip slow tests, then add heat job. | 20:20 |
| *** torandu has quit IRC | 20:23 | |
| sdague | dkranz: what does --collect-only do? | 20:25 |
| *** torandu has joined #openstack-qa | 20:25 | |
| dkranz | mtreinish: Actually it would be good if you put a tox job in for this. Then it coujld be tested | 20:25 |
| *** torandu has quit IRC | 20:25 | |
| dkranz | sdague: It prints out the names of the tests that will be run as usual, but doesn't actually run them | 20:25 |
| dkranz | sdague: I think it is an undocumented feature I found on stackoverflow | 20:26 |
| *** torandu has joined #openstack-qa | 20:26 | |
| mtreinish | dkranz: yeah it does 'testr list-tests' | 20:26 |
| dkranz | mtreinish: Cool. That's what I was looking for. | 20:26 |
| mtreinish | so to mimic the gate it would be 'testr list-tests tempest.api tempest.scenario tempest.thirdparty tempest.cli' | 20:26 |
| *** torandu has quit IRC | 20:27 | |
| mtreinish | dkranz: for the new tox job have it just run the same set as the gate but serially? | 20:27 |
| dkranz | mtreinish: Yes. | 20:28 |
| dkranz | mtreinish: I will then add the slow tag and make sure everything works. | 20:28 |
| *** torandu has joined #openstack-qa | 20:28 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add eventlet to requirements.txt https://review.openstack.org/39368 | 20:28 |
| mtreinish | dkranz: ok one sec | 20:29 |
| *** lexinator has joined #openstack-qa | 20:30 | |
| openstackgerrit | A change was merged to openstack/tempest: Fix neutron cli tests skip for testr https://review.openstack.org/39342 | 20:32 |
| *** annegentle is now known as anneisgentle | 20:40 | |
| afazekas | fungi: ping | 20:40 |
| fungi | afazekas: pong | 20:42 |
| dkranz | mtreinish: I see that in testr you can provide regexps that will be OR'ed but don't see how to exclude tests based on name. Is there? | 20:42 |
| afazekas | fungi: https://review.openstack.org/#/c/39291/ How could I use that script referred in the commit message without adding it to the tempest repo ? | 20:44 |
| fungi | afazekas: sphinx-autopackage-script? | 20:45 |
| afazekas | fungi: yes | 20:45 |
| fungi | afazekas: is the author willing to package it and upload to pypi? | 20:46 |
| mtreinish | dkranz: that was the negative lookahead stuff that clarkb was talking about the other day. You just make the regex the test name. | 20:47 |
| afazekas | fungi: I do not know the author will, I just found that script on the internet | 20:47 |
| dkranz | mtreinish: Can you be slightly more specific? | 20:47 |
| fungi | afazekas: we can run things that are either part of the project being tested, or which are packaged either for the target distribution (deb, rpm) or on pypi | 20:48 |
| openstackgerrit | A change was merged to openstack/tempest: Switch to using Oslo logging https://review.openstack.org/38879 | 20:48 |
| fungi | afazekas: i'd be hesitant to suggest embedding it in one of our infra repos like the utility scripts we've written, because that would basically be forking the original author's work | 20:49 |
| afazekas | fungi: can we just wget/curl that script ? it is one file. | 20:49 |
| fungi | afazekas: that makes the job flaky. what happens when there's a network blip? or when bitbucket is down for maintenance? | 20:49 |
| mtreinish | dkranz: testr doesn't differentiate tags and testname they all get combined in the testid. So if you want to filter out a test you just have to do the same thing you would do with filtering out a tag. | 20:49 |
| afazekas | fungi: can it be copied to some "internal" web server ? | 20:50 |
| mtreinish | dkranz: here is an example from nova which clarkb added to filter out the coverage unit tests: '^(?!.*test.*coverage).*$' | 20:50 |
| fungi | afazekas: we try to minimize the number of places on the internet a job needs to talk to. each one of them becomes a single point of failure | 20:50 |
| fungi | afazekas: if it's something we use, we'd have to publish it. then we'd be forking it | 20:50 |
| dkranz | mtreinish: I see. Just need to be a regex wizard. | 20:50 |
| fungi | afazekas: easiest path forward is to see if the author would mind terribly getting it onto pypi. we can even provide resources to help in that regard | 20:51 |
| mtreinish | dkranz: yeah, which I'm not. I pretty much fail whenever I try to use regex | 20:51 |
| dkranz | mtreinish: Ditto | 20:51 |
| fungi | afazekas: alternative approach, if the author has no interest in continuing to maintain that script long-term, is that we can talk about becoming the upstream maintainers of it | 20:53 |
| afazekas | fungi: I do not think we need ant maintenance or future version, it does what I need. | 20:53 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add tox job for serial testr https://review.openstack.org/39371 | 20:54 |
| fungi | clarkb: do you have any suggestions i'm overlooking for being able to run a third-party script for tempest doc generation when said script is only published unversioned on bitbucket? | 20:54 |
| mtreinish | dkranz: ^^^ there you go. Hot of the presses | 20:54 |
| mtreinish | *off | 20:55 |
| dkranz | mtreinish: Thanks. tempest.api should be tempest\.api right? | 20:56 |
| dkranz | mtreinish: And I have to turn those arguments into an alternation followed by exclusion of 'slow'. Blah. | 20:56 |
| *** comstud has joined #openstack-qa | 20:57 | |
| mtreinish | dkranz: you should be ok if you add the slow exclusion as another arg to the end of the list I think | 20:57 |
| mtreinish | dkranz: and yes you're right it should be \. but . works too :) | 20:58 |
| openstackgerrit | A change was merged to openstack/tempest: Add environmental variables to test.conf https://review.openstack.org/38828 | 20:58 |
| dkranz | mtreinish: I'll play, hopefully for not too long. | 20:58 |
| clarkb | fungi: uh, we could vendor it | 20:59 |
| clarkb | fungi: which is painful, but assuming the licenses are compatible is a thing we can do | 20:59 |
| fungi | clarkb: yeah, seems marginally impolite but it's a path forward i suppose | 21:00 |
| clarkb | mtreinish: dkranz if you describe the thing you want to exclude I can give you a regex | 21:00 |
| clarkb | s/thing/strings/ | 21:01 |
| mtreinish | afazekas: my google search for that script said it's now part of sphinx as apidoc | 21:01 |
| mtreinish | http://sphinx-doc.org/man/sphinx-apidoc.html does this do what you need? | 21:01 |
| afazekas | clarkb: https://xkcd.com/208/ :) | 21:01 |
| dkranz | mtreinish: \[\bslow\b\] | 21:01 |
| sdague | afazekas: ask dhelman what he's doing for ceilo, I thought he'd solved this one | 21:02 |
| afazekas | mtreinish: we are using relatively old sphinx | 21:02 |
| clarkb | oh yeah you want to exclude the slow tag. ^(?!.*\[\bslow\b\]).*$ should be close to what you want | 21:02 |
| comstud | sdague: ok, so ya, now that we're into tempest for the cells gate, there's a crapload of tests that I need to skip.. like aggregates, etc :) | 21:02 |
| sdague | afazekas: bump the requirement | 21:02 |
| fungi | afazekas: that xkcd was particularly apropos | 21:03 |
| sdague | comstud: so what's the eventual plan for these features? | 21:03 |
| afazekas | But there is way to use newer version in tempest and bypassing the common requirements .. | 21:03 |
| * fungi loled | 21:03 | |
| clarkb | I need the shirt from that comic | 21:03 |
| sdague | because up until this point we've really not let people turn off parts of nova testing, because it's all nova | 21:03 |
| comstud | sdague: Implement them... there's been a patch up for aggregates.. but I don't know that it's ready or the right way... | 21:03 |
| jeblair | afazekas: pbr does automatic api doc generation for sphinx | 21:03 |
| dkranz | clarkb: Thanks. | 21:03 |
| dkranz | Gotta run now. | 21:03 |
| comstud | sague: things like floating ips... well... there's tests against nova-network and there's not any point in implementing a bunch of cells work for nova-network since it'll be going away.. | 21:04 |
| afazekas | jeblair: can you show me an example usage ? | 21:04 |
| comstud | sdague even | 21:04 |
| sdague | comstud: not any time soon I don't think, as neutron isn't passing gate | 21:04 |
| comstud | sdague: If it's easier.. the gate is actually worthwhile JUST with the devstack exercises. | 21:04 |
| comstud | sdague: right | 21:04 |
| clarkb | sdague: yeah it won't be deprecated until icehouse at the earliest | 21:04 |
| sdague | so I think the past answer has been you are non-voting until you pass the tests, and so run at getting working as fast as possible | 21:05 |
| comstud | sdague: devstack exercises at least test 'build'... although it'd be nice to test basic actions with tempest.. like reboot, etc | 21:05 |
| mtreinish | comstud: if they need to be skipped there are 2 ways I'd go about it: 1. make a config option for tempest and skipIf each test (this is very time consuming but cleaner) or you could add a tox job which runs the subset you need | 21:05 |
| mtreinish | but I'll defer to sdague on whether we should or not | 21:05 |
| sdague | I think the problem is that it's a pretty extensive set of skips | 21:05 |
| comstud | sadgue: Yea, but a non-voting 'fail' on every single run is meaningless | 21:05 |
| sdague | comstud: right, which is the incentive to get it all working faster | 21:06 |
| comstud | It looks to be... although if I can skip a whole class, it's easier | 21:06 |
| comstud | sdague: Yeah, that's just not going to be possible in the relative near-term | 21:06 |
| comstud | security groups is an interesting problem | 21:06 |
| afazekas | mtreinish: cool, looks it is in my tox venv, I'll check it tomorrow | 21:07 |
| comstud | technically security groups and aggregates, etc are extensions. | 21:08 |
| mtreinish | comstud: you can skip a class. You just raise a skipException we do that a lot for config options. For example: https://review.openstack.org/#/c/39342/ | 21:08 |
| comstud | ok | 21:08 |
| jeblair | afazekas: i think you add 'autodoc_index_modules' to setup.cfg | 21:09 |
| jeblair | afazekas: but ask mordred | 21:09 |
| mordred | arro? | 21:09 |
| comstud | mtreinish: I'll go ahead and look at that and propose something similar for cells.. and then I guess we'll see how nasty it is :) | 21:09 |
| mordred | yeah. I believe that's the word | 21:09 |
| mordred | btw- | 21:09 |
| mordred | apparently there is a sphinx util to do that | 21:09 |
| mordred | so I might be ripping out the impl code and replacing it with an upstream thing | 21:09 |
| *** dkranz has quit IRC | 21:10 | |
| jeblair | mordred: http://sphinx-doc.org/man/sphinx-apidoc.html that? | 21:10 |
| jeblair | mordred: mtreinish pointed that out earlier | 21:10 |
| mordred | yeah | 21:10 |
| jeblair | mordred: what do you think the right path forward for afazekas is wrt to adding that to tempest? | 21:10 |
| jeblair | mordred: see this review https://review.openstack.org/#/c/39291/ | 21:11 |
| mordred | jeblair: to use the pbr interface | 21:11 |
| * afazekas regen my venv I had newer sphinx from the past .. | 21:11 | |
| mordred | oh yeah - not that | 21:11 |
| mordred | yup | 21:12 |
| mordred | autodoc_index_modules=true | 21:12 |
| mordred | in a [pbr] section | 21:12 |
| mordred | in setup.cfg | 21:12 |
| mordred | and that should write a file doc/source/api/autoindex.rst | 21:13 |
| mordred | that you can reference in your docs | 21:13 |
| afazekas | mordred: thx, I'll try that too | 21:15 |
| *** mkollaro has quit IRC | 21:24 | |
| *** meteorfox has quit IRC | 21:36 | |
| *** SergeyLukjanov has quit IRC | 21:37 | |
| afazekas | A simple long module list seams too flat to me | 21:44 |
| *** burt has quit IRC | 22:02 | |
| *** meteorfox has joined #openstack-qa | 22:07 | |
| *** meteorfox has quit IRC | 22:12 | |
| sdague | comstud: so I was thinking about the cells skips | 22:17 |
| sdague | on the drive home | 22:17 |
| sdague | I think what we need to do is the following | 22:17 |
| sdague | 1) add a available_service.cells to the config | 22:17 |
| sdague | 2) all skips in tempest need specific high priority nova bugs | 22:18 |
| sdague | we have a way to track skip bugs to know that they were resolved, which we run periodically | 22:18 |
| sdague | to unskip to make progress | 22:18 |
| *** dosaboy has quit IRC | 22:29 | |
| *** dosaboy has joined #openstack-qa | 22:30 | |
| *** anteaya has quit IRC | 22:45 | |
| *** EmilienM has quit IRC | 22:47 | |
| *** EmilienM has joined #openstack-qa | 22:52 | |
| *** krtaylor has quit IRC | 22:57 | |
| *** mlavalle has quit IRC | 23:02 | |
| openstackgerrit | A change was merged to openstack/tempest: Add locking to test_aggregates https://review.openstack.org/38857 | 23:04 |
| comstud | sdague: That's fair | 23:21 |
| comstud | sdague: Yea, I was def going to start with looking at #1 | 23:21 |
| comstud | I had some other distractions today so I didn't get started yet | 23:21 |
| sdague | comstud: that's a devstack change | 23:21 |
| sdague | well a devstack & a tempest change | 23:21 |
| comstud | ok | 23:21 |
| comstud | nod | 23:21 |
| sdague | to config it right for the gate | 23:21 |
| sdague | anyway, I think it's a path forward | 23:22 |
| comstud | I'm not planning on just forgetting that there's skips for cells | 23:23 |
| comstud | we'll work towards addressing them, I just don't know how quickly it's going to be | 23:23 |
| comstud | There's some other high priority things for H-3 for nova to complete | 23:24 |
| comstud | We need to revive this: https://review.openstack.org/#/c/25813/ | 23:25 |
| comstud | And make sure it's 'the right way' | 23:25 |
| comstud | sdague: I assume a link to a BP is also acceptable? Or maybe there should be a related tempest bug linking to the BP? | 23:26 |
| comstud | I'm thinking the latter | 23:27 |
| sdague | so it should be a nova bug | 23:27 |
| sdague | which can link to the BP | 23:27 |
| sdague | because it's actually a bug in nova that we are skipping on | 23:28 |
| comstud | depending on how you look at it | 23:28 |
| sdague | it's not a tempest bug that cells doesn't implement sec groups :) | 23:28 |
| comstud | it's really an unimplemented feature | 23:28 |
| comstud | :) | 23:28 |
| comstud | well.. depending on how you look at it ;) | 23:28 |
| comstud | But yes, that is true.. it's not a tempest bug. | 23:28 |
| comstud | I don't mind finding a nova bug for them at all | 23:29 |
| comstud | er filing | 23:29 |
| sdague | cool | 23:31 |
| sdague | they should be descreet enough to be plucked off one at a time. So one for aggregates support, one for sec groups, one for floating ips | 23:32 |
| comstud | yeah | 23:33 |
| comstud | i'll prob start working on that tomorrow.. i'm out for now | 23:35 |
| sdague | have a good evening | 23:49 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!