From: Sebastien Buisson <sbuisson@ddn.com>
Date: Fri, 25 Apr 2025 13:14:14 +0000 (+0200)
Subject: LU-18953 tests: verify identity_upcall is executable
X-Git-Tag: 2.16.55~28
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=28c477a87a22b58056d6f371786043e872057d0f;p=fs%2Flustre-release.git

LU-18953 tests: verify identity_upcall is executable

In test-framework, verify that the value of the L_GETIDENTITY
variable points to an executable. Otherwise, the identity_upcall
would fail.

Test-Parameters: trivial testlist=conf-sanity
Signed-off-by: Sebastien Buisson <sbuisson@ddn.com>
Change-Id: I5839c157c3678caa81908f950d3cc2cd2fc59c44
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/58961
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Marc Vef <mvef@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
---

diff --git a/lustre/tests/test-framework.sh b/lustre/tests/test-framework.sh
index f9f215e..8c24bcc 100755
--- a/lustre/tests/test-framework.sh
+++ b/lustre/tests/test-framework.sh
@@ -559,7 +559,7 @@ init_test_env() {
 	export PERM_CMD=$(echo ${PERM_CMD:-"$LCTL conf_param"})
 
 	export L_GETIDENTITY=${L_GETIDENTITY:-"$LUSTRE/utils/l_getidentity"}
-	if [ ! -f "$L_GETIDENTITY" ]; then
+	if [ ! -x "$L_GETIDENTITY" ]; then
 		if $(which l_getidentity > /dev/null 2>&1); then
 			export L_GETIDENTITY=$(which l_getidentity)
 		else