Skip to content

Commit 2c2dfc6

Browse files
sntiwari1claude
andcommitted
Fix ClickHouse health check: add retries and fix ambiguous YAML
The clickhouse health check had no interval/timeout/retries/start_period, so it used Docker defaults (retries=3, interval=30s). If ClickHouse takes more than 90s to be ready on CI (3 × 30s), it becomes permanently unhealthy, blocking metrics from starting since we added clickhouse:service_healthy as a metrics dependency. Also fix the multiline YAML for the test command (was split across two lines as a plain scalar) — replace with the unambiguous list form. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent dc9a9f2 commit 2c2dfc6

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

docker-compose-v1.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -373,9 +373,11 @@ services:
373373
ports:
374374
- '9002:9000'
375375
healthcheck:
376-
test:
377-
wget --no-verbose --tries=1 --spider http://localhost:8123/ping || exit
378-
1
376+
test: ['CMD-SHELL', 'wget -nv -t1 --spider http://localhost:8123/ping || exit 1']
377+
interval: 30s
378+
timeout: 10s
379+
retries: 10
380+
start_period: 30s
379381
id-gen-service:
380382
image: ghcr.io/sunbird-rc/id-gen-service:${RELEASE_VERSION}
381383
ports:

0 commit comments

Comments
 (0)