Merge pull request #169 from Test-Automation-Engineer/feature/ON-pdb-…

…for-worker Add pdb for worker
maykinmedia · Jan 22, 2025 · be90103 · be90103
2 parents 8739489 + 181d0e2
commit be90103
Show file tree

Hide file tree

Showing 5 changed files with 32 additions and 2 deletions.
diff --git a/charts/opennotificaties/CHANGELOG.md b/charts/opennotificaties/CHANGELOG.md
@@ -1,5 +1,8 @@
 # Changelog
 
+## 1.8.0-beta.0 (2025-22-01)
+- [#169] Add pdb for worker
+
 ## 1.7.0-beta.0 (2025-10-01)
 
 - [#148] Replace the worker liveness probe with the `celery inspect active` command. This should detect when a worker is down and should not interrupt long running tasks.

diff --git a/charts/opennotificaties/Chart.yaml b/charts/opennotificaties/Chart.yaml
@@ -3,7 +3,7 @@ name: opennotificaties
 description: API voor het routeren van notificaties
 
 type: application
-version: 1.7.0-beta.0
+version: 1.8.0-beta.0
 appVersion: latest
 
 dependencies:

diff --git a/charts/opennotificaties/README.md b/charts/opennotificaties/README.md
@@ -219,7 +219,9 @@ API voor het routeren van notificaties
 | worker.livenessProbe.successThreshold | int | `1` |  |
 | worker.livenessProbe.timeoutSeconds | int | `15` |  |
 | worker.maxWorkerLivenessDelta | string | `""` |  |
+| worker.pdb.create | bool | `false` |  |
+| worker.pdb.maxUnavailable | string | `""` |  |
+| worker.pdb.minAvailable | int | `1` |  |
 | worker.podLabels | object | `{}` |  |
 | worker.replicaCount | int | `2` |  |
 | worker.resources | object | `{}` |  |
-
diff --git a/charts/opennotificaties/templates/pdb.yaml b/charts/opennotificaties/templates/pdb.yaml
@@ -16,4 +16,25 @@ spec:
   selector:
     matchLabels:
       {{- include "opennotificaties.selectorLabels" . | nindent 6 }}
+{{- end }}
+
+---
+{{- if .Values.worker.pdb.create }}
+apiVersion: policy/v1
+kind: PodDisruptionBudget
+metadata:
+  name: {{ include "opennotificaties.workerFullname" . }}
+  namespace: {{ .Release.Namespace | quote }}
+  labels:
+    {{- include "opennotificaties.workerLabels" . | nindent 4 }}
+spec:
+  {{- if .Values.worker.pdb.minAvailable }}
+  minAvailable: {{ .Values.worker.pdb.minAvailable }}
+  {{- end }}
+  {{- if .Values.worker.pdb.maxUnavailable }}
+  maxUnavailable: {{ .Values.worker.pdb.maxUnavailable }}
+  {{- end }}
+  selector:
+    matchLabels:
+      {{- include "opennotificaties.workerSelectorLabels" . | nindent 6 }}
 {{- end }}
diff --git a/charts/opennotificaties/values.yaml b/charts/opennotificaties/values.yaml
@@ -353,6 +353,10 @@ worker:
     maxReplicas: 100
     targetCPUUtilizationPercentage: 80
     targetMemoryUtilizationPercentage: 80
+  pdb:
+    create: false
+    minAvailable: 1
+    maxUnavailable: ""
 
 flower:
   enabled: false