From 373245d5a13f4e05642a78177da162ab5c8f5831 Mon Sep 17 00:00:00 2001 From: vitor-aignosi Date: Fri, 7 Nov 2025 09:56:48 -0300 Subject: [PATCH] SIENTIAPDE-1325 Refactor OpcManager to emit connection metrics immediately after connection attempt - Moved the emission of OPC_CONNECTIONS_TOTAL metric to occur right after the connection attempt, ensuring accurate tracking of connection status. - Removed redundant metric emission logic that was previously placed after a successful connection. --- ingestor/managers/opc_manager.py | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/ingestor/managers/opc_manager.py b/ingestor/managers/opc_manager.py index 451b719..049b6f8 100644 --- a/ingestor/managers/opc_manager.py +++ b/ingestor/managers/opc_manager.py @@ -197,6 +197,13 @@ class OpcManager(SientiaMonitoring): - OPC_CONNECTION_STATUS: Set to 1 on successful connection """ + await self.emit_metric( + metric_object=metrics.OPC_CONNECTIONS_TOTAL, + tags={ + 'pod_id': self.pod_id, + 'server_name': self.name, + } + ) try: self.client = Client(self.url, timeout=10, watchdog_intervall=3600000) assert self.client is not None # Informa ao mypy que client não é None @@ -221,13 +228,6 @@ class OpcManager(SientiaMonitoring): 'server_url': self.url, }, ) - await self.emit_metric( - metric_object=metrics.OPC_CONNECTIONS_TOTAL, - tags={ - 'pod_id': self.pod_id, - 'server_name': self.name, - } - ) self.logger.info(f'Connection to {self.name} successful.') except Exception: await self.disconnect()