diff options
| author | Daniel J Walsh <dwalsh@redhat.com> | 2020-07-10 03:29:34 -0400 |
|---|---|---|
| committer | Daniel J Walsh <dwalsh@redhat.com> | 2020-07-10 12:46:16 -0400 |
| commit | 677ad10e0756212bf4fbbed2797d2c110aaa8374 (patch) | |
| tree | 9e7e25d7cbf3427943c047deffff94415f055a33 /test | |
| parent | 2ac8c6953481eb7391a6a7594709811f7ae3167f (diff) | |
| download | podman-677ad10e0756212bf4fbbed2797d2c110aaa8374.tar.gz podman-677ad10e0756212bf4fbbed2797d2c110aaa8374.tar.bz2 podman-677ad10e0756212bf4fbbed2797d2c110aaa8374.zip | |
Pids-limit should only be set if the user set it
Currently we are sending over pids-limits from the user even if they
never modified the defaults. The pids limit should be set at the server
side unless modified by the user.
This issue has led to failures on systems that were running with cgroups V1.
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Diffstat (limited to 'test')
| -rw-r--r-- | test/e2e/run_test.go | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/test/e2e/run_test.go b/test/e2e/run_test.go index 6cbedb457..9d48f1540 100644 --- a/test/e2e/run_test.go +++ b/test/e2e/run_test.go @@ -1072,4 +1072,13 @@ USER mail` Expect(session.OutputToString()).To(ContainSubstring(h)) }) + + It("podman run verify pids-limit", func() { + SkipIfCgroupV1() + limit := "4321" + session := podmanTest.Podman([]string{"run", "--pids-limit", limit, "--rm", ALPINE, "cat", "/sys/fs/cgroup/pids.max"}) + session.WaitWithDefaultTimeout() + Expect(session.ExitCode()).To(Equal(0)) + Expect(session.OutputToString()).To(ContainSubstring(limit)) + }) }) |