fix: sleep 5 at teardown violates AGENTS.md engineering principles (#845)

This commit is contained in:
openhands 2026-03-16 07:06:57 +00:00
parent 10ff61e6b5
commit 8986154d8f

View file

@ -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