Merge pull request 'fix: sleep 5 at teardown violates AGENTS.md engineering principles (#845)' (#861) from fix/issue-845 into master
This commit is contained in:
commit
39793ec0df
1 changed files with 9 additions and 2 deletions
|
|
@ -261,9 +261,16 @@ PYEOF
|
|||
'{completed: $arr, last_updated: now | todate}' > "$PROGRESS_FILE"
|
||||
log "DONE $seed_name"
|
||||
|
||||
# 6. Teardown
|
||||
# 6. Teardown — poll until all containers have exited (no fixed sleep)
|
||||
cd "$REPO_ROOT" && docker compose down -v 2>/dev/null || true
|
||||
sleep 5
|
||||
_deadline=$(( $(date +%s) + 30 ))
|
||||
while [[ -n "$(docker compose ps --quiet 2>/dev/null)" ]]; do
|
||||
if [[ $(date +%s) -ge $_deadline ]]; then
|
||||
log "WARNING: containers still present after 30s — proceeding anyway"
|
||||
break
|
||||
fi
|
||||
sleep 1
|
||||
done
|
||||
done
|
||||
|
||||
# Restore original
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue