From: Venky Shankar <vshankar@redhat.com>
Date: Tue, 18 Mar 2025 08:56:45 +0000 (+0000)
Subject: Revert "qa/cephfs-data-scan: Validate referent hardlink recovery"
X-Git-Tag: testing/wip-vshankar-testing-20260223.155722^2~39
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=6252a76db1d06c5b7aefd9ac2a37c4b9139e7a95;p=ceph-ci.git

Revert "qa/cephfs-data-scan: Validate referent hardlink recovery"

This reverts commit ba5df7ae75ad403d15881d2c9526eb797b0de77d.

Signed-off-by: Venky Shankar <vshankar@redhat.com>
---

diff --git a/qa/tasks/cephfs/test_data_scan.py b/qa/tasks/cephfs/test_data_scan.py
index b2690c2784c..9531bff0bd2 100644
--- a/qa/tasks/cephfs/test_data_scan.py
+++ b/qa/tasks/cephfs/test_data_scan.py
@@ -164,44 +164,6 @@ class SymlinkWorkload(Workload):
         self.assert_equal(target, "symdir/onemegs")
         return self._errors
 
-class HardlinkWorkload(Workload):
-    """
-    Hardlink file, check that it gets recovered as hardlink
-    with referent inode pointing to the remote inode
-    """
-    def write(self):
-        self._filesystem.set_allow_referent_inodes(True);
-        self._mount.run_shell(["mkdir", "hardlink_dir"])
-        self._mount.write_n_mb("hardlink_dir/file1", 1)
-        self._mount.run_shell(["ln", "hardlink_dir/file1", "hardlink_dir/hl_file1"])
-        self._mount.run_shell(["ln", "hardlink_dir/file1", "hardlink_dir/hl_file2"])
-        self._filesystem.set_allow_referent_inodes(False);
-
-    def validate(self):
-        self._mount.run_shell(["sudo", "ls", "hardlink_dir"], omit_sudo=False)
-        st_file1 = self._mount.lstat("hardlink_dir/file1")
-        st_hl_file1 = self._mount.lstat("hardlink_dir/hl_file1")
-        st_hl_file2 = self._mount.lstat("hardlink_dir/hl_file2")
-        # link count check
-        self.assert_equal(3, st_file1['st_nlink'])
-        self.assert_equal(3, st_hl_file1['st_nlink'])
-        self.assert_equal(3, st_hl_file2['st_nlink'])
-        # ino number check
-        self.assert_equal(st_file1['st_ino'], st_hl_file1['st_ino'])
-        self.assert_equal(st_file1['st_ino'], st_hl_file2['st_ino'])
-        # reverse link - referent inode list
-        file1_ino = self._mount.path_to_ino("hardlink_dir/file1", follow_symlinks=False)
-        file1_inode_dump = self._filesystem.mds_asok(['dump', 'inode', hex(file1_ino)])
-        self.assert_equal(len(file1_inode_dump['referent_inodes']), 2)
-        # remote link
-        referent1_ino = file1_inode_dump['referent_inodes'][0]
-        referent1_inode_dump = self._filesystem.mds_asok(['dump', 'inode', hex(referent1_ino)])
-        self.assert_equal(file1_ino, referent1_inode_dump['remote_ino'])
-        referent2_ino = file1_inode_dump['referent_inodes'][1]
-        referent2_inode_dump = self._filesystem.mds_asok(['dump', 'inode', hex(referent2_ino)])
-        self.assert_equal(file1_ino, referent2_inode_dump['remote_ino'])
-        return self._errors
-
 class NestedDirWorkload(Workload):
     """
     Nested directories, one is lost.
@@ -579,9 +541,6 @@ class TestDataScan(CephFSTestCase):
     def test_rebuild_symlink(self):
         self._rebuild_metadata(SymlinkWorkload(self.fs, self.mount_a))
 
-    def test_rebuild_hardlink(self):
-        self._rebuild_metadata(HardlinkWorkload(self.fs, self.mount_a))
-
     def test_rebuild_nested(self):
         self._rebuild_metadata(NestedDirWorkload(self.fs, self.mount_a))