Skip list FIP test for neutron new behaviour
Neutron is planning to employ a strict validation against filter parameters [1]. The motivation is to align with the OpenStack API guideline [2]. In particular, the API guideline mentioned the following: "... if the API supports query parameters and a request contains an unknown or unsupported parameter, the server should return a 400 Bad Request response. Invalid values in the request URL should never be silently ignored, as the response may not match the client’s expectation..." After discussion with the API work group [3], the neutron team decides to make a backward-incompatible change (with a config option to rollback to the old API behaviour). To proceed on neutron implementation of supporting both the behaviour, this commit skip this tests as per Tempest Bug fix process[4] Once neutron changes are merged then, we can fix this tests to verify the both supported behaviour according to neutron behaviour discovery. Needed-By: https://review.openstack.org/#/c/574907/ [1] https://bugs.launchpad.net/neutron/+bug/1749820 [2] http://specs.openstack.org/openstack/api-wg/guidelines/http/ response-codes.html#failure-code-clarifications [3] http://eavesdrop.openstack.org/meetings/api_sig/2018/ api_sig.2018-03-15-16.00.log.html#l-120 [4] https://docs.openstack.org/tempest/latest/HACKING.html#bug-fix-on-core-project-needing-tempest-changes Change-Id: I0b24a304cc3466a2c05426cdbb6f9d99f1797edd
This commit is contained in:
committed by
Ghanshyam Mann
parent
de5f0da10e
commit
0f02c1caed
@@ -95,6 +95,7 @@ class ExternalNetworksTestJSON(base.BaseAdminNetworkTest):
|
||||
self.assertEqual(self.network['id'], show_net['id'])
|
||||
self.assertFalse(show_net['router:external'])
|
||||
|
||||
@decorators.skip_because(bug="1749820")
|
||||
@decorators.idempotent_id('82068503-2cf2-4ed4-b3be-ecb89432e4bb')
|
||||
@testtools.skipUnless(CONF.network_feature_enabled.floating_ips,
|
||||
'Floating ips are not availabled')
|
||||
|
||||
Reference in New Issue
Block a user