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

pacific: rgw multisite: complete fix for metadata sync issue #51495

Closed
wants to merge 2 commits into from

Conversation

cbodley
Copy link
Contributor

@cbodley cbodley commented May 15, 2023

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


backport of #46148
parent tracker: https://tracker.ceph.com/issues/46563

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

moningchao and others added 2 commits May 15, 2023 14:51
…ariable

when the log entry in the meta.log object of the secondary cluster is empty,
the value of max_marker is also empty,which can't meet the requirement that
mdlog_marker <= max_marker,resulting in that the secondary cluster can't fetch
new log entry from the master cluster and infinite loop,finally, the secondary
cluster's metadata can't catch up the master cluster. when the truncate is false,
it means that the secondary cluster's meta.log is empyt,we can read more from
master cluster.

Fixes: https://tracker.ceph.com/issues/46563

Signed-off-by: gengjichao <gengjichao@jd.com>
(cherry picked from commit 2ed1c3e)
(cherry picked from commit 3b61154)
This is in addition to 2ed1c3e. Ensures that the we continue syncing logs
that have been fetched already and don't end up cloning more logs prematurely.

Signed-off-by: Shilpa Jagannath <smanjara@redhat.com>
(cherry picked from commit 256f6b3)
@cbodley cbodley added this to the pacific milestone May 15, 2023
@cbodley cbodley added the rgw label May 15, 2023
@cbodley cbodley requested a review from smanjara May 15, 2023 18:51
Copy link
Contributor

@smanjara smanjara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the backports!

@yuriw
Copy link
Contributor

yuriw commented Jun 21, 2023

@cbodley
Per @smanjara
`@yuriweinstein we have observed some issues with pacific: rgw multisite: complete fix for metadata sync issue by cbodley · Pull Request #51495 · ceph/ceph

I don’t think we should merge it until we have found a fix. otherwise rgw looks ok. approved.`

ref: https://trello.com/c/MTsDz3Xr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants