Skip to content

CI: Use NPROC in all Jenkins jobs - #11469

Closed
Alexey-Rivkin wants to merge 1 commit into
openucx:masterfrom
Alexey-Rivkin:nproc-jenkins-jobs
Closed

Alexey-Rivkin wants to merge 1 commit into
openucx:masterfrom
Alexey-Rivkin:nproc-jenkins-jobs

Conversation

@Alexey-Rivkin

Copy link
Copy Markdown
Contributor

What?

Replace remaining raw nproc with ${NPROC} in Jenkins-invoked CI scripts:

  • buildlib/tools/static_checks.sh
  • contrib/test_jenkins.sh (test_init_mt)

Why?

Follow-up to #11161. Both scripts source common.sh but still called nproc directly, missing the
adaptive cap on k8s.

@Alexey-Rivkin

Copy link
Copy Markdown
Contributor Author

/build

6 similar comments
@Alexey-Rivkin

Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin

Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin

Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin

Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin

Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin

Copy link
Copy Markdown
Contributor Author

/build

@dpressle

Copy link
Copy Markdown
Contributor

@svc-nvidia-pr-review

@svc-nvidia-pr-review

Copy link
Copy Markdown

🚫 @dpressle — not authorized to trigger a review on this repo. Contact a repo admin to be added to the allowed-users list.

@Alexey-Rivkin
Alexey-Rivkin requested a review from dpressle May 27, 2026 08:29
@Alexey-Rivkin

Copy link
Copy Markdown
Contributor Author

/build

1 similar comment
@Alexey-Rivkin

Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin

Copy link
Copy Markdown
Contributor Author

/build

@Alexey-Rivkin

Copy link
Copy Markdown
Contributor Author

/build

PR openucx#11161 introduced an adaptive NPROC cap in common.sh to avoid
OOM on k8s nodes where `nproc` reports the full host. Two
Jenkins-invoked spots still called `nproc` directly; switch them
to ${NPROC} so they honor the cap.

Signed-off-by: Alexey Rivkin <arivkin@nvidia.com>
@Alexey-Rivkin

Copy link
Copy Markdown
Contributor Author

/build

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