| harlowja | maybe tommorow | 00:00 |
|---|---|---|
| jogo | harlowja: so I have a use case | 00:00 |
| jogo | harlowja: https://review.openstack.org/#/c/151056/ | 00:00 |
| jogo | not sure how it fits there | 00:00 |
| harlowja | isn't that the same one as mtreinish ? | 00:00 |
| * harlowja confused, lol | 00:01 | |
| mtreinish | jogo: yeah I'm working on a patch right now adding that case | 00:01 |
| mtreinish | harlowja: heh, we're talking about the same thing. jogo wrote that patch and I -1'd it :) | 00:01 |
| * harlowja less confused now | 00:01 | |
| mtreinish | then I asked him if he could use debtcollector | 00:01 |
| harlowja | kk | 00:01 |
| harlowja | and he's like wtf is that | 00:01 |
| harlowja | hand over your $$ | 00:01 |
| harlowja | lol | 00:01 |
| harlowja | tax time | 00:01 |
| *** ajmiller has quit IRC | 00:04 | |
| jogo | exactly | 00:06 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/tempest: Updated from global requirements https://review.openstack.org/153428 | 00:08 |
| *** andreykurilin_ has joined #openstack-qa | 00:12 | |
| *** abhirc has joined #openstack-qa | 00:16 | |
| *** sabeen has quit IRC | 00:16 | |
| *** Sukhdev has joined #openstack-qa | 00:26 | |
| *** dims__ has quit IRC | 00:31 | |
| *** rfolco has joined #openstack-qa | 00:35 | |
| *** annashen has quit IRC | 00:35 | |
| *** achanda has quit IRC | 00:39 | |
| *** dmorita has joined #openstack-qa | 00:45 | |
| *** toure has quit IRC | 00:46 | |
| *** marun has quit IRC | 00:51 | |
| *** andreaf has quit IRC | 00:52 | |
| *** toure has joined #openstack-qa | 00:53 | |
| *** andreaf has joined #openstack-qa | 00:53 | |
| openstackgerrit | Emily Hugenbruch proposed openstack/tempest: Update compute base test to split up resource_setup https://review.openstack.org/141424 | 00:53 |
| openstackgerrit | Merged openstack-infra/devstack-gate: Bump ironic build/callback timeouts https://review.openstack.org/153200 | 00:54 |
| *** salv-orl_ has joined #openstack-qa | 00:58 | |
| *** itzikb has joined #openstack-qa | 01:00 | |
| *** salv-orlando has quit IRC | 01:01 | |
| *** salv-orl_ has quit IRC | 01:02 | |
| *** rfolco has quit IRC | 01:03 | |
| *** dims__ has joined #openstack-qa | 01:05 | |
| *** dimtruck is now known as zz_dimtruck | 01:11 | |
| *** andreykurilin_ has quit IRC | 01:11 | |
| itzikb | mtreinish: ping | 01:12 |
| *** abhirc has quit IRC | 01:13 | |
| *** weshay has quit IRC | 01:13 | |
| *** salv-orl_ has joined #openstack-qa | 01:14 | |
| *** shashankhegde has joined #openstack-qa | 01:14 | |
| *** toure has quit IRC | 01:18 | |
| *** toure has joined #openstack-qa | 01:24 | |
| *** Yogi1 has joined #openstack-qa | 01:25 | |
| *** weshay has joined #openstack-qa | 01:25 | |
| *** shashankhegde has quit IRC | 01:26 | |
| *** Sukhdev has quit IRC | 01:31 | |
| openstackrecheck | Console logs not available after 13:22s for gate-trove-python27 137452,9,76baf57 | 01:31 |
| *** openstack has joined #openstack-qa | 01:33 | |
| *** melwitt has quit IRC | 01:37 | |
| *** abhirc has joined #openstack-qa | 01:44 | |
| *** alex_xu has quit IRC | 01:44 | |
| *** alex_xu has joined #openstack-qa | 01:46 | |
| *** toure has quit IRC | 01:46 | |
| *** alex_xu has quit IRC | 01:48 | |
| *** mlavalle has quit IRC | 01:49 | |
| *** zz_dimtruck is now known as dimtruck | 01:50 | |
| *** salv-orl_ has quit IRC | 01:53 | |
| *** rwsu is now known as rwsu-afk | 01:53 | |
| *** shashankhegde has joined #openstack-qa | 01:55 | |
| *** Sukhdev has joined #openstack-qa | 02:00 | |
| *** Yogi1 has quit IRC | 02:06 | |
| *** ChuckC_ has joined #openstack-qa | 02:09 | |
| *** fandi has quit IRC | 02:11 | |
| *** ChuckC has quit IRC | 02:13 | |
| *** Yogi1 has joined #openstack-qa | 02:16 | |
| *** dimtruck is now known as zz_dimtruck | 02:19 | |
| *** shashankhegde has quit IRC | 02:20 | |
| *** zz_dimtruck is now known as dimtruck | 02:22 | |
| openstackgerrit | Joe Gordon proposed openstack-dev/grenade: Run javelin2 inside of tempest venv https://review.openstack.org/153081 | 02:22 |
| *** shashankhegde has joined #openstack-qa | 02:23 | |
| *** toure has joined #openstack-qa | 02:26 | |
| openstackgerrit | Joe Gordon proposed openstack-dev/devstack: Stop installing tempest https://review.openstack.org/153080 | 02:26 |
| *** vozcelik has joined #openstack-qa | 02:30 | |
| *** yamahata has quit IRC | 02:46 | |
| *** kan_ has quit IRC | 02:48 | |
| *** harlowja is now known as harlowja_away | 02:48 | |
| *** salv-orlando has joined #openstack-qa | 02:53 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/152814 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1412923, gate-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/1353939 | 02:55 |
| openstack | Launchpad bug 1412923 in devstack "heat api takes too long to come up, tempest tests fail" [Undecided,In progress] - Assigned to Davanum Srinivas (DIMS) (dims-v) | 02:55 |
| openstack | Launchpad bug 1353939 in OpenStack Compute (nova) "Rescue fails with 'Failed to terminate process: Device or resource busy' in the n-cpu log" [High,Confirmed] | 02:55 |
| *** vozcelik has quit IRC | 02:56 | |
| *** ChuckC has joined #openstack-qa | 02:59 | |
| *** ChuckC_ has quit IRC | 02:59 | |
| openstackgerrit | Itzik Brown proposed openstack/tempest: Adding the ability to use pre-created ports in Network scenarios https://review.openstack.org/139979 | 02:59 |
| *** patrickeast has quit IRC | 03:10 | |
| *** toure has quit IRC | 03:11 | |
| *** shashankhegde has quit IRC | 03:12 | |
| *** dims__ has quit IRC | 03:15 | |
| *** achanda has joined #openstack-qa | 03:17 | |
| *** itzikb has quit IRC | 03:21 | |
| *** achanda has quit IRC | 03:23 | |
| *** salv-orlando has quit IRC | 03:25 | |
| openstackgerrit | Al Miller proposed openstack-dev/devstack: Add devstack support for LBaaS Version 2 https://review.openstack.org/153079 | 03:26 |
| *** salv-orlando has joined #openstack-qa | 03:27 | |
| *** melwitt has joined #openstack-qa | 03:28 | |
| *** Yogi1 has quit IRC | 03:28 | |
| *** salv-orlando has quit IRC | 03:34 | |
| *** Longgeek has joined #openstack-qa | 03:36 | |
| *** dimtruck is now known as zz_dimtruck | 03:39 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/153420 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1365046 | 03:42 |
| openstack | Launchpad bug 1365046 in OpenStack-Gate "Job failed due to no devstack directory" [Undecided,Confirmed] | 03:42 |
| *** annashen has joined #openstack-qa | 03:52 | |
| *** Sukhdev has quit IRC | 04:00 | |
| *** shashankhegde has joined #openstack-qa | 04:02 | |
| *** early has quit IRC | 04:04 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack-dev/devstack: ofagent: Vendor code split https://review.openstack.org/149936 | 04:09 |
| *** BharatK has joined #openstack-qa | 04:16 | |
| *** pal has joined #openstack-qa | 04:23 | |
| *** achanda has joined #openstack-qa | 04:23 | |
| *** early has joined #openstack-qa | 04:25 | |
| *** Madan has joined #openstack-qa | 04:28 | |
| *** salv-orlando has joined #openstack-qa | 04:31 | |
| *** catohornet has quit IRC | 04:32 | |
| *** yamamoto_ has joined #openstack-qa | 04:32 | |
| *** early has quit IRC | 04:43 | |
| *** EmilienM|afk has quit IRC | 04:45 | |
| *** EmilienM has joined #openstack-qa | 04:46 | |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Switch to using the skip_because decorator from tempest-lib https://review.openstack.org/153461 | 04:46 |
| *** shashankhegde has quit IRC | 04:47 | |
| *** salv-orlando has quit IRC | 04:48 | |
| *** salv-orlando has joined #openstack-qa | 04:50 | |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Fix import in nova cli tests https://review.openstack.org/153462 | 04:50 |
| *** yamahata has joined #openstack-qa | 04:54 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove CONF values from volume clients https://review.openstack.org/149910 | 05:08 |
| *** achanda has quit IRC | 05:25 | |
| openstackgerrit | Merged openstack/tempest: Remove unused exceptions https://review.openstack.org/148433 | 05:25 |
| *** achanda has joined #openstack-qa | 05:27 | |
| *** salv-orlando has quit IRC | 05:29 | |
| openstackgerrit | Merged openstack-dev/devstack: Remove --skip-redirect form the tgtd service restart https://review.openstack.org/151166 | 05:35 |
| *** harlowja_away has quit IRC | 05:38 | |
| *** annashen has quit IRC | 05:38 | |
| *** annashen has joined #openstack-qa | 05:40 | |
| *** Madan has quit IRC | 05:41 | |
| *** annashen has quit IRC | 05:44 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Update compute base test to split up resource_setup https://review.openstack.org/141424 | 05:44 |
| *** early has joined #openstack-qa | 05:47 | |
| *** early has quit IRC | 05:50 | |
| *** Madan has joined #openstack-qa | 05:55 | |
| *** Madan has quit IRC | 05:55 | |
| *** Madan has joined #openstack-qa | 05:56 | |
| *** early has joined #openstack-qa | 06:06 | |
| *** anshul has joined #openstack-qa | 06:06 | |
| *** early has quit IRC | 06:12 | |
| *** shashankhegde has joined #openstack-qa | 06:14 | |
| *** yfried_ has joined #openstack-qa | 06:22 | |
| *** yfried_ is now known as yfried|afk | 06:22 | |
| *** early has joined #openstack-qa | 06:23 | |
| openstackgerrit | Abhishek G M proposed openstack/tempest: Floating IP test enhancements https://review.openstack.org/132000 | 06:23 |
| *** yfried has quit IRC | 06:25 | |
| *** SridharG has joined #openstack-qa | 06:28 | |
| *** rushiagr_away is now known as rushiagr | 06:29 | |
| *** melwitt has quit IRC | 06:30 | |
| *** openstackgerrit has quit IRC | 06:35 | |
| *** openstackgerrit has joined #openstack-qa | 06:35 | |
| *** salv-orlando has joined #openstack-qa | 06:37 | |
| *** early has quit IRC | 06:43 | |
| *** salv-orlando has quit IRC | 06:48 | |
| *** early has joined #openstack-qa | 06:49 | |
| *** achanda has quit IRC | 06:52 | |
| *** annashen has joined #openstack-qa | 06:52 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Separate token client from identity client https://review.openstack.org/151530 | 06:52 |
| *** afazekas has joined #openstack-qa | 06:57 | |
| *** achanda has joined #openstack-qa | 07:01 | |
| *** k4n0 has joined #openstack-qa | 07:04 | |
| *** pkoniszewski has joined #openstack-qa | 07:04 | |
| *** vigneshvar has joined #openstack-qa | 07:07 | |
| openstackgerrit | Abhishek G M proposed openstack/tempest: Floating IP test enhancements https://review.openstack.org/132000 | 07:14 |
| *** yfried|afk has quit IRC | 07:16 | |
| *** yfried|afk has joined #openstack-qa | 07:16 | |
| *** afazekas has quit IRC | 07:20 | |
| *** Madan has quit IRC | 07:20 | |
| openstackgerrit | Abhishek G M proposed openstack/tempest: Floating IP test enhancements https://review.openstack.org/132000 | 07:25 |
| *** achanda has quit IRC | 07:27 | |
| *** annashen has quit IRC | 07:31 | |
| *** mfalatic has quit IRC | 07:34 | |
| *** bdperkin has quit IRC | 07:34 | |
| *** neeti has joined #openstack-qa | 07:34 | |
| *** Madan has joined #openstack-qa | 07:35 | |
| *** yamahata has quit IRC | 07:36 | |
| *** bdperkin_gone has joined #openstack-qa | 07:47 | |
| *** bdperkin_gone is now known as bdperkin | 07:48 | |
| *** bdperkin has joined #openstack-qa | 07:48 | |
| *** zz_dimtruck is now known as dimtruck | 07:51 | |
| *** dimtruck is now known as zz_dimtruck | 08:16 | |
| *** andreaf has quit IRC | 08:20 | |
| *** andreaf has joined #openstack-qa | 08:20 | |
| *** yamamoto_ has quit IRC | 08:31 | |
| *** luqas has joined #openstack-qa | 08:32 | |
| *** markus_z has joined #openstack-qa | 08:42 | |
| *** jlanoux has joined #openstack-qa | 08:45 | |
| *** mpaolino has joined #openstack-qa | 08:47 | |
| *** ajo has joined #openstack-qa | 08:50 | |
| *** jlanoux_ has joined #openstack-qa | 08:50 | |
| *** jlanoux has quit IRC | 08:53 | |
| *** salv-orlando has joined #openstack-qa | 08:56 | |
| *** jordanP has joined #openstack-qa | 08:58 | |
| *** yamamoto_ has joined #openstack-qa | 08:58 | |
| *** gfidente has joined #openstack-qa | 08:58 | |
| *** gfidente has quit IRC | 08:58 | |
| *** gfidente has joined #openstack-qa | 08:58 | |
| *** jpich has joined #openstack-qa | 08:59 | |
| *** k4n0 has quit IRC | 09:01 | |
| *** yfried has joined #openstack-qa | 09:04 | |
| *** yfried|afk has quit IRC | 09:05 | |
| *** shashankhegde has quit IRC | 09:08 | |
| *** yamamoto_ has quit IRC | 09:09 | |
| *** yamamoto_ has joined #openstack-qa | 09:13 | |
| *** jgrimm is now known as zz_jgrimm | 09:14 | |
| openstackgerrit | Zhang Hua proposed openstack/tempest: VPNaaS Scenario Test https://review.openstack.org/153292 | 09:16 |
| *** safchain has joined #openstack-qa | 09:20 | |
| *** sergsh_away is now known as sergsh | 09:22 | |
| *** andreykurilin_ has joined #openstack-qa | 09:23 | |
| openstackgerrit | Zhang Hua proposed openstack/tempest: VPNaaS Scenario Test https://review.openstack.org/153292 | 09:23 |
| *** salv-orlando has quit IRC | 09:26 | |
| *** jlanoux_ has quit IRC | 09:27 | |
| *** andreaf has quit IRC | 09:28 | |
| *** salv-orlando has joined #openstack-qa | 09:28 | |
| *** andreaf has joined #openstack-qa | 09:29 | |
| *** yamamoto_ has quit IRC | 09:30 | |
| *** neeti has quit IRC | 09:33 | |
| openstackgerrit | Zhang Hua proposed openstack/tempest: VPNaaS Scenario Test https://review.openstack.org/153292 | 09:34 |
| *** mriedem has joined #openstack-qa | 09:35 | |
| *** mriedem_zz has quit IRC | 09:36 | |
| *** e0ne has joined #openstack-qa | 09:41 | |
| *** e0ne has quit IRC | 09:49 | |
| *** e0ne has joined #openstack-qa | 09:50 | |
| *** pkoniszewski has quit IRC | 09:51 | |
| *** pkoniszewski has joined #openstack-qa | 09:51 | |
| *** early has quit IRC | 09:51 | |
| *** jyuso has quit IRC | 09:52 | |
| *** jyuso has joined #openstack-qa | 09:52 | |
| *** tochi has quit IRC | 09:52 | |
| *** jpich has quit IRC | 09:52 | |
| *** vigneshvar has quit IRC | 09:52 | |
| *** SridharG has quit IRC | 09:53 | |
| *** tochi has joined #openstack-qa | 09:53 | |
| sergsh | dkranz, hi, about the patch https://review.openstack.org/137239 - which rename do you mean? thanks | 09:53 |
| sergsh | masayukig, about the patch https://review.openstack.org/137239 - what do you suggest to include in commit message? thanks | 09:53 |
| *** neeti has joined #openstack-qa | 09:54 | |
| *** salv-orlando has quit IRC | 10:00 | |
| *** mpaolino has quit IRC | 10:01 | |
| *** jlanoux has joined #openstack-qa | 10:04 | |
| *** vigneshvar has joined #openstack-qa | 10:05 | |
| *** SridharG has joined #openstack-qa | 10:06 | |
| *** jpich has joined #openstack-qa | 10:06 | |
| *** mpaolino has joined #openstack-qa | 10:13 | |
| *** aix has joined #openstack-qa | 10:16 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Update compute base test to split up resource_setup https://review.openstack.org/141424 | 10:21 |
| *** e0ne is now known as e0ne_ | 10:23 | |
| *** yamamoto has joined #openstack-qa | 10:23 | |
| gmann | ekhugen: mtreinish : andreaf: regarding gate failure for resource cleanup patch - https://review.openstack.org/#/c/141424/29 | 10:24 |
| *** yamamoto has quit IRC | 10:24 | |
| andreaf | gmann: thanks, looking at your comments | 10:24 |
| *** e0ne_ is now known as e0ne | 10:25 | |
| gmann | ekhugen: mtreinish : andreaf: actually issue was to set the appropriate network resources before getting credential | 10:25 |
| gmann | andreaf: just posted new patch, please have a look | 10:25 |
| andreaf | gmann: in principle I don't get how something wrong in the code could cause a failure on the juno run only - unless that test is only executed in the juno run? | 10:25 |
| *** zz_johnthetubagu is now known as johnthetubaguy | 10:25 | |
| andreaf | gmann: thanks, looking at it :) | 10:25 |
| andreaf | gmann: the original failure in test_delete_server is http://logs.openstack.org/24/141424/23/check/check-tempest-dsvm-neutron-full-juno/df22261/logs/testr_results.html.gz | 10:27 |
| gmann | andreaf: yes, that delete server issue was strange for me too. may be fixed something in nova or neutron in master in https://bugs.launchpad.net/tempest/+bug/1357055 | 10:28 |
| openstack | Launchpad bug 1357055 in tempest "Race to delete shared subnet in Tempest neutron full jobs" [Undecided,Confirmed] - Assigned to Salvatore Orlando (salvatore-orlando) | 10:28 |
| andreaf | gmann: that test does request any network resource | 10:28 |
| andreaf | gmann: yes I know that bug :) | 10:28 |
| andreaf | gmann: so if the test requires network resources, how did it pass before? I fail to see how this relates to ekhugen's patch | 10:29 |
| *** andreykurilin_ has quit IRC | 10:31 | |
| gmann | andreaf: tests require resources pass because cls.set_network_resources() was in base class->resource_setup and those tests override those also in same method | 10:32 |
| gmann | means tests require resources was set before credential | 10:32 |
| gmann | but if we put base class credential & network resource setting in set_credential ans leave derived tests network resource setup in resource_setup then it again cause issue | 10:33 |
| andreaf | gmann: oh, I see it finally, the set_network_resources was moved at the end of resource_setup in ekhugen's patch | 10:36 |
| gmann | andreaf: ya | 10:37 |
| andreaf | gmann: but why did that break juno only ? | 10:37 |
| andreaf | gmann: I think that setup_network_resources is a good use case for some pre_resource_setup hook that I had initially in :) | 10:38 |
| andreaf | gmann: or we could even convert that to class attributes | 10:38 |
| gmann | andreaf: juno only failure was for delete server tests which does not require any network resources but due to missing set_network_resources() before cred. where networks got created | 10:38 |
| *** dmorita has quit IRC | 10:39 | |
| gmann | andreaf: but those gets created and cleared in getting/clearing credential method | 10:39 |
| gmann | andreaf: tightly bounded with cred methods, so having those in prehook might not be good | 10:40 |
| andreaf | gmann: ah right because that's in tenant isolation | 10:40 |
| gmann | andreaf: yes | 10:40 |
| andreaf | gmann: we should probably change that | 10:41 |
| andreaf | gmann: tenant isolation should just create the tenant. then in resource_setup we can say tenant_isolation now create resources please | 10:42 |
| gmann | andreaf: ya, even i was thinking same, may be after setting up all cred then ask each tests to set their required network resources | 10:42 |
| openstackgerrit | Kashyap Chamarthy proposed openstack-dev/devstack: guides: Notes to setup DevStack with Nested KVM Virtualization https://review.openstack.org/148630 | 10:42 |
| andreaf | gmann: because the way it is now it's really a mix-up | 10:42 |
| gmann | andreaf: +1 | 10:42 |
| andreaf | gmann: I'll propose a couple of patches for that I think | 10:45 |
| gmann | andreaf: so for pre created account, how we setup network? | 10:45 |
| gmann | andreaf: i forgot that due to all mixup :( | 10:46 |
| andreaf | gmann: because it also fits with work in progress on two other bps, the ssh-auth-strategy, which also requires account bound resources, and the test-accounts-continued, which introduces pre-provisioned resources for pre-provisioned accounts | 10:46 |
| andreaf | gmann: yes that's not implemented yet, it's specified in the bp - but it should be pre-provisioned and specified via YAML file | 10:47 |
| gmann | andreaf: ohk | 10:47 |
| gmann | andreaf: because we should separate net resource from isolated_creds.py | 10:48 |
| gmann | andreaf: and put as class methods as you said earlier | 10:48 |
| gmann | andreaf: may be in base tests class | 10:48 |
| andreaf | gmann: I was thinking that cred_provider could hold those resources, because they are account specific resources anyways | 10:49 |
| *** neeti has quit IRC | 10:49 | |
| andreaf | such resources will be pre-provisioned or created on the fly depending on the type of cred_provider in use | 10:49 |
| gmann | andreaf: ah yes, forgot about they are account specific. in cred_provider it make sense | 10:50 |
| gmann | andreaf: ll you do that in resource cleanup BP or in others 2 you mentioned? | 10:50 |
| andreaf | gmann: but they should not be created when get_credentials is invoked, there should be a dedicated get_credentials_resources method | 10:52 |
| andreaf | gmann: I think I'll do the base setup as part of the resource-cleanup bp, because resource cleanup enforces the split of credentials creation and resources creation | 10:52 |
| gmann | andreaf: yes, something get_credentials_network_resource | 10:53 |
| gmann | andreaf: ya make sense. and till then we need to stop other patches update on that BP | 10:53 |
| *** dims__ has joined #openstack-qa | 10:54 | |
| andreaf | it shouldn't be too complicated I think a patch maybe two, I'll work on it today | 10:54 |
| gmann | andreaf: ok. thanks. I will review those on weekend if possible otherwise on monday | 10:55 |
| *** vrovachev1 has quit IRC | 10:56 | |
| *** amotoki has joined #openstack-qa | 10:56 | |
| gmann | andreaf: gotta go now. have a good day :) | 10:58 |
| andreaf | gmann: and you, thanks | 10:58 |
| andreaf | gmann, thank you for your help on this! | 10:59 |
| gmann | andreaf: np! | 10:59 |
| openstackgerrit | nayna-patel proposed openstack/tempest: Testcase GET/LIST datastore and datastore versions https://review.openstack.org/139953 | 11:00 |
| *** rfolco has joined #openstack-qa | 11:02 | |
| *** cdent has joined #openstack-qa | 11:04 | |
| *** pal has quit IRC | 11:09 | |
| *** yamamoto has joined #openstack-qa | 11:24 | |
| *** yamamoto has quit IRC | 11:29 | |
| *** cdent has quit IRC | 11:33 | |
| *** cdent has joined #openstack-qa | 11:35 | |
| *** gszasz has joined #openstack-qa | 11:36 | |
| *** cdent_ has joined #openstack-qa | 11:41 | |
| *** zigo has quit IRC | 11:42 | |
| *** cdent has quit IRC | 11:42 | |
| *** cdent_ is now known as cdent | 11:42 | |
| *** tochi has quit IRC | 11:48 | |
| *** aysyd has joined #openstack-qa | 11:48 | |
| *** e0ne is now known as e0ne_ | 11:54 | |
| *** cdent has quit IRC | 11:58 | |
| *** e0ne_ is now known as e0ne | 11:59 | |
| *** bdperkin has quit IRC | 12:00 | |
| *** tonyb has quit IRC | 12:00 | |
| *** cdent has joined #openstack-qa | 12:00 | |
| *** tonyb has joined #openstack-qa | 12:01 | |
| *** bdperkin has joined #openstack-qa | 12:04 | |
| *** bdperkin has quit IRC | 12:04 | |
| *** bdperkin has joined #openstack-qa | 12:04 | |
| *** jroll has quit IRC | 12:06 | |
| *** Yogi1 has joined #openstack-qa | 12:08 | |
| *** alex_xu has joined #openstack-qa | 12:09 | |
| *** sileht_ has joined #openstack-qa | 12:10 | |
| *** Yogi1 has quit IRC | 12:14 | |
| *** timello_ has quit IRC | 12:17 | |
| *** ekhugen has quit IRC | 12:17 | |
| *** maurosr has quit IRC | 12:17 | |
| *** ekhugen has joined #openstack-qa | 12:19 | |
| *** jgrimmn has joined #openstack-qa | 12:19 | |
| *** vigneshvar has quit IRC | 12:19 | |
| *** BharatK has quit IRC | 12:19 | |
| *** esmute has quit IRC | 12:19 | |
| *** beisner has quit IRC | 12:19 | |
| *** sweston has quit IRC | 12:19 | |
| *** dougwig has quit IRC | 12:19 | |
| *** comstud has quit IRC | 12:19 | |
| *** sc68cal has quit IRC | 12:19 | |
| *** sileht has quit IRC | 12:19 | |
| *** jvrbanac has quit IRC | 12:19 | |
| *** wendar has quit IRC | 12:19 | |
| *** zz_dimtruck has quit IRC | 12:19 | |
| *** dtroyer has quit IRC | 12:19 | |
| *** cyeoh has quit IRC | 12:19 | |
| *** rossella_s has quit IRC | 12:19 | |
| *** zz_jgrimm has quit IRC | 12:19 | |
| *** sileht_ is now known as sileht | 12:19 | |
| *** maurosr has joined #openstack-qa | 12:19 | |
| *** timello has joined #openstack-qa | 12:20 | |
| *** jroll has joined #openstack-qa | 12:21 | |
| *** vigneshvar has joined #openstack-qa | 12:21 | |
| *** BharatK has joined #openstack-qa | 12:21 | |
| *** esmute has joined #openstack-qa | 12:21 | |
| *** beisner has joined #openstack-qa | 12:21 | |
| *** sweston has joined #openstack-qa | 12:21 | |
| *** dougwig has joined #openstack-qa | 12:21 | |
| *** comstud has joined #openstack-qa | 12:21 | |
| *** sc68cal has joined #openstack-qa | 12:21 | |
| *** jvrbanac has joined #openstack-qa | 12:21 | |
| *** wendar has joined #openstack-qa | 12:21 | |
| *** zz_dimtruck has joined #openstack-qa | 12:21 | |
| *** dtroyer has joined #openstack-qa | 12:21 | |
| *** cyeoh has joined #openstack-qa | 12:21 | |
| *** rossella_s has joined #openstack-qa | 12:21 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/tempest: Updated from global requirements https://review.openstack.org/153428 | 12:24 |
| *** aysyd has quit IRC | 12:27 | |
| *** aysyd has joined #openstack-qa | 12:27 | |
| *** sileht has quit IRC | 12:28 | |
| *** sileht has joined #openstack-qa | 12:29 | |
| *** jaypipes has joined #openstack-qa | 12:31 | |
| *** ChanServ sets mode: +o jaypipes | 12:31 | |
| *** rfolco has quit IRC | 12:31 | |
| *** rfolco has joined #openstack-qa | 12:33 | |
| *** sergsh has quit IRC | 12:34 | |
| *** rfolco has quit IRC | 12:35 | |
| *** rfolco has joined #openstack-qa | 12:36 | |
| *** gszasz has quit IRC | 12:41 | |
| openstackgerrit | Sean Dague proposed openstack/qa-specs: Add devstack external plugin spec https://review.openstack.org/137054 | 12:41 |
| *** jhenner has joined #openstack-qa | 12:47 | |
| openstackgerrit | Christian Berendt proposed openstack-dev/hacking: Add hacking check H106: Use the correct capitalization of OpenStack https://review.openstack.org/153547 | 12:58 |
| *** early has joined #openstack-qa | 13:01 | |
| *** early has quit IRC | 13:08 | |
| *** rushiagr is now known as rushiagr_away | 13:10 | |
| *** dims__ has quit IRC | 13:17 | |
| openstackgerrit | Al Miller proposed openstack-dev/devstack: Add devstack support for LBaaS Version 2 https://review.openstack.org/153079 | 13:20 |
| *** dims_ has joined #openstack-qa | 13:25 | |
| *** bknudson has quit IRC | 13:28 | |
| *** Madan has quit IRC | 13:32 | |
| *** pal has joined #openstack-qa | 13:39 | |
| openstackrecheck | openstack/glance change: https://review.openstack.org/118775 failed gate-tempest-dsvm-neutron-full in the gate queue with an unrecognized error | 13:41 |
| *** pkoniszewski has quit IRC | 13:44 | |
| *** vigneshvar has quit IRC | 13:47 | |
| *** mwagner_lap has quit IRC | 13:48 | |
| *** cbader has quit IRC | 13:49 | |
| *** rushiagr_away is now known as rushiagr | 13:50 | |
| *** andreaf has quit IRC | 13:56 | |
| *** andreaf has joined #openstack-qa | 13:57 | |
| *** ajo has quit IRC | 13:57 | |
| *** ajo has joined #openstack-qa | 13:57 | |
| openstackgerrit | Merged openstack/qa-specs: meta data and uuid for tests https://review.openstack.org/144329 | 13:59 |
| *** jroll has quit IRC | 14:03 | |
| *** jroll has joined #openstack-qa | 14:03 | |
| *** jaypipes is now known as leakypipes | 14:07 | |
| *** dttocs has joined #openstack-qa | 14:07 | |
| *** dttocs has quit IRC | 14:08 | |
| *** Yogi1 has joined #openstack-qa | 14:08 | |
| *** dttocs has joined #openstack-qa | 14:08 | |
| *** dustins has joined #openstack-qa | 14:09 | |
| *** dttocs has left #openstack-qa | 14:15 | |
| *** jlibosva has joined #openstack-qa | 14:21 | |
| *** obutenko has joined #openstack-qa | 14:25 | |
| *** yamamoto has joined #openstack-qa | 14:27 | |
| *** jlibosva has quit IRC | 14:28 | |
| *** yamamoto_ has joined #openstack-qa | 14:30 | |
| *** yamamoto has quit IRC | 14:34 | |
| *** dttocs has joined #openstack-qa | 14:37 | |
| *** bknudson has joined #openstack-qa | 14:39 | |
| *** ChuckC has quit IRC | 14:52 | |
| *** alex_xu has quit IRC | 14:52 | |
| *** xu_alex has joined #openstack-qa | 14:53 | |
| *** zz_dimtruck is now known as dimtruck | 14:55 | |
| *** markus_z has quit IRC | 14:55 | |
| *** SridharG has left #openstack-qa | 15:00 | |
| *** ajo has quit IRC | 15:02 | |
| *** ajo has joined #openstack-qa | 15:02 | |
| *** Ananci has joined #openstack-qa | 15:03 | |
| *** yamamoto_ has quit IRC | 15:04 | |
| *** dimtruck is now known as zz_dimtruck | 15:05 | |
| *** jecarey has joined #openstack-qa | 15:06 | |
| mtreinish | ekhugen, gmann, andreaf: I left a review comment on https://review.openstack.org/#/c/141424/ that one test is failing now because networks aren't being configured for that test class | 15:07 |
| ekhugen | mtreinish I saw that thanks | 15:08 |
| mtreinish | so when it starts a guest it doesn't have any networking and therefore no address | 15:08 |
| *** dansmith is now known as superdan | 15:08 | |
| *** yamahata has joined #openstack-qa | 15:12 | |
| *** Yogi1 has quit IRC | 15:14 | |
| *** rushiagr is now known as rushiagr_away | 15:15 | |
| *** Yogi1 has joined #openstack-qa | 15:16 | |
| *** jgrimmn is now known as zz_jgrimmn | 15:21 | |
| *** yamahata has quit IRC | 15:23 | |
| *** zz_dimtruck is now known as dimtruck | 15:25 | |
| openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: WIP: testing path for doing external plugins in d-g https://review.openstack.org/153597 | 15:27 |
| mtreinish | sdague: ^^^ won't that fail because fail on clone is enabled? | 15:29 |
| sdague | mtreinish: probably, I want to figure out what's available | 15:29 |
| *** rbak has joined #openstack-qa | 15:30 | |
| mtreinish | sdague: well it should already be on the local disk on the slave in /opt/git/stackforge/ec2-api, does enable_plugin work with a local path? | 15:31 |
| mtreinish | although I guess it's not up to date so that's probably not a good idea to just point to the repo synced during image create | 15:32 |
| openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: WIP: testing path for doing external plugins in d-g https://review.openstack.org/153597 | 15:33 |
| sdague | so... that might actually do the right thing | 15:33 |
| *** anshul has quit IRC | 15:33 | |
| *** lordd has joined #openstack-qa | 15:37 | |
| *** marun has joined #openstack-qa | 15:39 | |
| *** sabeen1 has joined #openstack-qa | 15:40 | |
| *** hemna_ is now known as hemna | 15:40 | |
| *** hemna is now known as chewbaca | 15:41 | |
| *** chewbaca is now known as chewbacca | 15:41 | |
| *** chewbacca is now known as os-brick | 15:41 | |
| mtreinish | sdague: on https://review.openstack.org/#/c/153462/ do you have any idea why it's not failing everywhere? | 15:42 |
| mtreinish | before that patch I mean | 15:42 |
| *** os-brick is now known as flyfisher | 15:42 | |
| *** salv-orlando has joined #openstack-qa | 15:43 | |
| sdague | I think because that top level construct imported by being dragged in by the lower one | 15:43 |
| openstackgerrit | Maru Newby proposed openstack-dev/devstack: Add postgresql dev package as testonly neutron dep https://review.openstack.org/153091 | 15:46 |
| *** adalbas has joined #openstack-qa | 15:49 | |
| openstackgerrit | Emily Hugenbruch proposed openstack/tempest: Update compute base test to split up resource_setup https://review.openstack.org/141424 | 15:51 |
| andreaf | mtreinish, ekhugen: I discussed with gmann earlier that this problem is related to the fact that on one side we split credentials creation and resource setup, while in tenant isolation we create resources as part of the get_credentials call | 15:53 |
| andreaf | mtreinish, ekhugen: so I'm working on a couple of patches to split that, I think they'll make our life easier for the rest of the bp | 15:53 |
| ekhugen | andreaf okay, I'll see if the patch I submitted fixes the issues with compute tests | 15:54 |
| mtreinish | andreaf: well only if neutron is enabled, because if we didn't there wouldn't be any networking by default | 15:54 |
| mtreinish | but I think refactoring that side will be necessary long term even if just for test-accounts part2 | 15:55 |
| andreaf | mtreinish: exactly | 15:55 |
| andreaf | mtreinish: and it will come in handy for the ssh-auth bp as well were we need to setup validation resources (even with n-net in that case) | 15:56 |
| *** yamahata has joined #openstack-qa | 15:58 | |
| mtreinish | andreaf: although all I had envisioned was extending the public methods from isolated_creds.py to also exist for accounts.py and the abstract class for test-accounts part2 | 15:59 |
| mtreinish | which wouldn't have been too difficult | 15:59 |
| mtreinish | but it might not have been that simple now that I think about it... | 15:59 |
| *** pal has quit IRC | 16:02 | |
| sdague | mtreinish: https://jenkins07.openstack.org/job/gate-tempest-dsvm-nova-v21-full/79/console Conflict on isolated creds? | 16:03 |
| *** Yogi11 has joined #openstack-qa | 16:05 | |
| *** yamamoto has joined #openstack-qa | 16:05 | |
| mtreinish | sdague: that's what that stacktrace looks like. But that seems really unlikely we use a randomly generated suffix with the testclass as the prefix | 16:05 |
| *** rfolco has quit IRC | 16:05 | |
| mtreinish | I'll take a look at the keystone logs | 16:05 |
| *** Yogi1 has quit IRC | 16:07 | |
| andreaf | mtreinish: this is how my extended cred provider looks like for now: http://paste.openstack.org/show/168185/ | 16:07 |
| *** david-lyle_afk is now known as david-lyle | 16:07 | |
| *** mpaolino has quit IRC | 16:07 | |
| andreaf | mtreinish: we must pass the required resources at __init__ time because in case of pre-provisioned accounts we must match them as well before we can allocate an account | 16:08 |
| andreaf | mtreinish: but in case of tenant isolation, I only create resources when directly asked to | 16:08 |
| sdague | I think you are racing on creating roles | 16:08 |
| openstackgerrit | John Villalovos proposed openstack-dev/devstack: Use 'https' protocol by default as easier for proxies https://review.openstack.org/153397 | 16:09 |
| *** rfolco has joined #openstack-qa | 16:09 | |
| sdague | mtreinish: https://bugs.launchpad.net/tempest/+bug/1419043 | 16:10 |
| openstack | Launchpad bug 1419043 in tempest "Conflict on isolated credential setup" [High,New] | 16:10 |
| sdague | mtreinish: yeh | 16:10 |
| sdague | 2015-02-06 15:27:17.980468 25961 INFO keystone.assignment.core [-] Creating the default role 9fe2ff9ee4384b1894a90878d3e92bab because it does not exist. | 16:10 |
| sdague | 2015-02-06 15:27:17.982766 25959 INFO keystone.assignment.core [-] Creating the default role 9fe2ff9ee4384b1894a90878d3e92bab because it does not exist. | 16:10 |
| sdague | 2015-02-06 15:27:17.988287 25959 DEBUG keystone.common.sql.core [-] Conflict role: (IntegrityError) (1062, "Duplicate entry '9fe2ff9ee4384b1894a90878d3e92bab' for key 'PRIMARY'") 'INSERT INTO role (id, name, extra) VALUES (%s, %s, %s)' ('9fe2ff9ee4384b1894a90878d3e92bab', '_member_', '{}') wrapper /opt/stack/new/keystone/keystone/common/sql/core.py:399 | 16:10 |
| sdague | 2015-02-06 15:27:17.988978 25959 WARNING keystone.common.wsgi [-] Conflict occurred attempting to store role - Duplicate Entry | 16:10 |
| *** yamamoto has quit IRC | 16:10 | |
| mtreinish | sdague: but that was returned on the user create not the role assign | 16:10 |
| *** shashankhegde has joined #openstack-qa | 16:10 | |
| mtreinish | sdague: oh, it's because we stopped assigning that role in devstack | 16:10 |
| mtreinish | I saw that patch from jamielennox go through recently | 16:11 |
| sdague | you try to create it twice at the same time | 16:11 |
| mtreinish | sdague: it's being done under the covers | 16:11 |
| mtreinish | I think | 16:11 |
| sdague | by keystone or tempest? | 16:11 |
| mtreinish | keystone | 16:11 |
| mtreinish | tenant isolation doesn't create a default role | 16:12 |
| mtreinish | it'll only assign roles to a user specified from config (for either admin or ResellerAdmin) | 16:13 |
| sdague | ok | 16:13 |
| sdague | adding keystone as critical there, maybe morganfainberg can take a look | 16:13 |
| *** salv-orlando has quit IRC | 16:13 | |
| openstackgerrit | David Kranz proposed openstack/tempest: Fix stress test which was not changed to receive one client value https://review.openstack.org/153619 | 16:13 |
| *** salv-orlando has joined #openstack-qa | 16:13 | |
| *** rfolco has quit IRC | 16:15 | |
| mtreinish | oh the patch I was thinking from jamielennox was different and it hasn't merged yet | 16:15 |
| *** Yogi11 has quit IRC | 16:15 | |
| *** Yogi1 has joined #openstack-qa | 16:15 | |
| openstackgerrit | Merged openstack-dev/devstack: Remove the default project from all users https://review.openstack.org/150667 | 16:16 |
| *** rfolco has joined #openstack-qa | 16:16 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Drop the legacy and un-used _interface https://review.openstack.org/153621 | 16:18 |
| *** toure_ has joined #openstack-qa | 16:21 | |
| *** openstackgerrit has quit IRC | 16:21 | |
| *** openstackgerrit has joined #openstack-qa | 16:22 | |
| mtreinish | sdague: yeah there's a race in the keystone code I think: http://git.openstack.org/cgit/openstack/keystone/tree/keystone/assignment/core.py#n236 if 2 users are created at roughly the same time 2 create roles will be created with the same id | 16:22 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/153426 failed gate-tempest-dsvm-nova-v21-full in the gate queue with an unrecognized error | 16:22 |
| mtreinish | I'll push a patch adding a lock around the role create which should fix it | 16:22 |
| sdague | mtreinish: seems like a core keystone bug though | 16:24 |
| sdague | I suppose you could also put a retry loop | 16:25 |
| sdague | mtreinish: you should add to the keystone bug | 16:25 |
| ekhugen | andreaf is https://review.openstack.org/153621 for all the instances of _interface=json? there are a lot missing from the patch, like the database tests | 16:25 |
| andreaf | ekhugen: thanks - yes I was too quick in pushing it up - I'm fixing it | 16:27 |
| ekhugen | oh okay andreaf thanks! | 16:28 |
| openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: WIP: testing path for doing external plugins in d-g https://review.openstack.org/153597 | 16:28 |
| *** pal has joined #openstack-qa | 16:31 | |
| *** lordd has quit IRC | 16:32 | |
| morganfainberg | sdague: will look at that shortly | 16:32 |
| sdague | morganfainberg: also... default role uuid is a const? | 16:34 |
| *** dttocs has quit IRC | 16:34 | |
| morganfainberg | The "member" thing? | 16:35 |
| *** dttocs has joined #openstack-qa | 16:35 | |
| morganfainberg | Not sure what you're asking. | 16:35 |
| sdague | [-] Creating the default role 9fe2ff9ee4384b1894a90878d3e92bab because it does not exist. | 16:36 |
| sdague | also.... apache logs for keystone are not indexed in logstash? | 16:36 |
| sdague | 9fe2ff9ee4384b1894a90878d3e92bab seems to always be the default role | 16:36 |
| *** SridharG has joined #openstack-qa | 16:36 | |
| morganfainberg | It is defined in the keystone config. It is the "member" role | 16:37 |
| morganfainberg | It solved some migration thing I am not sure what at some point (maybe a chicken / egg) | 16:38 |
| sdague | ok, well there is definitely a race in default construction which assumes that keystone is running with 1 worker | 16:38 |
| morganfainberg | sdague: the Apache logs should get indexed. | 16:38 |
| sdague | they don't seem to be | 16:38 |
| *** dims_ is now known as dimsum__ | 16:39 | |
| mtreinish | morganfainberg: I updated the bug with where the race is, I'll push a fix after lunch | 16:39 |
| morganfainberg | Ok I think i know the race now | 16:39 |
| morganfainberg | It is simply missing a try/except in case the role was created. | 16:39 |
| *** mwagner_lap has joined #openstack-qa | 16:40 | |
| morganfainberg | mtreinish: just took me a moment to see what was going on. | 16:40 |
| sdague | yeh, none of the apache logs are getting indexed | 16:41 |
| morganfainberg | This means horizon logs are likely not indexed | 16:41 |
| sdague | it looks like there was an fs change in devstack-gate and nothing changed in the indexer | 16:41 |
| *** mlavalle has joined #openstack-qa | 16:42 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/153251 failed gate-tempest-dsvm-neutron-full in the gate queue with an unrecognized error | 16:42 |
| *** annashen has joined #openstack-qa | 16:46 | |
| *** annashen has quit IRC | 16:47 | |
| sdague | morganfainberg: also... http://logs.openstack.org/26/153426/2/gate/gate-tempest-dsvm-nova-v21-full/c80cf2c//logs/apache/keystone.txt.gz#_2015-02-06_15_27_17_988 - would be *really* nice to say what role failed to be created in the log message | 16:48 |
| morganfainberg | sdague: ++ yes | 16:49 |
| morganfainberg | I know what the bug is. | 16:49 |
| *** ylobankov has quit IRC | 16:51 | |
| *** gszasz has joined #openstack-qa | 16:51 | |
| *** ylobankov has joined #openstack-qa | 16:51 | |
| *** marun has quit IRC | 16:51 | |
| sdague | hmmm... it looks like the conflict stuff is so wrapped up in decorators that getting a more specific message might be hard | 16:52 |
| *** dhellmann_ has quit IRC | 16:53 | |
| *** anshul has joined #openstack-qa | 16:54 | |
| *** dhellmann_ has joined #openstack-qa | 16:54 | |
| *** dhellmann_ has quit IRC | 16:56 | |
| *** shashankhegde has quit IRC | 16:56 | |
| *** dhellmann_ has joined #openstack-qa | 16:57 | |
| *** jlanoux has quit IRC | 16:58 | |
| *** pkoniszewski has joined #openstack-qa | 16:58 | |
| *** ChuckC has joined #openstack-qa | 16:58 | |
| *** dhellmann_ has quit IRC | 16:59 | |
| *** vigneshvar has joined #openstack-qa | 16:59 | |
| *** cbader has joined #openstack-qa | 16:59 | |
| *** dhellmann_ has joined #openstack-qa | 16:59 | |
| sdague | hmmm.... also... other logging here is really weird. | 17:00 |
| sdague | I'm going to throw some patches post lunch, and you can yell at me :) | 17:00 |
| *** Nic has joined #openstack-qa | 17:02 | |
| morganfainberg | sdague: hehe ok | 17:02 |
| sdague | like, I mean - http://logs.openstack.org/26/153426/2/gate/gate-tempest-dsvm-nova-v21-full/c80cf2c//logs/apache/keystone.txt.gz#_2015-02-06_15_34_03_735 | 17:02 |
| sdague | so that's some args, but no reference to what the actual request was | 17:03 |
| mtreinish | morganfainberg: oh, is that your prefered fix catch the duplicate exception and ignore the failure. I just assumed lock to prevent racing db inserts | 17:04 |
| morganfainberg | mtreinish, that is likely the best bet | 17:05 |
| *** pkoniszewski has quit IRC | 17:05 | |
| mtreinish | also isn't the exception that will be raised going to be backend specific? (you guys still have an ldap backend right) | 17:05 |
| *** Yogi1 has quit IRC | 17:07 | |
| *** safchain has quit IRC | 17:07 | |
| morganfainberg | mtreinish, it should be the same exception - conflict | 17:08 |
| *** rwsu-afk is now known as rwsu | 17:09 | |
| morganfainberg | mtreinish, i will take a look in a few, on a call right now and I need to run a quick errand this morning. If sdague or you have a patch i can get that through or i can have something probably to land today in either case. | 17:11 |
| * morganfainberg will see if we can get keystone team on it before i can look at it directly | 17:12 | |
| *** gszasz has quit IRC | 17:13 | |
| *** vigneshvar has quit IRC | 17:14 | |
| *** salv-orlando has quit IRC | 17:16 | |
| *** abhirc has quit IRC | 17:17 | |
| *** jyuso has quit IRC | 17:18 | |
| *** jyuso has joined #openstack-qa | 17:18 | |
| *** heyongli has quit IRC | 17:18 | |
| *** tonyb has quit IRC | 17:19 | |
| morganfainberg | mtreinish, sdague: https://github.com/openstack/keystone/blob/master/keystone/assignment/core.py#L256-L261 just needs a try/except around it talking with keystone folks about it | 17:19 |
| *** tonyb has joined #openstack-qa | 17:20 | |
| *** rwsu has quit IRC | 17:20 | |
| *** heyongli has joined #openstack-qa | 17:23 | |
| *** nicodemos has joined #openstack-qa | 17:23 | |
| *** early has joined #openstack-qa | 17:23 | |
| *** zz_jgrimmn is now known as jgrimm | 17:23 | |
| *** Yogi1 has joined #openstack-qa | 17:24 | |
| *** e0ne has quit IRC | 17:24 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Drop the legacy and un-used _interface https://review.openstack.org/153621 | 17:28 |
| *** jpich has quit IRC | 17:28 | |
| *** marun has joined #openstack-qa | 17:28 | |
| andreaf | mtreinish, ekhugen ^^^ https://review.openstack.org/#/c/153621/ - is that review-able? It's tiny changes but 90 files affected... | 17:29 |
| *** shashankhegde has joined #openstack-qa | 17:32 | |
| *** obutenko has quit IRC | 17:34 | |
| *** pal has quit IRC | 17:35 | |
| *** Nic has quit IRC | 17:35 | |
| *** mfalatic has joined #openstack-qa | 17:36 | |
| *** vigneshvar has joined #openstack-qa | 17:37 | |
| *** shashankhegde has quit IRC | 17:42 | |
| *** rfolco has quit IRC | 17:43 | |
| *** bnemec is now known as beekneemech | 17:44 | |
| *** rfolco has joined #openstack-qa | 17:44 | |
| *** ChuckC has quit IRC | 17:46 | |
| *** rwsu has joined #openstack-qa | 17:49 | |
| *** salv-orlando has joined #openstack-qa | 17:49 | |
| *** jordanP has quit IRC | 17:51 | |
| *** luqas has quit IRC | 17:51 | |
| *** achanda has joined #openstack-qa | 17:51 | |
| *** annashen has joined #openstack-qa | 17:56 | |
| *** early has quit IRC | 18:00 | |
| dtroyer | sdague: looking at the e-s package bits for devstack got me thinking about the needs for services in general and I started doing it out loud at the bottom of https://etherpad.openstack.org/p/devstack-package-install-interface. Have a look, see if I'm straying too far downstream. | 18:01 |
| *** ChuckC has joined #openstack-qa | 18:02 | |
| *** anshul has quit IRC | 18:02 | |
| *** early has joined #openstack-qa | 18:06 | |
| openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/152959 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1285323 | 18:08 |
| openstack | Launchpad bug 1285323 in grenade "Services fail to shut down on the old side of Grenade" [Undecided,In progress] - Assigned to Sean Dague (sdague) | 18:08 |
| *** Nic has joined #openstack-qa | 18:09 | |
| *** patrickeast has joined #openstack-qa | 18:09 | |
| *** xavileon has quit IRC | 18:10 | |
| ekhugen | andreaf I think that's reasonable, it would be weird to try to break it up, I think | 18:14 |
| *** flyfisher is now known as hemna | 18:14 | |
| *** mlavalle has quit IRC | 18:14 | |
| *** annashen has quit IRC | 18:14 | |
| *** harlowja has joined #openstack-qa | 18:15 | |
| *** pal has joined #openstack-qa | 18:17 | |
| *** Ananci has quit IRC | 18:17 | |
| *** gyee has joined #openstack-qa | 18:18 | |
| gyee | sdague, mtrienish is there a bug filed for the keystone member role creation race condition? | 18:20 |
| *** early has quit IRC | 18:20 | |
| *** anshul has joined #openstack-qa | 18:24 | |
| *** shashankhegde has joined #openstack-qa | 18:25 | |
| *** e0ne has joined #openstack-qa | 18:26 | |
| *** early has joined #openstack-qa | 18:38 | |
| *** jogo is now known as flashgordon | 18:40 | |
| openstackgerrit | Wayne Okuma proposed openstack-dev/devstack: Adds elasticsearch support as a pre-req for the glance-index. https://review.openstack.org/149442 | 18:41 |
| openstackgerrit | Wayne Okuma proposed openstack-dev/devstack: implement elasticsearch pkg for devstack https://review.openstack.org/153048 | 18:41 |
| mtreinish | andreaf: well it seems to break things | 18:46 |
| *** pal has quit IRC | 18:47 | |
| sdague | gyee: https://bugs.launchpad.net/tempest/+bug/1419043 | 18:49 |
| openstack | Launchpad bug 1419043 in Keystone "Conflict on isolated credential setup" [Critical,Confirmed] - Assigned to Matthew Treinish (treinish) | 18:49 |
| openstackgerrit | David Kranz proposed openstack/tempest: Change keypair client to return one value and update tests https://review.openstack.org/153652 | 18:49 |
| gyee | sdague, thanks, I just created another bug in Keystone, let me mark it as dup | 18:49 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/147961 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 18:52 |
| *** abhirc has joined #openstack-qa | 18:52 | |
| *** early has quit IRC | 18:53 | |
| *** dhellmann has quit IRC | 18:58 | |
| *** dhellmann_ is now known as dhellmann | 18:58 | |
| *** annashen has joined #openstack-qa | 18:58 | |
| *** dhellmann_ has joined #openstack-qa | 18:59 | |
| *** dhellmann_ has quit IRC | 19:01 | |
| *** dhellmann_ has joined #openstack-qa | 19:01 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 19:03 | |
| mtreinish | morganfainberg: on https://review.openstack.org/#/c/153656 I'm looking for where to add a unit test to add this case, but I'm not sure where to put it | 19:04 |
| mtreinish | the tests dir in keystone is kinda confusing | 19:04 |
| mtreinish | any suggestions | 19:04 |
| bknudson | I don't think we've found a way to do testing for race conditions. | 19:06 |
| mtreinish | bknudson: I was just going to mock the driver call to raise an exception | 19:06 |
| mtreinish | there isn't a good way to simulate the race in the unit test | 19:06 |
| bknudson | mtreinish: I think the test will be in test_v3_assignment. | 19:07 |
| *** boden has joined #openstack-qa | 19:07 | |
| mtreinish | bknudson: ok thanks, I'll add it in there somewhere | 19:08 |
| *** yamamoto has joined #openstack-qa | 19:08 | |
| *** dhellmann has quit IRC | 19:09 | |
| *** dhellmann_ is now known as dhellmann | 19:09 | |
| *** yamamoto has quit IRC | 19:13 | |
| *** dustins has quit IRC | 19:13 | |
| mtreinish | bknudson: do all of the keystone unit tests go through the api? that seems weird to me | 19:13 |
| bknudson | mtreinish: not all of them... there's also tests in test_backend.py | 19:14 |
| bknudson | but I think those are supposed to be for testing the backend drivers (e.g., sql vs ldap) | 19:14 |
| *** marrusl has quit IRC | 19:15 | |
| andreaf | mtreinish: uhm yes I missed the verify_tempest_config bit | 19:15 |
| mtreinish | bknudson: ok then I'll put write it as going through the api, it just seems like it could be lower level than that but whatever | 19:15 |
| mtreinish | andreaf: ah, as much as I think we should be using that as part of setup in devstack, running it has been good for verifying we don't break it | 19:18 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Drop the legacy and un-used _interface https://review.openstack.org/153621 | 19:20 |
| *** dustins has joined #openstack-qa | 19:22 | |
| *** aysyd has quit IRC | 19:24 | |
| *** aysyd has joined #openstack-qa | 19:24 | |
| *** sergsh has joined #openstack-qa | 19:25 | |
| *** cdent has quit IRC | 19:25 | |
| *** e0ne has quit IRC | 19:28 | |
| *** nicodemos has quit IRC | 19:29 | |
| andreaf | mtreinish: I did a quick survey of the base classes for the various api and scenario tests, to check what kind of credentials they setup: http://paste.openstack.org/show/168269/ | 19:31 |
| sdague | dtroyer: you want to just approve the FUTURE.rst ? | 19:31 |
| andreaf | mtreinish: baremetal, identity and orchestration tests use pre-provisioned credentials always - do you know if that's on purpose for all three categories? | 19:32 |
| *** vigneshvar has quit IRC | 19:33 | |
| *** vigneshvar has joined #openstack-qa | 19:34 | |
| mtreinish | andreaf: so keystone tests being all preprovisioned makes sense | 19:34 |
| mtreinish | but I'm not sure why it's the case on ironic or heat | 19:34 |
| andreaf | mtreinish: I was thinking of moving the initial allocation of credentials up to the base class, because it would make things much simpler for the rest of the resource-cleanup bp | 19:35 |
| andreaf | mtreinish: so to handle those three I would need something like a "force_preprovisioned_account" to be set in the base class | 19:36 |
| sergsh | mtreinish: hi, about https://review.openstack.org/#/c/137239/ - what can I improve in commit message there? | 19:36 |
| sergsh | dkranz, can you please point me to renamed tests? https://review.openstack.org/#/c/137239/ | 19:37 |
| *** andreykurilin_ has joined #openstack-qa | 19:38 | |
| mtreinish | andreaf: so you don't actually have to I think they should all work with isolated creds | 19:38 |
| mtreinish | for keystone they just weren't because they're all admin tests and they didn't really need the separation | 19:38 |
| dkranz | sergsh: It is this part "Removing JSON name from classes in test_networks" | 19:39 |
| andreaf | mtreinish: right that was my thinking but I wasn't sure if it was worth making a change there | 19:39 |
| dkranz | sergsh: The class name is part of the name of the test | 19:39 |
| andreaf | mtreinish: it would look cleaner, for sure, but it might break things | 19:39 |
| sergsh | dkranz, ok, thanks, I'll revert it back | 19:39 |
| andreaf | mtreinish: also I wanted to give base classes (and tests) the ability to request certain credentials, and to leave the job of actually getting them to the overall base test class | 19:40 |
| mtreinish | andreaf: well it definitely shouldn't be an issue for the keystone tests, ironic is weird because it doesn't really have a concept of multi-tenancy. But i imagine if you use the same permissions level it wouldn't cause an issue | 19:41 |
| openstackrecheck | openstack/glance change: https://review.openstack.org/118775 failed gate-tempest-dsvm-neutron-full in the gate queue with an unrecognized error | 19:41 |
| mtreinish | I'm not sure what the deal is with the heat tests though. I assumed they were using isolated creds | 19:41 |
| andreaf | mtreinish: ok I will try and see if it works | 19:41 |
| mtreinish | sergsh: the issue with the commit message is it kinda of goes against the logical split of patches we expect | 19:42 |
| mtreinish | you're just listing a lot of individual changes you made, which would normally be a bunch of small commits | 19:43 |
| andreaf | mtreinish: the easier way would be to setup user, alt and admin for all, but because we setup nw resources for each credentials, it would mean allocate more networks and so | 19:43 |
| andreaf | mtreinish: the more complex way is to allow base classes and tests to request credentials - I was wondering whether to use a list at class level or an helper function to be invoked | 19:43 |
| mtreinish | sergsh: if you wrote it to explain you're doing a refactor and why all the cleanups and improvements are interelated and need to be in one patch it would be clearer | 19:44 |
| *** achanda has quit IRC | 19:44 | |
| mtreinish | because having a long list of items makes someone ask why is this 1 patch | 19:44 |
| sergsh | mtreinish, you're right, it's inter-related, but I try to figure out how to write it :) | 19:45 |
| *** annashen has quit IRC | 19:45 | |
| mtreinish | andreaf: the underlying issue is how neutron differs by default from n-net flat from the user perspective | 19:45 |
| mtreinish | which is why all of the network provisioning stuff is in isolated creds | 19:45 |
| *** SridharG has left #openstack-qa | 19:45 | |
| mtreinish | because each tenant needs its own separate network to be useable, so we dumped it in tenant isolation to keep the mapping automatic | 19:46 |
| sergsh | mtreinish, maybe so? "Converting all required functions in network tests to full transparent IPv6 support, removing duplicates and adding documentation to them" ? it's high-level of changes | 19:47 |
| mtreinish | sergsh: yeah that's a good start. Better than just listing all the individual pieces | 19:49 |
| sergsh | cool! then I change it, thanks | 19:49 |
| *** early has joined #openstack-qa | 19:50 | |
| openstackgerrit | Andrey Pavlov proposed openstack-dev/devstack: Make EC2 compatible with current AWS CLI. https://review.openstack.org/153669 | 19:51 |
| *** early has quit IRC | 19:53 | |
| *** toure_ has quit IRC | 19:53 | |
| *** achanda has joined #openstack-qa | 19:53 | |
| sergsh | dkranz, and what with new classes in tests? I shouldn't call them SomeClassJSON, right? | 19:53 |
| morganfainberg | mtreinish: +2 on that fix. Tests are hard in that case. You could mock the get to return a notfound once and then the create would check again. | 19:54 |
| mtreinish | morganfainberg: yeah I was looking at tests, but it's too much effort to figure out where to put them and it would just be a mock, so I'm ok with skipping it if you are | 19:55 |
| morganfainberg | Yep. Good with skipping the test here. | 19:55 |
| openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: Diagram the nova net multinode network setup https://review.openstack.org/153672 | 19:55 |
| dkranz | sergsh: right | 19:56 |
| *** anshul has quit IRC | 19:57 | |
| mtreinish | clarkb: ^^^ that's kinda hard to read, maybe a real image would be better? Although I'd hate not being able to read it in a terminal | 19:57 |
| clarkb | mtreinish: real image? boo | 19:57 |
| mtreinish | or maybe it's just how gerrit displays it, let me pull it | 19:57 |
| openstackgerrit | Sergey Shnaidman proposed openstack/tempest: Support for IPv6 for API (part 2) https://review.openstack.org/137239 | 19:57 |
| clarkb | it looked ok in vim | 19:57 |
| clarkb | the biggest issue is the vertical bridges but that seemed to be the best way to denote that | 19:58 |
| *** early has joined #openstack-qa | 19:58 | |
| mtreinish | clarkb: yeah it's far more legible in vim. I guess it's just the way gerrit is doing the spacing | 19:58 |
| *** mlavalle has joined #openstack-qa | 19:59 | |
| sdague | clarkb: svg is a real image format | 20:01 |
| *** mlavalle has quit IRC | 20:01 | |
| sdague | clarkb: it's very font sensitive | 20:01 |
| *** mlavalle has joined #openstack-qa | 20:01 | |
| clarkb | I can never figure out inkscape | 20:01 |
| sdague | honestly, that's why I do my diagrams inside of libreoffice most of the time | 20:02 |
| sdague | bug svg is just xml, you can do it in a text editor :) | 20:02 |
| *** rfolco has quit IRC | 20:03 | |
| *** early has quit IRC | 20:03 | |
| *** early has joined #openstack-qa | 20:07 | |
| morganfainberg | sdague, mtreinish, as soon as that fix passes check i'll +A it | 20:08 |
| mtreinish | morganfainberg: cool thanks | 20:09 |
| *** marrusl has joined #openstack-qa | 20:12 | |
| *** early has quit IRC | 20:14 | |
| *** achanda has quit IRC | 20:20 | |
| *** dustins has quit IRC | 20:22 | |
| openstackgerrit | Maru Newby proposed openstack-infra/devstack-gate: Refactor processing of testr artifacts for reuse https://review.openstack.org/153677 | 20:23 |
| openstackgerrit | Maru Newby proposed openstack-infra/devstack-gate: Refactor processing of testr artifacts for reuse https://review.openstack.org/153677 | 20:24 |
| *** early has joined #openstack-qa | 20:24 | |
| openstackgerrit | Maru Newby proposed openstack-infra/devstack-gate: Refactor processing of testr artifacts for reuse https://review.openstack.org/153677 | 20:25 |
| *** adalbas has quit IRC | 20:26 | |
| openstackgerrit | Sean Dague proposed openstack-dev/devstack: isolate unit tests from stackrc HOST_IP check https://review.openstack.org/153678 | 20:28 |
| *** annegentle has quit IRC | 20:29 | |
| dtroyer | sdague: I went ahead and +A it…I was hoping some of the other cores might stop by but we'll go on, there seems to be some good support anyway. | 20:30 |
| *** aysyd has quit IRC | 20:30 | |
| sdague | dtroyer: also https://review.openstack.org/153678 | 20:31 |
| sdague | because that just nuked a doc patch | 20:31 |
| dtroyer | just +2 it…I started seeing it occasionally today, any idea why now? | 20:32 |
| *** annashen has joined #openstack-qa | 20:32 | |
| sdague | dtroyer: nope, I'm asking in infra | 20:32 |
| sdague | my guess, some new network config rolled out on machines | 20:33 |
| sdague | but they are unit test machines... so we don't get all the machine logs | 20:33 |
| openstackgerrit | Merged openstack-dev/devstack: Stop neutron services more explicitly https://review.openstack.org/152603 | 20:34 |
| *** jhenner1 has joined #openstack-qa | 20:35 | |
| *** BharatK has quit IRC | 20:35 | |
| *** jhenner has quit IRC | 20:35 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Drop the legacy and un-used _interface https://review.openstack.org/153621 | 20:36 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Initial class creds creation in test base class https://review.openstack.org/153681 | 20:36 |
| *** Ananci has joined #openstack-qa | 20:37 | |
| *** vigneshvar has quit IRC | 20:38 | |
| *** jhenner1 has quit IRC | 20:40 | |
| openstackgerrit | Maru Newby proposed openstack-infra/devstack-gate: Refactor processing of testr artifacts for reuse https://review.openstack.org/153677 | 20:40 |
| *** pcrews has quit IRC | 20:42 | |
| *** jgrimm is now known as zz_jgrimm | 20:42 | |
| *** pcrews has joined #openstack-qa | 20:44 | |
| openstackgerrit | Merged openstack-dev/devstack: Split functions-common: python functions https://review.openstack.org/151811 | 20:45 |
| openstackgerrit | David Kranz proposed openstack/tempest: Change security-group clients to return one value and update tests https://review.openstack.org/153685 | 20:46 |
| *** jordanP has joined #openstack-qa | 20:47 | |
| *** early has quit IRC | 20:49 | |
| *** shashankhegde has quit IRC | 20:50 | |
| *** annashen has quit IRC | 20:50 | |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-dev/devstack: Add route table when failing to find host IP https://review.openstack.org/153688 | 20:50 |
| SpamapS | sdague: ^^ might help debug | 20:51 |
| *** andreaf has quit IRC | 20:51 | |
| openstackgerrit | Merged openstack-dev/grenade: Remove dead code in Neutron DHCP agent for dnsmasq lease script https://review.openstack.org/143299 | 20:51 |
| *** shashankhegde has joined #openstack-qa | 20:53 | |
| abhirc | newbie question - do i need start devstack's stack.sh flle to run the tempest test suite against it ? | 20:54 |
| sdague | SpamapS: so, actually, I found out what was happening | 20:55 |
| *** Yogi1 has quit IRC | 20:56 | |
| *** mwagner_lap has quit IRC | 20:57 | |
| SpamapS | sdague: subnet collision, right? | 21:00 |
| sdague | yeh | 21:00 |
| sdague | -infra | 21:00 |
| sdague | discussing it there | 21:00 |
| *** jordanP has quit IRC | 21:01 | |
| abhirc | answering my own question - no | 21:03 |
| *** melwitt has joined #openstack-qa | 21:03 | |
| *** melwitt_ has joined #openstack-qa | 21:04 | |
| *** annashen has joined #openstack-qa | 21:05 | |
| *** melwitt has quit IRC | 21:07 | |
| *** marun has quit IRC | 21:13 | |
| openstackgerrit | Merged openstack/tempest: Updated from global requirements https://review.openstack.org/153428 | 21:14 |
| *** jhenner has joined #openstack-qa | 21:15 | |
| *** achanda has joined #openstack-qa | 21:20 | |
| *** achanda has quit IRC | 21:25 | |
| *** melwitt_ is now known as melwitt | 21:31 | |
| *** ChuckC has quit IRC | 21:38 | |
| openstackgerrit | Al Miller proposed openstack-dev/devstack: Add devstack support for LBaaS Version 2 https://review.openstack.org/153079 | 21:38 |
| *** marun has joined #openstack-qa | 21:40 | |
| flashgordon | mtreinish: so I think the stable/juno dot release went out with stable/juno wedged | 21:41 |
| *** vigneshvar has joined #openstack-qa | 21:41 | |
| *** amotoki has quit IRC | 21:41 | |
| *** leakypipes has quit IRC | 21:41 | |
| sergsh | mtreinish, dkranz, can you please review my fixes? https://review.openstack.org/#/c/137239/ | 21:43 |
| mtreinish | flashgordon: heh, nice | 21:44 |
| mtreinish | that's a sign of people paying attention | 21:44 |
| openstackgerrit | Merged openstack/tempest: Fix import in nova cli tests https://review.openstack.org/153462 | 21:44 |
| openstackgerrit | Kyle Mestery proposed openstack-dev/devstack: Remove OpenDaylight from being in-tree for devstack https://review.openstack.org/153703 | 21:44 |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Switch to using the skip_because decorator from tempest-lib https://review.openstack.org/153461 | 21:45 |
| *** yamahata has quit IRC | 21:45 | |
| flashgordon | mtreinish: hmm I thought 2014.2.2 was cut, maybe not | 21:46 |
| flashgordon | but its still breaking for sure | 21:46 |
| flashgordon | aha the !dev ML | 21:47 |
| openstackgerrit | Anna Shen proposed openstack-dev/grenade: Add Trove upgrade process https://review.openstack.org/149416 | 21:48 |
| sdague | mtreinish: easy one - https://review.openstack.org/#/c/153678/ | 21:48 |
| mtreinish | sdague: +A | 21:50 |
| sdague | and... scene. See you guys next week. | 21:50 |
| flashgordon | and responded, this stable thing is such a pain | 21:52 |
| openstackgerrit | Kyle Mestery proposed openstack-infra/devstack-gate: Add DEVSTACK_GATE_NEUTRON_ODL option to enable ODL https://review.openstack.org/153705 | 21:52 |
| openstackgerrit | Merged openstack-dev/devstack: Document where we are going https://review.openstack.org/150789 | 21:54 |
| flashgordon | mtreinish: for https://review.openstack.org/#/c/151051/3/novaclient/tests/functional/test_readonly_nova.py,cm is global reqs bumped to 0.2 yet? | 21:58 |
| *** jecarey has quit IRC | 21:59 | |
| mtreinish | flashgordon: not yet, that was next on my list for today | 21:59 |
| mtreinish | if you want to go ahead and do that it saves me the step | 22:00 |
| *** rbak has quit IRC | 22:00 | |
| mtreinish | because we're already using it in tempest | 22:00 |
| flashgordon | mtreinish: doing it now | 22:00 |
| *** achanda has joined #openstack-qa | 22:00 | |
| mtreinish | flashgordon: just fyi it should just be like: https://review.openstack.org/153461 | 22:01 |
| sabeen1 | does tempest support data driven testing? | 22:01 |
| mestery | sdague: Thanks for the awesome email on how to use a devstack external plugin for gate testing! | 22:02 |
| mestery | sdague: Inspired me to start down that path for ODL :) | 22:02 |
| flashgordon | mtreinish: and done https://review.openstack.org/153708 | 22:03 |
| morganfainberg | mtreinish, sdague, going to backport that fix for the race to juno and icehouse. | 22:03 |
| morganfainberg | should be a clean cherry-pick | 22:03 |
| flashgordon | morganfainberg: stable/juno grenade is down so have fun | 22:03 |
| morganfainberg | flashgordon, doesn't mean it'll land today :P | 22:03 |
| morganfainberg | but it's an important fix to get in for sanity. | 22:04 |
| * morganfainberg dislikes being the reason gate is broken. | 22:04 | |
| flashgordon | morganfainberg: the reason is mtreinish this time :) | 22:04 |
| morganfainberg | ahaha | 22:04 |
| flashgordon | err stable gate | 22:04 |
| mtreinish | morganfainberg: to be fair I'm not even sure how we hit the keystone failure in the gate | 22:04 |
| morganfainberg | mtreinish, i am. | 22:04 |
| morganfainberg | mtreinish, multi workers plus multiple test runners | 22:05 |
| *** abhirc has quit IRC | 22:05 | |
| mtreinish | morganfainberg: well, the role should have existed from when devstack creates users | 22:05 |
| morganfainberg | maybe | 22:05 |
| mtreinish | morganfainberg: I understood the race, but the default role should have already existed I would think | 22:05 |
| morganfainberg | it's a wierd code-path | 22:05 |
| morganfainberg | that role should have been a migration like default domain | 22:05 |
| morganfainberg | but that ship has sailed | 22:05 |
| morganfainberg | it's likely because the setup users assigns specific roles. | 22:05 |
| mtreinish | morganfainberg: oh, yeah that would probably explain it | 22:06 |
| morganfainberg | in some cases a specific role supersedes the default, and if nothing used the default BOOM. | 22:06 |
| morganfainberg | very very very edge | 22:06 |
| morganfainberg | but still nasty. | 22:06 |
| morganfainberg | and a stupid reason for something to fail gate ;) | 22:06 |
| *** salv-orl_ has joined #openstack-qa | 22:07 | |
| morganfainberg | under eventlet this should be near impossible to happen | 22:07 |
| mtreinish | morganfainberg: I've fixed basically the same race on initial setup like 3 different times in n-net | 22:07 |
| morganfainberg | underapache much more liklely | 22:07 |
| mtreinish | but instead of db insert it was like setting up a bridge device with subprocess | 22:07 |
| morganfainberg | since sql is blocking and subprocess isn't | 22:08 |
| morganfainberg | that makes sense | 22:08 |
| *** salv-orlando has quit IRC | 22:08 | |
| morganfainberg | mysqldb can't yield in eventlet | 22:08 |
| mtreinish | flashgordon: it should be able to land because 0.2.0 will still be installed, the entry in test requirements will just not reflect reality | 22:09 |
| flashgordon | mtreinish: my patch tries changing the min version so the reqs sync job may bork | 22:10 |
| mtreinish | ah ok, I didn't look at it, just saw the notification with the comment | 22:10 |
| *** zigo has joined #openstack-qa | 22:11 | |
| mtreinish | sabeen1: do you have an example of what you're looking for? | 22:11 |
| *** salv-orl_ has quit IRC | 22:11 | |
| *** bknudson has quit IRC | 22:13 | |
| *** ChuckC has joined #openstack-qa | 22:13 | |
| sabeen1 | mtreinish: being able to send different data to the same function | 22:14 |
| *** cbader has quit IRC | 22:15 | |
| *** achanda has quit IRC | 22:17 | |
| *** achanda has joined #openstack-qa | 22:17 | |
| *** jhenner has quit IRC | 22:18 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/147961 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1372670 | 22:19 |
| openstack | Launchpad bug 1372670 in OpenStack Compute (nova) "libvirtError: operation failed: cannot read cputime for domain" [High,In progress] - Assigned to Eduardo Costa (ecosta) | 22:19 |
| openstackgerrit | Joe Gordon proposed openstack/tempest: Remove novaclient CLI tests https://review.openstack.org/153716 | 22:20 |
| flashgordon | mtreinish: ^ just so I can beat you to the punch | 22:20 |
| *** jhenner has joined #openstack-qa | 22:20 | |
| mtreinish | sabeen1: ok, yeah there are somethings which do that. Take a look at the autogenerated negative tests (there are only a few) | 22:21 |
| mtreinish | flashgordon: ugh, I never get the good removal patches... | 22:22 |
| flashgordon | if you port other CLI tests to tempest-lib ... | 22:22 |
| mtreinish | yeah, yeah | 22:23 |
| sabeen1 | thanks mtreinish! | 22:24 |
| flashgordon | mtreinish: http://logs.openstack.org/02/153702/1/check/check-grenade-dsvm/4bc843d//logs/grenade.sh.txt.gz#_2015-02-06_22_07_37_397 | 22:25 |
| flashgordon | huh? | 22:25 |
| mtreinish | flashgordon: do we do a reqs sync from juno or icehouse g-r? (which we really shouldn't) | 22:26 |
| *** erlon has quit IRC | 22:26 | |
| flashgordon | tahh right | 22:27 |
| flashgordon | ahh right* | 22:27 |
| *** Ananci has quit IRC | 22:28 | |
| *** marun has quit IRC | 22:30 | |
| * flashgordon wants to burn all stable branches to the ground | 22:33 | |
| * mtreinish brings some gasoline | 22:33 | |
| *** gfidente has quit IRC | 22:34 | |
| *** vigneshvar has quit IRC | 22:36 | |
| *** abhirc has joined #openstack-qa | 22:39 | |
| *** mriedem has quit IRC | 22:46 | |
| *** vigneshvar has joined #openstack-qa | 22:47 | |
| flashgordon | mtreinish: https://wiki.openstack.org/wiki/Magnum/Midcycle its local for me .. | 22:47 |
| mtreinish | flashgordon: so you're going to go? | 22:49 |
| flashgordon | mtreinish: maybe, if there is free food | 22:49 |
| *** yamamoto has joined #openstack-qa | 22:50 | |
| flashgordon | mtreinish: for the sync tempest global-reqs issue I am thinking solving it with https://review.openstack.org/#/c/153080/ | 22:51 |
| flashgordon | as that should side step the issue all together | 22:51 |
| flashgordon | (install tempest on master, but not on stable/*) | 22:52 |
| mtreinish | flashgordon: that might fix it, but if we still sync the reqs file with old g-r it won't | 22:54 |
| mtreinish | because isn't the conflict the stable g-r cap on oslo.config doesn't play nice with the version in tempest-lib's reqs file | 22:55 |
| mtreinish | so you'll still hit it when you install in the venv | 22:55 |
| flashgordon | that only happens during the install | 22:56 |
| flashgordon | AFAIK using tox -evenv means we don't do that | 22:56 |
| flashgordon | yup looks like it | 22:57 |
| *** marun has joined #openstack-qa | 22:58 | |
| openstackgerrit | Joe Gordon proposed openstack-dev/devstack: Stop installing tempest https://review.openstack.org/153080 | 22:58 |
| *** yamamoto has quit IRC | 23:00 | |
| flashgordon | I think that should work, still need to add the flag in to still run setup_develop $TEMPEST_DIR and turn that on by default. but set it to off in stable/* | 23:00 |
| openstackgerrit | Merged openstack-dev/grenade: Add partial ironic upgrade support https://review.openstack.org/148764 | 23:01 |
| openstackgerrit | Merged openstack/qa-specs: Update the template to add section for projects https://review.openstack.org/152668 | 23:03 |
| mtreinish | flashgordon: just do a raw patch blocking the install on the stable branches | 23:05 |
| mtreinish | and work on the flag for master | 23:05 |
| mtreinish | so future branches will be easy | 23:05 |
| mtreinish | flashgordon: oh yeah, I forgot that the sync get's triggered when we call setup_develop | 23:05 |
| *** yamamoto has joined #openstack-qa | 23:07 | |
| *** yamamoto has quit IRC | 23:13 | |
| *** marun has quit IRC | 23:13 | |
| *** Nic has quit IRC | 23:14 | |
| *** abhirc has quit IRC | 23:15 | |
| openstackgerrit | David Kranz proposed openstack/tempest: Change security-group clients to return one value and update tests https://review.openstack.org/153685 | 23:18 |
| *** shashankhegde has quit IRC | 23:20 | |
| *** boden has quit IRC | 23:20 | |
| *** jhenner has quit IRC | 23:25 | |
| *** bknudson has joined #openstack-qa | 23:25 | |
| *** salv-orlando has joined #openstack-qa | 23:29 | |
| *** annashen has quit IRC | 23:29 | |
| flashgordon | mtreinish: hmm now testr isn't installed | 23:33 |
| *** salv-orlando has quit IRC | 23:34 | |
| *** abhirc has joined #openstack-qa | 23:37 | |
| *** salv-orlando has joined #openstack-qa | 23:43 | |
| openstackgerrit | Joe Gordon proposed openstack-dev/devstack: Stop installing tempest https://review.openstack.org/153080 | 23:43 |
| *** salv-orlando has quit IRC | 23:45 | |
| *** annashen has joined #openstack-qa | 23:50 | |
| *** marun has joined #openstack-qa | 23:50 | |
| *** abhirc has quit IRC | 23:56 | |
| *** sabeen1 has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!