From: Venky Shankar <vshankar@redhat.com>
Date: Tue, 16 Jan 2024 04:19:39 +0000 (+0530)
Subject: qa: remove redundant test test_volumes.TestRenameCmd.test_rename_when_fs_is_online
X-Git-Tag: v19.1.0~403^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=ecf546f93b9676fab8df3dc606839a09600ccecd;p=ceph.git

qa: remove redundant test test_volumes.TestRenameCmd.test_rename_when_fs_is_online

See - https://github.com/ceph/ceph/pull/53899/files#r1382869363

Signed-off-by: Venky Shankar <vshankar@redhat.com>
---

diff --git a/qa/tasks/cephfs/test_volumes.py b/qa/tasks/cephfs/test_volumes.py
index f5894dbf79070..d86af3412dcf8 100644
--- a/qa/tasks/cephfs/test_volumes.py
+++ b/qa/tasks/cephfs/test_volumes.py
@@ -745,21 +745,6 @@ class TestRenameCmd(TestVolumesHelper):
         # data pool names unchanged
         self.assertCountEqual(orig_data_pool_names, list(self.fs.data_pools.values()))
 
-    def test_rename_when_fs_is_online(self):
-        for m in self.mounts:
-            m.umount_wait()
-        newvolname = self._gen_vol_name()
-
-        self.run_ceph_cmd(f'fs set {self.volname} refuse_client_session true')
-        self.negtest_ceph_cmd(
-            args=(f'fs volume rename {self.volname} {newvolname} '
-                   '--yes-i-really-mean-it'),
-            errmsgs=(f"CephFS '{self.volname}' is not offline. Before "
-                      "renaming a CephFS, it must be marked as down. See "
-                      "`ceph fs fail`."),
-            retval=errno.EPERM)
-        self.run_ceph_cmd(f'fs set {self.volname} refuse_client_session false')
-
     def test_rename_when_clients_arent_refused(self):
         newvolname = self._gen_vol_name()
         for m in self.mounts: