diff --git a/charts/posthog/Chart.yaml b/charts/posthog/Chart.yaml index 69e2b34c..105fbc85 100644 --- a/charts/posthog/Chart.yaml +++ b/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. diff --git a/charts/posthog/templates/worker-deployment.yaml b/charts/posthog/templates/worker-deployment.yaml index 8d7b2cec..b4a0d4cd 100644 --- a/charts/posthog/templates/worker-deployment.yaml +++ b/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 diff --git a/charts/posthog/tests/worker-deployment.yaml b/charts/posthog/tests/worker-deployment.yaml index f4b2b641..cd8f6600 100644 --- a/charts/posthog/tests/worker-deployment.yaml +++ b/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