Project

General

Profile

Actions

Bug #19733

closed

clean up min/max span warning

Added by David Zafman about 7 years ago. Updated over 6 years ago.

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

0%

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

Description

Sage cleaned up the name of the config and some minor output changes:

https://github.com/ceph/ceph/pull/14611


Related issues 4 (1 open3 closed)

Related to Ceph - Bug #15912: An OSD was seen getting ENOSPC even with osd_failsafe_full_ratio passedResolvedDavid Zafman05/17/2016

Actions
Related to RADOS - Bug #20730: need new OSD_SKEWED_USAGE implementationNew

Actions
Copied to Ceph - Backport #19841: kraken: clean up min/max span warningResolvedNathan CutlerActions
Copied to Ceph - Backport #19842: jewel: clean up min/max span warningResolvedAlexey SheplyakovActions
Actions #1

Updated by David Zafman about 7 years ago

  • Related to Bug #15912: An OSD was seen getting ENOSPC even with osd_failsafe_full_ratio passed added
Actions #2

Updated by Sage Weil almost 7 years ago

  • Status changed from 7 to Pending Backport
Actions #3

Updated by Nathan Cutler almost 7 years ago

Actions #4

Updated by Nathan Cutler almost 7 years ago

Actions #5

Updated by Nathan Cutler over 6 years ago

  • Status changed from Pending Backport to Resolved
Actions #6

Updated by David Zafman over 6 years ago

Unfortunately, I think Sage removed this code later, so that probably needs to be backported too.

Actions #7

Updated by Nathan Cutler over 6 years ago

David, do you mean daa0793c393a21bd2dd2ec52a0efd181e1032400 ?

smithfarm@wilbur:~/src/ceph/smithfarm/ceph> git log -S 'difference between min' src/mon/PGMonitor.cc
commit daa0793c393a21bd2dd2ec52a0efd181e1032400
Author: Sage Weil <sage@redhat.com>
Date:   Tue May 16 13:20:51 2017 -0400

    mon/PGMonitor: move most of get_health into PGMap

    Leave the legacy full/nearfull stuff in PGMonitor (although we'll still
    need to clean out the PGMap fields too).

    This is cut and paste, some stripping of pg_map., g_conf -> cct->_conf,
    and a bit of whitespace cleanup.

    Signed-off-by: Sage Weil <sage@redhat.com>

commit 489e810c37ed6fb9d32d1015634041a577501ee4
Author: Sage Weil <sage@redhat.com>
Date:   Tue Apr 18 13:54:56 2017 -0400

    mon/PGMonitor: clean up min/max span warning

    Clean up option naming.

    Signed-off-by: Sage Weil <sage@redhat.com>
Actions #8

Updated by David Zafman over 6 years ago

7832c53375ad24f3b54a68a3c63b056e69e3fa0d removed it after it was moved by daa0793c393a21bd2dd2ec52a0efd181e1032400

commit 7832c53375ad24f3b54a68a3c63b056e69e3fa0d
Author: Sage Weil <sage@redhat.com>
Date:   Fri Jul 21 11:58:08 2017 -0400

    mon/PGMap: remove skewed utilizatoin warning

    This has a few problems:

    1- It does not do it's analysis over CRUSH rule roots/classes, which
    means that an innocent user of classes will see skewed usage (bc hdds are
    more full than ssds, say)

    2- It does not take degraded clusters into account, which means the warning
    will appear when a fresh OSD is added.

    See http://tracker.ceph.com/issues/20730

    Signed-off-by: Sage Weil <sage@redhat.com>
Actions #9

Updated by David Zafman over 6 years ago

  • Related to Bug #20730: need new OSD_SKEWED_USAGE implementation added
Actions #10

Updated by David Zafman over 6 years ago

The tracker for the removal is http://tracker.ceph.com/issues/20730. I added it to related issues.

Actions

Also available in: Atom PDF