commit f47c3e7c72d798e9e6130c2b660514708e6add03 Author: asarfaty Date: Sun Oct 4 09:36:59 2020 +0200 V2T migration: remove irrelevant pre-flight checks Multiple LBs on the same router are now supported Change-Id: I450d218d19d5d5683017579ab6faac2477cef2e5 diff --git a/vmware_nsx/shell/admin/plugins/nsxv/resources/migration.py b/vmware_nsx/shell/admin/plugins/nsxv/resources/migration.py index 5323062..d99b6f0 100644 --- a/vmware_nsx/shell/admin/plugins/nsxv/resources/migration.py +++ b/vmware_nsx/shell/admin/plugins/nsxv/resources/migration.py @@ -161,7 +161,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: @@ -178,13 +177,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 @@ -194,9 +186,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 "