From: Rishabh Dave Date: Thu, 16 May 2024 07:00:49 +0000 (+0530) Subject: qa/cephfs: block buggy tests in test_admin.py X-Git-Tag: v20.0.0~1911^2 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=b7d07700d66c2a3c86fe985d4d5c83a635c260ac;p=ceph.git qa/cephfs: block buggy tests in test_admin.py Block test_idem_unaffected_root_squash temporarily and test_multifs_single_path_rootsquash. This test fails due to a known bug. Block it temporarily so that test_admin.py can run fully and PRs under QA can be tested fully. Otherwise, this test fails and that halts test_admin.py, which leaves the PR partially untested. This failure is then seen as an unrelated failure which lets the buggy code get merged. This has happened recently. Signed-off-by: Rishabh Dave --- diff --git a/qa/tasks/cephfs/test_admin.py b/qa/tasks/cephfs/test_admin.py index 6a84aad6f90ae..a56409c0909e5 100644 --- a/qa/tasks/cephfs/test_admin.py +++ b/qa/tasks/cephfs/test_admin.py @@ -839,6 +839,9 @@ class TestRenameCommand(TestAdminCommands): """ That renaming a non-existent file system fails. """ + self.skipTest('This test is broken ATM; see ' + 'https://tracker.ceph.com/issues/66088') + self.run_ceph_cmd(f'fs fail {self.fs.name}') self.run_ceph_cmd(f'fs set {self.fs.name} refuse_client_session true') sleep(5) @@ -1444,6 +1447,9 @@ class TestFsAuthorize(CephFSTestCase): """ Test root_squash with multi fs """ + self.skipTest('this test is broken ATM, see: ' + 'https://tracker.ceph.com/issues/66076.') + self.fs1 = self.fs self.fs2 = self.mds_cluster.newfs('testcephfs2') self.mount_b.remount(cephfs_name=self.fs2.name) @@ -1604,6 +1610,9 @@ class TestFsAuthorize(CephFSTestCase): That fs authorize command works on filesystems with names having [_.-] characters """ + self.skipTest('this test is broken ATM, see: ' + 'https://tracker.ceph.com/issues/66077') + self.mount_a.umount_wait(require_clean=True) self.mds_cluster.delete_all_filesystems() fs_name = "cephfs-_." @@ -1948,6 +1957,9 @@ class TestFsAuthorizeUpdate(CephFSTestCase): caps mon = "allow r fsname=a" caps osd = "allow rw tag cephfs data=a" """ + self.skipTest('this test is broken ATM, see ' + 'https://tracker.ceph.com/issues/65808') + PERM, PATH = 'rw', 'dir1' self.mount_a.run_shell(f'mkdir {PATH}') self.captester = CapTester(self.mount_a, PATH)