Skip to content

Commit

Permalink
not needed to create auth with cluster phz (#22)
Browse files Browse the repository at this point in the history
  • Loading branch information
morga471 committed Mar 18, 2026
1 parent 70fbf9e commit aff22dc
Show file tree
Hide file tree
Showing 2 changed files with 38 additions and 42 deletions.
4 changes: 0 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -45,15 +45,11 @@ No modules.
| [aws_route53_record.entry](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_record) | resource |
| [aws_route53_vpc_association_authorization.central_zone_east](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_vpc_association_authorization) | resource |
| [aws_route53_vpc_association_authorization.central_zone_west](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_vpc_association_authorization) | resource |
| [aws_route53_vpc_association_authorization.cluster_zone_east](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_vpc_association_authorization) | resource |
| [aws_route53_vpc_association_authorization.cluster_zone_west](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_vpc_association_authorization) | resource |
| [aws_route53_vpc_association_authorization.legacy_zone_east](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_vpc_association_authorization) | resource |
| [aws_route53_vpc_association_authorization.legacy_zone_west](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_vpc_association_authorization) | resource |
| [aws_route53_zone.cluster_domain](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_zone) | resource |
| [aws_route53_zone_association.central_zone_east](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_zone_association) | resource |
| [aws_route53_zone_association.central_zone_west](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_zone_association) | resource |
| [aws_route53_zone_association.cluster_zone_east](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_zone_association) | resource |
| [aws_route53_zone_association.cluster_zone_west](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_zone_association) | resource |
| [aws_route53_zone_association.legacy_zone_east](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_zone_association) | resource |
| [aws_route53_zone_association.legacy_zone_west](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_zone_association) | resource |
| [null_resource.git_version](https://registry.terraform.io/providers/hashicorp/null/latest/docs/resources/resource) | resource |
Expand Down
76 changes: 38 additions & 38 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -40,51 +40,51 @@ resource "aws_route53_zone" "cluster_domain" {
)
}

#---
# cluster domain associations with eks vpc
# east region
#---
resource "aws_route53_vpc_association_authorization" "cluster_zone_east" {
count = local.region != var.region_map["east"] ? 1 : 0

provider = aws.route53_self
vpc_id = local.vpc_id
vpc_region = var.region_map["east"]
zone_id = aws_route53_zone.cluster_domain.zone_id
}
# #---
# # cluster domain associations with eks vpc
# # east region
# #---
# resource "aws_route53_vpc_association_authorization" "cluster_zone_east" {
# count = local.region != var.region_map["east"] ? 1 : 0

# provider = aws.route53_self
# vpc_id = local.vpc_id
# vpc_region = var.region_map["east"]
# zone_id = aws_route53_zone.cluster_domain.zone_id
# }

resource "aws_route53_zone_association" "cluster_zone_east" {
count = local.region != var.region_map["east"] ? 1 : 0
# resource "aws_route53_zone_association" "cluster_zone_east" {
# count = local.region != var.region_map["east"] ? 1 : 0

provider = aws.route53_main
vpc_id = local.vpc_id
vpc_region = var.region_map["east"]
zone_id = aws_route53_zone.cluster_domain.zone_id
# provider = aws.route53_main
# vpc_id = local.vpc_id
# vpc_region = var.region_map["east"]
# zone_id = aws_route53_zone.cluster_domain.zone_id

depends_on = [aws_route53_vpc_association_authorization.cluster_zone_east]
}
# depends_on = [aws_route53_vpc_association_authorization.cluster_zone_east]
# }

#-------------------------------------------------
# cluster PHZ Association west region
#-------------------------------------------------
resource "aws_route53_vpc_association_authorization" "cluster_zone_west" {
count = local.region != var.region_map["west"] ? 1 : 0
# #-------------------------------------------------
# # cluster PHZ Association west region
# #-------------------------------------------------
# resource "aws_route53_vpc_association_authorization" "cluster_zone_west" {
# count = local.region != var.region_map["west"] ? 1 : 0

provider = aws.route53_self
vpc_id = local.vpc_id
vpc_region = var.region_map["west"]
zone_id = aws_route53_zone.cluster_domain.zone_id
}
# provider = aws.route53_self
# vpc_id = local.vpc_id
# vpc_region = var.region_map["west"]
# zone_id = aws_route53_zone.cluster_domain.zone_id
# }

resource "aws_route53_zone_association" "cluster_zone_west" {
count = local.region != var.region_map["west"] ? 1 : 0
provider = aws.route53_main
vpc_id = local.vpc_id
vpc_region = var.region_map["west"]
zone_id = aws_route53_zone.cluster_domain.zone_id
# resource "aws_route53_zone_association" "cluster_zone_west" {
# count = local.region != var.region_map["west"] ? 1 : 0
# provider = aws.route53_main
# vpc_id = local.vpc_id
# vpc_region = var.region_map["west"]
# zone_id = aws_route53_zone.cluster_domain.zone_id

depends_on = [aws_route53_vpc_association_authorization.cluster_zone_west]
}
# depends_on = [aws_route53_vpc_association_authorization.cluster_zone_west]
# }

#---
# cluster domain associations with central networking account
Expand Down

0 comments on commit aff22dc

Please sign in to comment.