Skip to content

Commit 2130ec8

Browse files
qa/cephfs: fix test_single_path_authorize_on_nonalphanumeric_fsname
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 <[email protected]>
1 parent d622325 commit 2130ec8

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

qa/tasks/cephfs/test_admin.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1798,10 +1798,9 @@ def test_single_path_authorize_on_nonalphanumeric_fsname(self):
17981798
That fs authorize command works on filesystems with names having [_.-]
17991799
characters
18001800
"""
1801-
self.skipTest('this test is broken ATM, see: '
1802-
'https://tracker.ceph.com/issues/66077')
1803-
18041801
self.mount_a.umount_wait(require_clean=True)
1802+
# let's unmount both client before deleting the FS
1803+
self.mount_b.umount_wait(require_clean=True)
18051804
self.mds_cluster.delete_all_filesystems()
18061805
fs_name = "cephfs-_."
18071806
self.fs = self.mds_cluster.newfs(name=fs_name)

0 commit comments

Comments
 (0)