diff options
author | Brent Baude <bbaude@redhat.com> | 2020-06-01 09:43:45 -0500 |
---|---|---|
committer | Brent Baude <bbaude@redhat.com> | 2020-06-01 11:54:15 -0500 |
commit | 990514ea92b9b50dfb95dc0e505e11a21578a649 (patch) | |
tree | 8ba9db4d111bd7379ac09780533d304154b69658 | |
parent | f559cec6c0d1694cca9530004aaba3c138f621e3 (diff) | |
download | podman-990514ea92b9b50dfb95dc0e505e11a21578a649.tar.gz podman-990514ea92b9b50dfb95dc0e505e11a21578a649.tar.bz2 podman-990514ea92b9b50dfb95dc0e505e11a21578a649.zip |
Add support for format {{.Label}}
the pod ps man page says .Label is valid go template format. i dont think the function was actually ever implemented.
Fixes #6448
Signed-off-by: Brent Baude <bbaude@redhat.com>
-rw-r--r-- | cmd/podman/pods/ps.go | 5 | ||||
-rw-r--r-- | pkg/api/handlers/utils/pods.go | 1 | ||||
-rw-r--r-- | pkg/domain/entities/pods.go | 1 | ||||
-rw-r--r-- | pkg/domain/infra/abi/pods.go | 1 | ||||
-rw-r--r-- | test/e2e/pod_ps_test.go | 16 |
5 files changed, 24 insertions, 0 deletions
diff --git a/cmd/podman/pods/ps.go b/cmd/podman/pods/ps.go index 5703bd172..1385ff270 100644 --- a/cmd/podman/pods/ps.go +++ b/cmd/podman/pods/ps.go @@ -178,6 +178,11 @@ func (l ListPodReporter) Created() string { return units.HumanDuration(time.Since(l.ListPodsReport.Created)) + " ago" } +// Labels returns a map of the pod's labels +func (l ListPodReporter) Labels() map[string]string { + return l.ListPodsReport.Labels +} + // NumberofContainers returns an int representation for // the number of containers belonging to the pod func (l ListPodReporter) NumberOfContainers() int { diff --git a/pkg/api/handlers/utils/pods.go b/pkg/api/handlers/utils/pods.go index fb795fa6a..5b6f6d34d 100644 --- a/pkg/api/handlers/utils/pods.go +++ b/pkg/api/handlers/utils/pods.go @@ -66,6 +66,7 @@ func GetPods(w http.ResponseWriter, r *http.Request) ([]*entities.ListPodsReport Namespace: pod.Namespace(), Status: status, InfraId: infraId, + Labels: pod.Labels(), } for _, ctr := range ctrs { state, err := ctr.State() diff --git a/pkg/domain/entities/pods.go b/pkg/domain/entities/pods.go index 1a38a7aa4..37acba6e6 100644 --- a/pkg/domain/entities/pods.go +++ b/pkg/domain/entities/pods.go @@ -29,6 +29,7 @@ type ListPodsReport struct { Name string Namespace string Status string + Labels map[string]string } type ListPodContainer struct { diff --git a/pkg/domain/infra/abi/pods.go b/pkg/domain/infra/abi/pods.go index 16c222cbd..320880920 100644 --- a/pkg/domain/infra/abi/pods.go +++ b/pkg/domain/infra/abi/pods.go @@ -360,6 +360,7 @@ func (ic *ContainerEngine) PodPs(ctx context.Context, options entities.PodPSOpti Name: p.Name(), Namespace: p.Namespace(), Status: status, + Labels: p.Labels(), }) } return reports, nil diff --git a/test/e2e/pod_ps_test.go b/test/e2e/pod_ps_test.go index 81d97b72d..8dcaf6af1 100644 --- a/test/e2e/pod_ps_test.go +++ b/test/e2e/pod_ps_test.go @@ -243,4 +243,20 @@ var _ = Describe("Podman ps", func() { infra.WaitWithDefaultTimeout() Expect(len(infra.OutputToString())).To(BeZero()) }) + + It("podman pod ps format with labels", func() { + _, ec, _ := podmanTest.CreatePod("") + Expect(ec).To(Equal(0)) + + _, ec1, _ := podmanTest.CreatePodWithLabels("", map[string]string{ + "io.podman.test.label": "value1", + "io.podman.test.key": "irrelevant-value", + }) + Expect(ec1).To(Equal(0)) + + session := podmanTest.Podman([]string{"pod", "ps", "--format", "{{.Labels}}"}) + session.WaitWithDefaultTimeout() + Expect(session.ExitCode()).To(Equal(0)) + Expect(session.OutputToString()).To(ContainSubstring("value1")) + }) }) |