From f1420059ac7d33cba65ab1b14fd5eade3c889684 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/30219 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Jinshan Xiong --- 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 881a3d7..a803cd0 100755 --- a/lustre/tests/sanity.sh +++ b/lustre/tests/sanity.sh @@ -9993,7 +9993,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 @@ -10008,7 +10008,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