]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
qa: remove shutdown lockdep not relevant to unit test
authorPatrick Donnelly <pdonnell@redhat.com>
Fri, 12 Mar 2021 04:58:53 +0000 (20:58 -0800)
committerPatrick Donnelly <pdonnell@redhat.com>
Wed, 31 Mar 2021 14:29:53 +0000 (07:29 -0700)
This is part of a fleet of commits trying to remove unnecessary mds_stop
calls in testing.

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

qa/tasks/cephfs/test_admin.py

index 4365f9c1f8680ef29aeee0314cb657508ba49403..408744ae63b047be6dd9074f68f6fc057512b300 100644 (file)
@@ -302,11 +302,6 @@ class TestConfigCommands(CephFSTestCase):
         out = self.mount_a.admin_socket(['config', 'get', test_key])
         self.assertEqual(out[test_key], test_val)
 
-        self.mount_a.write_n_mb("file.bin", 1);
-
-        # Implicitly asserting that things don't have lockdep error in shutdown
-        self.mount_a.umount_wait(require_clean=True)
-        self.fs.mds_stop()
 
     def test_mds_config_asok(self):
         test_key = "mds_max_purge_ops"
@@ -315,10 +310,6 @@ class TestConfigCommands(CephFSTestCase):
         out = self.fs.mds_asok(['config', 'get', test_key])
         self.assertEqual(out[test_key], test_val)
 
-        # Implicitly asserting that things don't have lockdep error in shutdown
-        self.mount_a.umount_wait(require_clean=True)
-        self.fs.mds_stop()
-
     def test_mds_config_tell(self):
         test_key = "mds_max_purge_ops"
         test_val = "123"
@@ -331,10 +322,6 @@ class TestConfigCommands(CephFSTestCase):
         out = self.fs.mds_asok(['config', 'get', test_key])
         self.assertEqual(out[test_key], test_val)
 
-        # Implicitly asserting that things don't have lockdep error in shutdown
-        self.mount_a.umount_wait(require_clean=True)
-        self.fs.mds_stop()
-
 
 class TestMirroringCommands(CephFSTestCase):
     CLIENTS_REQUIRED = 1