diff options
| author | baude <bbaude@redhat.com> | 2018-03-14 15:17:05 -0500 |
|---|---|---|
| committer | Atomic Bot <atomic-devel@projectatomic.io> | 2018-03-14 20:38:08 +0000 |
| commit | ff091cf731cb28e5d38ff843781115d352ea1530 (patch) | |
| tree | 5b14726f448ba8b0a6c30bdaff159d7ec9b18142 /test/e2e/top_test.go | |
| parent | b85b217f55993955da9ad0cae7735747b2f24390 (diff) | |
| download | podman-ff091cf731cb28e5d38ff843781115d352ea1530.tar.gz podman-ff091cf731cb28e5d38ff843781115d352ea1530.tar.bz2 podman-ff091cf731cb28e5d38ff843781115d352ea1530.zip | |
sleep does not catch SIGTERM
As Matt pointed out, when running sleep in a container, the clean up was taking a
full ten seconds to stop container because sleep does not catch SIGTERM which is
the default podman stop signal and it had to wait for SIGKILL. Changing sleep to
top should result in better test times.
Signed-off-by: baude <bbaude@redhat.com>
Closes: #492
Approved by: rhatdan
Diffstat (limited to 'test/e2e/top_test.go')
| -rw-r--r-- | test/e2e/top_test.go | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/test/e2e/top_test.go b/test/e2e/top_test.go index 410803353..4b4cb47e4 100644 --- a/test/e2e/top_test.go +++ b/test/e2e/top_test.go @@ -71,10 +71,10 @@ var _ = Describe("Podman top", func() { }) It("podman top on container invalid options", func() { - sleep := podmanTest.RunSleepContainer("") - sleep.WaitWithDefaultTimeout() - Expect(sleep.ExitCode()).To(Equal(0)) - cid := sleep.OutputToString() + top := podmanTest.RunTopContainer("") + top.WaitWithDefaultTimeout() + Expect(top.ExitCode()).To(Equal(0)) + cid := top.OutputToString() result := podmanTest.Podman([]string{"top", cid, "-o time"}) result.WaitWithDefaultTimeout() |