guettli/sharedinbox
1
0
Fork 0
Code Issues 21 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

ci: add timeouts to all CI/CD jobs, Dagger tasks, and runner scripts #432

Merged
guettlibot merged 1 commits from refs/pull/432/head into main 2026-06-05 09:49:31 +00:00
Conversation 0 Commits 1 Files Changed 5
+1251 -6504
guettlibot commented 2026-06-04 20:43:49 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

Closes #415

Summary

  • Adds missing timeout-minutes to ci.yml (check job, 60 min) and windows-nightly.yml (90 min, ready for when the Windows runner is registered)
  • Wraps ssh-keyscan and ssh -f -N -L tunnel creation in setup_dagger_remote.sh with timeout 30; emits a ::warning:: annotation when either takes more than 10 s
  • Adds timeout --kill-after=10 <N> to all bare dagger call invocations in Taskfile.yml: 600 s for test/query tasks, 1800 s for build/deploy tasks, 60 s for ci-graph; stalwart and check-dagger (already protected) left untouched
  • Adds timeout --kill-after=10 2400 per attempt in run_firebase_test.sh; emits ::warning:: on exit 124 instead of silently retrying

Test plan

  • CI passes on this PR (the check job now has timeout-minutes: 60 and will self-enforce)
  • All dagger call lines in Taskfile.yml now have a timeout prefix (visible in the diff)
  • setup_dagger_remote.sh logic is unchanged — only the two network calls are wrapped
Closes #415 ## Summary - Adds missing `timeout-minutes` to `ci.yml` (`check` job, 60 min) and `windows-nightly.yml` (90 min, ready for when the Windows runner is registered) - Wraps `ssh-keyscan` and `ssh -f -N -L` tunnel creation in `setup_dagger_remote.sh` with `timeout 30`; emits a `::warning::` annotation when either takes more than 10 s - Adds `timeout --kill-after=10 <N>` to all bare `dagger call` invocations in `Taskfile.yml`: 600 s for test/query tasks, 1800 s for build/deploy tasks, 60 s for `ci-graph`; `stalwart` and `check-dagger` (already protected) left untouched - Adds `timeout --kill-after=10 2400` per attempt in `run_firebase_test.sh`; emits `::warning::` on exit 124 instead of silently retrying ## Test plan - CI passes on this PR (the `check` job now has `timeout-minutes: 60` and will self-enforce) - All `dagger call` lines in `Taskfile.yml` now have a `timeout` prefix (visible in the diff) - `setup_dagger_remote.sh` logic is unchanged — only the two network calls are wrapped
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
NeedSupervisor
Issue escalated to a human supervisor; agentloop will skip it until cleared.
 State/InProgress
State/Later
State/Planned
automerge
Eligible for automatic merge by CI
 ci-failure
Issue opened by agentloop to track a failing CI workflow; used for deduplication.
 do-not-merge
Plan PR — review only, do not merge.
 loop/code
Add to run the built-in "code" prompt; override at prompts/code.md.
 loop/code-ci-pending
Prompt "code" finished; waiting for the PR's CI to pass before advancing.
 loop/code-done
Prompt "code" finished successfully.
 loop/code-in-process
Agent for the "code" prompt is currently running on this issue.
 loop/merge
Managed by agentloop
 loop/merge-done
Managed by agentloop
 loop/merge-in-process
Managed by agentloop
 loop/plan
Add to run the built-in "plan" prompt; override at prompts/plan.md.
 loop/plan-done
Prompt "plan" finished successfully.
 loop/plan-in-process
Agent for the "plan" prompt is currently running on this issue.
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
guettlibot guettli
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: guettli/sharedinbox#432