yahoo-eng-team team mailing list archive

[Launchpad Bug Tracker <1468457@xxxxxxxxxxxxxxxxxx>]

You have been subscribed to a public bug:

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.

** Affects: neutron
     Importance: Undecided
         Status: New

-- 
Invalid Tempest tests cause A10 CI to fail
https://bugs.launchpad.net/bugs/1468457
You received this bug notification because you are a member of Yahoo! Engineering Team, which is subscribed to neutron.