From 86c8d2fac37f216241125b570a42350f503ad6d2 Mon Sep 17 00:00:00 2001
From: chgl <chgl@users.noreply.github.com>
Date: Sun, 18 Apr 2021 22:18:07 +0200
Subject: [PATCH] enabled metrics endpoint and liveness probe by default

Signed-off-by: chgl <chgl@users.noreply.github.com>
---
 .../controller/workflow-controller-deployment.yaml   | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)

diff --git a/charts/argo-workflows/templates/controller/workflow-controller-deployment.yaml b/charts/argo-workflows/templates/controller/workflow-controller-deployment.yaml
index 61e878ed..66b6afbb 100644
--- a/charts/argo-workflows/templates/controller/workflow-controller-deployment.yaml
+++ b/charts/argo-workflows/templates/controller/workflow-controller-deployment.yaml
@@ -76,23 +76,15 @@ spec:
           {{- end }}
           resources:
             {{- toYaml .Values.controller.resources | nindent 12 }}
-          {{- if .Values.controller.metricsConfig.enabled }}
           ports:
             - name: metrics
-              containerPort: {{- .Values.controller.metricsConfig.port }}
+              containerPort: {{ .Values.controller.metricsConfig.port }}
           livenessProbe:
             httpGet:
               port: metrics
-              path: {{- .Values.controller.metricsConfig.path }}
+              path: {{ .Values.controller.metricsConfig.path }}
             initialDelaySeconds: 30
             periodSeconds: 30
-          readinessProbe:
-            httpGet:
-              port: metrics
-              path: {{- .Values.controller.metricsConfig.path }}
-            initialDelaySeconds: 30
-            periodSeconds: 30
-          {{- end }}
       {{- with .Values.images.pullSecrets }}
       imagePullSecrets:
         {{- toYaml . | nindent 8 }}