From fca069c5c8875a061624ab9e7724768f4a986172 Mon Sep 17 00:00:00 2001 From: badra001 Date: Mon, 30 Jan 2023 15:24:13 -0500 Subject: [PATCH] fix --- .../full-setup-tf-upgrade/variables.availability_zones.tf | 5 +++++ vpc-transit-gateway-association/common/README.md | 2 +- vpc-transit-gateway-association/common/variables.routing.tf | 2 +- vpc-transit-gateway-association/peer/README.md | 2 +- vpc-transit-gateway-association/peer/associate.tf | 1 - 5 files changed, 8 insertions(+), 4 deletions(-) create mode 100644 examples/full-setup-tf-upgrade/variables.availability_zones.tf diff --git a/examples/full-setup-tf-upgrade/variables.availability_zones.tf b/examples/full-setup-tf-upgrade/variables.availability_zones.tf new file mode 100644 index 0000000..3e906b9 --- /dev/null +++ b/examples/full-setup-tf-upgrade/variables.availability_zones.tf @@ -0,0 +1,5 @@ +variable "availability_zones" { + description = "AWS Availability zones for subnet (default: all)" + type = list(string) + default = [] +} diff --git a/vpc-transit-gateway-association/common/README.md b/vpc-transit-gateway-association/common/README.md index 0ee1a74..5c96ec6 100644 --- a/vpc-transit-gateway-association/common/README.md +++ b/vpc-transit-gateway-association/common/README.md @@ -21,7 +21,7 @@ No resources. | [create\_prefix\_list\_routing](#input\_create\_prefix\_list\_routing) | Flag to create (or not) prefix list routing. This is to be applied only on the TGW main account and VPCs | `bool` | `false` | no | | [create\_static\_peer\_routing](#input\_create\_static\_peer\_routing) | Flag to create (or not) static peer. This can be applied on every account including the TGW main account. This conflicts with craete\_prefix\_list\_routing | `bool` | `false` | no | | [data\_input](#input\_data\_input) | Map of data generated by vpc-transit-gateway-association-data | 
object({
    availablity_zone          = map(any)
    gateway_self              = string
    gateway_peer              = string
    route_tables_self         = map(any)
    route_tables_peer         = map(any)
    map_route_tables_self     = map(any)
    map_route_tables_peer     = map(any)
    map_vpn_route_tables_self = map(any)
    map_vpn_route_tables_peer = map(any)
    prefix_list_id_ipv4       = string
    vpn_prefix_list_id_ipv4   = string
    vpc_id                    = string
    vpc_cidr_block            = string
    vpc_cidr_blocks           = list(string)
  })
| n/a | yes | -| [enable\_vpn\_routing](#input\_enable\_vpn\_routing) | Flag to enable VPN routing, handled through a prefix list. This is used in the transition from per-VPC VPNs to TGW | `bool` | `false` | no | +| [enable\_vpn\_routing](#input\_enable\_vpn\_routing) | Flag to enable VPN routing, handled through a prefix list. This is used in the transition from per-VPC VPNs to TGW | `bool` | `true` | no | ## Outputs diff --git a/vpc-transit-gateway-association/common/variables.routing.tf b/vpc-transit-gateway-association/common/variables.routing.tf index 92c41e2..e1620ff 100644 --- a/vpc-transit-gateway-association/common/variables.routing.tf +++ b/vpc-transit-gateway-association/common/variables.routing.tf @@ -13,5 +13,5 @@ variable "create_static_peer_routing" { variable "enable_vpn_routing" { description = "Flag to enable VPN routing, handled through a prefix list. This is used in the transition from per-VPC VPNs to TGW" type = bool - default = false + default = true } diff --git a/vpc-transit-gateway-association/peer/README.md b/vpc-transit-gateway-association/peer/README.md index 72d328c..27bf6e5 100644 --- a/vpc-transit-gateway-association/peer/README.md +++ b/vpc-transit-gateway-association/peer/README.md @@ -98,7 +98,7 @@ No modules. | [create\_prefix\_list\_routing](#input\_create\_prefix\_list\_routing) | Flag to create (or not) prefix list routing. This is to be applied only on the TGW main account and VPCs | `bool` | `false` | no | | [create\_static\_peer\_routing](#input\_create\_static\_peer\_routing) | Flag to create (or not) static peer. This can be applied on every account including the TGW main account. This conflicts with craete\_prefix\_list\_routing | `bool` | `false` | no | | [data\_input](#input\_data\_input) | Map of data generated by vpc-transit-gateway-association-data | 
object({
    availablity_zone          = map(any)
    gateway_self              = string
    gateway_peer              = string
    route_tables_self         = map(any)
    route_tables_peer         = map(any)
    map_route_tables_self     = map(any)
    map_route_tables_peer     = map(any)
    map_vpn_route_tables_self = map(any)
    map_vpn_route_tables_peer = map(any)
    prefix_list_id_ipv4       = string
    vpn_prefix_list_id_ipv4   = string
    vpc_id                    = string
    vpc_cidr_block            = string
    vpc_cidr_blocks           = list(string)
  })
| n/a | yes | -| [enable\_vpn\_routing](#input\_enable\_vpn\_routing) | Flag to enable VPN routing, handled through a prefix list. This is used in the transition from per-VPC VPNs to TGW | `bool` | `false` | no | +| [enable\_vpn\_routing](#input\_enable\_vpn\_routing) | Flag to enable VPN routing, handled through a prefix list. This is used in the transition from per-VPC VPNs to TGW | `bool` | `true` | no | | [network\_account\_profile](#input\_network\_account\_profile) | AWS profile of the source account sharing the VPC resources | `string` | n/a | yes | | [override\_prefixes](#input\_override\_prefixes) | Override built-in prefixes by component. This should be used primarily for common infrastructure things | `map(string)` | `{}` | no | | [private\_route\_table\_ids](#input\_private\_route\_table\_ids) | Map of private route table IDs (which exclude the attachment subnets) | `map(string)` | `{}` | no | diff --git a/vpc-transit-gateway-association/peer/associate.tf b/vpc-transit-gateway-association/peer/associate.tf index 9aff5f0..ee17628 100644 --- a/vpc-transit-gateway-association/peer/associate.tf +++ b/vpc-transit-gateway-association/peer/associate.tf @@ -43,4 +43,3 @@ resource "aws_ec2_transit_gateway_route" "vpc_peer_all" { transit_gateway_route_table_id = each.value } -