Skip to content
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

[Fix] check for the CUDA version installed on image #1418

Merged
merged 2 commits into from
May 3, 2024

Conversation

jstourac
Copy link
Member

@jstourac jstourac commented May 3, 2024

Up to now we used nvidia-smi for the CUDA version check, but because of the [1] we should rather use the nvcc tool for such operation.

Also, correct version of CUDA is 12.1 in our images now, see [2].

This is a followup of #1413 [3].

[1] https://issues.redhat.com/browse/RHOAIENG-6673
[2] https://github.com/red-hat-data-services/notebooks/blob/rhoai-2.9/cuda/ubi9-python-3.9/Dockerfile#L29
[3] aae86d2


CI: good - failed test is unrelated issue to this PR
Screenshot from 2024-05-03 13-55-05

Up to now we used `nvidia-smi` for the CUDA version check, but because
of the [1] we should rather use the `nvcc` tool for such operation.

Also, correct version of CUDA is 12.1 in our images now, see [2].

This is a followup of red-hat-data-services#1413 [3].

[1] https://issues.redhat.com/browse/RHOAIENG-6673
[2] https://github.com/red-hat-data-services/notebooks/blob/rhoai-2.9/cuda/ubi9-python-3.9/Dockerfile#L29
[3] aae86d2
@jstourac jstourac added verified This PR has been tested with Jenkins enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) labels May 3, 2024
@jstourac jstourac self-assigned this May 3, 2024
@jstourac jstourac requested review from FedeAlonso, lugi0 and jiridanek May 3, 2024 11:56
Copy link
Contributor

github-actions bot commented May 3, 2024

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
446 0 0 446 100

@FedeAlonso
Copy link
Contributor

Thank you, Jan!
LGTM

Copy link

sonarqubecloud bot commented May 3, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@jstourac jstourac merged commit 3e55ab3 into red-hat-data-services:master May 3, 2024
8 checks passed
@jstourac jstourac deleted the fixCuda branch May 3, 2024 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) verified This PR has been tested with Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants