Skip to content

Allow setting the NVIDIA_DRIVER_CAPABILITIES environment variable#1684

Open
linkages wants to merge 2 commits into
NVIDIA:mainfrom
linkages:main
Open

Allow setting the NVIDIA_DRIVER_CAPABILITIES environment variable#1684
linkages wants to merge 2 commits into
NVIDIA:mainfrom
linkages:main

Conversation

@linkages

@linkages linkages commented Apr 7, 2026

Copy link
Copy Markdown

This will allow setting the NVIDIA_DRIVER_CAPABILITIES environment variable on the nvidia-device-plugin-ctr container in the daemonset.

In my environment I need to change it from the default of "compute,utility" to "all".

This is required in my environment where I am using MiG. If this is not set then we get Insufficient Permissions errors from the container when trying to read memory information.

Eli Ben-Shoshan added 2 commits April 7, 2026 10:51
@copy-pr-bot

copy-pr-bot Bot commented Apr 7, 2026

Copy link
Copy Markdown

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.

@rajatchopra rajatchopra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

{{- end }}
- name: NVIDIA_VISIBLE_DEVICES
value: all
{{ if typeIs "string" .Values.nvidiaDriverCapabilties }}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
{{ if typeIs "string" .Values.nvidiaDriverCapabilties }}
{{ if typeIs "string" .Values.nvidiaDriverCapabilities }}

@tariq1890

Copy link
Copy Markdown
Contributor

The templatisation is required here as well -

@tariq1890

Copy link
Copy Markdown
Contributor

Thank you for your PR. After addressing the comments, please ensure the following are taken care of as well:

  • Commits are pushed with --signoff; this satisfies the DCO check
  • Commits are signed and verified
  • Commit history is squashed

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.

4 participants