opensource/podman
1
0
Fork 0
mirror of https://github.com/containers/podman.git synced 2025-06-20 09:03:43 +08:00
Code Issues Packages Projects Releases Wiki Activity

New ulimit test: bump up minimum nfiles

Browse Source 
...and add a comment explaining why. The minimum, determined via
binary search, is actually 27! Anything under that will barf:

   $ bin/podman run --ulimit nofile=26:26 --rm quay.io/libpod/testimage:20221018 true
   Error: OCI runtime error: crun: openat2 `proc/sysrq-trigger`: Too many open files

Play it safe, go with 30.

(Does this seem alarming to anyone else, or am I the only one??)

Fixes: #17860

Signed-off-by: Ed Santiago <santiago@redhat.com>
This commit is contained in:
Ed Santiago
2023-03-20 13:06:38 -06:00
parent 39b0de19e1
commit 6b2101bc74
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
test/system/030-run.bats
View File

@ -1027,8 +1027,9 @@ EOF
@test "podman run ulimit from containers.conf" { @test "podman run ulimit from containers.conf" {
skip_if_remote "containers.conf has to be set on remote, only tested on E2E test" skip_if_remote "containers.conf has to be set on remote, only tested on E2E test"
containersconf=$PODMAN_TMPDIR/containers.conf containersconf=$PODMAN_TMPDIR/containers.conf
nofile1=$((RANDOM % 10000 + 5)) # Safe minimum: anything under 27 barfs w/ "crun: ... Too many open files"
nofile2=$((RANDOM % 10000 + 5)) nofile1=$((30 + RANDOM % 10000))
nofile2=$((30 + RANDOM % 10000))
cat >$containersconf <<EOF cat >$containersconf <<EOF
[containers] [containers]
default_ulimits = [ default_ulimits = [