fix: Fix string/comma problem in worker template (#782)

charts/posthog/Chart.yaml

@@ -11,7 +11,7 @@ type: application
 
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
-version: 30.37.0
+version: 30.38.0
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application.

charts/posthog/templates/worker-deployment.yaml

@@ -127,7 +127,7 @@ spec:
         {{- include "snippet.email-env" $ | nindent 8 }}
         {{- if .queues }}
         - name: CELERY_WORKER_QUEUES
-          value: "{{ .queues | join ',' }}"
+          value: "{{ .queues | join "," }}"
         {{- end }}
         {{- if .concurrency }}
         - name: CELERY_WORKER_CONCURRENCY

charts/posthog/tests/worker-deployment.yaml

@@ -125,3 +125,30 @@ tests:
       - equal:
           path: spec.template.spec.imagePullSecrets
           value: [name: secret]
+  - it: set consumer config correctly
+    template: templates/worker-deployment.yaml # TODO: remove once secrets.yaml will be fixed/removed
+    set:
+      cloud: local
+      worker.consumers:
+        - name: gevent
+          pool: gevent
+          concurrency: 50
+          queues:
+            - analytics_queries
+            - other
+    asserts:
+      - contains:
+          path: spec.template.spec.containers[0].env
+          content:
+            name: CELERY_WORKER_QUEUES
+            value: analytics_queries,other
+      - contains:
+          path: spec.template.spec.containers[0].env
+          content:
+            name: CELERY_WORKER_CONCURRENCY
+            value: "50"
+      - contains:
+          path: spec.template.spec.containers[0].env
+          content:
+            name: CELERY_WORKER_POOL
+            value: gevent