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 #1

Updated by David Zafman about 10 years ago

This was primarily fixed by 7357b6ed4bbc6ef44a80640ffb7e1ddacab10b5e for tracker #7517.

Actions #2

Updated by David Zafman about 10 years ago

  • Status changed from 7 to Duplicate
Actions

Also available in: Atom PDF