Skip to content

Conversation

@Benjiiim
Copy link
Contributor

@Benjiiim Benjiiim commented Mar 19, 2024

Why make this change?

To avoid the WorkloadProfilePropertyNotSupportedInApiVersion that can occur with containerApp API version 2022-10-01 when deploying with the bicep file.

Error with the az deployment group create command in the azure-container-apps-deploy.sh file:

ERROR: {"status":"Failed","error":{"code":"DeploymentFailed","target":"/subscriptions/xx-xx-xx-xx-xx/resourceGroups/Alpha_API_Tests_Auto_6/providers/Microsoft.Resources/deployments/dab-on-aca","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.","details":[{"code":"WorkloadProfilePropertyNotSupportedInApiVersion","message":"WorkloadProfiles configuration is not supported in 2022-10-01 api version. Use 2022-11-01-preview or above."}]}}

What is this change?

Upgrade the containerApp API version in the bicep file to the last stable version.

How was this tested?

Manual testing

Sample Request(s)

azure-container-apps-deploy.sh

Avoid the WorkloadProfilePropertyNotSupportedInApiVersion that can occur with 2022-10-01
@seantleonard
Copy link
Contributor

Hi @Benjiiim , can you highlight what documentation step failed with the old value? Are there any doc updates required to align with this api version bump?

@Benjiiim
Copy link
Contributor Author

Details added above.

No doc update needed as the corresponding doc is only mentioning the existence of this sample:

A sample shell script that can be run on Linux (using the [Cloud Shell](https://learn.microsoft.com/en-us/azure/cloud-shell/overview) if you don't have a Linux machine or WSL installed) is available in [/samples/azure](https://github.com/Azure/data-api-builder/tree/main/samples/azure) folder.

Copy link
Contributor

@aaronburtle aaronburtle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@seantleonard seantleonard enabled auto-merge (squash) March 26, 2024 18:27
@seantleonard
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@seantleonard seantleonard merged commit aad9c17 into Azure:main Mar 26, 2024
@Benjiiim Benjiiim deleted the patch-1 branch October 3, 2025 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants