[LTP] [PATCH 1/1] nfs_lib.sh: Require mount.nfs, print version

[Martin Doucha]

Hi,
looks good to me.

Reviewed-by: Martin Doucha <mdoucha@suse.cz>

On 11. 03. 22 14:22, Petr Vorel wrote:
> mount.nfs is required for mounting nfs.
> NOTE: we check only on lhost. Unlike for exportfs which is actually
> required on rhost when testing via ssh, mount.nfs is required on lhost
> (unless netns testing, but that is actually always on lhost).
> 
> Print its version for better debugging.
> 
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
>  testcases/network/nfs/nfs_stress/nfs_lib.sh | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/testcases/network/nfs/nfs_stress/nfs_lib.sh b/testcases/network/nfs/nfs_stress/nfs_lib.sh
> index 1bf31eed25..55ce1e11a4 100644
> --- a/testcases/network/nfs/nfs_stress/nfs_lib.sh
> +++ b/testcases/network/nfs/nfs_stress/nfs_lib.sh
> @@ -29,7 +29,7 @@ TST_PARSE_ARGS=nfs_parse_args
>  TST_USAGE=nfs_usage
>  TST_NEEDS_TMPDIR=1
>  TST_NEEDS_ROOT=1
> -TST_NEEDS_CMDS="$TST_NEEDS_CMDS mount exportfs"
> +TST_NEEDS_CMDS="$TST_NEEDS_CMDS mount exportfs mount.nfs"
>  TST_SETUP="${TST_SETUP:-nfs_setup}"
>  TST_CLEANUP="${TST_CLEANUP:-nfs_cleanup}"
>  TST_NEEDS_DRIVERS="nfsd"
> @@ -153,6 +153,8 @@ nfs_setup()
>  		done
>  	fi
>  
> +	tst_res TINFO "$(mount.nfs -V)"
> +
>  	for i in $VERSION; do
>  		type=$(get_socket_type $n)
>  		tst_res TINFO "setup NFSv$i, socket type $type"


-- 
Martin Doucha   mdoucha@suse.cz
QA Engineer for Software Maintenance
SUSE LINUX, s.r.o.
CORSO IIa
Krizikova 148/34
186 00 Prague 8
Czech Republic