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

rgw: fix transition from full to incremental meta sync #13920

Merged
merged 1 commit into from Mar 10, 2017

Conversation

Projects
None yet
2 participants
@cbodley
Contributor

cbodley commented Mar 10, 2017

when transitioning to incremental sync, we need to write temp_marker
instead of sync_marker, because temp_marker has replaced the full sync
marker (which is a metadata key, rather than an log marker) with
next_step_marker

this was breaking the metadata checkpoint in test_multi.py, because it
was comparing metadata keys (i.e. 'user:zone.user') against mdlog
markers

rgw: fix transition from full to incremental meta sync
when transitioning to incremental sync, we need to write temp_marker
instead of sync_marker, because temp_marker has replaced the full sync
marker (which is a metadata key, rather than an log marker) with
next_step_marker

this was breaking the metadata checkpoint in test_multi.py, because it
was comparing metadata keys (i.e. 'user:zone.user') against mdlog
markers

Signed-off-by: Casey Bodley <cbodley@redhat.com>
@cbodley

This comment has been minimized.

Contributor

cbodley commented Mar 10, 2017

this fix is related to the bucket cleanup, so i assume it won't need backport

@yehudasa

lgtm

@yehudasa yehudasa merged commit 5a91efd into ceph:master Mar 10, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@cbodley cbodley deleted the cbodley:wip-rgw-fix-cleanup-meta-sync branch Mar 10, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment