Project

General

Profile

Actions

Bug #40981

closed

mgr/dashboard: CephFS class issues with strings

Added by Volker Theile almost 5 years ago. Updated about 3 years ago.

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

0%

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

Description

The services/CephFS class does not handle strings correctly. Some calls of the Cython implementation require bytes instead of strings, other functions return bytes which are not converted to strings, thus the initial behaviour of the function is not working.
E.g. calling CephFS::get_dir_list() should filter out '.' and '..'.

The unfixed code returns e.g.:

{
    "paths": [
        "/b",
        "/b'.'",
        "/b'..'",
        "/b'volumes'" 
    ]
}

instead of:

{
    "paths": [
        "/",
        "/volumes" 
    ]
}

Actions #1

Updated by Volker Theile almost 5 years ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 29353
Actions #2

Updated by Volker Theile almost 5 years ago

  • Description updated (diff)
Actions #3

Updated by Volker Theile over 4 years ago

  • Status changed from Fix Under Review to Resolved
Actions #4

Updated by Ernesto Puerta about 3 years ago

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

Also available in: Atom PDF