Merge pull request #162 from devrandom0/sina/add_kubernetes_pdb

feat: add PodDisruptionBudget for high availability

Author: rosmo

deploy/chart/templates/poddisruptionbudget.yaml

@@ -0,0 +1,19 @@
+{{- if .Values.podDisruptionBudget.enabled }}
+---
+apiVersion: policy/v1
+kind: PodDisruptionBudget
+metadata:
+  name: {{ include "autoneg.fullname" . }}-pdb
+  namespace: {{ include "autoneg.namespace" . | quote }}
+  labels:
+    {{- include "autoneg.labels" . | nindent 4 }}
+spec:
+  selector:
+    matchLabels:
+      {{- include "autoneg.selectorLabels" . | nindent 6 }}
+  {{- if .Values.podDisruptionBudget.minAvailable }}
+  minAvailable: {{ .Values.podDisruptionBudget.minAvailable }}
+  {{- else if .Values.podDisruptionBudget.maxUnavailable }}
+  maxUnavailable: {{ .Values.podDisruptionBudget.maxUnavailable }}
+  {{- end }}
+{{- end }}

deploy/chart/values.yaml

@@ -21,6 +21,14 @@ autoscaling: {}
 
 replicas: 1
 
+# Pod Disruption Budget configuration
+podDisruptionBudget:
+  # Enable or disable PDB creation
+  enabled: true
+  # Specify either minAvailable or maxUnavailable, not both
+  minAvailable: 1
+  # maxUnavailable: 1
+
 serviceAccount:
   create: true
   name: autoneg-controller-manager

terraform/kubernetes/main.tf

@@ -382,3 +382,39 @@ resource "kubernetes_deployment" "deployment_autoneg_controller_manager" {
     kubernetes_cluster_role_binding.clusterrolebinding_autoneg_proxy_rolebinding,
   ]
 }
+
+resource "kubernetes_pod_disruption_budget" "pdb_autoneg_controller" {
+  count = var.pod_disruption_budget.enabled ? 1 : 0
+
+  metadata {
+    name      = "autoneg-controller-manager-pdb"
+    namespace = kubernetes_namespace.namespace_autoneg_system.metadata[0].name
+    labels = {
+      app           = "autoneg"
+      control-plane = "controller-manager"
+    }
+  }
+
+  spec {
+    dynamic "min_available" {
+      for_each = var.pod_disruption_budget.min_available != null ? [var.pod_disruption_budget.min_available] : []
+      content {
+        value = min_available.value
+      }
+    }
+
+    dynamic "max_unavailable" {
+      for_each = var.pod_disruption_budget.max_unavailable != null ? [var.pod_disruption_budget.max_unavailable] : []
+      content {
+        value = max_unavailable.value
+      }
+    }
+
+    selector {
+      match_labels = {
+        app           = "autoneg"
+        control-plane = "controller-manager"
+      }
+    }
+  }
+}

terraform/kubernetes/variables.tf

@@ -70,3 +70,17 @@ variable "priority_class_name" {
   type        = string
   default     = null
 }
+
+variable "pod_disruption_budget" {
+  description = "Pod Disruption Budget configuration"
+  type = object({
+    enabled       = bool
+    min_available = optional(number)
+    max_unavailable = optional(number)
+  })
+  default = {
+    enabled       = true
+    min_available = 1
+    max_unavailable = null
+  }
+}