Project

General

Profile

Actions

Bug #7535

closed

Switching to classic scrub unnecessarily

Added by David Zafman about 10 years ago. Updated about 10 years ago.

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.


Related issues 1 (0 open1 closed)

Has duplicate Ceph - Bug #7517: osd/PG.cc: 3673: FAILED assert(pool.info.type == pg_pool_t::TYPE_REPLICATED)ResolvedSamuel Just02/23/2014

Actions
Actions

Also available in: Atom PDF