Skip to content

ci: add concurrency cancellation + job timeouts to model-zoo CI#98

Merged
shujaatTracebloc merged 1 commit into
developfrom
ci/add-concurrency-timeouts
Jun 17, 2026
Merged

ci: add concurrency cancellation + job timeouts to model-zoo CI#98
shujaatTracebloc merged 1 commit into
developfrom
ci/add-concurrency-timeouts

Conversation

@LukasWodka

Copy link
Copy Markdown
Contributor

Summary

Mechanical CI-hygiene change to .github/workflows/ci.yml. No change to what any job does.

  • Adds a top-level concurrency: group keyed on github.ref, with cancel-in-progress scoped to pull_request events only — so a PR re-push cancels its own superseded run instead of stacking the heavy framework test matrices. Pushes to branches are not cancelled.
  • Adds timeout-minutes as the first key of every job:
    • ruff → 10
    • test-pytorch / test-tensorflow / test-sklearn / test-survival → 30 (these pip-install torch/tensorflow/etc., so they're heavier)

This caps hung installs at the configured limit instead of letting them run to the 6h GitHub default.

Type

CI hygiene

Test plan

  • python3 -c "import yaml; yaml.safe_load(open('.github/workflows/ci.yml'))" → YAML OK
  • Diff contains only the concurrency: block + per-job timeout-minutes additions; no job runs-on/steps lines changed.

🤖 Generated with Claude Code

Adds a per-ref concurrency group (cancels superseded PR runs only) and
timeout-minutes to every job, so a PR re-push no longer stacks the heavy
framework test matrices and hung installs can't run to the 6h default.
No change to job behavior.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@LukasWodka

Copy link
Copy Markdown
Contributor Author

👋 Heads-up — Code review queue is at 35 / 30

Above the WIP limit. The team convention is to review existing PRs before opening new work.

Open PRs currently in Code review (oldest first):

Pull from review before opening new work. (This is a nudge from the kanban WIP check, not a block.)

@LukasWodka LukasWodka added the devops DevOps / CI-CD / build & deploy label Jun 17, 2026
@shujaatTracebloc shujaatTracebloc merged commit 0538048 into develop Jun 17, 2026
8 checks passed
@shujaatTracebloc shujaatTracebloc deleted the ci/add-concurrency-timeouts branch June 17, 2026 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops DevOps / CI-CD / build & deploy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants