Update nvidia-ctk and use RUNTIME_DIRECTORY when calling nvidia-ctk

Updates nvidia-ctk since some patches were applied upstream. Also
changes the systemd service to use RUNTIME_DIRECTORY (this is what
upstream nixpkgs does when they call nvidia-ctk).

modules/default.nix

@@ -242,30 +242,28 @@ in
       otaUtils # Tools for UEFI capsule updates
     ];
 
-    systemd.tmpfiles.rules = lib.optional nvidiaContainerRuntimeActive "d /var/run/cdi 0755 root root - -";
-
     systemd.services.nvidia-cdi-generate = {
       enable = nvidiaContainerRuntimeActive;
       serviceConfig = {
         Type = "oneshot";
         RemainAfterExit = true;
-        ExecStart =
-          let
-            exe = "${pkgs.nvidia-jetpack.nvidia-ctk}/bin/nvidia-ctk";
-          in
-          toString [
-            exe
-            "cdi"
-            "generate"
-            "--nvidia-ctk-path=${exe}" # it is odd that this is needed, should be the same as /proc/self/exe?
-            "--driver-root=${pkgs.nvidia-jetpack.containerDeps}" # the root where nvidia libs will be resolved from
-            "--dev-root=/" # the root where chardevs will be resolved from
-            "--mode=csv"
-            "--csv.file=${pkgs.nvidia-jetpack.l4tCsv}"
-            "--output=/var/run/cdi/jetpack-nixos" # a yaml file extension is added by the nvidia-ctk tool
-          ];
+        RuntimeDirectory = "cdi";
       };
       wantedBy = [ "multi-user.target" ];
+      script =
+        let
+          exe = lib.getExe pkgs.nvidia-jetpack.nvidia-ctk;
+        in
+        ''
+          ${exe} cdi generate \
+            --nvidia-ctk-path=${exe} \
+            --driver-root=${pkgs.nvidia-jetpack.containerDeps} \
+            --ldconfig-path ${lib.getExe' pkgs.glibc "ldconfig"} \
+            --dev-root=/ \
+            --mode=csv \
+            --csv.file=${pkgs.nvidia-jetpack.l4tCsv} \
+            --output="$RUNTIME_DIRECTORY/jetpack-nixos"
+        '';
     };
 
     # Used by libEGL_nvidia.so.0

pkgs/containers/nvidia-ctk.nix

@@ -1,44 +1,39 @@
-{ fetchFromGitLab, buildGoModule, fetchpatch }:
+{ buildGoModule, fetchFromGitHub, fetchpatch }:
 
 buildGoModule rec {
   pname = "nvidia-ctk";
-  version = "unstable-${builtins.substring 0 7 src.rev}";
+  version = "1.15.0-rc.4";
 
   # TODO(jared): pin to v1.15.0 once it is released
   # We currently rely on some features in an unreleased version of nvidia
   # container toolkit.
-  src = fetchFromGitLab {
+  src = fetchFromGitHub {
     owner = "nvidia";
-    repo = "container-toolkit/container-toolkit";
-    rev = "a2262d00cc6d98ac2e95ae2f439e699a7d64dc17";
-    hash = "sha256-Oi04PIES0qTih/EiFBStIoBadM3H52+81KEfUumQcIs=";
+    repo = "nvidia-container-toolkit";
+    rev = "v${version}";
+    hash = "sha256-Ky0mGothIq5BOAHc4ujrMrh1niBYUoSgaRnv30ymjsE=";
   };
 
   patches = [
     # ensure nvidia-ctk can build with Go versions less than 1.20 (currently
     # required on their latest release)
     (fetchpatch {
-      name = "Fix-double-error-wrap-fmt";
-      url = "https://gitlab.com/nvidia/container-toolkit/container-toolkit/-/commit/80756d00a6b75761103c50f605cece5fa7e39392.patch";
-      hash = "sha256-hoeMUUPWKToCR7V/JG26wF6SCoHQwQORcGimH6EXDJ8=";
-    })
-    (fetchpatch {
-      name = "Use-golang-1.17";
-      url = "https://gitlab.com/nvidia/container-toolkit/container-toolkit/-/commit/5956b04096d1a92b241b13cc1f3e208f8b99eea0.patch";
-      hash = "sha256-VB3+ijc2Pdlm1W2LqvCjx9KDYKinWBkr/eiUJEwig/o=";
+      name = "Add-errors-Join-wrapper";
+      url = "https://github.com/NVIDIA/nvidia-container-toolkit/commit/92f17e94939bf8c213419749f5f7b48d2f0e618c.patch";
+      hash = "sha256-ioWstYky7LbIGtlfMMlbhIVN8yH7Qgp3z4wrkytT3TY=";
     })
     (fetchpatch {
-      name = "Draft-Compat-with-golang-1.17";
-      url = "https://gitlab.com/nvidia/container-toolkit/container-toolkit/-/commit/86f68a49014a4cffb7dcb51f14a02f6f1816b2ee.patch";
-      hash = "sha256-ioWstYky7LbIGtlfMMlbhIVN8yH7Qgp3z4wrkytT3TY=";
+      name = "Fix-double-error-wrap-fmt";
+      url = "https://github.com/NVIDIA/nvidia-container-toolkit/commit/f23fd2ce38ee3a9e87ac41c265b637cf97990ac7.patch";
+      hash = "sha256-hoeMUUPWKToCR7V/JG26wF6SCoHQwQORcGimH6EXDJ8=";
     })
-    # ensure nvidia-ctk can find ldconfig
-    ./nixos-ldconfig.patch
   ];
 
   subPackages = [ "cmd/nvidia-ctk" ];
 
   vendorHash = null;
 
   ldflags = [ "-s" "-w" "-extldflags=-Wl,-z,lazy" ];
+
+  meta.mainProgram = "nvidia-ctk";
 }