From 63f944e915395f2ed3adcca1294818b2faca03f3 Mon Sep 17 00:00:00 2001 From: asarfaty Date: Sun, 4 Oct 2020 09:36:59 +0200 Subject: [PATCH] V2T migration: remove irrelevant pre-flight checks Multiple LBs on the same router are now supported Change-Id: I450d218d19d5d5683017579ab6faac2477cef2e5 --- .../shell/admin/plugins/nsxv/resources/migration.py | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/vmware_nsx/shell/admin/plugins/nsxv/resources/migration.py b/vmware_nsx/shell/admin/plugins/nsxv/resources/migration.py index ac48f683ff..f0651b3663 100644 --- a/vmware_nsx/shell/admin/plugins/nsxv/resources/migration.py +++ b/vmware_nsx/shell/admin/plugins/nsxv/resources/migration.py @@ -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 "