From: Patrick Donnelly <pdonnell@redhat.com>
Date: Wed, 3 Apr 2024 13:08:19 +0000 (-0400)
Subject: qa: remove unnecessary background job cleanup
X-Git-Tag: v20.0.0~2106^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=98f161b21034f54ab4b105e3eb9c1a939a48c94e;p=ceph.git

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 245e25f34df..b5cd20cad97 100644
--- 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 72468a81361..410cb9d6a47 100644
--- 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 eff6b50932e..2672d801d0a 100644
--- 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 274cc238c32..2fe5a6aafc0 100644
--- 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