]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
osd/PeeringState: separate history's pruub from pg's 44584/head
authorSage Weil <sage@newdream.net>
Thu, 18 Nov 2021 20:46:06 +0000 (15:46 -0500)
committerVikhyat Umrao <vikhyat@redhat.com>
Fri, 14 Jan 2022 00:08:28 +0000 (16:08 -0800)
commit8eb78b4f0ec65992a7262565baf8583bacd20081
tree0aa92c6f88b9e6709edce844adb2dc1ed4626f30
parent5808c7532c6b9cc8020c194464a2848f78ba60e2
osd/PeeringState: separate history's pruub from pg's

(pruub = prior_readable_until_ub [upper bound])

During peering, a primary may conclude that it does not need to wait for
the prior interval(s)' read lease because it will query all such osds.
However, it is dangerous to reflect that local inference about future
peering effects in the info.history, which is freely shared with other
OSDs.  For example, if the primary cleared the history pruub, shared it,
and then failed, the next primary may conclude that it does not need to
wait for the lease to expire.

Instead, track the pruub in the conventional way.  Only at the end of
peering do we clear it (if there are no prior_interval_down_osds) before
ending our final activate infos out, when we have already talked to the
peer osds and they know the prior interval has finished.

Fixes: https://tracker.ceph.com/issues/53326
Signed-off-by: Sage Weil <sage@newdream.net>
(cherry picked from commit 96d6bf22a5dbf14da1de0ee6128585b9dd9d60d8)
src/osd/PeeringState.cc
src/osd/PeeringState.h