Sponsored Content
Skip to content

ci: Limit nproc on Linux builds temporarily - #7132

Merged
legleux merged 1 commit into
developfrom
legleux/limit-linux-nproc
May 12, 2026
Merged

ci: Limit nproc on Linux builds temporarily#7132
legleux merged 1 commit into
developfrom
legleux/limit-linux-nproc

Conversation

@legleux

@legleux legleux commented May 12, 2026

Copy link
Copy Markdown
Contributor

High Level Overview of Change

Temporary mitigation for OOMs in the "Build the binary" step on large Linux runners. Only this step is capped; deps, tests, and coverage are unchanged. macOS and Windows are unaffected.

Future Tasks

Assessing the actual resources required for GitHub runners will be in progress shortly.

@xrplf-ai-reviewer xrplf-ai-reviewer Bot 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.

No issues.

Review by Claude Opus 4.6 · Prompt: V15

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.

Pull request overview

This PR introduces a temporary CI mitigation to reduce out-of-memory (OOM) failures during the Linux “Build the binary” step by limiting build parallelism on Linux runners.

Changes:

  • Set BUILD_NPROC to a Linux-specific value for the “Build the binary” step.
  • Leave dependency build, tests, and coverage parallelism unchanged.

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

Comment thread .github/workflows/reusable-build-test-config.yml
@codecov

codecov Bot commented May 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.1%. Comparing base (aa55392) to head (6c4d7a1).
⚠️ Report is 3 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##           develop   #7132     +/-   ##
=========================================
- Coverage     82.1%   82.1%   -0.0%     
=========================================
  Files         1010    1010             
  Lines        76064   76064             
  Branches      7375    7375             
=========================================
- Hits         62470   62469      -1     
- Misses       13594   13595      +1     

see 1 file with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@legleux
legleux requested review from bthomee and mathbunnyru May 12, 2026 20:18
@legleux
legleux added this pull request to the merge queue May 12, 2026
Merged via the queue into develop with commit 170eb5e May 12, 2026
3 checks passed
@legleux
legleux deleted the legleux/limit-linux-nproc branch May 12, 2026 21:41
@mvadari mvadari added this to the 3.2.0 milestone May 20, 2026
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.

4 participants