application controller metrics application labels changed from map to list.

Updated templating to set the correct arguments.

Signed-off-by: Aaron Jones <aaron.jones@fundingcircle.com>
This commit is contained in:
Aaron Jones 2022-03-10 11:49:07 +00:00
parent ec6cd35fed
commit c3eb2efc56
4 changed files with 6 additions and 6 deletions

View file

@ -2,7 +2,7 @@ apiVersion: v2
appVersion: v2.2.5 appVersion: v2.2.5
description: A Helm chart for ArgoCD, a declarative, GitOps continuous delivery tool for Kubernetes. description: A Helm chart for ArgoCD, a declarative, GitOps continuous delivery tool for Kubernetes.
name: argo-cd name: argo-cd
version: 3.35.3 version: 3.35.4
home: https://github.com/argoproj/argo-helm home: https://github.com/argoproj/argo-helm
icon: https://argo-cd.readthedocs.io/en/stable/assets/logo.png icon: https://argo-cd.readthedocs.io/en/stable/assets/logo.png
keywords: keywords:
@ -21,4 +21,4 @@ dependencies:
condition: redis-ha.enabled condition: redis-ha.enabled
annotations: annotations:
artifacthub.io/changes: | artifacthub.io/changes: |
- "[Fixed]: Create service account for repo server by default" - "[Fixed]: ArgoCD application controller metric application label templating"

View file

@ -263,7 +263,7 @@ NAME: my-release
| controller.logFormat | string | `"text"` | Application controller log format. Either `text` or `json` | | controller.logFormat | string | `"text"` | Application controller log format. Either `text` or `json` |
| controller.logLevel | string | `"info"` | Application controller log level | | controller.logLevel | string | `"info"` | Application controller log level |
| controller.metrics.applicationLabels.enabled | bool | `false` | Enables additional labels in argocd_app_labels metric | | controller.metrics.applicationLabels.enabled | bool | `false` | Enables additional labels in argocd_app_labels metric |
| controller.metrics.applicationLabels.labels | object | `{}` | Additional labels | | controller.metrics.applicationLabels.labels | list | `[]` | Additional labels |
| controller.metrics.enabled | bool | `false` | Deploy metrics service | | controller.metrics.enabled | bool | `false` | Deploy metrics service |
| controller.metrics.rules.enabled | bool | `false` | Deploy a PrometheusRule for the application controller | | controller.metrics.rules.enabled | bool | `false` | Deploy a PrometheusRule for the application controller |
| controller.metrics.rules.spec | list | `[]` | PrometheusRule.Spec for the application controller | | controller.metrics.rules.spec | list | `[]` | PrometheusRule.Spec for the application controller |

View file

@ -57,9 +57,9 @@ spec:
- --loglevel - --loglevel
- {{ .Values.controller.logLevel }} - {{ .Values.controller.logLevel }}
{{- if .Values.controller.metrics.applicationLabels.enabled }} {{- if .Values.controller.metrics.applicationLabels.enabled }}
- --metrics-application-labels
{{- range .Values.controller.metrics.applicationLabels.labels }} {{- range .Values.controller.metrics.applicationLabels.labels }}
- {{- toYaml . | nindent 8 }} - --metrics-application-labels
- {{ . }}
{{- end }} {{- end }}
{{- end }} {{- end }}
{{- if or (and .Values.redis.enabled (not $redisHa.enabled)) (and $redisHa.enabled $redisHa.haproxy.enabled) }} {{- if or (and .Values.redis.enabled (not $redisHa.enabled)) (and $redisHa.enabled $redisHa.haproxy.enabled) }}

View file

@ -251,7 +251,7 @@ controller:
# -- Enables additional labels in argocd_app_labels metric # -- Enables additional labels in argocd_app_labels metric
enabled: false enabled: false
# -- Additional labels # -- Additional labels
labels: {} labels: []
service: service:
# -- Metrics service annotations # -- Metrics service annotations
annotations: {} annotations: {}