bugfix: we should set Running flag to true when started container

Signed-off-by: Michael Wan <zirenwan@gmail.com>

daemon/mgr/container_state.go

@@ -90,6 +90,7 @@ func (c *Container) SetStatusRunning(pid int64) {
 	c.State.StartedAt = time.Now().UTC().Format(utils.TimeLayout)
 	c.State.Pid = pid
 	c.State.ExitCode = 0
+	c.State.Running = true
 }
 
 // SetStatusStopped sets a container to be status stopped.

test/cli_run_test.go

@@ -403,3 +403,22 @@ func (suite *PouchRunMemorySuite) TestRunWithShm(c *check.C) {
 
 	util.PartialEqual(res.Stdout(), "1048576")
 }
+
+// TestRunSetRunningFlag is to verfy whether set Running Flag in ContainerState
+// when started a container
+func (suite *PouchRunSuite) TestRunSetRunningFlag(c *check.C) {
+	cname := "TestRunSetRunningFlag"
+	res := command.PouchRun("run", "-d", "--name", cname, busyboxImage, "top")
+	defer DelContainerForceMultyTime(c, cname)
+	res.Assert(c, icmd.Success)
+
+	// test if the value is in inspect result
+	res = command.PouchRun("inspect", cname)
+	res.Assert(c, icmd.Success)
+
+	result := []types.ContainerJSON{}
+	if err := json.Unmarshal([]byte(res.Stdout()), &result); err != nil {
+		c.Errorf("failed to decode inspect output: %v", err)
+	}
+	c.Assert(result[0].State.Running, check.Equals, true)
+}