V2T migration: remove irrelevant pre-flight checks
Multiple LBs on the same router are now supported Change-Id: I450d218d19d5d5683017579ab6faac2477cef2e5
This commit is contained in:
parent
ad6fd2d11b
commit
4cf42d5ed9
@ -159,7 +159,6 @@ def validate_config_for_migration(resource, event, trigger, **kwargs):
|
|||||||
filters = {'device_owner': [nl_constants.DEVICE_OWNER_LOADBALANCERV2,
|
filters = {'device_owner': [nl_constants.DEVICE_OWNER_LOADBALANCERV2,
|
||||||
oct_const.DEVICE_OWNER_OCTAVIA]}
|
oct_const.DEVICE_OWNER_OCTAVIA]}
|
||||||
lb_ports = plugin.get_ports(admin_context, filters=filters)
|
lb_ports = plugin.get_ports(admin_context, filters=filters)
|
||||||
lb_routers = []
|
|
||||||
for port in lb_ports:
|
for port in lb_ports:
|
||||||
fixed_ips = port.get('fixed_ips', [])
|
fixed_ips = port.get('fixed_ips', [])
|
||||||
if 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 "
|
LOG.error("Loadbalancer %s subnet %s is not external "
|
||||||
"nor connected to a router.",
|
"nor connected to a router.",
|
||||||
port.get('device_id'), subnet_id)
|
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
|
# TODO(asarfaty): Multiple listeners on the same pool is not
|
||||||
# supported, but currently the admin utility has no access to this
|
# 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
|
# but currently the admin utility has no access to this information
|
||||||
# from octavia
|
# from octavia
|
||||||
|
|
||||||
# General validations:
|
|
||||||
# TODO(asarfaty): multiple transport zones (migrator limitation)?
|
|
||||||
|
|
||||||
if n_errors > 0:
|
if n_errors > 0:
|
||||||
plural = n_errors > 1
|
plural = n_errors > 1
|
||||||
LOG.error("The NSX-V plugin configuration is not ready to be "
|
LOG.error("The NSX-V plugin configuration is not ready to be "
|
||||||
|
Loading…
Reference in New Issue
Block a user