tasks/cephfs: use kill in test_reconnect_eviction

Previously relied on client being able to unmount
while the MDS was offline, which is not necessarily
so.  Use kill instead.

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

diff --git a/tasks/cephfs/test_client_recovery.py b/tasks/cephfs/test_client_recovery.py
index 8839345e96075f0020cd16142394489c8bb9b288..4b4085a915704553da119b5ffd49a5f7659c37a2 100644 (file)
--- a/tasks/cephfs/test_client_recovery.py
+++ b/tasks/cephfs/test_client_recovery.py
@@ -176,11 +176,13 @@ class TestClientRecovery(CephFSTestCase):
     def test_reconnect_eviction(self):
         # Eviction during reconnect
         # =========================
+        mount_a_client_id = self.mount_a.get_global_id()
+
         self.fs.mds_stop()
         self.fs.mds_fail()
 
-        mount_a_client_id = self.mount_a.get_global_id()
-        self.mount_a.umount_wait(force=True)
+        # The mount goes away while the MDS is offline
+        self.mount_a.kill()
 
         self.fs.mds_restart()
 
@@ -201,6 +203,9 @@ class TestClientRecovery(CephFSTestCase):
                             evict_til_active
                         ))
 
+        # We killed earlier so must clean up before trying to use again
+        self.mount_a.kill_cleanup()
+
         # Bring the client back
         self.mount_a.mount()
         self.mount_a.wait_until_mounted()