]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
qa/cephfs: fix TestRenameCommand and unmount the clinet before failing fs 59399/head
authorXiubo Li <xiubli@redhat.com>
Thu, 22 Aug 2024 07:57:25 +0000 (15:57 +0800)
committerXiubo Li <xiubli@redhat.com>
Thu, 22 Aug 2024 07:58:47 +0000 (15:58 +0800)
We should else bring and wait for MDS to be up since it is needed
for unmounting of CephFS in CephFSTestCase.tearDown() to be successful,
or just unmount the mountpoints before failing the filesystem.

Since the mountpoint won't be used in later tests so we just unmount
it.

Fixes: https://tracker.ceph.com/issues/66946
Signed-off-by: Xiubo Li <xiubli@redhat.com>
(cherry picked from commit ae3faac004e64fa2e3bd4a3ff0d823db4da2685c)
Conflicts:
  - Just fixed the code needed in reef.

qa/tasks/cephfs/test_admin.py

index 34dc59ab2a230754a07fbd2ebd665859fa1d2d5d..657f53d8de7eb253df5f390b888fc3c501028010 100644 (file)
@@ -1827,6 +1827,9 @@ class TestFSFail(TestAdminCommands):
         health_warn = 'MDS_CACHE_OVERSIZED'
         self.gen_health_warn_mds_cache_oversized()
 
+        # Failing the file system breaks this mount
+        self.mount_a.umount_wait(require_clean=True)
+
         # actual testing begins now.
         self.negtest_ceph_cmd(args=f'fs fail {self.fs.name}',
                               retval=1, errmsgs=health_warn)
@@ -1846,6 +1849,9 @@ class TestFSFail(TestAdminCommands):
         health_warn = 'MDS_TRIM'
         self.gen_health_warn_mds_trim()
 
+        # Failing the file system breaks this mount
+        self.mount_a.umount_wait(require_clean=True)
+
         # actual testing begins now.
         self.negtest_ceph_cmd(args=f'fs fail {self.fs.name}',
                               retval=1, errmsgs=health_warn)
@@ -1867,6 +1873,9 @@ class TestFSFail(TestAdminCommands):
         self.fs.set_max_mds(2)
         self.gen_health_warn_mds_cache_oversized()
 
+        # Failing the file system breaks this mount
+        self.mount_a.umount_wait(require_clean=True)
+
         # actual testing begins now.
         self.negtest_ceph_cmd(args=f'fs fail {self.fs.name}',
                               retval=1, errmsgs=health_warn)