Merge pull request #1 from silinternational/aws-5

Release 1.0.0 -- upgrade to AWS provider version 5

main.tf

@@ -103,7 +103,7 @@ resource "aws_eip" "gateway_eip" {
     app_name = var.app_name
     app_env  = var.app_env
   }
-  vpc = true
+  domain = "vpc"
 }
 
 resource "aws_nat_gateway" "nat_gateway" {

variables.tf

@@ -2,19 +2,20 @@
  * Required variables
  */
 variable "app_name" {
-  type    = string
-  default = "terraform"
+  description = "Application name, used with app_env to create resource names"
+  type        = string
+  default     = "terraform"
 }
 
 variable "app_env" {
-  type    = string
-  default = "testing"
+  description = "Application environment, used with app_name to create resource names"
+  type        = string
+  default     = "testing"
 }
 
 variable "aws_zones" {
   description = "A list of zones to create subnets (max 8)"
   type        = list(string)
-
   default = [
     "us-east-1c",
     "us-east-1d",
@@ -23,8 +24,9 @@ variable "aws_zones" {
 }
 
 variable "enable_dns_hostnames" {
-  type    = bool
-  default = false
+  description = "A boolean flag to enable/disable DNS hostnames in the VPC. Defaults false."
+  type        = bool
+  default     = false
 }
 
 variable "create_nat_gateway" {

versions.tf

@@ -5,7 +5,7 @@ terraform {
   required_providers {
     aws = {
       source  = "hashicorp/aws"
-      version = ">= 4.0.0, < 5.0.0"
+      version = ">= 5.0.0, < 6.0.0"
     }
   }
 }