Merge pull request #7261 from zhangguanzhang/ps-format-add-field

Add the `Status` field in the ps --format=json

cmd/podman/containers/ps.go

@@ -109,6 +109,7 @@ func jsonOut(responses []entities.ListContainer) error {
 	r := make([]entities.ListContainer, 0)
 	for _, con := range responses {
 		con.CreatedAt = units.HumanDuration(time.Since(time.Unix(con.Created, 0))) + " ago"
+		con.Status = psReporter{con}.Status()
 		r = append(r, con)
 	}
 	b, err := json.MarshalIndent(r, "", "  ")

pkg/domain/entities/container_ps.go

@@ -56,6 +56,8 @@ type ListContainer struct {
 	StartedAt int64
 	// State of container
 	State string
+	// Status is a human-readable approximation of a duration for json output
+	Status string
 }
 
 // ListContainer Namespaces contains the identifiers of the container's Linux namespaces

test/e2e/ps_test.go

@@ -188,6 +188,21 @@ var _ = Describe("Podman ps", func() {
 		Expect(result.IsJSONOutputValid()).To(BeTrue())
 	})
 
+	It("podman ps print a human-readable `Status` with json format", func() {
+		_, ec, _ := podmanTest.RunLsContainer("test1")
+		Expect(ec).To(Equal(0))
+
+		result := podmanTest.Podman([]string{"ps", "-a", "--format", "json"})
+		result.WaitWithDefaultTimeout()
+		Expect(result.ExitCode()).To(Equal(0))
+		Expect(result.IsJSONOutputValid()).To(BeTrue())
+		// must contain "Status"
+		match, StatusLine := result.GrepString(`Status`)
+		Expect(match).To(BeTrue())
+		// container is running or exit, so it must contain `ago`
+		Expect(StatusLine[0]).To(ContainSubstring("ago"))
+	})
+
 	It("podman ps namespace flag with go template format", func() {
 		Skip(v2fail)
 		_, ec, _ := podmanTest.RunLsContainer("test1")