Add option to set DAPR protocol for Container Apps

[fatim]

This PR closes #1083

The changes in this PR are as follows:

• Container Apps: Add support for setting Dapr protocol.

I have read the contributing guidelines and have completed the following:

• Tested my code end-to-end against a live Azure subscription.
• Updated the documentation in the docs folder for the affected changes.
• Written unit tests against the modified code that I have made.
• Updated the release notes with a new entry for this PR.
• Checked the coding standards outlined in the contributions guide and ensured my code adheres to them.

If I haven't completed any of the tasks above, I include the reasons why here:

I don't have experience using container apps with Dapr in Azure.
This change produces correct ARM config, I've used this doc as a reference - https://learn.microsoft.com/en-us/azure/container-apps/enable-dapr?tabs=arm1
Could somebody test this in Azure.

Below is a minimal example configuration that includes the new features, which can be used to deploy to Azure:

let httpContainerApp =
    containerApp {
        ...
        // Dapr config
        dapr_app_id "http"
        dapr_app_port 5000us

        // new parameter added by this PR
        dapr_app_protocol "grpc"
        ...
    }

[jwthomson]

Thanks for this contribution!

This looks great in principle. @isaacabraham is going to have a look at it, apologies for the delay.

[isaacabraham]

This looks great - just a small change which will add some type safety and improve usability.

[isaacabraham]

I would make a type for the protocol e.g.

[<RequireQualifiedAccess>]
type DaprProtocol = Grpc | Http

Doing this will make it easier for individuals to know what the available valid options are, rather than a raw string.

[isaacabraham]

With Protocol now a union rather than a string, this code will need a pattern match to emit the correct string e.g.

match settings.Protocol with Http -> "http" | Gprc -> "grpc"

etc.

[mattgallagher92]

Hi @fatim, thanks for your contribution 🙂 Would you like to make the changes that Isaac suggested, or would you prefer that someone else picks up this PR?

[isaacabraham]

Let's give this another week - if it's not worked on, we should close this PR but add an issue to revisit it at a later date.