Skip to content

Use GHA runners instead of custom runners#3397

Draft
jakebailey wants to merge 2 commits intomainfrom
jabaile/ci-no-1es
Draft

Use GHA runners instead of custom runners#3397
jakebailey wants to merge 2 commits intomainfrom
jabaile/ci-no-1es

Conversation

@jakebailey
Copy link
Copy Markdown
Member

@jakebailey jakebailey commented Apr 13, 2026

Testing to see if it's even worth it to use custom runners.

Before:

image

These are the jobs on custom runners:

  • race: 11m25s
  • release-build: 10m59s

After:

image

So:

  • race: 20m35s
  • release-build: 17m17s

So, something like twice as slow.

But, windows-latest already takes nearly 19 minutes. So, why bother having these runners?

@jakebailey jakebailey changed the title Try running longer CI on GHA runners Use GHA runners instead of custom runners Apr 13, 2026
@jakebailey jakebailey marked this pull request as ready for review April 13, 2026 22:03
Copilot AI review requested due to automatic review settings April 13, 2026 22:03
Copy link
Copy Markdown
Contributor

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

Note

Copilot was unable to run its full agentic suite in this review.

This PR switches select CI jobs from custom self-hosted runners (1ES pool) to GitHub-hosted runners to evaluate performance and whether custom runners are worth maintaining.

Changes:

  • Move release-build job execution to ubuntu-latest GitHub-hosted runner.
  • Update the matrix entry for the “race mode” job to run on ubuntu-latest instead of the self-hosted runner pool.

Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml
Copy link
Copy Markdown
Member

@RyanCavanaugh RyanCavanaugh left a comment

Choose a reason for hiding this comment

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

Either way

@jakebailey jakebailey marked this pull request as draft April 16, 2026 21:16
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.

3 participants