RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2025-02-24 03:27:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

qa/cephfs: fix test_single_path_authorize_on_nonalphanumeric_fsname

Browse Source 
This test deletes the CephFS already present on the cluster at the very
beginning and unmounts the first client beforehand. But it leaves the
second client mounted on this deleted CephFS that doesn't exist for the
rest of the test. And then at the very end of this test it attempts to
remount the second client (during tearDown()) which hangs and causes
test runner to crash.

Unmount the second client beforehand to prevent the bug and delete
mount_b object to avoid confusion for the readers in future about
whether or not 2nd mountpoint exists.

Fixes: https://tracker.ceph.com/issues/66077
Signed-off-by: Rishabh Dave <ridave@redhat.com>
This commit is contained in:
Rishabh Dave 2024-06-26 19:05:35 +05:30
parent d622325c17
commit 2130ec8ebc
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
qa/tasks/cephfs/test_admin.py
View File

@ -1798,10 +1798,9 @@ class TestFsAuthorize(CephFSTestCase):
That fs authorize command works on filesystems with names having [_.-]
characters
"""
self.skipTest('this test is broken ATM, see: '
'https://tracker.ceph.com/issues/66077')
self.mount_a.umount_wait(require_clean=True)
# let's unmount both client before deleting the FS
self.mount_b.umount_wait(require_clean=True)
self.mds_cluster.delete_all_filesystems()
fs_name = "cephfs-_."
self.fs = self.mds_cluster.newfs(name=fs_name)