From 58a1c2c3210c4e10e5e55d19a5f357f7c6788f80 Mon Sep 17 00:00:00 2001
From: Lei Feng <flei@whamcloud.com>
Date: Wed, 6 Sep 2023 10:39:49 +0800
Subject: [PATCH] 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
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/52285
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Arshad Hussain <arshad.hussain@aeoncomputing.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
---
 lustre/tests/test-framework.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lustre/tests/test-framework.sh b/lustre/tests/test-framework.sh
index 54e5f35..bb38de0 100755
--- a/lustre/tests/test-framework.sh
+++ b/lustre/tests/test-framework.sh
@@ -2321,7 +2321,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
-- 
1.8.3.1