terraform-provider-iterative icon indicating copy to clipboard operation
terraform-provider-iterative copied to clipboard

nvidia-docker2 might not be needed

Open DavidGOrtega opened this issue 1 year ago • 6 comments

With the release of Docker 19.03, usage of nvidia-docker2 packages is deprecated since NVIDIA GPUs are now natively supported as devices in the Docker runtime.

DavidGOrtega avatar Sep 16 '22 22:09 DavidGOrtega

Replace nvidia-docker2 with nvidia-container-toolkit here:

https://github.com/iterative/terraform-provider-iterative/blob/1e888fbb77b1be0d640ac95dd0df40168b066b20/environment/setup.sh#L40

0x2b3bfa0 avatar Sep 17 '22 06:09 0x2b3bfa0

Hello @DavidGOrtega! I would like to take over this issue. Can you please assign it to me?

aliabbasjaffri avatar Oct 06 '22 21:10 aliabbasjaffri

Yes, sure!

0x2b3bfa0 avatar Oct 06 '22 22:10 0x2b3bfa0

Are there any other changes required apart from the one mentioend in the comment above? I tried to look for instances where the old --runtime was being used; either in code or readme, but none found.

aliabbasjaffri avatar Oct 07 '22 07:10 aliabbasjaffri

I think that no other changes are required

0x2b3bfa0 avatar Oct 07 '22 16:10 0x2b3bfa0

After seeing that https://github.com/NVIDIA/nvidia-docker/commit/88a2fda83cb3cf42221931dbd0c08aa4370d4fe5 points to a documentation page explicitly requiring the installation of the allegedly deprecated nvidia-docker2 package, the best thing we can do is ~question their sanity~ close this issue.

0x2b3bfa0 avatar Oct 09 '22 03:10 0x2b3bfa0