From: Lei Feng <flei@whamcloud.com>
Date: Wed, 6 Sep 2023 02:39:49 +0000 (+0800)
Subject: LU-17091 tests: check correct return value in lfs_df
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=8d0b87768034a766a301f945b7a51bf3a3cf0c40;p=fs%2Flustre-release.git

LU-17091 tests: check correct return value in lfs_df

$? is the return value of last command in a pipe.
We should check the return value of first command 'lfs df'
in this case.

Signed-off-by: Lei Feng <flei@whamcloud.com>
Test-Parameters: trivial testlist=sanityn
Change-Id: I7daa38f27c878e5195181ed82717cd28ca345dbc
---

diff --git a/lustre/tests/test-framework.sh b/lustre/tests/test-framework.sh
index b25fcb4..1ca18f0 100755
--- a/lustre/tests/test-framework.sh
+++ b/lustre/tests/test-framework.sh
@@ -2318,7 +2318,7 @@ restore_quota() {
 # This will allow fixing the "lfs df" summary line in the future.
 lfs_df() {
 	$LFS df $* | sed -e 's/filesystem /filesystem_/'
-	check_lfs_df_ret_val $?
+	check_lfs_df_ret_val ${PIPESTATUS[0]}
 }
 
 # Get free inodes on the MDT specified by mdt index, free indoes on