Closed iammuho closed 4 months ago
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.
Description
This pull request updates the
customer_gateways
variable to use an object type instead ofany
, as outlined in Terraform's best practices. By providing an object structure formap
, the configuration becomes clearer and easier to validate, leading to more predictable outcomes.Official : https://developer.hashicorp.com/terraform/language/expressions/type-constraints#dynamic-types-the-any-constraint
Motivation and Context
bgp_asn
andip_address
are consistently provided.any
type, which is discouraged due to the absence of specific typing.Breaking Changes
bgp_asn
andip_address
), they will encounter validation errors during deployment.null
or another specified default value, which should be accounted for in downstream logic.customer_gateways
definitions and update them to match the new object structure to ensure successful deployment.How Has This Been Tested?
[ ] I have updated at least one of the
examples/*
projects to verify this change.[x] I have tested and validated these changes using one or more of the provided
examples/*
projects:[ ] I have executed
pre-commit run -a
on this pull request to ensure compliance.[ ] I have verified these changes do not introduce breaking errors in deployment.