Actions
Bug #50195
closedmgr/dashboard: filesystem pool size should use stored stat instead of bytes_used
% 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
Updated by Avan Thakkar about 3 years ago
- Assignee set to Avan Thakkar
- Backport set to nautilus, octopus, pacific
Updated by Ernesto Puerta about 3 years ago
- Project changed from mgr to Dashboard
- Category changed from 142 to Component - CephFS
Updated by Kefu Chai about 3 years ago
- Status changed from New to Pending Backport
- Pull request ID set to 40874
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
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
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
Updated by Ernesto Puerta almost 3 years ago
- Status changed from Pending Backport to Resolved
Actions