From 9bd833bcfd07bf2b3c258a617d88255327b91669 Mon Sep 17 00:00:00 2001 From: Paul Holzinger Date: Wed, 12 Apr 2023 21:24:28 +0200 Subject: [PATCH] test/e2e: fix "podman run ipcns ipcmk container test" The test will leak processes because the rm -fa in the cleanup failed. This happens because podman tried to remove the contianers in the wrong order and thus ppodman failed with: `contianer XXX has dependent containers which must be removed before it` For now I patch the test but it should be much better if we can fix it in podman to remove in the correct order. `--all` should mean all I do not care if there is a dependent container, just get rid of it. Signed-off-by: Paul Holzinger --- test/e2e/run_ns_test.go | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/test/e2e/run_ns_test.go b/test/e2e/run_ns_test.go index 4431b3284e..9eac8801b0 100644 --- a/test/e2e/run_ns_test.go +++ b/test/e2e/run_ns_test.go @@ -85,7 +85,7 @@ var _ = Describe("Podman run ns", func() { }) It("podman run ipcns ipcmk container test", func() { - setup := podmanTest.Podman([]string{"run", "-d", "--name", "test1", fedoraMinimal, "sleep", "999"}) + setup := podmanTest.Podman([]string{"run", "-d", "--name", "test1", fedoraMinimal, "sleep", "30"}) setup.WaitWithDefaultTimeout() Expect(setup).Should(Exit(0)) @@ -94,7 +94,12 @@ var _ = Describe("Podman run ns", func() { Expect(session).Should(Exit(0)) output := strings.Split(session.OutputToString(), " ") ipc := output[len(output)-1] - session = podmanTest.Podman([]string{"run", "--ipc=container:test1", fedoraMinimal, "ipcs", "-m", "-i", ipc}) + session = podmanTest.Podman([]string{"run", "--name=t2", "--ipc=container:test1", fedoraMinimal, "ipcs", "-m", "-i", ipc}) + session.WaitWithDefaultTimeout() + Expect(session).Should(Exit(0)) + + // We have to remove the dependency container first, rm --all fails in the cleanup because of the unknown ordering. + session = podmanTest.Podman([]string{"rm", "t2"}) session.WaitWithDefaultTimeout() Expect(session).Should(Exit(0)) })