Skip to content

[release-1.19] feat(nvcdi): Allow IPC sockets to not be discovered.#1808

Merged
cdesiniotis merged 1 commit intorelease-1.19from
backport-1790-to-release-1.19
May 7, 2026
Merged

[release-1.19] feat(nvcdi): Allow IPC sockets to not be discovered.#1808
cdesiniotis merged 1 commit intorelease-1.19from
backport-1790-to-release-1.19

Conversation

@github-actions
Copy link
Copy Markdown

@github-actions github-actions Bot commented May 7, 2026

🤖 Automated backport of #1790 to release-1.19

✅ Cherry-pick completed successfully with no conflicts.

Original PR: #1790
Original Author: @LandonTClipp

Cherry-picked commits (1):

  • 6dad053 feat(nvcdi): Allow IPC sockets to not be discovered.

This backport was automatically created by the backport bot.

There are cases we are dealing with where the containers should not have access
to any NVIDIA IPC sockets like fabricmanager/persistenced. We want the ability
to disable this.

This commit adds a feature flag `disable-ipc-discoverer` that if provided will
cause the IPC discoverer.Discoverer to be nil. Ultimately, this is needed so that
the CDI spec file generated by k8s-device-plugin does not include these sockets.

Signed-off-by: LandonTClipp <lclipp@coreweave.com>
(cherry picked from commit 6dad053)
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 7, 2026

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.

@cdesiniotis
Copy link
Copy Markdown
Contributor

/ok to test e1a3e95

@cdesiniotis cdesiniotis enabled auto-merge May 7, 2026 19:50
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 25518459206

Coverage increased (+0.005%) to 43.325%

Details

  • Coverage increased (+0.005%) from the base build.
  • Patch coverage: 2 uncovered changes across 1 file (4 of 6 lines covered, 66.67%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
pkg/nvcdi/driver-nvml.go 6 4 66.67%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 14862
Covered Lines: 6439
Line Coverage: 43.33%
Coverage Strength: 0.48 hits per line

💛 - Coveralls

@cdesiniotis cdesiniotis merged commit 83eb602 into release-1.19 May 7, 2026
15 checks passed
@cdesiniotis cdesiniotis deleted the backport-1790-to-release-1.19 branch May 7, 2026 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants