Make rendering faster by rendering once (instead of rendering each layer separately)

modulestf/cloudcraft/graph.py

@@ -80,7 +80,9 @@ def populate_graph(data):  # noqa: C901
         for edge in G.edges.data():
             edge = list(edge)
 
-            if edge[0] in connectors or edge[1] in connectors:
+            if edge[0] == edge[1]:
+                edge = []
+            elif edge[0] in connectors or edge[1] in connectors:
                 break
             else:
                 edge = []

modulestf/const.py

@@ -9,7 +9,9 @@
 
 OUTPUT_DIR = "output"
 WORK_DIR = "work"
-FINAL_DIR = "../final"
+WORK_DIR_FOR_COOKIECUTTER = "{{ cookiecutter.dir_name }}"
+
+FINAL_DIR = "final"
 
 S3_BUCKET = "dl.modules.tf"
 S3_BUCKET_REGION = "eu-west-1"

modulestf/render.py

@@ -1,8 +1,9 @@
 import glob
 import json
-import os
+import pathlib
 import re
 import shutil
+from os import chdir, getcwd, makedirs, mkdir, path
 from pprint import pformat, pprint
 
 from cookiecutter.exceptions import NonTemplatedInputDirException
@@ -21,73 +22,101 @@ def mkdir_safely(dir):
         pass
 
     try:
-        os.mkdir(dir)
+        mkdir(dir)
     except OSError:
         pass
 
 
 def prepare_render_dirs():
-    output_dir = os.path.join(tmp_dir, OUTPUT_DIR)
+    # return
+    output_dir = path.join(tmp_dir, OUTPUT_DIR)
 
     mkdir_safely(output_dir)
-    os.chdir(output_dir)
+    chdir(output_dir)
 
     mkdir_safely(WORK_DIR)
-    os.chdir(WORK_DIR)
+    chdir(WORK_DIR)
 
-    mkdir_safely(FINAL_DIR)
+    # mkdir_safely(FINAL_DIR)
 
+    mkdir(WORK_DIR_FOR_COOKIECUTTER)
 
-def render_single_layer(resource, region):
+
+def find_templates_files(dir):
+    pprint("DIR = %s" % dir)
+
+    files = glob.glob(dir + "/*") + \
+        glob.glob(dir + "/.*")
+
+    return files
+
+
+def prepare_single_layer(resource, region, templates_dir, templates_files):
 
     dir_name = resource.get("dir_name")
 
-    full_dir_name = ("single_layer/%s/%s" % (region, dir_name)).lower()
+    full_dir_name = ("%s/%s" % (region, dir_name)).lower()
 
     single_layer = {
-        "dir_name": full_dir_name,
-        "region": region,
-        "module_source": MODULES[resource["type"]]["source"],
-        "module_variables": MODULES[resource["type"]]["variables"],
+        "module_type": resource["type"]
     }
 
     extra_context = resource.update(single_layer) or resource
 
-    cookiecutter(os.path.join(COOKIECUTTER_TEMPLATES_DIR, COOKIECUTTER_TEMPLATES_PREFIX + "-single-layer"),
-                 config_file=os.path.join(COOKIECUTTER_TEMPLATES_DIR, "config_aws_lambda.yaml"),
-                 no_input=True,
-                 extra_context=extra_context)
+    data = '{%- set this = ' + str(extra_context) + ' -%}'
 
+    dst_dir = path.join(getcwd(), WORK_DIR_FOR_COOKIECUTTER, full_dir_name)
 
-def render_common_layer(region):
+    for file in templates_files:
+        # pprint("original = %s" % file)
+        part_of_path_to_keep = "".join(file[len(templates_dir):])
 
-    common_layer = {
-        "dir_name": "common_layer",
-        "region": region,
-    }
+        # pprint("just relative file = %s" % part_file)
+        # pprint("getcwd = %s" % getcwd())
 
-    try:
-        cookiecutter(os.path.join(COOKIECUTTER_TEMPLATES_DIR, COOKIECUTTER_TEMPLATES_PREFIX + "-common-layer"),
-                     config_file=os.path.join(COOKIECUTTER_TEMPLATES_DIR, "config_aws_lambda.yaml"),
-                     no_input=True,
-                     extra_context=common_layer)
-    except NonTemplatedInputDirException:
-        pass
+        dst_file = dst_dir + part_of_path_to_keep
+        # pprint("new file = %s" % dst_file)
 
+        with open(file, "r") as original:
+            original_data = original.read()
+            original.close()
 
-def render_root_dir(source, region, dirs):
+        makedirs(path.dirname(dst_file), exist_ok=True)
 
-    root_dir = {
-        "dir_name": "root_dir",
-        "source_name": source["name"],
-        "region": region,
-        "dirs": dirs,
-    }
+        with open(dst_file, "w") as modified:
+            modified.write(data + "\n" + original_data)
+            modified.close()
+
+    shutil.copy(templates_dir + "/../cookiecutter.json", "cookiecutter.json")
+
+    return resource["type"]
+
+
+# Copy all files and subdirectories into working directory
+def copy_to_working_dir(templates_dir):
 
-    cookiecutter(os.path.join(COOKIECUTTER_TEMPLATES_DIR, "root"),
-                 config_file=os.path.join(COOKIECUTTER_TEMPLATES_DIR, "config_aws_lambda.yaml"),
+    dst_dir = path.realpath(WORK_DIR_FOR_COOKIECUTTER)
+
+    files = find_templates_files(templates_dir)
+
+    for file in files:
+        pprint("FILE == %s" % file)
+        pprint("dst_dir == %s" % dst_dir)
+        if path.isdir(file):
+            dst = path.join(dst_dir, path.basename(file))
+            shutil.copytree(file, dst)
+        else:
+            shutil.copy(file, dst_dir)
+
+
+def render_all(extra_context):
+
+    output_dir = path.join(tmp_dir, OUTPUT_DIR, WORK_DIR)
+
+    cookiecutter(output_dir,
+                 config_file=path.join(COOKIECUTTER_TEMPLATES_DIR, "config_aws_lambda.yaml"),
                  no_input=True,
-                 extra_context=root_dir)
+                 extra_context=extra_context)
 
 
 # Count unique combination of type and text to decide if to append unique resource id
@@ -169,6 +198,13 @@ def render_from_modulestf_config(config, source, regions):
 
         types_text[t] = new_appendix
 
+    # Find all templates for single layer once
+    templates_dir = path.realpath(path.join(COOKIECUTTER_TEMPLATES_DIR, COOKIECUTTER_TEMPLATES_PREFIX + "-single-layer/template"))
+    templates_files = find_templates_files(templates_dir)
+
+    # Set of used module to load data once
+    used_modules = set()
+
     # render single layers in a loop
     for resource in resources:
 
@@ -201,23 +237,33 @@ def render_from_modulestf_config(config, source, regions):
 
         # Render the layer
         logger.info("Rendering single layer resource id: %s" % resource.get("ref_id"))
-        render_single_layer(resource, region)
+        used_module_type = prepare_single_layer(resource, region, templates_dir, templates_files)
 
-    logger.info("Rendering common layer")
-    render_common_layer(region)
+        used_modules.add(used_module_type)
 
-    logger.info("Rendering root dir")
-    render_root_dir(source, region, dirs)
+    extra_context = dict({"module_sources": {}, "module_variables": {}})
+    for module_type in used_modules:
+        extra_context["module_sources"].update({
+            module_type: MODULES[module_type]["source"],
+        })
 
-    files = glob.glob("single_layer/*") + \
-        glob.glob("single_layer/.*") + \
-        glob.glob("common_layer/*") + \
-        glob.glob("common_layer/.*") + \
-        glob.glob("root_dir/*") + \
-        glob.glob("root_dir/.*")
+        extra_context["module_variables"].update({
+            module_type: MODULES[module_type]["variables"],
+        })
 
-    logger.info("Moving files into final dir: %s" % FINAL_DIR)
-    for file in files:
-        shutil.move(file, FINAL_DIR)
+    logger.info("Prepare common layer")
+    templates_dir = path.realpath(path.join(COOKIECUTTER_TEMPLATES_DIR, COOKIECUTTER_TEMPLATES_PREFIX + "-common-layer/template"))
+    copy_to_working_dir(templates_dir)
+
+    logger.info("Prepare root dir")
+    templates_dir = path.realpath(path.join(COOKIECUTTER_TEMPLATES_DIR, "root/template"))
+    copy_to_working_dir(templates_dir)
+
+    extra_context["source_name"] = source["name"]
+    extra_context["dirs"] = dirs
+    extra_context["region"] = region
+
+    logger.info("Rendering all")
+    render_all(extra_context)
 
     logger.info("Complete!")

serverless.yml

@@ -9,7 +9,7 @@ provider:
   region: eu-west-1
   profile: private-anton
   tracing: true
-  timeout: 10
+  timeout: 30
   environment:
     S3_BUCKET: "dl.modules.tf"
     S3_DIR: ${self:custom.stage}

templates/terragrunt-single-layer/cookiecutter.json

@@ -1,6 +1,9 @@
 {
-  "dir_name": "template",
-  "module_source": "",
+  "dir_name": "final",
+  "region": "",
+  "source_name": "",
+  "dirs": "",
+  "module_sources": {},
   "module_variables": {},
   "params": {},
   "dependencies": "",

templates/terragrunt-single-layer/{{ cookiecutter.dir_name }}/terraform.tfvars → templates/terragrunt-single-layer/template/terraform.tfvars

@@ -1,33 +1,35 @@
+{%- set module_source = cookiecutter.module_sources[this.module_type] -%}
+{%- set module_variables = cookiecutter.module_variables[this.module_type] -%}
 terragrunt = {
   terraform {
-    source = "{{ cookiecutter.module_source }}"
+    source = "{{ module_source }}"
   }
 
   include = {
     path = "${find_in_parent_folders()}"
   }
 
-  {% if cookiecutter.dependencies|default("") -%}
+  {% if this.dependencies|default("") -%}
   dependencies {
     paths = [
-      {%- for value in cookiecutter.dependencies.split(",") -%}
+      {%- for value in this.dependencies.split(",") -%}
       "../{{ value }}"{%- if not loop.last -%}, {% endif -%}
       {%- endfor -%}
     ]
   }
   {%- endif %}
 }
 
-{% for key, value in cookiecutter.module_variables|dictsort -%}
+{% for key, value in module_variables|dictsort -%}
 
 {%- if value.cloudcraft_name is defined -%}
-{%- if value.cloudcraft_name in cookiecutter.params -%}
-{%- set tmp_value = cookiecutter.params[value.cloudcraft_name] -%}
+{%- if value.cloudcraft_name in this.params -%}
+{%- set tmp_value = this.params[value.cloudcraft_name] -%}
 {%- else -%}
 {%- set tmp_value = None -%}
 {%- endif-%}
-{%- elif key in cookiecutter.params -%}
-{%- set tmp_value = cookiecutter.params[key] -%}
+{%- elif key in this.params -%}
+{%- set tmp_value = this.params[key] -%}
 {%- else -%}
 {%- set tmp_value = None -%}
 {%- endif -%}
@@ -57,8 +59,8 @@ terragrunt = {
 {%- endif -%}
 
 {#- Inline comment for dynamic params -#}
-{%- if key in cookiecutter.dynamic_params -%}
-{%- set comment_dynamic_param = " # @modulestf:" ~ cookiecutter.dynamic_params[key] -%}
+{%- if key in this.dynamic_params -%}
+{%- set comment_dynamic_param = " # @modulestf:" ~ this.dynamic_params[key] -%}
 {%- else -%}
 {%- set comment_dynamic_param = "" -%}
 {%- endif %}