Fix celery log folder creation (#1787)

core/celery/tasks/custom_list.py

@@ -125,5 +125,7 @@ def update_custom_lists(task: Task) -> None:
             select(CustomList.id).where(CustomList.auto_update_enabled == True)
         ).all()
 
+    task.log.info(f"Updating {len(custom_lists)} custom lists.")
+
     for custom_list in custom_lists:
         update_custom_list.delay(custom_list.id)

docker/Dockerfile

@@ -46,10 +46,6 @@ FROM common AS scripts
 # Set the local timezone and setup cron
 ENV TZ=US/Eastern
 RUN ln -snf /usr/share/zoneinfo/$TZ /etc/localtime && \
-    mkdir /var/log/celery && \
-    chown simplified:simplified /var/log/celery && \
-    mkdir /var/run/celery && \
-    chown simplified:simplified /var/run/celery && \
     touch /var/log/cron.log
 
 # Copy cron config into container

docker/runit-scripts/beat/run

@@ -1,6 +1,17 @@
 #!/bin/bash
 set -e
 
+# Set the working directory to the root of the project
 cd /var/www/circulation
 source env/bin/activate
+
+# Make sure the log directory exists and is writable
+mkdir -p /var/log/celery
+chown 1000:adm /var/log/celery
+
+# Make sure the run directory exists and is writable
+mkdir -p /var/run/celery
+chown 1000:1000 /var/run/celery
+
+# Start the celery beat process
 exec env/bin/celery -A "core.celery.worker.app" beat --uid 1000 --gid 1000 --logfile /var/log/celery/beat.log -s /var/run/celery/beat-schedule

docker/runit-scripts/worker-default/run

@@ -1,6 +1,13 @@
 #!/bin/bash
 set -e
 
+# Set the working directory to the root of the project
 cd /var/www/circulation
 source env/bin/activate
+
+# Make sure the log directory exists and is writable
+mkdir -p /var/log/celery
+chown 1000:adm /var/log/celery
+
+# Start the celery worker
 exec env/bin/celery -A "core.celery.worker.app" worker --uid 1000 --gid 1000 --autoscale 4,1 --hostname default@%h -Q high,default --logfile /var/log/celery/default.log

docker/runit-scripts/worker-high/run

@@ -1,6 +1,13 @@
 #!/bin/bash
 set -e
 
+# Set the working directory to the root of the project
 cd /var/www/circulation
 source env/bin/activate
+
+# Make sure the log directory exists and is writable
+mkdir -p /var/log/celery
+chown 1000:adm /var/log/celery
+
+# Start the celery worker
 exec env/bin/celery -A "core.celery.worker.app" worker --uid 1000 --gid 1000 --concurrency 1 --hostname high@%h -Q high --logfile /var/log/celery/high.log