add health checks to rabbit and postgres to ensure they are healthy before CollectOSS starts.

This avoids some race conditions with the startup process that could create issues, especially on first initialization and especially in CI where we are still replaying migrations

Co-Authored-By: guptapratykshh <pratykshgupta9999@gmail.com>
Co-Authored-By: Sukuna0007Abhi <appsonly310@gmail.com>
Signed-off-by: Adrian Edwards <adredwar@redhat.com>

Author: 3 people

docker-compose.yml

@@ -14,6 +14,15 @@ services:
       - "${COLLECTOSS_DB_PORT:-5432}:5432"
     volumes:
       - augurpostgres:/var/lib/postgresql/data
+    healthcheck:
+      test:
+        [
+          "CMD-SHELL",
+          "pg_isready -U ${COLLECTOSS_DB_USER:-augur} -d ${COLLECTOSS_DB_NAME:-augur}",
+        ]
+      interval: 10s
+      timeout: 5s
+      retries: 5
 
   redis:
     image: "redis:alpine"
@@ -39,6 +48,12 @@ services:
         - RABBIT_MQ_DEFAULT_USER=${COLLECTOSS_RABBITMQ_USERNAME:-augur}
         - RABBIT_MQ_DEFAULT_PASSWORD=${COLLECTOSS_RABBITMQ_PASSWORD:-password123}
         - RABBIT_MQ_DEFAULT_VHOST=${COLLECTOSS_RABBITMQ_VHOST:-collectoss_vhost}
+    healthcheck:
+      test: ["CMD", "rabbitmq-diagnostics", "ping"]
+      interval: 10s
+      timeout: 5s
+      retries: 10
+      start_period: 10s
 
   core:
     image: collectoss:latest
@@ -74,10 +89,14 @@ services:
       - CACHE_LOCKDIR=/cache
       - CELERYBEAT_SCHEDULE_DB=/tmp/celerybeat-schedule.db
     depends_on:
-      - database
-      - redis
-      - keyman
-      - rabbitmq
+      database:
+        condition: service_healthy
+      redis:
+        condition: service_started
+      keyman:
+        condition: service_started
+      rabbitmq:
+        condition: service_healthy
     user: 2345:2345 # Run as an arbitrary non-root user
     post_start:
       # Make sure the user has access to the volumes