Skip to content

reef: rgw/multisite: metadata polling event based on unmodified mdlog_marker#60793

Merged
smanjara merged 1 commit intoceph:reeffrom
smanjara:wip-69004-reef
Jan 28, 2025
Merged

reef: rgw/multisite: metadata polling event based on unmodified mdlog_marker#60793
smanjara merged 1 commit intoceph:reeffrom
smanjara:wip-69004-reef

Conversation

@smanjara
Copy link
Copy Markdown
Contributor

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


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

@smanjara smanjara requested a review from a team as a code owner November 21, 2024 17:12
@smanjara smanjara added this to the reef milestone Nov 21, 2024
@smanjara
Copy link
Copy Markdown
Contributor Author

smanjara commented Jan 2, 2025

jenkins test api

@smanjara
Copy link
Copy Markdown
Contributor Author

smanjara commented Jan 2, 2025

jenkins retest

@smanjara smanjara requested a review from adamemerson January 2, 2025 20:35
@smanjara
Copy link
Copy Markdown
Contributor Author

smanjara commented Jan 3, 2025

jenkins test docs

@smanjara
Copy link
Copy Markdown
Contributor Author

smanjara commented Jan 6, 2025

jenkins retest

…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 <smanjara@redhat.com>
(cherry picked from commit bae10f2)

Signed-off-by: Shilpa Jagannath <smanjara@redhat.com>
@adamemerson adamemerson modified the milestones: reef, squid, v18.2.5 Jan 8, 2025
@adamemerson
Copy link
Copy Markdown
Contributor

make check

@smanjara
Copy link
Copy Markdown
Contributor Author

smanjara commented Jan 9, 2025

jenkins make check

1 similar comment
@smanjara
Copy link
Copy Markdown
Contributor Author

jenkins make check

@yuriw
Copy link
Copy Markdown
Contributor

yuriw commented Jan 27, 2025

jenkins test make check

@yuriw
Copy link
Copy Markdown
Contributor

yuriw commented Jan 27, 2025

@smanjara pls merge when all checks passed
ref: https://tracker.ceph.com/issues/69133

@smanjara smanjara merged commit 90f5e52 into ceph:reef Jan 28, 2025
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.

3 participants