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/pg: bound the portion of the log we request in GetLog::GetLog() #12233

Merged
merged 1 commit into from Jan 13, 2017

Conversation

Projects
None yet
4 participants
@363921219
Contributor

363921219 commented Nov 30, 2016

osd/PG: bound the portion of the log we request in PG::RecoveryState::GetLog::GetLog

Signed-off-by: Jie Wang jie.wang@kylin-cloud.com

osd/PG: bound the portion of the log we request in PG::RecoveryState:…
…:GetLog::GetLog

Signed-off-by: Jie Wang <jie.wang@kylin-cloud.com>

@363921219 363921219 closed this Nov 30, 2016

@363921219 363921219 reopened this Nov 30, 2016

@liewegas liewegas requested a review from athanatos Dec 29, 2016

@athanatos

This comment has been minimized.

Contributor

athanatos commented Jan 3, 2017

Ok, the observation is that we don't need/want to request more logging unless the primary doesn't have the entries. Seems right to me. Needs testing.

@athanatos athanatos added the needs-qa label Jan 3, 2017

@yuriw yuriw merged commit b1c2153 into ceph:master Jan 13, 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
@yuriw

This comment has been minimized.

Contributor

yuriw commented Jan 13, 2017

http://pulpito.front.sepia.ceph.com/yuriw-2017-01-10_22:24:29-rados-wip-yuri-testing_2017_1_8-distro-basic-smithi/

rerun failed
(teuthology-suite -m smithi -c wip-yuri-testing_2017_1_8 -r yuriw-2017-01-10_22:24:29-rados-wip-yuri-testing_2017_1_8-distro-basic-smithi --ceph-repo https://github. com/ceph/ceph-ci.git -p 100)

http://pulpito.ceph.com/yuriw-2017-01-12_17:38:33-rados-wip-yuri-testing_2017_1_8---basic-smithi/ (centos 7.2 is not installed on smithi)

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