openstack/neutron
Code Issues Proposed changes

Merge "[DVR] Set arp entries only for single IPs given as allowed addr pair"

Browse Source
This commit is contained in:
Zuul 2021-09-15 09:59:24 +00:00 committed by Gerrit Code Review
commit 6db2619628
2 changed files with 28 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
neutron/agent/l3/dvr_local_router.py
View File

@ -366,7 +366,13 @@ class DvrLocalRouter(dvr_router_base.DvrRouterBase):
device=device, device=device,
device_exists=device_exists) device_exists=device_exists)
for allowed_address_pair in p.get('allowed_address_pairs', []): for allowed_address_pair in p.get('allowed_address_pairs', []):
self._update_arp_entry(allowed_address_pair['ip_address'], if ('/' not in str(allowed_address_pair['ip_address']) or
common_utils.is_cidr_host(
allowed_address_pair['ip_address'])):
ip_address = common_utils.cidr_to_ip(
allowed_address_pair['ip_address'])
self._update_arp_entry(
ip_address,
allowed_address_pair['mac_address'], allowed_address_pair['mac_address'],
subnet_id, subnet_id,
'add', 'add',

16
neutron/tests/functional/agent/l3/test_dvr_router.py
View File

@ -1005,13 +1005,18 @@ class TestDvrRouter(DvrRouterTestFramework, framework.L3AgentTestFramework):
# cache is properly populated. # cache is properly populated.
self.agent.conf.agent_mode = 'dvr_snat' self.agent.conf.agent_mode = 'dvr_snat'
router_info = self.generate_dvr_router_info(enable_snat=True) router_info = self.generate_dvr_router_info(enable_snat=True)
expected_neighbors = ['35.4.1.10', '10.0.0.10'] expected_neighbors = ['35.4.1.10', '10.0.0.10', '10.200.0.3']
allowed_address_net = netaddr.IPNetwork('10.100.0.0/30')
port_data = { port_data = {
'fixed_ips': [{'ip_address': expected_neighbors[0]}], 'fixed_ips': [{'ip_address': expected_neighbors[0]}],
'mac_address': 'fa:3e:aa:bb:cc:dd', 'mac_address': 'fa:3e:aa:bb:cc:dd',
'device_owner': DEVICE_OWNER_COMPUTE, 'device_owner': DEVICE_OWNER_COMPUTE,
'allowed_address_pairs': [ 'allowed_address_pairs': [
{'ip_address': expected_neighbors[1], {'ip_address': expected_neighbors[1],
'mac_address': 'fa:3e:aa:bb:cc:dd'},
{'ip_address': '10.200.0.3/32',
'mac_address': 'fa:3e:aa:bb:cc:dd'},
{'ip_address': str(allowed_address_net),
'mac_address': 'fa:3e:aa:bb:cc:dd'}] 'mac_address': 'fa:3e:aa:bb:cc:dd'}]
} }
self.agent.plugin_rpc.get_ports_by_subnet.return_value = [port_data] self.agent.plugin_rpc.get_ports_by_subnet.return_value = [port_data]
@ -1019,11 +1024,18 @@ class TestDvrRouter(DvrRouterTestFramework, framework.L3AgentTestFramework):
internal_device = router1.get_internal_device_name( internal_device = router1.get_internal_device_name(
router_info['_interfaces'][0]['id']) router_info['_interfaces'][0]['id'])
for expected_neighbor in expected_neighbors: for expected_neighbor in expected_neighbors:
neighbor = ip_lib.dump_neigh_entries(4, internal_device, neighbor = ip_lib.dump_neigh_entries(
lib_constants.IP_VERSION_4, internal_device,
router1.ns_name, router1.ns_name,
dst=expected_neighbor) dst=expected_neighbor)
self.assertNotEqual([], neighbor) self.assertNotEqual([], neighbor)
self.assertEqual(expected_neighbor, neighbor[0]['dst']) self.assertEqual(expected_neighbor, neighbor[0]['dst'])
for not_expected_neighbor in allowed_address_net:
neighbor = ip_lib.dump_neigh_entries(
lib_constants.IP_VERSION_4, internal_device,
router1.ns_name,
dst=str(not_expected_neighbor))
self.assertEqual([], neighbor)
def _assert_rfp_fpr_mtu(self, router, expected_mtu=1500): def _assert_rfp_fpr_mtu(self, router, expected_mtu=1500):
dev_mtu = self.get_device_mtu( dev_mtu = self.get_device_mtu(