| *** mkollaro has quit IRC | 00:06 | |
| *** marun has quit IRC | 00:10 | |
| *** lexinator has quit IRC | 00:23 | |
| *** lexinator has joined #openstack-qa | 00:27 | |
| *** harlowja has quit IRC | 00:47 | |
| *** bclifford has joined #openstack-qa | 01:18 | |
| *** bclifford has left #openstack-qa | 01:24 | |
| *** lexinator has quit IRC | 01:46 | |
| *** anteaya has quit IRC | 01:56 | |
| *** lexinator has joined #openstack-qa | 02:00 | |
| *** sdague has quit IRC | 02:34 | |
| *** sdague has joined #openstack-qa | 02:44 | |
| *** HenryG has joined #openstack-qa | 02:45 | |
| Anju | h | 03:23 |
|---|---|---|
| Anju | fazekas:good morning | 03:24 |
| Anju | afazekas_zz :good morning | 03:24 |
| *** afazekas_zz is now known as afazekas | 03:29 | |
| afazekas | Anju: good morning | 03:29 |
| Anju | :) | 03:29 |
| Anju | afazekas: https://review.openstack.org/#/c/38102/7 | 03:30 |
| Anju | i correct the comments | 03:30 |
| Anju | given | 03:30 |
| Anju | and it si running in my devstack | 03:30 |
| Anju | resp, server = self.create_server(wait_unti='ACTIVE') | 03:31 |
| Anju | i am doing this | 03:31 |
| Anju | and remove this self.client.wait_for_server_status(self.server_id, 'ACTIVE') | 03:31 |
| Anju | is only this Treinishwants | 03:33 |
| Anju | afzekas:in all tests the wait call call is after the creation | 03:36 |
| openstackgerrit | Sreeram Yerrapragada proposed a change to openstack-dev/devstack: Add support for vmdk files as glance images https://review.openstack.org/38576 | 03:43 |
| *** prekarat has joined #openstack-qa | 03:43 | |
| openstackgerrit | Sreeram Yerrapragada proposed a change to openstack-dev/devstack: Add support for vmdk files as glance images https://review.openstack.org/38576 | 03:46 |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Added a server-pause test https://review.openstack.org/38102 | 03:54 |
| *** lexinator has quit IRC | 04:19 | |
| *** SergeyLukjanov has joined #openstack-qa | 04:27 | |
| *** harlowja has joined #openstack-qa | 04:46 | |
| openstackgerrit | A change was merged to openstack/tempest: Use nose skip exception conditionally https://review.openstack.org/38284 | 04:53 |
| Anju | afazekas:ping | 04:56 |
| Anju | sean :ping | 05:12 |
| Anju | sean :please revoiew this https://review.openstack.org/#/c/38102/7 | 05:13 |
| *** afazekas_ has joined #openstack-qa | 05:15 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: unskip test_register_get_deregister_ari_image https://review.openstack.org/38744 | 05:42 |
| *** prekarat has quit IRC | 05:46 | |
| *** SergeyLukjanov has quit IRC | 05:57 | |
| *** mkollaro has joined #openstack-qa | 05:58 | |
| mkoderer | Anju: pong - I reviewed it.. just some minor findings | 06:01 |
| Anju | mkoderer: | 06:14 |
| Anju | ping | 06:14 |
| Anju | hii I saw .. | 06:14 |
| Anju | yesterday the same problme with the blueprint name | 06:14 |
| Anju | mkoderer: i change the name of the blueprint | 06:15 |
| Anju | and the i commit in the same branch | 06:15 |
| mkoderer | Anju: but it isn't a blueprint for my understanding | 06:16 |
| Anju | mkoderer:https://blueprints.launchpad.net/tempest/+spec/add-negative-test-for-server | 06:17 |
| mkoderer | Anju: ok so you change it to be more general | 06:18 |
| Anju | means?? | 06:19 |
| mkoderer | Anju: so pls correct your commit message and add a line "Implements: bp add-negative-test-for-server" | 06:19 |
| Anju | ok ok | 06:19 |
| mkoderer | this will change the topic | 06:19 |
| Anju | and the commit message is correct? | 06:19 |
| mkoderer | mhmh you have still the lowercase issue.. | 06:20 |
| Anju | in "Neagtive" | 06:20 |
| mkoderer | so Negative = negative and Pausing = pausing | 06:20 |
| Anju | yeah | 06:20 |
| Anju | :P | 06:20 |
| mkoderer | ;) | 06:20 |
| mkoderer | afazekas: ping | 06:21 |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Added a server-pause test https://review.openstack.org/38102 | 06:24 |
| Anju | mkoderer: have a look | 06:24 |
| Anju | :) | 06:24 |
| Anju | 11 patches i have committed for a small one | 06:25 |
| Anju | :( | 06:25 |
| *** mkollaro has quit IRC | 06:25 | |
| mkoderer | thats typical for the beginning | 06:31 |
| mkoderer | don't worry, it becomes easier with the time :) | 06:32 |
| Anju | no idea if it is good or bad for me | 06:32 |
| mkoderer | Anju: it's good.. since you learn and improve | 06:34 |
| Anju | mkoderer:what is blueprint defination | 06:34 |
| *** prekarat has joined #openstack-qa | 06:34 | |
| mkoderer | Anju: you defined a blueprint in launchpad, and usually this must be approved by Sean | 06:35 |
| Anju | oh god | 06:35 |
| Anju | :) | 06:35 |
| Anju | i dont know what will be his suggestions | 06:35 |
| mkoderer | So it's just that one of the core reviews give a statement about it | 06:36 |
| mkoderer | the patch is good so let's see | 06:36 |
| Anju | :) | 06:36 |
| Anju | thanks mkoderer | 06:36 |
| mkoderer | just wait until someone awakes ;) | 06:36 |
| Anju | he he | 06:36 |
| Anju | :) | 06:36 |
| Anju | afazekas is quite busy | 06:37 |
| openstackgerrit | nayna-patel proposed a change to openstack/tempest: Added 2 quota related test cases for Neutron API https://review.openstack.org/35487 | 06:48 |
| afazekas_ | mkoderer: Pong | 07:21 |
| afazekas_ | sorry I left my another machine logged in .. | 07:21 |
| *** SergeyLukjanov has joined #openstack-qa | 07:26 | |
| mkoderer | afazekas_: regarding the "skipped" issue | 07:28 |
| mkoderer | afazekas_: was there an action item about it? | 07:28 |
| mkoderer | is it a blueprint? | 07:28 |
| mkoderer | I am just curios of the solution | 07:29 |
| *** plomakin has quit IRC | 07:30 | |
| afazekas_ | mkoderer: I have checked several skipped tests, ec2 ones will be solved (or became easily solvable) after https://wiki.openstack.org/w/index.php?title=Blueprint-ec2-error-codes&mobileaction=toggle_view_desktop (Looks like the wiki formats the text in different way, than at the writing time) | 07:31 |
| mkoderer | afazekas_: I mean this solution with the time stamp | 07:32 |
| *** plomakin has joined #openstack-qa | 07:32 | |
| afazekas_ | mkoderer: IMHO git blame + git show combination can do it | 07:33 |
| mkoderer | ok so we could enhance the tool for this | 07:35 |
| mkoderer | afazekas_: https://github.com/openstack/tempest/blob/master/tools/skip_tracker.py | 07:36 |
| mkoderer | so enhancing it that is show old skips and ignores new ones maybe? | 07:37 |
| afazekas_ | git blame tempest/api/compute/images/test_images_oneserver.py | grep testtools.skip\( | cut -f 1 -d' ' | while read a; do echo -n "$a: "; git show --pretty="%cr" $a| head -1; done | 07:41 |
| afazekas_ | 770e5a44: 4 months ago | 07:41 |
| afazekas_ | Poc | 07:41 |
| afazekas_ | we can collect the the age of each line | 07:41 |
| mkoderer | yes something like this | 07:42 |
| afazekas_ | https://bugs.launchpad.net/nova/+bug/1160749 looks like it became an extension, so the skip of the quota tests should be a config option | 07:55 |
| afazekas_ | Or just the usage of the 'force' .. | 07:56 |
| afazekas_ | mkoderer: I think the skip tracker can be anhanced by age and be test case lister. Even an auto unskip patch is possible, but looks like most the skipped issues are real issues and not solved yet | 07:57 |
| afazekas_ | One swift related issue is devstack config issue.. | 07:57 |
| afazekas_ | (or swift default) | 07:57 |
| mkoderer | afazekas_: yes I think this would be a good idea - should we call the git commands or should we use a python lib for that? | 08:00 |
| afazekas_ | For me both approach is ok | 08:05 |
| afazekas_ | AFAIK most people prefers the python libraries in tempest when it is doable | 08:06 |
| Anju | afazekas: | 08:08 |
| Anju | how to parse an json to xml | 08:08 |
| Anju | https://github.com/openstack/tempest/blob/master/tempest/services/volume/xml/volumes_client.py | 08:10 |
| Anju | in this why the volume_parse function is avialable | 08:10 |
| Anju | what is the functionality of these functions | 08:11 |
| Anju | in some xml | 08:11 |
| Anju | only a xml_to _json function is used | 08:11 |
| Anju | afazekas_ | 08:12 |
| Anju | :ping | 08:12 |
| afazekas_ | Long time ago the json client implemented first, and the xml client just came after the json | 08:16 |
| Anju | afazekas_ | 08:16 |
| afazekas_ | actually that function outputs a python dict (or data structure) | 08:16 |
| afazekas_ | in this way the test cases can be the same, just the rest_client needs to be different | 08:16 |
| Anju | i see that in some files | 08:18 |
| Anju | a function exmaple parse _volume is availble in xml file | 08:18 |
| *** SergeyLukjanov has quit IRC | 08:18 | |
| afazekas_ | https://review.openstack.org/#/c/32258/ | 08:19 |
| Anju | https://github.com/openstack/tempest/blob/master/tempest/services/compute/json/aggregates_client.py | 08:19 |
| Anju | https://github.com/openstack/tempest/blob/master/tempest/services/compute/xml/aggregates_client.py | 08:19 |
| Anju | see the difference | 08:19 |
| afazekas_ | some cases the automated conversion does not working, in this case they are converted by a dedicated function | 08:20 |
| Anju | how one can know that these can not be converted? | 08:20 |
| afazekas_ | when it is not working it is clear indicator :) | 08:21 |
| Anju | now what is the way | 08:21 |
| Anju | i mean how a list aggreagte funtion is converted | 08:21 |
| Anju | what is basic rule | 08:21 |
| Anju | as some xml tests are missing | 08:21 |
| afazekas_ | you can try to use the xml_to_json at the first time, if you see an issue you need to create a function | 08:22 |
| Anju | afazekas: | 08:22 |
| Anju | i got a issue | 08:22 |
| Anju | but o dont know to to coverse it | 08:22 |
| Anju | but dont know how to covert it | 08:23 |
| Anju | what should i have to maintain | 08:23 |
| Anju | to convert it | 08:23 |
| Anju | example :https://github.com/openstack/tempest/blob/master/tempest/services/compute/xml/aggregates_client.py#L34 | 08:24 |
| Anju | format aggregate | 08:24 |
| afazekas_ | FIXME: the aggregate xml test are working | 08:26 |
| afazekas_ | s/test/tests/ | 08:27 |
| Anju | means? | 08:28 |
| Anju | afazekas:please dont reply will ask from home | 08:28 |
| Anju | will anot able to see your answer | 08:29 |
| Anju | wiil ask from home | 08:29 |
| Anju | thanks | 08:29 |
| Anju | :) | 08:29 |
| afazekas_ | http://logs.openstack.org/44/38744/1/check/gate-tempest-devstack-vm-full/35785/console.html | 08:29 |
| afazekas_ | tempest.api.compute.admin.test_aggregates.AggregatesAdminTestJSON.test_aggregate_create_as_user[gate,negative] ... ok | 08:29 |
| Anju | wiil continue in the same | 08:29 |
| afazekas_ | tempest.api.compute.admin.test_aggregates.AggregatesAdminTestXML.test_aggregate_create_as_user[gate,negative] ... ok | 08:29 |
| Anju | no not in this | 08:30 |
| Anju | there are many files in which some test cases for xml | 08:30 |
| Anju | is not working | 08:30 |
| Anju | so give you details from home | 08:30 |
| Anju | :) | 08:30 |
| Anju | this was just an example | 08:31 |
| Anju | i know that this was working | 08:31 |
| Anju | :) | 08:31 |
| Anju | thanks | 08:31 |
| Anju | :) | 08:31 |
| afazekas_ | bye | 08:31 |
| afazekas_ | you are welcome | 08:31 |
| *** SergeyLukjanov has joined #openstack-qa | 08:55 | |
| *** afazekas has quit IRC | 09:04 | |
| *** afazekas_ is now known as afazekas | 09:08 | |
| afazekas | mkoderer: https://review.openstack.org/#/c/36652/4/tempest/stress/actions/volume_attach_delete.py | 09:12 |
| afazekas | Will you change this case to use the nova api ? | 09:12 |
| afazekas | https://github.com/openstack/tempest/blob/master/tempest/services/compute/json/servers_client.py#L294 | 09:15 |
| afazekas | https://github.com/openstack/tempest/blob/master/tempest/services/compute/json/servers_client.py#L306 | 09:15 |
| *** turul_ has joined #openstack-qa | 09:18 | |
| *** mkollaro has joined #openstack-qa | 09:36 | |
| openstackgerrit | nayna-patel proposed a change to openstack/tempest: Added 2 quota related test cases for Neutron API https://review.openstack.org/35487 | 09:40 |
| *** fbo is now known as fbo_away | 09:49 | |
| *** mkollaro1 has joined #openstack-qa | 09:54 | |
| *** mkollaro has quit IRC | 09:54 | |
| openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Enable Heat and Neutron and Horizon service https://review.openstack.org/38263 | 09:57 |
| openstackgerrit | A change was merged to openstack/tempest: add image tests https://review.openstack.org/37524 | 10:18 |
| *** prekarat has quit IRC | 10:24 | |
| afazekas | fungi: ping | 10:46 |
| afazekas | https://bugs.launchpad.net/tempest/+bug/1202978 | 10:47 |
| afazekas | What kind of steps are necessary to close this bug ? | 10:48 |
| *** prekarat has joined #openstack-qa | 10:53 | |
| *** mkollaro1 has quit IRC | 10:59 | |
| afazekas | mkoderer: ping | 11:01 |
| *** afazekas is now known as afazekas_zz | 11:07 | |
| mkoderer | afazekas_zz: sry quite busy .. whats up? | 11:12 |
| afazekas_zz | Are you planning to move to nova volume api this change: https://review.openstack.org/#/c/36652/4/tempest/stress/actions/volume_attach_delete.py ? | 11:14 |
| mkoderer | afazekas_zz: yes I will | 11:14 |
| afazekas_zz | mkoderer: Will you have time for improving the skip_tracker ? | 11:14 |
| mkoderer | just lack of time | 11:14 |
| mkoderer | afazekas_zz: that's my plan - but still quite busy with other tasks | 11:15 |
| afazekas_zz | I know the issue, I still did not read my e-mails, but I try to do some coding today :) | 11:15 |
| afazekas_zz | thank you | 11:15 |
| mkoderer | I think I need to work in the night for upstream... ;) | 11:16 |
| *** prekarat has quit IRC | 11:41 | |
| *** krtaylor has joined #openstack-qa | 11:59 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Use vmware section https://review.openstack.org/38460 | 12:01 |
| *** fbo_away is now known as fbo | 12:05 | |
| 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 | 12:13 |
| openstackgerrit | A change was merged to openstack/tempest: Added a server-pause test https://review.openstack.org/38102 | 12:25 |
| openstackgerrit | A change was merged to openstack/tempest: Basic starter scenario for testing the dashboard https://review.openstack.org/38495 | 12:25 |
| 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 | 12:26 |
| afazekas_zz | sdague: It should be approved IMHO https://blueprints.launchpad.net/tempest/+spec/fix-gate-tempest-devstack-vm-quantum-full | 12:32 |
| *** dkranz has joined #openstack-qa | 12:33 | |
| openstackgerrit | A change was merged to openstack/tempest: add service tests https://review.openstack.org/37696 | 12:34 |
| openstackgerrit | A change was merged to openstack/tempest: Add scenario test of instance/volume snapshot https://review.openstack.org/36199 | 12:34 |
| afazekas_zz | dkranz: https://review.openstack.org/#/c/35302/ can you review it | 12:36 |
| dkranz | afazekas_zz: Done. | 12:41 |
| *** jasondotstar has joined #openstack-qa | 12:41 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Simplify whitebox/manager https://review.openstack.org/35302 | 12:53 |
| afazekas_zz | dkranz: rebased | 12:53 |
| *** krtaylor has quit IRC | 12:54 | |
| *** SergeyLukjanov has quit IRC | 12:57 | |
| *** afazekas_zz has quit IRC | 13:03 | |
| *** anteaya has joined #openstack-qa | 13:06 | |
| *** SergeyLukjanov has joined #openstack-qa | 13:20 | |
| *** turul_ is now known as afazekas | 13:28 | |
| fbo | Hello guys, what is the differences between gate and smoke tests in tempest ? or where can I find an explanation ? | 13:34 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Bug 1074904 is fixed https://review.openstack.org/38805 | 13:44 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: unskip test_register_get_deregister_ari_image https://review.openstack.org/38744 | 13:47 |
| afazekas | fbo: the smoke is a smaller set of test cases | 13:49 |
| afazekas | Somke one should be selected to be minimal/fast with good coverage, (it can be improved) | 13:50 |
| fbo | afazekas, so what is the rule when I create a new test class if it seems those tests can be run pretty fast then i set smoke attr ? | 13:51 |
| afazekas | fbo: IMHO we will remove lot of smoke attr in the future, but if the test case touches a relatively big thing and nothing else does it and it is even fast, it may get a smoke attr | 13:56 |
| afazekas | the test case should get the gate attr anyway, which implies the smoke | 13:59 |
| afazekas | sorry | 13:59 |
| afazekas | the smoke implies the gate :) | 13:59 |
| fbo | afazekas yes that what i saw my tests was set with smoke but gate is also display as attr when running nose | 14:00 |
| fbo | afazekas, thx for that clarification | 14:00 |
| fbo | afazekas, just another stuff what the meaning of negative attr ? | 14:02 |
| afazekas | they are usually small simple tests which ends with an expected 4xx error code | 14:03 |
| afazekas | So they tries an operation which expected to be fail/rejected | 14:04 |
| fbo | afazekas, ok that seems clear thx | 14:05 |
| afazekas | the 'positive' attribute is deprecated | 14:05 |
| *** anteaya is now known as ant-tree-ya | 14:05 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add jsonschema to requirements.txt https://review.openstack.org/38810 | 14:07 |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add stress test to attach volumes to vm's https://review.openstack.org/36652 | 14:07 |
| *** burt has joined #openstack-qa | 14:08 | |
| mkoderer | afazekas: I fixed it as weekend present for you ;) | 14:09 |
| mkoderer | I am out for today.. | 14:09 |
| afazekas | mkoderer: Have a nice evening/weekend | 14:10 |
| afazekas | mkoderer: thx :) | 14:11 |
| sdague | afazekas: agreed, I approved it | 14:11 |
| sdague | mkoderer: enjoy the weekend! | 14:12 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add jsonschema to requirements.txt https://review.openstack.org/38810 | 14:12 |
| mkoderer | thanks! I really need it ;) | 14:12 |
| *** sdague changes topic to "All things OpenStack QA || QA reviews with no negative feedback - https://review.openstack.org/#/q/status:open+-Verified-1+-CodeReview-1+-CodeReview-2+(project:openstack-dev/grenade+OR+project:openstack/tempest),n,z" | 14:13 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add jsonschema to requirements.txt https://review.openstack.org/38810 | 14:13 |
| afazekas | sdague: thx | 14:17 |
| *** mlavalle has joined #openstack-qa | 14:19 | |
| *** krtaylor has joined #openstack-qa | 14:19 | |
| afazekas | dkranz: https://bugs.launchpad.net/tempest/+bug/1176151 this is not a tempest only "bug" | 14:24 |
| afazekas | the devstack first install the dependencies with a "normal" pip-install | 14:24 |
| dkranz | I think setup.py should work without venv but don't know how to do that. | 14:29 |
| *** matel has joined #openstack-qa | 14:31 | |
| matel | Hi all | 14:31 |
| matel | Hey, I would like to ask about this review: https://review.openstack.org/#/c/38444/ | 14:31 |
| matel | So would it help, if I introduce a variable, like XEN_SETUP_TEXTCONSOLE with a default of "false" - So that by default it's not installed? | 14:31 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Unskip test_register_get_deregister_ari_image https://review.openstack.org/38744 | 14:33 |
| *** lexinator has joined #openstack-qa | 14:37 | |
| afazekas | dkranz: try install the dependencies first; sudo pip-python -r requirements.txt ; sudo pip-python test-requirements.txt | 14:43 |
| dkranz | afazekas: I eventually got it to work but I think the point of the bug ticket is that 'python setup.py develop' should just work. | 14:44 |
| *** mkollaro has joined #openstack-qa | 14:45 | |
| afazekas | dkranz: I see, but I think the not working logic is outside to tempest | 14:45 |
| dkranz | afazekas: What do you mean? Tempest does not assume devstack is running or that tempest is on the same machine as the cloud. | 14:47 |
| dkranz | afazekas: Or at least it should not be assuming that. | 14:47 |
| afazekas | dkranz: I mean the setuptools installs the dependencies with too restrictive permissions | 14:48 |
| dkranz | afazekas: Is this different than if pip is used? | 14:49 |
| HenryG | dtroyer: ping | 14:50 |
| sdague | matel: ok, so the crontab item isn't idempotent is it? I guess I don't fully understand why the cron job is actually needed (especially one that runs every minute) | 14:50 |
| dkranz | afazekas: I'm just not sure why none of the other projects have an issue like this. | 14:50 |
| afazekas | dkranz: AFAIK yes | 14:50 |
| afazekas | dkranz: they have | 14:50 |
| sdague | dtroyer: you about? I've got a couple of fixes for default devstack to work that would be nice to get your eyes on | 14:50 |
| sdague | dtroyer: https://review.openstack.org/#/q/status:open+project:openstack-dev/devstack+branch:master+topic:heatclient,n,z | 14:51 |
| dkranz | afazekas: How did they solve it? | 14:51 |
| dkranz | afazekas: python setup.py develop works on other projects, right? | 14:51 |
| sdague | dkranz: might be worth asking mordred on that | 14:51 |
| sdague | he's the master of setup.py hacking | 14:52 |
| dkranz | sdague: Good idea. I'll point him at the bug ticket. | 14:52 |
| sdague | well, I'd ask him more about how it might be fixable | 14:52 |
| dkranz | sdague: I was going to put that question in the bug ticket. | 14:52 |
| matel | sdauge: Yes, it's idempotent, in the sense, that it always sets the crontab, doesn't append it. This cron job is actually doing two things: Sets up xen, so guest consoles are logged, and rotates the log files. | 14:52 |
| afazekas | dkranz: https://github.com/openstack-dev/devstack/blob/master/functions#L1134 | 14:52 |
| afazekas | this is the magic | 14:53 |
| matel | sdague: Yes, it's idempotent, in the sense, that it always sets the crontab, doesn't append it. This cron job is actually doing two things: Sets up xen, so guest consoles are logged, and rotates the log files. | 14:53 |
| dkranz | afazekas: So do you know how to fix it now? | 14:53 |
| * mordred didn't do it | 14:53 | |
| mordred | sdague, dkranz what? | 14:54 |
| dkranz | mordred: We're trying to understand why python setup.py develop does not work in tempest | 14:54 |
| afazekas | I'll try it with sudo pip-python -r requirements.txt ; sudo pip-python test-requirements.txt ; sudo python setup.py develop | 14:54 |
| matel | sdague: This change is needed to support text console with XenServer - it's the last missing bit to pass full tempest tests. | 14:54 |
| mordred | dkranz: it should definitely work. let me look | 14:54 |
| dkranz | mordred: https://bugs.launchpad.net/tempest/+bug/1176151 | 14:55 |
| HenryG | Sorry to interrupt. Does anyone know of any progress on bug 1199529 ? | 14:55 |
| sdague | matel: is this running in the guests or the host? | 14:56 |
| matel | sdague: this cronjob will run on dom0 | 14:56 |
| sdague | sorry, I haven't fully traced where the tools/xen stuff run | 14:57 |
| mordred | dkranz: couple of things | 14:57 |
| mordred | develop will not install test-requirements | 14:57 |
| matel | sdague: I guess that is called host | 14:57 |
| dkranz | afazekas: ^^^ | 14:57 |
| sdague | yeh, dom0. So in that case what about the unstack case to remove that? | 14:57 |
| matel | sdague: technically it is just a guest :-) | 14:58 |
| mordred | dkranz: I'm tracking down the other things | 14:58 |
| sdague | yeh, I know, I worked on xen back in the day :) | 14:58 |
| dkranz | mordred: Thanks! | 14:58 |
| sdague | my name is still in the man pages | 14:58 |
| mordred | however, I _do_ recommend adding "sudo pip -U setuptools" to your recipe right after sudo apt-get -y build-dep python-mysqldb | 14:58 |
| matel | sdague: unstack runs in domU, so it won't remove this. | 14:58 |
| mordred | dkranz: because of the recent setuptools/distribute upgrade saga | 14:58 |
| afazekas | dkranz: the tempest is running without test requires | 14:58 |
| sdague | oh, right, because you guys only have py24 in dom0 | 14:58 |
| sdague | so you do the split management thing | 14:59 |
| mordred | the permission denied is very weird - I'm looking in to that right now | 14:59 |
| afazekas | dkranz: test-requirements are for tools like flake8 | 14:59 |
| sdague | so that means this cron job is forever running in dom0 after that | 14:59 |
| matel | sdague: Yes, devstack sits in a vm. | 14:59 |
| dkranz | afazekas: But we want that, don't we? | 14:59 |
| matel | sdague: yes | 14:59 |
| sdague | that's the part that seems odd to me | 14:59 |
| dkranz | afazekas: IMO the distinction between test and non-test requirements in tempest is not worthwhile | 15:00 |
| sdague | but I guess if that's the way you want it, I'm fine with it now | 15:00 |
| afazekas | dkranz: sudo pip-python install -r test-requirements.txt and you get it | 15:00 |
| sdague | dkranz: to pass the docs job we need the distinction | 15:01 |
| sdague | because the docs job only installs test-requirements, not requirements | 15:01 |
| dkranz | sdague: Oh, OK. | 15:01 |
| matel | sdague: Ok, it's pretty harmless stuff, it's like adding a feature to your hypervisor, like xenapi plugins. | 15:01 |
| sdague | matel: ok, I just +2ed it | 15:02 |
| *** krtaylor has quit IRC | 15:02 | |
| matel | sdague: Thanks. | 15:02 |
| *** lexinator has quit IRC | 15:04 | |
| afazekas | dkranz: I am not sure why to move tempest to the sys.path, just installing the dependencies is usually good for me | 15:06 |
| afazekas | dkranz: http://fpaste.org/28126/13748513/ this is working | 15:10 |
| sdague | mtreinish: so my testr stuff stalled last night, I think because I had 12 abandoned guests hanging out | 15:10 |
| sdague | rerunning now | 15:10 |
| mtreinish | sdague: ok yeah that might do it | 15:11 |
| sdague | though why it stalled on that... is interesting | 15:11 |
| sdague | quota limit? | 15:11 |
| sdague | I would have thought we'd explode instead | 15:11 |
| sdague | mtreinish: is there an easy way to pass testr args through tox? | 15:14 |
| *** lexinator has joined #openstack-qa | 15:16 | |
| clarkb | if tox is configured propery `tox -epy27 -- testr args here` should work | 15:16 |
| clarkb | doesn't look like tempest tox is setup that way | 15:17 |
| mtreinish | clarkb: no it's not I forgot to add posargs | 15:18 |
| mtreinish | I'll push out a change to add it | 15:18 |
| mtreinish | sdague: I've just been running testr run --parallel manually | 15:19 |
| dkranz | afazekas: Great. So we just have to put that in the README. | 15:20 |
| dkranz | afazekas: What OS were you using? | 15:20 |
| afazekas | dkranz: without the setup.py develop your cwd must be the tempest dir , it is implicitly added to the sys.path in addition | 15:20 |
| openstackgerrit | Fabien Boucher proposed a change to openstack/tempest: Add test for swift ACLs https://review.openstack.org/38823 | 15:21 |
| afazekas | custom F19 image with pre-installed C/C++ headers and dependencies | 15:21 |
| dkranz | afazekas: I guess this needs to work on Ubuntu and vanilla Fedora and RHEL | 15:22 |
| afazekas | dkranz: yes, on ubuntu the pip install command is different | 15:23 |
| afazekas | just 'pip' instead of 'pip-python' | 15:23 |
| dtroyer | HIPpwd01 | 15:24 |
| afazekas | dkranz: https://github.com/openstack-dev/devstack/blob/master/functions#L1480 | 15:25 |
| dkranz | afazekas: So why can't this stuff go in setup.py and you do 'sudo python setup.py develop' ? | 15:26 |
| afazekas | dkranz: https://bugs.launchpad.net/tempest/+bug/1188932 just the xsltlib-dev / xsltlib-devel is missing now | 15:27 |
| dtroyer | sdague: +2 | 15:27 |
| afazekas | dkranz: IMHO it is setuptools bug/feature , may be the mainual can say something smart about the permission issue | 15:28 |
| afazekas | dkranz: As I remember you can extend it to use two requirements.txt | 15:29 |
| afazekas | dkranz: You can check the git history as well, long time ago it was solved in a different way .. | 15:30 |
| *** fbo is now known as fbo_away | 15:31 | |
| *** fbo_away is now known as fbo | 15:31 | |
| *** fbo is now known as fbo_away | 15:31 | |
| *** Anju_ has joined #openstack-qa | 15:31 | |
| *** krtaylor has joined #openstack-qa | 15:33 | |
| *** lexinator has quit IRC | 15:37 | |
| afazekas | dkranz: https://github.com/openstack-dev/pbr and the https://pypi.python.org/pypi/d2to1 are new here | 15:38 |
| sdague | dtroyer: cool, if you could +A it in, it's blocking a few folks | 15:42 |
| sdague | I know it would make dansmith happy | 15:42 |
| dtroyer | oh, well, fine... | 15:42 |
| sdague | :) | 15:43 |
| sdague | I don't know who else would +2 besides us anyway :) | 15:43 |
| dtroyer | I'm trying to book a hotel in CA and nothing is available…1 week in advance? go figure... | 15:43 |
| sdague | wow, bummer | 15:47 |
| sdague | is there some big event? | 15:48 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add posargs to testr-full tox job. https://review.openstack.org/38827 | 15:48 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add environmental variables to test.conf https://review.openstack.org/38828 | 15:48 |
| sdague | seems odd that everything would be sold out | 15:48 |
| dtroyer | maybe, but I'm just going to the Mother Ship for a visit | 15:48 |
| openstackgerrit | saurabh proposed a change to openstack/tempest: Add Neutron CLI tests to tempest https://review.openstack.org/38829 | 15:48 |
| afazekas | mtreinish: https://review.openstack.org/#/c/38828/1 can it work without an additional tempest change ? | 15:51 |
| mtreinish | afazekas: yeah, it doesn't have any requirements. I just got lazy and pushed it as a series with the tox update | 15:52 |
| mtreinish | those are options for testr | 15:52 |
| afazekas | mtreinish: in nova they processed by the test.py | 15:53 |
| mtreinish | afazekas: they are, hmm the STDOUT stuff worked for me when I tested it | 15:54 |
| mtreinish | let me take a look | 15:54 |
| sdague | afazekas: is julie on irc? | 15:54 |
| sdague | the timestamp scenario test looks like it has a race - https://bugs.launchpad.net/tempest/+bug/1205344 | 15:54 |
| mtreinish | afazekas: yep you're right let me port that over to tempest | 15:55 |
| *** marun has joined #openstack-qa | 15:56 | |
| afazekas | sdague: at least the ssh connection was successful | 15:57 |
| sdague | afazekas: yep | 15:58 |
| sdague | it's a good test, just curious why it failed | 15:58 |
| sdague | but someone should look at it soon, otherwise we'll need to skip, as it's already failed twice this morning | 15:59 |
| *** SergeyLukjanov has quit IRC | 15:59 | |
| afazekas | the boto test waiting in this way https://github.com/openstack/tempest/blob/master/tempest/thirdparty/boto/test_ec2_instance_run.py#L272 | 16:01 |
| afazekas | sdague: I can add hard coded /dev/vdb wait or something like used in the boto test | 16:07 |
| afazekas | Which one do you prefer ? | 16:07 |
| *** bclifford has joined #openstack-qa | 16:15 | |
| *** Anju_ has quit IRC | 16:21 | |
| *** hemnafk is now known as hemna | 16:29 | |
| *** mkollaro has quit IRC | 16:33 | |
| afazekas | I do it in the fast way.. | 16:42 |
| *** anju_ has joined #openstack-qa | 16:43 | |
| anju_ | afazekas:ping | 16:43 |
| anju_ | afazekas:please tell me how to make the parsing functions | 16:44 |
| *** krtaylor has quit IRC | 16:52 | |
| * afazekas is trying to add something to a sloooow test case in order to fix a critical bug | 16:53 | |
| *** lexinator has joined #openstack-qa | 17:06 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Fix attachment visibility wait/race issue https://review.openstack.org/38841 | 17:06 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add posargs to testr-full tox job https://review.openstack.org/38827 | 17:11 |
| sdague | mtreinish: so the timeouts are hooked in with fixtures in nova | 17:13 |
| *** mlavalle has quit IRC | 17:13 | |
| mtreinish | sdague: yeah that's what it looks like | 17:14 |
| *** krtaylor has joined #openstack-qa | 17:44 | |
| fungi | afazekas: sorry, at a conference and mostly afk... did you still need me on something? | 17:46 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Fix attachment visibility wait/race issue https://review.openstack.org/38841 | 17:48 |
| afazekas | fungi: What needs to be done in order to close this bug https://bugs.launchpad.net/tempest/+bug/1202978 ? | 17:50 |
| afazekas | sdague: ^^^ | 17:51 |
| afazekas | anju_: can you be more specific ? | 17:52 |
| anju_ | afazekas:i gave you an example | 17:52 |
| anju_ | wait | 17:52 |
| fungi | afazekas: the tarballs can be generated with just a few lines changed to openstack-infra/config in modules/openstack_project/zuul/layout.yaml and maybe an additional line to the project jobs definition for tempest under modules/openstack_project/files/jenkins_jobs/config/ | 17:54 |
| anju_ | https://github.com/openstack/tempest/blob/master/tempest/services/compute/json/aggregates_client.py | 17:54 |
| anju_ | https://github.com/openstack/tempest/blob/master/tempest/services/compute/xml/aggregates_client.py | 17:54 |
| fungi | afazekas: i'm happy to draft up the change to add them, but first we need to know whether that's counter to the desires of the tempest team | 17:54 |
| anju_ | afazekas: | 17:55 |
| anju_ | in theses two | 17:55 |
| anju_ | first one is json and the other is for xml | 17:55 |
| anju_ | the function is implemented | 17:56 |
| anju_ | in xml file | 17:56 |
| anju_ | to format aggregate in xml | 17:56 |
| anju_ | what is the fundamental to do ths | 17:56 |
| *** marun has quit IRC | 17:57 | |
| afazekas | fungi: Would it be a daily job or on every commit ? | 17:58 |
| afazekas | anju_: imho first you should capture an xml and json response to compare them | 18:00 |
| anju_ | then | 18:01 |
| *** bclifford has quit IRC | 18:02 | |
| *** bclifford has joined #openstack-qa | 18:02 | |
| afazekas | etree.fromstring(body) creates more python friendly type from the xml string http://lxml.de/tutorial.html | 18:06 |
| afazekas | you need to manipulate this structure to be the same as it came as a json | 18:07 |
| anju_ | ohhhhhhhhhhhhh | 18:08 |
| anju_ | its very hard | 18:08 |
| fungi | afazekas: tempest-master.tar.gz would be regenerated and replaced each time a new commit merges to that branch | 18:08 |
| anju_ | to do | 18:08 |
| anju_ | https://github.com/openstack/tempest/blob/master/tempest/services/compute/xml/aggregates_client.py | 18:08 |
| afazekas | sdague, dkranz, mtreinish, cyeoh, jaypipes: Do you have any objection against generating tarballs ? | 18:10 |
| *** matel has quit IRC | 18:10 | |
| mtreinish | afazekas: none from me | 18:11 |
| dkranz | afazekas: Need to read back... | 18:11 |
| sdague | afazekas: so we don't really do releases, so it seems a little weird | 18:11 |
| mtreinish | sdague: yeah but there's no real harm is there. Especially if some packagers prefer a tarball for whatever dumb reason they come up with. | 18:12 |
| sdague | sure, I guess the versioning might be odd | 18:13 |
| dkranz | afazekas: I have no objection. | 18:13 |
| jaypipes | afazekas: no objections at all. | 18:13 |
| dkranz | sdague: I think a lot of use cases are coming that will make tempest more like a package to be delivered. | 18:13 |
| sdague | dkranz: sure | 18:14 |
| dkranz | sdague: I think we should treat it like any other openstack thing. | 18:14 |
| dkranz | sdague: Including making 'python setup.py develop' work :) | 18:14 |
| sdague | dkranz: well patches welcomed :) | 18:14 |
| mtreinish | sdague: it'll be something like: 2013.2.a1435.g3d3f87c | 18:14 |
| dkranz | sdague: We're working on it. | 18:14 |
| sdague | presumably setup.py might need some tweaks to make it a sane tarball | 18:14 |
| afazekas | dkranz: are you sure it is working with other opestack project ? | 18:15 |
| dkranz | afazekas: Not sure. But I think so. At least some of them. | 18:15 |
| dkranz | sdague: First I have to fix devstack so it can be deployed on an openstack vm and tempest run on a machine on a different network. | 18:17 |
| *** prekarat has joined #openstack-qa | 18:18 | |
| sdague | mtreinish: are you trying to tackle the fixtures timeout thing, or should I give it a shot? | 18:18 |
| mtreinish | sdague: right now I'm down the locking and logging rabbit hole with oslo in tempest if you want to take that over go for it | 18:18 |
| sdague | ok will do | 18:19 |
| afazekas | sdague: dev snapshot :) | 18:22 |
| afazekas | fungi: looks like we are ok with that, I hope it will not cause verify issues | 18:23 |
| fungi | afazekas: not at all. it would run from the post pipeline, so it doesn't even start running until a commit has merged to the branch | 18:24 |
| afazekas | fungi: cool | 18:24 |
| afazekas | dkranz: I have doubts this egg issue is real issue to any packaging system, normally you have the dependent packages installed before packaging the new one. If there is another system what I do not know, all know system can workaround it. | 18:26 |
| fungi | afazekas: i'll try to get the change proposed to add that job some time today | 18:26 |
| dkranz | afazekas: This is beyond my knowledge | 18:27 |
| afazekas | dkranz: I played with 4 different kind of packaging stuff long time ago, I hope it is not changed nowadays :) , not just Linux .. | 18:28 |
| * afazekas may be more .. | 18:29 | |
| *** prekarat has quit IRC | 18:36 | |
| *** bclifford has quit IRC | 18:37 | |
| *** bclifford has joined #openstack-qa | 18:37 | |
| *** mkollaro has joined #openstack-qa | 18:40 | |
| afazekas | sdague,dkranz,mtrienish : can you review this before jenkins says anything https://review.openstack.org/#/c/38841/ Hopefully it will fix the issue, and will be more verbose if not | 18:41 |
| afazekas | anju_: you can try create a small python program for practice | 18:43 |
| anju_ | afazekas:yeah i tried | 18:44 |
| anju_ | will diccuss with you when i will in trobule | 18:45 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Add ceilometer to the sys.path before the nova starts https://review.openstack.org/38702 | 18:48 |
| * afazekas is entering to power state S3 soon | 18:52 | |
| *** anju_ has quit IRC | 18:54 | |
| *** mlavalle has joined #openstack-qa | 19:07 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Sync lockutils from oslo. https://review.openstack.org/38856 | 19:12 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add locking to test_aggregates https://review.openstack.org/38857 | 19:12 |
| mtreinish | sdague: about https://review.openstack.org/#/c/38857/1/etc/tempest.conf.sample will devstack pull that in because it's in the sample or do I need to add an iniset for it? | 19:13 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: fix race condition in service list compares https://review.openstack.org/38859 | 19:13 |
| sdague | mtreinish: I think it comes in from the sample. However you should initset it to the TMPDIR variable that devstack sets up | 19:14 |
| mtreinish | sdague: ok I'll write up a devstack patch for that then | 19:14 |
| *** bclifford has quit IRC | 19:18 | |
| *** bclifford has joined #openstack-qa | 19:18 | |
| *** mlavalle has left #openstack-qa | 19:20 | |
| afazekas | mtreinish: the values from the sample is used unless it is over-written by an iniset | 19:20 |
| mtreinish | afazekas: yeah I'm going to push out a change to devstack to iniset it to devstack's tmpdir | 19:22 |
| *** harlowja has quit IRC | 19:23 | |
| *** bclifford has quit IRC | 19:24 | |
| *** afazekas is now known as afazekas_zz | 19:24 | |
| *** mlavalle has joined #openstack-qa | 19:25 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-dev/devstack: Set lock_path for Tempest to TMPDIR https://review.openstack.org/38861 | 19:28 |
| *** SergeyLukjanov has joined #openstack-qa | 19:35 | |
| *** marun has joined #openstack-qa | 19:55 | |
| *** bclifford has joined #openstack-qa | 20:19 | |
| dkranz | mtreinish: ping | 20:28 |
| mtreinish | dkranz: pong | 20:29 |
| dkranz | mtreinish: Where does the tempest log file go? | 20:29 |
| dkranz | mtreinish: I can't seem to find it. Is more configuration needed to prevent messages from just getting eaten? | 20:30 |
| mtreinish | dkranz: I'm actually looking at that right now because I'm going to convert to oslo logging. From what I can tell nowhere by default. | 20:31 |
| mtreinish | There are 2 environ variables, TEMPEST_LOG_CONFIG_DIR and TEMPEST_LOG_CONFIG to set an output dir | 20:31 |
| mtreinish | by my testing with those didn't work | 20:31 |
| dkranz | mtreinish: :( | 20:31 |
| mtreinish | what we seem to do by default is let nose grab it, and nose will print it on errors and fails | 20:32 |
| mtreinish | but that won't work with testr | 20:32 |
| dkranz | mtreinish: Unfortunately I need to debug a hang that I have to ^c and nothing comes out. | 20:32 |
| dkranz | mtreinish: Is there an easy way to run a test without nose? | 20:32 |
| mtreinish | dkranz: well you can try the variables (maybe you'll have more luck) | 20:33 |
| dkranz | mtreinish: Actually without any test runner? | 20:33 |
| dkranz | mtreinish: OK, I'll try that. | 20:33 |
| mtreinish | dkranz: you'll need to use a test runner | 20:33 |
| mtreinish | it doesn't have to be nose though | 20:34 |
| dkranz | mtreinish: I'll let you know if the variable works | 20:34 |
| mtreinish | In these kind of scenarios I normally force a fail with something like self.assertTrue(False) to trigger output | 20:35 |
| dkranz | mtreinish: That's a good idea. | 20:36 |
| dkranz | mtreinish: Supplying a log variable gives ConfigFileParseError: Failed to parse /tmp/tempest.log: No section: 'formatters' | 20:37 |
| dkranz | mtreinish: I think that variable is for a log formatting file, not the name of the file to log to. | 20:37 |
| dkranz | mtreinish: Blah. | 20:37 |
| mtreinish | dkranz: they're definitely for specifying the output: http://paste.openstack.org/show/42043/ | 20:42 |
| mtreinish | but you might be hitting some other configuration | 20:42 |
| mtreinish | actually I just figured out why it wasn't working for me I was using them wrong let me give it a try | 20:43 |
| dkranz | mtreinish: ok | 20:43 |
| mtreinish | dkranz: oh nm I'm being silly that's to point it to the logging config file | 20:44 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/devstack-gate: Build a devstack-gate image with diskimage-builder https://review.openstack.org/38871 | 20:44 |
| mtreinish | doh | 20:44 |
| dkranz | mtreinish: Yeah, that's what I found out. | 20:44 |
| dkranz | mtreinish: Obvious, isn't it? | 20:45 |
| mtreinish | dkranz: do TEMPEST_LOG_CONFIG_DIR=/opt/stack/tempest/etc TEMPEST_LOG_CONFIG=logging.conf.sample and it'll dump it into tempest.log in /opt/stack/tempest | 20:46 |
| mtreinish | assuming you're using devstack | 20:46 |
| dkranz | mtreinish: I'm running tempest on a different machine than the cluster (which happens to be devstack) but that's probably enough info. | 20:47 |
| mtreinish | dkranz: yeah I know, this is super obvious. Hopefully when I get the move to oslo logging finished it'll be a bit better | 20:47 |
| dkranz | mtreinish: :) | 20:47 |
| *** harlowja has joined #openstack-qa | 20:50 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add environmental variables to test.conf https://review.openstack.org/38828 | 20:52 |
| mtreinish | sdague: ^^^ that should fix the timeout for you | 20:52 |
| sdague | mtreinish: did you try it? | 20:55 |
| sdague | because my code was very much like that | 20:55 |
| sdague | and it didn't time out | 20:55 |
| mtreinish | sdague: I tested that by changing the testr.conf timeout to 1 and most of the tests failed with TimeoutException | 20:56 |
| mtreinish | also all the fails had _StringException: Empty attachments: stderr stdout | 20:56 |
| mtreinish | which means it was trying to capture those too | 20:56 |
| *** burt has quit IRC | 20:57 | |
| sdague | hmmmm... I wonder what I messed up | 20:58 |
| *** stevebaker has joined #openstack-qa | 20:59 | |
| stevebaker | Hi all, feel free to ping me for anything heat related | 21:01 |
| *** jasondotstar has quit IRC | 21:03 | |
| *** bclifford has quit IRC | 21:03 | |
| dkranz | stevebaker: Welcome :) | 21:03 |
| dkranz | stevebaker: I think the plan is to move the slow test to scenario and mark it with a slow tag. | 21:04 |
| dkranz | stevebaker: Then we will change the main tempest run to exclude slow tests. | 21:04 |
| dkranz | stevebaker: Then we create a separate gate job for heat that runs the tests we choose. | 21:04 |
| stevebaker | yep, that all sounds good | 21:04 |
| dkranz | stevebaker: Obviously the second has to be committed before the first. | 21:05 |
| dkranz | stevebaker: I hope to submit that soon but am having loads of trouble trying to run tempest on my laptop and talk to a devstack running on an openstack vm. | 21:06 |
| dkranz | stevebaker: But I just found the last problem :) | 21:06 |
| stevebaker | also, no slow tests will be run until there is an image for them to run against | 21:06 |
| stevebaker | my first attempt will be to invoke diskimage-builder from devstack when heat and tempest are enabled | 21:07 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/devstack-gate: Build a devstack-gate image with diskimage-builder https://review.openstack.org/38871 | 21:10 |
| *** openstack has joined #openstack-qa | 21:14 | |
| dkranz | mtreinish: Do we know how to exclude a test tagged as 'slow' when using testr? | 21:15 |
| *** HenryG has quit IRC | 21:16 | |
| mtreinish | dkranz: that may be a problem. Right now I believe testr only filters on regex matches | 21:16 |
| *** woodspa has quit IRC | 21:16 | |
| mtreinish | so we'd probably need to add an exclude option or something similar | 21:16 |
| mtreinish | lifeless: ^^^ is that right? | 21:17 |
| *** woodspa has joined #openstack-qa | 21:17 | |
| *** bclifford has joined #openstack-qa | 21:17 | |
| mtreinish | clarkb: ^^^ do you know? | 21:18 |
| clarkb | mtreinish: dkranz: regexes are sufficient | 21:18 |
| clarkb | you can do negative lookahead | 21:18 |
| mtreinish | clarkb: yeah that'll work, but it's ugly | 21:18 |
| clarkb | eg https://github.com/openstack/nova/blob/master/tox.ini#L38 | 21:18 |
| clarkb | its not ugly | 21:18 |
| *** HenryG has joined #openstack-qa | 21:18 | |
| dkranz | mtreinish: Well I don't think I can check in the slow skip until that is possible, right? | 21:18 |
| dkranz | mtreinish: It is easy with nose. | 21:18 |
| clarkb | I mean I don't know how it appliesto tags but you can exlcude tests today | 21:19 |
| mtreinish | clarkb: ok that's not as bad as I thought it was. I googled it at one point and it didn't look that simple. | 21:19 |
| dkranz | clarkb: Does that apply to a tag using attr like we do in tempest? | 21:20 |
| clarkb | I have no idea. the whole tag business is foreign to me | 21:20 |
| clarkb | but if you can regex on the tags it should | 21:20 |
| mtreinish | it shouldn't be that different because the tags are at the end of the test id in [] | 21:21 |
| dkranz | clarkb: I have not tried to use testr yet. | 21:21 |
| dkranz | mtreinish: Ah, then it will be easy. | 21:21 |
| *** zigo has quit IRC | 21:21 | |
| *** zigo has joined #openstack-qa | 21:21 | |
| mtreinish | dkranz: for example: tempest.api.image.v1.test_images.CreateRegisterImagesTest.test_register_image_with_min_ram[gate] | 21:22 |
| dkranz | mtreinish: Yeah, I got it. | 21:22 |
| mtreinish | and multiple tags: tempest.api.compute.flavors.test_flavors.FlavorsTestXML.test_invalid_minRam_filter[gate,negative] | 21:22 |
| clarkb | yeah in that case ^(?!.*\[slow\]).*$ | 21:23 |
| clarkb | I think that is the correct regex unless you need to account for [slow,negative] | 21:23 |
| dkranz | clarkb: We don't do no stinking slow negative tests | 21:23 |
| dkranz | clarkb: Well, we'll figure it out if we do :) | 21:24 |
| mtreinish | yeah I don't think such a thing could exist the negative tests are almost instantaneous. | 21:24 |
| mtreinish | you could just do it so slow with any characters in front and after slow inside [] | 21:25 |
| mtreinish | I think that would work as long as we don't have any tags that have slow as part of them | 21:25 |
| clarkb | and if you do you can get more exact with the regex | 21:25 |
| clarkb | wrap with some \bs | 21:25 |
| mtreinish | dkranz: when we start adding service tags that will be a concern | 21:26 |
| clarkb | \[\bslow\b\] | 21:26 |
| *** SergeyLukjanov has quit IRC | 21:30 | |
| mtreinish | dkranz: so you turned on logging for this channel? | 21:35 |
| mtreinish | you should've probably announced that on the ML. Because I had no idea | 21:35 |
| mtreinish | now I have to watch what I say | 21:35 |
| sdague | mtreinish: well there were 50 people in here anyway, most of them probably had local logs :) | 21:38 |
| mtreinish | sdague: fair enough :) | 21:39 |
| lifeless | mtreinish: regex can exclude | 21:41 |
| mtreinish | lifeless: thanks | 21:43 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Switch to using oslo logging. https://review.openstack.org/38879 | 21:43 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Sync lockutils and log from oslo https://review.openstack.org/38856 | 21:43 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add locking to test_aggregates https://review.openstack.org/38857 | 21:43 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add locking to test_aggregates https://review.openstack.org/38857 | 21:47 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Switch to using oslo logging https://review.openstack.org/38879 | 21:47 |
| lifeless | mtreinish: (?!foo) - negatice lookahead on foo, from memory | 21:55 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add locking to test_aggregates https://review.openstack.org/38857 | 22:03 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Switch to using oslo logging https://review.openstack.org/38879 | 22:03 |
| clarkb | sdague: mtreinish http://logs.openstack.org/71/22471/14/gate/gate-tempest-devstack-vm-postgres-full/35820/console.html is that a known problem? | 22:17 |
| clarkb | it just reset the gate, and it looks like something that would be simple to fix in theory | 22:18 |
| *** mkollaro has quit IRC | 22:19 | |
| *** mlavalle has quit IRC | 22:31 | |
| *** mlavalle has joined #openstack-qa | 22:32 | |
| *** harlowja has quit IRC | 22:35 | |
| *** mlavalle has quit IRC | 23:00 | |
| mtreinish | clarkb: it's probably this right? https://bugs.launchpad.net/tempest/+bug/1205344 | 23:00 |
| mtreinish | if so afazekas_zz has a fix pending the gate: https://review.openstack.org/#/c/38841/ | 23:00 |
| clarkb | mtreinish: ya Alex_Gaynor pointed it out to me | 23:01 |
| clarkb | thanks | 23:01 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Set service_available config options in tempest https://review.openstack.org/38379 | 23:04 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/devstack-gate: Proof of concept of kexec/takeovernode https://review.openstack.org/38887 | 23:09 |
| *** mlavalle has joined #openstack-qa | 23:16 | |
| *** mlavalle has quit IRC | 23:19 | |
| *** mlavalle has joined #openstack-qa | 23:19 | |
| *** marun has quit IRC | 23:25 | |
| *** marun has joined #openstack-qa | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!