Fix worker HPA to target new worker groups

Also fix matchLabel clash between different worker deployments
PostHog · Jan 17, 2024 · e9ba29a · e9ba29a
1 parent 1a65077
commit e9ba29a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/charts/posthog/templates/worker-deployment.yaml b/charts/posthog/templates/worker-deployment.yaml
@@ -11,7 +11,7 @@ spec:
     matchLabels:
         app: {{ template "posthog.fullname" $ }}
         release: "{{ $.Release.Name }}"
-        role: worker
+        role: worker-{{ .name }}
   {{- if not $.Values.worker.hpa.enabled }}
   replicas: {{ $.Values.worker.replicacount }}
   {{- end }}
@@ -32,7 +32,7 @@ spec:
       labels:
         app: {{ template "posthog.fullname" $ }}
         release: "{{ $.Release.Name }}"
-        role: worker
+        role: worker-{{ .name }}
         {{- if (eq (default $.Values.image.tag "none") "latest") }}
         date: "{{ now | unixEpoch }}"
         {{- end }}

diff --git a/charts/posthog/templates/worker-hpa.yaml b/charts/posthog/templates/worker-hpa.yaml
@@ -1,14 +1,15 @@
 {{- if and .Values.worker.enabled .Values.worker.hpa.enabled -}}
+{{- range .Values.worker.consumers }}
 apiVersion: autoscaling/v2
 kind: HorizontalPodAutoscaler
 metadata:
-  name: {{ template "posthog.fullname" . }}-worker
+  name: {{ template "posthog.fullname" . }}-{{ .name }}-worker
   labels: {{- include "_snippet-metadata-labels-common" . | nindent 4 }}
 spec:
   scaleTargetRef:
     kind: Deployment
     apiVersion: apps/v1
-    name: {{ template "posthog.fullname" . }}-worker
+    name: {{ template "posthog.fullname" . }}-{{ .name }}-worker
   minReplicas: {{ .Values.worker.hpa.minpods }}
   maxReplicas: {{ .Values.worker.hpa.maxpods }}
   metrics:
@@ -23,3 +24,4 @@ spec:
   behavior: 
     {{ toYaml .Values.worker.hpa.behavior | nindent 4 }}
 {{- end }}
+{{- end }}