feat(account-app): adding a PodDisruptionBudget

charts/accountapp/templates/_helpers.tpl

@@ -44,3 +44,11 @@ app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
 app.kubernetes.io/managed-by: {{ .Release.Service }}
 jenkins.io/maintainer: {{ (index .Chart.Maintainers 0).Name }}
 {{- end -}}
+
+{{/*
+Selector labels
+*/}}
+{{- define "accountapp.selectorLabels" -}}
+app.kubernetes.io/name: {{ include "accountapp.name" . }}
+app.kubernetes.io/instance: {{ .Release.Name }}
+{{- end }}

charts/accountapp/templates/pdb.yaml

@@ -0,0 +1,18 @@
+{{- if (gt (int .Values.replicaCount) 1) }}
+apiVersion: policy/v1
+kind: PodDisruptionBudget
+metadata:
+  name: {{ include "accountapp.fullname" . }}
+  labels:
+    {{- include "accountapp.labels" . | nindent 4 }}
+spec:
+  {{- with .Values.poddisruptionbudget.minAvailable }}
+  minAvailable: {{ . }}
+  {{- end }}
+  {{- with .Values.poddisruptionbudget.maxUnavailable }}
+  maxUnavailable: {{ . }}
+  {{- end }}
+  selector:
+    matchLabels:
+      {{- include "accountapp.selectorLabels" . | nindent 6 }}
+{{- end }}

charts/accountapp/tests/custom_values_test.yaml

@@ -3,6 +3,7 @@ templates:
   - deployment.yaml
   - ingress.yaml
   - secret.yaml # Direct dependency of deployment.yaml
+  - pdb.yaml
 tests:
   - it: should create a Secret with the correct value
     set:
@@ -51,3 +52,18 @@ tests:
       - equal:
           path: spec.template.spec.containers[0].resources.requests.cpu
           value: 500m
+  - it: should ensure the pdb has correct spec
+    template: pdb.yaml
+    set:
+      replicaCount: 2
+      poddisruptionbudget.minAvailable: 2
+      poddisruptionbudget.maxUnavailable: 1
+    asserts:
+      - isKind:
+          of: PodDisruptionBudget
+      - equal:
+          path: spec.minAvailable
+          value: 2
+      - equal:
+          path: spec.selector.matchLabels['app.kubernetes.io/name']
+          value: "accountapp"

charts/accountapp/tests/defaults_test.yaml

@@ -3,6 +3,7 @@ templates:
   - deployment.yaml
   - ingress.yaml
   - secret.yaml # Direct dependency of deployment.yaml
+  - pdb.yaml
 tests:
   - it: should create an ingress by default
     template: ingress.yaml
@@ -21,3 +22,8 @@ tests:
           value: IfNotPresent
       - notExists:
           path: spec.template.spec.containers[0].resources
+  - it: should not generate any pdb with default values
+    template: pdb.yaml
+    asserts:
+      - hasDocuments:
+          count: 0

charts/accountapp/values.yaml

@@ -66,3 +66,5 @@ election:
   close: 1970-01-02
   logDir: /var/log/accountapp/elections
   candidates: bob,alice
+poddisruptionbudget:
+  minAvailable: 1