It is OK to start an already running container (with no attach)

Signed-off-by: baude <bbaude@redhat.com>

Closes: #637
Approved by: rhatdan

1 files changed, 9 insertions, 6 deletions

diff --git a/cmd/podman/start.go b/cmd/podman/start.go
index 8f6fc5800..06b8f815f 100644
--- a/cmd/podman/start.go
+++ b/cmd/podman/start.go
@@ -90,16 +90,16 @@ func startCmd(c *cli.Context) error {
 			continue
 		}
 
+		ctrState, err := ctr.State()
+		if err != nil {
+			return errors.Wrapf(err, "unable to get container state")
+		}
+
 		if attach {
 			inputStream := os.Stdin
 			if !c.Bool("interactive") {
 				inputStream = nil
 			}
-
-			ctrState, err := ctr.State()
-			if err != nil {
-				return errors.Wrapf(err, "unable to get container state")
-			}
 			if ctrState == libpod.ContainerStateRunning {
 				return attachCtr(ctr, os.Stdout, os.Stderr, inputStream, c.String("detach-keys"), c.BoolT("sig-proxy"))
 			}
@@ -116,7 +116,10 @@ func startCmd(c *cli.Context) error {
 
 			return ctr.Cleanup()
 		}
-
+		if ctrState == libpod.ContainerStateRunning {
+			fmt.Println(ctr.ID())
+			continue
+		}
 		// Handle non-attach start
 		if err := ctr.Start(); err != nil {
 			if lastError != nil {