From: Michal Nasiadka <mnasiadka@gmail.com>
Date: Mon, 27 Feb 2023 08:15:50 +0000 (+0100)
Subject: cephadm: set --ulimit nofiles with Docker
X-Git-Tag: v17.2.7~482^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F50890%2Fhead;p=ceph.git

cephadm: set --ulimit nofiles with Docker

cephadm is setting LimitNOFILE in systemd units, but it doesn't
get picked up by Docker.

Fixes: https://tracker.ceph.com/issues/58855
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
(cherry picked from commit ce169004f749f088c4f05505b6457e7a190db680)

Conflicts:
	src/cephadm/tests/test_cephadm.py
---

diff --git a/src/cephadm/cephadm b/src/cephadm/cephadm
index dbe2e639b661..8595f72153c0 100755
--- a/src/cephadm/cephadm
+++ b/src/cephadm/cephadm
@@ -4018,6 +4018,9 @@ class CephContainer:
             if os.path.exists('/etc/ceph/podman-auth.json'):
                 cmd_args.append('--authfile=/etc/ceph/podman-auth.json')
 
+        if isinstance(self.ctx.container_engine, Docker):
+            cmd_args.extend(['--ulimit', 'nofile=1048576'])
+
         envs: List[str] = [
             '-e', 'CONTAINER_IMAGE=%s' % self.image,
             '-e', 'NODE_NAME=%s' % get_hostname(),