Hotfix: Smoke tests didn't allow customizing the worker's command arg…

…uments, now it does (celery#8937)
stegayet · Apr 1, 2024 · 010af00 · 010af00
1 parent c3a988c
commit 010af00
Show file tree

Hide file tree

Showing 3 changed files with 29 additions and 2 deletions.
diff --git a/t/smoke/workers/dev.py b/t/smoke/workers/dev.py
@@ -53,6 +53,7 @@ def worker_queue(cls) -> str:
     },
     wrapper_class=SmokeWorkerContainer,
     timeout=defaults.DEFAULT_WORKER_CONTAINER_TIMEOUT,
+    command=fxtr("default_worker_command"),
 )
 
 

diff --git a/t/smoke/workers/docker/dev b/t/smoke/workers/docker/dev
@@ -4,7 +4,20 @@ FROM python:3.11-bookworm
 RUN adduser --disabled-password --gecos "" test_user
 
 # Install system dependencies
-RUN apt-get update && apt-get install -y build-essential
+RUN apt-get update && apt-get install -y build-essential \
+    git \
+    wget \
+    make \
+    curl \
+    apt-utils \
+    debconf \
+    lsb-release \
+    libmemcached-dev \
+    libffi-dev \
+    ca-certificates \
+    pypy3 \
+    pypy3-lib \
+    sudo
 
 # Set arguments
 ARG CELERY_LOG_LEVEL=INFO

diff --git a/t/smoke/workers/docker/pypi b/t/smoke/workers/docker/pypi
@@ -4,7 +4,20 @@ FROM python:3.10-bookworm
 RUN adduser --disabled-password --gecos "" test_user
 
 # Install system dependencies
-RUN apt-get update && apt-get install -y build-essential
+RUN apt-get update && apt-get install -y build-essential \
+    git \
+    wget \
+    make \
+    curl \
+    apt-utils \
+    debconf \
+    lsb-release \
+    libmemcached-dev \
+    libffi-dev \
+    ca-certificates \
+    pypy3 \
+    pypy3-lib \
+    sudo
 
 # Set arguments
 ARG CELERY_VERSION=""
-Original file line number
+Diff line change
@@ Expand Up / @@ -53,6 +53,7 @@ def worker_queue(cls) -> str: @@
         },
         wrapper_class=SmokeWorkerContainer,
         timeout=defaults.DEFAULT_WORKER_CONTAINER_TIMEOUT,
+        command=fxtr("default_worker_command"),
     )
@@ Expand Down @@