Skip to content

Commit

Permalink
Mount all of /sys in GFD instead of just /sys/class/dmi/id directory
Browse files Browse the repository at this point in the history
This change is motivated by WSL2 where the /sys/class/dmi directory does not exist.
Mount all of /sys as read-only is a safe approach to account for systems where this
directory does not exist.

Signed-off-by: Christopher Desiniotis <[email protected]>
  • Loading branch information
cdesiniotis committed Dec 20, 2024
1 parent e83c34a commit 2d8f120
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions assets/gpu-feature-discovery/0500_daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -127,8 +127,8 @@ spec:
volumeMounts:
- name: output-dir
mountPath: "/etc/kubernetes/node-feature-discovery/features.d"
- name: dmi-info-dir
mountPath: "/sys/class/dmi/id"
- name: host-sys
mountPath: /sys
readOnly: true
- name: config
mountPath: /config
Expand Down Expand Up @@ -169,9 +169,9 @@ spec:
- name: output-dir
hostPath:
path: "/etc/kubernetes/node-feature-discovery/features.d"
- name: dmi-info-dir
- name: host-sys
hostPath:
path: "/sys/class/dmi/id"
path: /sys
- name: run-nvidia-validations
hostPath:
path: "/run/nvidia/validations"
Expand Down

0 comments on commit 2d8f120

Please sign in to comment.