Skip to content

update PR validation and daily runs to reduce PR dependance on ARM64 runners#1603

Merged
simongdavies merged 3 commits into
hyperlight-dev:mainfrom
simongdavies:trim-pr-validation-on-aarch64
Jun 30, 2026
Merged

update PR validation and daily runs to reduce PR dependance on ARM64 runners#1603
simongdavies merged 3 commits into
hyperlight-dev:mainfrom
simongdavies:trim-pr-validation-on-aarch64

Conversation

@simongdavies

Copy link
Copy Markdown
Member

Helps to speed up PR validation whilst we have limited ARM runners, some validation/tests are moved to a daily job that runs at 5am UTC daily, failures will result in a release blocking issue

…runners

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 30, 2026 12:35
@simongdavies simongdavies added kind/enhancement For PRs adding features, improving functionality, docs, tests, etc. area/infrastructure Concerns infrastructure rather than core functionality labels Jun 30, 2026
@simongdavies simongdavies added the ready-for-review PR is ready for (re-)review label Jun 30, 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 adjusts GitHub Actions CI to reduce arm64 runner usage on pull requests by moving full aarch64 coverage (examples + fuzzing + expanded test set) to a daily scheduled workflow, while keeping PR validation faster and less dependent on limited arm64 capacity.

Changes:

  • Update PR validation to skip arm64 example runs and run arm64 fuzzing only on the daily schedule.
  • Add a full_aarch64 switch to the reusable build/test workflow to skip Miri + single-driver tests on aarch64 PR runs while still running them in the daily workflow.
  • Introduce a new scheduled DailyArm64.yml workflow that restores full aarch64 coverage and files a release-blocking issue on failure.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
.github/workflows/ValidatePullRequest.yml Stops running arm64 examples/fuzzing on PRs (moved to daily) to conserve arm64 runners.
.github/workflows/dep_build_test.yml Adds full_aarch64 to control whether Miri/single-driver tests run on aarch64.
.github/workflows/DailyArm64.yml New daily scheduled workflow to run full aarch64 coverage and open a release-blocking issue on failure.

Comment thread .github/workflows/DailyArm64.yml
Comment thread .github/workflows/DailyArm64.yml
Comment thread .github/workflows/dep_build_test.yml Outdated
Comment thread .github/workflows/dep_build_test.yml Outdated
Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>

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

Mostly LGTM. Just one question.

Comment thread .github/workflows/DailyArm64.yml
ludfjig
ludfjig previously approved these changes Jun 30, 2026
Comment thread .github/workflows/ValidatePullRequest.yml Outdated
Comment thread .github/workflows/DailyArm64.yml
@simongdavies

Copy link
Copy Markdown
Member Author

yeah I'm wondering why this looks different to report-ci-status in ValidatePullRquest.yml (the if statement that is)

All the daily jobs failure notify steps are defined like this e.g. https://github.com/hyperlight-dev/hyperlight/blob/main/.github/workflows/DailyBenchmarks.yml

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>

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

Ah I see didn't see that other workflow already use similar pattern

@simongdavies simongdavies merged commit 1f91c87 into hyperlight-dev:main Jun 30, 2026
44 checks passed
@simongdavies simongdavies removed the ready-for-review PR is ready for (re-)review label Jun 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/infrastructure Concerns infrastructure rather than core functionality kind/enhancement For PRs adding features, improving functionality, docs, tests, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants