chore(argo-rollouts): Change controller deployment strategy to rollinUpdate

[ilyastoliar]

Signed-off-by: Ilya Stoliar [email protected]

Change controller deployment strategy to rollingUpdate due to chore: change controller's deploy strategy to RollingUpdate due to leader election #3334
argoproj/argo-rollouts#3334

Checklist:

• I have bumped the chart version according to versioning
• I have updated the documentation according to documentation
• I have updated the chart changelog with all the changes that come with this pull request according to changelog.
• Any new values are backwards compatible and/or have sensible default.
• I have signed off all my commits as required by DCO.
• My build is green (troubleshooting builds).

[jmeridth]

Thank you for your contribution. Unfortunately, argoproj/argo-rollouts#3334 is merged but not released, so we can't apply it to the helm chart yet. There was a release after it was merged but it did not include the change.

Also #2412 is the more configurable approach.

[jmeridth]

Thank you for your contribution but I believe #2412 is the better approach and we're still waiting for upstream to release the change also.

[yu-croco]

Hi @ilyastoliar , thank you for your PR. I agree with @jmeridth .

[mkilchhofer]

Closing this in favor of #2412