openstack
/
neutron
Code Issues Proposed changes

Mark TestVirtualPorts tests as unstable 

Based on debug performed in [1] the TestVirtualPorts
are unstable. At lease one of tests per 20
retries of functional job fail in various
places.
We don't think it is worth to spend more
time on debugging those tests, because there is
a possible easier solution: set virtual port
type based on DEVICE_TYPE, and do not rely
on allowed address pairs.

[1] https://review.opendev.org/#/c/712888/

Related-Bug: #1865453

Change-Id: Iffef8ceb9ca458820a7b3e604a53281f7e93b810
This commit is contained in:
Maciej Józefczyk 2020-03-19 12:03:49 +00:00
parent 5e84289f68
commit ceb27f2c7d
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/test_mech_driver.py
View File

@ -23,6 +23,7 @@ from neutron.common.ovn import constants as ovn_const
from neutron.common.ovn import utils
from neutron.common import utils as n_utils
from neutron.db import ovn_revision_numbers_db as db_rev
from neutron.tests import base as tests_base
from neutron.tests.functional import base
@ -246,6 +247,7 @@ class TestVirtualPorts(base.TestOVNFunctionalBase):
check = functools.partial(self._is_ovn_port_type, port_id, type)
n_utils.wait_until_true(check, timeout=10)
@tests_base.unstable_test("bug 1865453")
def test_virtual_port_created_before(self):
virt_port = self._create_port()
virt_ip = virt_port['fixed_ips'][0]['ip_address']
@ -281,6 +283,7 @@ class TestVirtualPorts(base.TestOVNFunctionalBase):
backup['id'],
ovn_vport.options[ovn_const.LSP_OPTIONS_VIRTUAL_PARENTS_KEY])
@tests_base.unstable_test("bug 1865453")
def test_virtual_port_update_address_pairs(self):
master = self._create_port()
backup = self._create_port()
@ -350,6 +353,7 @@ class TestVirtualPorts(base.TestOVNFunctionalBase):
self.assertNotIn(ovn_const.LSP_OPTIONS_VIRTUAL_IP_KEY,
ovn_vport.options)
@tests_base.unstable_test("bug 1865453")
def test_virtual_port_created_after(self):
master = self._create_port(fixed_ip='10.0.0.11')
backup = self._create_port(fixed_ip='10.0.0.12')
@ -376,6 +380,7 @@ class TestVirtualPorts(base.TestOVNFunctionalBase):
backup['id'],
ovn_vport.options[ovn_const.LSP_OPTIONS_VIRTUAL_PARENTS_KEY])
@tests_base.unstable_test("bug 1865453")
def test_virtual_port_delete_parents(self):
master = self._create_port()
backup = self._create_port()