Fix ignored --stop-timeout flag to 'podman create'

Also add some extra debug information to help figure out what's
going on when stop goes bad.

Fixes: #2472

Signed-off-by: Matthew Heon <matthew.heon@pm.me>

1 files changed, 11 insertions, 3 deletions

diff --git a/cmd/podman/stop.go b/cmd/podman/stop.go
index d86894a6f..ab9a2cf38 100644
--- a/cmd/podman/stop.go
+++ b/cmd/podman/stop.go
@@ -73,21 +73,29 @@ func stopCmd(c *cliconfig.StopValues) error {
 		fmt.Println(err.Error())
 	}
 
+	if c.Flag("timeout").Changed && c.Flag("time").Changed {
+		return errors.New("the --timeout and --time flags are mutually exclusive")
+	}
+
 	var stopFuncs []shared.ParallelWorkerInput
 	for _, ctr := range containers {
 		con := ctr
 		var stopTimeout uint
-		if c.Flag("timeout").Changed {
+		if c.Flag("timeout").Changed || c.Flag("time").Changed {
 			stopTimeout = c.Timeout
 		} else {
 			stopTimeout = ctr.StopTimeout()
+			logrus.Debugf("Set timeout to container %s default (%d)", ctr.ID(), stopTimeout)
 		}
 		f := func() error {
-			if err := con.StopWithTimeout(stopTimeout); err != nil && errors.Cause(err) != libpod.ErrCtrStopped {
+			if err := con.StopWithTimeout(stopTimeout); err != nil {
+				if errors.Cause(err) == libpod.ErrCtrStopped {
+					logrus.Debugf("Container %s already stopped", con.ID())
+					return nil
+				}
 				return err
 			}
 			return nil
-
 		}
 		stopFuncs = append(stopFuncs, shared.ParallelWorkerInput{
 			ContainerID:  con.ID(),