Project

General

Profile

Bug #40981

mgr/dashboard: CephFS class issues with strings

Added by Volker Theile over 3 years ago. Updated over 1 year 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" 
    ]
}

History

#1 Updated by Volker Theile over 3 years ago

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

#2 Updated by Volker Theile over 3 years ago

  • Description updated (diff)

#3 Updated by Volker Theile over 3 years ago

  • Status changed from Fix Under Review to Resolved

#4 Updated by Ernesto Puerta over 1 year ago

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

Also available in: Atom PDF