DevFW-CICD/argocd-helm
16
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Allow setting the metrics port name

Browse source 
Here, we scrape every pod that has a .*-metrics port here
But the helm chart is locked to `metrics` which our Prometheus doesn't see

This PR will make it configurable with a default value of `metrics`

Signed-off-by: Julien Duchesne <julien.duchesne@grafana.com>
This commit is contained in:
Julien Duchesne 2021-09-17 13:48:46 -04:00 committed by Julien Duchesne
parent 7477b33cba
commit 98d8c55e16
2 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
charts/argo-workflows/templates/controller/workflow-controller-deployment.yaml
View file

@ -75,7 +75,7 @@ spec:
resources: resources:
{{- toYaml .Values.controller.resources | nindent 12 }} {{- toYaml .Values.controller.resources | nindent 12 }}
ports: ports:
- name: metrics - name: {{ .Values.controller.metricsConfig.portName }}
containerPort: {{ .Values.controller.metricsConfig.port }} containerPort: {{ .Values.controller.metricsConfig.port }}
- containerPort: 6060 - containerPort: 6060
livenessProbe: {{ .Values.controller.livenessProbe | toYaml | nindent 12 }} livenessProbe: {{ .Values.controller.livenessProbe | toYaml | nindent 12 }}

1
charts/argo-workflows/values.yaml
View file

@ -57,6 +57,7 @@ controller:
enabled: false enabled: false
path: /metrics path: /metrics
port: 9090 port: 9090
portName: metrics
servicePort: 8080 servicePort: 8080
servicePortName: metrics servicePortName: metrics
# the controller container's securityContext # the controller container's securityContext