Skip to content

dra: fix HOST_ROOT envvar and add missing mounts for PassthroughSupport - #2707

Merged
cdesiniotis merged 1 commit into
NVIDIA:mainfrom
cdesiniotis:dra-passthrough-fixes
Aug 1, 2026
Merged

dra: fix HOST_ROOT envvar and add missing mounts for PassthroughSupport#2707
cdesiniotis merged 1 commit into
NVIDIA:mainfrom
cdesiniotis:dra-passthrough-fixes

Conversation

@cdesiniotis

@cdesiniotis cdesiniotis commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Description

This PR fixes a few bugs encountered when enabling the PassthroughSupport feature gate in the NVIDIA DRA Driver for GPUS. First, the HOST_ROOT variable was not set correctly, leading to the following error message in the kubelet-plugin:

Error: invalid CLI flags: host root is not mounted at "/host-root"

After this issue was fixed, the below issue was encountered as we were missing some volume mounts required for the PassthroughSupport feature.

Error: error creating driver: error enumerating all possible devices: error enumerating GPU PCI devices: error getting GPU info from PCI device: error finding best VFIO driver for device "0009:01:00.0": failed to get vfio_pci aliases in modules.alias file: failed to read file /lib/modules/5.14.0-687.20.1.el9_8.aarch64/modules.alias: open /lib/modules/5.14.0-687.20.1.el9_8.aarch64/modules.alias: no such file or directory

See upstream DRA driver manifest: https://github.com/kubernetes-sigs/dra-driver-nvidia-gpu/blob/50e0e3568f633d87266e3aabf8afa510843d6c7d/deployments/helm/dra-driver-nvidia-gpu/templates/kubeletplugin.yaml#L338-L353

Checklist

  • No secrets, sensitive information, or unrelated changes
  • Lint checks passing (make lint)
  • Generated assets in-sync (make validate-generated-assets)
  • Go mod artifacts in-sync (make validate-modules)
  • Test cases are added for new code paths

Testing

Manual testing on a single node OpenShift cluster. Deployed with and without this change.

Comment thread internal/state/dra_driver.go Outdated
Comment thread manifests/state-dra-driver/0500_daemonset.yaml
@cdesiniotis
cdesiniotis force-pushed the dra-passthrough-fixes branch from 72e7224 to 0dbf00c Compare July 31, 2026 22:59
Comment thread internal/state/dra_driver.go Outdated
Comment thread internal/state/dra_driver.go Outdated
@cdesiniotis
cdesiniotis force-pushed the dra-passthrough-fixes branch 2 times, most recently from 6d53db6 to 879401f Compare July 31, 2026 23:36
@tariq1890

Copy link
Copy Markdown
Contributor

LGTM. Once gofmt is applied, this should be good to merge.

Signed-off-by: Christopher Desiniotis <cdesiniotis@nvidia.com>
@cdesiniotis
cdesiniotis force-pushed the dra-passthrough-fixes branch from 879401f to 49ce973 Compare August 1, 2026 00:05
@cdesiniotis

Copy link
Copy Markdown
Contributor Author

LGTM. Once gofmt is applied, this should be good to merge.

Fixed now.

@cdesiniotis
cdesiniotis enabled auto-merge August 1, 2026 00:17
@cdesiniotis
cdesiniotis merged commit 14ca11b into NVIDIA:main Aug 1, 2026
36 of 38 checks passed
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