From: David Zafman <dzafman@redhat.com>
Date: Wed, 25 Jul 2018 19:10:57 +0000 (-0700)
Subject: osd: Improve logic in be_compare_scrubmaps()
X-Git-Tag: v13.2.2~117^2~7
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=30bbc95f07b66f60a0925a4fae0dc6e08b0dbda1;p=ceph.git

osd: Improve logic in be_compare_scrubmaps()

Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit 135d5d0209fd6441a470fd2301c6c0e2ec803091)
---

diff --git a/src/osd/PGBackend.cc b/src/osd/PGBackend.cc
index 432faa580842..8a96348880b4 100644
--- a/src/osd/PGBackend.cc
+++ b/src/osd/PGBackend.cc
@@ -1001,11 +1001,12 @@ void PGBackend::be_compare_scrubmaps(
 	  if (found)
 	    errorstream << pgid << " shard " << j->first << ": soid " << *k
 		      << " " << ss.str() << "\n";
-	} else if (found) {
+	} else if (object_error.errors != 0) {
 	  // Track possible shard to use as authoritative, if needed
 	  // There are errors, without identifying the shard
 	  object_errors.insert(j->first);
-	  errorstream << pgid << " : soid " << *k << " " << ss.str() << "\n";
+	  if (found)
+	    errorstream << pgid << " : soid " << *k << " " << ss.str() << "\n";
 	} else {
 	  // XXX: The auth shard might get here that we don't know
 	  // that it has the "correct" data.