Enable pod disruption budget by default

thoughtbot · Oct 24, 2024 · bd066bf · bd066bf
1 parent 6b750a5
commit bd066bf
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/platform/modules/prometheus-instance/chart/templates/podDisruptionBudget.yaml b/platform/modules/prometheus-instance/chart/templates/podDisruptionBudget.yaml
@@ -5,8 +5,7 @@ metadata:
   name: {{ template "prometheus.fullname" . }}
   namespace: {{ $.Release.Namespace }}
   labels:
-    app: {{ template "kube-prometheus-stack.name" . }}-prometheus
-{{ include "kube-prometheus-stack.labels" . | indent 4 }}
+    {{- include "prometheus.labels" . | nindent 4 }}
 spec:
   {{- if .Values.prometheus.podDisruptionBudget.minAvailable }}
   minAvailable: {{ .Values.prometheus.podDisruptionBudget.minAvailable }}

diff --git a/platform/modules/prometheus-instance/chart/values.yaml b/platform/modules/prometheus-instance/chart/values.yaml
@@ -41,8 +41,8 @@ prometheus:
     version: v2.26.0
 
     ## Pod anti-affinity can prevent the scheduler from placing Prometheus replicas on the same node.
-    ## The default value "soft" means that the scheduler should *prefer* to not schedule two replica pods onto the same node but no guarantee is provided.
-    ## The value "hard" means that the scheduler is *required* to not schedule two replica pods onto the same node.
+    ## The value "soft" means that the scheduler should *prefer* to not schedule two replica pods onto the same node but no guarantee is provided.
+    ## The default value "hard" means that the scheduler is *required* to not schedule two replica pods onto the same node.
     ## The value "" will disable pod anti-affinity so that no anti-affinity rules will be configured.
     podAntiAffinity: "hard"
 
@@ -58,7 +58,7 @@ prometheus:
   ## https://github.com/kubernetes/kubernetes/issues/45398
   ##
   podDisruptionBudget:
-    enabled: false
+    enabled: true
     minAvailable: 1
     maxUnavailable: ""