From d4c907818c5d43ce37578076942474fa805281af Mon Sep 17 00:00:00 2001 From: Frode Nordahl Date: Tue, 31 Oct 2023 14:57:46 +0100 Subject: [PATCH] cms/openstack: Set route policy to '""' instead of not setting it. Having the transaction error in the run makes the run fail on larger scale. Signed-off-by: Frode Nordahl --- ovn-tester/cms/openstack/openstack.py | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/ovn-tester/cms/openstack/openstack.py b/ovn-tester/cms/openstack/openstack.py index 002cb995..9fe2c821 100644 --- a/ovn-tester/cms/openstack/openstack.py +++ b/ovn-tester/cms/openstack/openstack.py @@ -246,12 +246,8 @@ def connect_external_network_to_project( self.external_port = lr_port gw_net = DualStackSubnet(netaddr.IPNetwork("0.0.0.0/0")) - # XXX: ovsdbapp does not allow setting external IDs to static route - # XXX: Setting 'policy' to "" throws "constraint violation" error in - # logs because ovsdbapp does not allow not specifying policy. - # However, the route itself is created successfully with no - # policy, the same way Neutron does it. - self.nbctl.route_add(project.router, gw_net, lr_port.ip, "") + # NOTE(mkalcok): Neutron actually sets policy to '""' + self.nbctl.route_add(project.router, gw_net, lr_port.ip, '""') gw_nodes = self._get_gateway_chassis(external_network.num_gw_nodes) for index, chassis in enumerate(gw_nodes):