diff mbox series

Fix incorrect tst_brk usage in nfslock01.sh

Message ID 20240122122518.14149-1-mdoucha@suse.cz
State Accepted
Headers show
Series Fix incorrect tst_brk usage in nfslock01.sh | expand

Commit Message

Martin Doucha Jan. 22, 2024, 12:25 p.m. UTC 
Signed-off-by: Martin Doucha <mdoucha@suse.cz>
---

In the previous release, the nfs_lock process failure would skip any
remaining filesystems. After this change, the single filesystem test run
will exit but all remeaining filesystems will be tested.

 testcases/network/nfs/nfslock01/nfslock01.sh | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Petr Vorel Jan. 22, 2024, 3:48 p.m. UTC | #1 
Hi Martin,

> In the previous release, the nfs_lock process failure would skip any
> remaining filesystems. After this change, the single filesystem test run
> will exit but all remeaining filesystems will be tested.

Thanks a lot for a fix, merged!

Kind regards,
Petr
diff mbox series

Patch

diff --git a/testcases/network/nfs/nfslock01/nfslock01.sh b/testcases/network/nfs/nfslock01/nfslock01.sh
index 01d59ce85..1f8c2d755 100755
--- a/testcases/network/nfs/nfslock01/nfslock01.sh
+++ b/testcases/network/nfs/nfslock01/nfslock01.sh
@@ -68,7 +68,8 @@  do_test()
 	for p in $pids; do
 		wait $p
 		if [ $? -ne 0 ]; then
-			tst_brk TFAIL "nfs_lock process failed"
+			tst_res TFAIL "nfs_lock process failed"
+			return
 		else
 			tst_res TINFO "$p completed"
 		fi