| *** weshay has joined #openstack-qa | 00:20 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/tempest: Add more detailed message about the volumes missing https://review.openstack.org/42964 | 00:41 |
|---|---|---|
| *** weshay has quit IRC | 01:07 | |
| *** LiangZheng has joined #openstack-qa | 01:18 | |
| *** jasondotstar has joined #openstack-qa | 01:51 | |
| *** anteaya has quit IRC | 02:00 | |
| openstackgerrit | A change was merged to openstack/tempest: Update requirements from global requirements https://review.openstack.org/43168 | 02:01 |
| openstackgerrit | Steve Baker proposed a change to openstack/tempest: Rename heat logical_resource_id to resource_name https://review.openstack.org/43819 | 02:17 |
| *** LiangZheng has quit IRC | 02:29 | |
| *** LiangZheng has joined #openstack-qa | 03:00 | |
| *** LiangZheng has quit IRC | 03:04 | |
| *** HenryG has quit IRC | 03:07 | |
| *** LiangZheng has joined #openstack-qa | 03:30 | |
| *** jhenner has joined #openstack-qa | 03:34 | |
| *** LiangZheng has quit IRC | 03:39 | |
| *** jasondotstar has quit IRC | 03:39 | |
| *** Anju has joined #openstack-qa | 03:42 | |
| *** prekarat has joined #openstack-qa | 04:05 | |
| *** yfried has joined #openstack-qa | 04:17 | |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add common "snapshot_server" method https://review.openstack.org/43658 | 04:38 |
| *** jhenner has quit IRC | 04:45 | |
| afazekas | mtreinish: pong | 04:51 |
| *** afazekas has quit IRC | 05:00 | |
| openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds verfication for Bootable Volume https://review.openstack.org/42621 | 05:11 |
| *** SergeyLukjanov has joined #openstack-qa | 05:17 | |
| *** afazekas has joined #openstack-qa | 05:18 | |
| openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds verfication for Bootable Volume https://review.openstack.org/42621 | 05:30 |
| *** giulivo has joined #openstack-qa | 05:43 | |
| mkoderer | afazekas: ping | 05:57 |
| afazekas | mkoderer: pong | 05:58 |
| mkoderer | afazekas: is there any presentation that shows the full continuous integration process of openstack? | 05:59 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Remove wait_for_image_resp_code https://review.openstack.org/38995 | 05:59 |
| mkoderer | I am searching for a figure that I can show in a internal presentation... | 05:59 |
| afazekas | mkoderer: I can't recall to any picture, IMHO you can ask it on the -infra channel as well, the infra team has a lot of good documentation | 06:00 |
| mkoderer | afazekas: ok - do you know somebody that is in our timezone? | 06:01 |
| afazekas | You can google some presentation as well, there is several general overview about QA, but I do not have them in a bookmark | 06:01 |
| afazekas | mkoderer: no | 06:02 |
| mkoderer | yes am already google around | 06:02 |
| mkoderer | afazekas: thank you | 06:03 |
| afazekas | https://www.google.cz/search?q=infra+openstack+gate&client=firefox-a&hs=zEt&rls=org.mozilla:en-US:official&channel=fflb&source=lnms&tbm=isch&sa=X&ei=TEEcUsiUHYah0QXOsYDQDA&ved=0CAkQ_AUoAQ&biw=1920&bih=874#facrc=_&imgdii=_&imgrc=rUPBjYhooJNFNM%3A%3BuZVX2Ia1t6feXM%3Bhttp%253A%252F%252Fdague.net%252Fwp-content%252Fuploads%252F2013%252F02%252Fscreenshot_016-600x451.png%3Bhttp%253A%252F%252Fdague.net%252Fcategory%252Fopenstack%252F%3B600 | 06:04 |
| afazekas | %3B451 | 06:04 |
| afazekas | mkoderer: http://dague.net/category/openstack/ | 06:06 |
| mkoderer | afazekas: ok cool that's what I am searching | 06:08 |
| *** jhenner has joined #openstack-qa | 06:26 | |
| *** raies has quit IRC | 06:33 | |
| Anju | afazekas : ping | 06:48 |
| afazekas | Anju: pong | 06:49 |
| *** tkammer has joined #openstack-qa | 06:50 | |
| Anju | afazekas : | 06:50 |
| Anju | https://github.com/openstack/tempest/blob/master/tempest/scenario/test_network_basic_ops.py#L172 | 06:50 |
| Anju | in this have added a create_port option in this test | 06:51 |
| afazekas | Why would you create port in that test case ? | 06:53 |
| Anju | afazekas : in this router,subnets added | 06:54 |
| Anju | so i did it with port | 06:54 |
| afazekas | that test case a little bit abnormal, because the the test_ methods depends on each other | 06:55 |
| afazekas | test_008 and test_006 are the real test cases the others are preparation steps for these | 06:56 |
| Anju | i added in 004 and in 003 | 06:56 |
| afazekas | Will that port will be used in the 006 or 008 ? | 06:57 |
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/tempest: Append some operations to boot from volume pattern https://review.openstack.org/41048 | 06:58 |
| Anju | afazekas : https://github.com/openstack/tempest/blob/master/tempest/scenario/test_network_basic_ops.py#L175 | 06:58 |
| Anju | where this subnet is in use ? | 06:59 |
| afazekas | Anju: the server will get ip from this subnet | 06:59 |
| Anju | afazekas : @attr(type='smoke') def test_004_check_networks(self): # Checks that we see the newly created network/subnet/router/port via # checking the result of list_[networks,routers,subnets,ports] seen_nets = self._list_networks() seen_names = [n['name'] for n in seen_nets] seen_ids = [n['id'] for n in seen_nets] for mynet in self.networks: self.assertIn(mynet.na | 07:02 |
| Anju | afazekas : http://dpaste.com/1357753/ | 07:02 |
| afazekas | you can boot servers with a port-uuid | 07:03 |
| Anju | have a look what i have done | 07:03 |
| kashyap | afazekas, Morning, do you test with postgres or mostly mariadb? | 07:06 |
| afazekas | Anju: that test case is scheduled to be re-factored to real independent test cases, in independent logic, would be better if the newly created port would be used explicitly for server boot | 07:09 |
| afazekas | kashyap: I am using the mariadb more frequently | 07:10 |
| Anju | afazekas : can u be a bit more clear | 07:14 |
| afazekas | Anju: nova help boot | 07:14 |
| afazekas | attach NIC | 07:15 |
| afazekas | to port with this UUID (optional) | 07:15 |
| kashyap | afazekas, Ok, thanks. | 07:15 |
| *** sileht has quit IRC | 07:19 | |
| *** sileht has joined #openstack-qa | 07:20 | |
| *** thomasbiege has joined #openstack-qa | 07:20 | |
| *** thomasbiege has quit IRC | 07:20 | |
| Anju | he test_server_basic_ops.py | 07:24 |
| Anju | afazekas : in the test_server_basic_ops.py ?? | 07:24 |
| Anju | yeah there ins na optianla argument of port | 07:25 |
| Anju | yeah there is an optional argument of port | 07:25 |
| *** SergeyLukjanov has quit IRC | 07:27 | |
| afazekas | giulivo: https://review.openstack.org/#/c/43723/2/tempest/api/volume/test_volumes_actions.py here the wait is necessary, you need to ensure the volume is active before any test case. You can do it in setUp ( or create a combined cleanUp method) | 07:31 |
| giulivo | afazekas, I see what you mean, thanks | 07:36 |
| Anju | afazekas :^^^ | 07:41 |
| afazekas | Anju: can you add it to the server creation ? | 07:42 |
| Anju | in the same file ? | 07:42 |
| afazekas | create_kwargs is there | 07:43 |
| *** jgallard has joined #openstack-qa | 07:44 | |
| *** shanks has joined #openstack-qa | 07:45 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add expired tokens tests https://review.openstack.org/43301 | 07:47 |
| Anju | afazekas : noe these tsts are very confusing for me | 07:52 |
| Anju | can u plaese give me short descryption of these scnario tests ? | 07:52 |
| afazekas | http://docs.openstack.org/developer/tempest/field_guide/scenario.html | 07:53 |
| *** thomasbiege1 has joined #openstack-qa | 07:57 | |
| *** thomasbiege1 has quit IRC | 08:00 | |
| *** fbo_away is now known as fbo | 08:10 | |
| jhenner | psedlak: afazekas: I like the build stability plugin: http://jenkins.rhev.lab.eng.brq.redhat.com:8080/user/jaroslav.henner@gmail.com/my-views/view/Jarynovo/job/rhos-tempest-all/80/testReport/ | 08:13 |
| afazekas | psedlak: https://review.openstack.org/#/c/42069/ | 08:19 |
| *** thomasbiege1 has joined #openstack-qa | 08:20 | |
| *** thomasbiege1 has quit IRC | 08:24 | |
| *** giulivo has quit IRC | 08:34 | |
| *** giulivo has joined #openstack-qa | 08:35 | |
| *** yassine has joined #openstack-qa | 08:58 | |
| openstackgerrit | Mate Lakat proposed a change to openstack-dev/devstack: xenapi: devstack support for raw ovf image upload https://review.openstack.org/42172 | 09:08 |
| Anju | afazekas : need to make a create_port like this also ? | 09:12 |
| Anju | https://github.com/openstack/tempest/blob/master/tempest/scenario/test_network_basic_ops.py#L167 | 09:12 |
| afazekas | IMHO try to add it to the 004 or 005 . The steps from 001..005 will be merged sooner or later | 09:14 |
| afazekas | as you can see the other scenario tests: | 09:15 |
| afazekas | - just have a single test_method | 09:15 |
| Anju | i ahve added the ports in 005 | 09:15 |
| afazekas | - using the pythonclients | 09:15 |
| Anju | https://github.com/openstack/tempest/blob/master/tempest/scenario/test_network_basic_ops.py#L219 | 09:15 |
| afazekas | Anju: You can send it to the review, and you also can ask marun's opinion (he added the test case originally) | 09:16 |
| Anju | ok afazekas thanks | 09:19 |
| Anju | afazeka s: have a look plesae http://dpaste.com/1357846/ | 09:20 |
| Anju | afazekas : ^^^ | 09:22 |
| afazekas | I guess the ports belongs to one of the nics dict element | 09:24 |
| Anju | oh yeah afzeka s | 09:25 |
| Anju | afazekas | 09:25 |
| *** yassine has quit IRC | 09:27 | |
| afazekas | you can try it with cli with --debug option in order to see how it is used | 09:27 |
| *** yassine_ has joined #openstack-qa | 09:28 | |
| openstackgerrit | Julien Danjou proposed a change to openstack-dev/devstack: Switch Ceilometer default backend to MySQL https://review.openstack.org/43851 | 09:46 |
| Anju | afazekas : but i am not making any new port here | 09:53 |
| afazekas | Anju: you created in the code you pasted before | 10:00 |
| openstackgerrit | Mate Lakat proposed a change to openstack-dev/devstack: xenapi: devstack support for raw tgz image upload https://review.openstack.org/42172 | 10:00 |
| openstackgerrit | Giulio Fidente proposed a change to openstack/tempest: removes self.fail as suggested by HACKING.rst https://review.openstack.org/43723 | 10:03 |
| openstackgerrit | Mate Lakat proposed a change to openstack-dev/devstack: xenapi: devstack support for raw tgz image upload https://review.openstack.org/42172 | 10:16 |
| openstackgerrit | Mate Lakat proposed a change to openstack-dev/devstack: xenapi: devstack support for raw tgz image upload https://review.openstack.org/42172 | 10:17 |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add common "snapshot_server" method https://review.openstack.org/43658 | 10:22 |
| Anju | afazekas : https://github.com/openstack/tempest/blob/master/tempest/scenario/test_network_basic_ops.py#L209 | 10:31 |
| Anju | when i am printing network here it si blank | 10:31 |
| *** Anju has quit IRC | 10:41 | |
| *** Anju has joined #openstack-qa | 10:41 | |
| afazekas | Anju: 0 <DeletableNetwork id="d350fe18-af88-48f7-b70c-5a98b0ba05b4" name="network-smoke-69311350"> | 10:42 |
| Anju | u did that? | 10:42 |
| afazekas | + open('/dev/tty','w').write("{0} {1}\n".format(i, network)) | 10:43 |
| afazekas | this bypasses all capture/ monkey patch attempts what we ever used in tempest | 10:45 |
| *** prekarat has quit IRC | 11:12 | |
| *** weshay has joined #openstack-qa | 11:21 | |
| *** HenryG has joined #openstack-qa | 11:37 | |
| openstackgerrit | Giulio Fidente proposed a change to openstack/tempest: removes self.fail as suggested by HACKING.rst https://review.openstack.org/43723 | 11:42 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Add logging to the python-clients https://review.openstack.org/43868 | 11:58 |
| *** giulivo has quit IRC | 12:38 | |
| *** dkranz has joined #openstack-qa | 12:39 | |
| *** thomasbiege has joined #openstack-qa | 12:45 | |
| *** jasondotstar has joined #openstack-qa | 12:49 | |
| *** Anju1 has joined #openstack-qa | 12:56 | |
| *** dkranz has quit IRC | 12:57 | |
| *** anteaya has joined #openstack-qa | 13:12 | |
| yfried | in tempest/api - if a test result is dependent on execution order (alphabetical or order-written) is this a problem with test (bug)? | 13:15 |
| *** Anju1 has quit IRC | 13:16 | |
| *** mriedem has joined #openstack-qa | 13:27 | |
| *** yfried1 has joined #openstack-qa | 13:35 | |
| *** yfried has quit IRC | 13:37 | |
| *** Anju1 has joined #openstack-qa | 13:43 | |
| mtreinish | afazekas: don't worry about it anymore. I was going to ask you to +a the parallel migration patch. But, I just self approved it last night. | 13:53 |
| afazekas | mtreinish: how the neutron parallel solved ? | 13:55 |
| afazekas | looks like it is disabled :( | 13:57 |
| mtreinish | afazekas: it's not we just pushed through the full tox jobs. the smoke tox job is still in serial | 13:57 |
| afazekas | We have two possible solution | 13:57 |
| afazekas | - 1. switching the 'private' named net to shared | 13:57 |
| afazekas | -2. we add some "code" to the credentials isolation, to create router + net + subnet for every tenant when it is required | 13:58 |
| afazekas | BTW: looks like the aggregate some cases still have an issue https://bugs.launchpad.net/tempest/+bug/1217163 | 13:59 |
| mtreinish | afazekas: yeah there are still a few flaky tests. But this was the right time to switch it to gating and get everyone to work on fixing flaky tests together | 13:59 |
| mtreinish | I can't keep up with queue myself | 14:00 |
| mtreinish | I was just looking at the logs for that one | 14:00 |
| mtreinish | it looks like the same issue I've seen elsewhere | 14:00 |
| mtreinish | where networking fails | 14:00 |
| mtreinish | http://logs.openstack.org/68/43868/1/check/gate-tempest-devstack-vm-postgres-full/37db98a/logs/screen-n-cpu.txt.gz#_2013-08-27_12_05_34_411 | 14:00 |
| mtreinish | and then it's followed by a fun db insertion error where it tries to dump the whole traceback into the db and fails because it's >255 characters | 14:01 |
| *** burt has joined #openstack-qa | 14:01 | |
| afazekas | mtreinish: so, nothing to do on tempest side | 14:02 |
| mtreinish | afazekas: not sure yet, I need to figure out what's causing the network error. It might be we need to wait for a periodic or something before we can do certain operations. (I don't know the nova network code very well) | 14:03 |
| *** dkranz has joined #openstack-qa | 14:03 | |
| *** dkranz has quit IRC | 14:04 | |
| * afazekas 5 min git grep chain .. rpc_setup_network_on_host | 14:13 | |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Create discovery option for stress tests https://review.openstack.org/43444 | 14:13 |
| mtreinish | afazekas: you found the bug? | 14:14 |
| afazekas | mtreinish: not yet, I just find the possible rpc call which was handled .. | 14:14 |
| *** dkranz has joined #openstack-qa | 14:20 | |
| afazekas | 'ip route del default dev br100' not too many code path leads to this command IMHO | 14:23 |
| mtreinish | afazekas: yeah if you look at the corresponding network log for the link about. You can see the full code path on the network side. | 14:25 |
| mtreinish | http://logs.openstack.org/68/43868/1/check/gate-tempest-devstack-vm-postgres-full/37db98a/logs/screen-n-net.txt.gz#_2013-08-27_12_05_34_349 | 14:26 |
| mtreinish | I was just tracing through it | 14:26 |
| *** mlavalle has joined #openstack-qa | 14:28 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Move the network api tests to smoke https://review.openstack.org/42517 | 14:29 |
| *** adalbas has quit IRC | 14:32 | |
| afazekas | https://review.openstack.org/42517 rebased , without a full gate these should be part of the smoke job anyway | 14:34 |
| mlavalle | mtreinish: ping | 14:45 |
| mtreinish | mlavalle: hi | 14:46 |
| mtreinish | sorry yesterday I was at the dentist when you pinged me | 14:46 |
| mlavalle | mrtreinish: np. I thought you were busy. Did you have a chance to read my response in https://bugs.launchpad.net/tempest/+bug/1216076 | 14:46 |
| *** yfried1 has quit IRC | 14:48 | |
| mtreinish | mlavalle: I did, if you think that'll work that'd be great nice and simple. But, I like afazekas idea of setting up a network per isolated tenant creation though | 14:48 |
| *** adalbas has joined #openstack-qa | 14:48 | |
| mtreinish | it seems to go better with the tenant isolation model then using one large shared network | 14:48 |
| mlavalle | mtreinish: that's fine. is he going to implement it? I just want to help | 14:49 |
| afazekas | probbaly we might need to extend the external IP pool for separated network creation | 14:49 |
| mlavalle | afazekas: how can I help? | 14:50 |
| afazekas | I got 2* -1 here, so if it really needed it goes to the gate script https://review.openstack.org/#/c/36627/ | 14:51 |
| afazekas | https://github.com/openstack/tempest/blob/master/tempest/common/isolated_creds.py this code is responsible for most isolated creds | 14:52 |
| afazekas | It has the power to create and delete network at the correct time | 14:52 |
| mlavalle | yeah, I've been looking at that code since Sunday afternoon | 14:53 |
| afazekas | mtreinish: are you ok with adding an optional network element creation to this class ? | 14:53 |
| afazekas | mlavalle: the new tenant user needs to chat the networking with the neutron.. | 14:54 |
| mlavalle | And I think what you proposed to do is described here:http://docs.openstack.org/grizzly/openstack-network/admin/content/use_cases_tenant_router.html | 14:54 |
| afazekas | I think the isolated network creation should be optional, because if the system has a shared network, it would cause an issue | 14:55 |
| afazekas | most of the test cases does not specify the nics | 14:55 |
| mlavalle | The commands for this scenario are here: http://docs.openstack.org/grizzly/openstack-network/admin/content/demo_per_tenant_router_network_config.html | 14:56 |
| afazekas | mlavalle: yes , this looks like close to the existing devstack setup | 14:56 |
| afazekas | (BTW the shared and private networks can co-exists in the same system ..) | 14:56 |
| mlavalle | Yes, they can | 14:56 |
| *** giulivo has joined #openstack-qa | 14:57 | |
| afazekas | isolated_creds.IsolatedCreds could get a set as parameter | 14:58 |
| afazekas | the parameter can be defined similarly as the _interface | 14:58 |
| mtreinish | afazekas: yeah that should be fine | 14:58 |
| mtreinish | that's probably the best place for it | 14:59 |
| afazekas | the BaseTestCase should define both 3 tenant | 14:59 |
| mtreinish | unless you wanted to make separate isolated_network class and call that if neutron is enabled | 14:59 |
| afazekas | we should override it in keysone by an empty set | 14:59 |
| mlavalle | what would be passed in that set parameter? | 14:59 |
| afazekas | set(( 'primary' , 'alt' , 'admin' )) # means create network for both three tenant | 15:00 |
| mlavalle | ok | 15:00 |
| afazekas | mtreinish: I am thinking about rename to IsolationManager , and let it do all isolation related stuff | 15:01 |
| mtreinish | afazekas: that's fine | 15:01 |
| afazekas | mlavalle: can you work on it ? | 15:02 |
| mlavalle | afazekas: Yes. I'll start tonight | 15:03 |
| mlavalle | I'll ping you if I have questions | 15:03 |
| * afazekas for me it is end of the day | 15:03 | |
| *** dkranz has quit IRC | 15:03 | |
| mlavalle | I know. I meant tomorrow | 15:03 |
| mlavalle | You are Budapest, aren't' you? | 15:04 |
| * afazekas I will not be here for more than week from Friday | 15:04 | |
| afazekas | mlavalle: In the same tz, but in Brno | 15:04 |
| afazekas | mlavalle: and you ? | 15:04 |
| mlavalle | afazekas: San Antonio, Texas. I am with Rackspace | 15:05 |
| afazekas | mtreinish: I am not sure that operation really needs to remove the default route, and if it already removed by concurrent operation, it should not cry about that.. | 15:06 |
| afazekas | mlavalle: cool | 15:06 |
| mtreinish | afazekas: yeah I'm thinking catching the error and just moving on. | 15:07 |
| mtreinish | but I'll ping vishy about it first | 15:07 |
| afazekas | mtreinish: looks like it is parametric, IMHO it should be logged at least on WARN level... | 15:08 |
| mtreinish | afazekas: yeah, log something like tried to delete but couldn't find route, and then just continue | 15:09 |
| mlavalle | afazekas, mtreinish: ok, we have a plan for the networking isolation stuff. i'll keep you posted | 15:09 |
| afazekas | FYI: this change might solve more random failure issue then in the commit message https://review.openstack.org/#/c/43279/ | 15:09 |
| mtreinish | mlavalle: ok cool | 15:09 |
| mtreinish | afazekas: leave a comment about that then | 15:12 |
| *** dkranz has joined #openstack-qa | 15:17 | |
| *** Anju1 has quit IRC | 15:24 | |
| openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack-dev/devstack: Skip DHCP variables for Nova Network FlatManager https://review.openstack.org/43468 | 15:29 |
| *** amscanne has joined #openstack-qa | 15:31 | |
| *** dkranz has quit IRC | 15:44 | |
| *** afazekas has quit IRC | 15:47 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add unittest framework + tests for wrapper scripts https://review.openstack.org/41770 | 15:56 |
| *** dkranz has joined #openstack-qa | 15:58 | |
| *** Anju1 has joined #openstack-qa | 16:05 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: change quantum into neutron for neutron configuration values in nova.conf https://review.openstack.org/43604 | 16:07 |
| openstackgerrit | A change was merged to openstack-dev/devstack: VMware: update cinder support for VMware configuration settings https://review.openstack.org/42824 | 16:07 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Add support for setting Neutron DHCP agent options. https://review.openstack.org/43212 | 16:07 |
| psedlak | what does the 'Empty attachments:\nstdout\nstderr' means in the output from testr|subunit2pyunit? | 16:31 |
| mtreinish | psedlak: that means that tests are running with a fixture to capture stdout and stderr | 16:32 |
| mtreinish | if the test printed anything out to either it'll be printed when that shows up | 16:32 |
| mtreinish | but it should only happen during failure | 16:33 |
| psedlak | mtreinish: so having those empty is a good result? | 16:33 |
| mtreinish | psedlak: I meant you should only see those during a test failure | 16:33 |
| mtreinish | it just means that nothing was printed to stdout or stderr if it's empty | 16:33 |
| psedlak | mtreinish: i'm trying to improve the cli tests so that commands executed always logs their stdout/err for example ... (to make those current cli.keystone tests debuggable) | 16:34 |
| psedlak | and i'm doing it using LOG.info/debug (same way as rest_client uses for api tests) | 16:35 |
| mtreinish | psedlak: that should work but you'll have to dig it out of the log | 16:35 |
| psedlak | and i was wondering if it wouldn't be better to get it to those testtools results ... | 16:35 |
| psedlak | mtreinish: well i don't have a way how to know in that part of code (comparable to rest_client in this context) to decide if test which called me will fail or not ... | 16:36 |
| mtreinish | psedlak: well the easiest way to do that is just print output and it should be captured by the fixture and printed in the attachments section. | 16:36 |
| psedlak | so i have to log.debug it always ... | 16:36 |
| psedlak | but i'm not sure if just printing it out would be a good thing for nosetests compatability | 16:37 |
| mtreinish | psedlak: no it wouldn't be good for nose. It would show up during all the cli tests | 16:38 |
| psedlak | yeah, so i will go with log.debug for now, thanks :) | 16:38 |
| mtreinish | psedlak: you could also save the output and catch the failure and then log it at the error log level | 16:39 |
| psedlak | mtreinish: i was thinking about something like that before, but it would be really tricky thing to do in this context | 16:40 |
| psedlak | mtreinish: the context i'm talking about is https://github.com/openstack/tempest/blob/master/tempest/cli/__init__.py#L122 | 16:41 |
| *** sthaha has quit IRC | 16:43 | |
| *** jgallard has quit IRC | 16:48 | |
| *** fbo is now known as fbo_away | 16:52 | |
| *** jhenner has quit IRC | 16:54 | |
| *** yassine_ has quit IRC | 17:00 | |
| Anju1 | afazekas : ping | 17:01 |
| *** adalbas has quit IRC | 17:01 | |
| *** thomasbiege has joined #openstack-qa | 17:07 | |
| *** thomasbiege has quit IRC | 17:09 | |
| openstackgerrit | Pavel Sedlák proposed a change to openstack/tempest: Alway log stdout and stderr of CLI commands https://review.openstack.org/43923 | 17:13 |
| openstackgerrit | Pavel Sedlák proposed a change to openstack/tempest: (WIP) Alway log stdout and stderr of CLI commands https://review.openstack.org/43923 | 17:15 |
| *** mlavalle has quit IRC | 17:16 | |
| *** adalbas has joined #openstack-qa | 17:18 | |
| Anju1 | mtrenish : stop/start basic ops can be added here :basic set of operations | 17:25 |
| Anju1 | https://github.com/openstack/tempest/blob/master/tempest/scenario/test_server_basic_ops.py | 17:25 |
| *** HenryG has quit IRC | 17:25 | |
| Anju1 | mtreinish : stop/start basic ops can be added here :basic set of operations | 17:26 |
| Anju1 | (04:29:07 PM) Anju1: https://github.com/openstack/tempest/blob/master/tempest/scenario/test_server_basic_ops.py | 17:26 |
| openstackgerrit | David Kranz proposed a change to openstack/tempest: Force an empty patch to use 'check experimental' https://review.openstack.org/43927 | 17:26 |
| Anju1 | mtreinish: ^^^ | 17:27 |
| *** tkammer has quit IRC | 17:27 | |
| *** thomasbiege has joined #openstack-qa | 17:34 | |
| mtreinish | Anju1: I think if you want to add a scenario test that does start and stop of an instance you should add another scenario test | 17:47 |
| *** thomasbiege has quit IRC | 17:48 | |
| mtreinish | while start/stop is a basic operation I think that it's a different scenario and should be tested separately. | 17:48 |
| Anju1 | mtreinish : but in the same file ? | 17:49 |
| mtreinish | Anju1: no, the trend for scenario tests seems to be one scenario per file. | 17:50 |
| mtreinish | so if you're adding a new scenario you should make a new file for it | 17:50 |
| Anju1 | mtreinish : but unpause/pause , suspend /resume are in the same file | 17:51 |
| *** fbo_away is now known as fbo | 17:54 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: add TEMPEST_LARGE_OPS_NUMBER to option https://review.openstack.org/43778 | 17:54 |
| mtreinish | Anju1: I'm not following did you want to combine 2 scenarios? Or add a new one? | 17:57 |
| mtreinish | psedlak: on https://review.openstack.org/#/c/43923/ why the requirement change? | 17:57 |
| Anju1 | mtreinish : : https://github.com/openstack/tempest/blob/master/tempest/scenario/test_server_basic_ops.py | 17:58 |
| Anju1 | in this file all the basic tests are added | 17:58 |
| Anju1 | have a look | 17:58 |
| Anju1 | sa i am asking to add like stop/start lock/unlock | 17:59 |
| mtreinish | Anju1: no that's a basic usage scenario test. Not the tests for all basic operations. | 18:00 |
| *** SergeyLukjanov has joined #openstack-qa | 18:00 | |
| mtreinish | if you want to add a new scenario that includes start/stop and lock/unlock that would be fine | 18:00 |
| mtreinish | but I don't think you should change that scenario to add more steps to it | 18:00 |
| mtreinish | jog0: do you have any ideas about: https://bugs.launchpad.net/python-keystoneclient/+bug/1217159 | 18:03 |
| mtreinish | (only asking because you're the cli test guy) | 18:03 |
| jog0 | mtreinish: there is a patch to make the cli output better (record all std*) | 18:04 |
| jog0 | mtreinish: I am not sure at the moment sorry | 18:05 |
| mtreinish | jog0: this one?: https://review.openstack.org/#/c/43923/ | 18:05 |
| jog0 | haven't dug into that one | 18:05 |
| jog0 | mtreinish: yeah | 18:05 |
| mtreinish | jog0: ok no problem just thought that I'd ask | 18:05 |
| jog0 | I think that will help sort that one out | 18:05 |
| mtreinish | it's flaky test #1 or 2 right now | 18:05 |
| mtreinish | jog0: yeah, hopefully it'll make it easier to debug | 18:06 |
| *** mlavalle has joined #openstack-qa | 18:06 | |
| *** bodepd has quit IRC | 18:25 | |
| *** thomasbiege has joined #openstack-qa | 18:32 | |
| *** anteaya has quit IRC | 18:33 | |
| *** Anju1 has left #openstack-qa | 18:33 | |
| *** thomasbiege has quit IRC | 18:37 | |
| *** anteaya has joined #openstack-qa | 18:54 | |
| dkranz | mtreinish: Do you know why we are emitting two log messages for each test (one right after another)? | 18:57 |
| mtreinish | dkranz: do you have an example? | 18:59 |
| dkranz | mtreinish: http://logs.openstack.org/91/43791/3/check/gate-tempest-devstack-vm-full/07c8c7f/console.html | 18:59 |
| mtreinish | dkranz: oh I thought you meant in tempest log. That's just what the subunit2pyunit output does. I think it prints a line when the test starts and then anotherone with ... result | 19:01 |
| dkranz | mtreinish: I don't tink so because the times are always the same for both messages. | 19:01 |
| dkranz | mtreinish: If that was the intent then something is not right. | 19:02 |
| mtreinish | dkranz: I think it prints them when they show up in the subunit stream which is probably after the test finishes. | 19:02 |
| dkranz | mtreinish: It would be nice to get rid of that. | 19:03 |
| mtreinish | dkranz: I'd ask lifeless when he comes online he would know better then me. :) | 19:03 |
| dkranz | mtreinish: OK, will do. | 19:03 |
| mtreinish | dkranz: it was a tradeoff because I don't think using the colorized output would work well for the gate | 19:03 |
| mtreinish | and we don't want it to sit there silently for ~20min | 19:03 |
| clarkb | I like it being silent >_> | 19:04 |
| clarkb | it does output stuff on failure | 19:04 |
| mtreinish | clarkb: so did I actually. I should have put "sdague said:" in front of my last message :) | 19:05 |
| *** HenryG has joined #openstack-qa | 19:11 | |
| *** shanks has quit IRC | 19:13 | |
| *** mberwanger has joined #openstack-qa | 19:21 | |
| lifeless | mtreinish: oh, I should have pinged here, sorry ;) | 19:24 |
| mtreinish | lifeless: np, dkranz was just asking me about subunit2pyunit and I didn't have all the answers | 19:24 |
| mtreinish | I figured that you would :) | 19:24 |
| dkranz | lifeless: I was just asking why the tempest output now has two entries for each test, put out at the same time. | 19:25 |
| dkranz | lifeless: As in http://logs.openstack.org/91/43791/3/check/gate-tempest-devstack-vm-full/07c8c7f/console.html | 19:25 |
| lifeless | dkranz: thats probably tightly related to the current bug in testr where things are counted twice | 19:27 |
| lifeless | dkranz: off the cuff I'm going to go with an enumeration event + actually running being mishandled | 19:27 |
| lifeless | dkranz: please feel free to add this data point to the testrepository bug on same | 19:28 |
| dkranz | lifeless: OK, I just wanted to make sure folks knew about it. Will do. | 19:28 |
| lifeless | mtreinish: is this running as 'testr run --subunit | subunit2pyunit' ? | 19:28 |
| dkranz | lifeless: python setup.py testr --slowest --testr-args="--subunit $TESTRARGS" | subunit2pyunit | 19:31 |
| dkranz | lifeless: That's from https://github.com/openstack/tempest/blob/master/tools/pretty_tox.sh | 19:32 |
| *** mberwanger has quit IRC | 19:41 | |
| mtreinish | lifeless: yeah it is | 19:55 |
| mtreinish | sorry I stepped out for a bit | 19:55 |
| lifeless | np | 19:55 |
| mtreinish | dkranz: should I rename the file to not_so_pretty_tox? | 19:56 |
| dkranz | mtreinish: :) | 19:56 |
| *** weshay has quit IRC | 19:57 | |
| *** weshay has joined #openstack-qa | 19:57 | |
| *** ianw has quit IRC | 20:04 | |
| *** ianw has joined #openstack-qa | 20:05 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Switch run_tests.sh to run in parallel by default https://review.openstack.org/43935 | 20:05 |
| *** HenryG has quit IRC | 20:10 | |
| *** thomasbiege has joined #openstack-qa | 20:27 | |
| *** mberwanger has joined #openstack-qa | 20:38 | |
| *** mberwanger has quit IRC | 20:51 | |
| *** mberwanger has joined #openstack-qa | 20:53 | |
| *** openstackgerrit has quit IRC | 20:55 | |
| *** openstackgerrit has joined #openstack-qa | 20:56 | |
| *** mberwanger has quit IRC | 21:13 | |
| *** cyeoh_ has joined #openstack-qa | 21:18 | |
| *** cyeoh has quit IRC | 21:19 | |
| *** jasondotstar has quit IRC | 21:25 | |
| *** HenryG has joined #openstack-qa | 21:31 | |
| *** thomasbiege has quit IRC | 21:37 | |
| *** thomasbiege has joined #openstack-qa | 21:37 | |
| *** thomasbiege has quit IRC | 21:37 | |
| *** dkranz has quit IRC | 21:42 | |
| *** mriedem has quit IRC | 21:54 | |
| *** weshay has quit IRC | 21:56 | |
| *** burt has quit IRC | 22:03 | |
| *** mlavalle has quit IRC | 22:04 | |
| *** SergeyLukjanov has quit IRC | 22:05 | |
| openstackgerrit | SlickNik proposed a change to openstack-dev/devstack: Added Trove (Database as a Service). https://review.openstack.org/38169 | 22:11 |
| *** giulivo has quit IRC | 22:16 | |
| openstackgerrit | Kui Shi proposed a change to openstack/tempest: emit warning while running flake8 without virtual env https://review.openstack.org/43367 | 22:35 |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Fix is_package_installed() check with dpkg https://review.openstack.org/43966 | 22:50 |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Add support for Docker as Nova hypervisor https://review.openstack.org/40759 | 23:16 |
| *** anteaya has quit IRC | 23:21 | |
| *** dkranz has joined #openstack-qa | 23:44 | |
| *** rgrober has joined #openstack-qa | 23:51 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
