Skip to content

Use core available for CUDA plugin ARM64 build. Build time 4 hours -> 1.5 hours. - #32399

Merged
Tianlei Wu (tianleiwu) merged 1 commit into
mainfrom
dev/erscor/2026/9/2-build-parallelism
Sep 3, 2026
Merged

Tianlei Wu (tianleiwu) merged 1 commit into
mainfrom
dev/erscor/2026/9/2-build-parallelism

Conversation

@eserscor

@eserscor eserscor commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Description

Motivation and Context

Copilot AI balanced review requested due to automatic review settings September 2, 2026 21:19

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.

🟢 Approval recommended

The implementation is straightforward; only a non-blocking stale comment needs updating.

Pull request overview

Adjusts ARM64 CUDA plugin build parallelism to reserve two CPU cores, reducing build time.

Changes:

  • Replaces the eight-job cap with nproc - 2 on larger ARM64 hosts.
File summaries
File Description
tools/ci_build/github/linux/build_cuda_plugin_package.sh Updates ARM64 build concurrency.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Balanced

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread tools/ci_build/github/linux/build_cuda_plugin_package.sh

@tianleiwu Tianlei Wu (tianleiwu) 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.

LGTM

@tianleiwu
Tianlei Wu (tianleiwu) merged commit c41c291 into main Sep 3, 2026
91 checks passed
@tianleiwu
Tianlei Wu (tianleiwu) deleted the dev/erscor/2026/9/2-build-parallelism branch September 3, 2026 01:21
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