openstack/neutron
Code Issues Proposed changes

[FT] Make LR name random

Browse Source 
In the ``TestNbApi`` tests, make the Logical_Router name random.
This patch also makes use of the ``lookup`` method to retrieve
a DB row from the database.

Closes-Bug: #2073387
Change-Id: I304c5199815ff583ac306f4de6cea771fdf00391
This commit is contained in:
Rodolfo Alonso Hernandez 2024-07-17 13:39:45 +00:00
parent 9b6c485b93
commit ed303c464e
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_impl_idl.py
View File

@ -627,15 +627,14 @@ class TestNbApi(BaseOvnIdlTest):
self.nbapi.lookup('HA_Chassis_Group', router_name, default=None))
def _assert_routes_exist(self, lr_name, expected_count):
with self.nbapi.transaction(check_error=True) as txn:
lr = txn.add(self.nbapi.lr_get(lr_name))
actual_count = len(lr.result.static_routes)
lr = self.nbapi.lookup('Logical_Router', lr_name)
actual_count = len(lr.static_routes)
self.assertEqual(actual_count, expected_count,
f"Expected {expected_count} routes, "
f"found {actual_count}.")
def test_del_static_routes(self):
lr_name = 'router_with_static_routes_del'
lr_name = ovn_utils.ovn_name(uuidutils.generate_uuid())
routes = [('0.0.0.0/0', '192.0.2.1'), ('10.0.0.0/24', '192.0.3.1')]
with self.nbapi.transaction(check_error=True) as txn:
@ -653,7 +652,7 @@ class TestNbApi(BaseOvnIdlTest):
self._assert_routes_exist(lr_name, 0)
def test_del_no_static_routes(self):
lr_name = 'router_with_static_routes_del'
lr_name = ovn_utils.ovn_name(uuidutils.generate_uuid())
routes = []
with self.nbapi.transaction(check_error=True) as txn: