yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #42242
[Bug 1468457] Re: Invalid Tempest tests cause A10 CI to fail
not related to neutronclient, it should be filed to neutron.
** Project changed: python-neutronclient => neutron
** Tags added: lbaas
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1468457
Title:
Invalid Tempest tests cause A10 CI to fail
Status in neutron:
New
Bug description:
The following tests will not pass in A10's CI due to what appear to be incorrect tests.
neutron_lbaas.tests.tempest.v2.api.test_pools_admin.TestPools.test_create_pool_for_another_tenant[smoke]
neutron_lbaas.tests.tempest.v2.api.test_pools_admin.TestPools.test_create_pool_missing_tenant_id_for_admin[smoke]
neutron_lbaas.tests.tempest.v2.api.test_pools_admin.TestPools.test_create_pool_missing_tenant_id_for_other_tenant[smoke]
neutron_lbaas.tests.tempest.v2.api.test_pools_admin.TestPools.test_create_pool_using_empty_tenant_field[smoke]
--
I'm creating this bug so I have one to reference when I @skip the tests per dougwig.
The empty tenant ID tests need to be modified to expect an error
condition, but this is not possible as Neutron's request handling
fills in missing tenant IDs with the tenant ID of the logged in user.
This is an error condition and should be handled as such. Fixing it
in the request handling is going to require fixes in a lot more places
in Neutron, I believe. I'll look for other similar tests that would
expose such functionality.
To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1468457/+subscriptions