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 <indira.sawant@ibm.com>

diff --git a/src/pybind/mgr/mgr_util.py b/src/pybind/mgr/mgr_util.py
index 246476f8ed26f31cf273d69f05fb6d51c426eb66..5d12d0722142e9a8837b10bf3bb545beeb0cf599 100644 (file)
--- 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: