From: Yuri Weinstein <yweinste@redhat.com>
Date: Thu, 7 Apr 2022 21:26:58 +0000 (-0700)
Subject: Merge pull request #45785 from ronen-fr/wip-rf-45640-pacific
X-Git-Tag: v16.2.8~29
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=adbb2295c126c0099b01db0e1406ad226f6dbc44;p=ceph.git

Merge pull request #45785 from ronen-fr/wip-rf-45640-pacific

pacific: osd/scrub: restart snap trimming only after scrubbing is done

Reviewed-by: Neha Ojha <nojha@redhat.com>
---

adbb2295c126c0099b01db0e1406ad226f6dbc44
diff --cc src/osd/pg_scrubber.cc
index f733f8ae8b2,bca92a5f46f..59372e552fc
--- a/src/osd/pg_scrubber.cc
+++ b/src/osd/pg_scrubber.cc
@@@ -176,8 -176,12 +176,11 @@@ void PgScrubber::initiate_regular_scrub
    if (check_interval(epoch_queued)) {
      dout(10) << "scrubber event -->> StartScrub epoch: " << epoch_queued << dendl;
      reset_epoch(epoch_queued);
 -    m_fsm->my_states();
      m_fsm->process_event(StartScrub{});
      dout(10) << "scrubber event --<< StartScrub" << dendl;
+   } else {
+     // and just in case snap trimming was blocked by the aborted scrub
+     m_pg->snap_trimmer_scrub_complete();
    }
  }
  
@@@ -188,11 -192,13 +191,12 @@@ void PgScrubber::initiate_scrub_after_r
    if (check_interval(epoch_queued)) {
      dout(10) << "scrubber event -->> AfterRepairScrub epoch: " << epoch_queued << dendl;
      reset_epoch(epoch_queued);
 -    m_fsm->my_states();
      m_fsm->process_event(AfterRepairScrub{});
      dout(10) << "scrubber event --<< AfterRepairScrub" << dendl;
+   } else {
+     m_pg->snap_trimmer_scrub_complete();
    }
  }
- 
  void PgScrubber::send_scrub_unblock(epoch_t epoch_queued)
  {
    dout(10) << "scrubber event -->> " << __func__ << " epoch: " << epoch_queued << dendl;