]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mon: Fix status output warning for mon_warn_osd_usage_min_max_delta 16134/head
authorDavid Zafman <dzafman@redhat.com>
Fri, 7 Jul 2017 17:53:41 +0000 (10:53 -0700)
committerNathan Cutler <ncutler@suse.com>
Tue, 18 Jul 2017 20:19:42 +0000 (22:19 +0200)
Fixes: http://tracker.ceph.com/issues/20544
Caued by: 489e810c37ed6fb9d32d1015634041a577501ee4

Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit 56f9808016ab289bdc0ce7cfbb0503c78b509593)

Conflicts:
    src/mon/PGMap.cc:
        kraken has no daa0793c393a21bd2dd2ec52a0efd181e1032400 so we manually
        apply the changes from 56f9808016ab289bdc0ce7cfbb0503c78b509593 to
        src/mon/PGMonitor.cc instead of to src/mon/PGMap.cc

src/mon/PGMonitor.cc

index 5e5644daf88391b90542457e5b9e066e6295413d..5b36af5a8f31aabc5ecdc797c69e44f0bfbf5367 100644 (file)
@@ -1811,11 +1811,11 @@ void PGMonitor::get_health(list<pair<health_status_t,string> >& summary,
     float diff = max_osd_usage - min_osd_usage;
     if (diff > g_conf->mon_warn_osd_usage_min_max_delta) {
       ostringstream ss;
-      ss << "difference between min (" << roundf(min_osd_usage*1000.0)/100.0
-        << "%) and max (" << roundf(max_osd_usage*1000.0)/100.0
-        << "%) osd usage " << roundf(diff*1000.0)/100.0 << "% > "
-        << roundf(g_conf->mon_warn_osd_usage_min_max_delta*1000.0)/100.0
-        << " (mon_warn_osd_usage_min_max_delta)";
+      ss << "difference between min (" << roundf(min_osd_usage*1000.0)/10.0
+         << "%) and max (" << roundf(max_osd_usage*1000.0)/10.0
+         << "%) osd usage " << roundf(diff*1000.0)/10.0 << "% > "
+         << roundf(cct->_conf->mon_warn_osd_usage_min_max_delta*1000.0)/10.0
+         << "% (mon_warn_osd_usage_min_max_delta)";
       summary.push_back(make_pair(HEALTH_WARN, ss.str()));
       if (detail)
         detail->push_back(make_pair(HEALTH_WARN, ss.str()));