mds: change priority of mds rss perf counter to useful

mds_cache_mem_rss perf counter can be a useful metric to detect
the usage of mds cache and detect mds cache oversize before it happens.
The PR changes the priority of the mds_rss metric from Debug to useful

Fixes: https://tracker.ceph.com/issues/63765
Signed-off-by: sp98 <sapillai@redhat.com>
(cherry picked from commit 89b9fa299c0fe80b3e8619dd9dd8cb77e5b08016)

diff --git a/src/mds/MDSRank.cc b/src/mds/MDSRank.cc
index 8d41a4678f88aa249525464b340e67b7880df988..4eb94e4106a9d68f979dae39ab566d6fa66a2835 100644 (file)
--- a/src/mds/MDSRank.cc
+++ b/src/mds/MDSRank.cc
@@ -3472,6 +3472,9 @@ void MDSRank::create_logger()
                     PerfCountersBuilder::PRIO_INTERESTING);
     mdm_plb.add_u64(l_mdm_dn, "dn", "Dentries", "dn",
                     PerfCountersBuilder::PRIO_INTERESTING);
+    // mds rss metric is set to PRIO_USEFUL as it can be useful to detect mds cache oversizing
+    mdm_plb.add_u64(l_mdm_rss, "rss", "RSS", "rss",
+                    PerfCountersBuilder::PRIO_USEFUL);
 
     mdm_plb.set_prio_default(PerfCountersBuilder::PRIO_USEFUL);
     mdm_plb.add_u64_counter(l_mdm_inoa, "ino+", "Inodes opened");
@@ -3486,9 +3489,6 @@ void MDSRank::create_logger()
     mdm_plb.add_u64_counter(l_mdm_caps, "cap-", "Capabilities removed");
     mdm_plb.add_u64(l_mdm_heap, "heap", "Heap size");
 
-    mdm_plb.set_prio_default(PerfCountersBuilder::PRIO_DEBUGONLY);
-    mdm_plb.add_u64(l_mdm_rss, "rss", "RSS");
-
     mlogger = mdm_plb.create_perf_counters();
     g_ceph_context->get_perfcounters_collection()->add(mlogger);
   }