Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not try to create the DNS records when aliases is specified. #70

Open
vikas027 opened this issue Sep 23, 2021 · 2 comments
Open

Do not try to create the DNS records when aliases is specified. #70

vikas027 opened this issue Sep 23, 2021 · 2 comments

Comments

@vikas027
Copy link

Describe the Feature

Do not try to create the DNS records when aliases is specified.

Expected Behavior

I should be able to use aliases without creating them using this module.

Use Case

As of now, if you specify aliases, the module tries to find the R53 zone and create the record. In my case, my R53 zone is in a different account which is why fails with the below error

  on .terraform/modules/waf.cdn.dns/main.tf line 1, in data "aws_route53_zone" "default":
   1: data "aws_route53_zone" "default" {

Describe Ideal Solution

I would like to have one of these options

  • Use of aliases without creating the R53 record
  • Option to create an alias by using a different provider (with a different region). This would require to pass two was providers to the module

Alternatives Considered

Create the aliases manually or some script outside of the terraform module

@nnsense
Copy link

nnsense commented Nov 28, 2022

Isn’t this option doing that?

dns_aliases_enabled: Set to false to prevent dns records for aliases from being created

@HectorQuiroz
Copy link

Isn’t this option doing that?

dns_aliases_enabled: Set to false to prevent dns records for aliases from being created

This does worked for me. I think we can close this Issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants