Project

General

Profile

Actions

Bug #50195

closed

mgr/dashboard: filesystem pool size should use stored stat instead of bytes_used

Added by David Rivera about 3 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Component - CephFS
Target version:
-
% Done:

0%

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

Description

Description of problem

Filesystem pool size should use stored stat instead of bytes_used.

CephFS pool stats percent_used reports ~60% used:

[ceph: root@ceph0 /]# ceph df -f json | jq .pools[2]
{
  "name": "cephfs",
  "id": 3,
  "stats": {
    "stored": 10414261796864,
    "objects": 8814736,
    "kb_used": 20335999808,
    "bytes_used": 20824063803392,
    "percent_used": 0.5955945253372192,
    "max_avail": 7069713301504
  }
}

The dashboard does not match the percent used.

Environment

  • ceph version string: ceph version 15.2.10 (27917a557cca91e4da407489bbaa64ad4352cc02) octopus (stable)
  • Platform (OS/distro/release): CentOS Linux release 8.3.2011
  • Cluster details (nodes, monitors, OSDs): 4 nodes, 3 monitors, 48 OSDs
  • Browser used (e.g.: Version 86.0.4240.198 (Official Build) (64-bit)): Google Chrome Version 89.0.4389.114 (Official Build) (64-bit)

How reproducible

Dashboard > Filesystems > Cephfs filesystem details > Pools Detail Table

Actual results

Ceph Filesystem Dashboard is incorrect (Reporting 75% instead of 60%):

Expected results

Replacing bytes_used with stored returns the expected result:

Additional info

This occurs because the dashboard is using bytes_used stat, that looks to be the same as raw bytes used, and max_avail stat which takes replication or erasure coding into consideration.

Requires change in https://github.com/ceph/ceph/blob/master/src/pybind/mgr/dashboard/controllers/cephfs.py on line 259.


Files


Related issues 3 (0 open3 closed)

Copied to Dashboard - Backport #50416: octopus: mgr/dashboard: filesystem pool size should use stored stat instead of bytes_usedResolvedAvan ThakkarActions
Copied to Dashboard - Backport #50417: nautilus: mgr/dashboard: filesystem pool size should use stored stat instead of bytes_usedResolvedAvan ThakkarActions
Copied to Dashboard - Backport #50418: pacific: mgr/dashboard: filesystem pool size should use stored stat instead of bytes_usedResolvedAvan ThakkarActions
Actions #1

Updated by Avan Thakkar about 3 years ago

  • Assignee set to Avan Thakkar
  • Backport set to nautilus, octopus, pacific
Actions #2

Updated by Ernesto Puerta about 3 years ago

  • Project changed from mgr to Dashboard
  • Category changed from 142 to Component - CephFS
Actions #3

Updated by Kefu Chai about 3 years ago

  • Status changed from New to Pending Backport
  • Pull request ID set to 40874
Actions #4

Updated by Backport Bot about 3 years ago

  • Copied to Backport #50416: octopus: mgr/dashboard: filesystem pool size should use stored stat instead of bytes_used added
Actions #5

Updated by Backport Bot about 3 years ago

  • Copied to Backport #50417: nautilus: mgr/dashboard: filesystem pool size should use stored stat instead of bytes_used added
Actions #6

Updated by Backport Bot about 3 years ago

  • Copied to Backport #50418: pacific: mgr/dashboard: filesystem pool size should use stored stat instead of bytes_used added
Actions #7

Updated by Ernesto Puerta almost 3 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF