ucx icon indicating copy to clipboard operation
ucx copied to clipboard

DEBIAN: Moved driver level dependencies under Recommends section.

Open rakhmets opened this issue 7 months ago • 3 comments

What?

Moved ucx-cuda package dependency on libnvidia-compute | libnvidia-ml1 from Depends to Recommends.

Why?

The default behavior of apt-install remains unchanged. And --no-install-recommends can be used to avoid installing driver level packages in a container.

rakhmets avatar May 30 '25 18:05 rakhmets

@tzafrir-mellanox could you please review the changes.

rakhmets avatar Jun 02 '25 07:06 rakhmets

Is the package functional without one of those installed? What are they required for? That said, seems reasonable.

tzafrir-mellanox avatar Jun 03 '25 14:06 tzafrir-mellanox

Is the package functional without one of those installed? What are they required for? That said, seems reasonable.

ucx-cuda depends on two libraries from these packages: libcuda.so from libnvidia-compute, and libnvidia-ml1.so from libnvidia-ml1. ucx-cuda is not functional without these two. However, these libraries are driver level libraries. Which is an issue, since installing them in a container means you're installing drivers in a container, which should be mounted from the host instead.

Adding these packages as Recommends will not break the default behavior of apt-get install. And this will allow to not include them with --no-install-recommends option, which could be used in container.

rakhmets avatar Jun 03 '25 18:06 rakhmets