Skip to content

Commit 2167731

Browse files
committed
Add logging for resource ID and node status in SCS cluster status checker
1 parent 1a928e2 commit 2167731

1 file changed

Lines changed: 18 additions & 2 deletions

File tree

src/modules/get_cluster_status_scs.py

Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
Python script to get and validate the status of an SCS cluster.
66
"""
77

8+
import logging
89
import xml.etree.ElementTree as ET
910
from typing import Dict, Any
1011
from ansible.module_utils.basic import AnsibleModule
@@ -123,7 +124,6 @@ def _process_node_attributes(self, cluster_status_xml: ET.Element) -> Dict[str,
123124
ascs_resource_id = f"rsc_sap_{self.sap_sid.upper()}_ASCS{self.ascs_instance_number}"
124125
ers_resource_id = f"rsc_sap_{self.sap_sid.upper()}_ERS{self.ers_instance_number}"
125126

126-
all_nodes = [node.attrib.get("name") for node in node_attributes]
127127
for node in node_attributes:
128128
node_name = node.attrib["name"]
129129
for attribute in node:
@@ -134,22 +134,38 @@ def _process_node_attributes(self, cluster_status_xml: ET.Element) -> Dict[str,
134134
self.result["ascs_node"] = node_name
135135

136136
if resources is not None:
137+
self.log(
138+
logging.INFO,
139+
f"ASCS resource ID: {ascs_resource_id}, ERS resource ID: {ers_resource_id}",
140+
)
137141
ascs_resource = resources.find(f"./resource[@id='{ascs_resource_id}']")
138142
ers_resource = resources.find(f"./resource[@id='{ers_resource_id}']")
139-
143+
self.log(
144+
logging.INFO,
145+
f"ASCS resource ID: {ascs_resource}, ERS resource ID: {ers_resource}",
146+
)
140147
if ascs_resource is not None:
141148
is_failed = ascs_resource.attrib.get("failed", "false").lower() == "true"
142149
if not is_failed:
143150
node_element = ascs_resource.find("node")
151+
self.log(
152+
logging.INFO,
153+
f"ASCS node element: {node_element}, ASCS node: {self.result['ascs_node']}",
154+
)
144155
if node_element is not None:
145156
self.result["ascs_node"] = node_element.attrib.get(
146157
"name", self.result["ascs_node"]
147158
)
159+
148160
else:
149161
self.result["ascs_node"] = ""
150162

151163
if ers_resource is not None:
152164
is_failed = ers_resource.attrib.get("failed", "false").lower() == "true"
165+
self.log(
166+
logging.INFO,
167+
f"ERS resource ID: {ers_resource_id}, ERS resource failed: {is_failed}",
168+
)
153169
if not is_failed:
154170
node_element = ers_resource.find("node")
155171
if node_element is not None:

0 commit comments

Comments
 (0)