From: Laura Flores <lflores@redhat.com>
Date: Wed, 14 Sep 2022 20:41:39 +0000 (-0500)
Subject: Revert "osd/PeeringState: fix missed `recheck_readable` from laggy"
X-Git-Tag: v17.2.6~505^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=cccef47a6ccb7c31d9efcf76723dcf56a740dbe4;p=ceph.git

Revert "osd/PeeringState: fix missed `recheck_readable` from laggy"

This reverts commit c71ee14cdbcc499d0ccaa84f4714ad5f29821f40.

This commit caused a regression in the rados suite, as evidenced by:

- with the commit:
http://pulpito.front.sepia.ceph.com/lflores-2022-09-14_15:11:39-rados-quincy-release-distro-default-smithi/

- with the commit reverted:
http://pulpito.front.sepia.ceph.com/lflores-2022-09-14_17:02:02-rados-wip-lflores-testing-quincy-release-distro-default-smithi/

Fixes: https://tracker.ceph.com/issues/57546
Signed-off-by: Laura Flores <lflores@redhat.com>
---

diff --git a/src/osd/PeeringState.cc b/src/osd/PeeringState.cc
index 8ad0a08e9b4c..68b8d2225995 100644
--- a/src/osd/PeeringState.cc
+++ b/src/osd/PeeringState.cc
@@ -1220,7 +1220,7 @@ void PeeringState::proc_lease_ack(int from, const pg_lease_ack_t& a)
   if (was_min) {
     auto old_ru = readable_until;
     recalc_readable_until();
-    if (now >= old_ru) {
+    if (now < old_ru) {
       pl->recheck_readable();
     }
   }