diff options
| author | Urvashi Mohnani <umohnani@redhat.com> | 2021-09-29 16:22:55 -0400 |
|---|---|---|
| committer | Matthew Heon <matthew.heon@pm.me> | 2021-10-19 13:55:41 -0400 |
| commit | 14509a92bb87bc328c67182b9e985fd81f6f7b53 (patch) | |
| tree | e7176a7ce73949cbf274a82ea4077fbffab02520 /pkg | |
| parent | c15c1540837b25305729bc72d61372098e204151 (diff) | |
| download | podman-14509a92bb87bc328c67182b9e985fd81f6f7b53.tar.gz podman-14509a92bb87bc328c67182b9e985fd81f6f7b53.tar.bz2 podman-14509a92bb87bc328c67182b9e985fd81f6f7b53.zip | |
Allow a value of -1 to set unlimited pids limit
Users can set --pids-limit to -1 now to set unlimited
pids limit for a container - this matches the convention.
[NO TESTS NEEDED]
Signed-off-by: Urvashi Mohnani <umohnani@redhat.com>
Diffstat (limited to 'pkg')
| -rw-r--r-- | pkg/specgen/generate/validate.go | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/pkg/specgen/generate/validate.go b/pkg/specgen/generate/validate.go index 50efe7fa3..b0d84825e 100644 --- a/pkg/specgen/generate/validate.go +++ b/pkg/specgen/generate/validate.go @@ -72,10 +72,9 @@ func verifyContainerResourcesCgroupV1(s *specgen.SpecGenerator) ([]string, error // Pids checks if s.ResourceLimits.Pids != nil { - pids := s.ResourceLimits.Pids // TODO: Should this be 0, or checking that ResourceLimits.Pids // is set at all? - if pids.Limit > 0 && !sysInfo.PidsLimit { + if s.ResourceLimits.Pids.Limit >= 0 && !sysInfo.PidsLimit { warnings = append(warnings, "Your kernel does not support pids limit capabilities or the cgroup is not mounted. PIDs limit discarded.") s.ResourceLimits.Pids = nil } |