From ea78489e2fc0083f46d8e6c932fcd09ba822939f Mon Sep 17 00:00:00 2001 From: Andreas Dilger Date: Fri, 15 Oct 2021 11:31:57 -0600 Subject: [PATCH] LU-12784 tests: fix large_xattr_enabled() for ZFS Fix large_xattr_enabled() check for ZFS filesystems, since bash functions return "0" for true. Otherwise, all ZFS tests that check large_xattr_enabled() will be skipped. Fixes: 84097792f56c ("LU-12784 llite: limit max xattr size by kernel value") Signed-off-by: Andreas Dilger Change-Id: Ie566244c6b1f46b947a96331e7623b9b863ebbe5 Reviewed-on: https://review.whamcloud.com/45264 Tested-by: jenkins Tested-by: Maloo Reviewed-by: Arshad Hussain Reviewed-by: Elena Gryaznova Reviewed-by: Oleg Drokin --- lustre/tests/test-framework.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lustre/tests/test-framework.sh b/lustre/tests/test-framework.sh index 3d0398f..b5b87ec 100755 --- a/lustre/tests/test-framework.sh +++ b/lustre/tests/test-framework.sh @@ -8649,7 +8649,7 @@ get_block_count() { # ldiskfs xattrs over one block in size. Allow both the historical # Lustre feature name (large_xattr) and the upstream name (ea_inode). large_xattr_enabled() { - [[ $(facet_fstype $SINGLEMDS) == zfs ]] && return 1 + [[ $(facet_fstype $SINGLEMDS) == zfs ]] && return 0 local mds_dev=$(mdsdevname ${SINGLEMDS//mds/}) -- 1.8.3.1