Merge pull request #5753 from jefvantongerloo/jefvt-fix-5739-docker-p…

…ostgress-healthcheck Fix docker task-manager-db postgress healthcheck
opsmill · Feb 14, 2025 · 3a98a35 · 3a98a35
2 parents 09c6581 + 3945331
commit 3a98a35
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 3 deletions.
diff --git a/.vale/styles/spelling-exceptions.txt b/.vale/styles/spelling-exceptions.txt
@@ -51,6 +51,7 @@ graphene
 graphiql
 graphql
 greymatter
+healthcheck
 hfid
 hfids
 hostname
@@ -95,6 +96,7 @@ order_weight
 openconfig
 opentelemetry
 order_by
+Postgress
 Prefetch
 profile_name
 pseudocode

diff --git a/changelog/5739.fixed.md b/changelog/5739.fixed.md
@@ -0,0 +1 @@
+Fix Docker task-manager-db Postgress healthcheck test by adding database and user parameters.
diff --git a/development/docker-compose-deps-nats.yml b/development/docker-compose-deps-nats.yml
@@ -49,7 +49,9 @@ services:
     volumes:
       - workflow_db:/var/lib/postgresql/data
     healthcheck:
-      test: ["CMD-SHELL", "pg_isready"]
+      test:
+        - "CMD-SHELL"
+        - "pg_isready -q -d ${INFRAHUB_TASKMANAGER_DB_DATABASE:-prefect} -U ${INFRAHUB_TASKMANAGER_DB_USER:-postgres}"
       interval: 10s
       timeout: 5s
       retries: 5

diff --git a/development/docker-compose-deps.yml b/development/docker-compose-deps.yml
@@ -47,7 +47,9 @@ services:
     volumes:
       - workflow_db:/var/lib/postgresql/data
     healthcheck:
-      test: ["CMD-SHELL", "pg_isready"]
+      test:
+        - "CMD-SHELL"
+        - "pg_isready -q -d ${INFRAHUB_TASKMANAGER_DB_DATABASE:-prefect} -U ${INFRAHUB_TASKMANAGER_DB_USER:-postgres}"
       interval: 10s
       timeout: 5s
       retries: 5

diff --git a/docker-compose.yml b/docker-compose.yml
@@ -199,7 +199,9 @@ services:
     volumes:
       - workflow_db:/var/lib/postgresql/data
     healthcheck:
-      test: ["CMD-SHELL", "pg_isready"]
+      test:
+        - "CMD-SHELL"
+        - "pg_isready -q -d ${INFRAHUB_TASKMANAGER_DB_DATABASE:-prefect} -U ${INFRAHUB_TASKMANAGER_DB_USER:-postgres}"
       interval: 10s
       timeout: 5s
       retries: 5
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Fix Docker task-manager-db Postgress healthcheck test by adding database and user parameters.