Merge pull request #48 from silinternational/develop

Release 5.0.0

aws/alb/versions.tf

@@ -1,4 +1,11 @@
 
 terraform {
   required_version = ">= 0.12"
+
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = ">= 2.0.0, < 5.0.0"
+    }
+  }
 }

aws/asg-ebs/main.tf

@@ -1,10 +1,8 @@
 /*
  * Generate user_data from template file
  */
-data "template_file" "user_data" {
-  template = file("${path.module}/user-data.sh")
-
-  vars = {
+locals {
+  user_data = templatefile("${path.module}/user-data.sh", {
     ecs_cluster_name     = var.ecs_cluster_name
     additional_user_data = var.additional_user_data
     aws_region           = var.aws_region
@@ -18,7 +16,7 @@ data "template_file" "user_data" {
     ebs_mkfs_extraopts   = var.ebs_mkfs_extraopts
     ebs_fs_type          = var.ebs_fs_type
     ebs_mountopts        = var.ebs_mountopts
-  }
+  })
 }
 
 /*
@@ -37,7 +35,7 @@ resource "aws_launch_configuration" "as_conf" {
     volume_size = var.aws_instance["volume_size"]
   }
 
-  user_data = data.template_file.user_data.rendered
+  user_data = local.user_data
 
   lifecycle {
     create_before_destroy = true

aws/asg-ebs/versions.tf

@@ -1,4 +1,11 @@
 
 terraform {
   required_version = ">= 0.12"
+
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = ">= 2.0.0, < 5.0.0"
+    }
+  }
 }

aws/asg-efs/main.tf

@@ -1,15 +1,13 @@
 /*
  * Generate user_data from template file
  */
-data "template_file" "user_data" {
-  template = file("${path.module}/user-data.sh")
-
-  vars = {
+locals {
+  user_data = templatefile("${path.module}/user-data.sh", {
     ecs_cluster_name     = var.ecs_cluster_name
     efs_dns_name         = var.efs_dns_name
     mount_point          = var.mount_point
     additional_user_data = var.additional_user_data
-  }
+  })
 }
 
 /*
@@ -28,7 +26,7 @@ resource "aws_launch_configuration" "as_conf" {
     volume_size = var.aws_instance["volume_size"]
   }
 
-  user_data = data.template_file.user_data.rendered
+  user_data = local.user_data
 
   lifecycle {
     create_before_destroy = true

aws/asg-efs/versions.tf

@@ -1,4 +1,11 @@
 
 terraform {
   required_version = ">= 0.12"
+
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = ">= 2.0.0, < 5.0.0"
+    }
+  }
 }

aws/asg/main.tf

@@ -1,13 +1,11 @@
 /*
  * Generate user_data from template file
  */
-data "template_file" "user_data" {
-  template = file("${path.module}/user-data.sh")
-
-  vars = {
+locals {
+  user_data = templatefile("${path.module}/user-data.sh", {
     ecs_cluster_name     = var.ecs_cluster_name
     additional_user_data = var.additional_user_data
-  }
+  })
 }
 
 /*
@@ -26,7 +24,7 @@ resource "aws_launch_configuration" "as_conf" {
     volume_size = var.aws_instance["volume_size"]
   }
 
-  user_data = data.template_file.user_data.rendered
+  user_data = local.user_data
 
   lifecycle {
     create_before_destroy = true

aws/asg/versions.tf

@@ -1,4 +1,11 @@
 
 terraform {
   required_version = ">= 0.12"
+
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = ">= 2.0.0, < 5.0.0"
+    }
+  }
 }

aws/cloudflare-sg/versions.tf

@@ -1,4 +1,11 @@
 
 terraform {
   required_version = ">= 0.12"
+
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = ">= 2.0.0, < 5.0.0"
+    }
+  }
 }

aws/cloudtrail/main.tf

@@ -1,38 +1,40 @@
 resource "aws_s3_bucket" "cloudtrail" {
   bucket        = var.s3_bucket_name
   force_destroy = true
+}
+
+resource "aws_s3_bucket_policy" "cloudtrail" {
+  bucket = aws_s3_bucket.cloudtrail.id
 
-  policy = <<POLICY
-{
-    "Version": "2012-10-17",
-    "Statement": [
+  policy = jsonencode(
+    {
+      Version = "2012-10-17"
+      Statement = [
         {
-            "Sid": "AWSCloudTrailAclCheck",
-            "Effect": "Allow",
-            "Principal": {
-              "Service": "cloudtrail.amazonaws.com"
-            },
-            "Action": "s3:GetBucketAcl",
-            "Resource": "arn:aws:s3:::${var.s3_bucket_name}"
+          Sid    = "AWSCloudTrailAclCheck"
+          Effect = "Allow"
+          Principal = {
+            Service = "cloudtrail.amazonaws.com"
+          }
+          Action   = "s3:GetBucketAcl"
+          Resource = "arn:aws:s3:::${var.s3_bucket_name}"
         },
         {
-            "Sid": "AWSCloudTrailWrite",
-            "Effect": "Allow",
-            "Principal": {
-              "Service": "cloudtrail.amazonaws.com"
-            },
-            "Action": "s3:PutObject",
-            "Resource": "arn:aws:s3:::${var.s3_bucket_name}/*",
-            "Condition": {
-                "StringEquals": {
-                    "s3:x-amz-acl": "bucket-owner-full-control"
-                }
+          Sid    = "AWSCloudTrailWrite"
+          Effect = "Allow"
+          Principal = {
+            Service = "cloudtrail.amazonaws.com"
+          }
+          Action   = "s3:PutObject"
+          Resource = "arn:aws:s3:::${var.s3_bucket_name}/*"
+          Condition = {
+            StringEquals = {
+              "s3:x-amz-acl" = "bucket-owner-full-control"
             }
-        }
-    ]
-}
-POLICY
-
+          }
+        },
+      ]
+  })
 }
 
 resource "aws_iam_user" "cloudtrail-s3" {
@@ -43,26 +45,24 @@ resource "aws_iam_user_policy" "cloudtrail-s3" {
   name = "cloudtrail-s3"
   user = aws_iam_user.cloudtrail-s3.name
 
-  policy = <<EOF
-{
-  "Version": "2012-10-17",
-  "Statement": [
+  policy = jsonencode(
     {
-      "Effect": "Allow",
-      "Action": [
-        "s3:GetBucketPolicy",
-        "s3:GetObject",
-        "s3:ListBucket"
-      ],
-      "Resource": [
-          "${aws_s3_bucket.cloudtrail.arn}",
-          "${aws_s3_bucket.cloudtrail.arn}/*"
+      Version = "2012-10-17"
+      Statement = [
+        {
+          Effect = "Allow"
+          Action = [
+            "s3:GetBucketPolicy",
+            "s3:GetObject",
+            "s3:ListBucket",
+          ],
+          Resource = [
+            aws_s3_bucket.cloudtrail.arn,
+            "${aws_s3_bucket.cloudtrail.arn}/*",
+          ]
+        },
       ]
-    }
-  ]
-}
-EOF
-
+  })
 }
 
 resource "aws_cloudtrail" "cloudtrail" {

aws/cloudtrail/versions.tf

@@ -1,4 +1,11 @@
 
 terraform {
   required_version = ">= 0.12"
+
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = ">= 2.0.0, < 5.0.0"
+    }
+  }
 }

aws/ecr/main.tf

@@ -5,18 +5,16 @@ resource "aws_ecr_repository" "repo" {
   name = var.repo_name
 }
 
-data "template_file" "repo_policy" {
-  template = file("${path.module}/ecr-policy.json")
-
-  vars = {
+locals {
+  repo_policy = templatefile("${path.module}/ecr-policy.json", {
     ecsInstanceRole_arn = var.ecsInstanceRole_arn
     ecsServiceRole_arn  = var.ecsServiceRole_arn
     cd_user_arn         = var.cd_user_arn
-  }
+  })
 }
 
 resource "aws_ecr_repository_policy" "policy" {
   repository = aws_ecr_repository.repo.name
-  policy     = data.template_file.repo_policy.rendered
+  policy     = local.repo_policy
 }
 

aws/ecr/versions.tf

@@ -1,4 +1,11 @@
 
 terraform {
   required_version = ">= 0.12"
+
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = ">= 2.0.0, < 5.0.0"
+    }
+  }
 }