Project

General

Profile

Bug #21752

ceph.in: ceph fs status returns error

Added by Patrick Donnelly 2 months ago. Updated 2 months ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
-
Target version:
Start date:
10/10/2017
Due date:
% Done:

0%

Source:
Development
Tags:
Backport:
luminous
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Release:
Needs Doc:
No

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)

History

#1 Updated by Patrick Donnelly 2 months ago

  • Assignee set to John Spray

John, can you take a look?

#2 Updated by Yanhu Cao 2 months 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.

#4 Updated by Kefu Chai 2 months ago

  • Status changed from New to Need Review

#5 Updated by Kefu Chai 2 months ago

  • Project changed from fs to mgr
  • Category deleted (Administration/Usability)

#6 Updated by John Spray 2 months ago

  • Status changed from Need 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.

Also available in: Atom PDF