diff options
| author | baude <bbaude@redhat.com> | 2018-12-23 11:47:46 -0600 |
|---|---|---|
| committer | baude <bbaude@redhat.com> | 2018-12-23 11:47:46 -0600 |
| commit | df923b26138a449a522cbbf22e4adcc630606a60 (patch) | |
| tree | b00dfa35b4d9ef22ffee8a602b78fd01576f599e | |
| parent | eb982193c5bb949ec75308e91d5698aebe4f9429 (diff) | |
| download | podman-df923b26138a449a522cbbf22e4adcc630606a60.tar.gz podman-df923b26138a449a522cbbf22e4adcc630606a60.tar.bz2 podman-df923b26138a449a522cbbf22e4adcc630606a60.zip | |
rootless tests using stop is more reliable
when testing rootless containers, it is more reliable to stop
a container with a zero timeout than kill a container. We made
this change in non-rootless tests as well. When IO or CPU are
taxed, it avoids a situation where the kill signal is sent but the
container has not been able to update its status when a subsequent
action occurs.
Signed-off-by: baude <bbaude@redhat.com>
| -rw-r--r-- | test/e2e/rootless_test.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/e2e/rootless_test.go b/test/e2e/rootless_test.go index 5a7f0359a..8e9f9fc8d 100644 --- a/test/e2e/rootless_test.go +++ b/test/e2e/rootless_test.go @@ -210,7 +210,7 @@ var _ = Describe("Podman rootless", func() { cmd.WaitWithDefaultTimeout() Expect(cmd.ExitCode()).To(Equal(0)) - cmd = rootlessTest.PodmanAsUser([]string{"kill", "-l"}, 1000, 1000, env) + cmd = rootlessTest.PodmanAsUser([]string{"stop", "-l", "-t", "0"}, 1000, 1000, env) cmd.WaitWithDefaultTimeout() Expect(cmd.ExitCode()).To(Equal(0)) |