Skip to content

Conversation

@elezar
Copy link
Contributor

@elezar elezar commented Nov 25, 2025

This change switches to using CDI to handle the --gpus flag. This removes the custom implementation that invoked the nvidia-container-cli directly. This mechanism does not align with existing implementations.

See also:

Copy link
Member

Choose a reason for hiding this comment

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

Thanks, does this need any change on https://github.com/containerd/nerdctl/blob/main/docs/gpu.md ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have updated this as well in the latest revision.

@AkihiroSuda AkihiroSuda added this to the v2.3.0 milestone Nov 26, 2025
@elezar
Copy link
Contributor Author

elezar commented Nov 26, 2025 via email

This change switches to using CDI to handle the --gpus flag.
This removes the custom implementation that invoked the nvidia-container-cli
directly. This mechanism does not align with existing implementations.

Signed-off-by: Evan Lezar <elezar@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants