From 2d46c9f03920a4c4e371978518295304d5c48b62 Mon Sep 17 00:00:00 2001 From: Michal Nasiadka Date: Mon, 27 Feb 2023 09:15:50 +0100 Subject: [PATCH] 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 (cherry picked from commit ce169004f749f088c4f05505b6457e7a190db680) Conflicts: src/cephadm/tests/test_cephadm.py --- src/cephadm/cephadm | 3 +++ 1 file changed, 3 insertions(+) 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(), -- 2.47.3