From: Patrick Donnelly <pdonnell@redhat.com>
Date: Fri, 3 May 2024 00:49:22 +0000 (-0400)
Subject: mon/MDSMonitor: add note about missing metadata inclusion
X-Git-Tag: testing/wip-pdonnell-testing-20240508.183908-debug~14^2~6
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=6517b704e311fd24dbf5bfbdec2ddd23b17d4092;p=ceph-ci.git

mon/MDSMonitor: add note about missing metadata inclusion

There is a "client_count" metadata on the health warning that apparently was
intended to be used for aggregating warnings but never was. Add a TODO item for
that.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
---

diff --git a/src/mon/MDSMonitor.cc b/src/mon/MDSMonitor.cc
index ab1257566eb..357f944df21 100644
--- a/src/mon/MDSMonitor.cc
+++ b/src/mon/MDSMonitor.cc
@@ -313,6 +313,7 @@ void MDSMonitor::encode_pending(MonitorDBStore::TransactionRef t)
   }
   pending.get_health_checks(&new_checks);
   for (auto& p : new_checks.checks) {
+    // TODO: handle "client_count" metadata when summarizing
     p.second.summary = std::regex_replace(
       p.second.summary,
       std::regex("%num%"),