Require TF >=1.3, make variables optional if we already accepted null

.github/workflows/ci.yml

@@ -19,7 +19,7 @@ jobs:
       - name: Terraform Setup
         uses: hashicorp/setup-terraform@v1
         with:
-          terraform_version: '1.1.x'
+          terraform_version: '1.3.x'
 
       - name: Terraform Format
         working-directory: "./"
@@ -30,7 +30,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        terraform: [ '0.12.x', '0.13.x', '0.14.x', '0.15.x', '1.0.x', '1.1.x' ]
+        terraform: [ '1.3.x' ]
       fail-fast: false
     steps:
       - uses: actions/checkout@v3

examples/ci/ci.tf

@@ -1,5 +1,5 @@
 terraform {
-  required_version = ">= 0.12"
+  required_version = ">= 1.3"
 
   required_providers {
     aws = {

examples/logging/logging.tf

@@ -1,6 +1,16 @@
+terraform {
+  required_version = ">= 1.3"
+
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = "~> 3.0"
+    }
+  }
+}
+
 provider "aws" {
-  version = "~> 3.0"
-  region  = "us-west-2"
+  region = "us-west-2"
 }
 
 module "acs" {
@@ -14,7 +24,7 @@ data "aws_elb_service_account" "main" {}
 //  name = "fake-example-cluster"
 //}
 module "fargate_api" {
-  source = "github.com/byu-oit/terraform-aws-fargate-api?ref=v4.0.1"
+  source = "github.com/byu-oit/terraform-aws-fargate-api?ref=v5.0.0"
   //  source           = "../../" // for local testing
   app_name = "example-api"
   //  ecs_cluster_name = aws_ecs_cluster.existing.name
@@ -29,18 +39,11 @@ module "fargate_api" {
     secrets = {
       foo = "/super-secret"
     }
-    efs_volume_mounts = null
-    ulimits           = null
   }
 
-  autoscaling_config            = null
   codedeploy_test_listener_port = 8443
   codedeploy_lifecycle_hooks = {
-    BeforeInstall         = null
-    AfterInstall          = null
     AfterAllowTestTraffic = "testLifecycle"
-    BeforeAllowTraffic    = null
-    AfterAllowTraffic     = null
   }
 
   hosted_zone                   = module.acs.route53_zone

examples/simple/simple.tf

@@ -1,6 +1,16 @@
+terraform {
+  required_version = ">= 1.3"
+
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = "~> 3.0"
+    }
+  }
+}
+
 provider "aws" {
-  version = "~> 3.0"
-  region  = "us-west-2"
+  region = "us-west-2"
 }
 
 module "acs" {
@@ -11,8 +21,8 @@ module "acs" {
 //  name = "fake-example-cluster"
 //}
 module "fargate_api" {
-  source = "github.com/byu-oit/terraform-aws-fargate-api?ref=v4.0.1"
-  //  source           = "../../" // for local testing
+  source = "github.com/byu-oit/terraform-aws-fargate-api?ref=v5.0.0"
+  // source   = "../../" // for local testing
   app_name = "example-api"
   //  ecs_cluster_name = aws_ecs_cluster.existing.name
   container_port = 8000
@@ -26,18 +36,11 @@ module "fargate_api" {
     secrets = {
       foo = "/super-secret"
     }
-    efs_volume_mounts = null
-    ulimits           = null
   }
 
-  autoscaling_config            = null
   codedeploy_test_listener_port = 8443
   codedeploy_lifecycle_hooks = {
-    BeforeInstall         = null
-    AfterInstall          = null
     AfterAllowTestTraffic = "testLifecycle"
-    BeforeAllowTraffic    = null
-    AfterAllowTraffic     = null
   }
 
   hosted_zone                   = module.acs.route53_zone

main.tf

@@ -1,5 +1,5 @@
 terraform {
-  required_version = ">= 0.12.21"
+  required_version = ">= 1.3.0"
   required_providers {
     aws = ">= 3.0.0"
   }

variables.tf

@@ -12,19 +12,19 @@ variable "primary_container_definition" {
     name                  = string
     image                 = string
     ports                 = list(number)
-    environment_variables = map(string)
-    secrets               = map(string)
-    efs_volume_mounts = list(object({
+    environment_variables = optional(map(string))
+    secrets               = optional(map(string))
+    efs_volume_mounts = optional(list(object({
       name           = string
       file_system_id = string
       root_directory = string
       container_path = string
-    }))
-    ulimits = list(object({
+    })))
+    ulimits = optional(list(object({
       name       = string
       soft_limit = number
       hard_limit = number
-    }))
+    })))
   })
   description = "The primary container definition for your application. This one will be the only container that receives traffic from the ALB, so make sure the 'ports' field contains the same port as the 'image_port'"
 }
@@ -33,19 +33,19 @@ variable "extra_container_definitions" {
     name                  = string
     image                 = string
     ports                 = list(number)
-    environment_variables = map(string)
-    secrets               = map(string)
-    efs_volume_mounts = list(object({
+    environment_variables = optional(map(string))
+    secrets               = optional(map(string))
+    efs_volume_mounts = optional(list(object({
       name           = string
       file_system_id = string
       root_directory = string
       container_path = string
-    }))
-    ulimits = list(object({
+    })))
+    ulimits = optional(list(object({
       name       = string
       soft_limit = number
       hard_limit = number
-    }))
+    })))
   }))
   description = "A list of extra container definitions. Defaults to []"
   default     = []
@@ -156,11 +156,11 @@ variable "codedeploy_test_listener_port" {
 }
 variable "codedeploy_lifecycle_hooks" {
   type = object({
-    BeforeInstall         = string
-    AfterInstall          = string
-    AfterAllowTestTraffic = string
-    BeforeAllowTraffic    = string
-    AfterAllowTraffic     = string
+    BeforeInstall         = optional(string)
+    AfterInstall          = optional(string)
+    AfterAllowTestTraffic = optional(string)
+    BeforeAllowTraffic    = optional(string)
+    AfterAllowTraffic     = optional(string)
   })
   description = "Define Lambda Functions for CodeDeploy lifecycle event hooks. Or set this variable to null to not have any lifecycle hooks invoked. Defaults to null"
   default     = null
@@ -211,6 +211,7 @@ variable "autoscaling_config" {
     max_capacity = number
   })
   description = "Configuration for default autoscaling policies and alarms. Set to null if you want to set up your own autoscaling policies and alarms."
+  default     = null
 }
 variable "scaling_up_policy_config" {
   type = object({