On 12/9/24 8:49 AM, Christoph Hellwig wrote:
+_supported_fs btrfs xfs
This is not how generic tests works. They need a helper in common
to check if a feature is present or not, which then probes for the
feature.
Someone who has knowledge of the interfaces and facilities that
duperemove depends on will need to construct such a helper. I can
only guess.
We should figure out what the issue is here first before
doing hacks like that as well.
To be clear, there are two issues:
1. Dave's report of unreliability on filesystems where it still makes
sense to use duperemove, and
2. How to disable this test on filesystems (like NFS) where duperemove
is not supported or where the test is not meaningful. The current check
for the presence of the duperemove executable is IMO inadequate.
For now I've expunged these three tests from the NFSD CI workflow.
--
Chuck Lever
