Project

General

Profile

Bug #12402

get pools health'info have error

Added by huanwen ren about 3 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
07/20/2015
Due date:
% Done:

0%

Source:
Community (user)
Tags:
Backport:
firefly, hammer
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:

Description

file:OSDMonitor.cc
function: void OSDMonitor::get_pools_health(
list<pair<health_status_t,string> >& summary,
list<pair<health_status_t,string> > *detail) const
describe:
......
} else if (crit_threshold > 0 &&
sum.num_bytes >= pool.quota_max_bytes*crit_threshold) {
ss << "pool '" << pool_name
<< "' has " << si_t(sum.num_bytes) << " bytes"
<< " (max " << si_t(pool.quota_max_bytes) << ")";
status = HEALTH_ERR;
} else if (warn_threshold > 0 &&
sum.num_bytes >= pool.quota_max_bytes*warn_threshold) {
ss << "pool '" << pool_name
<< "' has " << si_t(sum.num_bytes) << " objects"
<< " (max " << si_t(pool.quota_max_bytes) << ")";
status = HEALTH_WARN;
}
......

I think the "objects" replaced by "bytes"?

OSDMonitor.cc View - code file (217 KB) huanwen ren, 07/20/2015 02:27 AM

errors.jpg View - errors.jpg (96.3 KB) huanwen ren, 07/20/2015 02:27 AM


Related issues

Copied to Ceph - Backport #12497: get pools health'info have error Resolved 07/20/2015
Copied to Ceph - Backport #12498: get pools health'info have error Resolved 07/20/2015

Associated revisions

Revision 7fc13c9d (diff)
Added by renhwztetecs about 3 years ago

Update OSDMonitor.cc

OSDMonitor::get_pools_health(), s/objects/bytes/

Fixes: #12402
Signed-off-by:

Revision 4457d3eb (diff)
Added by renhwztetecs about 3 years ago

Update OSDMonitor.cc

OSDMonitor::get_pools_health(), s/objects/bytes/

Fixes: #12402
Signed-off-by:
(cherry picked from commit 7fc13c9d6b9a4962d7640240416105d8f558d600)

Revision 31ff7d14 (diff)
Added by renhwztetecs about 3 years ago

Update OSDMonitor.cc

OSDMonitor::get_pools_health(), s/objects/bytes/

Fixes: #12402
Signed-off-by:
(cherry picked from commit 7fc13c9d6b9a4962d7640240416105d8f558d600)

History

#1 Updated by huang jun about 3 years ago

yes, it's a wrong hints, you can send a pull request in github, or send a patch in mailinglist.

#2 Updated by Kefu Chai about 3 years ago

  • Status changed from New to Need Review
  • Priority changed from High to Normal

#3 Updated by Kefu Chai about 3 years ago

  • Status changed from Need Review to Pending Backport
  • Backport set to firefly, hammer

#4 Updated by Nathan Cutler almost 3 years ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF