tasks/cephfs: skip TestVolumeClient.test_evict on kcephfs

This test was probably buggy and only happened to work
with ceph-fuse, because it expects the MDS to immediately
respond to updates to the client's auth caps, but that
doesn't happen.

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

diff --git a/suites/kcephfs/recovery/tasks/volume-client.yaml b/suites/kcephfs/recovery/tasks/volume-client.yaml
index e8c850a05cdd65e27fffb02be34deaf1f32ff001..d738eebf12f6ad898119833a8b903d27373eb5ff 100644 (file)
--- a/suites/kcephfs/recovery/tasks/volume-client.yaml
+++ b/suites/kcephfs/recovery/tasks/volume-client.yaml
@@ -7,5 +7,6 @@ overrides:
 
 tasks:
   - cephfs_test_runner:
+      fail_on_skip: false
       modules:
         - tasks.cephfs.test_volume_client

diff --git a/tasks/cephfs/test_volume_client.py b/tasks/cephfs/test_volume_client.py
index 1549ce64f4e7da0629c581bb56c832b6e18ab6f5..101acf50853b3626b4e25eb37b3307ab69d8caf7 100644 (file)
--- a/tasks/cephfs/test_volume_client.py
+++ b/tasks/cephfs/test_volume_client.py
@@ -4,6 +4,7 @@ import time
 import os
 from textwrap import dedent
 from tasks.cephfs.cephfs_test_case import CephFSTestCase
+from tasks.cephfs.fuse_mount import FuseMount
 from teuthology.exceptions import CommandFailedError
 
 log = logging.getLogger(__name__)
@@ -432,6 +433,9 @@ vc.disconnect()
         path it has mounted.
         """
 
+        if not isinstance(self.mount_a, FuseMount):
+            self.skipTest("Requires FUSE client to inject client metadata")
+
         # mounts[1] would be used as handle for driving VolumeClient. mounts[2]
         # and mounts[3] would be used as guests to mount the volumes/shares.