Project

General

Profile

Bug #40981

mgr/dashboard: CephFS class issues with strings

Added by Volker Theile 5 months ago. Updated 3 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
dashboard/cephfs
Target version:
% Done:

0%

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

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 5 months ago

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

#2 Updated by Volker Theile 5 months ago

  • Description updated (diff)

#3 Updated by Volker Theile 3 months ago

  • Status changed from Fix Under Review to Resolved

Also available in: Atom PDF