From bd066bf9711af3481c3e50a153b53572799e454c Mon Sep 17 00:00:00 2001
From: Olamide <Olamide@thoughtbot.com>
Date: Thu, 24 Oct 2024 15:19:02 +0100
Subject: [PATCH] Enable pod disruption budget by default

---
 .../chart/templates/podDisruptionBudget.yaml                | 3 +--
 platform/modules/prometheus-instance/chart/values.yaml      | 6 +++---
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/platform/modules/prometheus-instance/chart/templates/podDisruptionBudget.yaml b/platform/modules/prometheus-instance/chart/templates/podDisruptionBudget.yaml
index fd89009..b71809d 100644
--- 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
index 27d618c..f940f3e 100644
--- 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: ""