From bf158dc2d9d44caad13265bd37d3a89a936516e1 Mon Sep 17 00:00:00 2001 From: "Joshua M. Boniface" Date: Thu, 7 Dec 2023 11:23:25 -0500 Subject: [PATCH] Shorten debug output --- health-daemon/plugins/psql | 5 ++--- health-daemon/pvchealthd/objects/MonitoringInstance.py | 7 +------ 2 files changed, 3 insertions(+), 9 deletions(-) diff --git a/health-daemon/plugins/psql b/health-daemon/plugins/psql index 909ccce2..3daa11de 100644 --- a/health-daemon/plugins/psql +++ b/health-daemon/plugins/psql @@ -117,9 +117,8 @@ class MonitoringPluginScript(MonitoringPlugin): # Handle some exceptional cases if health_delta > 0: if coordinator_state in ["takeover", "relinquish"]: - # This scenario occurrs if this plugin run catches a node transitioning from primary to secondary coordinator - - # We can ignore it. + # This scenario occurrs if this plugin run catches a node transitioning from primary to + # secondary coordinator. We can ignore it. health_delta = 0 message = "Patroni PostgreSQL error reported but currently transitioning coordinator state; ignoring." diff --git a/health-daemon/pvchealthd/objects/MonitoringInstance.py b/health-daemon/pvchealthd/objects/MonitoringInstance.py index 61d99479..0a6832e2 100644 --- a/health-daemon/pvchealthd/objects/MonitoringInstance.py +++ b/health-daemon/pvchealthd/objects/MonitoringInstance.py @@ -571,14 +571,9 @@ class MonitoringInstance(object): if self.config["debug"]: self.logger.out( - f"Entries for fault check {fault_type}:", + f"Entries for fault check {fault_type}: {dumps(entries)}", state="d", ) - for line in dumps(entries, indent=2).split("\n"): - self.logger.out( - line, - state="d", - ) for _entry in entries: entry = _entry["entry"]