From d3ead6e7a1b4763366e950abb423cdd6cca213e4 Mon Sep 17 00:00:00 2001 From: Indira Sawant Date: Wed, 21 Jan 2026 11:46:31 -0600 Subject: [PATCH] mgr/volumes: reduce noisy health check logs Previously, the manager logged connection cleanup messages at info level even when no idle connections existed, adding unnecessary noise to logs. This change logs cleanup actions at info level only when idle connections are found, and moves the 'no idle connections' message to debug level. Fixes: https://tracker.ceph.com/issues/73635 Signed-off-by: Indira Sawant --- src/pybind/mgr/mgr_util.py | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/pybind/mgr/mgr_util.py b/src/pybind/mgr/mgr_util.py index 246476f8ed26..5d12d0722142 100644 --- a/src/pybind/mgr/mgr_util.py +++ b/src/pybind/mgr/mgr_util.py @@ -296,9 +296,13 @@ class CephfsConnectionPool(object): for connection in connections: if connection.is_connection_idle(CephfsConnectionPool.CONNECTION_IDLE_INTERVAL): idle_conns.append((fs_name, connection)) - for idle_conn in idle_conns: - logger.info(f'cleaning up connection: {idle_conn}') - self._del_connection(idle_conn[0], idle_conn[1]) + # Log only if there are idle connections to clean up + if len(idle_conns) > 0: + logger.debug(f'cleaning up connections: {idle_conns}') + for idle_conn in idle_conns: + self._del_connection(idle_conn[0], idle_conn[1]) + else: + logger.debug("No idle connections to clean up.") def get_fs_handle(self, fs_name: str) -> "cephfs.LibCephFS": with self.lock: -- 2.47.3