Bug #43224: ceph osd status error - mgr - Ceph

Actions

Copy link

Bug #43224

closed

ceph osd status error

Added by Alexander Kazansky over 4 years ago. Updated over 4 years ago.

Status:

Duplicate

Priority:

Urgent

Assignee:

Category:

Target version:

Ceph - v14.2.5

% Done:

Source:

Tags:

low-hanging-fruit

Backport:

Regression:

Severity:

3 - minor

Reviewed:

Affected Versions:

ceph-qa-suite:

Pull request ID:

28603

Crash signature (v1):

Crash signature (v2):

Description

Hello.

I received error when try execute 'ceph osd status', it happens when cluster have some osd down.

# ceph osd status
Error EINVAL: Traceback (most recent call last):
  File "/usr/share/ceph/mgr/mgr_module.py", line 914, in _handle_command
    return self.handle_command(inbuf, cmd)
  File "/usr/share/ceph/mgr/status/module.py", line 253, in handle_command
    return self.handle_osd_status(cmd)
  File "/usr/share/ceph/mgr/status/module.py", line 237, in handle_osd_status
    mgr_util.format_dimless(self.get_rate("osd", osd_id.__str__(), "osd.op_w") +
  File "/usr/share/ceph/mgr/status/module.py", line 47, in get_rate
    return (data[-1][1] - data[-2][1]) / float(data[-1][0] - data[-2][0])
ZeroDivisionError: float division by zero

if all osd up, command executed normal.

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Brad Hubbard over 4 years ago

Project changed from Ceph to mgr
Category deleted (~~ceph cli~~)

Actions

Copy link

Updated by Sage Weil over 4 years ago

Priority changed from Normal to Urgent

Actions

Copy link

Updated by Neha Ojha over 4 years ago

Tags set to low-hanging-fruit

Actions

Copy link

Updated by Stephan Müller over 4 years ago

This will be fixed with my PR (https://github.com/ceph/ceph/pull/28603) as soon it is merged.

Actions

Copy link

Updated by Sebastian Wagner over 4 years ago

Note that https://github.com/ceph/ceph/pull/28603 fixes a copy of this code in the dashboard module. someone would need to adapt the code in the status module, too.

Actions

Copy link

Updated by Nathan Cutler over 4 years ago

Related to Feature #40365: mgr: Add get_rates_from_data from the dashboard to the mgr_util.py added

Actions

Copy link

Updated by Sage Weil over 4 years ago

Pull request ID set to 28603

Actions

Copy link

Updated by Nathan Cutler over 4 years ago

Related to deleted (Feature #40365: mgr: Add get_rates_from_data from the dashboard to the mgr_util.py)

Actions

Copy link

Updated by Nathan Cutler over 4 years ago

Is duplicate of Feature #40365: mgr: Add get_rates_from_data from the dashboard to the mgr_util.py added

Actions

Copy link

#10

Updated by Nathan Cutler over 4 years ago

Status changed from New to Duplicate

Actions

Copy link

#11

Updated by Nathan Cutler over 4 years ago

Status changed from Duplicate to New
Pull request ID deleted (~~28603~~)

Sebastian wrote above: "Note that https://github.com/ceph/ceph/pull/28603 fixes a copy of this code in the dashboard module. someone would need to adapt the code in the status module, too."

which I take to mean that this issue is tracking the problem in the status module.

Actions

Copy link

#12