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

octopus: radosgw-admin: 'sync status' is not behind if there are no mdlog entries #45443

Merged
merged 1 commit into from May 10, 2022

Conversation

cfsnyder
Copy link
Contributor

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


backport of #42708
parent tracker: https://tracker.ceph.com/issues/52091

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

if remote mdlogs are trimmed prematurely, sync status will report
that it's behind the remote's max-marker even if there are no mdlog
entries to sync

for each behind shard, we fetch the next mdlog entry from the remote. if
we get an empty listing, remove that shard from behind_shards. this
logic now has to run before we print "behind shards:" so that empty
shards aren't listed

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

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 14d43f7)
@yuriw
Copy link
Contributor

yuriw commented May 9, 2022

jenkins test make check

@yuriw yuriw merged commit 9f1b4c5 into ceph:octopus May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants