]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
qa: remove redundant rmr
authorPatrick Donnelly <pdonnell@redhat.com>
Tue, 6 Oct 2020 16:37:21 +0000 (09:37 -0700)
committerPatrick Donnelly <pdonnell@redhat.com>
Fri, 23 Oct 2020 03:11:22 +0000 (20:11 -0700)
The mount.cleanup method will remove the mount point. This `rm -rf` will
always fail (with exit status 0).

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 8e7a89009c41570e9f87f57d92d519b7c49d7706)

Conflicts:
qa/tasks/cephfs/fuse_mount.py

        Notes: convert to cleanup call.

qa/tasks/cephfs/fuse_mount.py

index 1f5fe1b0b663583deaf1928e781c782e655e59da..4e0a3d553bcee43490eb452b4956c613fdfed6f2 100644 (file)
@@ -368,7 +368,6 @@ class FuseMount(CephFSMount):
                 cwd=self.test_dir,
                 stderr=stderr,
                 timeout=(60*5),
-                check_status=False,
             )
         except CommandFailedError:
             if b"No such file or directory" in stderr.getvalue():
@@ -410,16 +409,7 @@ class FuseMount(CephFSMount):
             except CommandFailedError:
                 pass
 
-        # Indiscriminate, unlike the touchier cleanup()
-        self.client_remote.run(
-            args=[
-                'rm',
-                '-rf',
-                self.mountpoint,
-            ],
-            cwd=self.test_dir,
-            timeout=(60*5)
-        )
+        return self.cleanup()
 
     def _asok_path(self):
         return "/var/run/ceph/ceph-client.{0}.*.asok".format(self.client_id)