Project

General

Profile

Bug #40981

mgr/dashboard: CephFS class issues with strings

Added by Volker Theile about 2 months ago. Updated 23 days ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
dashboard/cephfs
Target version:
Start date:
07/26/2019
Due date:
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:

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 about 2 months ago

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

#2 Updated by Volker Theile about 2 months ago

  • Description updated (diff)

#3 Updated by Volker Theile 23 days ago

  • Status changed from Need Review to Resolved

Also available in: Atom PDF