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

Fix bug - dns_zone_id not available during planning #228

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

tsaodown
Copy link

@tsaodown tsaodown commented Mar 2, 2023

what

  • dns_zone_id now accepts both string and list(string)

why

  • When spinning up a new EB environment pointing at a Terraform-managed Route 53 hosted zone, an "Invalid Count Argument" error is thrown

references

@tsaodown tsaodown requested review from a team as code owners March 2, 2023 03:39
@hans-d
Copy link

hans-d commented Mar 2, 2024

/terratest

Copy link
Contributor

mergify bot commented Mar 9, 2024

Thanks @tsaodown for creating this pull request!

A maintainer will review your changes shortly. Please don't be discouraged if it takes a while.

While you wait, make sure to review our contributor guidelines.

Tip

Need help or want to ask for a PR review to be expedited?

Join us on Slack in the #pr-reviews channel.

@mergify mergify bot added the triage Needs triage label Mar 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage Needs triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants