From: Jian Yu <jian.yu@intel.com>
Date: Fri, 7 Jul 2017 06:16:06 +0000 (-0700)
Subject: LU-9663 tests: replace “ls -s” with “stat -c '%s'” to get file size
X-Git-Tag: 2.10.51~49
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=45b8c6cf6916cf012ad0512f9cf249c152b45236;p=fs%2Flustre-release.git

LU-9663 tests: replace “ls -s” with “stat -c '%s'” to get file size

This patch improves acl/2561{,_zfs}.test to use “stat -c '%s'” to
get file size instead of using “ls -s”, which prints the allocated
size of the file in blocks.

Test-Parameters: trivial \
envdefinitions=ONLY=103 \
mdtfilesystemtype=zfs ostfilesystemtype=zfs \
testlist=sanity

Change-Id: I8bfea5197102baeece244e20731c90172c5cc2f1
Signed-off-by: Jian Yu <jian.yu@intel.com>
Reviewed-on: https://review.whamcloud.com/27961
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Fan Yong <fan.yong@intel.com>
---

diff --git a/lustre/tests/acl/2561.test b/lustre/tests/acl/2561.test
index 073d1a0..ac38131 100644
--- a/lustre/tests/acl/2561.test
+++ b/lustre/tests/acl/2561.test
@@ -4,8 +4,8 @@ LU-2561 newly created file is same size as directory
 	$ cd 2561
 	$ getfacl --access . | setfacl -d -M- .
 	$ touch f1
-	$ ls -s f1
-	> 0 f1
+	$ stat -c '%s' f1
+	> 0
 	$ cd ..
 	$ rm -rf 2561
 
diff --git a/lustre/tests/acl/2561_zfs.test b/lustre/tests/acl/2561_zfs.test
index 8ab0657..ac38131 100644
--- a/lustre/tests/acl/2561_zfs.test
+++ b/lustre/tests/acl/2561_zfs.test
@@ -4,8 +4,8 @@ LU-2561 newly created file is same size as directory
 	$ cd 2561
 	$ getfacl --access . | setfacl -d -M- .
 	$ touch f1
-	$ ls -s f1
-	> 1 f1
+	$ stat -c '%s' f1
+	> 0
 	$ cd ..
 	$ rm -rf 2561
 
diff --git a/lustre/tests/acl/run b/lustre/tests/acl/run
index 6a86c22..ffffd5a 100755
--- a/lustre/tests/acl/run
+++ b/lustre/tests/acl/run
@@ -42,7 +42,7 @@ if (!defined($ARGV[0])) {
 open($testfile, $ARGV[0]) or die "Can't open file $ARGV[0]: $!";
 for (;;) {
   my $line = <$testfile>; $line_number++;
-  if (defined $line) {
+  if (defined $line && $line !~ /\%s/) {
     # Substitute %VAR and %{VAR} with environment variables.
     $line =~ s[%(?:(\w+)|\{(\w+)\})][$ENV{"$1$2"}]eg;
   }