Skip to content

Commit

Permalink
Fix pair_aggregator idempotency check
Browse files Browse the repository at this point in the history
We need parentheses around the right hand side of the `||` so that
everything get skipped if `/tmp/done` exists.

Part of #1096
  • Loading branch information
tgeoghegan committed Jun 18, 2024
1 parent 152f352 commit c67ce13
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ services:
- -c
- |
test -f /tmp/done || \
/divviup --url=http://localhost:8080 --token="" \
(/divviup --url=http://localhost:8080 --token="" \
aggregator create \
--name=leader --api-url=http://janus_1_aggregator:8080/aggregator-api \
--bearer-token=0000 \
Expand All @@ -101,7 +101,7 @@ services:
aggregator create \
--name=helper --api-url=http://janus_2_aggregator:8080/aggregator-api \
--bearer-token=0000 && \
touch /tmp/done
touch /tmp/done)
network_mode: service:divviup_api
depends_on:
divviup_api:
Expand Down

0 comments on commit c67ce13

Please sign in to comment.