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

Use cuda_keyring for avoiding installing keys manually in install_cuda-scripts in the future? #449

Closed
mskyttner opened this issue May 9, 2022 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@mskyttner
Copy link
Contributor

mskyttner commented May 9, 2022

In a recent fix for #442 the list of keys for cuda libraries is manually updated with the latest key so that "apt-get update" does not fail (after April 27) when rebuilding with messages such as "The following signatures couldn't be verified because the public key is not available: NO_PUBKEY A4B469963BF863CC".

Nvidia recommends to use cuda_keyring which can avoid manual steps next time they rotate keys for CUDA Linux repositories, if I understand correctly.

This Dockerfile stanza seems to work for switching to their new repositories and installing cuda_keyring:

# use cuda_keyring instead of manually installed keys
RUN sed -i '/developer\.download\.nvidia\.com\/compute\/cuda\/repos/d' /etc/apt/sources.list.d/cuda.list && \
	sed -i '/developer\.download\.nvidia\.com\/compute\/cuda\/repos/d' /etc/apt/sources.list.d/nvidia-ml.list && \
	wget -O cuda-keyring.deb "https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64/cuda-keyring_1.0-1_all.deb" && \
	dpkg -i cuda-keyring.deb && \
	rm cuda-keyring.deb && \
	apt-get update

Would something similar to install cuda_keyring be nice to have in the install_cuda scripts?

@cboettig
Copy link
Member

cboettig commented May 9, 2022

thanks for looking into this, sounds like a good idea to me. Would you be interested in sending a PR for this?

@eitsupi eitsupi added the enhancement New feature or request label May 11, 2022
@eitsupi eitsupi added the good first issue Good for newcomers label Jun 6, 2022
@eitsupi
Copy link
Member

eitsupi commented Feb 8, 2025

We are deciding to stop supporting anything like rocker/ml here any more (#903), so I am closing this.
Thank you for your understanding.

@eitsupi eitsupi closed this as not planned Won't fix, can't repro, duplicate, stale Feb 8, 2025
@cboettig
Copy link
Member

Just for clarity -- by "here" I believe @eitsupi means "rocker-versioned2" and not "rocker-org" -- as per #903 thread we have a new setup in rocker/ml seeking to support these use cases. The discussion in #903 also mentions several other approaches outside the rocker project that might also be suitable.

(cuda install is handled upstream in this approach)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants