fix review issues, add linter fixes

Author: kiryazovi-redis

pyproject.toml

@@ -80,11 +80,11 @@ filterwarnings = [
     # Ignore a coverage warning when COVERAGE_CORE=sysmon for Pythons < 3.12.
     "ignore:sys.monitoring isn't available:coverage.exceptions.CoverageWarning",
 ]
-log_cli_level = "DEBUG"
+log_cli_level = "INFO"
 log_cli_date_format = "%H:%M:%S:%f"
-log_cli = true
+log_cli = false
 log_cli_format = "%(asctime)s %(levelname)s %(threadName)s: %(message)s"
-log_level = "DEBUG"
+log_level = "INFO"
 capture = "yes"
 
 [tool.ruff]

tests/test_scenario/maint_notifications_helpers.py

@@ -168,9 +168,13 @@ def find_target_node_and_empty_node(
 
         logging.debug(f"All nodes: {all_nodes}")
         logging.debug(f"Nodes with shards from any database: {nodes_with_any_shards}")
-        logging.debug(f"Nodes with target database shards: {nodes_with_target_db_shards}")
+        logging.debug(
+            f"Nodes with target database shards: {nodes_with_target_db_shards}"
+        )
         logging.debug(f"Master nodes (target database only): {master_nodes}")
-        logging.debug(f"Nodes with no shards from target database: {nodes_with_no_shards_target_bdb}")
+        logging.debug(
+            f"Nodes with no shards from target database: {nodes_with_no_shards_target_bdb}"
+        )
 
         if not nodes_with_no_shards_target_bdb:
             raise ValueError("All nodes have shards from target database")
@@ -179,7 +183,9 @@ def find_target_node_and_empty_node(
             raise ValueError("No nodes with master shards from target database found")
 
         # Return the first available empty node and master node (numeric part only)
-        empty_node = next(iter(nodes_with_no_shards_target_bdb)).split(":")[1]  # node:1 -> 1
+        empty_node = next(iter(nodes_with_no_shards_target_bdb)).split(":")[
+            1
+        ]  # node:1 -> 1
         target_node = next(iter(master_nodes)).split(":")[1]  # node:2 -> 2
 
         return target_node, empty_node