Implement Piped mode for provision

cloudbender/exceptions.py

@@ -0,0 +1,2 @@
+class ParameterNotFound(Exception):
+    """My documentation"""

cloudbender/stack.py

@@ -10,10 +10,11 @@ from dateutil.tz import tzutc
 
 from botocore.exceptions import ClientError
 
-from .utils import dict_merge
+from .utils import dict_merge, search_refs
 from .connection import BotoConnection
 from .jinja import JinjaEnv, read_config_file
 from . import __version__
+from .exceptions import ParameterNotFound
 
 import cfnlint.core
 
@@ -265,41 +266,42 @@ class Stack(object):
 
         self.read_template_file()
 
-        # Inspect all outputs of the running Conglomerate members
+        # if we run in Piped Mode, inspect all outputs of the running Conglomerate members
-        # if we run in Piped Mode
+        if self.mode == "Piped":
-        # if self.mode == "Piped":
+            stack_outputs = {}
-        #     try:
+            try:
-        #         stack_outputs = inspect_stacks(config['tags']['Conglomerate'])
+                stack_outputs = self._inspect_stacks(self.tags['Conglomerate'])
-        #         logger.info(pprint.pformat(stack_outputs))
+            except KeyError:
-        #     except KeyError:
+                pass
-        #        pass
 
         if 'Parameters' in self.cfn_data:
+            _errors = []
             self.cfn_parameters = []
             for p in self.cfn_data['Parameters']:
                 # In Piped mode we try to resolve all Paramters first via stack_outputs
-                # if self.mode == "Piped":
+                if self.mode == "Piped":
-                #    try:
+                    try:
-                #        # first reverse the rename due to AWS alphanumeric restriction for parameter names
+                        # first reverse the rename due to AWS alphanumeric restriction for parameter names
-                #        _p = p.replace('DoT','.')
+                        _p = p.replace('DoT', '.')
-                #        value = str(stack_outputs[_p])
+                        value = str(stack_outputs[_p])
-                #        parameters.append({'ParameterKey': p, 'ParameterValue': value })
+                        self.cfn_parameters.append({'ParameterKey': p, 'ParameterValue': value})
-                #        logger.info('Got {} = {} from running stack'.format(p,value))
+                        logger.info('Got {} = {} from running stack'.format(p, value))
-                #        continue
+                        continue
-                #    except KeyError:
+                    except KeyError:
-                #        pass
+                        pass
 
                 # Key name in config tree is: stacks.<self.stackname>.parameters.<parameter>
-                try:
+                if p in self.parameters:
                     value = str(self.parameters[p])
                     self.cfn_parameters.append({'ParameterKey': p, 'ParameterValue': value})
                     logger.info('{} {} Parameter {}={}'.format(self.region, self.stackname, p, value))
-                except KeyError:
-                    # If we have a Default defined in the CFN skip, as AWS will use it
-                    if 'Default' in self.cfn_data['Parameters'][p]:
-                        continue
                 else:
-                        logger.error('Cannot find value for parameter {0}'.format(p))
+                    # If we have a Default defined in the CFN skip, as AWS will use it
+                    if not 'Default' in self.cfn_data['Parameters'][p]:
+                        _errors.append(p)
+
+            if _errors:
+                raise ParameterNotFound('Cannot find value for parameters: {0}'.format(_errors))
 
     def write_parameter_file(self):
         parameter_file = os.path.join(self.ctx['parameter_path'], self.rel_path, self.stackname + ".yaml")
@@ -515,33 +517,41 @@ class Stack(object):
         if not os.path.exists(os.path.join(self.ctx[path], self.rel_path)):
             os.makedirs(os.path.join(self.ctx[path], self.rel_path))
 
+    # stackoutput inspection
+    def _inspect_stacks(self, conglomerate):
+        # Get all stacks of the conglomertate
+        running_stacks = self.connection_manager.call(
+            "cloudformation",
+            "describe_stacks",
+            profile=self.profile, region=self.region)
 
-def search_refs(template, attributes, mode):
+        stacks = []
-    """ Traverses a template and searches for all Fn::GetAtt calls to FortyTwo
+        for stack in running_stacks['Stacks']:
-        adding them to the passed in attributes set
+            for tag in stack['Tags']:
-    """
+                if tag['Key'] == 'Conglomerate' and tag['Value'] == conglomerate:
-    if isinstance(template, dict):
+                    stacks.append(stack)
-        for k, v in template.items():
+                    break
-            # FortyTwo Fn::GetAtt
+
-            if k == "Fn::GetAtt" and isinstance(v, list):
+        # Gather stack outputs, use Tag['Artifact'] as name space: Artifact.OutputName, same as FortyTwo
-                if v[0] == "FortyTwo":
+        stack_outputs = {}
-                    attributes.append(v[1])
+        for stack in stacks:
+            # If stack has an Artifact Tag put resources into the namespace Artifact.Resource
+            artifact = None
+            for tag in stack['Tags']:
+                if tag['Key'] == 'Artifact':
+                    artifact = tag['Value']
+
+            if artifact:
+                key_prefix = "{}.".format(artifact)
+            else:
+                key_prefix = ""
 
-            # CloudBender::StackRef
-            if k == "CloudBender::StackRef":
             try:
-                    attributes.append(v['StackTags']['Artifact'])
+                for output in stack['Outputs']:
+                    # Gather all outputs of the stack into one dimensional key=value structure
+                    stack_outputs[key_prefix + output['OutputKey']] = output['OutputValue']
             except KeyError:
                 pass
 
-            # PipedMode Refs
+        # Add outputs from stacks into the data for jinja under StackOutput
-            if mode == "Piped" and k == "Ref" and "DoT" in v:
+        return stack_outputs
-                attributes.append(v)
-
-            if isinstance(v, dict) or isinstance(v, list):
-                search_refs(v, attributes, mode)
-
-    elif isinstance(template, list):
-        for k in template:
-            if isinstance(k, dict) or isinstance(k, list):
-                search_refs(k, attributes, mode)

cloudbender/utils.py

@@ -58,3 +58,34 @@ def setup_logging(debug):
     logger.addHandler(log_handler)
     logger.setLevel(our_level)
     return logger
+
+
+def search_refs(template, attributes, mode):
+    """ Traverses a template and searches for all Fn::GetAtt calls to FortyTwo
+        adding them to the passed in attributes set
+    """
+    if isinstance(template, dict):
+        for k, v in template.items():
+            # FortyTwo Fn::GetAtt
+            if k == "Fn::GetAtt" and isinstance(v, list):
+                if v[0] == "FortyTwo":
+                    attributes.append(v[1])
+
+            # CloudBender::StackRef
+            if k == "CloudBender::StackRef":
+                try:
+                    attributes.append(v['StackTags']['Artifact'])
+                except KeyError:
+                    pass
+
+            # PipedMode Refs
+            if mode == "Piped" and k == "Ref" and "DoT" in v:
+                attributes.append(v)
+
+            if isinstance(v, dict) or isinstance(v, list):
+                search_refs(v, attributes, mode)
+
+    elif isinstance(template, list):
+        for k in template:
+            if isinstance(k, dict) or isinstance(k, list):
+                search_refs(k, attributes, mode)