Message ID | 20230919114701.15327-2-mdoucha@suse.cz |
---|---|
State | Accepted |
Headers | show |
Series | NFS test fixes | expand |
Hi Martin,
Reviewed-by: Petr Vorel <pvorel@suse.cz>
Kind regards,
Petr
Reviewed-by: Richard Palethorpe <rpalethorpe@suse.com> Martin Doucha <mdoucha@suse.cz> writes: > Signed-off-by: Martin Doucha <mdoucha@suse.cz> > --- > testcases/network/nfs/nfs_stress/nfs_lib.sh | 9 +++++++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/testcases/network/nfs/nfs_stress/nfs_lib.sh b/testcases/network/nfs/nfs_stress/nfs_lib.sh > index abf7ba5a2..a996f7cc8 100644 > --- a/testcases/network/nfs/nfs_stress/nfs_lib.sh > +++ b/testcases/network/nfs/nfs_stress/nfs_lib.sh > @@ -191,7 +191,9 @@ nfs_setup() > > remote_dir="$(get_remote_dir $i $type)" > nfs_setup_server "$remote_dir" "$(($$ + n))" > - nfs_mount "$(get_local_dir $i $n)" "$remote_dir" "-o proto=$type,vers=$i" > + local_dir="$(get_local_dir $i $n)" > + tst_res TINFO "Mounting $local_dir" > + nfs_mount "$local_dir" "$remote_dir" "-o proto=$type,vers=$i" > > n=$(( n + 1 )) > done > @@ -210,7 +212,10 @@ nfs_cleanup() > local n=0 > for i in $VERSION; do > local_dir="$(get_local_dir $i $n)" > - grep -q "$local_dir" /proc/mounts && umount $local_dir > + if grep -q "$local_dir" /proc/mounts; then > + tst_res TINFO "Unmounting $local_dir" > + umount $local_dir > + fi > n=$(( n + 1 )) > done > sleep 2 > -- > 2.42.0
Hi!
Reviewed-by: Cyril Hrubis <chrubis@suse.cz>
diff --git a/testcases/network/nfs/nfs_stress/nfs_lib.sh b/testcases/network/nfs/nfs_stress/nfs_lib.sh index abf7ba5a2..a996f7cc8 100644 --- a/testcases/network/nfs/nfs_stress/nfs_lib.sh +++ b/testcases/network/nfs/nfs_stress/nfs_lib.sh @@ -191,7 +191,9 @@ nfs_setup() remote_dir="$(get_remote_dir $i $type)" nfs_setup_server "$remote_dir" "$(($$ + n))" - nfs_mount "$(get_local_dir $i $n)" "$remote_dir" "-o proto=$type,vers=$i" + local_dir="$(get_local_dir $i $n)" + tst_res TINFO "Mounting $local_dir" + nfs_mount "$local_dir" "$remote_dir" "-o proto=$type,vers=$i" n=$(( n + 1 )) done @@ -210,7 +212,10 @@ nfs_cleanup() local n=0 for i in $VERSION; do local_dir="$(get_local_dir $i $n)" - grep -q "$local_dir" /proc/mounts && umount $local_dir + if grep -q "$local_dir" /proc/mounts; then + tst_res TINFO "Unmounting $local_dir" + umount $local_dir + fi n=$(( n + 1 )) done sleep 2
Signed-off-by: Martin Doucha <mdoucha@suse.cz> --- testcases/network/nfs/nfs_stress/nfs_lib.sh | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-)