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

osd: scrub_to specifies clone ver, but transaction include head write… #16404

Merged
merged 1 commit into from Jul 26, 2017

Conversation

Projects
None yet
3 participants
@dzafman
Member

dzafman commented Jul 18, 2017

… ver

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

Signed-off-by: David Zafman dzafman@redhat.com

@dzafman

This comment has been minimized.

Member

dzafman commented Jul 19, 2017

Waiting for testing of this based on Jewel cluster from original reporter of tracker.

5 fails: dzafman-2017-07-18_21:53:08-rados-wip-zafman-testing-distro-basic-smithi
1418366 Error ENOENT: osd.3 does not exist. create it before updating the crush map
1418387 Segmentation violation in cephx_verify_authorizer()
1418434 status 124: 'sudo adjust-ulimits ceph-coverage /home/ubuntu/cephtest/archive/coverage timeout 120 ceph --cluster ceph quorum_status'
1418448 Error compiling test crush map; probably need newer crushtool
1418521 src/osd/OSD.cc: 4701: FAILED assert(curmap) in handle_osd_ping()

@dzafman dzafman removed the needs-qa label Jul 19, 2017

@dzafman dzafman requested a review from jdurgin Jul 20, 2017

@dzafman

This comment has been minimized.

Member

dzafman commented Jul 20, 2017

@jdurgin Before merging should we wait for Stefan to test the Jewel version on his cluster?

@jdurgin

This comment has been minimized.

Member

jdurgin commented Jul 24, 2017

Yeah, let's see how it goes on Stefan's cluster

@jdurgin jdurgin added the needs-qa label Jul 26, 2017

@jdurgin jdurgin added this to the luminous milestone Jul 26, 2017

@dzafman

This comment has been minimized.

Member

dzafman commented Jul 26, 2017

Stefan is reporting no hanging scrubs on original reporting cluster.

@dzafman dzafman merged commit c9a5e47 into ceph:master Jul 26, 2017

4 checks passed

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
make check (arm64) make check succeeded
Details

@dzafman dzafman deleted the dzafman:wip-20041 branch Jul 26, 2017

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