Fix the error reporting.

- Don't return a dict error
- in manager return full response
- return the result in Resource.validate()

Change-Id: I585ea9dd9cf747927fb4effb90cfff49cba20931
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>

diff --git a/heat/engine/checkeddict.py b/heat/engine/checkeddict.py
index ec9c6fe35b7f444c263b67a18e6c4c155aff66c3..41367a945b53011e1b29faf9ff877b8de425706c 100644 (file)
--- a/heat/engine/checkeddict.py
+++ b/heat/engine/checkeddict.py
@@ -154,11 +154,9 @@ class Properties(CheckedDict):
             if 'Required' in self.data[key]:
                 if self.data[key]['Required'] \
                     and not 'Value' in self.data[key]:
-                    return {'Error':
-                        '%s Property must be provided' % key}
+                    return '%s Property must be provided' % key
 
             # are there unimplemented Properties
             if not self.data[key]['Implemented'] and 'Value' in self.data[key]:
-                return {'Error':
-                    '%s Property not implemented yet' % key}
+                return '%s Property not implemented yet' % key
         return None

diff --git a/heat/engine/manager.py b/heat/engine/manager.py
index f8bb9c516e989774c233dd53beaad5ec5bc862a2..299221363afbc2d883b8caa38e40114badf6738a 100644 (file)
--- a/heat/engine/manager.py
+++ b/heat/engine/manager.py
@@ -203,7 +203,7 @@ class EngineManager(manager.Manager):
         template_copy = None
         if 'Malformed Query Response' in \
                 response['ValidateTemplateResult']['Description']:
-            return response['ValidateTemplateResult']['Description']
+            return response
 
         stack = parser.Stack(context, stack_name, template, 0, params,
                              metadata_server=metadata_server)

diff --git a/heat/engine/parser.py b/heat/engine/parser.py
index 2b72bb9e60f8d24b9dcafcd7ee6a32568dd58153..8494e5c7d0ec3d3399fd44d2d206a015af3cc2e4 100644 (file)
--- a/heat/engine/parser.py
+++ b/heat/engine/parser.py
@@ -103,7 +103,7 @@ class Stack(object):
                 res = str(ex)
             finally:
                 if res:
-                    err_str = 'Malformed Query Response [%s]' % (res)
+                    err_str = 'Malformed Query Response %s' % (res)
                     response = {'ValidateTemplateResult': {
                                 'Description': err_str,
                                 'Parameters': []}}

diff --git a/heat/engine/resources.py b/heat/engine/resources.py
index 137b41be0b6dbdd6413623780904d3cdce42726e..82b791540ae32867adc6abe6fc10fe878dfa9736 100644 (file)
--- a/heat/engine/resources.py
+++ b/heat/engine/resources.py
@@ -128,8 +128,8 @@ class Resource(object):
         try:
             self.calculate_properties()
         except ValueError as ex:
-                return {'Error': '%s' % str(ex)}
-        self.properties.validate()
+                return str(ex)
+        return self.properties.validate()
 
     def instance_id_set(self, inst):
         self.instance_id = inst