From: Ronen Friedman <rfriedma@redhat.com>
Date: Fri, 27 Aug 2021 13:37:07 +0000 (+0000)
Subject: osd/scrub: stats_of_handled_objects(): reduce irrelevant logs
X-Git-Tag: v17.1.0~945^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=77335d475148350d5aee10520fba1c5546d5772e;p=ceph.git

osd/scrub: stats_of_handled_objects(): reduce irrelevant logs

PrimaryLogScrub::stats_of_handled_objects() is responsible for a large
share of all scrub-related logs, as it emits a log message even when
called for non-active or not scrubbing PGs.

This PR removes these useless logs.

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
---

diff --git a/src/osd/PrimaryLogScrub.cc b/src/osd/PrimaryLogScrub.cc
index 244be9e861e8..dd5aa84e8bab 100644
--- a/src/osd/PrimaryLogScrub.cc
+++ b/src/osd/PrimaryLogScrub.cc
@@ -574,15 +574,13 @@ void PrimaryLogScrub::stats_of_handled_objects(const object_stat_sum_t& delta_st
   // scrubbed and their stats have already been added to the scrubber. Objects after that
   // point haven't been included in the scrubber's stats accounting yet, so they will be
   // included when the scrubber gets to that object.
-  dout(15) << __func__ << " soid: " << soid << " scrub is active? " << is_scrub_active()
-	   << dendl;
   if (is_primary() && is_scrub_active()) {
     if (soid < m_start) {
       dout(20) << __func__ << " " << soid << " < [" << m_start << "," << m_end << ")"
 	       << dendl;
       m_scrub_cstat.add(delta_stats);
     } else {
-      dout(20) << __func__ << " " << soid << " >= [" << m_start << "," << m_end << ")"
+      dout(25) << __func__ << " " << soid << " >= [" << m_start << "," << m_end << ")"
 	       << dendl;
     }
   }