diff options
author | baude <bbaude@redhat.com> | 2019-03-17 10:09:10 -0500 |
---|---|---|
committer | baude <bbaude@redhat.com> | 2019-03-18 08:58:48 -0500 |
commit | 46f18764f205ead97fb5da70104951cf90bf5b6b (patch) | |
tree | 4f6512ae594ec9b475af29716afb9916428a7542 /test/e2e/logs_test.go | |
parent | ea54a1c2f51d3173649277939738ce9b1c392076 (diff) | |
download | podman-46f18764f205ead97fb5da70104951cf90bf5b6b.tar.gz podman-46f18764f205ead97fb5da70104951cf90bf5b6b.tar.bz2 podman-46f18764f205ead97fb5da70104951cf90bf5b6b.zip |
podman logs on created container should exit
when running podman logs on a created container (which has no logs),
podman should return gracefully (like docker) with a 0 return code. if
multiple containers are provided and one is only in the created state
(and no follow is used), we still display the logs for the other ids.
fixes issue #2677
Signed-off-by: baude <bbaude@redhat.com>
Diffstat (limited to 'test/e2e/logs_test.go')
-rw-r--r-- | test/e2e/logs_test.go | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/test/e2e/logs_test.go b/test/e2e/logs_test.go index d383a83b3..d051e3dba 100644 --- a/test/e2e/logs_test.go +++ b/test/e2e/logs_test.go @@ -132,4 +132,14 @@ var _ = Describe("Podman logs", func() { Expect(len(output)).To(Equal(6)) Expect(strings.Contains(output[0], cid1[:12]) || strings.Contains(output[0], cid2[:12])).To(BeTrue()) }) + + It("podman logs on a created container should result in 0 exit code", func() { + session := podmanTest.Podman([]string{"create", "-dt", "--name", "log", ALPINE}) + session.WaitWithDefaultTimeout() + Expect(session.ExitCode()).To(BeZero()) + + results := podmanTest.Podman([]string{"logs", "log"}) + results.WaitWithDefaultTimeout() + Expect(results.ExitCode()).To(BeZero()) + }) }) |