From 81af24d9197a3690c17e001ffb00eb16b98b8f2e Mon Sep 17 00:00:00 2001
From: Kamoltat Sirivadhna <ksirivad@redhat.com>
Date: Mon, 14 Jul 2025 18:46:02 +0000
Subject: [PATCH] src/mon/ConnectionTracker.cc: && HealthMonitor.cc Improve
 logging

Improve logging for Netsplit pairs and
other variables in HealthMonitor::check_netsplit

Fixes: https://tracker.ceph.com/issues/71344

Signed-off-by: Kamoltat Sirivadhna <ksirivad@redhat.com>
---
 src/mon/ConnectionTracker.cc |  9 ++++++---
 src/mon/HealthMonitor.cc     | 32 ++++++++++++++++++++------------
 2 files changed, 26 insertions(+), 15 deletions(-)

diff --git a/src/mon/ConnectionTracker.cc b/src/mon/ConnectionTracker.cc
index 513d823011928..5ef3903dbb1a1 100644
--- a/src/mon/ConnectionTracker.cc
+++ b/src/mon/ConnectionTracker.cc
@@ -425,11 +425,14 @@ std::set<std::pair<unsigned, unsigned>> ConnectionTracker::get_netsplit(
     }
     // For debugging purposes:
     if (cct->_conf->subsys.should_gather(ceph_subsys_mon, 30)) {
-      ldout(cct, 30) << "Netsplit pairs: " << dendl;
+      ldout(cct, 30) << "Netsplit pairs: {";
+      bool first = true;
       for (const auto& nsp_pair : nsp_pairs) {
-        ldout(cct, 30) << "(" << nsp_pair.first << ", "
-          << nsp_pair.second << ") " << dendl;
+        if (!first) *_dout << ", ";
+        first = false;
+        *_dout << "(" << nsp_pair.first << ", " << nsp_pair.second << ")";
       }
+      *_dout << "}" << dendl;
     }
     return nsp_pairs;
 }
diff --git a/src/mon/HealthMonitor.cc b/src/mon/HealthMonitor.cc
index 007b2e7a6f231..85b199958fba1 100644
--- a/src/mon/HealthMonitor.cc
+++ b/src/mon/HealthMonitor.cc
@@ -1203,30 +1203,38 @@ void HealthMonitor::check_netsplit(health_check_map_t *checks, std::set<std::str
     }
     *_dout << "}" << dendl;
 
-    dout(30) << "mon_loc_map: " << dendl;
+    dout(30) << "mon_loc_map: { ";
+    bool outer_first = true;
     for (const auto& mon_pair : mon_loc_map) {
-      dout(30) << mon_pair.first << ": {";
-      bool first = true;
+      if (!outer_first) *_dout << ", ";
+      outer_first = false;
+      *_dout << mon_pair.first << ": {";
+      bool inner_first = true;
       for (const auto& loc_pair : mon_pair.second) {
-        if (!first) *_dout << ", ";
-        first = false;
+        if (!inner_first) *_dout << ", ";
+        inner_first = false;
         *_dout << loc_pair.first << ": " << loc_pair.second;
       }
-      *_dout << "}" << dendl;
+      *_dout << "}";
     }
+    *_dout << " }" << dendl;
 
 
-    dout(30) << "location_to_mons: " << dendl;
+    dout(30) << "location_to_mons: {";
+    bool outer_first = true;
     for (const auto& loc_pair : location_to_mons) {
-      dout(30) << loc_pair.first << ": {";
-      bool first = true;
+      if (!outer_first) *_dout << ", ";
+      outer_first = false;
+      *_dout << loc_pair.first << ": {";
+      bool inner_first = true;
       for (const auto& monitor : loc_pair.second) {
-        if (!first) *_dout << ", ";
-        first = false;
+        if (!inner_first) *_dout << ", ";
+        inner_first = false;
         *_dout << monitor;
       }
-      *_dout << "}" << dendl;
+      *_dout << "}";
     }
+    *_dout << " }" << dendl;
 
     dout(30) << "detected_location_netsplits: {";
     bool first = true;
-- 
2.39.5