From: John Spray Date: Wed, 13 Sep 2017 11:07:50 +0000 (-0400) Subject: osd: upgrade a bunch of perf counters to PRIO_USEFUL X-Git-Tag: v12.2.2~61^2~56 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=8b9a18d3a0c40cb0ef10e32018ea1b23ff53f51e;p=ceph.git osd: upgrade a bunch of perf counters to PRIO_USEFUL These are broadly the OSD-wide IO stats, which happen to also be the ones that were named in the prometheus plugin until I changed it to be priority-based. Signed-off-by: John Spray (cherry picked from commit a1cc4ba2993de62b60fd1e58a9704877a6da5fe4) --- diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc index c586afccefda..c2f563107a76 100644 --- a/src/osd/OSD.cc +++ b/src/osd/OSD.cc @@ -2964,6 +2964,9 @@ void OSD::create_logger() }; + // All the basic OSD operation stats are to be considered useful + osd_plb.set_prio_default(PerfCountersBuilder::PRIO_USEFUL); + osd_plb.add_u64( l_osd_op_wip, "op_wip", "Replication operations currently being processed (primary)"); @@ -3051,6 +3054,10 @@ void OSD::create_logger() l_osd_op_rw_prepare_lat, "op_rw_prepare_latency", "Latency of read-modify-write operations (excluding queue time and wait for finished)"); + // Now we move on to some more obscure stats, revert to assuming things + // are low priority unless otherwise specified. + osd_plb.set_prio_default(PerfCountersBuilder::PRIO_DEBUGONLY); + osd_plb.add_time_avg(l_osd_op_before_queue_op_lat, "op_before_queue_op_lat", "Latency of IO before calling queue(before really queue into ShardedOpWq)"); // client io before queue op_wq latency osd_plb.add_time_avg(l_osd_op_before_dequeue_op_lat, "op_before_dequeue_op_lat",