Actions
Bug #7535
closedSwitching to classic scrub unnecessarily
Status:
Duplicate
Priority:
Normal
Assignee:
David Zafman
Category:
-
Target version:
-
% Done:
0%
Source:
other
Tags:
Backport:
Regression:
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
test/vstart_wrapped_tests.sh output:
pg[0.0( empty local-les=20 n=0 ec=1 les/c 20/20 14/14/6) [0,2,1] r=-1 lpr=14 crt=0'0 mlcod 0'0 active+clean+scrubbing+d
eep+repair] OSD 0 does not support chunky scrubs, falling back to classic
assert(deep_scrub);
We need to fix the code that determines that a class scrub is needed because that should NOT be the case here. We may want to remove the assert() for backwards compatibility.
Changes I made to the scrubbing code were not tested with the classic scrub which may not even be possible with current upgrade scenarios. We should determine if the classic scrub code should be removed but not likely for firefly.
Actions