Project

General

Profile

Actions

Bug #48062

closed

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

Added by Yongseok Oh over 3 years ago. Updated almost 3 years 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 3 (0 open3 closed)

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

Updated by Volker Theile about 3 years ago

  • Status changed from New to Fix Under Review
Actions #2

Updated by Volker Theile about 3 years ago

  • Backport set to nautilus, octopus, pacific
Actions #3

Updated by Ernesto Puerta about 3 years 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 142
  • Status changed from Fix Under Review to Pending Backport
  • Pull request ID set to 36518
Actions #4

Updated by Backport Bot about 3 years ago

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

Updated by Backport Bot about 3 years ago

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

Updated by Backport Bot about 3 years ago

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

Updated by Ernesto Puerta about 3 years ago

  • Status changed from Pending Backport to Resolved
Actions #8

Updated by Ernesto Puerta almost 3 years ago

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

Also available in: Atom PDF