Supress error when deleting non-existing container with --force

This addresses the first issue reported in #59. CRI-O expects runsc to
return success to delete when --force is used with a non-existing container.

PiperOrigin-RevId: 198487418
Change-Id: If7660e8fdab1eb29549d0a7a45ea82e20a1d4f4a

3 files changed, 65 insertions, 7 deletions

diff --git a/runsc/cmd/BUILD b/runsc/cmd/BUILD
index 08aaee996..4b4afa4a0 100644
--- a/runsc/cmd/BUILD
+++ b/runsc/cmd/BUILD
@@ -44,13 +44,17 @@ go_library(
 go_test(
     name = "cmd_test",
     size = "small",
-    srcs = ["exec_test.go"],
+    srcs = [
+        "delete_test.go",
+        "exec_test.go",
+    ],
     embed = [":cmd"],
     deps = [
         "//pkg/abi/linux",
         "//pkg/sentry/control",
         "//pkg/sentry/kernel/auth",
         "//pkg/urpc",
+        "//runsc/boot",
         "@com_github_google_go-cmp//cmp:go_default_library",
         "@com_github_google_go-cmp//cmp/cmpopts:go_default_library",
         "@com_github_opencontainers_runtime-spec//specs-go:go_default_library",
diff --git a/runsc/cmd/delete.go b/runsc/cmd/delete.go
index 769a11c45..46de5f348 100644
--- a/runsc/cmd/delete.go
+++ b/runsc/cmd/delete.go
@@ -15,9 +15,13 @@
 package cmd
 
 import (
+	"fmt"
+	"os"
+
 	"context"
 	"flag"
 	"github.com/google/subcommands"
+	"gvisor.googlesource.com/gvisor/pkg/log"
 	"gvisor.googlesource.com/gvisor/runsc/boot"
 	"gvisor.googlesource.com/gvisor/runsc/container"
 )
@@ -56,19 +60,28 @@ func (d *Delete) Execute(_ context.Context, f *flag.FlagSet, args ...interface{}
 	}
 
 	conf := args[0].(*boot.Config)
+	if err := d.execute(f.Args(), conf); err != nil {
+		Fatalf("%v", err)
+	}
+	return subcommands.ExitSuccess
+}
 
-	for i := 0; i < f.NArg(); i++ {
-		id := f.Arg(i)
+func (d *Delete) execute(ids []string, conf *boot.Config) error {
+	for _, id := range ids {
 		c, err := container.Load(conf.RootDir, id)
 		if err != nil {
-			Fatalf("error loading container %q: %v", id, err)
+			if os.IsNotExist(err) && d.force {
+				log.Warningf("couldn't find container %q: %v", id, err)
+				return nil
+			}
+			return fmt.Errorf("error loading container %q: %v", id, err)
 		}
 		if !d.force && (c.Status == container.Running) {
-			Fatalf("cannot stop running container without --force flag")
+			return fmt.Errorf("cannot stop running container without --force flag")
 		}
 		if err := c.Destroy(); err != nil {
-			Fatalf("error destroying container: %v", err)
+			return fmt.Errorf("error destroying container: %v", err)
 		}
 	}
-	return subcommands.ExitSuccess
+	return nil
 }
diff --git a/runsc/cmd/delete_test.go b/runsc/cmd/delete_test.go
new file mode 100644
index 000000000..928e9ee2c
--- /dev/null
+++ b/runsc/cmd/delete_test.go
@@ -0,0 +1,41 @@
+// Copyright 2018 Google Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package cmd
+
+import (
+	"io/ioutil"
+	"testing"
+
+	"gvisor.googlesource.com/gvisor/runsc/boot"
+)
+
+func TestNotFound(t *testing.T) {
+	ids := []string{"123"}
+	dir, err := ioutil.TempDir("", "metadata")
+	if err != nil {
+		t.Fatalf("error creating dir: %v", err)
+	}
+	conf := &boot.Config{RootDir: dir}
+
+	d := Delete{}
+	if err := d.execute(ids, conf); err == nil {
+		t.Error("Deleting non-existend container should have failed")
+	}
+
+	d = Delete{force: true}
+	if err := d.execute(ids, conf); err != nil {
+		t.Errorf("Deleting non-existend container with --force should NOT have failed: %v", err)
+	}
+}