From: Shaun Tancheff <shaun.tancheff@hpe.com>
Date: Wed, 2 Apr 2025 00:53:07 +0000 (+0700)
Subject: LU-18751 lnet: test lnetctl fault reset directly
X-Git-Tag: 2.16.54~20
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=refs%2Fchanges%2F23%2F58623%2F4;p=fs%2Flustre-release.git

LU-18751 lnet: test lnetctl fault reset directly

Using 'do_lnetctl' confounds the required command line arguments
when calling:

    lnetctl fault reset <None>

So avoid the helper function for this case.

Test-Parameters: trivial testlist=sanity-lnet env=ONLY=410,ONLY_REPEAT=50
Fixes: dc54ff3310 ("LU-18751 lnet: fix sanity-lnet/410 test case")
Signed-off-by: Shaun Tancheff <shaun.tancheff@hpe.com>
Change-Id: I72ab8a06bbf1a95fd51aac4fb13955c0c03f8dd0
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/58623
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Frank Sehr <fsehr@whamcloud.com>
Reviewed-by: Timothy Day <timday@amazon.com>
Reviewed-by: James Simmons <jsimmons@infradead.org>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
---

diff --git a/lustre/tests/sanity-lnet.sh b/lustre/tests/sanity-lnet.sh
index 5d442fa..ee04c1f 100755
--- a/lustre/tests/sanity-lnet.sh
+++ b/lustre/tests/sanity-lnet.sh
@@ -4601,10 +4601,12 @@ run_test 402 "Destination net rule should not panic"
 test_410() {
 	reinit_dlc || return $?
 
-	do_lnetctl fault reset "" ||
-		return 0
+	$LCTL mark "$LNETCTL fault reset"
+	echo "$LNETCTL fault reset"
+	$LNETCTL fault reset '' &&
+		error "Command should fail"
 
-	error "Command should fail"
+	return 0
 }
 run_test 410 "No segfault in lnetctl fault command"