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: Improve size scrub error handling and ignore system attrs in xattr checking #17196

Merged
merged 8 commits into from Aug 24, 2017

Conversation

Projects
None yet
4 participants
@theanalyst
Member

theanalyst commented Aug 23, 2017

dzafman added some commits Jun 29, 2017

osd: Fix test op error message
Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit 8325639)
test: Add undocumented corrupt-size for testing
Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit c0606b9)
osd: Add whether shard is primary in list-inconsistent-obj
Add new field in the client interface
Update test case

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

Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit 8ad4b29)
osd: Change a check to an assert() since it can't happen anymore
Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit 8e2b9a0)
osd: Compare all object info even when can't consider for auth copy
Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit 437e5cf)
osd, rados: Improve size scrub error handling
Fixes: http://tracker.ceph.com/issues/20243

Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit 5f58301)
osd, rados: Adding ss_attr_missing and ss_attr_corrupt errors to list…
…-inconsistent-obj

Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit 4c949b6)
osd: In scrub's be_select_auth_object() detect multiple errors better
Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit 75b4256)

@theanalyst theanalyst self-assigned this Aug 23, 2017

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

@theanalyst theanalyst requested review from dzafman and tchaikov Aug 23, 2017

@theanalyst

This comment has been minimized.

Member

theanalyst commented Aug 23, 2017

jenkins test this please

@dzafman

It looks like the 8 commits applied cleanly, so it looks good.

@liewegas liewegas merged commit 5e0a7a2 into ceph:luminous Aug 24, 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