[LU-14034] test-framework and sanity-hsm use libtool nonsensically

• Type: Task
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.14.0
• Assignee: John Hammond
• Reporter: John Hammond

Description

We use libtool several places in lustre/tests/ but only one place uses it correctly:

lustre/tests/conf-sanity.sh:            do_facet ${facet} "libtool execute l_tunedisk ${device}"

All of the following uses are nonsensical:

lustre/tests/sanity-hsm.sh:     do_nodesv $hosts "libtool execute pgrep -x $HSMTOOL"
lustre/tests/sanity-hsm.sh:             "do_nodesv $agents libtool execute pkill -CONT -x '$HSMTOOL' || true" EXIT
lustre/tests/sanity-hsm.sh:     do_nodesv $agents "libtool execute pkill -STOP -x $HSMTOOL" || return 0
lustre/tests/sanity-hsm.sh:     local mntpnt=$(do_rpc_nodes $agent libtool execute ps -C $HSMTOOL -o args= |
lustre/tests/sanity.sh:         # strip out libtool wrappers for in-tree executables
lustre/tests/test-framework.sh: do_nodesv $agents "libtool execute pkill -CONT -x $HSMTOOL" || return 0
lustre/tests/test-framework.sh: do_nodesv $hosts "libtool execute killall -q $HSMTOOL" || true
lustre/tests/test-framework.sh: stack_trap "do_facet $facet libtool execute pkill -x '$HSMTOOL' || true" EXIT

Asking libtool to execute pkill, pgrep, or killall does not fix the problem that this was intended to solve (namely that lhmstool_posix is sometimes a libtool script that execs lt-lhsmtool_posix (which needs to be passed as lt-lhsmtool_pos when used with pkill and pgrep)).

Issue Links

• is duplicated by: LU-13931 lhsm_posix with lt- prefix when build from source tree with --disable-shared
• is related to: LU-15560 Interop: conf-sanity test 125 fails due to ‘libtool: command not found’
• is related to: LU-12870 sanity-hsm test 9A fails with “uuid Dumping not found in agent list on mds1”
• is related to: LU-13688 Handle in-tree executed HSM copytools correctly

Comments

[Peter Jones]

Landed for 2.14

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/40253/
Subject: LU-14034 hsm: add PID file handling to lhsmtool_posix
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 71a4616b38e81d021a65ee5305b692b271ed0955

[Gerrit Updater]

John L. Hammond ([e-mail]) uploaded a new patch: https://review.whamcloud.com/40253
Subject: LU-14034 tests: remove libtool usage
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 627b8ff32c6c4c3032d17362545b6613d4178176