Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

squid: rgw/multisite: metadata polling event based on unmodified mdlog_marker #60792

Open
wants to merge 1 commit into
base: squid
Choose a base branch
from

Conversation

smanjara
Copy link
Contributor

backport tracker: https://tracker.ceph.com/issues/69003


backport of #52568
parent tracker: https://tracker.ceph.com/issues/61743

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

…ck if the mdlog_marker is not modified by comparing

mdlog_marker and max_marker. but max_marker is exposed to changes from RGWReadMDLogEntriesCR, and if there is a race
coming from mdlog trimming which could render max_marker empty, then its comparison with mdlog polling can be incorrect.
To fix this, we now save the previous mdlog marker and compare with the updated mdlog marker.

Signed-off-by: Shilpa Jagannath <[email protected]>
(cherry picked from commit bae10f2)
@smanjara smanjara requested a review from a team as a code owner November 21, 2024 16:51
@smanjara smanjara added this to the squid milestone Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants