remove default livenessProbe for compactor and storage-gateway (#502)

* remove default livenessProbe for compactor and storage-gateway

Signed-off-by: Niclas Schad <[email protected]>

* pr review: use correct brackets

Signed-off-by: Niclas Schad <[email protected]>

---------

Signed-off-by: Niclas Schad <[email protected]>

Author: nschad

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## master / unreleased
 
+* [CHANGE] Removed the default `livenessProbe` for store-gateway and compactor. You can still use a `livenessProbe` but we advise against it #502
 * [DEPENDENCY] Update quay.io/cortexproject/cortex Docker tag to v1.17.1 #501
 
 ## 2.3.0 / 2024-04-12

README.md

@@ -179,9 +179,7 @@ Kubernetes: `^1.19.0-0`
 | compactor.​extraVolumeMounts | list | `[]` |  |
 | compactor.​extraVolumes | list | `[]` |  |
 | compactor.​initContainers | list | `[]` |  |
-| compactor.​livenessProbe.​httpGet.​path | string | `"/ready"` |  |
-| compactor.​livenessProbe.​httpGet.​port | string | `"http-metrics"` |  |
-| compactor.​livenessProbe.​httpGet.​scheme | string | `"HTTP"` |  |
+| compactor.​livenessProbe | object | `{}` |  |
 | compactor.​nodeSelector | object | `{}` |  |
 | compactor.​persistentVolume.​accessModes | list | `["ReadWriteOnce"]` | compactor data Persistent Volume access modes Must match those of existing PV or dynamic provisioner Ref: http://kubernetes.io/docs/user-guide/persistent-volumes/ |
 | compactor.​persistentVolume.​annotations | object | `{}` | compactor data Persistent Volume Claim annotations |
@@ -811,9 +809,7 @@ Kubernetes: `^1.19.0-0`
 | store_gateway.​extraVolumeMounts | list | `[]` |  |
 | store_gateway.​extraVolumes | list | `[]` |  |
 | store_gateway.​initContainers | list | `[]` |  |
-| store_gateway.​livenessProbe.​httpGet.​path | string | `"/ready"` |  |
-| store_gateway.​livenessProbe.​httpGet.​port | string | `"http-metrics"` |  |
-| store_gateway.​livenessProbe.​httpGet.​scheme | string | `"HTTP"` |  |
+| store_gateway.​livenessProbe | object | `{}` |  |
 | store_gateway.​nodeSelector | object | `{}` |  |
 | store_gateway.​persistentVolume.​accessModes | list | `["ReadWriteOnce"]` | Store-gateway data Persistent Volume access modes Must match those of existing PV or dynamic provisioner Ref: http://kubernetes.io/docs/user-guide/persistent-volumes/ |
 | store_gateway.​persistentVolume.​annotations | object | `{}` | Store-gateway data Persistent Volume Claim annotations |

templates/compactor/compactor-statefulset.yaml

@@ -127,8 +127,10 @@ spec:
               protocol: TCP
           startupProbe:
             {{- toYaml .Values.compactor.startupProbe | nindent 12 }}
+          {{- if .Values.compactor.livenessProbe }}
           livenessProbe:
             {{- toYaml .Values.compactor.livenessProbe | nindent 12 }}
+          {{- end }}
           readinessProbe:
             {{- toYaml .Values.compactor.readinessProbe | nindent 12 }}
           resources:

templates/store-gateway/store-gateway-statefulset.yaml

@@ -133,8 +133,10 @@ spec:
               protocol: TCP
           startupProbe:
             {{- toYaml .Values.store_gateway.startupProbe | nindent 12 }}
+          {{- if .Values.store_gateway.livenessProbe }}
           livenessProbe:
             {{- toYaml .Values.store_gateway.livenessProbe | nindent 12 }}
+          {{- end }}
           readinessProbe:
             {{- toYaml .Values.store_gateway.readinessProbe | nindent 12 }}
           resources:

values.yaml

@@ -1410,11 +1410,7 @@ store_gateway:
       path: /ready
       port: http-metrics
       scheme: HTTP
-  livenessProbe:
-    httpGet:
-      path: /ready
-      port: http-metrics
-      scheme: HTTP
+  livenessProbe: {}
   readinessProbe:
     httpGet:
       path: /ready
@@ -1530,11 +1526,7 @@ compactor:
       path: /ready
       port: http-metrics
       scheme: HTTP
-  livenessProbe:
-    httpGet:
-      path: /ready
-      port: http-metrics
-      scheme: HTTP
+  livenessProbe: {}
   readinessProbe:
     httpGet:
       path: /ready