Skip to content

Commit

Permalink
Merge pull request #50728 from lxbsz/wip-59195
Browse files Browse the repository at this point in the history
qa: switch to use the merge fragment for fscrypt
  • Loading branch information
lxbsz committed Jun 12, 2023
2 parents 680201c + 04fab9b commit 4a60f67
Show file tree
Hide file tree
Showing 19 changed files with 8 additions and 26 deletions.
1 change: 1 addition & 0 deletions qa/suites/fs/fscrypt/mount
1 change: 0 additions & 1 deletion qa/suites/fs/fscrypt/mount/.qa

This file was deleted.

1 change: 0 additions & 1 deletion qa/suites/fs/fscrypt/mount/kclient/.qa

This file was deleted.

1 change: 0 additions & 1 deletion qa/suites/fs/fscrypt/mount/kclient/mount-syntax/.qa

This file was deleted.

1 change: 0 additions & 1 deletion qa/suites/fs/fscrypt/mount/kclient/mount-syntax/v1.yaml

This file was deleted.

1 change: 0 additions & 1 deletion qa/suites/fs/fscrypt/mount/kclient/mount.yaml

This file was deleted.

Empty file.
1 change: 0 additions & 1 deletion qa/suites/fs/fscrypt/mount/kclient/overrides/.qa

This file was deleted.

1 change: 0 additions & 1 deletion qa/suites/fs/fscrypt/mount/kclient/overrides/distro/.qa

This file was deleted.

This file was deleted.

This file was deleted.

File renamed without changes.
7 changes: 7 additions & 0 deletions qa/suites/fs/fscrypt/tasks/0-client.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
teuthology:
postmerge:
# Once can we make sure the distro kernels have included the fscrypt feature
# or the ceph-fuse have supported the fscrypt feature we can remove this
# restriction.
- if not is_kupstream() then reject() end
task:
File renamed without changes.
17 changes: 0 additions & 17 deletions qa/tasks/cephfs/test_fscrypt.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,22 +12,7 @@ class TestFscrypt(XFSTestsDev):
def setup_xfsprogs_devs(self):
self.install_xfsprogs = True

def require_kernel_mount(self):
from tasks.cephfs.fuse_mount import FuseMount
from tasks.cephfs.kernel_mount import KernelMount

# TODO: make xfstests-dev compatible with ceph-fuse. xfstests-dev
# remounts CephFS before running tests using kernel, so ceph-fuse
# mounts are never actually tested.
if isinstance(self.mount_a, FuseMount):
self.skipTest('Requires kernel client; xfstests-dev not '\
'compatible with ceph-fuse ATM.')
elif isinstance(self.mount_a, KernelMount):
log.info('client is kernel mounted')

def test_fscrypt_encrypt(self):
self.require_kernel_mount()

# XXX: check_status is set to False so that we can check for command's
# failure on our own (since this command doesn't set right error code
# and error message in some cases) and print custom log messages
Expand All @@ -51,8 +36,6 @@ def test_fscrypt_encrypt(self):
self.assertIn('Passed all 26 tests', stdout)

def test_fscrypt_dummy_encryption_with_quick_group(self):
self.require_kernel_mount()

self.write_local_config('test_dummy_encryption')

# XXX: check_status is set to False so that we can check for command's
Expand Down

0 comments on commit 4a60f67

Please sign in to comment.