]> git.apps.os.sepia.ceph.com Git - ceph-ci.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 Feb 2023 22:09:38 +0000 (17:09 -0500)
commit84c988dcfd1db20e2eca478e9bf89899b2029c5c
tree9d4a11bf6d195e0f342a0933362d04b0d1b303e0
parenta9cbbba3388d0974456074429c7172cc26b85d74
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>
src/cephadm/cephadm.py