Project

General

Profile

Actions
Copy link

Bug #21752

closed

ceph.in: ceph fs status returns error

Added by Patrick Donnelly over 6 years ago. Updated over 6 years ago.

Status:
Resolved
Priority:
High
Assignee:
John Spray
Category:
-
Target version:
Ceph - v12.2.2
% Done:

0%

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

Description

$ bin/ceph fs status cephfs_a
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
2017-10-10 13:19:21.820458 7f5eeec71700 -1 WARNING: all dangerous and experimental features are enabled.
2017-10-10 13:19:21.832435 7f5eeec71700 -1 WARNING: all dangerous and experimental features are enabled.
Error EINVAL: Traceback (most recent call last):
  File "/home/pdonnell/ceph/src/pybind/mgr/status/module.py", line 301, in handle_command
    return self.handle_fs_status(cmd)
  File "/home/pdonnell/ceph/src/pybind/mgr/status/module.py", line 173, in handle_fs_status
    ) + "/s" 
TypeError: format_units() takes exactly 5 arguments (3 given)
Actions
Copy link
 #1

Updated by Patrick Donnelly over 6 years ago

  • Assignee set to John Spray

John, can you take a look?

Actions
Copy link
 #2

Updated by Yanhu Cao over 6 years ago

see https://github.com/ceph/ceph/pull/17380

if state == "active":
-    activity = "Reqs: " + self.format_dimless(
+    activity = "Reqs: " + self.format_units(

should use format_dimless, But it changed and did not revert.

Actions
Copy link
 #4

Updated by Kefu Chai over 6 years ago

  • Status changed from New to Fix Under Review
Actions
Copy link
 #5

Updated by Kefu Chai over 6 years ago

  • Project changed from CephFS to mgr
  • Category deleted (Administration/Usability)
Actions
Copy link
 #6

Updated by John Spray over 6 years ago

  • Status changed from Fix Under Review to Resolved

Merged, and noted on http://tracker.ceph.com/issues/21699 to pull in the patch rather than creating a separate backport for this.

Actions
Copy link

Also available in: Atom PDF