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.

Sign up for GitHub

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

Jump to bottom

Add nvidia-imex package #200

Merged
merged 2 commits into from
Jan 28, 2025
Merged

Add nvidia-imex package #200

merged 2 commits into from
Jan 28, 2025

Conversation

cdesiniotis
Copy link
Contributor

@cdesiniotis cdesiniotis commented Jan 27, 2025 •
edited
Loading 

$ sudo chroot /run/nvidia/driver apt list --installed | grep imex
nvidia-imex-570/now 570.86.15-1 amd64 [installed,local]

$ sudo chroot /run/nvidia/driver which nvidia-imex
/usr/bin/nvidia-imex
$ sudo chroot /run/nvidia/driver which nvidia-imex-ctl
/usr/bin/nvidia-imex-ctl


$ sudo chroot /run/nvidia/driver dpkg-query -L nvidia-imex-570
/.
/etc
/etc/nvidia-imex
/etc/nvidia-imex/config.cfg
/lib
/lib/systemd
/lib/systemd/system
/lib/systemd/system/nvidia-imex.service
/usr
/usr/bin
/usr/bin/nvidia-imex
/usr/bin/nvidia-imex-ctl
/usr/share
/usr/share/doc
/usr/share/doc/nvidia-imex
/usr/share/doc/nvidia-imex/third-party-notices.txt.gz
/usr/share/doc/nvidia-imex-570
/usr/share/doc/nvidia-imex-570/changelog.Debian.gz
/usr/share/doc/nvidia-imex-570/copyright

@cdesiniotis cdesiniotis requested a review from tariq1890 January 27, 2025 18:29
@tariq1890
Copy link
Contributor

Looks like the minimum supported driver branch for IMEX is R550

@tariq1890
Copy link
Contributor

Please make the changes in the precompiled docker images as well.

@cdesiniotis cdesiniotis force-pushed the imex-pkg branch 4 times, most recently from 7fd422d to 39c6826 Compare January 27, 2025 21:28
@cdesiniotis
Copy link
Contributor Author

@tariq1890 addressed both of your comments. PTAL.

@cdesiniotis cdesiniotis marked this pull request as ready for review January 27, 2025 22:16
@cdesiniotis
Copy link
Contributor Author

Verified that the nvidia-imex package is now included in the precompiled containers where driver branch >= 550:

$ docker run --rm -it --entrypoint /bin/sh ghcr.io/nvidia/driver:550-5.15.0-131-generic-ubuntu22.04@sha256:d7ecca08f1f1354fb1379b431c17fd27c847ceb8159bddbc358aecda44839964
# dpkg-query -L nvidia-imex-550
/.
/etc
/etc/nvidia-imex
/etc/nvidia-imex/config.cfg
/lib
/lib/systemd
/lib/systemd/system
/lib/systemd/system/nvidia-imex.service
/usr
/usr/bin
/usr/bin/nvidia-imex
/usr/bin/nvidia-imex-ctl
/usr/share
/usr/share/doc
/usr/share/doc/nvidia-imex
/usr/share/doc/nvidia-imex/third-party-notices.txt.gz
/usr/share/doc/nvidia-imex-550
/usr/share/doc/nvidia-imex-550/changelog.Debian.gz
/usr/share/doc/nvidia-imex-550/copyright


$ docker run --rm -it --entrypoint /bin/sh ghcr.io/nvidia/driver:535-5.15.0-131-generic-ubuntu22.04@sha256:d7ecca08f1f1354fb1379b431c17fd27c847ceb8159bddbc358aecda44839964
# dpkg-query -L nvidia-imex-535
dpkg-query: package 'nvidia-imex-535' is not installed
Use dpkg --contents (= dpkg-deb --contents) to list archive files contents.

@cdesiniotis
Add nvidia-imex package
ef2d9d1 
Signed-off-by: Christopher Desiniotis <cdesiniotis@nvidia.com>
@cdesiniotis cdesiniotis force-pushed the imex-pkg branch 2 times, most recently from 7a8e290 to c191f81 Compare January 28, 2025 07:52
@cdesiniotis
[ubuntu20.04] force install nvidia-imex pkg to workaround unnecessary…
acf3325 
… dependency issues

The debian packages for Ubuntu 20.04 incorrectly mark openssl >= 3 as a dependency
of the nvidia-imex-$DRIVER_BRANCH package, even though openssl is statically linked.
This commit acts as workaround until the upstream imex packages are fixed.

Signed-off-by: Christopher Desiniotis <cdesiniotis@nvidia.com>
@cdesiniotis cdesiniotis merged commit f2ce110 into main Jan 28, 2025
12 checks passed
@tariq1890 tariq1890 deleted the imex-pkg branch January 28, 2025 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tariq1890 tariq1890 tariq1890 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cdesiniotis @tariq1890