Skip to content

Change pr-builder runner from ubuntu-latest to ubuntu-slim - #617

Open
bdice wants to merge 2 commits into
rapidsai:mainfrom
bdice:pr-builder-self-hosted
Open

Change pr-builder runner from ubuntu-latest to ubuntu-slim#617
bdice wants to merge 2 commits into
rapidsai:mainfrom
bdice:pr-builder-self-hosted

Conversation

@bdice

@bdice bdice commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

GitHub’s ubuntu-latest runners are heavily backlogged, and pr-builder cannot run, which blocks merges. Switching to use self-hosted CPU runners should help alleviate the problem.

@bdice
bdice requested a review from a team as a code owner August 20, 2026 03:20
@bdice
bdice requested review from jameslamb and removed request for a team August 20, 2026 03:20
@gforsyth

Copy link
Copy Markdown
Contributor

Do we still want to do this?

@jakirkham mentioned we've had our limit bumped for those runners last night at ~9pm eastern: https://nvidia.slack.com/archives/C01PF84NXCL/p1787186487740469?thread_ts=1787181186.430319&cid=C01PF84NXCL

@jameslamb

Copy link
Copy Markdown
Member

I just asked our GitHub contacts a clarifying question... pr-builder might be a good fit for ubuntu-slim runners: https://github.blog/changelog/2025-10-28-1-vcpu-linux-runner-now-available-in-github-actions-in-public-preview/

Those are physically smaller and we might face higher concurrency limits for them... if so I'd rather switch to that than put more burden on our self-hosted CPU runners.

@bdice

bdice commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

The reason I chose self-hosted infrastructure is because it isolates us from the recurring issues in GitHub infrastructure. I'll try ubuntu-slim, please approve and merge if you are fine with that.

Comment thread .github/workflows/pr-builder.yaml Outdated
@bdice bdice changed the title Change pr-builder runner from ubuntu-latest to linux-amd64-cpu4 Change pr-builder runner from ubuntu-latest to ubuntu-slim Aug 25, 2026
@bdice bdice added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Aug 25, 2026
@bdice bdice self-assigned this Aug 25, 2026
@jameslamb

jameslamb commented Aug 26, 2026

Copy link
Copy Markdown
Member

Now that that outage is resolved and our ubuntu-latest concurrency limit has been increased, I think this should just be closed.

Our GitHub contacts said that ubuntu-slim and ubuntu-latest compete for the same concurrency, so unfortunately switching to ubuntu-slim doesn't help us.

Our self-hosted runners are plentiful but they do cost money and aren't truly unlimited.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants