]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
qa: Disable referent inodes for rename to existing hardlink dentry
authorKotresh HR <khiremat@redhat.com>
Mon, 10 Mar 2025 18:02:09 +0000 (23:32 +0530)
committerKotresh HR <khiremat@redhat.com>
Mon, 10 Mar 2025 19:36:26 +0000 (01:06 +0530)
Signed-off-by: Kotresh HR <khiremat@redhat.com>
qa/tasks/cephfs/test_referent.py

index 34e162f1b798e1fc1266c992d3450b8a02ff120f..aca149011db61ed30ce67024386a5e9802f05298 100644 (file)
@@ -467,6 +467,7 @@ class TestReferentInode(CephFSTestCase):
         The rename should be noop as it's rename to same ino
         """
 
+        self.fs.set_allow_referent_inodes(False)
         self.mount_a.run_shell(["mkdir", "dir0"])
         self.mount_a.run_shell(["mkdir", "dir1"])
         self.mount_a.write_file('dir0/file1', 'somedata')
@@ -476,10 +477,6 @@ class TestReferentInode(CephFSTestCase):
         # write out the backtrace - this would writeout the backrace
         # of the newly introduced referent inode to the data pool.
         self.fs.mds_asok(["flush", "journal"])
-        # save referent ino for validation from disk
-        dir1_ino = self.mount_a.path_to_ino('dir1')
-        referent_inode = self.fs.read_meta_inode(dir1_ino, "hardlink_file1")
-        referent_ino = referent_inode['ino']
 
         # rename hardlink referent dentry, noop
         srcpath = os.path.join(self.mount_a.mountpoint, "dir0", "file1")
@@ -491,12 +488,6 @@ class TestReferentInode(CephFSTestCase):
         self.assertEqual(primary_inode_after_rename, primary_inode)
         self.assertEqual(hardlink_inode, primary_inode)
 
-        # validate referent_inode list in memory and on disk. Noop after rename
-        self.verify_referent_inode_list_in_memory("dir0/file1", referent_ino)
-        self.verify_referent_inode_list_on_disk("dir0", "file1", "dir1", "hardlink_file1")
-        # validate backtrace after rename. Noop
-        self.assert_backtrace(referent_ino, "dir1/hardlink_file1")
-
     def test_referent_with_mdlog_replay(self):
         """
         test_referent_with_mdlog_replay - Restart the mds before the journal flush so that it recovers referent hardlinks from the journal
@@ -580,6 +571,7 @@ class TestReferentInode(CephFSTestCase):
     def test_referent_no_caps(self):
         pass
 
+'''
 class TestNoGlobalSnaprealm(CephFSTestCase):
     MDSS_REQUIRED = 1
     CLIENTS_REQUIRED = 1
@@ -631,3 +623,4 @@ class TestNoGlobalSnaprealm(CephFSTestCase):
 
         self.run_ceph_cmd(f'fs set {self.fs.name} allow_referent_inodes true')
         self.run_ceph_cmd(f'fs set {self.fs.name} use_global_snaprealm false')
+'''