Skip to content

Try fix 1es runners#439

Merged
ludfjig merged 1 commit intomainfrom
1es_runner_jobid
Apr 1, 2026
Merged

Try fix 1es runners#439
ludfjig merged 1 commit intomainfrom
1es_runner_jobid

Conversation

@ludfjig
Copy link
Copy Markdown
Contributor

@ludfjig ludfjig commented Apr 1, 2026

Signed-off-by: Ludvig Liljenberg <4257730+ludfjig@users.noreply.github.com>
Copy link
Copy Markdown

Copilot AI left a comment

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 GitHub Actions runs-on label selection for the repo’s self-hosted 1ES runner pools by adding a JobId=... label to help disambiguate/route jobs (per the approach referenced in hyperlight-dev/hyperlight#1353).

Changes:

  • Add a JobId=... label to the runs-on selector in the Rust dependency workflow (dep_rust.yml) for matrix jobs.
  • Add a JobId=... label to the runs-on selector in the Benchmarks workflow for matrix jobs.
  • Add a JobId=... label to the cargo publish workflow runner selector.

Reviewed changes

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

File Description
.github/workflows/dep_rust.yml Appends a JobId=build-... label to the existing 1ES pool runner label selection for the build matrix.
.github/workflows/dep_cargo_publish.yml Adds a JobId=publish-hyperlight-wasm-... label to the 1ES pool runner label list for the publish job.
.github/workflows/Benchmarks.yml Appends a JobId=benchmark-... label to the existing 1ES pool runner label selection for the benchmark matrix.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ludfjig ludfjig marked this pull request as ready for review April 1, 2026 19:18
@ludfjig ludfjig merged commit 651bb34 into main Apr 1, 2026
29 of 30 checks passed
@ludfjig ludfjig deleted the 1es_runner_jobid branch April 1, 2026 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants