search

Azure / terraform-azurerm-avm-res-app-managedenvironment

AVM Terraform module for Container Apps Managed Environments
MIT License
7 stars 6 forks source link

workload profile fix #87

Open kewalaka opened 1 month ago

kewalaka commented 1 month ago

From @alefteris

Setting a workload profile wasn't working, because the check for consumption profile to set workload_profiles_consumption_enabled didn't account for the fact that the workload_profile variable includes also the max/min count with default values. So I fixed this part. Also fixed is the part that sets the workloadProfiles for the azapi resource to use the correct format.

The rest of the changes are white-space changes from running the avm pre-commit and adding the modtm provider that was needed for the module to work, because of previous commits in main the branch.

From @kewalaka

Added linting fixes to appease pr-check

kewalaka commented 3 weeks ago

@segraef do you have any feedback? I don't believe I can trigger the E2E run