Project

General

Profile

Bug #48062

mgr/dashboard: missing root path of each session in Cephfs dashboard

Added by Yongsoek Oh 7 months ago. Updated about 2 months ago.

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

100%

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

The current version of the dashboard (src/pybind/mgr/dashboard/controllers/cephfs.py) missed the client root path in the _clients() function. So I have fixed and added it as described in the below code. You can also confirm that the attachment properly displays the session root path.

        for client in clients:
            if "ceph_version" in client['client_metadata']:  # pragma: no cover - no complexity
                client['type'] = "userspace" 
                client['version'] = client['client_metadata']['ceph_version']
                client['hostname'] = client['client_metadata']['hostname']
                client['root'] = client['client_metadata']['root']
            elif "kernel_version" in client['client_metadata']:  # pragma: no cover - no complexity
                client['type'] = "kernel" 
                client['version'] = client['client_metadata']['kernel_version']
                client['hostname'] = client['client_metadata']['hostname']
                client['root'] = client['client_metadata']['root']


Related issues

Copied to Dashboard - Backport #49083: octopus: mgr/dashboard: missing root path of each session in Cephfs dashboard Resolved
Copied to Dashboard - Backport #49084: nautilus: mgr/dashboard: missing root path of each session in Cephfs dashboard Resolved
Copied to Dashboard - Backport #49085: pacific: mgr/dashboard: missing root path of each session in Cephfs dashboard Resolved

History

#1 Updated by Volker Theile 4 months ago

  • Status changed from New to Fix Under Review

#2 Updated by Volker Theile 4 months ago

  • Backport set to nautilus, octopus, pacific

#3 Updated by Ernesto Puerta 4 months ago

  • Project changed from Ceph to mgr
  • Subject changed from Missing root path of each session in Cephfs dashboard to mgr/dashboard: missing root path of each session in Cephfs dashboard
  • Category changed from Monitoring/Alerting to dashboard/cephfs
  • Status changed from Fix Under Review to Pending Backport
  • Pull request ID set to 36518

#4 Updated by Backport Bot 4 months ago

  • Copied to Backport #49083: octopus: mgr/dashboard: missing root path of each session in Cephfs dashboard added

#5 Updated by Backport Bot 4 months ago

  • Copied to Backport #49084: nautilus: mgr/dashboard: missing root path of each session in Cephfs dashboard added

#6 Updated by Backport Bot 4 months ago

  • Copied to Backport #49085: pacific: mgr/dashboard: missing root path of each session in Cephfs dashboard added

#7 Updated by Ernesto Puerta 3 months ago

  • Status changed from Pending Backport to Resolved

#8 Updated by Ernesto Puerta about 2 months ago

  • Tracker changed from Fix to Bug
  • Project changed from mgr to Dashboard
  • Category changed from dashboard/cephfs to Component - CephFS
  • Regression set to No
  • Severity set to 3 - minor

Also available in: Atom PDF