int128 / terraform-aws-nat-instance

Terraform module to provision a NAT Instance using an Auto Scaling Group and Spot Instance from $1/month
https://registry.terraform.io/modules/int128/nat-instance/aws/
Apache License 2.0
175 stars 90 forks source link

Documentation change for VPC #9

Closed juicedM3 closed 4 years ago

juicedM3 commented 4 years ago

Following your documentation, I was getting an error that azs was empty for module.vpc. Maybe it's just a newer version of the VPC module? I was able to run things after this change:

data "aws_availability_zones" "available" {
  state = "available"
}

module "vpc" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "2.17.0"

  name               = "test-vpc"
  azs                  = formatlist("%s",data.aws_availability_zones.available.names)
  cidr                 = "172.18.0.0/16"
  private_subnets      = ["172.18.64.0/20", "172.18.80.0/20", "172.18.96.0/20"]
  public_subnets       = ["172.18.128.0/20", "172.18.144.0/20", "172.18.160.0/20"]
  enable_dns_hostnames = true
}
int128 commented 4 years ago

Thank you for opening the issue. I just fixed the example.