Skip to content

ci: add timeout-minutes to workflow jobs (sized from CI history) - #76

Merged
rehan-fr merged 1 commit into
masterfrom
chore/ci-job-timeouts
Aug 18, 2026
Merged

ci: add timeout-minutes to workflow jobs (sized from CI history)#76
rehan-fr merged 1 commit into
masterfrom
chore/ci-job-timeouts

Conversation

@rehan-fr

Copy link
Copy Markdown
Contributor

Sets timeout-minutes on GitHub Actions jobs that had none, so a hung job
can't hold a runner for GitHub's 6-hour default.

How the values were computed — for each job, the last 100 successful
runs (90-day lookback) were pulled from Datadog CI Visibility; the timeout is
ceil(max_duration) + 20 minutes of headroom, rounded up to the nearest
multiple of 5. Jobs that already declare a
timeout-minutes were left untouched, and jobs with no recent CI data were
skipped rather than given a guessed value.

Notes for review:

  • Only .github/workflows/*.y*ml files are touched; one inserted line per job.
  • Reusable-workflow caller jobs (uses:) are unchanged — GitHub doesn't
    allow timeout-minutes there; the limit lives on the called workflow's jobs.
  • If a job is known to legitimately run longer than its history suggests
    (e.g. rare full rebuilds), bump the value in this PR rather than dropping it.

Part of the platform CI-hardening effort (freshaengineering/team-devex#483).

🤖 Generated with Claude Code

Sized from Datadog CI Visibility: max of the last 100 successful runs
plus 20 minutes headroom, rounded up to the nearest multiple of 5. Jobs with an existing timeout-minutes or no
recent CI data are untouched.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@shpendnilafresha shpendnilafresha left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

Verified before approving: the diff is confined to .github/workflows/*.y*ml, adds exactly one job-level timeout-minutes line per job with no other edits, and the workflow files at head parse cleanly with no duplicate keys.

🤖 Generated with Claude Code

@rehan-fr
rehan-fr merged commit 7fdcfab into master Aug 18, 2026
9 checks passed
@rehan-fr
rehan-fr deleted the chore/ci-job-timeouts branch August 18, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants