]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
PGMonitor: unconfuse object count skew message 7882/head
authorPiotr Dałek <piotr.dalek@ts.fujitsu.com>
Wed, 2 Mar 2016 12:22:38 +0000 (13:22 +0100)
committerPiotr Dałek <piotr.dalek@ts.fujitsu.com>
Thu, 3 Mar 2016 14:01:56 +0000 (15:01 +0100)
"Pool <pool> has too few pgs" is okay assuming it does not take other
pools into account. And since it does, it is confusing in the following
scenario:

1. Create two pools, one with small pg count and one with large
   pg count
2. Put a whole lot of objects in smaller pool, resulting in "too few
   pgs" warning on that pool, which is expected behavior.
3. Put a whole lot of objects in larger pool, warning goes away.
   Suddenly smaller pool has plenty of PGs?

Current message suggests adding more nodes (or PGs) to pool, when
actually it's warning about significantly more objects in that
particular pool than in the other pools.

Signed-off-by: Piotr Dałek <piotr.dalek@ts.fujitsu.com>
src/mon/PGMonitor.cc

index 99e2287cf36ff302a309431c4cf58f0aeb86b3b1..0a5f2277a8d5bcd66cde0c00efd4fd9475e61a03 100644 (file)
@@ -2395,7 +2395,7 @@ void PGMonitor::get_health(list<pair<health_status_t,string> >& summary,
        if (g_conf->mon_pg_warn_max_object_skew > 0 &&
            ratio > g_conf->mon_pg_warn_max_object_skew) {
          ostringstream ss;
-         ss << "pool " << name << " has too few pgs";
+         ss << "pool " << name << " has many more objects per pg than average (too few pgs?)";
          summary.push_back(make_pair(HEALTH_WARN, ss.str()));
          if (detail) {
            ostringstream ss;