Added Container Apps Workload Profiled #1947 #1948
Conversation
|
Addresses #1947 by adding workload profiles to Container App Environments |
|
@arnaudlh this PR now contains a breaking change because there is a breaking change in version 3.82.0 of terraform-provider-azurerm How should it be handled? |
There was a problem hiding this comment.
hi @darrenhull thanks for the PR, you can see from the pipeline run here, there are a couple of things to address: https://github.com/aztfmod/terraform-azurerm-caf/actions/runs/9353085218/job/25744923700
| @@ -83,6 +83,12 @@ container_app_environments = { | |||
| } | |||
| internal_load_balancer_enabled = true | |||
| zone_redundancy_enabled = true | |||
| workload_profile { | |||
There was a problem hiding this comment.
| workload_profile { | |
| workload_profile = { |
There was a problem hiding this comment.
sorry i didnt get a notification on this i will update it, i think i have a newer version on my fork.
There was a problem hiding this comment.
I have updated this now @arnaudlh is there any documentation on how to run the tests locally?
|
👍 We need this feature |
its working on my forked dev branch if you need it before its pulled in. |
1947
PR Checklist
Description
Does this introduce a breaking change
Testing