Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Dec 9, 2025. It is now read-only.

Fix docs to correctly install nvidia driver #142

Merged
michaelasp merged 1 commit into from
Jun 28, 2025
Merged

Fix docs to correctly install nvidia driver #142

michaelasp merged 1 commit into from
Jun 28, 2025

Conversation

@michaelasp
Copy link
Collaborator

@michaelasp michaelasp commented Jun 27, 2025

No description provided.

@michaelasp
Copy link
Collaborator Author

michaelasp commented Jun 27, 2025

Tested locally but with

helm upgrade --install --create-namespace --namespace nvidia-dra-driver-gpu nvidia-dra-driver-gpu ./k8s-dra-driver-gpu/deployments/helm/nvidia-dra-driver-gpu --set gpuResourcesEnabledOverride=true --set nvidiaDriverRoot=/home/kubernetes/bin/nvidia --values=https://gist.githubusercontent.com/michaelasp/b051fba634d5615169dcb6357f2a5dc4/raw/214cc5c79acd6287e1ff5e36c141a12bc025c9f9/values.yaml --wait

Instead since the yaml is not yet in main, should work identically once this PR is merged.

@michaelasp michaelasp requested a review from aojea June 27, 2025 19:06
aojea
aojea reviewed Jun 27, 2025
View reviewed changes
gauravkghildiyal
gauravkghildiyal reviewed Jun 27, 2025
View reviewed changes
Copy link
Member

@gauravkghildiyal gauravkghildiyal left a comment

Choose a reason for hiding this comment

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

Thanks @michaelasp. Left one question

aojea
aojea previously approved these changes Jun 27, 2025
View reviewed changes
@michaelasp michaelasp merged commit 7d12923 into google:main Jun 28, 2025
7 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@gauravkghildiyal gauravkghildiyal gauravkghildiyal approved these changes

@aojea aojea aojea left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@michaelasp @aojea @gauravkghildiyal