diff --git a/ovn_bgp_agent/drivers/openstack/ovn_bgp_driver.py b/ovn_bgp_agent/drivers/openstack/ovn_bgp_driver.py index b077bbdb..5bcb1c0a 100644 --- a/ovn_bgp_agent/drivers/openstack/ovn_bgp_driver.py +++ b/ovn_bgp_agent/drivers/openstack/ovn_bgp_driver.py @@ -580,7 +580,7 @@ class OVNBGPDriver(driver_api.AgentDriverBase): elif row.type == constants.OVN_PATCH_VIF_PORT_TYPE: if (associated_port and self.sb_idl.is_port_on_chassis( associated_port, self.chassis)): - if not self.ovn_local_cr_lrps.get(associated_port): + if not self.sb_idl.is_provider_network(row.datapath): # Only exposing IPs if the associated network is a # provider network return [] @@ -722,7 +722,7 @@ class OVNBGPDriver(driver_api.AgentDriverBase): self.sb_idl.is_port_on_chassis( associated_port, self.chassis) or self.sb_idl.is_port_deleted(associated_port))): - if not self.ovn_local_cr_lrps.get(associated_port): + if not self.sb_idl.is_provider_network(row.datapath): # Only exposing IPs if the associated network is a # provider network return diff --git a/ovn_bgp_agent/tests/unit/drivers/openstack/test_ovn_bgp_driver.py b/ovn_bgp_agent/tests/unit/drivers/openstack/test_ovn_bgp_driver.py index 96b29b13..4dc78a44 100644 --- a/ovn_bgp_agent/tests/unit/drivers/openstack/test_ovn_bgp_driver.py +++ b/ovn_bgp_agent/tests/unit/drivers/openstack/test_ovn_bgp_driver.py @@ -1245,7 +1245,7 @@ class TestOVNBGPDriver(test_base.TestCase): @mock.patch.object(linux_net, 'add_ips_to_dev') def test__expose_ip_fip_association_to_vm( self, mock_add_ip_dev, mock_add_rule, mock_add_route): - self.sb_idl.is_provider_network.return_value = False + self.sb_idl.is_provider_network.return_value = True self.sb_idl.is_port_on_chassis.return_value = True mock_get_bridge = mock.patch.object( self.bgp_driver, '_get_bridge_for_datapath').start() @@ -1500,7 +1500,7 @@ class TestOVNBGPDriver(test_base.TestCase): @mock.patch.object(linux_net, 'del_ips_from_dev') def test_withdraw_ip_fip_association_to_vm( self, mock_del_ip_dev, mock_del_rule, mock_del_route): - self.sb_idl.is_provider_network.return_value = False + self.sb_idl.is_provider_network.return_value = True self.sb_idl.is_port_on_chassis.return_value = True mock_get_bridge = mock.patch.object( self.bgp_driver, '_get_bridge_for_datapath').start()