Skip to content

Adds optional repo variables for job timeout values#1600

Merged
simongdavies merged 2 commits into
hyperlight-dev:mainfrom
simongdavies:make-job-timeouts-configurable
Jun 29, 2026
Merged

Adds optional repo variables for job timeout values#1600
simongdavies merged 2 commits into
hyperlight-dev:mainfrom
simongdavies:make-job-timeouts-configurable

Conversation

@simongdavies

Copy link
Copy Markdown
Member

The timeout values on GH jobs are hardcoded, this means that if we need to change any of these values even temporarily then we need to submit a PR, this PR allows the timeout value for each job to be optionally read from a repo variable. The hardcoded (fallback) values are not changed.

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 29, 2026 10:40
@simongdavies simongdavies added the kind/enhancement For PRs adding features, improving functionality, docs, tests, etc. label Jun 29, 2026
@simongdavies simongdavies requested a review from danbugs as a code owner June 29, 2026 10:40
@simongdavies simongdavies added the ready-for-review PR is ready for (re-)review label Jun 29, 2026

Copilot AI 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.

Pull request overview

This PR updates several GitHub Actions workflows to allow job-level timeout-minutes values to be overridden via repository variables, reducing the need for PRs when timeouts need temporary adjustment.

Changes:

  • Replaced hardcoded timeout-minutes values with expressions that read optional repo variables and fall back to the existing defaults.
  • Applied this pattern across nightly, coverage, cache-priming, and dependency-related workflows.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 10 comments.

Show a summary per file
File Description
.github/workflows/RustNightly.yml Makes the nightly musl job timeout configurable via a repo variable (with fallback).
.github/workflows/PrimeCaches.yml Makes the prime-caches job timeout configurable via a repo variable (with fallback).
.github/workflows/dep_update_guest_locks.yml Makes the update-guest-locks job timeout configurable via a repo variable (with fallback).
.github/workflows/dep_run_examples.yml Makes the run-examples job timeout configurable via a repo variable (with fallback).
.github/workflows/dep_code_checks.yml Makes linux/windows code-check jobs’ timeouts configurable via a repo variable (with fallback).
.github/workflows/dep_build_test.yml Makes the build-and-test job timeout configurable via a repo variable (with fallback).
.github/workflows/dep_build_guests.yml Makes the build-guests job timeout configurable via a repo variable (with fallback).
.github/workflows/dep_benchmarks.yml Makes the benchmarks job timeout configurable via a repo variable (with fallback).
.github/workflows/Coverage.yml Makes the coverage job timeout configurable via a repo variable (with fallback).

Comment thread .github/workflows/dep_code_checks.yml Outdated
Comment thread .github/workflows/dep_code_checks.yml Outdated
Comment thread .github/workflows/dep_build_guests.yml Outdated
Comment thread .github/workflows/dep_run_examples.yml Outdated
Comment thread .github/workflows/dep_update_guest_locks.yml Outdated
Comment thread .github/workflows/dep_build_test.yml Outdated
Comment thread .github/workflows/dep_benchmarks.yml Outdated
Comment thread .github/workflows/Coverage.yml Outdated
Comment thread .github/workflows/PrimeCaches.yml Outdated
Comment thread .github/workflows/RustNightly.yml Outdated
jprendes
jprendes previously approved these changes Jun 29, 2026

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

LGTM

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
@simongdavies simongdavies removed the ready-for-review PR is ready for (re-)review label Jun 29, 2026
@simongdavies simongdavies merged commit da8e366 into hyperlight-dev:main Jun 29, 2026
310 of 404 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement For PRs adding features, improving functionality, docs, tests, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants