diff options
| author |   stark, steven <steven.stark@att.com> | 2019-02-15 13:28:11 -0800 |
|---|---|---|
| committer | stark, steven <steven.stark@att.com> | 2019-02-15 13:28:44 -0800 |
| commit | 3a37d68be925c92c9a540cf589dfbe4c901d7911 (patch) | |
| tree | b1a5faf94c572a86c7c6df01d80438556faf1819 /ice_validator/tests/utils | |
| parent | 36731efa2c3de9cb52941058fde0dcbc4beeed5b (diff) | |
[VVP] ports.py check port type
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>
Diffstat (limited to 'ice_validator/tests/utils')
| -rw-r--r-- | ice_validator/tests/utils/ports.py | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/ice_validator/tests/utils/ports.py b/ice_validator/tests/utils/ports.py index f5db5a4..89440eb 100644 --- a/ice_validator/tests/utils/ports.py +++ b/ice_validator/tests/utils/ports.py @@ -37,13 +37,13 @@ # # -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( - 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 @@ -138,6 +138,8 @@ def is_valid_ip_address( 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: @@ -190,8 +192,8 @@ def get_invalid_ip_addresses(resources, port_property, parameters): else: continue - 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(): @@ -220,6 +222,7 @@ def get_invalid_ip_addresses(resources, port_property, parameters): network_role, port_property, parameter_type, + network_type, ) if not valid_ip_address: |