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: osd: read object attrs failed at EC recovery #24327

Merged
merged 1 commit into from Oct 4, 2018

Conversation

Projects
None yet
3 participants
@smithfarm
Contributor

smithfarm commented Sep 28, 2018

osd: retry to read object attrs at EC recovery
In EC recovery read, if the object's attrs read failed or with errors, we erase the attrs we have read and
try to read it again from left shards. This will make the primary osd get the object's attrs correct and
avoid assert.

Signed-off-by: xiaofei cui <cuixiaofei@sangfor.com>
(cherry picked from commit 3eb1679)

@smithfarm smithfarm self-assigned this Sep 28, 2018

@smithfarm smithfarm added this to the luminous milestone Sep 28, 2018

@smithfarm smithfarm requested review from dzafman and tchaikov Sep 28, 2018

@dzafman

dzafman approved these changes Oct 1, 2018

Looks good without conflicts

@yuriw

This comment has been minimized.

Contributor

yuriw commented Oct 3, 2018

@yuriw yuriw merged commit 4fed9d5 into ceph:luminous Oct 4, 2018

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

@smithfarm smithfarm deleted the smithfarm:wip-24478-luminous branch Oct 5, 2018

@smithfarm smithfarm changed the title from luminous: read object attrs failed at EC recovery to luminous: osd: read object attrs failed at EC recovery Oct 26, 2018

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