chrisp/brackt
1
1
Fork 0
Code Issues 42 Pull requests Projects Releases Packages Wiki Activity Actions 2

Fix migration check in deploy script to use reliable exit code

Browse source 
`docker compose wait migrate | grep -qx "0"` was unreliable because
`docker compose wait` output format may not match the literal string "0".

Replace with `docker compose up --no-deps migrate` (foreground, no -d),
which exits with the container's actual exit code — no grep needed.

https://claude.ai/code/session_01WRe4qZuV2NQqDYurpDodF4
This commit is contained in:
Claude 2026-04-06 04:38:43 +00:00
parent 0f97ab258f
commit 23194a590c
No known key found for this signature in database
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
.github/workflows/deploy.yml vendored
View file

@ -149,8 +149,7 @@ jobs:
docker login $CONTAINER_REGISTRY -u $REGISTRY_USERNAME -p $REGISTRY_PASSWORD docker login $CONTAINER_REGISTRY -u $REGISTRY_USERNAME -p $REGISTRY_PASSWORD
cd brackt cd brackt
docker compose pull docker compose pull
docker compose up -d migrate if ! docker compose up --no-deps migrate; then
if ! docker compose wait migrate | grep -qx "0"; then
echo "Migration failed:" echo "Migration failed:"
docker compose logs migrate docker compose logs migrate
exit 1 exit 1