qa: remove unnecessary background job cleanup

This is done in CephFSTestCase.tearDown.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>

diff --git a/qa/tasks/cephfs/test_client_recovery.py b/qa/tasks/cephfs/test_client_recovery.py
index 245e25f34df5f38f14af33ae382128e9fab60b55..b5cd20cad97ac2573cc3dddb1c3a29ef2684e639 100644 (file)
--- a/qa/tasks/cephfs/test_client_recovery.py
+++ b/qa/tasks/cephfs/test_client_recovery.py
@@ -377,7 +377,7 @@ class TestClientRecovery(CephFSTestCase):
         """
 
         flockable = self._is_flockable()
-        lock_holder = self.mount_a.lock_background(do_flock=flockable)
+        self.mount_a.lock_background(do_flock=flockable)
 
         self.mount_b.wait_for_visible("background_file-2")
         self.mount_b.check_filelock(do_flock=flockable)
@@ -387,8 +387,6 @@ class TestClientRecovery(CephFSTestCase):
 
         self.mount_b.check_filelock(do_flock=flockable)
 
-        self.mount_a._kill_background(lock_holder)
-
     def test_filelock_eviction(self):
         """
         Check that file lock held by evicted client is given to

diff --git a/qa/tasks/cephfs/test_misc.py b/qa/tasks/cephfs/test_misc.py
index 72468a81361ae6a4c6afb80b648db9eb271b1f1e..410cb9d6a47d46d9527690d6066caf7c95aed824 100644 (file)
--- a/qa/tasks/cephfs/test_misc.py
+++ b/qa/tasks/cephfs/test_misc.py
@@ -69,15 +69,13 @@ class TestMisc(CephFSTestCase):
 
         # create a file and hold it open. MDS will issue CEPH_CAP_EXCL_*
         # to mount_a
-        p = self.mount_a.open_background("testfile")
+        self.mount_a.open_background("testfile")
         self.mount_b.wait_for_visible("testfile")
 
         # this triggers a lookup request and an open request. The debug
         # code will check if lookup/open reply contains xattrs
         self.mount_b.run_shell(["cat", "testfile"])
 
-        self.mount_a.kill_background(p)
-
     def test_root_rctime(self):
         """
         Check that the root inode has a non-default rctime on startup.

diff --git a/qa/tasks/cephfs/test_openfiletable.py b/qa/tasks/cephfs/test_openfiletable.py
index eff6b50932e4ac78614f90e14391c2e7c1938a94..2672d801d0a3dfc4746cf9dc8236c98915fd7287 100644 (file)
--- a/qa/tasks/cephfs/test_openfiletable.py
+++ b/qa/tasks/cephfs/test_openfiletable.py
@@ -31,7 +31,7 @@ class OpenFileTable(CephFSTestCase):
         file_count = 8
         for i in range(0, file_count):
             filename = "open_file{}".format(i)
-            p = self.mount_a.open_background(filename)
+            self.mount_a.open_background(filename)
             self.mount_a.write_n_mb(filename, size_mb)
 
         time.sleep(10)
@@ -44,9 +44,6 @@ class OpenFileTable(CephFSTestCase):
 
         self.fs.radosm(["stat", "mds0_openfiles.1"])
 
-        # Now close the file
-        self.mount_a.kill_background(p)
-
     def test_perf_counters(self):
         """
         Opening a file should increment omap_total_updates by 1.

diff --git a/qa/tasks/cephfs/test_strays.py b/qa/tasks/cephfs/test_strays.py
index 274cc238c325383a91996c438bf7f7efef40aea1..2fe5a6aafc065bf0bde21768bb999fdcb1d7890a 100644 (file)
--- a/qa/tasks/cephfs/test_strays.py
+++ b/qa/tasks/cephfs/test_strays.py
@@ -712,7 +712,7 @@ touch pin/placeholder
         self._force_migrate("pin")
 
         # Hold the dir open so it cannot be purged
-        p = self.mount_a.open_dir_background("pin/to-be-unlinked")
+        self.mount_a.open_dir_background("pin/to-be-unlinked")
 
         # Unlink the dentry
         self.mount_a.run_shell(["rmdir", "pin/to-be-unlinked"])
@@ -736,8 +736,6 @@ touch pin/placeholder
         # self.assertEqual(self.get_mdc_stat("strays_created", mds_id=rank_0_id), 1)
         # https://github.com/ceph/ceph/pull/44335#issuecomment-1125940158
 
-        self.mount_a.kill_background(p)
-
     def assert_backtrace(self, ino, expected_path):
         """
         Assert that the backtrace in the data pool for an inode matches