From 0f02c1caed35951a1804269786323403324fc61d Mon Sep 17 00:00:00 2001 From: ghanshyam Date: Wed, 18 Jul 2018 02:41:44 +0000 Subject: [PATCH] Skip list FIP test for neutron new behaviour MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 --- tempest/api/network/admin/test_external_network_extension.py | 1 + 1 file changed, 1 insertion(+) diff --git a/tempest/api/network/admin/test_external_network_extension.py b/tempest/api/network/admin/test_external_network_extension.py index 49a9cdb5f0..39d03e7c67 100644 --- a/tempest/api/network/admin/test_external_network_extension.py +++ b/tempest/api/network/admin/test_external_network_extension.py @@ -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')