Martchus
/
mdadm
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

mdadm/test: mdadm needn't make install on the system 

Fixes: beb71de04d ("mdadm/test: enable clustermd testing under clustermd_tests/")
clustermd_tests/func.sh:
remove unnecessary 'make install', just ensure 'make everything' has done.
the original idea is to make the /sbin/mdadm version same as ./mdadm, and
this breakage has pointed out by commit:
59416da78f ("tests/func.sh: Fix some total breakage in the test scripts")

Signed-off-by: Zhilong Liu <zlliu@suse.com>
Signed-off-by: Jes Sorensen <jsorensen@fb.com>
This commit is contained in:
Zhilong Liu 2018-05-30 15:04:05 +08:00 committed by Jes Sorensen
parent 27e39ad31c
commit 38e955cbf0
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
clustermd_tests/func.sh
View File

@ -86,10 +86,13 @@ check_env()
echo "testing can only be done as 'root'."
exit 1
}
[ \! -x $mdadm ] && {
echo "test: please run make everything before perform testing."
exit 1
}
check_ssh
commands=(mdadm iscsiadm bc modinfo dlm_controld
udevadm crm crm_mon lsblk pgrep sbd)
mdadm_src_ver="$($mdadm -V 2>&1)"
for ip in $NODE1 $NODE2
do
for cmd in ${commands[@]}
@ -99,12 +102,6 @@ check_env()
exit 1
}
done
mdadm_sbin_ver="$(ssh $ip "mdadm -V 2>&1")"
if [ "$mdadm_src_ver" != "$mdadm_sbin_ver" ]
then
echo "$ip: please run 'make install' before testing."
exit 1
fi
mods=(raid1 raid10 md_mod dlm md-cluster)
for mod in ${mods[@]}
do