x/vmware-nsx
Code Issues Proposed changes

[NSX-P] Ensure multicast is disabled for dualstack use cases

Browse Source 
For some dual-stack use cases it will be mandatory to disable multicast
routing on NSX-T segments.

Change-Id: I821b6038ec4b0404d54c03c8802bdbbf8d211ed4
This commit is contained in:
Salvatore Orlando 2021-10-14 13:46:13 -07:00
parent 8031a85420
commit 5c7c09d56a
1 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
vmware_nsx/plugins/nsx_p/plugin.py
View File

@ -1141,12 +1141,11 @@ class NsxPolicyPlugin(nsx_plugin_common.NsxPluginV3Base):
return bool(dhcp_port) return bool(dhcp_port)
def _get_segment_subnets_versions(self, context, net_id): def _get_segment_subnets_versions(self, context, net_id):
# Find networks DHCP enabled subnets versions = {4: [], 6: []}
versions = set()
with db_api.CONTEXT_READER.using(context): with db_api.CONTEXT_READER.using(context):
network = self._get_network(context, net_id) network = self._get_network(context, net_id)
for subnet in network.subnets: for subnet in network.subnets:
versions.add(subnet.ip_version) versions[subnet.ip_version].append(subnet)
return versions return versions
def _get_segment_multicast_setting(self, context, net_id): def _get_segment_multicast_setting(self, context, net_id):
@ -1154,8 +1153,16 @@ class NsxPolicyPlugin(nsx_plugin_common.NsxPluginV3Base):
seg_subnets_ip_ver = self._get_segment_subnets_versions( seg_subnets_ip_ver = self._get_segment_subnets_versions(
context, net_id) context, net_id)
# Multicast cannot be enabled on segments with v6 subnets only # Multicast cannot be enabled on segments with v6 subnets only
if len(seg_subnets_ip_ver) == 1 and seg_subnets_ip_ver.pop() == 6: if not seg_subnets_ip_ver[4]:
return False return False
if seg_subnets_ip_ver[6]:
for subnet in seg_subnets_ip_ver[6]:
if subnet.enable_dhcp:
break
else:
# There are no v4 subnets stored in NSX, must
# disable multicast
return False
# Ignore value of multicast setting (go with defaults) # Ignore value of multicast setting (go with defaults)
return core_resources.IGNORE return core_resources.IGNORE