Project

General

Profile

Actions

Bug #42767

open

mgr/pg_autoscaler: warning message should mention which pool is affected

Added by Lenz Grimmer over 4 years ago. Updated about 1 year ago.

Status:
Fix Under Review
Priority:
Normal
Assignee:
-
Category:
pg_autoscaler module
Target version:
-
% Done:

0%

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

Description

When using the PG autoscaler in "warn" mode, it correctly sets the HEALTH_WARN state and shows the following warning:

1 pools have too many placement groups

This message is not very helpful, as it does not indicate which pool is the one to check/adjust. It would be great if the name(s) of the affected pool(s) would be displayed here.

And a minor nit: if only one pool is affected, the correct message would be "1 pool has too many placement groups"...

Actions #1

Updated by Kefu Chai over 4 years ago

  • Assignee set to Kefu Chai
Actions #2

Updated by Kefu Chai over 4 years ago

  • Assignee deleted (Kefu Chai)
Actions #3

Updated by Kiefer Chang over 4 years ago

FWIW, `bin/ceph health detail` display the pool name:

╰─# bin/ceph health detail
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
2019-11-15T07:26:56.946+0000 7f3444d78700 -1 WARNING: all dangerous and experimental features are enabled.
2019-11-15T07:26:56.974+0000 7f3444d78700 -1 WARNING: all dangerous and experimental features are enabled.
HEALTH_WARN 1 pool(s) have no replicas configured; 1 pools have too many placement groups
[WRN] POOL_NO_REDUNDANCY: 1 pool(s) have no replicas configured
    pool 'ab' has no replicas configured
[WRN] POOL_TOO_MANY_PGS: 1 pools have too many placement groups
    Pool ab has 128 placement groups, should have 4

Actions #4

Updated by Laura Flores over 1 year ago

Can also fix "{0} pools have too few placement groups".

Actions #5

Updated by Laura Flores over 1 year ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 48151
Actions #6

Updated by Radoslaw Zarzynski about 1 year ago

Fixing commits.

Actions

Also available in: Atom PDF