From e5eaaff6e378b8c95d0a809f4dd3b4817d9fd492 Mon Sep 17 00:00:00 2001 From: "John L. Hammond" Date: Wed, 15 Nov 2017 10:05:57 -0600 Subject: [PATCH] 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 Change-Id: Ia51ccdacb995ff51a6467cbdd280c8965cfc5f47 Reviewed-on: https://review.whamcloud.com/30105 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: James Nunez Reviewed-by: Andreas Dilger Reviewed-by: Oleg Drokin --- lustre/tests/badarea_io.c | 3 +++ lustre/tests/sanity.sh | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) 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 #include +#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 -- 1.8.3.1