Fix to terraform helper and add changes list to more pythonic way (#193)

* Item: #58 Desc: Change template to match query parameter from ansible inventory creation

* Item: #58 Desc: Changed list creation to more pythonic way

* Item: #58 Desc: Fix issue with terraform file helper

core/src/epicli/cli/engine/AnsibleInventoryCreator.py

@@ -25,7 +25,7 @@ def create(self):
         save_inventory(inventory, self.cluster_model.specification.name)
 
     def get_inventory(self):
-        inventory = list()
+        inventory = []
         for component_key, component_value in self.cluster_model.specification.components.items():
             if component_value.count < 1:
                 continue
@@ -56,7 +56,7 @@ def group_duplicated(inventory):
                 if host not in groups[item.role]:
                     groups[item.role].append(host)
 
-        result = list()
+        result = []
         for key, value in groups.items():
             result.append(AnsibleInventoryItem(key, value))
         return result

core/src/epicli/cli/engine/aws/AWSAPIProxy.py

@@ -48,7 +48,7 @@ def get_ips_for_feature(self, feature_key, look_for_public_ip=False):
                 }]
         )
 
-        result = list()
+        result = []
         for instance in running_instances:
             if look_for_public_ip:
                 result.append(AnsibleHostModel(instance.public_dns_name, instance.public_ip_address))

core/src/epicli/cli/engine/aws/AWSConfigBuilder.py

@@ -7,7 +7,7 @@
 
 class AWSConfigBuilder(InfrastructureConfigBuilder):
     def build(self, cluster_model, user_input):
-        result = list()
+        result = []
         vpc_config = self.get_vpc_config(cluster_model, user_input)
         result.append(vpc_config)
         vpc_name = vpc_config.specification.name

core/src/epicli/cli/helpers/doc_list_helpers.py

@@ -9,7 +9,7 @@ def select_first(documents, query):
 
 def select_all(documents, query):
     if documents is not None:
-        result = list()
+        result = []
         for x in documents:
             if query(x):
                 result.append(x)

core/src/epicli/cli/helpers/terraform_file_helper.py

@@ -11,10 +11,10 @@ def generate_terraform_file(infrastructure, template_generator, template_generat
                                                                      template_generator_config.
                                                                      templates_paths)
 
-        if yaml_document.kind != "epiphany-cluster":
-            terraform_file_name = "{:03d}".format(idx + 1) + "_" + yaml_document.specification.name + ".tf"
+        if yaml_document["kind"] != "epiphany-cluster":
+            terraform_file_name = "{:03d}".format(idx + 1) + "_" + yaml_document["specification"]["name"] + ".tf"
         else:
-            terraform_file_name = "000_" + yaml_document.name + ".tf"
+            terraform_file_name = "000_" + yaml_document["specification"]["name"] + ".tf"
 
         terraform_output_file_path = os.path.join(terraform_build_directory, terraform_file_name)
 

core/src/epicli/cli/static/terraform_templates_aws/vm_template.tf.j2

@@ -35,7 +35,7 @@ resource "aws_autoscaling_group" "{{ specification.name }}" {
   tag {
     {%- for tag_key, tag_value in tag.items() %}
     key = "{{ tag_key }}"
-    value = "{% if tag_value is defined and tag_value|length %}{{ tag_value }}{% else %}-{% endif %}"
+    value = "{{ tag_value }}"
     {%- endfor %}
     propagate_at_launch = "true"
   }