| *** yangyapeng has quit IRC | 00:01 | |
| *** ihrachys has quit IRC | 00:04 | |
| mtreinish | mriedem: yeah, what gmann said. It's only being used in api right now, but nothing about scenario should be different | 00:05 |
|---|---|---|
| *** Gen has joined #openstack-qa | 00:05 | |
| *** Swami has quit IRC | 00:05 | |
| mriedem | mtreinish: gmann: ok, about the network thingy, is there any way in tempest to set it up so the tenant doesn't have a network available to it already? | 00:09 |
| mtreinish | mriedem: yes, there is a variable you set to an empty dict or something in one of the setup class functions | 00:09 |
| mtreinish | let me dig up an example | 00:09 |
| mtreinish | (all the network tests do it | 00:09 |
| mtreinish | mriedem: https://github.com/openstack/tempest/blob/master/tempest/api/network/base.py#L62-L66 | 00:12 |
| mtreinish | do that | 00:12 |
| mriedem | sweet | 00:12 |
| mriedem | what do you think about where this should live? api or scenario? | 00:13 |
| mriedem | the test is really just booting a server at the given microversion when the tenant doesn't have a network | 00:13 |
| mriedem | and verify that it works and the tenant then has a network auto allocated for it | 00:13 |
| mtreinish | mriedem: that sounds like an api test | 00:13 |
| mriedem | it is testing an orchestration thing between multiple services | 00:13 |
| mtreinish | mriedem: scenario tests are normally supposed to simulate an end user workflow (which I guess booting a server is) | 00:14 |
| mtreinish | but this feels more like just testing an api works | 00:14 |
| mriedem | alright, i can buy that | 00:14 |
| mriedem | i very much look forward to this exciting adventure | 00:15 |
| *** Gen has quit IRC | 00:16 | |
| openstackrecheck | Console logs not available after 13:28s for gate-tempest-dsvm-postgres-full 281546,6,00d7e3c | 00:21 |
| *** ihrachys has joined #openstack-qa | 00:22 | |
| *** Suyi has quit IRC | 00:23 | |
| *** Gen has joined #openstack-qa | 00:27 | |
| *** mtanino has quit IRC | 00:27 | |
| *** david-lyle has quit IRC | 00:28 | |
| *** david-lyle has joined #openstack-qa | 00:28 | |
| *** amotoki has joined #openstack-qa | 00:31 | |
| *** david-lyle has quit IRC | 00:33 | |
| *** ihrachys has quit IRC | 00:33 | |
| openstackrecheck | Console logs not available after 13:23s for gate-grenade-dsvm-multinode 326756,1,af6a743 | 00:34 |
| *** amotoki has quit IRC | 00:36 | |
| *** anteaya has quit IRC | 00:36 | |
| *** ihrachys has joined #openstack-qa | 00:39 | |
| *** pvaneck has quit IRC | 00:40 | |
| *** hoangcx has joined #openstack-qa | 00:45 | |
| openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: DNM: experimenting with local.conf https://review.openstack.org/326585 | 00:51 |
| *** hoangcx has left #openstack-qa | 00:51 | |
| *** salv-orlando has quit IRC | 00:53 | |
| slowrie | mtreinish: Take a look @ https://review.openstack.org/#/c/283770/30/tempest/cmd/init.py line 76. Your upstream patch didn't fix the error that crashed out tempest init. Can you put that into your patch and I'll pull it out of mine? | 00:53 |
| *** salv-orlando has joined #openstack-qa | 00:53 | |
| mtreinish | slowrie: we can't do that. It's making an assumption about the value of sys.prefix having write perms | 00:54 |
| mtreinish | I'll take a look at the failure in a bit and see if I can come up with something else | 00:54 |
| slowrie | Ok | 00:54 |
| slowrie | Wouldn't you always have perms if you run into that case though? | 00:54 |
| *** nwkarsten has joined #openstack-qa | 00:55 | |
| *** nwkarste_ has joined #openstack-qa | 00:56 | |
| mtreinish | not necessarily, there are a lot of weird edge conditions there | 00:56 |
| slowrie | gotcha. I'm going to pull that back out then, just put it up there to test it upstream | 00:57 |
| openstackgerrit | Stephen Lowrie proposed openstack/tempest: Centralized Workspaces https://review.openstack.org/283770 | 00:57 |
| *** nwkarsten has quit IRC | 00:59 | |
| *** ihrachys has quit IRC | 01:01 | |
| *** ihrachys has joined #openstack-qa | 01:04 | |
| *** ihrachys has quit IRC | 01:09 | |
| masayukig | mtreinish: I left a comment about py34 on https://review.openstack.org/#/c/284411/ . I'm not sure this is caused by tempest-run or os-testr.. | 01:13 |
| *** markvoelker has joined #openstack-qa | 01:13 | |
| *** yangyapeng has joined #openstack-qa | 01:14 | |
| mtreinish | hmm, yeah I tried it on py35 after I saw your comment and got the same thing | 01:14 |
| mtreinish | I expect it's related to the str vs unicode py3 changes | 01:14 |
| masayukig | yeah, agree | 01:14 |
| mtreinish | but it's really annoying that there aren't any stack traces on failure with tempest run. We need to fix that before we land it | 01:14 |
| mtreinish | if we had a stack trace we could see exactly where it's coming from | 01:14 |
| masayukig | yup, agree too | 01:15 |
| *** tqtran has quit IRC | 01:16 | |
| *** zz_dimtruck is now known as dimtruck | 01:16 | |
| masayukig | and I found that tempest 'run' conflicts in the interactive mode command.. | 01:17 |
| masayukig | I'm not sure we should care about it or not. | 01:17 |
| *** markvoelker has quit IRC | 01:18 | |
| mtreinish | masayukig: interactive mode? | 01:19 |
| masayukig | mtreinish: yeah, just type 'tempest' | 01:19 |
| mtreinish | oh, wow I didn't realize it did that | 01:20 |
| mtreinish | hmm, I wonder what I did that broke that | 01:20 |
| masayukig | heh, I suppose it's cliff's feature | 01:21 |
| masayukig | and I think 'help' command always conflicts | 01:21 |
| *** Apoorva has quit IRC | 01:23 | |
| *** jbernard1 has joined #openstack-qa | 01:23 | |
| jbernard1 | jbernard: test | 01:23 |
| masayukig | I think the interactive mode should provide a way to call application commands directly. It's not tempest stuff, though. | 01:24 |
| *** jbernard1 has quit IRC | 01:24 | |
| *** ramishra has joined #openstack-qa | 01:26 | |
| mtreinish | masayukig: hmm, yeah I don't know. TBH, I'm not too familiar with cliff | 01:28 |
| *** frontrunner has quit IRC | 01:31 | |
| *** amotoki has joined #openstack-qa | 01:32 | |
| *** EinstCrazy has joined #openstack-qa | 01:35 | |
| *** amotoki has quit IRC | 01:36 | |
| *** gouthamr has quit IRC | 01:40 | |
| *** EinstCra_ has joined #openstack-qa | 01:41 | |
| *** EinstCr__ has joined #openstack-qa | 01:44 | |
| *** EinstCrazy has quit IRC | 01:44 | |
| *** EinstCra_ has quit IRC | 01:46 | |
| *** mriedem has quit IRC | 01:52 | |
| *** yamahata has quit IRC | 01:53 | |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Centralized Workspaces https://review.openstack.org/283770 | 01:53 |
| mtreinish | slowrie: ^^^ I broke the working tests, but I'm going for better isolation | 01:53 |
| mtreinish | I'm also gonna respin my init fix patch to skip the sample config generation if it can't find the oslo.config sample generator config file | 01:54 |
| slowrie | The new test_absolute_path doesn't really test the regression that broke though | 01:55 |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Make missing global config dir not fatal in tempest init https://review.openstack.org/326725 | 01:56 |
| mtreinish | slowrie: oh, what did I change? I thought I just moved the subprocess call to the helper function | 01:56 |
| slowrie | The issue that was found was to do with relative paths not being expanded, the new version is just changing where it's looking for the .yaml | 01:56 |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Centralized Workspaces https://review.openstack.org/283770 | 01:56 |
| slowrie | line 95 | 01:56 |
| mtreinish | slowrie: ah, ok. I misunderstood the bug | 01:57 |
| mtreinish | we can change it back | 01:57 |
| *** r-mibu has quit IRC | 01:57 | |
| *** r-mibu has joined #openstack-qa | 01:57 | |
| mtreinish | I was just staring at the tests and thinking these are totally system dependent and didn't like that | 01:57 |
| mtreinish | but if we're specifically testing that condition that's ok (we should have a comment about that though | 01:57 |
| slowrie | Yeah, I'll respin the test to just use register in a relative path | 01:57 |
| slowrie | I was originally just attempting to replicate it exactly like the bug was found though which is why I used tempest init instead of just registering it via a rel path | 01:58 |
| slowrie | lines 38-43 (of _run_cmd_gets_return_code) are also duplicated to the rest of the function and the values are thrown away. I'll take those out when I fix that when I come back from dinner | 02:01 |
| *** akshai has quit IRC | 02:01 | |
| mtreinish | slowrie: oops, sloppy copy and paste | 02:04 |
| slowrie | mtreinish: Going to move the test to the other class now that I'm going to be using just the WorkspaceManager | 02:07 |
| mtreinish | ok, makes sense | 02:07 |
| *** cody-somerville has joined #openstack-qa | 02:10 | |
| *** akshai has joined #openstack-qa | 02:12 | |
| openstackgerrit | Stephen Lowrie proposed openstack/tempest: Centralized Workspaces https://review.openstack.org/283770 | 02:21 |
| *** Suyi has joined #openstack-qa | 02:22 | |
| *** lutostag has joined #openstack-qa | 02:24 | |
| *** cody-somerville has quit IRC | 02:28 | |
| *** yamamoto_ has joined #openstack-qa | 02:29 | |
| *** amotoki has joined #openstack-qa | 02:32 | |
| *** ramishra_ has joined #openstack-qa | 02:33 | |
| *** mtanino has joined #openstack-qa | 02:34 | |
| *** ramishra has quit IRC | 02:36 | |
| *** amotoki has quit IRC | 02:37 | |
| *** woodster_ has quit IRC | 02:38 | |
| openstackgerrit | Merged openstack/tempest: Cleanup wording and section for remote access config doc https://review.openstack.org/326628 | 02:41 |
| *** lutostag has quit IRC | 02:46 | |
| *** openstackgerrit has quit IRC | 02:47 | |
| *** openstackgerrit has joined #openstack-qa | 02:48 | |
| *** ihrachys has joined #openstack-qa | 02:53 | |
| *** dave-mccowan has quit IRC | 02:54 | |
| *** ihrachys has quit IRC | 02:59 | |
| *** rossella_s has quit IRC | 03:04 | |
| *** mtanino has quit IRC | 03:08 | |
| *** EinstCr__ has quit IRC | 03:13 | |
| *** fnaval_ has joined #openstack-qa | 03:23 | |
| *** fnaval has quit IRC | 03:25 | |
| *** Gen has quit IRC | 03:26 | |
| *** jrcloud has joined #openstack-qa | 03:28 | |
| *** dimtruck is now known as zz_dimtruck | 03:28 | |
| *** zackf has quit IRC | 03:29 | |
| *** amotoki has joined #openstack-qa | 03:34 | |
| openstackrecheck | Console logs not available after 13:24s for gate-nova-python34-db 325684,3,400fdec | 03:34 |
| *** amotoki has quit IRC | 03:38 | |
| *** binbincong has joined #openstack-qa | 03:40 | |
| *** EinstCrazy has joined #openstack-qa | 03:44 | |
| *** tochi has quit IRC | 03:46 | |
| *** EinstCrazy has quit IRC | 03:47 | |
| *** tochi has joined #openstack-qa | 03:50 | |
| *** EinstCrazy has joined #openstack-qa | 04:22 | |
| *** tochi has quit IRC | 04:24 | |
| *** salv-orl_ has joined #openstack-qa | 04:24 | |
| *** salv-orlando has quit IRC | 04:27 | |
| *** fnaval_ has quit IRC | 04:30 | |
| *** amotoki has joined #openstack-qa | 04:34 | |
| *** edtubill has joined #openstack-qa | 04:38 | |
| *** amotoki has quit IRC | 04:39 | |
| *** Alex_Stef has joined #openstack-qa | 04:46 | |
| *** sshnaidm|afk has quit IRC | 04:51 | |
| *** fnaval has joined #openstack-qa | 04:53 | |
| *** yamahata has joined #openstack-qa | 04:54 | |
| *** tochi has joined #openstack-qa | 04:54 | |
| openstackrecheck | Console logs not available after 13:37s for gate-tempest-dsvm-neutron-full 282169,2,0ec3f89 | 04:58 |
| *** salv-orl_ has quit IRC | 05:03 | |
| *** salv-orlando has joined #openstack-qa | 05:03 | |
| *** gmann has quit IRC | 05:10 | |
| *** gmann has joined #openstack-qa | 05:12 | |
| *** edtubill has quit IRC | 05:15 | |
| *** nwkarste_ has quit IRC | 05:15 | |
| *** itzikb has joined #openstack-qa | 05:16 | |
| *** Poornima has joined #openstack-qa | 05:17 | |
| *** tfreger has joined #openstack-qa | 05:17 | |
| *** gvrangan has joined #openstack-qa | 05:27 | |
| *** fawadkhaliq has joined #openstack-qa | 05:33 | |
| *** amotoki has joined #openstack-qa | 05:35 | |
| *** amotoki has quit IRC | 05:39 | |
| *** Suyi has quit IRC | 05:48 | |
| *** pgadiya has joined #openstack-qa | 05:48 | |
| *** mtanino has joined #openstack-qa | 05:56 | |
| *** binbincong has quit IRC | 05:58 | |
| *** EinstCrazy has quit IRC | 05:59 | |
| *** EinstCrazy has joined #openstack-qa | 05:59 | |
| *** EinstCrazy has quit IRC | 05:59 | |
| *** yangyape_ has joined #openstack-qa | 06:01 | |
| *** yangyapeng has quit IRC | 06:02 | |
| *** EinstCrazy has joined #openstack-qa | 06:05 | |
| *** sridharg has joined #openstack-qa | 06:06 | |
| *** lifeless has quit IRC | 06:13 | |
| *** rcernin has joined #openstack-qa | 06:13 | |
| openstackgerrit | watanabe isao proposed openstack-dev/devstack: Don't configure ML2_L3_PLUGIN if q-l3 is disabled https://review.openstack.org/326844 | 06:19 |
| *** e0ne has joined #openstack-qa | 06:21 | |
| *** nadya has joined #openstack-qa | 06:22 | |
| *** Alex_Stef has quit IRC | 06:23 | |
| openstackgerrit | Benny Kopilov proposed openstack-dev/devstack: Add a multibackend list to tempest.conf https://review.openstack.org/325895 | 06:28 |
| openstackrecheck | Console logs not available after 13:23s for gate-tempest-dsvm-full-dibtest 326788,1,4c7a3c4 | 06:29 |
| *** nadya has quit IRC | 06:41 | |
| *** nadya has joined #openstack-qa | 06:45 | |
| *** openstackgerrit has quit IRC | 06:48 | |
| *** ajmiller_ has quit IRC | 06:48 | |
| *** openstackgerrit has joined #openstack-qa | 06:48 | |
| *** ajmiller_ has joined #openstack-qa | 06:48 | |
| *** jpena|off is now known as jpena | 06:50 | |
| *** nadya has quit IRC | 06:53 | |
| openstackgerrit | Christian Schwede proposed openstack-infra/devstack-gate: Use subnodes for Swift storage nodes in a multinode setup https://review.openstack.org/304465 | 06:55 |
| *** EinstCra_ has joined #openstack-qa | 06:59 | |
| *** tesseract has joined #openstack-qa | 07:00 | |
| *** EinstCrazy has quit IRC | 07:02 | |
| *** salv-orlando has quit IRC | 07:02 | |
| *** ajmiller_ has quit IRC | 07:03 | |
| *** ajmiller has joined #openstack-qa | 07:03 | |
| *** jecarey has joined #openstack-qa | 07:03 | |
| *** salv-orlando has joined #openstack-qa | 07:03 | |
| *** EinstCra_ has quit IRC | 07:05 | |
| *** EinstCrazy has joined #openstack-qa | 07:05 | |
| *** e0ne has quit IRC | 07:06 | |
| *** jecarey has quit IRC | 07:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack-dev/devstack: Updated from generate-devstack-plugins-list https://review.openstack.org/326868 | 07:11 |
| *** zeih has joined #openstack-qa | 07:15 | |
| *** gamado has joined #openstack-qa | 07:18 | |
| *** mtanino has quit IRC | 07:19 | |
| *** ifarkas has joined #openstack-qa | 07:20 | |
| openstackgerrit | Benny Kopilov proposed openstack/tempest: Fix cinder volume nameing for admin https://review.openstack.org/326875 | 07:21 |
| *** yfried has joined #openstack-qa | 07:21 | |
| *** fawadkhaliq has quit IRC | 07:23 | |
| *** jlanoux has joined #openstack-qa | 07:29 | |
| *** salv-orlando has quit IRC | 07:32 | |
| *** salv-orlando has joined #openstack-qa | 07:32 | |
| *** ajmiller has quit IRC | 07:35 | |
| *** anshul_behl has joined #openstack-qa | 07:35 | |
| *** amotoki has joined #openstack-qa | 07:36 | |
| *** Liuqing has joined #openstack-qa | 07:37 | |
| *** shardy has joined #openstack-qa | 07:39 | |
| openstackrecheck | Console logs not available after 13:35s for gate-horizon-npm-run-test 304999,6,ad2aa13 | 07:40 |
| *** amotoki has quit IRC | 07:41 | |
| *** gvrangan has quit IRC | 07:43 | |
| *** dtantsur|afk is now known as dtantsur | 07:44 | |
| *** rossella_s has joined #openstack-qa | 07:44 | |
| *** openstackgerrit has quit IRC | 07:48 | |
| *** openstackgerrit has joined #openstack-qa | 07:48 | |
| *** apevec has joined #openstack-qa | 07:48 | |
| *** apevec has quit IRC | 07:48 | |
| *** apevec has joined #openstack-qa | 07:48 | |
| *** jlanoux_ has joined #openstack-qa | 07:50 | |
| *** Liuqing has quit IRC | 07:52 | |
| *** jlanoux has quit IRC | 07:54 | |
| *** Alex_Stef has joined #openstack-qa | 08:00 | |
| *** ajmiller has joined #openstack-qa | 08:05 | |
| *** jrcloud has quit IRC | 08:05 | |
| *** cdent has joined #openstack-qa | 08:05 | |
| *** fawadkhaliq has joined #openstack-qa | 08:09 | |
| *** salv-orlando has quit IRC | 08:10 | |
| *** ajmiller_ has joined #openstack-qa | 08:10 | |
| *** lucas-dinner is now known as lucasagomes | 08:10 | |
| *** salv-orlando has joined #openstack-qa | 08:11 | |
| *** ajmiller has quit IRC | 08:14 | |
| *** yangyape_ has quit IRC | 08:15 | |
| *** EinstCrazy has quit IRC | 08:15 | |
| *** yamahata has quit IRC | 08:16 | |
| *** EinstCrazy has joined #openstack-qa | 08:18 | |
| *** yangyapeng has joined #openstack-qa | 08:19 | |
| masayukig | mtreinish: I found out a secret word to get stack traces 'TESTR_PDB' - http://paste.openstack.org/show/508860/ | 08:25 |
| *** aarefiev_ is now known as aarefiev | 08:26 | |
| masayukig | And I noticed we need to check the return value of testrepository.commands.run_argv(). It doesn't raise an exception.. https://github.com/testing-cabal/testrepository/blob/master/testrepository/commands/__init__.py#L191 | 08:28 |
| *** apevec has left #openstack-qa | 08:29 | |
| openstackrecheck | Console logs not available after 13:24s for gate-grenade-dsvm-multinode 326874,1,f848250 | 08:32 |
| *** yfried has quit IRC | 08:35 | |
| bkopilov | afazekas, please review https://review.openstack.org/#/c/325895 | 08:35 |
| *** guardianJ has joined #openstack-qa | 08:37 | |
| *** amotoki has joined #openstack-qa | 08:37 | |
| gmann | masayukig: can you have look into these -https://review.openstack.org/#/c/326205/ https://review.openstack.org/#/c/294364/ | 08:38 |
| *** guardianJ has left #openstack-qa | 08:38 | |
| masayukig | gmann: looking :) | 08:39 |
| gmann | masayukig: Thanks | 08:39 |
| masayukig | gmann: one question about https://review.openstack.org/#/c/294364, why should we separate this? for consistency? | 08:41 |
| gmann | masayukig: yea, each service client should be separate as per resource | 08:41 |
| *** amotoki has quit IRC | 08:42 | |
| masayukig | gmann: sure, make sense | 08:42 |
| *** Serlex has joined #openstack-qa | 08:42 | |
| *** EinstCrazy has quit IRC | 08:43 | |
| *** EinstCrazy has joined #openstack-qa | 08:43 | |
| *** EinstCrazy has quit IRC | 08:45 | |
| *** fawadkhaliq has quit IRC | 08:45 | |
| openstackrecheck | Console logs not available after 13:21s for gate-nova-pep8 326326,3,689cc0c | 08:45 |
| openstackgerrit | Genadi Chereshnya proposed openstack/tempest: Remove creation of egress rules for _create_loginable_secgroup https://review.openstack.org/316429 | 08:47 |
| *** EinstCrazy has joined #openstack-qa | 08:47 | |
| *** openstackgerrit has quit IRC | 08:48 | |
| *** openstackgerrit has joined #openstack-qa | 08:48 | |
| *** yfried has joined #openstack-qa | 08:49 | |
| *** e0ne has joined #openstack-qa | 08:50 | |
| *** kzaitsev_mb has joined #openstack-qa | 08:53 | |
| masayukig | gmann: I have one question https://review.openstack.org/#/c/326205, Why do you make the variable name different from v2's one? | 08:54 |
| masayukig | v2: cls.os_image_member_client , v1:cls.image_member_client? | 08:54 |
| masayukig | I don't have strong opinion here, just wondering. | 08:54 |
| gmann | masayukig: let me check | 08:54 |
| masayukig | https://review.openstack.org/#/c/326205/4/tempest/api/image/base.py | 08:55 |
| afazekas | bkopilov: 'be' is also local | 08:55 |
| bkopilov | afazekas, ok thanks | 08:55 |
| gmann | masayukig: yea, actually those should without 'os', i can update that if you want | 08:56 |
| masayukig | oops, I already +A for https://review.openstack.org/#/c/294364/ | 08:56 |
| *** dtantsur is now known as dtantsur|bbl | 08:57 | |
| openstackgerrit | Steve Heyman proposed openstack/tempest: Remove neutron OO wrappers https://review.openstack.org/320495 | 08:57 |
| gmann | masayukig: no prob, i can quickly submit follow up for that | 08:58 |
| openstackrecheck | Console logs not available after 13:21s for gate-tempest-dsvm-full 326875,1,1dfde3f | 08:59 |
| openstackgerrit | Liron Kuchlani proposed openstack/tempest: Backup create using force flag https://review.openstack.org/326507 | 08:59 |
| masayukig | gmann, cool, thanks | 09:00 |
| openstackgerrit | Liron Kuchlani proposed openstack/tempest: Backup create using force flag https://review.openstack.org/326507 | 09:01 |
| *** tosky has joined #openstack-qa | 09:02 | |
| *** tosky has quit IRC | 09:02 | |
| *** mvk has quit IRC | 09:03 | |
| *** gamado has quit IRC | 09:04 | |
| *** gamado has joined #openstack-qa | 09:05 | |
| *** tosky has joined #openstack-qa | 09:05 | |
| openstackgerrit | Genadi Chereshnya proposed openstack/tempest: Remove creation of egress rules for _create_loginable_secgroup https://review.openstack.org/316429 | 09:09 |
| openstackgerrit | Benny Kopilov proposed openstack-dev/devstack: Add a multibackend list to tempest.conf https://review.openstack.org/325895 | 09:10 |
| *** anshul_behl has quit IRC | 09:12 | |
| *** EinstCrazy has quit IRC | 09:18 | |
| openstackrecheck | Console logs not available after 13:21s for gate-horizon-pep8 326244,2,48ac4da | 09:20 |
| *** EinstCrazy has joined #openstack-qa | 09:21 | |
| *** ajmiller__ has joined #openstack-qa | 09:22 | |
| *** anshul has joined #openstack-qa | 09:24 | |
| *** anshul is now known as Guest8385 | 09:24 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Fix the v2 image client variable name https://review.openstack.org/326948 | 09:25 |
| gmann | masayukig: ^^ | 09:25 |
| *** ajmiller_ has quit IRC | 09:26 | |
| *** nadya has joined #openstack-qa | 09:27 | |
| masayukig | thanks, looking | 09:29 |
| *** fawadkhaliq has joined #openstack-qa | 09:32 | |
| *** fawadkhaliq has quit IRC | 09:33 | |
| *** fawadkhaliq has joined #openstack-qa | 09:33 | |
| *** fawadkhaliq has quit IRC | 09:33 | |
| openstackgerrit | watanabe isao proposed openstack-dev/devstack: Don't configure ML2_L3_PLUGIN if q-l3 is disabled https://review.openstack.org/326844 | 09:34 |
| *** mvk has joined #openstack-qa | 09:35 | |
| *** nadya_ has joined #openstack-qa | 09:35 | |
| *** nadya has quit IRC | 09:36 | |
| *** amotoki has joined #openstack-qa | 09:38 | |
| *** sambetts|afk is now known as sambetts | 09:41 | |
| *** fawadkhaliq has joined #openstack-qa | 09:42 | |
| *** amotoki has quit IRC | 09:42 | |
| *** kzaitsev_mb has quit IRC | 09:43 | |
| openstackrecheck | Console logs not available after 13:27s for gate-horizon-dsvm-integration 144153,8,f3604a2 | 09:50 |
| openstackgerrit | Merged openstack/tempest: Separate Image v2 members client https://review.openstack.org/294364 | 09:52 |
| *** markvoelker has joined #openstack-qa | 09:53 | |
| *** amotoki has joined #openstack-qa | 09:56 | |
| *** rossella_s has quit IRC | 10:02 | |
| *** sridharg has quit IRC | 10:03 | |
| openstackrecheck | Console logs not available after 13:22s for gate-horizon-npm-run-test 144152,15,dc9295b | 10:04 |
| *** nadya_ has quit IRC | 10:05 | |
| *** yangyapeng has quit IRC | 10:09 | |
| *** dtantsur|bbl is now known as dtantsur | 10:11 | |
| *** jogo has quit IRC | 10:17 | |
| *** zackf has joined #openstack-qa | 10:17 | |
| openstackrecheck | Console logs not available after 13:29s for gate-horizon-python34 272644,7,ff3ee55 | 10:17 |
| *** yfried has quit IRC | 10:21 | |
| *** zackf has quit IRC | 10:21 | |
| andreaf | mtreinish: one for you: https://review.openstack.org/326976 - there was another leftover from legacy credentials to be removed | 10:22 |
| *** salv-orl_ has joined #openstack-qa | 10:25 | |
| *** salv-orlando has quit IRC | 10:28 | |
| *** salv-orl_ has quit IRC | 10:34 | |
| *** salv-orlando has joined #openstack-qa | 10:34 | |
| *** lifeless has joined #openstack-qa | 10:35 | |
| *** gszasz has joined #openstack-qa | 10:35 | |
| *** Guest8385 has quit IRC | 10:36 | |
| *** weshay has joined #openstack-qa | 10:37 | |
| *** jogo has joined #openstack-qa | 10:38 | |
| *** nadya has joined #openstack-qa | 10:41 | |
| openstackgerrit | Andreas Scheuring proposed openstack-dev/devstack: Docs for devstack MacVTap agent setup https://review.openstack.org/292778 | 10:41 |
| *** nadya_ has joined #openstack-qa | 10:42 | |
| *** sdague has joined #openstack-qa | 10:42 | |
| *** nadya has quit IRC | 10:42 | |
| *** yamamoto_ has quit IRC | 10:43 | |
| *** gamado has quit IRC | 10:47 | |
| *** Guest8385 has joined #openstack-qa | 10:47 | |
| *** lifeless has quit IRC | 10:47 | |
| *** lifeless_ has joined #openstack-qa | 10:47 | |
| *** kzaitsev_mb has joined #openstack-qa | 10:48 | |
| openstackrecheck | Console logs not available after 13:29s for gate-tempest-pep8 320495,12,2d1331a | 10:48 |
| *** yfried has joined #openstack-qa | 10:49 | |
| *** tesseract has quit IRC | 10:50 | |
| *** nwkarsten has joined #openstack-qa | 10:50 | |
| *** nadya_ has quit IRC | 10:50 | |
| *** nadya has joined #openstack-qa | 10:51 | |
| *** tesseract has joined #openstack-qa | 10:51 | |
| *** nwkarsten has quit IRC | 10:54 | |
| *** nadya has quit IRC | 10:57 | |
| openstackgerrit | Sean Dague proposed openstack-dev/grenade: DNM: local.conf support https://review.openstack.org/326993 | 10:58 |
| *** rossella_s has joined #openstack-qa | 10:58 | |
| openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: DNM: experimenting with local.conf https://review.openstack.org/326585 | 10:59 |
| *** nadya has joined #openstack-qa | 11:00 | |
| *** gamado has joined #openstack-qa | 11:01 | |
| openstackrecheck | Console logs not available after 13:31s for gate-tempest-dsvm-neutron-full 270286,11,5db5bfc | 11:02 |
| *** ihrachys has joined #openstack-qa | 11:02 | |
| *** nadya has quit IRC | 11:04 | |
| openstackgerrit | Steve Heyman proposed openstack/tempest: Remove neutron OO wrappers https://review.openstack.org/320495 | 11:04 |
| *** afazekas is now known as __afazekas | 11:05 | |
| *** weshay has quit IRC | 11:05 | |
| *** e0ne_ has joined #openstack-qa | 11:11 | |
| *** weshay has joined #openstack-qa | 11:11 | |
| andreaf | gmann, oomichi: I guess your plan regarding the image client is to first split and then move to tempest.lib, right? Any special reason for not doing that in one step? | 11:12 |
| *** fawadkhaliq has quit IRC | 11:12 | |
| *** e0ne has quit IRC | 11:13 | |
| openstackgerrit | Benny Kopilov proposed openstack/tempest: Fix cinder volume nameing for admin https://review.openstack.org/326875 | 11:15 |
| openstackrecheck | Console logs not available after 13:26s for gate-tempest-dsvm-full 326875,2,6e33cfb | 11:15 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Define _set_image_clients https://review.openstack.org/326671 | 11:17 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Define _set_network_clients https://review.openstack.org/326672 | 11:17 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Prepare the Manager class for tempest.lib https://review.openstack.org/326683 | 11:18 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Factor up (most) CONF value in clients.Manager https://review.openstack.org/326667 | 11:18 |
| andreaf | gmann, oomichi: also you may want to look at this https://review.openstack.org/326671 - I don't mind rebasing after the image client split is done - but I think it would be nice to have | 11:19 |
| *** fawadkhaliq has joined #openstack-qa | 11:19 | |
| *** gszasz has quit IRC | 11:20 | |
| *** gszasz has joined #openstack-qa | 11:21 | |
| *** gszasz has quit IRC | 11:21 | |
| *** gszasz has joined #openstack-qa | 11:21 | |
| *** gszasz has quit IRC | 11:21 | |
| *** gszasz has joined #openstack-qa | 11:22 | |
| *** fawadkhaliq has quit IRC | 11:25 | |
| *** tochi has quit IRC | 11:26 | |
| *** moravec has quit IRC | 11:27 | |
| *** nadya has joined #openstack-qa | 11:28 | |
| openstackrecheck | Console logs not available after 13:28s for gate-keystone-python27-db 326918,9,aa55bde | 11:29 |
| *** kzaitsev_mb has quit IRC | 11:32 | |
| *** yamamoto has joined #openstack-qa | 11:33 | |
| *** yamamoto_ has joined #openstack-qa | 11:34 | |
| *** lucasagomes is now known as lucas-afk | 11:35 | |
| *** kiran-r has joined #openstack-qa | 11:36 | |
| *** fawadkhaliq has joined #openstack-qa | 11:38 | |
| *** yamamoto has quit IRC | 11:38 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 11:38 |
| andreaf | oomichi: I hope I addressed your comments in ^^^ | 11:40 |
| *** fawadkhaliq has quit IRC | 11:40 | |
| andreaf | __afazekas: small change to https://review.openstack.org/221020 if you can re-review | 11:40 |
| *** fawadkhaliq has joined #openstack-qa | 11:40 | |
| *** jpena is now known as jpena|lunch | 11:40 | |
| *** moravec has joined #openstack-qa | 11:42 | |
| openstackrecheck | Console logs not available after 13:25s for gate-grenade-dsvm-neutron 326948,1,52a3827 | 11:42 |
| *** fawadk has joined #openstack-qa | 11:43 | |
| *** fawadkhaliq has quit IRC | 11:44 | |
| *** fawadkhaliq has joined #openstack-qa | 11:44 | |
| *** nadya_ has joined #openstack-qa | 11:44 | |
| *** kzaitsev_mb has joined #openstack-qa | 11:44 | |
| *** nadya has quit IRC | 11:46 | |
| *** fawadk has quit IRC | 11:48 | |
| *** alyson_ has joined #openstack-qa | 11:48 | |
| *** yolanda has joined #openstack-qa | 11:49 | |
| *** __afazekas is now known as afazekas | 11:49 | |
| *** markvoelker has quit IRC | 11:56 | |
| openstackrecheck | Console logs not available after 13:37s for gate-nova-python27-db 326940,1,142c7cb | 11:56 |
| *** sridharg has joined #openstack-qa | 11:56 | |
| *** ifarkas has quit IRC | 11:58 | |
| *** salv-orlando has quit IRC | 11:59 | |
| *** zeih has quit IRC | 11:59 | |
| *** moravec has quit IRC | 12:00 | |
| *** salv-orlando has joined #openstack-qa | 12:00 | |
| openstackgerrit | Scott DAngelo proposed openstack/tempest: Migrate volume while attached to an instance https://review.openstack.org/326681 | 12:00 |
| *** aysyd has joined #openstack-qa | 12:00 | |
| *** kzaitsev_mb has quit IRC | 12:01 | |
| *** tfreger has quit IRC | 12:02 | |
| *** kzaitsev_mb has joined #openstack-qa | 12:03 | |
| *** fawadkhaliq has quit IRC | 12:03 | |
| openstackgerrit | Sean Dague proposed openstack-dev/grenade: DNM: local.conf support https://review.openstack.org/326993 | 12:04 |
| *** ifarkas has joined #openstack-qa | 12:06 | |
| *** sridharg has quit IRC | 12:09 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Prepare the Manager class for tempest.lib https://review.openstack.org/326683 | 12:10 |
| openstackrecheck | Console logs not available after 13:33s for gate-horizon-npm-run-test 308139,7,6e2fad1 | 12:10 |
| *** ihrachys has quit IRC | 12:10 | |
| *** nadya_ has quit IRC | 12:10 | |
| *** ihrachys has joined #openstack-qa | 12:10 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Factor up (most) CONF value in clients.Manager https://review.openstack.org/326667 | 12:11 |
| *** markvoelker has joined #openstack-qa | 12:12 | |
| *** EinstCrazy has quit IRC | 12:14 | |
| *** moravec has joined #openstack-qa | 12:14 | |
| *** EinstCrazy has joined #openstack-qa | 12:14 | |
| *** nadya has joined #openstack-qa | 12:14 | |
| *** frontrunner has joined #openstack-qa | 12:14 | |
| *** salv-orl_ has joined #openstack-qa | 12:15 | |
| *** fnaval_ has joined #openstack-qa | 12:15 | |
| *** nadya_ has joined #openstack-qa | 12:17 | |
| *** fnaval has quit IRC | 12:19 | |
| *** EinstCrazy has quit IRC | 12:19 | |
| *** nadya has quit IRC | 12:19 | |
| *** nadya_ has quit IRC | 12:19 | |
| *** salv-orlando has quit IRC | 12:19 | |
| *** tfreger has joined #openstack-qa | 12:20 | |
| *** sshnaidm|afk has joined #openstack-qa | 12:21 | |
| *** fawadkhaliq has joined #openstack-qa | 12:22 | |
| openstackrecheck | Console logs not available after 13:25s for gate-horizon-npm-run-lint 317456,11,aa72029 | 12:23 |
| *** sridharg has joined #openstack-qa | 12:23 | |
| *** mriedem has joined #openstack-qa | 12:24 | |
| *** yamamoto_ has quit IRC | 12:26 | |
| *** fawadkhaliq has quit IRC | 12:26 | |
| *** zeih has joined #openstack-qa | 12:27 | |
| *** yamamoto has joined #openstack-qa | 12:28 | |
| *** yamamoto has quit IRC | 12:30 | |
| *** nadya has joined #openstack-qa | 12:30 | |
| *** zeih has quit IRC | 12:32 | |
| *** fawadkhaliq has joined #openstack-qa | 12:33 | |
| openstackgerrit | Merged openstack/tempest: Separate namespaces_client from v2 images_client https://review.openstack.org/326187 | 12:35 |
| *** annemccormick has joined #openstack-qa | 12:35 | |
| *** rossella_s has quit IRC | 12:36 | |
| *** tfreger has quit IRC | 12:36 | |
| *** annemccormick has quit IRC | 12:37 | |
| openstackrecheck | Console logs not available after 13:24s for gate-horizon-dsvm-integration 144153,9,46eaf9b | 12:37 |
| *** pgadiya has quit IRC | 12:40 | |
| *** cdent has quit IRC | 12:41 | |
| *** rossella_s has joined #openstack-qa | 12:42 | |
| *** lucas-afk is now known as lucasagomes | 12:44 | |
| *** kzaitsev_mb has quit IRC | 12:48 | |
| *** jpena|lunch is now known as jpena | 12:49 | |
| openstackrecheck | Console logs not available after 13:25s for gate-horizon-npm-run-test 144152,17,8dd537e | 12:50 |
| *** tfreger has joined #openstack-qa | 12:51 | |
| *** sridharg has quit IRC | 12:53 | |
| *** sridharg has joined #openstack-qa | 12:53 | |
| *** rodrigods has quit IRC | 12:56 | |
| *** rodrigods has joined #openstack-qa | 12:56 | |
| *** gmann_ has joined #openstack-qa | 12:58 | |
| *** edmondsw has joined #openstack-qa | 13:00 | |
| *** nadya has quit IRC | 13:01 | |
| *** gouthamr has joined #openstack-qa | 13:02 | |
| *** jecarey has joined #openstack-qa | 13:03 | |
| *** zeih has joined #openstack-qa | 13:03 | |
| openstackrecheck | Console logs not available after 13:27s for gate-cinder-pep8 326961,1,128988f | 13:04 |
| *** fawadkhaliq has quit IRC | 13:04 | |
| *** jecarey has quit IRC | 13:06 | |
| *** jecarey has joined #openstack-qa | 13:06 | |
| *** rlrossit has joined #openstack-qa | 13:08 | |
| *** otherwiseguy has joined #openstack-qa | 13:09 | |
| *** Poornima has quit IRC | 13:09 | |
| *** cdent has joined #openstack-qa | 13:10 | |
| *** jecarey has quit IRC | 13:11 | |
| *** kzaitsev_mb has joined #openstack-qa | 13:13 | |
| openstackgerrit | Merged openstack/tempest: Separate schemas_client from v2 images_client https://review.openstack.org/326506 | 13:13 |
| *** jecarey has joined #openstack-qa | 13:14 | |
| *** jecarey has quit IRC | 13:15 | |
| *** jecarey has joined #openstack-qa | 13:15 | |
| *** jecarey has quit IRC | 13:15 | |
| *** dustins has joined #openstack-qa | 13:16 | |
| *** jecarey has joined #openstack-qa | 13:16 | |
| openstackrecheck | Console logs not available after 13:30s for gate-nova-python34-db 324536,6,e1bbe36 | 13:17 |
| mriedem | mtreinish: oomichi: looking at a gate job, tempest.conf has max_microversion = latest under [compute] - does that means the latest microversion is sent for each compute api test unless the test itself overrides max_microversion to cap it? | 13:20 |
| *** porrua has joined #openstack-qa | 13:21 | |
| *** zz_dimtruck is now known as dimtruck | 13:22 | |
| *** dkranz has joined #openstack-qa | 13:23 | |
| *** gcheresh has joined #openstack-qa | 13:25 | |
| tosky | I'm trying to fix Sahara jobs to use tempest tests part of sahara-tests, but I'm not sure if https://review.openstack.org/#/c/326980/ is the right way, reviews appreciated! | 13:25 |
| openstackrecheck | Console logs not available after 13:26s for gate-nova-pep8 326874,3,4b425d4 | 13:31 |
| *** dimtruck is now known as zz_dimtruck | 13:34 | |
| *** zz_dimtruck is now known as dimtruck | 13:36 | |
| *** sridharg has quit IRC | 13:36 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 13:38 |
| andreaf | afazekas, oomichi: I managed to get it wrong the first time :( ^^^ fixed it now | 13:38 |
| afazekas | andreaf, what do you think about this ? https://review.openstack.org/#/c/316429/ | 13:41 |
| *** rbak has joined #openstack-qa | 13:42 | |
| *** edtubill has joined #openstack-qa | 13:43 | |
| *** kzaitsev_mb has quit IRC | 13:44 | |
| openstackrecheck | Console logs not available after 13:25s for gate-nova-pep8 327011,1,1f4d756 | 13:45 |
| *** nwkarsten has joined #openstack-qa | 13:45 | |
| *** nwkarsten has quit IRC | 13:46 | |
| *** kzaitsev_mb has joined #openstack-qa | 13:46 | |
| *** nwkarsten has joined #openstack-qa | 13:46 | |
| openstackgerrit | Matt Riedemann proposed openstack/tempest: Cleanup some small issues in the microversion testing doc https://review.openstack.org/327100 | 13:47 |
| *** jlanoux_ has quit IRC | 13:47 | |
| andreaf | afazekas: well, if we want to enforce that rules are already there I guess it makes sense. Re-creating them will hide an issue if the auto-creation stops working | 13:47 |
| andreaf | afazekas: I guess a good detailed comment would be nice to have there | 13:48 |
| *** asselin_ has joined #openstack-qa | 13:48 | |
| *** Gen has joined #openstack-qa | 13:48 | |
| *** fnaval_ has quit IRC | 13:49 | |
| afazekas | gcheresh, ^ | 13:50 |
| afazekas | andreaf: Yes, a little longer commit message is required. | 13:52 |
| andreaf | afazekas: heh, I think it would be nice to have a comment in the code | 13:52 |
| gcheresh | afazekas: So comment that those rules will be always created by default will be good enough? | 13:54 |
| afazekas | gcheresh: probably the doc string should have how the group will like like and the and and why | 13:57 |
| * afazekas is leaving to computer.. | 13:58 | |
| openstackrecheck | Console logs not available after 13:25s for gate-nova-pep8 327012,1,e50564e | 13:59 |
| *** Guest8385 has quit IRC | 14:00 | |
| *** Swami has joined #openstack-qa | 14:01 | |
| *** fnaval has joined #openstack-qa | 14:02 | |
| *** dimtruck is now known as zz_dimtruck | 14:03 | |
| *** gmann_ has quit IRC | 14:05 | |
| *** zz_dimtruck is now known as dimtruck | 14:05 | |
| *** Swami_ has joined #openstack-qa | 14:05 | |
| *** itzikb has quit IRC | 14:06 | |
| *** rossella_s has quit IRC | 14:07 | |
| *** cdent has quit IRC | 14:08 | |
| *** yolanda_ has joined #openstack-qa | 14:08 | |
| *** Swami has quit IRC | 14:09 | |
| *** rlrossit has quit IRC | 14:09 | |
| *** yolanda has quit IRC | 14:09 | |
| *** yolanda_ has quit IRC | 14:09 | |
| *** yolanda has joined #openstack-qa | 14:09 | |
| *** rlrossit has joined #openstack-qa | 14:12 | |
| *** zeih has quit IRC | 14:13 | |
| *** rossella_s has joined #openstack-qa | 14:13 | |
| openstackrecheck | Console logs not available after 13:42s for gate-cinder-python34-db 327043,1,9877bca | 14:13 |
| *** Swami__ has joined #openstack-qa | 14:14 | |
| *** dimtruck is now known as zz_dimtruck | 14:16 | |
| *** gcheresh has quit IRC | 14:16 | |
| *** Swami has joined #openstack-qa | 14:17 | |
| *** pglass has joined #openstack-qa | 14:17 | |
| *** vsaienko has joined #openstack-qa | 14:18 | |
| *** Swami_ has quit IRC | 14:18 | |
| *** Swami__ has quit IRC | 14:19 | |
| *** ramishra_ has quit IRC | 14:19 | |
| *** ramishra has joined #openstack-qa | 14:20 | |
| *** erlon has joined #openstack-qa | 14:21 | |
| *** dave-mccowan has joined #openstack-qa | 14:26 | |
| *** mriedem1 has joined #openstack-qa | 14:27 | |
| *** mriedem has quit IRC | 14:27 | |
| openstackrecheck | Console logs not available after 13:24s for gate-nova-python27-db 234659,25,529f4de | 14:27 |
| *** spzala has joined #openstack-qa | 14:28 | |
| *** yamahata has joined #openstack-qa | 14:31 | |
| *** bdperkin has quit IRC | 14:32 | |
| *** pglass has quit IRC | 14:32 | |
| mtreinish | mriedem: right max_microversion just tells tempest what things are allowed, it doesn't actually dictate what version is sent | 14:36 |
| mtreinish | mriedem: that's basically just the either determined by the tempest min_microversion option or the test case's attrs | 14:37 |
| *** zeih has joined #openstack-qa | 14:37 | |
| mtreinish | masayukig: cool, we should bake TESTR_PDB="" into the default env then (unless it's already defined) | 14:38 |
| mtreinish | I'll add that to the next rev | 14:38 |
| openstackrecheck | Console logs not available after 13:35s for gate-tempest-dsvm-postgres-full-liberty 221020,45,2eef1bb | 14:41 |
| *** pauloewerton has joined #openstack-qa | 14:42 | |
| *** yangyapeng has joined #openstack-qa | 14:43 | |
| *** Alex_Stef has quit IRC | 14:45 | |
| mriedem1 | mtreinish: in my test i want to validate early that the project doesn't have access to any networks - is that ok in setup_clients or should be done in resource_setup - or doesn't matter? | 14:46 |
| *** anteaya has joined #openstack-qa | 14:46 | |
| mriedem1 | i can't do it in skip_checks since that's too early | 14:46 |
| *** mriedem1 is now known as mriedem | 14:46 | |
| mriedem | resource_setup seems to make the most sense | 14:47 |
| *** mtanino has joined #openstack-qa | 14:49 | |
| *** zeih has quit IRC | 14:49 | |
| mtreinish | mriedem: yeah I would do it in resource_setup | 14:50 |
| openstackrecheck | Console logs not available after 13:30s for gate-grenade-dsvm-neutron-multinode 326585,6,cc89852 | 14:55 |
| *** lutostag has joined #openstack-qa | 14:55 | |
| *** salv-orl_ has quit IRC | 14:56 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Define _set_network_clients https://review.openstack.org/326672 | 14:56 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Prepare the Manager class for tempest.lib https://review.openstack.org/326683 | 14:56 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Define _set_image_clients https://review.openstack.org/326671 | 14:56 |
| *** salv-orlando has joined #openstack-qa | 14:57 | |
| *** nadya has joined #openstack-qa | 14:58 | |
| *** fawadkhaliq has joined #openstack-qa | 15:00 | |
| *** nadya_ has joined #openstack-qa | 15:01 | |
| *** zz_dimtruck is now known as dimtruck | 15:03 | |
| *** nadya has quit IRC | 15:03 | |
| *** cdent has joined #openstack-qa | 15:05 | |
| *** gvrangan has joined #openstack-qa | 15:05 | |
| *** frickler has quit IRC | 15:06 | |
| *** frickler has joined #openstack-qa | 15:06 | |
| *** pglass has joined #openstack-qa | 15:08 | |
| openstackrecheck | Console logs not available after 13:29s for gate-grenade-dsvm-neutron 326667,7,322b16f | 15:08 |
| *** e0ne_ is now known as e0ne | 15:11 | |
| *** nadya_ has quit IRC | 15:12 | |
| *** mriedem1 has joined #openstack-qa | 15:15 | |
| *** raissa_ is now known as raissa | 15:16 | |
| *** mriedem has quit IRC | 15:17 | |
| *** thingee has quit IRC | 15:18 | |
| *** david-lyle has joined #openstack-qa | 15:19 | |
| *** davidlenwell has quit IRC | 15:20 | |
| *** zackf has joined #openstack-qa | 15:21 | |
| *** akshai has quit IRC | 15:21 | |
| *** akshai has joined #openstack-qa | 15:21 | |
| openstackrecheck | Console logs not available after 13:29s for periodic-tempest-dsvm-stress-master 302332,6,eca4b9d | 15:22 |
| openstackgerrit | Mark T. Voelker proposed openstack/tempest: Add simple test for Neutron GET / https://review.openstack.org/311747 | 15:22 |
| *** woodster_ has joined #openstack-qa | 15:24 | |
| *** mriedem1 is now known as mriedem | 15:24 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Add unit test coverage for account generator https://review.openstack.org/327182 | 15:25 |
| andreaf | mtreinish, afazekas, gmann, oomichi: I wrote and locally ran unit tests for the test account generator tool, and forgot to add them to the commit - here they are ^^^ | 15:26 |
| *** bdperkin has joined #openstack-qa | 15:26 | |
| *** bdperkin has quit IRC | 15:26 | |
| *** bdperkin has joined #openstack-qa | 15:26 | |
| *** fnaval has quit IRC | 15:26 | |
| *** yamamoto has joined #openstack-qa | 15:28 | |
| *** ajmiller has joined #openstack-qa | 15:29 | |
| *** ajmiller__ has quit IRC | 15:31 | |
| *** fnaval has joined #openstack-qa | 15:31 | |
| openstackgerrit | Matt Riedemann proposed openstack/tempest: WIP: Add compute API tests for 'get-me-a-network' https://review.openstack.org/327191 | 15:32 |
| *** yamamoto has quit IRC | 15:33 | |
| *** tfreger has quit IRC | 15:36 | |
| openstackrecheck | Console logs not available after 13:28s for gate-nova-python34-db 280301,11,fd0277f | 15:36 |
| *** Swami_ has joined #openstack-qa | 15:38 | |
| openstackgerrit | Rob Crittenden proposed openstack-dev/devstack: Make wait_for_service more robust by checking HTTP response https://review.openstack.org/321249 | 15:39 |
| *** nadya has joined #openstack-qa | 15:40 | |
| *** Swami__ has joined #openstack-qa | 15:40 | |
| *** nadya has quit IRC | 15:40 | |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Add tempest run command https://review.openstack.org/284411 | 15:40 |
| mtreinish | masayukig: ^^^ that should address the things you pointed out | 15:41 |
| mtreinish | and some more that I found with additional testing | 15:41 |
| *** Swami has quit IRC | 15:42 | |
| *** zeih has joined #openstack-qa | 15:43 | |
| *** Swami_ has quit IRC | 15:43 | |
| *** zeih has quit IRC | 15:43 | |
| *** bkopilov has quit IRC | 15:43 | |
| *** zeih has joined #openstack-qa | 15:44 | |
| *** Swami__ has quit IRC | 15:45 | |
| *** vhosakot has joined #openstack-qa | 15:46 | |
| *** yamamoto has joined #openstack-qa | 15:46 | |
| mtreinish | mriedem: you'll still need the skip in skip_checks too, because of the whole n-net vs neutron thing, the network service tag doesn't actually ever skip | 15:48 |
| mtreinish | mriedem: https://github.com/openstack/tempest/blob/master/tempest/test.py#L77 | 15:48 |
| *** zeih has quit IRC | 15:48 | |
| mriedem | mtreinish: ok... | 15:49 |
| mriedem | not sure why the network tag decoration is needed then, just info i guess? | 15:49 |
| mtreinish | it was more about just tagging the test so if you wanted to filter to just network tests this would get run | 15:49 |
| mtreinish | like 'testr run network' | 15:50 |
| openstackrecheck | Console logs not available after 13:27s for gate-horizon-tox-py27dj18 316244,28,34659d3 | 15:50 |
| *** Apoorva has joined #openstack-qa | 15:51 | |
| *** bkopilov has joined #openstack-qa | 15:52 | |
| *** dwalleck has joined #openstack-qa | 15:53 | |
| openstackgerrit | Daniel Berrange proposed openstack-dev/devstack: nova.conf: set privsep helper command for os-vif plugins https://review.openstack.org/327199 | 15:54 |
| *** rlrossit has quit IRC | 15:55 | |
| andreaf | mtreinish, oomichi: I'm trying to decouple client lazy load in the client manager from making client manager ready to be a stable interface, if you want to have a look https://review.openstack.org/#/c/326683 I'm looking for early feedback on the approach | 15:56 |
| *** yamamoto has quit IRC | 15:56 | |
| *** davidlenwell has joined #openstack-qa | 15:56 | |
| *** yamamoto has joined #openstack-qa | 15:56 | |
| mtreinish | andreaf: I'll trade you for a look at a quick fix: https://review.openstack.org/326725 and tempest run: https://review.openstack.org/284411 | 15:57 |
| *** gvrangan has quit IRC | 15:57 | |
| *** sabeen has joined #openstack-qa | 15:57 | |
| andreaf | +A and the 2nd one I need some time to review - but now I'm off to the airport to pick up my brother in law - so later tonight or tomorrow | 16:00 |
| *** yamamoto has quit IRC | 16:00 | |
| andreaf | mtreinish: ^^ | 16:00 |
| *** yamamoto has joined #openstack-qa | 16:00 | |
| mtreinish | andreaf: ok cool, thanks | 16:00 |
| andreaf | mtreinish: for the client manger ones, they are still WIP but should be give an idea, no rush | 16:00 |
| mtreinish | andreaf: I just +2'd the unit test patch | 16:00 |
| *** timello has joined #openstack-qa | 16:01 | |
| andreaf | this one is of course my favourite one: https://review.openstack.org/#/c/221020/ :) | 16:01 |
| andreaf | thanks | 16:01 |
| timello | ci | 16:01 |
| * andreaf wonders why git add -u does not selectively add to the commit the new files that he meant to add :D | 16:02 | |
| *** cody-somerville has joined #openstack-qa | 16:02 | |
| andreaf | mtreinish: git should learn how to read my mind... | 16:02 |
| openstackrecheck | Console logs not available after 13:24s for gate-tempest-dsvm-neutron-full-liberty 326948,1,d66d776 | 16:03 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/325641 failed gate-grenade-dsvm in the gate queue with an unrecognized error | 16:05 |
| *** fawadkhaliq has quit IRC | 16:07 | |
| *** edtubill has joined #openstack-qa | 16:07 | |
| *** yfried has quit IRC | 16:07 | |
| *** cody-somerville has quit IRC | 16:08 | |
| mtreinish | andreaf: heh, it's worse when you run the latest git releases and the behavior changes on patterns you have committed to muscle memory | 16:10 |
| *** nadya has joined #openstack-qa | 16:10 | |
| *** vhosakot has quit IRC | 16:15 | |
| *** salv-orlando has quit IRC | 16:18 | |
| *** salv-orlando has joined #openstack-qa | 16:18 | |
| *** rossella_s has quit IRC | 16:19 | |
| *** thingee has joined #openstack-qa | 16:22 | |
| *** nadya has quit IRC | 16:23 | |
| *** salv-orl_ has joined #openstack-qa | 16:24 | |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Add tempest run command https://review.openstack.org/284411 | 16:25 |
| *** suyash has joined #openstack-qa | 16:26 | |
| *** salv-orlando has quit IRC | 16:27 | |
| *** sabeen has quit IRC | 16:28 | |
| *** sabeen has joined #openstack-qa | 16:28 | |
| *** e0ne has quit IRC | 16:30 | |
| *** yamahata has quit IRC | 16:30 | |
| *** yamamoto has quit IRC | 16:30 | |
| *** Gen has quit IRC | 16:31 | |
| *** Gen has joined #openstack-qa | 16:31 | |
| *** Gen has quit IRC | 16:31 | |
| *** Gen has joined #openstack-qa | 16:31 | |
| *** tesseract has quit IRC | 16:31 | |
| *** dtantsur is now known as dtantsur|afk | 16:32 | |
| *** Gen has quit IRC | 16:33 | |
| *** yamamoto has joined #openstack-qa | 16:34 | |
| *** weshay is now known as weshay_brb | 16:34 | |
| *** sabeen has quit IRC | 16:36 | |
| *** frontrunner has quit IRC | 16:36 | |
| *** yolanda has quit IRC | 16:37 | |
| *** jpena is now known as jpena|off | 16:37 | |
| *** nadya has joined #openstack-qa | 16:37 | |
| *** dwalleck has quit IRC | 16:37 | |
| *** yamamoto has quit IRC | 16:39 | |
| *** yolanda has joined #openstack-qa | 16:40 | |
| david-lyle | mtreinish, does this look reasonable re: stable branch concerns? https://review.openstack.org/327226 | 16:41 |
| david-lyle | I basically just removed the stable filter | 16:41 |
| *** rcernin has quit IRC | 16:42 | |
| *** fawadkhaliq has joined #openstack-qa | 16:42 | |
| david-lyle | after I finally got the plugin working properly | 16:42 |
| *** fawadkhaliq has quit IRC | 16:42 | |
| *** fawadkhaliq has joined #openstack-qa | 16:42 | |
| mtreinish | david-lyle: yeah, I think that will work fine | 16:43 |
| david-lyle | mtreinish, thanks | 16:43 |
| *** Serlex has left #openstack-qa | 16:47 | |
| *** frontrunner has joined #openstack-qa | 16:47 | |
| *** lucasagomes is now known as lucas-dinner | 16:48 | |
| mtreinish | david-lyle: if you get a sec: https://review.openstack.org/#/c/326570/ should be quick | 16:49 |
| *** mvk has quit IRC | 16:50 | |
| *** nwkarsten has quit IRC | 16:52 | |
| *** rlrossit has joined #openstack-qa | 16:52 | |
| *** gvrangan has joined #openstack-qa | 16:52 | |
| *** Swami has joined #openstack-qa | 16:54 | |
| jlvillal | sdague: https://review.openstack.org/#/c/316662/ is our final patch needed to get Grenade working for Ironic :) | 16:54 |
| jlvillal | Reviews greatly appreciated... | 16:54 |
| sdague | jlvillal: +2 | 16:56 |
| jlvillal | sdague: Thanks! :) | 16:56 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/326810 failed because of: gate-grenade-dsvm-multinode: https://bugs.launchpad.net/bugs/1298006 | 16:56 |
| openstack | Launchpad bug 1298006 in OpenStack-Gate "network sometimes unreachable accessing our git server" [Medium,Triaged] | 16:56 |
| *** weshay_brb is now known as weshay | 16:57 | |
| *** jay-mehta has joined #openstack-qa | 16:59 | |
| openstackgerrit | Paulo Ewerton Gomes Fragoso proposed openstack/eslint-config-openstack: Enable func-names https://review.openstack.org/244257 | 16:59 |
| *** sabeen has joined #openstack-qa | 17:00 | |
| *** gamado has quit IRC | 17:00 | |
| jay-mehta | Could anyone please review unit test patch https://review.openstack.org/#/c/319511/ . It is +1 | 17:00 |
| mtreinish | jay-mehta: you're probably gonna have more luck asking for reviews on that in #openstack-cinder | 17:02 |
| *** cody-somerville has joined #openstack-qa | 17:04 | |
| *** yamahata has joined #openstack-qa | 17:11 | |
| *** cody-somerville has quit IRC | 17:12 | |
| openstackgerrit | Franklin Naval proposed openstack/tempest: Allows specifying a name for a particular endpoint. https://review.openstack.org/284273 | 17:12 |
| openstackrecheck | Console logs not available after 13:29s for gate-grenade-dsvm-multinode 326667,9,e321878 | 17:13 |
| fnaval | andreaf: this should be ready for reviews again after it passes gates: https://review.openstack.org/#/c/284273 | 17:14 |
| fnaval | thanks in advance! | 17:14 |
| *** nadya has quit IRC | 17:18 | |
| openstackgerrit | Franklin Naval proposed openstack/tempest: Allows specifying a name for a particular endpoint. https://review.openstack.org/284273 | 17:18 |
| *** kiran-r has quit IRC | 17:20 | |
| *** vhosakot has joined #openstack-qa | 17:20 | |
| openstackgerrit | Franklin Naval proposed openstack/tempest: Allows specifying a name for a particular endpoint. https://review.openstack.org/284273 | 17:22 |
| *** nadya has joined #openstack-qa | 17:23 | |
| *** dimtruck is now known as zz_dimtruck | 17:30 | |
| *** e0ne has joined #openstack-qa | 17:32 | |
| *** yamahata has quit IRC | 17:33 | |
| *** zz_dimtruck is now known as dimtruck | 17:33 | |
| *** yamahata has joined #openstack-qa | 17:33 | |
| *** Gen has joined #openstack-qa | 17:35 | |
| *** gszasz has quit IRC | 17:40 | |
| *** sabeen has quit IRC | 17:41 | |
| *** nwkarsten has joined #openstack-qa | 17:42 | |
| *** nwkarsten has quit IRC | 17:43 | |
| *** e0ne has quit IRC | 17:43 | |
| *** nwkarsten has joined #openstack-qa | 17:43 | |
| *** kzaitsev_mb has quit IRC | 17:44 | |
| *** nadya has quit IRC | 17:45 | |
| *** ihrachys has quit IRC | 17:47 | |
| *** e0ne has joined #openstack-qa | 17:50 | |
| *** sambetts is now known as sambetts|afk | 17:53 | |
| *** salv-orl_ has quit IRC | 17:53 | |
| *** salv-orlando has joined #openstack-qa | 17:53 | |
| *** e0ne has quit IRC | 17:53 | |
| *** sabeen has joined #openstack-qa | 17:54 | |
| openstackgerrit | Castulo J. Martinez proposed openstack/tempest: Add missing tests for listing stacks for Heat API https://review.openstack.org/325516 | 17:57 |
| *** sabeen has quit IRC | 18:01 | |
| *** sabeen has joined #openstack-qa | 18:01 | |
| openstackgerrit | Andrew Kerr proposed openstack/tempest: Add tests for Cinder user messages v3 API https://review.openstack.org/300639 | 18:01 |
| openstackgerrit | Andrew Kerr proposed openstack-dev/devstack: Set volume service min and max microversions https://review.openstack.org/316144 | 18:02 |
| *** rlrossit has quit IRC | 18:04 | |
| *** pvaneck has joined #openstack-qa | 18:04 | |
| *** rlrossit has joined #openstack-qa | 18:08 | |
| *** lopin has joined #openstack-qa | 18:09 | |
| *** asselin has quit IRC | 18:10 | |
| *** frontrunner has quit IRC | 18:10 | |
| -openstackstatus- NOTICE: Zuul has been restarted to correct an error condition. Events since 17:30 may have been missed; please 'recheck' your changes if they were uploaded since then, or have "NOT_REGISTERED" errors. | 18:11 | |
| *** jecarey has quit IRC | 18:11 | |
| openstackgerrit | Castulo J. Martinez proposed openstack/tempest: Add missing tests for the identity v3 API https://review.openstack.org/316182 | 18:11 |
| *** lopin has quit IRC | 18:13 | |
| *** lopin has joined #openstack-qa | 18:15 | |
| *** jecarey has joined #openstack-qa | 18:17 | |
| *** mriedem1 has joined #openstack-qa | 18:17 | |
| *** dpaterson has joined #openstack-qa | 18:18 | |
| *** mriedem has quit IRC | 18:18 | |
| *** asselin has joined #openstack-qa | 18:21 | |
| *** mriedem1 is now known as mriedem | 18:26 | |
| dpaterson | masayukig: Is there a plan for a mid-cycle sprint? If so, I would like to participate but need to budget travel soon. | 18:28 |
| *** lopin has quit IRC | 18:33 | |
| openstackgerrit | Richard Theis proposed openstack-dev/devstack: Support "geneve" ML2 plugin type driver https://review.openstack.org/327278 | 18:35 |
| *** zeih has joined #openstack-qa | 18:35 | |
| *** frontrunner has joined #openstack-qa | 18:36 | |
| *** lopin has joined #openstack-qa | 18:36 | |
| *** alyson_ has quit IRC | 18:38 | |
| *** gvrangan has quit IRC | 18:40 | |
| *** gouthamr has quit IRC | 18:45 | |
| *** lopin has quit IRC | 18:47 | |
| *** lopin has joined #openstack-qa | 18:48 | |
| *** yolanda has quit IRC | 18:48 | |
| *** vhosakot has quit IRC | 18:50 | |
| *** krtaylor has quit IRC | 18:52 | |
| *** Kennan has quit IRC | 18:52 | |
| *** vhosakot has joined #openstack-qa | 18:52 | |
| *** Kennan has joined #openstack-qa | 18:52 | |
| *** cody-somerville has joined #openstack-qa | 18:53 | |
| *** cody-somerville has quit IRC | 18:53 | |
| *** cody-somerville has joined #openstack-qa | 18:53 | |
| mtreinish | dpaterson: if you get a sec you might want to take a look at the lastest rev on tempest run: https://review.openstack.org/284411 | 18:54 |
| dpaterson | mtreinish: will do | 18:54 |
| mtreinish | it should be about ready now | 18:54 |
| *** fnaval has quit IRC | 18:57 | |
| *** fnaval has joined #openstack-qa | 18:58 | |
| *** fnaval has quit IRC | 18:58 | |
| *** fnaval has joined #openstack-qa | 18:59 | |
| *** gouthamr has joined #openstack-qa | 19:01 | |
| *** alex_xu has quit IRC | 19:02 | |
| *** jecarey has quit IRC | 19:03 | |
| *** lopin has quit IRC | 19:03 | |
| *** lopin has joined #openstack-qa | 19:05 | |
| *** Apoorva has quit IRC | 19:05 | |
| *** krtaylor has joined #openstack-qa | 19:05 | |
| *** alex_xu has joined #openstack-qa | 19:06 | |
| andreaf | fnaval: cool I'll check it tomorrow | 19:09 |
| fnaval | andreaf: thank you sir! | 19:09 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/devstack-gate: DNM playing with tempest run https://review.openstack.org/327291 | 19:10 |
| *** jecarey has joined #openstack-qa | 19:11 | |
| *** jecarey has quit IRC | 19:11 | |
| *** jecarey has joined #openstack-qa | 19:12 | |
| *** lopin has quit IRC | 19:13 | |
| *** lopin has joined #openstack-qa | 19:13 | |
| *** e0ne has joined #openstack-qa | 19:13 | |
| *** salv-orlando has quit IRC | 19:14 | |
| *** salv-orlando has joined #openstack-qa | 19:14 | |
| *** kzaitsev_mb has joined #openstack-qa | 19:19 | |
| *** gvrangan has joined #openstack-qa | 19:19 | |
| *** frontrunner has quit IRC | 19:21 | |
| mtreinish | timothyb89: do you want to add your subunit karma reporter to gerrit and add it to qa? Or just keep it on your github? | 19:23 |
| *** rcernin has joined #openstack-qa | 19:25 | |
| *** gvrangan has quit IRC | 19:26 | |
| *** ifarkas has quit IRC | 19:31 | |
| *** nadya has joined #openstack-qa | 19:37 | |
| *** lopin has quit IRC | 19:37 | |
| openstackgerrit | Matthew Treinish proposed openstack-infra/devstack-gate: DNM playing with tempest run https://review.openstack.org/327291 | 19:38 |
| *** salv-orlando has quit IRC | 19:39 | |
| *** salv-orlando has joined #openstack-qa | 19:40 | |
| *** lopin has joined #openstack-qa | 19:40 | |
| *** spzala has quit IRC | 19:46 | |
| *** lopin has quit IRC | 19:46 | |
| *** lopin has joined #openstack-qa | 19:46 | |
| *** Apoorva has joined #openstack-qa | 19:47 | |
| *** yolanda has joined #openstack-qa | 19:50 | |
| *** timello has quit IRC | 19:52 | |
| *** cody-somerville has quit IRC | 19:54 | |
| *** dimtruck is now known as zz_dimtruck | 19:54 | |
| *** lopin has quit IRC | 19:56 | |
| *** zz_dimtruck is now known as dimtruck | 19:57 | |
| timothyb89 | mtreinish: I'm open to adding it to qa if you think that's a good home for it | 19:58 |
| timothyb89 | mtreinish: there's also node-subunit to worry about as well | 19:59 |
| *** fawadkhaliq has quit IRC | 20:01 | |
| *** fawadkhaliq has joined #openstack-qa | 20:01 | |
| *** e0ne has quit IRC | 20:04 | |
| *** nwkarsten has quit IRC | 20:04 | |
| *** fawadkhaliq has quit IRC | 20:05 | |
| *** nwkarsten has joined #openstack-qa | 20:08 | |
| *** nwkarsten has quit IRC | 20:08 | |
| *** nwkarsten has joined #openstack-qa | 20:08 | |
| *** timello has joined #openstack-qa | 20:09 | |
| *** jeblair has quit IRC | 20:09 | |
| *** jeblair has joined #openstack-qa | 20:10 | |
| *** minwang2 has joined #openstack-qa | 20:10 | |
| *** minwang2 has quit IRC | 20:12 | |
| mtreinish | timothyb89: yeah, I think node-subunit just belongs in subunit upstream, it already has multiple language bindings. The only thing I don't know about there is the packaging story, you'll have to ask lifeless | 20:12 |
| mtreinish | timothyb89: but for the karma reporter I think QA is a good home for it | 20:12 |
| mtreinish | timothyb89: although having a random js lib on github is the norm, not sure you want to deviate from that :) | 20:13 |
| timothyb89 | mtreinish: heh, I suppose that's true as well | 20:13 |
| openstackgerrit | Rob Crittenden proposed openstack-dev/devstack: Make wait_for_service more robust by checking HTTP response https://review.openstack.org/321249 | 20:14 |
| timothyb89 | mtreinish: should I get a patch up to create a karma-subunit-reporter project? | 20:14 |
| *** timello has quit IRC | 20:15 | |
| mtreinish | timothyb89: yeah, and then a governance patch to add it to qa | 20:16 |
| timothyb89 | mtreinish: okay cool, I'll get on that | 20:16 |
| mtreinish | mriedem: hmm, that error message isn't super useful: http://logs.openstack.org/91/327191/1/check/gate-tempest-dsvm-neutron-full/615a890/console.html#_2016-06-08_20_09_50_702 | 20:17 |
| mtreinish | doesn't say where the network is coming from | 20:17 |
| mriedem | mtreinish: damn you're faster than i was on that one | 20:18 |
| *** minwang2 has joined #openstack-qa | 20:19 | |
| *** mvk has joined #openstack-qa | 20:19 | |
| mtreinish | mriedem: I wonder if it's picking up a shared network or something? We've had problems with that before | 20:21 |
| mriedem | yeah i guess my setup check for that doesn't match what's in the nova code for getting available networks for the tenant | 20:21 |
| *** nadya has quit IRC | 20:22 | |
| *** timello has joined #openstack-qa | 20:22 | |
| openstackrecheck | Console logs not available after 13:30s for gate-nova-python27-db 327183,1,7c1fd09 | 20:24 |
| *** spzala has joined #openstack-qa | 20:25 | |
| *** spzala has quit IRC | 20:26 | |
| *** spzala has joined #openstack-qa | 20:26 | |
| *** gvrangan has joined #openstack-qa | 20:27 | |
| *** ihrachys has joined #openstack-qa | 20:28 | |
| *** timello has quit IRC | 20:29 | |
| *** cody-somerville has joined #openstack-qa | 20:30 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Define 3 image clients as libraries https://review.openstack.org/326751 | 20:36 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Separate resource_types_client from images_client https://review.openstack.org/326700 | 20:36 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Add unit tests for glance v2 service clients https://review.openstack.org/327320 | 20:36 |
| *** cody-somerville has quit IRC | 20:38 | |
| *** timello has joined #openstack-qa | 20:38 | |
| *** timello has quit IRC | 20:38 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Define 3 image clients as libraries https://review.openstack.org/326751 | 20:39 |
| mtreinish | andreaf: did you see my comment on your scope patch? it looks like it's breaking the puppet jobs. I didn't dig into it too much, but it's probably running in a config you weren't expecting | 20:40 |
| *** nadya has joined #openstack-qa | 20:40 | |
| openstackrecheck | Console logs not available after 13:32s for gate-nova-python34-db 327188,1,e89669b | 20:40 |
| *** timello has joined #openstack-qa | 20:42 | |
| *** gouthamr has quit IRC | 20:43 | |
| *** dave-mccowan has quit IRC | 20:44 | |
| *** nadya has quit IRC | 20:45 | |
| *** spzala has quit IRC | 20:45 | |
| *** dimtruck is now known as zz_dimtruck | 20:46 | |
| *** zeih has quit IRC | 20:47 | |
| *** openstackgerrit has quit IRC | 20:48 | |
| *** openstackgerrit has joined #openstack-qa | 20:48 | |
| openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: DNM: experimenting with local.conf https://review.openstack.org/326585 | 20:53 |
| *** jbernard has quit IRC | 20:56 | |
| *** erlon has quit IRC | 20:56 | |
| *** nadya has joined #openstack-qa | 20:58 | |
| *** porrua has quit IRC | 20:58 | |
| *** dave-mccowan has joined #openstack-qa | 21:00 | |
| *** jbernard has joined #openstack-qa | 21:03 | |
| *** nadya has quit IRC | 21:03 | |
| *** edtubill has quit IRC | 21:06 | |
| *** dkranz has quit IRC | 21:08 | |
| *** zhufl has quit IRC | 21:10 | |
| *** zhufl has joined #openstack-qa | 21:11 | |
| openstackrecheck | Console logs not available after 13:25s for gate-tempest-pep8 284411,14,1b09610 | 21:12 |
| mriedem | mtreinish: hmm, looks like when setting up credentials there shouldn't be a network | 21:15 |
| mriedem | http://logs.openstack.org/91/327191/1/check/gate-tempest-dsvm-neutron-full/615a890/logs/tempest.txt.gz#_2016-06-08_19_39_36_871 | 21:15 |
| mriedem | but yeah that's probably a private tenant network? | 21:15 |
| mriedem | not accounting for the public shared network that devstack sets up? | 21:15 |
| *** nadya has joined #openstack-qa | 21:17 | |
| mriedem | ah yeah it's the public network | 21:17 |
| mriedem | http://logs.openstack.org/91/327191/1/check/gate-tempest-dsvm-neutron-full/615a890/logs/tempest.txt.gz#_2016-06-08_19_39_37_654 | 21:17 |
| *** dustins has quit IRC | 21:17 | |
| *** ylobankov has quit IRC | 21:17 | |
| *** ylobankov has joined #openstack-qa | 21:17 | |
| mtreinish | mriedem: yeah, the set_network_resources call you added before the super() in setup_credentials tells dynamic_creds to not create anything networking related | 21:20 |
| mtreinish | mriedem: we should be able to filter out the public networks, this has come up before | 21:21 |
| mriedem | mtreinish: yeah | 21:21 |
| mtreinish | let me try to find an example (or just looking at the json blob might be faster) | 21:21 |
| mriedem | i really just need to mimic this https://github.com/openstack/nova/blob/a0bedb332c96b363f0fd6acfa81b5937a82244eb/nova/network/neutronv2/api.py#L155-L160 | 21:21 |
| mriedem | but need to figure out what tenant id the test is using | 21:21 |
| *** nadya has quit IRC | 21:22 | |
| mtreinish | mriedem: cls.os.credentials.project_id, or something like that | 21:22 |
| mriedem | cls.os.credentials.tenant_id | 21:24 |
| mtreinish | mriedem: that would only work on v2 I think | 21:25 |
| *** mvk_ has joined #openstack-qa | 21:25 | |
| mriedem | it's what the volume api tests are using | 21:26 |
| openstackrecheck | Console logs not available after 13:27s for gate-nova-pep8 320063,7,3363a8a | 21:26 |
| mtreinish | hmm, well whatever. It's not like keystone v3 works in the gate anyway | 21:27 |
| mtreinish | if it's an issue we can always fix it later | 21:27 |
| *** rlrossit has quit IRC | 21:27 | |
| *** mvk has quit IRC | 21:28 | |
| mriedem | can you get the tenant_id from any rest client? | 21:28 |
| mriedem | cls.tenant_id = cls.flavors_client.tenant_id | 21:29 |
| mriedem | i see stuff like that | 21:29 |
| mtreinish | mriedem: I was thinking you could just do this: https://github.com/openstack/tempest/blob/master/tempest/api/network/test_networks.py#L377-L382 | 21:29 |
| *** dpaterson has quit IRC | 21:30 | |
| mtreinish | mriedem: yeah tenant_id and stuff are properties on the rest_client class (which is the base for all the service clients) | 21:30 |
| *** lifeless_ is now known as lifeless | 21:30 | |
| mtreinish | mriedem: https://github.com/openstack/tempest/blob/master/tempest/lib/common/rest_client.py#L163-L170 | 21:31 |
| mriedem | kick ass | 21:31 |
| mriedem | kick | 21:31 |
| mriedem | ass | 21:31 |
| mriedem | for dramatic effect | 21:31 |
| mtreinish | heh | 21:32 |
| lifeless | mtreinish: timothyb89: we can make a repo for node-subunit trivially and give you commit access | 21:32 |
| lifeless | like the go-subunit repo | 21:32 |
| lifeless | just tell me the one to adopt | 21:32 |
| lifeless | / a name | 21:32 |
| lifeless | and whom | 21:32 |
| mtreinish | lifeless: https://github.com/timothyb89/node-subunit | 21:32 |
| mtreinish | lifeless: it's all timothyb89 | 21:33 |
| lifeless | mtreinish: cool; so I'll wait for an ack from timothyb89 and then do that | 21:34 |
| timothyb89 | lifeless: sure, whatever you think is bet | 21:34 |
| timothyb89 | best, even | 21:34 |
| *** nadya has joined #openstack-qa | 21:35 | |
| *** yamahata has quit IRC | 21:35 | |
| lifeless | timothyb89: is it node specific, or js but you built it to use with node ? | 21:35 |
| openstackgerrit | Matt Riedemann proposed openstack/tempest: WIP: Add compute API tests for 'get-me-a-network' https://review.openstack.org/327191 | 21:36 |
| timothyb89 | lifeless: node specific at the moment | 21:36 |
| *** ihrachys has quit IRC | 21:36 | |
| lifeless | timothyb89: ok, so we should expect a subunit-js thats different in future? Or would you refactor / add more modules within this code base to e.g. support selenium | 21:37 |
| *** ihrachys has joined #openstack-qa | 21:37 | |
| timothyb89 | potentially more modules in the future, though I'd expect the base lib to be mostly the same | 21:39 |
| *** nadya has quit IRC | 21:39 | |
| openstackrecheck | Console logs not available after 13:27s for gate-horizon-pep8 327292,1,eb04c77 | 21:39 |
| timothyb89 | though it would be possible to add support for browser usage (vanilla js) as well | 21:39 |
| lifeless | timothyb89: ok, so I propose to call it subunit-js | 21:41 |
| *** lutostag has quit IRC | 21:41 | |
| lifeless | timothyb89: matching subunit-go in namning convention | 21:41 |
| lifeless | timothyb89: ok? | 21:41 |
| timothyb89 | lifeless: sure, sounds fine to me | 21:42 |
| timothyb89 | though I'm unsure what renaming the npm package will entail | 21:42 |
| *** cody-somerville has joined #openstack-qa | 21:42 | |
| timothyb89 | I suppose just mark as deprecated and publish under the new name | 21:42 |
| lifeless | ok, so https://github.com/timothyb89/node-subunit is the remote you'll need to add | 21:43 |
| *** lutostag has joined #openstack-qa | 21:43 | |
| *** nadya has joined #openstack-qa | 21:44 | |
| lifeless | timothyb89: you should have an invite to the committers team for testing-cabal now | 21:44 |
| timothyb89 | ah, cool, just saw that | 21:45 |
| lifeless | timothyb89: with great power comes great responsibility :) - e.g. don't push into other repos without chatting with folk that are feeding it first, we have a trust-folk-will-be-sensible policy rather than lots of acls | 21:45 |
| timothyb89 | lifeless: for sure :) | 21:45 |
| lifeless | lol @ testymctestface | 21:46 |
| timothyb89 | I was feeling especially uncreative that day... :) | 21:47 |
| lifeless | timothyb89: I don't know if you need to rename the node package or not | 21:47 |
| lifeless | timothyb89: it might be nice to do so but I certainly don't care any which way :) | 21:47 |
| lifeless | let me know when you've pushed and I'll tweet that its there | 21:47 |
| *** pauloewerton has quit IRC | 21:48 | |
| timothyb89 | lifeless: sure... are there any other users I should add as owners of the npm package? | 21:48 |
| *** timello has quit IRC | 21:49 | |
| *** nadya has quit IRC | 21:49 | |
| lifeless | timothyb89: on the npm site? No idea :) I'd wait and see who else collaborates with you | 21:49 |
| timothyb89 | okay, fair enough | 21:50 |
| lifeless | timothyb89: I don't know who else in testing-cabal even has an npm uid | 21:50 |
| timothyb89 | lifeless: are there any concerns with licensing of the repo as it is now? | 21:53 |
| lifeless | Apache V2 is fine | 21:53 |
| lifeless | which is what it looked like when I checked it out, no ? | 21:53 |
| *** ihrachys_ has joined #openstack-qa | 21:53 | |
| *** rcernin is now known as rcernin|off | 21:54 | |
| *** cdent has quit IRC | 21:54 | |
| timothyb89 | lifeless: yup, no changes planned, just making sure that's fine | 21:54 |
| openstackrecheck | Console logs not available after 13:30s for gate-tempest-dsvm-postgres-full 327291,2,70eac94 | 21:54 |
| *** ihrachys has quit IRC | 21:56 | |
| *** Gen has quit IRC | 21:58 | |
| *** nadya has joined #openstack-qa | 22:02 | |
| *** cody-somerville has quit IRC | 22:03 | |
| *** vhosakot has quit IRC | 22:03 | |
| *** timello has joined #openstack-qa | 22:05 | |
| *** gouthamr has joined #openstack-qa | 22:06 | |
| openstackrecheck | Console logs not available after 13:28s for gate-glance-python27-db 326126,4,9b1a816 | 22:08 |
| *** pglass has quit IRC | 22:08 | |
| *** nadya has quit IRC | 22:08 | |
| timothyb89 | lifeless: okay, I've updated docs and pushed, and re-published to npm under the new name: https://www.npmjs.com/package/subunit-js | 22:10 |
| lifeless | cool | 22:11 |
| *** ihrachys_ has quit IRC | 22:11 | |
| *** nadya has joined #openstack-qa | 22:11 | |
| *** ihrachys has joined #openstack-qa | 22:12 | |
| *** ihrachys has quit IRC | 22:12 | |
| lifeless | timothyb89: and publicised. wicked :) | 22:12 |
| timothyb89 | lifeless: awesome! | 22:13 |
| *** nadya has quit IRC | 22:16 | |
| *** nadya has joined #openstack-qa | 22:20 | |
| *** rcernin|off has quit IRC | 22:21 | |
| openstackrecheck | Console logs not available after 13:23s for gate-nova-pep8 327302,1,56e1fea | 22:22 |
| andreaf | mtreinish: thanks nice catch on https://review.openstack.org/#/c/221020/ - I know nothing about those puppet jobs, I'll try to find out - do you have an overview you can point me at? | 22:23 |
| *** nwkarsten has quit IRC | 22:23 | |
| *** gvrangan has quit IRC | 22:23 | |
| *** gvrangan has joined #openstack-qa | 22:25 | |
| *** salv-orl_ has joined #openstack-qa | 22:25 | |
| *** nadya has quit IRC | 22:25 | |
| *** tosky has quit IRC | 22:25 | |
| mtreinish | andreaf: from our perspective it's mostly just everything is ipv6, ssl, and keystone v3 by default | 22:26 |
| andreaf | mtreinish: I guess the problem may be that the default admin user is not assigned a role on the domain, like we do in devstack, but I have no idea where to look for that | 22:26 |
| mtreinish | andreaf: heh, probably puppet-keystone or puppet-tempest | 22:26 |
| andreaf | mtreinish: I guess that's created by a puppet module | 22:26 |
| mtreinish | andreaf: I would ask EmilienM he'll probably know off the top of his head | 22:27 |
| andreaf | EmilienM: ping - I'm looking for the place where the admin user is setup in puppet jobs running against tempest - i.e. http://logs.openstack.org/20/221020/46/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7-nv/277ebe2/console.html#_2016-06-08_14_34_27_013 | 22:27 |
| *** salv-orlando has quit IRC | 22:28 | |
| *** fnaval has quit IRC | 22:29 | |
| *** fnaval has joined #openstack-qa | 22:29 | |
| *** nadya has joined #openstack-qa | 22:29 | |
| *** mriedem has quit IRC | 22:31 | |
| mtreinish | andreaf: https://github.com/openstack/puppet-openstack-integration/blob/6bcc7756fc7eeff4f32e795165c0008e1fdb258c/manifests/keystone.pp#L70-L73 and https://github.com/openstack/puppet-keystone/blob/49bf9e85ece425adafdf03246318281c6c233cc2/manifests/roles/admin.pp | 22:32 |
| mtreinish | I think that's it | 22:32 |
| mtreinish | so it doesn't look like it's setting a role on the domain | 22:33 |
| mtreinish | andreaf: but is that a requirement? | 22:33 |
| andreaf | for using domain scoped tokens yes | 22:34 |
| andreaf | which is what my patch tries to turn on for identity v3 admin clients | 22:34 |
| *** timello has quit IRC | 22:34 | |
| *** nadya has quit IRC | 22:35 | |
| mtreinish | andreaf: but I'm not sure that's something we can really assume for config file admin creds | 22:35 |
| mtreinish | for dynamic creds we can, we just assign it | 22:35 |
| *** zz_dimtruck is now known as dimtruck | 22:35 | |
| mtreinish | and for preprov we can add a flag or something | 22:35 |
| mtreinish | but for the config file we don't have anything to really describe it | 22:35 |
| *** Gen has joined #openstack-qa | 22:35 | |
| mtreinish | why is v3 so confusing... | 22:35 |
| andreaf | mtreinish: if a cloud uses this policy https://github.com/openstack/keystone/blob/master/etc/policy.v3cloudsample.json, a domain token is required to do any admin operation, so the initial admin must have that | 22:36 |
| andreaf | role on the domain | 22:36 |
| *** edmondsw has quit IRC | 22:37 | |
| andreaf | mtreinish: I could introduce a configuration flag then, whether the role on domain is required for admins or not | 22:37 |
| andreaf | dynamic cred will add it, and now users pre-provisioned via the account generator will also do the same | 22:37 |
| mtreinish | andreaf: heh, well we can't assume the accounts.yaml came from the script | 22:38 |
| *** nwkarsten has joined #openstack-qa | 22:38 | |
| EmilienM | andreaf: looking | 22:38 |
| andreaf | mtreinish: the base policy instead does not enforce that https://github.com/openstack/keystone/blob/master/etc/policy.json but it works with domain scoped tokens as well | 22:38 |
| * EmilienM reading backlog | 22:38 | |
| *** nadya has joined #openstack-qa | 22:39 | |
| *** timello has joined #openstack-qa | 22:39 | |
| mtreinish | andreaf: sure, I got that. I'm just trying to figure out how we want to handle domain roles on provided credentials | 22:39 |
| mtreinish | because right now your patch does it as an implicit assumption which is what tripped up the puppet jobs | 22:40 |
| mtreinish | and if we do that we need to really document that | 22:40 |
| mtreinish | and very clearly so | 22:40 |
| andreaf | mtreinish: ok that's a good point | 22:40 |
| EmilienM | let me check if we set a role on the domain | 22:40 |
| mtreinish | or we make it configurable somehow | 22:40 |
| andreaf | mtreinish: well if a cloud uses the cloudv3 policy its admin users wherever they come from should have the role on the domain else they won't be able to do anything | 22:41 |
| mtreinish | andreaf: especially because whether it's a hard requirement or not comes from the configured policy.json which is completely undiscoverable from the api side | 22:41 |
| mtreinish | as that failure shows | 22:41 |
| mtreinish | no real indication of what was wrong | 22:41 |
| andreaf | mtreinish: so perhaps we can have a new config item about this - it it's enabled, the domain scoped is used, and admins are expected to have that role - we can document that | 22:42 |
| *** minwang2 has quit IRC | 22:42 | |
| EmilienM | I don't think we set a role on the domain | 22:42 |
| mtreinish | EmilienM: yeah, that's what it looked like when I quickly skimmed things | 22:42 |
| andreaf | mtreinish: if it's disable is behaves like today, so project scope is used, and admin is not required the scope on the domain | 22:43 |
| *** nwkarsten has quit IRC | 22:43 | |
| andreaf | EmilienM: thanks! | 22:43 |
| mtreinish | andreaf: yeah I think a config flag is the only way to handle this | 22:43 |
| EmilienM | if you want I can add the parameters to do it | 22:43 |
| EmilienM | and see how your patch behaves with Depends-On | 22:43 |
| *** nadya has quit IRC | 22:43 | |
| *** fawadkhaliq has joined #openstack-qa | 22:44 | |
| *** kzaitsev_mb has quit IRC | 22:44 | |
| andreaf | EmilienM: I will add a configuration on tempest side, so it's not mandatory for you to have that role - you will really need it only if you want to support the cloudv3sample.json policy in keystone | 22:44 |
| *** kzaitsev_mb has joined #openstack-qa | 22:44 | |
| EmilienM | andreaf: we might want to test it, once your feature is done, we'll have this case in our scenarios | 22:45 |
| andreaf | EmilienM: I assume you deploy with the base policy in the gate job, not sure if you are interested in that policy | 22:45 |
| *** salv-orl_ has quit IRC | 22:45 | |
| EmilienM | it will improve our v3 coverage | 22:45 |
| andreaf | EmilienM: cool :) | 22:45 |
| EmilienM | andreaf: we are using policy proved by upstream repos | 22:45 |
| EmilienM | (packaging, but upstream repo) | 22:45 |
| EmilienM | we don't touch it | 22:45 |
| * EmilienM bookmarks your patch and TODO add it to scenarios once it merges | 22:46 | |
| andreaf | EmilienM, mtreinish : thanks | 22:46 |
| *** salv-orlando has joined #openstack-qa | 22:46 | |
| *** JayF has joined #openstack-qa | 22:47 | |
| *** jecarey_ has joined #openstack-qa | 22:47 | |
| *** nadya has joined #openstack-qa | 22:48 | |
| andreaf | mtreinish: I would put the new config flag in the identity section, since it's about how identity is configured | 22:48 |
| mtreinish | andreaf: yeah, that makes sense | 22:48 |
| mtreinish | andreaf: and we'll definitely need some prose on this for configuration.rst | 22:48 |
| mtreinish | because it effects the config opts and preprov | 22:49 |
| andreaf | mtreinish: but I'm struggling with the name, admin_requires_domain_scope ? | 22:49 |
| mtreinish | heh, that's a mouthful | 22:49 |
| mtreinish | but it's accurate | 22:49 |
| andreaf | mtreinish: I know :( nothing shorted comes to mind, perhaps later, I can start with that | 22:49 |
| mtreinish | either that admin_scoped_domain, or admin_domain_scope wfm | 22:50 |
| mtreinish | we can bikeshed on name in review | 22:50 |
| andreaf | what about domain_scoped_tokens | 22:50 |
| *** jecarey has quit IRC | 22:51 | |
| * andreaf bikeshedding.... | 22:51 | |
| *** jecarey_ has quit IRC | 22:52 | |
| andreaf | mtreinish: a config options raises the question of whether we should run that configuration in the gate | 22:52 |
| *** nadya has quit IRC | 22:52 | |
| andreaf | mtreinish: would it be ok to turn the option on for one of the jobs? | 22:52 |
| mtreinish | andreaf: that makes it sound like every token will be domain scoped :) | 22:52 |
| andreaf | domain_scoped_identity_admin_tokens :D that's even longer | 22:53 |
| mtreinish | andreaf: I don't think it really matters if we turn it on by default on master everywhere | 22:53 |
| mtreinish | it's not like we use v3 auth by default | 22:53 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Rename image members clients https://review.openstack.org/327404 | 22:53 |
| andreaf | mtreinish: ok but the puppet jobs then would have to either set the role or disable the config option | 22:54 |
| mtreinish | andreaf: sorry, I meant in devstack | 22:55 |
| mtreinish | it has to be default false in tempest | 22:55 |
| *** yolanda has quit IRC | 22:55 | |
| andreaf | mtreinish: ah right ok | 22:55 |
| *** amotoki has quit IRC | 22:56 | |
| *** nadya has joined #openstack-qa | 22:57 | |
| *** sdague has quit IRC | 22:59 | |
| *** amotoki has joined #openstack-qa | 23:02 | |
| *** rbak has quit IRC | 23:02 | |
| *** nadya has quit IRC | 23:02 | |
| openstackrecheck | Console logs not available after 13:28s for gate-keystone-pep8 320586,33,e10f14e | 23:02 |
| *** timello has quit IRC | 23:03 | |
| *** nadya has joined #openstack-qa | 23:06 | |
| *** sabeen has quit IRC | 23:07 | |
| *** vhosakot has joined #openstack-qa | 23:08 | |
| *** nadya has quit IRC | 23:11 | |
| *** bnemec has quit IRC | 23:11 | |
| openstackgerrit | Merged openstack/tempest: Make missing global config dir not fatal in tempest init https://review.openstack.org/326725 | 23:13 |
| *** markvoelker has quit IRC | 23:13 | |
| *** vhosakot has quit IRC | 23:14 | |
| *** jgriffith is now known as jgriffith_away | 23:14 | |
| *** vhosakot has joined #openstack-qa | 23:14 | |
| *** gouthamr_ has joined #openstack-qa | 23:15 | |
| *** kzaitsev_mb has quit IRC | 23:15 | |
| *** timello has joined #openstack-qa | 23:16 | |
| openstackrecheck | Console logs not available after 13:33s for gate-tempest-dsvm-neutron-full 325589,13,754a060 | 23:16 |
| *** fawadkhaliq has quit IRC | 23:17 | |
| *** fawadkhaliq has joined #openstack-qa | 23:18 | |
| *** gouthamr has quit IRC | 23:18 | |
| *** bnemec has joined #openstack-qa | 23:19 | |
| *** fawadkhaliq has quit IRC | 23:21 | |
| *** nwkarsten has joined #openstack-qa | 23:22 | |
| *** nadya has joined #openstack-qa | 23:24 | |
| *** timello has quit IRC | 23:24 | |
| *** vhosakot has quit IRC | 23:25 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Use scope in v3 identity client https://review.openstack.org/221020 | 23:26 |
| *** vhosakot has joined #openstack-qa | 23:26 | |
| andreaf | mtreinish: ^^^ first attempt | 23:26 |
| openstackgerrit | Merged openstack/tempest: Centralized Workspaces https://review.openstack.org/283770 | 23:28 |
| *** nadya has quit IRC | 23:29 | |
| openstackrecheck | Console logs not available after 13:26s for gate-horizon-tox-py27dj18 316244,31,16f6dc4 | 23:29 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: WIP: Add option to include run_id test_runs by run meta call https://review.openstack.org/327411 | 23:30 |
| openstackgerrit | Andrea Frittoli proposed openstack-dev/devstack: Enable admin_domain_scope by default in Tempest https://review.openstack.org/327414 | 23:37 |
| andreaf | mtreinish: and the devstack change ^^^ | 23:38 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Test change https://review.openstack.org/327416 | 23:39 |
| andreaf | mtreinish: ...and a test change on top that pulls all together and tests on experimental queue ^^^ | 23:41 |
| *** tochi has joined #openstack-qa | 23:42 | |
| *** nadya has joined #openstack-qa | 23:42 | |
| andreaf | masayukig. oomichi, gmann: a couple of easy ones: https://review.openstack.org/#/c/326976/ and https://review.openstack.org/#/c/327182/ | 23:42 |
| openstackrecheck | Console logs not available after 13:31s for gate-tempest-dsvm-postgres-full 320156,52,baf03d9 | 23:43 |
| oomichi | andreaf: heh, ok. checking :) | 23:43 |
| *** markvoelker has joined #openstack-qa | 23:44 | |
| andreaf | mtreinish: how would you implement those unit tests without a Mixin? Just repeating all the mocks everywhere? | 23:44 |
| * andreaf wants to learn how to write better code :) | 23:45 | |
| *** timello has joined #openstack-qa | 23:47 | |
| *** nadya has quit IRC | 23:47 | |
| *** markvoelker has quit IRC | 23:49 | |
| oomichi | andreaf: about https://review.openstack.org/#/c/326976/1/tempest/common/preprov_creds.py | 23:49 |
| oomichi | andreaf: why can we just remove the lines 332-334? the above 2 seem good as the logic, but the last one is different | 23:50 |
| andreaf | oomichi: use_default_creds is not valid anymore, so we don't need to check for it - the rest of the logic isn't changed | 23:51 |
| *** nwkarsten has quit IRC | 23:51 | |
| *** nadya has joined #openstack-qa | 23:51 | |
| *** nwkarsten has joined #openstack-qa | 23:52 | |
| andreaf | oomichi: the logic was - when using legacy configured credentials just return False on any request for role, as not supported in that configuration, else check if the role is there | 23:52 |
| andreaf | oomichi: so now we just check if the role is there | 23:52 |
| andreaf | oomichi: going to bed now, leave me a comment on the review if you have more questions :) | 23:53 |
| andreaf | oomichi: and thanks for the review | 23:53 |
| oomichi | andreaf: maybe it would be fine if I dig the code more ;) | 23:54 |
| oomichi | s/it/the patch/ | 23:54 |
| *** nadya has quit IRC | 23:56 | |
| openstackrecheck | Console logs not available after 13:31s for gate-tempest-dsvm-postgres-full 326918,15,c2e6119 | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!