Skip to content

Use larger memory CPU runners for build jobs#9737

Open
jrhemstad wants to merge 2 commits into
NVIDIA:mainfrom
jrhemstad:codex/use-memory-cpu-runners
Open

Use larger memory CPU runners for build jobs#9737
jrhemstad wants to merge 2 commits into
NVIDIA:mainfrom
jrhemstad:codex/use-memory-cpu-runners

Conversation

@jrhemstad

@jrhemstad jrhemstad commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

Description

Updates non-GPU CI jobs to use the larger memory CPU runner labels (cpu16m) instead of the standard cpu16 labels to avoid OOMs during builds.

@copy-pr-bot

copy-pr-bot Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@cccl-authenticator-app cccl-authenticator-app Bot moved this from Todo to In Progress in CCCL Jul 7, 2026
@jrhemstad jrhemstad changed the title Use memory CPU runners for generated CI jobs Use larger memory CPU runners for build jobs Jul 7, 2026
@jrhemstad

Copy link
Copy Markdown
Collaborator Author

/ok to test 7e6f374

@jrhemstad jrhemstad marked this pull request as ready for review July 7, 2026 19:24
@jrhemstad jrhemstad requested a review from a team as a code owner July 7, 2026 19:24
@jrhemstad jrhemstad requested a review from wmaxey July 7, 2026 19:24
@cccl-authenticator-app cccl-authenticator-app Bot moved this from In Progress to In Review in CCCL Jul 7, 2026
@jrhemstad

Copy link
Copy Markdown
Collaborator Author

/ok to test 424668e

@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: b13a1672-e5f5-45f8-97dd-9ee8a734ffa7

📥 Commits

Reviewing files that changed from the base of the PR and between 84c2c01 and 424668e.

📒 Files selected for processing (1)
  • .github/actions/workflow-build/build-workflow.py

📝 Walkthrough

Summary by CodeRabbit

  • Bug Fixes
    • Updated CPU-only CI dispatch jobs to use the correct runner label, improving job assignment and build reliability.

Walkthrough

important: verify cpu16m runner label exists in CI runner pool before merging; a mismatched label will cause dispatch jobs to fail to schedule.

Changes the CPU-only runner label in generate_dispatch_job_runner from a cpu16 suffix to cpu16m, affecting runner assignment for non-GPU CI dispatch jobs.

Changes

Runner Label Update

Layer / File(s) Summary
CPU runner suffix change
.github/actions/workflow-build/build-workflow.py
Runner label string changed from *-cpu16 to *-cpu16m for CPU-only job dispatch.

Related PRs: None identified.

Suggested labels: ci, github-actions

Suggested reviewers: None identified.

Poem
A rabbit hops through YAML lines,
Changes one small suffix, "m" it finds.
Sixteen cores now marked anew,
cpu16m, a label true.


Comment @coderabbitai help to get the list of available commands.

@trxcllnt

trxcllnt commented Jul 7, 2026

Copy link
Copy Markdown
Member

Other alternative is to create a swapfile (PR here). I imagine cpu16m runners are more costly and availability may be limited.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

😬 CI Workflow Results

🟥 Finished in 2h 50m: Pass: 99%/506 | Total: 16d 15h | Max: 2h 35m | Hits: 54%/1797913

See results here.

@Jacobfaib

Copy link
Copy Markdown
Contributor

Other alternative is to create a swapfile (#9739). I imagine cpu16 runners are more costly and availability may be limited.

If that also unblocks CI (which based on your 1-shot green CI it does) while keeping existing runner capacity, I'd be in favor of this.

@jrhemstad what do you think?

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

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

3 participants