Merge pull request #12 from silvercar/sg-0.12

Update security group template to terraform 0.12 format

.circleci/config.yml

@@ -14,8 +14,6 @@ jobs:
           command: pylint src
 
 
-
-
 workflows:
   version: 2
   build:

.pylintrc

@@ -15,7 +15,7 @@ ignore-patterns=
 
 # Python code to execute, usually for sys.path manipulation such as
 # pygtk.require().
-#init-hook=
+init-hook="from pylint.config import find_pylintrc; import os, sys; sys.path.append(os.path.dirname(find_pylintrc()))"
 
 # Use multiple processes to speed up Pylint. Specifying 0 will auto-detect the
 # number of processors available to use.

src/terrafort/renderer.py

@@ -57,5 +57,6 @@ def fmt(content):
         formatted = subprocess.run(["terraform", "fmt", "-"],
                                    stdout=subprocess.PIPE,
                                    input=content,
-                                   encoding='ascii')
+                                   encoding='ascii',
+                                   check=False)
         return formatted.stdout

src/terrafort/templates/aws_security_group.tf

@@ -4,8 +4,8 @@ resource "aws_security_group" "{{ resource.GroupName }}" {
   vpc_id                 = "{{ resource.VpcId }}"
   revoke_rules_on_delete = true
 
-  tags {
-    {% for tag in resource.Tags %} "{{ tag.Key }}" = "{{tag.Value}}"
+  tags = {
+    {% for tag in resource.Tags %} {{ tag.Key }} = "{{tag.Value}}"
     {% endfor %}
   }
 }