Fix the --runtime option of container create is ignored

Signed-off-by: Kay Yan <kay.yan@daocloud.io>
containerd · Aug 8, 2023 · d78d883 · d78d883
1 parent 9cd00de
commit d78d883
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/cmd/nerdctl/compose_up_test.go b/cmd/nerdctl/compose_up_test.go
@@ -46,7 +46,7 @@ services:
 	c := base.ComposeCmd("-f", comp.YAMLFullPath(), "up", "-d")
 	expected := icmd.Expected{
 		ExitCode: 1,
-		Err:      `exec: \"invalid\": executable file not found in $PATH`,
+		Err:      `failed to resolve runtime path: invalid runtime name invalid`,
 	}
 	if base.Target == testutil.Docker {
 		expected.Err = `Unknown runtime specified invalid`

diff --git a/pkg/cmd/container/run_runtime.go b/pkg/cmd/container/run_runtime.go
@@ -18,6 +18,7 @@ package container
 
 import (
 	"context"
+	"path/filepath"
 	"strings"
 
 	"github.com/containerd/containerd"
@@ -47,6 +48,9 @@ func generateRuntimeCOpts(cgroupManager, runtimeStr string) ([]containerd.NewCon
 				}
 				runtimeOpts = nil
 			}
+		} else if filepath.IsAbs(runtimeStr) {
+			// runtimeStr is absolute path to runtime binary
+			runtime = runtimeStr
 		} else {
 			// runtimeStr is a runc binary
 			runcOpts.BinaryName = runtimeStr