qa: no longer need to explicitly enable multimds

Signed-off-by: John Spray <john.spray@redhat.com>

diff --git a/qa/tasks/ceph.py b/qa/tasks/ceph.py
index fa43530a6f5cffd72b16110e927c7550873bc57a..5363697168d885e861f6bb0724fcbeddb6c67cce 100644 (file)
--- a/qa/tasks/ceph.py
+++ b/qa/tasks/ceph.py
@@ -346,7 +346,6 @@ def cephfs_setup(ctx, config):
         all_roles = [item for remote_roles in mdss.remotes.values() for item in remote_roles]
         num_active = len([r for r in all_roles if is_active_mds(r)])
 
-        fs.set_allow_multimds(True)
         fs.set_max_mds(num_active)
         fs.set_allow_dirfrags(True)
 

diff --git a/qa/tasks/cephfs/filesystem.py b/qa/tasks/cephfs/filesystem.py
index 01e1ca588c1e8ee3076ad5f9b0fd10efa44a63c5..0d3fc547b7e2b6f0b9e3f3bffca50f0300d63b85 100644 (file)
--- a/qa/tasks/cephfs/filesystem.py
+++ b/qa/tasks/cephfs/filesystem.py
@@ -425,9 +425,6 @@ class Filesystem(MDSCluster):
     def set_allow_dirfrags(self, yes):
         self.mon_manager.raw_cluster_cmd("fs", "set", self.name, "allow_dirfrags", str(yes).lower(), '--yes-i-really-mean-it')
 
-    def set_allow_multimds(self, yes):
-        self.mon_manager.raw_cluster_cmd("fs", "set", self.name, "allow_multimds", str(yes).lower(), '--yes-i-really-mean-it')
-
     def get_pgs_per_fs_pool(self):
         """
         Calculate how many PGs to use when creating a pool, in order to avoid raising any

diff --git a/qa/tasks/cephfs/test_exports.py b/qa/tasks/cephfs/test_exports.py
index 80f876ebca301d749969106ec8692784012a46b3..df16e1939734292ea7f7e1259197fbdffe8db3ca 100644 (file)
--- a/qa/tasks/cephfs/test_exports.py
+++ b/qa/tasks/cephfs/test_exports.py
@@ -23,7 +23,6 @@ class TestExports(CephFSTestCase):
         raise RuntimeError("rank {0} failed to reach desired subtree state", rank)
 
     def test_export_pin(self):
-        self.fs.set_allow_multimds(True)
         self.fs.set_max_mds(2)
 
         status = self.fs.status()

diff --git a/qa/tasks/cephfs/test_failover.py b/qa/tasks/cephfs/test_failover.py
index faefec458d6107bece8dca1bad283e149e13035a..1bb2ff7322b339b5223d882009211e56b69d1049 100644 (file)
--- a/qa/tasks/cephfs/test_failover.py
+++ b/qa/tasks/cephfs/test_failover.py
@@ -257,7 +257,6 @@ class TestStandbyReplay(CephFSTestCase):
 
         # Create FS alpha and get mds_a to come up as active
         fs_a = self.mds_cluster.newfs("alpha")
-        fs_a.set_allow_multimds(True)
         fs_a.set_max_mds(2)
 
         self.mds_cluster.mds_restart(mds_a)
@@ -412,8 +411,6 @@ class TestMultiFilesystems(CephFSTestCase):
     def test_grow_shrink(self):
         # Usual setup...
         fs_a, fs_b = self._setup_two()
-        fs_a.set_allow_multimds(True)
-        fs_b.set_allow_multimds(True)
 
         # Increase max_mds on fs_b, see a standby take up the role
         fs_b.set_max_mds(2)
@@ -570,10 +567,8 @@ class TestMultiFilesystems(CephFSTestCase):
 
         # Create two filesystems which should have two ranks each
         fs_a = self.mds_cluster.newfs("alpha")
-        fs_a.set_allow_multimds(True)
 
         fs_b = self.mds_cluster.newfs("bravo")
-        fs_b.set_allow_multimds(True)
 
         fs_a.set_max_mds(2)
         fs_b.set_max_mds(2)

diff --git a/qa/tasks/cephfs/test_journal_repair.py b/qa/tasks/cephfs/test_journal_repair.py
index 1b03afc0fc49b9adcc8300c77820e09625fed2b3..62cbbb0684a76a19b3fc4fbd75aa5d48c187f901 100644 (file)
--- a/qa/tasks/cephfs/test_journal_repair.py
+++ b/qa/tasks/cephfs/test_journal_repair.py
@@ -160,7 +160,6 @@ class TestJournalRepair(CephFSTestCase):
         """
 
         # Set max_mds to 2
-        self.fs.set_allow_multimds(True)
         self.fs.set_max_mds(2)
 
         # See that we have two active MDSs

diff --git a/qa/tasks/cephfs/test_mantle.py b/qa/tasks/cephfs/test_mantle.py
index 8e0526332e65ef5592e0287cb56c62891c2ce213..6cd86ad12dcff41244f4de42fda2117875ada5c1 100644 (file)
--- a/qa/tasks/cephfs/test_mantle.py
+++ b/qa/tasks/cephfs/test_mantle.py
@@ -9,7 +9,6 @@ success = "mantle balancer version changed: "
 class TestMantle(CephFSTestCase):
     def start_mantle(self):
         self.wait_for_health_clear(timeout=30)
-        self.fs.set_allow_multimds(True)
         self.fs.set_max_mds(2)
         self.wait_until_equal(lambda: len(self.fs.get_active_names()), 2, 30,
                               reject_fn=lambda v: v > 2 or v < 1)

diff --git a/qa/tasks/cephfs/test_sessionmap.py b/qa/tasks/cephfs/test_sessionmap.py
index e9b4b646d8f184ac3638c15d64044303c6e39af2..9d12ab6d8bee6dbf345ab470547f896b82f4983f 100644 (file)
--- a/qa/tasks/cephfs/test_sessionmap.py
+++ b/qa/tasks/cephfs/test_sessionmap.py
@@ -99,7 +99,6 @@ class TestSessionMap(CephFSTestCase):
         self.fs.wait_for_daemons()
 
         # I would like two MDSs, so that I can do an export dir later
-        self.fs.set_allow_multimds(True)
         self.fs.set_max_mds(2)
         self.fs.wait_for_daemons()
 

diff --git a/qa/tasks/cephfs/test_strays.py b/qa/tasks/cephfs/test_strays.py
index 3335d89dd90023ceff5d77b878a9e28f2a2f4922..6e70243a36b937d6be350bef287377625408de9d 100644 (file)
--- a/qa/tasks/cephfs/test_strays.py
+++ b/qa/tasks/cephfs/test_strays.py
@@ -492,7 +492,6 @@ class TestStrays(CephFSTestCase):
 
     def _setup_two_ranks(self):
         # Set up two MDSs
-        self.fs.set_allow_multimds(True)
         self.fs.set_max_mds(2)
 
         # See that we have two active MDSs