[stable/influxdb] probes (#20800)

* Use include function instead of template. Signed-off-by: Naseem <naseem@transit.app> * Hardcode probe path. There is no value in making the path configurable. By hardcoding it to /ping we remove unecessary templating and config complexity. Signed-off-by: Naseem <naseem@transit.app>

[stable/influxdb] probes (#20800)
524fe3f5 · Naseem · GitHub · 0c23b2f3 · 524fe3f5 · 524fe3f5
Commit 524fe3f5 authored Feb 17, 2020 by Naseem Committed by GitHub Feb 17, 2020
4 changed files
--- a/stable/influxdb/Chart.yaml
+++ b/stable/influxdb/Chart.yaml
 apiVersion: v1
 name: influxdb
-version: 4.2.1
+version: 4.2.2
 appVersion: 1.7.6
 description: Scalable datastore for metrics, events, and real-time analytics.
 keywords:

--- a/stable/influxdb/templates/meta-statefulset.yaml
+++ b/stable/influxdb/templates/meta-statefulset.yaml
@@ -52,20 +52,20 @@ spec:
        {{- end }}
        livenessProbe:
          httpGet:
-            path: {{ .Values.livenessProbe.probePath }}
+            path: /ping
            port: meta
          initialDelaySeconds: {{ .Values.livenessProbe.initialDelaySeconds | default 30 }}
          timeoutSeconds: {{ .Values.livenessProbe.timeoutSeconds | default 5 }}
        readinessProbe:
          httpGet:
-            path: {{ .Values.readinessProbe.probePath }}
+            path: /ping
            port: meta
          initialDelaySeconds: {{ .Values.readinessProbe.initialDelaySeconds | default 5 }}
          timeoutSeconds: {{ .Values.readinessProbe.timeoutSeconds | default 1 }}
        {{- if .Values.startupProbe.enabled }}
        startupProbe:
          httpGet:
-            path: {{ .Values.startupProbe.probePath }}
+            path: /ping
            port: meta
          failureThreshold: {{ .Values.startupProbe.failureThreshold | default 6 }}
          periodSeconds: {{ .Values.startupProbe.periodSeconds | default 5 }}

--- a/stable/influxdb/templates/statefulset.yaml
+++ b/stable/influxdb/templates/statefulset.yaml
@@ -86,20 +86,20 @@ spec:
        {{- end }}
        livenessProbe:
          httpGet:
-            path: {{ .Values.livenessProbe.probePath }}
+            path: /ping
            port: api
          initialDelaySeconds: {{ .Values.livenessProbe.initialDelaySeconds | default 30 }}
          timeoutSeconds: {{ .Values.livenessProbe.timeoutSeconds | default 5 }}
        readinessProbe:
          httpGet:
-            path: {{ .Values.readinessProbe.probePath }}
+            path: /ping
            port: api
          initialDelaySeconds: {{ .Values.readinessProbe.initialDelaySeconds | default 5 }}
          timeoutSeconds: {{ .Values.readinessProbe.timeoutSeconds | default 1 }}
        {{- if .Values.startupProbe.enabled }}
        startupProbe:
          httpGet:
-            path: {{ .Values.startupProbe.probePath }}
+            path: /ping
            port: api
          failureThreshold: {{ .Values.startupProbe.failureThreshold | default 6 }}
          periodSeconds: {{ .Values.startupProbe.periodSeconds | default 5 }}

--- a/stable/influxdb/values.yaml
+++ b/stable/influxdb/values.yaml
@@ -14,18 +14,20 @@ serviceAccount:
  name:
  annotations: {}

-## Customize livenessProbe and readinessProbe
+## Customize liveness, readiness and startup probes
 ## ref: https://docs.influxdata.com/influxdb/v1.7/tools/api/#ping-http-endpoint
+## ref: https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-startup-probes/
 ##
-livenessProbe:
-  probePath: "/ping"
+livenessProbe: {}
+  # initialDelaySeconds: 30
+  # timeoutSeconds: 5

-readinessProbe:
-  probePath: "/ping"
+readinessProbe: {}
+  # initialDelaySeconds: 5
+  # timeoutSeconds: 1

 startupProbe:
  enabled: false
-  probePath: "/ping"
  # failureThreshold: 6
  # periodSeconds: 5