Project

General

Profile

Actions

Bug #6922

closed

mon: do not allow unreasonable PG splitting

Added by Greg Farnum over 10 years ago. Updated about 10 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Monitor
Target version:
% Done:

0%

Source:
Support
Tags:
Backport:
emperor
Regression:
Severity:
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

We've had two different kinds of issues with PG splits:
1) the incorrectly-interpreted PG values from the Dumpling->Emperor upgrade (with a Dumpling leader) causing creation of millions of PGs,
2) Users splitting by a 255x multiple within a pool.

This causes issues with the OSD of various kinds; the first one for obvious reasons and the second one because the OSD internals are simply not designed to handle a sufficiently-large number of multiples in one go. In particular, in reproducing (2) locally I saw it taking 4 minutes to process the splits, during which time the internal heartbeating failed while waiting for stuff to get into the queue (which was at the default 50-op limit). To deal with both issues, have the monitors specify a maximum PG split ratio and enforce it.


Related issues 1 (0 open1 closed)

Related to Ceph - Fix #6908: OSD: do not do PG creates in-line with incoming requestRejected11/27/2013

Actions
Actions #1

Updated by Greg Farnum over 10 years ago

  • Status changed from In Progress to Fix Under Review
  • Backport set to emperor
Actions #2

Updated by Greg Farnum over 10 years ago

Hrm, a customer has asked that we backport this patch to dumpling as well; is that sort of UI change appropriate on an established version?

Actions #3

Updated by Greg Farnum over 10 years ago

  • Target version changed from v0.73 to v0.74
Actions #4

Updated by Sage Weil over 10 years ago

  • Tracker changed from Fix to Bug
  • Status changed from Fix Under Review to Pending Backport
Actions #6

Updated by Greg Farnum about 10 years ago

  • Status changed from Pending Backport to Resolved

Sam merged this.

Actions

Also available in: Atom PDF