From 1e5fb484c9f1570906f9d5a866f5b76d27c156ad Mon Sep 17 00:00:00 2001 From: Tariq Ibrahim Date: Wed, 16 Sep 2026 09:43:46 -0700 Subject: [PATCH] fix: [apply-cuda-memory-limits] exit gracefully on oci spec load failure With this fix, containers started with "userns --nomap" no longer fail as the OCI spec load operation is no longer non-blocking. This is required as the containers with "userns --nomap" fail even if they don't set the cuda memory limits environment variables. Signed-off-by: Tariq Ibrahim --- .../apply-cuda-memory-limits/apply-cuda-memory-limits.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cmd/nvidia-cdi-hook/apply-cuda-memory-limits/apply-cuda-memory-limits.go b/cmd/nvidia-cdi-hook/apply-cuda-memory-limits/apply-cuda-memory-limits.go index ce07f6636..2341034f5 100644 --- a/cmd/nvidia-cdi-hook/apply-cuda-memory-limits/apply-cuda-memory-limits.go +++ b/cmd/nvidia-cdi-hook/apply-cuda-memory-limits/apply-cuda-memory-limits.go @@ -115,7 +115,8 @@ func (m command) run(_ *cli.Command, cfg *config) error { fs := oci.NewFileSpec(specFilePath, false) _, err = fs.Load() if err != nil { - return fmt.Errorf("failed to load OCI container spec: %w", err) + m.logger.Warningf("skipping apply-cuda-memory-limits; failed to load OCI container spec: %v", err) + return nil } memReqStr, hasRequest := fs.LookupEnv(GPUMemoryRequestEnvName)