Skip to content

ci: Pin more nvidia-container dependencies#6637

Merged
malfet merged 1 commit into
mainfrom
seemethere-patch-3
May 16, 2025
Merged

ci: Pin more nvidia-container dependencies#6637
malfet merged 1 commit into
mainfrom
seemethere-patch-3

Conversation

@seemethere

@seemethere seemethere commented May 16, 2025

Copy link
Copy Markdown
Member

This pull request updates the .github/actions/setup-nvidia/action.yml file to enhance the NVIDIA container setup process by including additional dependencies for better compatibility and functionality.

Enhancements to NVIDIA container setup:

  • Updated the yum install command to include additional packages: libnvidia-container-tools-1.16.2, libnvidia-container1-1.16.2, and nvidia-container-toolkit-base-1.16.2, alongside the existing nvidia-docker2 and nvidia-container-toolkit-1.16.2. This ensures a more complete installation of NVIDIA container tools.

@vercel

vercel Bot commented May 16, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
torchci ⬜️ Ignored (Inspect) Visit Preview May 16, 2025 9:07pm

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 16, 2025
@malfet malfet merged commit 04c89fb into main May 16, 2025
7 checks passed
@malfet malfet deleted the seemethere-patch-3 branch May 16, 2025 21:09
@nWEIdia

nWEIdia commented May 17, 2025

Copy link
Copy Markdown
Contributor

I wonder whether the following installation sequence would be any better:

https://docs.nvidia.com/datacenter/cloud-native/container-toolkit/latest/install-guide.html#with-dnf-rhel-centos-fedora-amazon-linux

atalman pushed a commit to atalman/test-infra that referenced this pull request May 20, 2025
This pull request updates the `.github/actions/setup-nvidia/action.yml`
file to enhance the NVIDIA container setup process by including
additional dependencies for better compatibility and functionality.

Enhancements to NVIDIA container setup:

* Updated the `yum install` command to include additional packages:
`libnvidia-container-tools-1.16.2`, `libnvidia-container1-1.16.2`, and
`nvidia-container-toolkit-base-1.16.2`, alongside the existing
`nvidia-docker2` and `nvidia-container-toolkit-1.16.2`. This ensures a
more complete installation of NVIDIA container tools.
malfet pushed a commit that referenced this pull request May 20, 2025
Cherry pick of #6637

Co-authored-by: Eli Uriegas <1700823+seemethere@users.noreply.github.com>
sudo yum-config-manager --add-repo "${YUM_REPO_URL}"
sudo yum install -y nvidia-docker2 nvidia-container-toolkit-1.16.2
sudo yum install -y \
nvidia-docker2 \

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

As a note: nvidia-docker2 can be removed. It is an empty package that only serves to introduce a dependency on the nvidia-container-toolkit package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants