generic/{159,160}: require chattr instead of lsattr
authorAmir Goldstein <amir73il@gmail.com>
Sun, 6 May 2018 06:36:08 +0000 (09:36 +0300)
committerEryu Guan <guaneryu@gmail.com>
Fri, 11 May 2018 10:40:04 +0000 (18:40 +0800)
The tests _require_test_lsattr, but don't actually use lsattr. They
use chattr +i/-i, so require the appropriate command.

_require_test_lsattr checks the FS_IOC_GETFLAGS ioctl on a directory
and that is not supported in overlayfs. _require_chattr checks the
ioctl on a file, which is supported in overlayfs, so this change
makes the tests run and pass on overlayfs.

Signed-off-by: Amir Goldstein <amir73il@gmail.com>
Reviewed-by: Eryu Guan <guaneryu@gmail.com>
Signed-off-by: Eryu Guan <guaneryu@gmail.com>
tests/generic/159
tests/generic/160

index 7e52d5c..f517b55 100755 (executable)
@@ -43,7 +43,7 @@ _cleanup()
 
 # real QA test starts here
 _supported_os Linux
-_require_test_lsattr
+_require_chattr i
 _require_test_reflink
 
 rm -f $seqres.full
index 409b15e..cd1044d 100755 (executable)
@@ -43,7 +43,7 @@ _cleanup()
 
 # real QA test starts here
 _supported_os Linux
-_require_test_lsattr
+_require_chattr i
 _require_test_dedupe
 
 rm -f $seqres.full