From: Andreas Dilger <adilger@whamcloud.com>
Date: Wed, 13 Oct 2021 20:34:14 +0000 (-0600)
Subject: LU-13997 tests: fix sanity test_418 lock cancellation
X-Git-Tag: 2.14.56~149
X-Git-Url: https://git.whamcloud.com/?p=fs%2Flustre-release.git;a=commitdiff_plain;h=c0596c4d8af3bce6565c0e16a1469fcc65191404

LU-13997 tests: fix sanity test_418 lock cancellation

Use "do_nodes" directly to cancel DLM locks, rather than
"do_rpc_nodes", since that is very heavy to use in a loop
(each call takes 3s, but the loop delay is only 0.2s).

Due to DoM reserving grant space for the DoM files, the "avail"
space shown by "df" may be smaller in the aggregate returned by
the MDT compared to the individual values from "lfs df".

Skip this part of the check until MDC grant cancel is fixed.

Test-Parameters: trivial
Signed-off-by: Andreas Dilger <adilger@whamcloud.com>
Change-Id: I83d989688ce671f0ff9c62ebdf3144746a3ebbe5
Reviewed-on: https://review.whamcloud.com/45231
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Alex Zhuravlev <bzzz@whamcloud.com>
Reviewed-by: James Nunez <jnunez@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
---

diff --git a/lustre/tests/sanity.sh b/lustre/tests/sanity.sh
index 308d6a5..6f1e2aa 100755
--- a/lustre/tests/sanity.sh
+++ b/lustre/tests/sanity.sh
@@ -25434,7 +25434,8 @@ check_lfs_df() {
 	[ "$1" == "blocks" ] && inodes= || inodes="-i"
 
 	for count in {1..100}; do
-		do_rpc_nodes "$CLIENTS" cancel_lru_locks
+		do_nodes "$CLIENTS" \
+			$LCTL set_param ldlm.namespaces.*.lru_size=clear
 		sync; sleep 0.2
 
 		# read the lines of interest
@@ -25447,7 +25448,9 @@ check_lfs_df() {
 		# "<NID>:/<fsname>" for df, "filesystem_summary:" for lfs df
 		# compare the two outputs
 		passed=true
-		for i in {1..5}; do
+		#  skip "available" on MDT until LU-13997 is fixed.
+		#for i in {1..5}; do
+		for i in 1 2 4 5; do
 			[ "${df_out[i]}" != "${lfs_df_out[i]}" ] && passed=false
 		done
 		$passed && break