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

luminous: multisite: FAILED assert(prev_iter != pos_to_prev.end()) in RGWMetaSyncShardCR::collect_children() #17234

Merged
merged 1 commit into from Aug 25, 2017

Conversation

Projects
None yet
3 participants
@theanalyst
Member

theanalyst commented Aug 24, 2017

cls/log: cls_log_list always returns next marker
commit 5334622 changed cls_log_list()
to only return the next marker if the results were truncated

this broke RGWMetaSyncShardCR in rgw_sync.cc, which relies on
cls_log_list() to track its max_marker

Fixes: http://tracker.ceph.com/issues/20906

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit f7ea4ea)

@theanalyst theanalyst self-assigned this Aug 24, 2017

@theanalyst theanalyst added this to the luminous milestone Aug 24, 2017

@theanalyst theanalyst requested review from yehudasa, cbodley and mattbenjamin Aug 24, 2017

@theanalyst

This comment has been minimized.

Member

theanalyst commented Aug 25, 2017

@liewegas is this ok to merge?

@liewegas

This comment has been minimized.

Member

liewegas commented Aug 25, 2017

@cbodley

This comment has been minimized.

Contributor

cbodley commented Aug 25, 2017

@liewegas confirmed unrelated. the two s3a-hadoop failures appear to be broken due to ceph-ansible. the two POOL_APP_NOT_ENABLED failures were addressed by #17162

@liewegas liewegas merged commit 491e582 into ceph:luminous Aug 25, 2017

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment