aboutsummaryrefslogtreecommitdiffstats
path: root/ice_validator/tests/test_vm_class_has_unique_type.py
diff options
context:
space:
mode:
authorstark, steven <steven.stark@att.com>2019-11-18 13:45:47 -0800
committerstark, steven <steven.stark@att.com>2019-11-18 14:30:09 -0800
commit1d281ce111a76c2b056bcaa572c1e929c90237cf (patch)
treeaefc59fd8aae102d60c75aed484750075e892423 /ice_validator/tests/test_vm_class_has_unique_type.py
parent0c4e64d87728b89aa9cd4d41d738f5bfe64ceee3 (diff)
[VVP] Updating vm_type class test to proceed if no cinder
Other minor changes - updating error message to be more clear for vm_type class - updating test_network_format to handle nested files w/ no resources - updating ports.py to handle hardcoded str_replace param/values Issue-ID: VVP-342 Signed-off-by: stark, steven <steven.stark@att.com> Change-Id: I63fcda582fe25dcedca87e874327b7ea56fe6ef3
Diffstat (limited to 'ice_validator/tests/test_vm_class_has_unique_type.py')
-rw-r--r--ice_validator/tests/test_vm_class_has_unique_type.py12
1 files changed, 7 insertions, 5 deletions
diff --git a/ice_validator/tests/test_vm_class_has_unique_type.py b/ice_validator/tests/test_vm_class_has_unique_type.py
index b158f5b..5938535 100644
--- a/ice_validator/tests/test_vm_class_has_unique_type.py
+++ b/ice_validator/tests/test_vm_class_has_unique_type.py
@@ -73,8 +73,8 @@ class VmClassValidator(object):
self.vm_rids = collections.defaultdict(set)
self.vm_types = collections.defaultdict(set)
va_config, self.va_count = CinderVolumeAttachmentProcessor.get_config(resources)
- if not va_config:
- pytest.skip("No Cinder Volume Attachment configurations found")
+ # if not va_config:
+ # pytest.skip("No Cinder Volume Attachment configurations found")
for rid, resource in resources.items():
vm_class = NovaServerProcessor.get_vm_class(resource)
if vm_class:
@@ -98,13 +98,15 @@ class VmClassValidator(object):
for k, v in self.vm_types.items():
if len(v) > 1:
errors.append(
- "vm-type %s has class conflict %s"
+ "OS::Nova::Server resources with the same vm_type must have identical configurations. "
+ "The following OS::Nova::Server resources for vm-type %s do not have identical configurations: %s"
% (k, ", ".join(str(list(self.vm_classes[c])) for c in v))
)
classes = list(v)
errors.append(
- "Differences %s"
- % ", ".join([str(key_diff(classes[0], c)) for c in classes[1:]])
+ "The following attributes are detected differences between "
+ "OS::Nova::Server's with vm_type %s: %s"
+ % (k, ", ".join([str(key_diff(classes[0], c)) for c in classes[1:]]))
)
for k, v in self.vm_counts.items():
if len(v) > 1: