diff --git a/cmapi/cmapi_server/managers/process.py b/cmapi/cmapi_server/managers/process.py index 2feaec267..9b0d2264d 100644 --- a/cmapi/cmapi_server/managers/process.py +++ b/cmapi/cmapi_server/managers/process.py @@ -471,9 +471,6 @@ class MCSProcessManager: # undefined behaviour when primary gone and then recovers (failover # triggered 2 times). for prog_name in cls._get_sorted_progs(True, reverse=True): - if is_read_only and prog_name == MCSProgs.WRITE_ENGINE_SERVER.value: - logging.debug('Node is in read-only mode, not stopping WriteEngine') - continue if not cls.stop(prog_name, is_primary, use_sudo): logging.error(f'Process "{prog_name}" not stopped properly.') raise CMAPIBasicError(f'Error while stopping "{prog_name}"') diff --git a/cmapi/cmapi_server/node_manipulation.py b/cmapi/cmapi_server/node_manipulation.py index 5e19c1488..975197891 100644 --- a/cmapi/cmapi_server/node_manipulation.py +++ b/cmapi/cmapi_server/node_manipulation.py @@ -1016,6 +1016,9 @@ def _add_read_only_node(root, node) -> None: else: for n in read_only_nodes.findall("./Node"): if n.text == node: + logging.warning( + f"_add_read_only_node(): node {node} already exists in ReadOnlyNodes" + ) return etree.SubElement(read_only_nodes, "Node").text = node diff --git a/cmapi/mcs_cluster_tool/__main__.py b/cmapi/mcs_cluster_tool/__main__.py index 4c15284b6..3b0d9d0bf 100644 --- a/cmapi/mcs_cluster_tool/__main__.py +++ b/cmapi/mcs_cluster_tool/__main__.py @@ -70,7 +70,10 @@ def setup_logging(verbose: bool = False) -> None: add_logging_level('TRACE', 5) dict_config(MCS_CLI_LOG_CONF_PATH) if verbose: - enable_console_logging(logging.getLogger()) + for logger_name in ("", "mcs_cli"): + logger = logging.getLogger(logger_name) + logger.setLevel(logging.DEBUG) + enable_console_logging(logger) if __name__ == '__main__':