x
/
vmware-nsx
Code Issues Proposed changes

V2T migration: remove irrelevant pre-flight checks 

Multiple LBs on the same router are now supported

Change-Id: I450d218d19d5d5683017579ab6faac2477cef2e5
This commit is contained in:
asarfaty 2020-10-04 09:36:59 +02:00 committed by Adit Sarfaty
parent 9dbe5e3d38
commit 63f944e915
1 changed files with 0 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
vmware_nsx/shell/admin/plugins/nsxv/resources/migration.py
View File

@ -159,7 +159,6 @@ def validate_config_for_migration(resource, event, trigger, **kwargs):
filters = {'device_owner': [nl_constants.DEVICE_OWNER_LOADBALANCERV2,
oct_const.DEVICE_OWNER_OCTAVIA]}
lb_ports = plugin.get_ports(admin_context, filters=filters)
lb_routers = []
for port in lb_ports:
fixed_ips = port.get('fixed_ips', [])
if fixed_ips:
@ -176,13 +175,6 @@ def validate_config_for_migration(resource, event, trigger, **kwargs):
LOG.error("Loadbalancer %s subnet %s is not external "
"nor connected to a router.",
port.get('device_id'), subnet_id)
# Multiple loadbalancers on the same router cannot be supported
if router_id in lb_routers:
n_errors = n_errors + 1
LOG.error("Router %s has multiple loadbalancers which is "
"not supported.", router_id)
else:
lb_routers.append(router_id)
# TODO(asarfaty): Multiple listeners on the same pool is not
# supported, but currently the admin utility has no access to this
@ -192,9 +184,6 @@ def validate_config_for_migration(resource, event, trigger, **kwargs):
# but currently the admin utility has no access to this information
# from octavia
# General validations:
# TODO(asarfaty): multiple transport zones (migrator limitation)?
if n_errors > 0:
plural = n_errors > 1
LOG.error("The NSX-V plugin configuration is not ready to be "