From 96d7a8875b68ca49973e5b23e5351686f5d92643 Mon Sep 17 00:00:00 2001
From: Ghislain LE MEUR <ghislain.lemeur@gmail.com>
Date: Fri, 17 Oct 2025 11:45:29 +0200
Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9B(helm)=20add=20default=20commands?=
 =?UTF-8?q?=20for=20celery=20workers?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Without explicit commands in values.yaml,
celeryTranscribe and  celerySummarize pods
were using the Dockerfile's default CMD (uvicorn),
which started the REST API instead of Celery workers.

This fix adds default commands to values.yaml for both services,
ensuring they run as Celery workers processing their respective
queues (transcribe-queue and summarize-queue).
---
 src/helm/meet/values.yaml | 20 ++++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

diff --git a/src/helm/meet/values.yaml b/src/helm/meet/values.yaml
index f693d028..9a395d26 100644
--- a/src/helm/meet/values.yaml
+++ b/src/helm/meet/values.yaml
@@ -546,7 +546,15 @@ celeryTranscribe:
   dpAnnotations: {}
 
   ## @param celeryTranscribe.command Override the celeryTranscribe container command
-  command: []
+  command:
+    - "celery"
+    - "-A"
+    - "summary.core.celery_worker"
+    - "worker"
+    - "--pool=solo"
+    - "--loglevel=info"
+    - "-Q"
+    - "transcribe-queue"
 
   ## @param celeryTranscribe.args Override the celeryTranscribe container args
   args: []
@@ -647,7 +655,15 @@ celerySummarize:
   dpAnnotations: {}
 
   ## @param celerySummarize.command Override the celerySummarize container command
-  command: []
+  command:
+    - "celery"
+    - "-A"
+    - "summary.core.celery_worker"
+    - "worker"
+    - "--pool=solo"
+    - "--loglevel=info"
+    - "-Q"
+    - "summarize-queue"
 
   ## @param celerySummarize.args Override the celerySummarize container args
   args: []