Skip NetworkWritableMtuTest when driver is ML2/OVN

East/west icmp fragmentation is not supported with ML2/OVN since
there is no production use case, as mentioned in [1].
With this change NetworkWritableMtuTest will be skipped
automatically when ML2/OVN agents found and thus ci jobs
configs can be simplified by removing exclude list.

[1] https://docs.openstack.org/neutron/latest/ovn/gaps.html

Change-Id: I7f690b1a260ff70e12bcfac82f63e36ddca7cfbc
This commit is contained in:
Roman Safronov 2024-09-17 14:57:48 +03:00
parent 26adc41889
commit 9be9375131
6 changed files with 3 additions and 27 deletions

View File

@ -169,19 +169,13 @@ class NetworkWritableMtuTest(NetworkMtuBaseTest):
servers = [] servers = []
networks = [] networks = []
@classmethod
def skip_checks(cls):
super(NetworkWritableMtuTest, cls).skip_checks()
supported_type_drivers = ['vxlan', 'geneve']
if not any(type_driver in supported_type_drivers for type_driver in
config.CONF.neutron_plugin_options.available_type_drivers):
raise cls.skipException(
"Neither VXLAN nor GENEVE type_driver is enabled")
@classmethod @classmethod
@utils.requires_ext(extension="net-mtu-writable", service="network") @utils.requires_ext(extension="net-mtu-writable", service="network")
def resource_setup(cls): def resource_setup(cls):
super(NetworkWritableMtuTest, cls).resource_setup() super(NetworkWritableMtuTest, cls).resource_setup()
if cls.is_driver_ovn:
raise cls.skipException("East/west icmp fragmentation is not "
"supported with ML2/OVN")
def _create_setup(self): def _create_setup(self):
self.admin_client = self.os_admin.network_client self.admin_client = self.os_admin.network_client

View File

@ -210,10 +210,6 @@
(^neutron_tempest_plugin.scenario)|\ (^neutron_tempest_plugin.scenario)|\
(^tempest.api.compute.servers.test_attach_interfaces)|\ (^tempest.api.compute.servers.test_attach_interfaces)|\
(^tempest.api.compute.servers.test_multiple_create)" (^tempest.api.compute.servers.test_multiple_create)"
# TODO(jlibosva): Remove the NetworkWritableMtuTest test from the list
# once east/west fragmentation is supported in core OVN
tempest_exclude_regex: "\
(^neutron_tempest_plugin.scenario.test_mtu.NetworkWritableMtuTest)"
devstack_localrc: devstack_localrc:
NETWORK_API_EXTENSIONS: "{{ (network_api_extensions_common + network_api_extensions_ovn) | join(',') }}" NETWORK_API_EXTENSIONS: "{{ (network_api_extensions_common + network_api_extensions_ovn) | join(',') }}"
devstack_services: devstack_services:

View File

@ -205,10 +205,6 @@
(^neutron_tempest_plugin.scenario)|\ (^neutron_tempest_plugin.scenario)|\
(^tempest.api.compute.servers.test_attach_interfaces)|\ (^tempest.api.compute.servers.test_attach_interfaces)|\
(^tempest.api.compute.servers.test_multiple_create)" (^tempest.api.compute.servers.test_multiple_create)"
# TODO(jlibosva): Remove the NetworkWritableMtuTest test from the list
# once east/west fragmentation is supported in core OVN
tempest_exclude_regex: "\
(^neutron_tempest_plugin.scenario.test_mtu.NetworkWritableMtuTest)"
devstack_localrc: devstack_localrc:
NETWORK_API_EXTENSIONS: "{{ (network_api_extensions_common + network_api_extensions_ovn) | join(',') }}" NETWORK_API_EXTENSIONS: "{{ (network_api_extensions_common + network_api_extensions_ovn) | join(',') }}"
devstack_services: devstack_services:

View File

@ -207,10 +207,6 @@
(^neutron_tempest_plugin.scenario)|\ (^neutron_tempest_plugin.scenario)|\
(^tempest.api.compute.servers.test_attach_interfaces)|\ (^tempest.api.compute.servers.test_attach_interfaces)|\
(^tempest.api.compute.servers.test_multiple_create)" (^tempest.api.compute.servers.test_multiple_create)"
# TODO(jlibosva): Remove the NetworkWritableMtuTest test from the list
# once east/west fragmentation is supported in core OVN
tempest_exclude_regex: "\
(^neutron_tempest_plugin.scenario.test_mtu.NetworkWritableMtuTest)"
devstack_localrc: devstack_localrc:
NETWORK_API_EXTENSIONS: "{{ (network_api_extensions_common + network_api_extensions_ovn) | join(',') }}" NETWORK_API_EXTENSIONS: "{{ (network_api_extensions_common + network_api_extensions_ovn) | join(',') }}"
devstack_local_conf: devstack_local_conf:

View File

@ -627,10 +627,6 @@
network_api_extensions_ovn: network_api_extensions_ovn:
- vlan-transparent - vlan-transparent
- external-gateway-multihoming - external-gateway-multihoming
# TODO(jlibosva): Remove the NetworkWritableMtuTest test from the list
# once east/west fragmentation is supported in core OVN
tempest_exclude_regex: "\
(^neutron_tempest_plugin.scenario.test_mtu.NetworkWritableMtuTest)"
devstack_localrc: devstack_localrc:
Q_AGENT: ovn Q_AGENT: ovn
OVN_AGENT_EXTENSIONS: 'metadata' OVN_AGENT_EXTENSIONS: 'metadata'

View File

@ -186,8 +186,6 @@
(^neutron_tempest_plugin.scenario)|\ (^neutron_tempest_plugin.scenario)|\
(^tempest.api.compute.servers.test_attach_interfaces)|\ (^tempest.api.compute.servers.test_attach_interfaces)|\
(^tempest.api.compute.servers.test_multiple_create)" (^tempest.api.compute.servers.test_multiple_create)"
tempest_exclude_regex: "\
(^neutron_tempest_plugin.scenario.test_mtu.NetworkWritableMtuTest)"
network_api_extensions: *api_extensions network_api_extensions: *api_extensions
network_api_extensions_ovn: network_api_extensions_ovn:
- vlan-transparent - vlan-transparent