Skip to content

bump cuda base image for precompiled ubuntu24.04#664

Open
rahulait wants to merge 1 commit intoNVIDIA:mainfrom
rahulait:bump-cuda-base
Open

bump cuda base image for precompiled ubuntu24.04#664
rahulait wants to merge 1 commit intoNVIDIA:mainfrom
rahulait:bump-cuda-base

Conversation

@rahulait
Copy link
Copy Markdown
Contributor

This PR bumps the cuda base image used when building ubuntu24.04 precompiled image
Fixes failures seen in #655

Base image contains the correct content:

docker run -it nvcr.io/nvidia/cuda:13.2.0-base-ubuntu24.04 /bin/bash
root@b973177b8614:/# sha256sum /usr/share/keyrings/cuda-archive-keyring.gpg
25100d6f2eccaee7d6719a70e3c0c6145064aa9edc2c32148f693ac4c524a376  /usr/share/keyrings/cuda-archive-keyring.gpg
root@b973177b8614:/# ls /etc/apt/sources.list.d/
cuda-ubuntu2404-x86_64.list  ubuntu.sources
root@b973177b8614:/# cat /etc/apt/sources.list.d/cuda-ubuntu2404-x86_64.list
deb [signed-by=/usr/share/keyrings/cuda-archive-keyring.gpg] https://developer.download.nvidia.com/compute/cuda/repos/ubuntu2404/x86_64/ /
root@b973177b8614:/# exit

Signed-off-by: Rahul Sharma <rahulsharm@nvidia.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Ubuntu 24.04 precompiled image build to use a newer NVIDIA CUDA base image that already includes the correct CUDA APT keyring and repo configuration, resolving the build failures referenced in #655.

Changes:

  • Bump CUDA base image for ubuntu24.04/precompiled from 13.1.1-base-ubuntu24.04 to 13.2.0-base-ubuntu24.04.
  • Remove the Dockerfile step that manually deletes/adds CUDA repo keys and rewrites the CUDA APT source (now provided correctly by the base image).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants