diff options
author | Giuseppe Scrivano <gscrivan@redhat.com> | 2022-05-10 13:23:04 +0200 |
---|---|---|
committer | Giuseppe Scrivano <gscrivan@redhat.com> | 2022-05-10 20:11:16 +0200 |
commit | d0ca90b3ed3ed543372158633d89811604e7797c (patch) | |
tree | d1f0c47ad5c6374f4f4c07c4ed32faa5f137a79e | |
parent | 28588235d2051bf673d528c293261f926fad0d92 (diff) | |
download | podman-d0ca90b3ed3ed543372158633d89811604e7797c.tar.gz podman-d0ca90b3ed3ed543372158633d89811604e7797c.tar.bz2 podman-d0ca90b3ed3ed543372158633d89811604e7797c.zip |
test: simplify cleanup code
do not try to first stop and then rm but combine the two operations in
a single command.
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
-rw-r--r-- | test/e2e/common_test.go | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/test/e2e/common_test.go b/test/e2e/common_test.go index 28991af7f..db194b777 100644 --- a/test/e2e/common_test.go +++ b/test/e2e/common_test.go @@ -516,17 +516,13 @@ func (p *PodmanTestIntegration) PodmanPID(args []string) (*PodmanSessionIntegrat // Cleanup cleans up the temporary store func (p *PodmanTestIntegration) Cleanup() { - // Remove all containers - stopall := p.Podman([]string{"stop", "-a", "--time", "0"}) - stopall.WaitWithDefaultTimeout() - - podstop := p.Podman([]string{"pod", "stop", "-a", "-t", "0"}) - podstop.WaitWithDefaultTimeout() - podrm := p.Podman([]string{"pod", "rm", "-fa"}) + // Remove all pods... + podrm := p.Podman([]string{"pod", "rm", "-fa", "-t", "0"}) podrm.WaitWithDefaultTimeout() - session := p.Podman([]string{"rm", "-fa"}) - session.WaitWithDefaultTimeout() + // ...and containers + rmall := p.Podman([]string{"rm", "-fa", "-t", "0"}) + rmall.WaitWithDefaultTimeout() p.StopRemoteService() // Nuke tempdir |