Skip to content

feat: mount devinfo directory in sriov device plugin pod#56

Closed
tsorya wants to merge 1 commit into
NVIDIA:public-mainfrom
tsorya:igal/sriov-dp-devinfo-mount
Closed

feat: mount devinfo directory in sriov device plugin pod#56
tsorya wants to merge 1 commit into
NVIDIA:public-mainfrom
tsorya:igal/sriov-dp-devinfo-mount

Conversation

@tsorya

@tsorya tsorya commented Jun 9, 2026

Copy link
Copy Markdown

Summary

  • Mount /var/run/k8s.cni.cncf.io/devinfo/dp as a HostPath volume (DirectoryOrCreate) in the sriov device plugin container
  • This path is required by the device plugin to write device information files per the device-info-spec, enabling CNI plugins to discover extended device properties (PCI address, PF mapping, etc.)
  • Matches the upstream sriov-network-device-plugin DaemonSet volume configuration

Test plan

  • Built dpf-system image and deployed to dev cluster
  • Verified controller detects outdated pod and recreates it with the new volume mount
  • Confirmed the volume mount is present at the expected path in the running device plugin pod

Mount /var/run/k8s.cni.cncf.io/devinfo/dp as a HostPath volume
(DirectoryOrCreate) in the sriov device plugin container. This path
is required by the device plugin to expose device information files
to CNI plugins per the device-info-spec.

Signed-off-by: Igal Tsoiref <itsoiref@redhat.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@souleb

souleb commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

@tsorya this is merged. Can you close the PR?

@tsorya

tsorya commented Jun 15, 2026

Copy link
Copy Markdown
Author

merged

@tsorya tsorya closed this Jun 15, 2026
AsherBond pushed a commit to ElasticProvisioner/doca-platform that referenced this pull request Jun 15, 2026
* feat: add devinfo volume to sriovdp pod

Implement upstream contrib NVIDIA#56

Signed-off-by: Soule BA <souleb@nvidia.com>

Based on main@3c2351fda3116efe434e832a81a40d436ff35dd8
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.

2 participants