]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
qa: unmount volumes before removal
authorPatrick Donnelly <pdonnell@redhat.com>
Fri, 30 Oct 2020 21:40:25 +0000 (14:40 -0700)
committerKotresh HR <khiremat@redhat.com>
Fri, 12 Feb 2021 09:55:07 +0000 (15:25 +0530)
To avoid potential failures/hangs in umount.

Fixes: https://tracker.ceph.com/issues/23718
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 4ca8aaafa32bb7397eec80a8eb366d2dd781a72f)

qa/tasks/cephfs/test_volumes.py

index 62bc18165d656c3dc95892a32590c2ecb353d25c..9570c3ef5d1957ba6d08468eb412216b18dde341 100644 (file)
@@ -456,6 +456,8 @@ class TestVolumes(CephFSTestCase):
         That the volume can only be removed when --yes-i-really-mean-it is used
         and verify that the deleted volume is not listed anymore.
         """
+        for m in self.mounts:
+            m.umount_wait()
         try:
             self._fs_cmd("volume", "rm", self.volname)
         except CommandFailedError as ce:
@@ -514,6 +516,8 @@ class TestVolumes(CephFSTestCase):
         That the arbitrary pool added to the volume out of band is removed
         successfully on volume removal.
         """
+        for m in self.mounts:
+            m.umount_wait()
         new_pool = "new_pool"
         # add arbitrary data pool
         self.fs.add_data_pool(new_pool)
@@ -534,6 +538,8 @@ class TestVolumes(CephFSTestCase):
         That the volume can only be removed when mon_allowd_pool_delete is set
         to true and verify that the pools are removed after volume deletion.
         """
+        for m in self.mounts:
+            m.umount_wait()
         self.config_set('mon', 'mon_allow_pool_delete', False)
         try:
             self._fs_cmd("volume", "rm", self.volname, "--yes-i-really-mean-it")