Skip to content

Commit

Permalink
compose.yaml rough edges
Browse files Browse the repository at this point in the history
Addresses some rough edges in compose.yaml:

- Adds a healthcheck on the postgres service to make startup of
  dependent schema migrator services more reliable.
- `pair_aggregator` would re-run every time `docker compose up` is run,
  and fail after the first time. It now drops a file `/tmp/done` to
  indicate it has already run to completion.

Part of #1096
  • Loading branch information
tgeoghegan committed Jun 17, 2024
1 parent 3064de8 commit c06447c
Showing 1 changed file with 16 additions and 3 deletions.
19 changes: 16 additions & 3 deletions compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ x-janus-migrate: &janus_migrate
- "60"
depends_on:
postgres:
condition: service_started
condition: service_healthy

x-janus-environment: &janus_environment
RUST_LOG: info
Expand Down Expand Up @@ -54,6 +54,17 @@ services:
image: docker.io/library/postgres:16
ports:
- "5432:5432"
healthcheck:
test:
- "CMD"
- "/bin/sh"
- "-c"
- |
psql postgres://postgres@127.0.0.1/divviup -c "" && \
psql postgres://postgres@127.0.0.1/janus_1 -c "" && \
psql postgres://postgres@127.0.0.1/janus_2 -c ""
interval: 1s
retries: 10
restart: always
environment:
POSTGRES_PASSWORD: postgres
Expand All @@ -70,14 +81,15 @@ services:
DATABASE_URL: postgres://postgres:postgres@postgres:5432/divviup
depends_on:
postgres:
condition: service_started
condition: service_healthy

pair_aggregator:
<<: *divviup_image
entrypoint:
- /bin/sh
- -c
- |
test -f /tmp/done || \
/divviup --url=http://localhost:8080 --token="" \
aggregator create \
--name=leader --api-url=http://janus_1_aggregator:8080/aggregator-api \
Expand All @@ -88,7 +100,8 @@ services:
/divviup --url=http://localhost:8080 --token="" \
aggregator create \
--name=helper --api-url=http://janus_2_aggregator:8080/aggregator-api \
--bearer-token=0000
--bearer-token=0000 && \
touch /tmp/done
network_mode: service:divviup_api
depends_on:
divviup_api:
Expand Down

0 comments on commit c06447c

Please sign in to comment.