From: John L. Hammond <john.hammond@intel.com>
Date: Wed, 15 Nov 2017 16:05:57 +0000 (-0600)
Subject: LU-10038 test: improve error reporting in sanity test_133g()
X-Git-Tag: 2.10.56~28
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=e5eaaff6e378b8c95d0a809f4dd3b4817d9fd492;p=fs%2Flustre-release.git

LU-10038 test: improve error reporting in sanity test_133g()

In sanity test_133g() remove the output redirection to /dev/null so
that errors from do_facet or find go to the test logs. Suppress error
messages in badarea_io.c.

Test-Parameters: trivial envdefinitions=ONLY=133 testlist=sanity,sanity,sanity,sanity,sanity,sanity,sanity,sanity,sanity,sanity
Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: Ia51ccdacb995ff51a6467cbdd280c8965cfc5f47
Reviewed-on: https://review.whamcloud.com/30105
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
---

diff --git a/lustre/tests/badarea_io.c b/lustre/tests/badarea_io.c
index b926ce9..b9da6db 100644
--- a/lustre/tests/badarea_io.c
+++ b/lustre/tests/badarea_io.c
@@ -4,6 +4,9 @@
 #include <fcntl.h>
 #include <unistd.h>
 
+#undef perror
+#define perror(str) ((void)0)
+
 int main(int argc, char **argv)
 {
 	int rc;
diff --git a/lustre/tests/sanity.sh b/lustre/tests/sanity.sh
index d8b62ae..3ee0492 100755
--- a/lustre/tests/sanity.sh
+++ b/lustre/tests/sanity.sh
@@ -10063,7 +10063,7 @@ test_133g() {
 		-type f \
 		-not -name force_lbug \
 		-not -name changelog_mask \
-		-exec badarea_io '{}' \; &> /dev/null ||
+		-exec badarea_io '{}' \; ||
 		error "find $proc_dirs failed"
 
 	local facet
@@ -10078,7 +10078,7 @@ test_133g() {
 			-type f \
 			-not -name force_lbug \
 			-not -name changelog_mask \
-			-exec badarea_io '{}' \\\; &> /dev/null ||
+			-exec badarea_io '{}' \\\; ||
 				error "$facet find $facet_proc_dirs failed"
 	done