[helm] - Add deploymentStrategy to user deployment helm chart

[marijncv]

Summary & Motivation

Closes #26445

How I Tested These Changes

Added a test and ran helm template on a values file like:

deployments:
  - name: "k8s-example-user-code-1"
    image:
      repository: "docker.io/dagster/user-code-example"
      tag: ~
      pullPolicy: Always
    dagsterApiGrpcArgs:
      - "-f"
      - "/example_project/example_repo/repo.py"
    port: 3030
    deploymentStrategy:
      type: Recreate

[ion-elgreco]

This is very useful! This change would simplify our dagster user code deployment cli

[gibsondan]

Hi @marijncv , thanks for sending this out. This looks good to me but it looks like it may need a rebase first due to some conflicts.

[gibsondan]

(apologies for the delay, I imagine those conflicts are new since you first sent this out)

[marijncv]

@gibsondan no worries, should be fixed now!

[gibsondan]

@marijncv something has gone a little strange with the rebase here I think in a way that is confusing our CI - it looks like this PR is currently trying to do a third merge commit on top of your original two changes, but I think you want to rebase so that you're only trying to merge two commits into the current mas ter.

[gibsondan]

here's an example external PR where the commits are lined up correctly: #26893 (there's no additional merge PR if you click on Commits, just the commits that are intended to merge into master)

If it's easier to just send our a new PR that would be fine too

[marijncv]

@gibsondan ah my mistake. I tried to force push a rebase but it still seems to cause some issues in the CI so I created another PR here. Hope that one works and thanks for your patience!