From: Rishabh Dave <ridave@redhat.com>
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 <ridave@redhat.com>
---

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)