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: Object level shard errors are tracked and used if no auth available #15397

Merged
merged 3 commits into from Jun 2, 2017

Conversation

Projects
None yet
4 participants
@dzafman
Member

dzafman commented May 31, 2017

No description provided.

@dzafman dzafman requested a review from liewegas May 31, 2017

@dzafman

This comment has been minimized.

Member

dzafman commented May 31, 2017

Waiting for shaman build to do a repair_test rados test of it.

Revert "osd: When scrub finds an attr error mark shard inconsistent"
This reverts commit 44736aa.

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

This comment has been minimized.

Contributor

tchaikov commented Jun 1, 2017

retest this please. see #15406

dzafman added some commits Jun 1, 2017

osd-scrub-repair: Fix broken test and correctly handle ec overwrite t…
…esting

Caused by: 3ca750d

Signed-off-by: David Zafman <dzafman@redhat.com>
osd: Object level shard errors are tracked and used if no auth available
Shards with object mismatch are tracked to mark them inconsistent
Fix test because storing omap_digest in object_info not behaving as before

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

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

@dzafman dzafman requested a review from jdurgin Jun 1, 2017

@jdurgin

jdurgin approved these changes Jun 2, 2017

@liewegas liewegas merged commit 1244c0a into ceph:master Jun 2, 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

@dzafman dzafman deleted the dzafman:wip-20089 branch Jun 2, 2017

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