Merge pull request #3 from osodevops/update-route53

Updated the vpc module to allow for the route 53 association to compl…

aws_eip.tf

@@ -1,4 +1,5 @@
 resource "aws_eip" "nateip" {
   count = 3
+  vpc   = true
   tags  = "${var.common_tags}"
 }

aws_nat_gateway.tf

@@ -2,7 +2,6 @@ resource "aws_nat_gateway" "natgateway" {
   count         = 3
   allocation_id = "${element(aws_eip.nateip.*.id, count.index)}"
   subnet_id     = "${element(aws_subnet.public.*.id, count.index)}"
-  public_ip     = "${element(aws_eip.nateip.*.public_ip, count.index)}"
 
   tags {
     Name = "${var.account_name}-AZ${count.index+1}"

aws_route53_zone_private.tf

@@ -1,6 +1,7 @@
 resource "aws_route53_zone" "private" {
-  comment = "Managed by Terraform"
   name    = "${lower(var.account_name)}.${data.aws_region.current.name}.cloud."
+  comment = "Managed by Terraform"
+
   vpc {
     vpc_id     = "${aws_vpc.main.id}"
     vpc_region = "${data.aws_region.current.name}"

aws_route53_zone_ptr.tf

@@ -1,6 +1,7 @@
 resource "aws_route53_zone" "reverse" {
   name       = "10.in-addr.arpa"
   comment    = "Reverse DNS Lookup for ${upper(var.account_name)} - Managed by Terraform"
+
   vpc {
     vpc_id     = "${aws_vpc.main.id}"
     vpc_region = "${data.aws_region.current.name}"