From: Ronen Friedman <rfriedma@redhat.com>
Date: Wed, 28 Sep 2022 13:52:52 +0000 (+0000)
Subject: pacific: osd/scrub: use the actual active set when requesting replicas to scrub
X-Git-Tag: v16.2.11~95^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=8f330ae6aba9a7b05346b155b19371f5a95caf8a;p=ceph.git

pacific: osd/scrub: use the actual active set when requesting replicas to scrub

Note that get_acting_set() may differ from
get_acting_recovery_backfill() - if we are remapped

Fixes: https://tracker.ceph.com/issues/57698
Signed-off-by: Ronen Friedman <rfriedma@redhat.com>

(cherry picked from commit 6b57a469e36ae150f8ac4973b9b72f37cedee286)
---

diff --git a/src/osd/pg_scrubber.cc b/src/osd/pg_scrubber.cc
index a686d8a4c4a..c7a286c2d20 100644
--- a/src/osd/pg_scrubber.cc
+++ b/src/osd/pg_scrubber.cc
@@ -786,7 +786,7 @@ void PgScrubber::get_replicas_maps(bool replica_can_preempt)
   m_primary_scrubmap_pos.reset();
 
   // ask replicas to scan and send maps
-  for (const auto& i : m_pg->get_acting_recovery_backfill()) {
+  for (const auto& i : m_pg->get_actingset()) {
 
     if (i == m_pg_whoami)
       continue;
@@ -1293,7 +1293,7 @@ void PgScrubber::scrub_compare_maps()
   map<pg_shard_t, ScrubMap*> maps;
   maps[m_pg_whoami] = &m_primary_scrubmap;
 
-  for (const auto& i : m_pg->get_acting_recovery_backfill()) {
+  for (const auto& i : m_pg->get_actingset()) {
     if (i == m_pg_whoami)
       continue;
     dout(2) << __func__ << " replica " << i << " has "
@@ -1317,7 +1317,7 @@ void PgScrubber::scrub_compare_maps()
     m_osds->clog->warn(ss);
   }
 
-  if (m_pg->recovery_state.get_acting_recovery_backfill().size() > 1) {
+  if (m_pg->recovery_state.get_actingset().size() > 1) {
 
     dout(10) << __func__ << "  comparing replica scrub maps" << dendl;