Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to deploy to a container app that uses workload profiles #2402

Closed
1 task done
ruslany opened this issue Jun 13, 2023 · 5 comments · Fixed by #2412
Closed
1 task done

Unable to deploy to a container app that uses workload profiles #2402

ruslany opened this issue Jun 13, 2023 · 5 comments · Fixed by #2412
Assignees
Labels
aca Azure Container Apps

Comments

@ruslany
Copy link

ruslany commented Jun 13, 2023

Output from azd version
azd version 1.0.1 (commit e0cd1ac)

Describe the bug
When trying to deploy to a container app that uses workload profiles I get this error:

ERROR: failed deploying service 'acachat': failing invoking action 'deploy', updating container app service: updating container app revision: begin updating ingress traffic: PATCH https://management.azure.com/subscriptions/<sub>/resourceGroups/<rg>/providers/Microsoft.App/containerApps/<appname>
--------------------------------------------------------------------------------
RESPONSE 400: 400 Bad Request
ERROR CODE: WorkloadProfilePropertyNotSupportedInApiVersion
--------------------------------------------------------------------------------
{
  "error": {
    "code": "WorkloadProfilePropertyNotSupportedInApiVersion",
    "message": "WorkloadProfiles configuration is not supported in  api version. Use 2022-11-01-preview or above."
  }
}
--------------------------------------------------------------------------------

TraceID: 3cfceb3ea54d1ceb7c9c46876069b8c3

To Reproduce
Use api version 2022-11-01-preview in the bicep files for creating containerapp environment and apps. Run azd provision. Then run azd deploy

Expected behavior
azd deploy should work with workload profiles or there should be a way to specify an API version to use when PATCH a container app.

Environment
Information on your environment:
* Language name and version - py
* IDE and version : Visual Studio Code

@ghost ghost added the needs-triage For new issues label Jun 13, 2023
@rajeshkamal5050 rajeshkamal5050 added the aca Azure Container Apps label Jun 13, 2023
@ghost ghost removed the needs-triage For new issues label Jun 13, 2023
@rajeshkamal5050
Copy link

rajeshkamal5050 commented Jun 13, 2023

@wbreza is this a change in the Bicep which users could modify or something in the core azd bits? Please adjust milestone based on findings.

@wbreza
Copy link
Contributor

wbreza commented Jun 14, 2023

@ruslany Please take a look at the private azd build from PR #2412 and confirm that this addresses your issue. Thank you for reporting this!

wbreza added a commit that referenced this issue Jun 16, 2023
Updates Azure SDK for go armappcontainers module to v2-beta to support workload profiles

Fixes #2402
@savannahostrowski
Copy link
Contributor

@wbreza Is this okay to close now, given that we have #2413?

@wbreza
Copy link
Contributor

wbreza commented Jul 7, 2023

@wbreza Is this okay to close now, given that we have #2413?

@savannahostrowski This one is already closed :) Let me know if I misunderstood the question.

@cyptus
Copy link

cyptus commented Nov 23, 2023

@ruslany how can i influence the environment type after this patch (from an azure.yaml or using DistributedApplication builder)? i dont find any documentation of how you did this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aca Azure Container Apps
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants