Add precondition
for var.container_app_environment_internal_load_balancer_enabled
#33
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe your changes
It looks like the azurerm provider has introduced a new constriction on
internal_load_balancer_enabled
, once it has been set no matter the value, you must setinfrastructure_subnet_id
too or an error will be thrown. I've opened a pr in provider's repo for this issue, this pr changed the variable's default value tonull
to avoid the error caused by default values.Update: since #39 has already updated the variable's default value, this pr only add a
precondition
block for this variable to provide a better user experience.Issue number
#32 #40
Checklist before requesting a review
CHANGELOG.md
fileThanks for your cooperation!