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 Route53 Hosted Zone Output #550

Merged
merged 2 commits into from
Feb 19, 2024
Merged

Conversation

joseapeinado
Copy link
Contributor

What?

It changes the ooutput value on Hosted Zones from list to individual string

Why?

Bugfixing

References

Closes #18

@joseapeinado joseapeinado requested a review from a team as a code owner January 25, 2024 16:55
Copy link

Infracost report

💰 Monthly cost will not change

Cost details
──────────────────────────────────
Project: binbashar/le-tf-infra-aws/apps-devstg/us-east-1/databases-aurora/rds-export-to-s3
Module path: apps-devstg/us-east-1/databases-aurora/rds-export-to-s3
Errors:
  Passed var file does not exist:
    apps-devstg/us-east-1/config/backend.tfvars. Make sure you are passing the var file path relative to the --path flag.
  Diff baseline error:
    Passed var file does not exist:
      apps-devstg/us-east-1/config/backend.tfvars. Make sure you are passing the var file path relative to the --path flag.

──────────────────────────────────
Project: binbashar/le-tf-infra-aws/apps-devstg/us-east-1/databases-mysql --/rds-export-to-s3
Module path: apps-devstg/us-east-1/databases-mysql --/rds-export-to-s3
Errors:
  Passed var file does not exist:
    apps-devstg/us-east-1/config/backend.tfvars. Make sure you are passing the var file path relative to the --path flag.
  Diff baseline error:
    Passed var file does not exist:
      apps-devstg/us-east-1/config/backend.tfvars. Make sure you are passing the var file path relative to the --path flag.

──────────────────────────────────

2538 cloud resources were detected:
∙ 595 were estimated, 456 of which include usage-based costs, see https://infracost.io/usage-file
∙ 1786 were free, rerun with --show-skipped to see details
∙ 157 are not supported yet, rerun with --show-skipped to see details
This comment will be updated when code changes.

@exequielrafaela exequielrafaela merged commit 6fe6057 into master Feb 19, 2024
6 checks passed
@exequielrafaela exequielrafaela deleted the feature/fix_dns_output branch March 6, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

outputs.tf dns value won't be a list
3 participants