MDSMonitor: fix mds health printed in bad format

Fixes: http://tracker.ceph.com/issues/23582
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>

diff --git a/src/mon/MDSMonitor.cc b/src/mon/MDSMonitor.cc
index 14c5e10511825716d609cecce49e30649e466d72..b436015461243b08aba807151ade155a1d694032 100644 (file)
--- a/src/mon/MDSMonitor.cc
+++ b/src/mon/MDSMonitor.cc
@@ -220,13 +220,15 @@ void MDSMonitor::encode_pending(MonitorDBStore::TransactionRef t)
        mds_metric_summary(metric.type));
       ostringstream ss;
       ss << "mds" << info.name << "(mds." << rank << "): " << metric.message;
-      for (auto p = metric.metadata.begin();
-          p != metric.metadata.end();
-          ++p) {
-       if (p != metric.metadata.begin()) {
+      bool first = true;
+      for (auto &p : metric.metadata) {
+       if (first) {
+         ss << " ";
+       } else {
          ss << ", ";
-       }
-       ss << p->first << ": " << p->second;
+        }
+       ss << p.first << ": " << p.second;
+        first = false;
       }
       check->detail.push_back(ss.str());
     }