From 1be8ded964a52a0bbdc753d47ad6141b3aa1a2e4 Mon Sep 17 00:00:00 2001
From: Daniel J Walsh <dwalsh@redhat.com>
Date: Wed, 22 Apr 2020 11:50:52 -0400
Subject: Fix podman rm to have correct exit codes

If you attempt to remove a running container is it supposed to exit with
2
If you attempt to remove a non existing container is is supposed to exit with
1

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
---
 cmd/podman/containers/rm.go | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

(limited to 'cmd/podman')

diff --git a/cmd/podman/containers/rm.go b/cmd/podman/containers/rm.go
index 12a7a3d89..3021853a9 100644
--- a/cmd/podman/containers/rm.go
+++ b/cmd/podman/containers/rm.go
@@ -95,11 +95,9 @@ func rm(cmd *cobra.Command, args []string) error {
 	}
 	responses, err := registry.ContainerEngine().ContainerRm(context.Background(), args, rmOptions)
 	if err != nil {
-		// TODO exitcode is a global main variable to track exit codes.
-		// we need this enabled
-		//if len(c.InputArgs) < 2 {
-		//	exitCode = setExitCode(err)
-		//}
+		if len(args) < 2 {
+			setExitCode(err)
+		}
 		return err
 	}
 	for _, r := range responses {
@@ -108,6 +106,7 @@ func rm(cmd *cobra.Command, args []string) error {
 			if errors.Cause(err) == define.ErrWillDeadlock {
 				logrus.Errorf("Potential deadlock detected - please run 'podman system renumber' to resolve")
 			}
+			setExitCode(r.Err)
 			errs = append(errs, r.Err)
 		} else {
 			fmt.Println(r.Id)
@@ -115,3 +114,13 @@ func rm(cmd *cobra.Command, args []string) error {
 	}
 	return errs.PrintErrors()
 }
+
+func setExitCode(err error) {
+	cause := errors.Cause(err)
+	switch cause {
+	case define.ErrNoSuchCtr:
+		registry.SetExitCode(1)
+	case define.ErrCtrStateInvalid:
+		registry.SetExitCode(2)
+	}
+}
-- 
cgit v1.2.3-54-g00ecf