Skip to content

Commit

Permalink
fix(argo-rollouts): use integet instead of float for controller replicas
Browse files Browse the repository at this point in the history
Fixes: #2308

Cast to int since default from values.yaml is int64 not int

Signed-off-by: jmeridth <jmeridth@gmail.com>
  • Loading branch information
jmeridth committed Oct 24, 2023
1 parent 14e2e32 commit f29ffa5
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions charts/argo-rollouts/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
appVersion: v1.6.0
description: A Helm chart for Argo Rollouts
name: argo-rollouts
version: 2.32.0
version: 2.32.1
home: https://github.com/argoproj/argo-helm
icon: https://argoproj.github.io/argo-rollouts/assets/logo.png
keywords:
Expand All @@ -18,5 +18,5 @@ annotations:
fingerprint: 2B8F22F57260EFA67BE1C5824B11F800CD9D2252
url: https://argoproj.github.io/argo-helm/pgp_keys.asc
artifacthub.io/changes: |
- kind: changed
description: Upgrade Argo Rollouts to v1.6.0
- kind: fixed
description: Use integer instead of float for controller replicas
2 changes: 1 addition & 1 deletion charts/argo-rollouts/templates/controller/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ spec:
{{- if not .Values.clusterInstall }}
- --namespaced
{{- end }}
{{- if gt .Values.controller.replicas 1.0 }}
{{- if gt (int .Values.controller.replicas) 1 }}
- --leader-elect
{{- end }}
{{- with .Values.controller.extraArgs }}
Expand Down

0 comments on commit f29ffa5

Please sign in to comment.