get_invalid_ip_addresses needs to be enhanced to check
the neutron port type (internal or external) before
validating the IP parameter format.
The IP parameter format should not be validated against
both internal and external, it should only be validated
against the formats for the ports type.
Change-Id: Ic8b66b83729f6ea5d268d55816e03dcbbfec12fd
Issue-ID: VVP-167
Signed-off-by: stark, steven <steven.stark@att.com>
-from .network_roles import get_network_role_from_port
+from .network_roles import get_network_role_and_type
from .vm_types import get_vm_type_for_nova_server
import re
def is_valid_ip_address(
from .vm_types import get_vm_type_for_nova_server
import re
def is_valid_ip_address(
- ip_address, vm_type, network_role, port_property, parameter_type
+ ip_address, vm_type, network_role, port_property, parameter_type, network_type
):
"""
Check the ip_address to make sure it is properly formatted and
):
"""
Check the ip_address to make sure it is properly formatted and
continue
if v3[0] != port_property:
continue
continue
if v3[0] != port_property:
continue
+ if v3[2] != network_type:
+ continue
# check if pattern matches
m = v3[3].match(ip_address)
if m:
# check if pattern matches
m = v3[3].match(ip_address)
if m:
- network_role = get_network_role_from_port(port_resource)
- if not network_role:
+ network_role, network_type = get_network_role_and_type(port_resource)
+ if not network_role or not network_type:
continue
for k1, v1 in port_resource["properties"].items():
continue
for k1, v1 in port_resource["properties"].items():
network_role,
port_property,
parameter_type,
network_role,
port_property,
parameter_type,
)
if not valid_ip_address:
)
if not valid_ip_address: