search

Azure / azure-rest-api-specs

The source for REST API specifications for Microsoft Azure.
MIT License
2.68k stars 5.1k forks source link

Container Apps seems to be missing appProtocol #17144

Open XavierGeerinck opened 2 years ago

XavierGeerinck commented 2 years ago

Hi All,

I would like to create an application that uses gRPC instead of HTTP (as supported by Dapr). It however seems that port and id can be passed but protocol cannot even though the dapr docs state that --app-protocol can be passed to switch between http or grpc

Note: Protocol is crucial as the Python SDK doesn't support HTTP for anything else than Actors. So, for most workers in the AI space, gRPC will have to be used.

Potential fix: add appProtocol to the CommonDefinitions.json but then this has to be added in the backend I suppose?

https://github.com/Azure/azure-rest-api-specs/blob/c0a3d9e02bef53de9b9dcd43e9be3a460d96a4e0/specification/web/resource-manager/Microsoft.Web/stable/2021-03-01/CommonDefinitions.json#L828-L853

ghost commented 2 years ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @AzureAppServiceCLI, @antcp.

Issue Details
Hi All, I would like to create an application that uses gRPC instead of HTTP (as supported by [Dapr](https://dapr.io)). It however seems that `port` and `id` can be passed but `protocol` cannot even though the [dapr docs](https://docs.dapr.io/reference/cli/dapr-run/) state that `--app-protocol` can be passed to switch between `http` or `grpc` Note: Protocol is crucial as the Python SDK doesn't support HTTP for anything else than Actors. So, for most workers in the AI space, gRPC will have to be used. Potential fix: add `appProtocol` to the `CommonDefinitions.json` but then this has to be added in the backend I suppose? https://github.com/Azure/azure-rest-api-specs/blob/c0a3d9e02bef53de9b9dcd43e9be3a460d96a4e0/specification/web/resource-manager/Microsoft.Web/stable/2021-03-01/CommonDefinitions.json#L828-L853
Author: XavierGeerinck
Assignees: -
Labels: `question`, `Service Attention`, `customer-reported`, `Web Apps`, `needs-triage`
Milestone: -
XavierGeerinck commented 2 years ago

Any update?

ghost commented 2 years ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @AzureAppServiceCLI, @antcp.

Issue Details
Hi All, I would like to create an application that uses gRPC instead of HTTP (as supported by [Dapr](https://dapr.io)). It however seems that `port` and `id` can be passed but `protocol` cannot even though the [dapr docs](https://docs.dapr.io/reference/cli/dapr-run/) state that `--app-protocol` can be passed to switch between `http` or `grpc` Note: Protocol is crucial as the Python SDK doesn't support HTTP for anything else than Actors. So, for most workers in the AI space, gRPC will have to be used. Potential fix: add `appProtocol` to the `CommonDefinitions.json` but then this has to be added in the backend I suppose? https://github.com/Azure/azure-rest-api-specs/blob/c0a3d9e02bef53de9b9dcd43e9be3a460d96a4e0/specification/web/resource-manager/Microsoft.Web/stable/2021-03-01/CommonDefinitions.json#L828-L853
Author: XavierGeerinck
Assignees: -
Labels: `question`, `Service Attention`, `customer-reported`, `Web Apps`
Milestone: -
navba-MSFT commented 2 years ago

@XavierGeerinck Apologies for the late reply. Thanks for reaching out to us and sharing this feedback. I have marked our Service Team in this issue. @AzureAppServiceCli @antcp Could you please look into this ask ?