Azure / terraform-azure-container-apps

A Terraform module to deploy a container app in Azure
https://github.com/Azure/terraform-azure-container-apps
35 stars 22 forks source link

Add `precondition` for `var.container_app_environment_internal_load_balancer_enabled` #33

Closed lonegunmanb closed 7 months ago

lonegunmanb commented 8 months ago

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 set infrastructure_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 to null 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

Thanks for your cooperation!