Project

General

Profile

Actions

Bug #42317

closed

mimic: incomplete backport LibCephFS.mount in cephfs.pyx needs it's method signature updated

Added by Rishabh Dave over 4 years ago. Updated over 4 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Category:
-
Target version:
% Done:

0%

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

Description

I came across this issue while trying to reproduce the issue here.

On mimic branch ceph_volume_client.py's CephFSVolumeClient.connect() passes an arg to mount (see [1]) -

            log.debug("Premount eviction of {0} completes".format(premount_evict))
        log.debug("CephFS mounting...")
        self.fs.mount(filesystem_name=to_bytes(self.fs_name))

But cephfs.pyx's `LibCephFS.mount` doesn't accept any arg besides `self` (see [2]) -

 def mount(self):

This is due to an incomplete backport - https://github.com/ceph/ceph/commit/1ab18fcf056ef8e4fcb9ed27e64ee4f8a75866dc

How to reproduce

On a latest mimic cluster setup by vstart.sh, run test_volclient.py (attached) to see the error and the traceback.
Running any test from test_volume_client.TestVolumeClient would also work but the real error message would be hidden behind CommandFailedError.


Files

test_volclient.py (501 Bytes) test_volclient.py Rishabh Dave, 10/15/2019 05:53 AM

Related issues 1 (0 open1 closed)

Related to CephFS - Backport #40896: mimic: ceph_volume_client: fs_name must be converted to string before using itRejectedNathan CutlerActions
Actions

Also available in: Atom PDF