From f44a4604921f2450aab8997d80ba21ff1282b5b7 Mon Sep 17 00:00:00 2001 From: James Nunez Date: Fri, 26 Jul 2013 11:43:42 -0600 Subject: [PATCH] LU-3648 test: large_xattr_enable doesn't detect large xattrs The large_xattr_enable function looks for "large_xattr" from the output of dumpe2fs to determine if large xattrs is enabled. When large extended attributes are enabled, the file system flag "ea_inode" is set and not "large_xattr". Since "large_xattr" may be used in the future for this attribute, the fix is to grep for both "ea_inode" and "large_xattr". Signed-off-by: James Nunez Change-Id: Iebe744e7fc70c878f1897b853e37f85cd6e59031 Reviewed-on: http://review.whamcloud.com/7133 Tested-by: Hudson Reviewed-by: Keith Mannthey Reviewed-by: Jian Yu Tested-by: Maloo Reviewed-by: Oleg Drokin --- lustre/tests/test-framework.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lustre/tests/test-framework.sh b/lustre/tests/test-framework.sh index 0da57f6..963db0b 100644 --- a/lustre/tests/test-framework.sh +++ b/lustre/tests/test-framework.sh @@ -6233,7 +6233,8 @@ large_xattr_enabled() { local mds_dev=$(mdsdevname ${SINGLEMDS//mds/}) - do_facet $SINGLEMDS "$DUMPE2FS -h $mds_dev 2>&1 | grep -q large_xattr" + do_facet $SINGLEMDS "$DUMPE2FS -h $mds_dev 2>&1 | + grep -E -q '(ea_inode|large_xattr)'" return ${PIPESTATUS[0]} } -- 1.8.3.1