Project

General

Profile

Bug #6291

Recovery can take more resources than it should

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

Status:
Resolved
Priority:
Normal
Assignee:
David Zafman
Category:
-
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

The code in do_recovery() uses MAX instead of MIN so is can exceed osd_recovery_max_active.

Associated revisions

Revision 139a714e (diff)
Added by David Zafman over 10 years ago

osd/OSD.cc: Use MIN so that we don't exceed osd_recovery_max_active

Caused by 944f3b73531af791c90f0f061280160003545c63

Fixes: #6291

Backport: dumpling

Signed-off-by: David Zafman <>
Reviewed-by: Samuel Just <>

Revision e0203c61 (diff)
Added by David Zafman over 10 years ago

osd/OSD.cc: Use MIN so that we don't exceed osd_recovery_max_active

Caused by 944f3b73531af791c90f0f061280160003545c63

Fixes: #6291

Backport: dumpling

Signed-off-by: David Zafman <>
Reviewed-by: Samuel Just <>
(cherry picked from commit 139a714e13aa3c7f42091270b55dde8a17b3c4b8)

Conflicts:

src/osd/OSD.cc

History

#1 Updated by David Zafman over 10 years ago

Caused by 944f3b73 "OSD: only start osd_recovery_max_single_start at once"

#2 Updated by David Zafman over 10 years ago

  • Status changed from New to Pending Backport

139a714e13aa3c7f42091270b55dde8a17b3c4b8

#3 Updated by Samuel Just over 10 years ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF