Skip to content

Conversation

zvonkok
Copy link
Contributor

@zvonkok zvonkok commented Jul 14, 2025

Remove gpu admin tools and pin toolkit version.

zvonkok added 2 commits July 14, 2025 21:59
NVRC got a new feature reading the CC mode directly from register

Signed-off-by: Zvonko Kaiser <zkaiser@nvidia.com>
New versions have incompatibilites, pin toolkit to a working
version

Signed-off-by: Zvonko Kaiser <zkaiser@nvidia.com>
@@ -67,7 +67,7 @@ install_nvidia_ctk() {
echo "chroot: Installing NVIDIA GPU container runtime"
apt list nvidia-container-toolkit-base -a
# Base gives a nvidia-ctk and the nvidia-container-runtime
eval "${APT_INSTALL}" nvidia-container-toolkit-base
eval "${APT_INSTALL}" nvidia-container-toolkit-base=1.17.6-1
Copy link
Member

Choose a reason for hiding this comment

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

This is a nit, but should this be added as part of versions.yaml?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes and No we want to test always the latest version to see if things break, so essentially we do not care until it breaks, but may be a good thing to have it in the versions.yaml I haven't had a need for that, famous last words.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, leave it out for now and we re-evaluate it after the release.
Fair?

Copy link
Contributor

Choose a reason for hiding this comment

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

Does the package repo where this comes from keep old versions around? At least for the standard debian repos, I noticed that pinning packages like this only works for some time, but eventually the mirrors all drop the older versions. Maybe not a concern, because "we want to test always the latest version"?

Copy link
Member

@fidencio fidencio left a comment

Choose a reason for hiding this comment

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

lgtm, assuming green tests.
thanks @zvonkok!

@@ -67,7 +67,7 @@ install_nvidia_ctk() {
echo "chroot: Installing NVIDIA GPU container runtime"
apt list nvidia-container-toolkit-base -a
# Base gives a nvidia-ctk and the nvidia-container-runtime
eval "${APT_INSTALL}" nvidia-container-toolkit-base
eval "${APT_INSTALL}" nvidia-container-toolkit-base=1.17.6-1
Copy link
Contributor

Choose a reason for hiding this comment

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

Does the package repo where this comes from keep old versions around? At least for the standard debian repos, I noticed that pinning packages like this only works for some time, but eventually the mirrors all drop the older versions. Maybe not a concern, because "we want to test always the latest version"?

@fidencio fidencio merged commit 11c744c into kata-containers:main Jul 15, 2025
973 of 1025 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants