-
Notifications
You must be signed in to change notification settings - Fork 223
GPU: Add unsupported distro list, add NVIDIA GPU driver installation for Debian #4132
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
|
||
| self._log.info("Successfully installed CUDA driver for Ubuntu") | ||
|
|
||
| def _install_cuda_debian(self) -> None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try to create a common method that works for both Debian and Ubuntu, avoiding duplicated code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ack
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated
b35f262 to
cc89971
Compare
|
Test is ongoing. The PR potentially impacts a lot of distro and variants. |
81d8473 to
429a732
Compare
Move logic to skip GPU cases based on distro to case requirement
Checking of GPU support is not supposed to happen in verification process. The is_supported checks are spread across multiple functions for GPU. We need to gradually reduce this incosistency
ecf55d7 to
447f69c
Compare
447f69c to
6c67f3f
Compare
|
Not include changes for SUSE in this PR due to multiple issues faced while enabling SUSE. It may need more time |
No description provided.