Skip to content

Add job timeouts and PR concurrency to workflows - #18

Merged
woksin merged 2 commits into
mainfrom
chore/ci-timeouts-concurrency
Aug 25, 2026
Merged

Add job timeouts and PR concurrency to workflows#18
woksin merged 2 commits into
mainfrom
chore/ci-timeouts-concurrency

Conversation

@woksin

@woksin woksin commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Part of the org-wide Actions throttling remediation. Mechanical, rule-based sweep:

  • timeout-minutes on every job that had none (previously 6 h default): quick checks 15, builds 30, publish/release 60, integration/benchmarks 120.
  • concurrency + cancel-in-progress only on pull-request-triggered verification workflows — never on publish/release/deploy, pull_request_target, workflow_run or reusable (workflow_call) workflows.

Every edited file re-parsed as YAML before commit. Files changed:

  • .github/workflows/auto-approve-publish-deployments.yml
  • .github/workflows/cleanup-pr-artifacts.yml
  • .github/workflows/dotnet-build.yml
  • .github/workflows/publish.yml
  • .github/workflows/verify-semver-label.yml

woksin added 2 commits August 25, 2026 22:20
Jobs had no timeout-minutes, so a hung job could hold one of the
organization's 20 shared hosted-runner slots for the 6 hour default - a
contributor to the 2026-08-25 runner starvation. Verification workflows
triggered by pull requests also lacked a concurrency group, so rapid
successive pushes stacked duplicate runs instead of cancelling stale ones.

Rules applied: quick checks 15 min, builds 30 min, publish/release 60 min,
integration/benchmarks 120 min; cancel-in-progress only on pull-request
verification workflows, never on publish, release or deploy workflows.
@woksin woksin added the patch label Aug 25, 2026
@woksin
woksin merged commit 4cc2917 into main Aug 25, 2026
1 of 2 checks passed
@woksin
woksin deleted the chore/ci-timeouts-concurrency branch August 25, 2026 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant