Skip to content

Conversation

@jgerry
Copy link
Contributor

@jgerry jgerry commented Feb 1, 2022

Description

Add a new boolean variable route53_private_zone to pass into the ALB module

Motivation and Context

Add support for Route53 private hosted zones. Fixes #249.

Breaking Changes

no breaking changes

How Has This Been Tested?

  • I have tested and validated these changes using one or more of the provided examples/* projects.

I have tested this by setting route53_zone_name to a public hosted zone, the module works as it did before my change. By setting route53_zone_name to a private hosted zone, and setting route53_private_zone to true, I was able to create the appropriate Route53 entry in my private hosted zone.

Terraform v1.1.4
on darwin_amd64
+ provider registry.terraform.io/hashicorp/aws v3.74.0
+ provider registry.terraform.io/hashicorp/local v2.1.0
+ provider registry.terraform.io/hashicorp/random v3.1.0

@jgerry jgerry changed the title add var and config to allow private hosted zone use feat: add var and config to allow private hosted zone use Feb 1, 2022
@jgerry jgerry changed the title feat: add var and config to allow private hosted zone use feat: Add var and config to allow private hosted zone use Feb 1, 2022
@antonbabenko
Copy link
Member

LGTM, please fix the docs by running pre-commit run -a and committing the changes.

Copy link
Member

@bryantbiggs bryantbiggs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, thanks @jgerry

@antonbabenko 👍🏽

@antonbabenko
Copy link
Member

@bryantbiggs Now we have some more extra work with AWS provider version 4. For now, I just locked it to version 3.

@bryantbiggs
Copy link
Member

Ya I just saw that dropped, I'll start taking a look later today 😅

@antonbabenko antonbabenko changed the title feat: Add var and config to allow private hosted zone use feat: Added a possibility to allow private hosted zone usage Feb 10, 2022
@antonbabenko antonbabenko merged commit 5a53f39 into terraform-aws-modules:master Feb 10, 2022
antonbabenko pushed a commit that referenced this pull request Feb 10, 2022
## [3.11.0](v3.10.0...v3.11.0) (2022-02-10)

### Features

* Added a possibility to allow private hosted zone usage ([#250](#250)) ([5a53f39](5a53f39))
@antonbabenko
Copy link
Member

This PR is included in version 3.11.0 🎉

titanlien pushed a commit to Flaconi/terraform-aws-atlantis that referenced this pull request Jun 8, 2022
titanlien pushed a commit to Flaconi/terraform-aws-atlantis that referenced this pull request Jun 8, 2022
titanlien pushed a commit to Flaconi/terraform-aws-atlantis that referenced this pull request Jun 8, 2022
titanlien pushed a commit to Flaconi/terraform-aws-atlantis that referenced this pull request Jun 8, 2022
titanlien pushed a commit to Flaconi/terraform-aws-atlantis that referenced this pull request Jun 9, 2022
titanlien pushed a commit to Flaconi/terraform-aws-atlantis that referenced this pull request Jun 9, 2022
@alin-grecu
Copy link

alin-grecu commented Aug 30, 2022

Sorry if this is not a good place, but I think because of:

zone_id = var.certificate_arn == "" ? element(concat(data.aws_route53_zone.this.*.id, [""]), 0) : ""
and:
private_zone = var.route53_private_zone
if you set route53_private_zone = true AWS is unable to validate ACM because the validation record is created on a private zone.

@github-actions
Copy link

github-actions bot commented Nov 8, 2022

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for Route53 private hosted zones

4 participants