]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
cephadm: set pids-limit unlimited for all ceph daemons
authorAdam King <adking@redhat.com>
Sun, 12 Feb 2023 20:28:10 +0000 (15:28 -0500)
committerAdam King <adking@redhat.com>
Mon, 13 Mar 2023 21:36:37 +0000 (17:36 -0400)
commitb2fd8e32acd616868899ba17681f1fea0f833296
tree7abe6d7e60f23c5b9a906cdada327b2de665fbb3
parente3ec2a21a3ebb52b80620068b385374b661f016a
cephadm: set pids-limit unlimited for all ceph daemons

We actually had this setup before, but ran into issues.
Some teuthology test had failed in the fs suite, so it was
modified to only affect iscsi and rgw daemons (https://github.com/ceph/ceph/pull/45798)
and then the changes were reverted entirely (so no pids-limit
modifying code at all) in quincy and pacific because
the LRC ran into issues with the change related to the podman
version (https://github.com/ceph/ceph/pull/45932). This new patch
now addresses the podman versions, specifically that the patch
that makes -1 work for a pids-limit seems to have landed in
podman 3.4.1 based on https://github.com/containers/podman/pull/12040.
We'll need to make sure that this doesn't break anything in the
fs suites again as I don't remember the details of the first
issue, or why having it only set the pids-limit for iscsi and rgw fixes it.
Assuming that isn't a problem we should hopefully be able to unify
at least how reef and quincy handle this now that the podman version
issue is being addressed in this patch.

See the linked tracker issue for a discussion on why we're going at
this again and why I'm trying to do this for all ceph daemon types.

Fixes: https://tracker.ceph.com/issues/58685
Signed-off-by: Adam King <adking@redhat.com>
(cherry picked from commit 84c988dcfd1db20e2eca478e9bf89899b2029c5c)

Conflicts:
src/cephadm/cephadm
src/cephadm/cephadm