Skip to content

Commit

Permalink
fix(argo-cd): make ServiceMonitor annotations in ArgoCD server condit…
Browse files Browse the repository at this point in the history
…ional (argoproj#2273)

* fix: add argocd servicemonitor annotations conditionally
Signed-off-by: Mats Willemsen <[email protected]>

* chore(argo-cd): add changelog

Signed-off-by: Mats Willemsen <[email protected]>

---------

Signed-off-by: Mats Willemsen <[email protected]>
Signed-off-by: Kripu Khadka <[email protected]>
  • Loading branch information
ma-ts authored and Kripu77 committed Nov 16, 2023
1 parent 1796378 commit c4fbf53
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions charts/argo-cd/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ appVersion: v2.8.4
kubeVersion: ">=1.23.0-0"
description: A Helm chart for Argo CD, a declarative, GitOps continuous delivery tool for Kubernetes.
name: argo-cd
version: 5.46.4
version: 5.46.5
home: https://github.com/argoproj/argo-helm
icon: https://argo-cd.readthedocs.io/en/stable/assets/logo.png
sources:
Expand All @@ -27,4 +27,4 @@ annotations:
url: https://argoproj.github.io/argo-helm/pgp_keys.asc
artifacthub.io/changes: |
- kind: fixed
description: Align redis-ha's affinity type to upstream due to warnings
description: fixed issue with argocd-server servicemonitor annotation attribute being added even when no annotations where defined
6 changes: 3 additions & 3 deletions charts/argo-cd/templates/argocd-server/servicemonitor.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,10 @@ metadata:
{{- with .Values.server.metrics.serviceMonitor.additionalLabels }}
{{- toYaml . | nindent 4 }}
{{- end }}
{{- with .Values.server.metrics.serviceMonitor.annotations }}
annotations:
{{- range $key, $value := .Values.server.metrics.serviceMonitor.annotations }}
{{ $key }}: {{ $value | quote }}
{{- end }}
{{- toYaml . | nindent 4 }}
{{- end }}
spec:
endpoints:
- port: {{ .Values.server.metrics.service.portName }}
Expand Down

0 comments on commit c4fbf53

Please sign in to comment.