public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
* [LTP] [PATCH 1/2] tst_test.sh: Print test command + params
@ 2023-10-31 16:09 Petr Vorel
  2023-10-31 16:09 ` [LTP] [PATCH 2/2] runtest/net.nfs: Rename test names Petr Vorel
  2024-01-19 16:31 ` [LTP] [PATCH 1/2] tst_test.sh: Print test command + params Cyril Hrubis
  0 siblings, 2 replies; 8+ messages in thread
From: Petr Vorel @ 2023-10-31 16:09 UTC (permalink / raw)
  To: ltp

To be more obvious what is being run
(test names in runtest files aren't always obvious).

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
 testcases/lib/tst_test.sh | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/testcases/lib/tst_test.sh b/testcases/lib/tst_test.sh
index b5b38f524..95ebdda9d 100644
--- a/testcases/lib/tst_test.sh
+++ b/testcases/lib/tst_test.sh
@@ -898,6 +898,8 @@ if [ -z "$TST_NO_DEFAULT_RUN" ]; then
 
 	TST_ARGS="$@"
 
+	tst_res TINFO "Running: $(basename $0) $TST_ARGS"
+
 	OPTIND=1
 
 	while getopts ":hi:$TST_OPTS" _tst_name $TST_ARGS; do
-- 
2.42.0


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [LTP] [PATCH 2/2] runtest/net.nfs: Rename test names
  2023-10-31 16:09 [LTP] [PATCH 1/2] tst_test.sh: Print test command + params Petr Vorel
@ 2023-10-31 16:09 ` Petr Vorel
  2024-01-19 16:28   ` Cyril Hrubis
  2024-01-30 12:29   ` Martin Doucha
  2024-01-19 16:31 ` [LTP] [PATCH 1/2] tst_test.sh: Print test command + params Cyril Hrubis
  1 sibling, 2 replies; 8+ messages in thread
From: Petr Vorel @ 2023-10-31 16:09 UTC (permalink / raw)
  To: ltp

Test names weren't obvious, rename to be more descriptive.

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
 runtest/net.nfs | 188 ++++++++++++++++++++++++------------------------
 1 file changed, 94 insertions(+), 94 deletions(-)

diff --git a/runtest/net.nfs b/runtest/net.nfs
index 15a960017..7556ed4fc 100644
--- a/runtest/net.nfs
+++ b/runtest/net.nfs
@@ -2,107 +2,107 @@
 #
 # PLEASE READ THE README FILE network/README.md BEFORE RUNNING THESE.
 #
-nfs3_01 nfs01.sh -v 3 -t udp
-nfs3t_01 nfs01.sh -v 3 -t tcp
-nfs4_01 nfs01.sh -v 4 -t tcp
-nfs41_01 nfs01.sh -v 4.1 -t tcp
-nfs42_01 nfs01.sh -v 4.2 -t tcp
-nfs3_ipv6_01 nfs01.sh -6 -v 3 -t udp
-nfs3t_ipv6_01 nfs01.sh -6 -v 3 -t tcp
-nfs4_ipv6_01 nfs01.sh -6 -v 4 -t tcp
-nfs41_ipv6_01 nfs01.sh -6 -v 4.1 -t tcp
-nfs42_ipv6_01 nfs01.sh -6 -v 4.2 -t tcp
+nfs01.sh_v3_ipv4_udp nfs01.sh -v 3 -t udp
+nfs01.sh_v3_ipv4_tcp nfs01.sh -v 3 -t tcp
+nfs01.sh_v4_ipv4_tcp nfs01.sh -v 4 -t tcp
+nfs01.sh_v4.1_ipv4_tcp nfs01.sh -v 4.1 -t tcp
+nfs01.sh_v4.2_ipv4_tcp nfs01.sh -v 4.2 -t tcp
+nfs01.sh_v3_ipv6_udp nfs01.sh -6 -v 3 -t udp
+nfs01.sh_v3_ipv6_tcp nfs01.sh -6 -v 3 -t tcp
+nfs01.sh_v4_ipv6_tcp nfs01.sh -6 -v 4 -t tcp
+nfs01.sh_v4.1_ipv6_tcp nfs01.sh -6 -v 4.1 -t tcp
+nfs01.sh_v4.2_ipv6_tcp nfs01.sh -6 -v 4.2 -t tcp
 
-nfs3_02 nfs02.sh -v 3 -t udp
-nfs3t_02 nfs02.sh -v 3 -t tcp
-nfs4_02 nfs02.sh -v 4 -t tcp
-nfs41_02 nfs02.sh -v 4.1 -t tcp
-nfs42_02 nfs02.sh -v 4.2 -t tcp
-nfs3_ipv6_02 nfs02.sh -6 -v 3 -t udp
-nfs3t_ipv6_02 nfs02.sh -6 -v 3 -t tcp
-nfs4_ipv6_02 nfs02.sh -6 -v 4 -t tcp
-nfs41_ipv6_02 nfs02.sh -6 -v 4.1 -t tcp
-nfs42_ipv6_02 nfs02.sh -6 -v 4.2 -t tcp
+nfs02.sh_v3_ipv4_udp nfs02.sh -v 3 -t udp
+nfs02.sh_v3_ipv4_tcp nfs02.sh -v 3 -t tcp
+nfs02.sh_v4_ipv4_tcp nfs02.sh -v 4 -t tcp
+nfs02.sh_v4.1_ipv4_tcp nfs02.sh -v 4.1 -t tcp
+nfs02.sh_v4.2_ipv4_tcp nfs02.sh -v 4.2 -t tcp
+nfs02.sh_v3_ipv6_udp nfs02.sh -6 -v 3 -t udp
+nfs02.sh_v3_ipv6_tcp nfs02.sh -6 -v 3 -t tcp
+nfs02.sh_v4_ipv6_tcp nfs02.sh -6 -v 4 -t tcp
+nfs02.sh_v4.1_ipv6_tcp nfs02.sh -6 -v 4.1 -t tcp
+nfs02.sh_v4.2_ipv6_tcp nfs02.sh -6 -v 4.2 -t tcp
 
-nfs3_03 nfs03.sh -v 3 -t udp
-nfs3t_03 nfs03.sh -v 3 -t tcp
-nfs4_03 nfs03.sh -v 4 -t tcp
-nfs41_03 nfs03.sh -v 4.1 -t tcp
-nfs42_03 nfs03.sh -v 4.2 -t tcp
-nfs3_ipv6_03 nfs03.sh -6 -v 3 -t udp
-nfs3t_ipv6_03 nfs03.sh -6 -v 3 -t tcp
-nfs4_ipv6_03 nfs03.sh -6 -v 4 -t tcp
-nfs41_ipv6_03 nfs03.sh -6 -v 4.1 -t tcp
-nfs42_ipv6_03 nfs03.sh -6 -v 4.2 -t tcp
+nfs03.sh_v3_ipv4_udp nfs03.sh -v 3 -t udp
+nfs03.sh_v3_ipv4_tcp nfs03.sh -v 3 -t tcp
+nfs03.sh_v4_ipv4_tcp nfs03.sh -v 4 -t tcp
+nfs03.sh_v4.1_ipv4_tcp nfs03.sh -v 4.1 -t tcp
+nfs03.sh_v4.2_ipv4_tcp nfs03.sh -v 4.2 -t tcp
+nfs03.sh_v3_ipv6_udp nfs03.sh -6 -v 3 -t udp
+nfs03.sh_v3_ipv6_tcp nfs03.sh -6 -v 3 -t tcp
+nfs03.sh_v4_ipv6_tcp nfs03.sh -6 -v 4 -t tcp
+nfs03.sh_v4.1_ipv6_tcp nfs03.sh -6 -v 4.1 -t tcp
+nfs03.sh_v4.2_ipv6_tcp nfs03.sh -6 -v 4.2 -t tcp
 
-nfs3_04 nfs04.sh -v 3 -t udp
-nfs3t_04 nfs04.sh -v 3 -t tcp
-nfs4_04 nfs04.sh -v 4 -t tcp
-nfs41_04 nfs04.sh -v 4.1 -t tcp
-nfs42_04 nfs04.sh -v 4.2 -t tcp
-nfs3_ipv6_04 nfs04.sh -6 -v 3 -t udp
-nfs3t_ipv6_04 nfs04.sh -6 -v 3 -t tcp
-nfs4_ipv6_04 nfs04.sh -6 -v 4 -t tcp
-nfs41_ipv6_04 nfs04.sh -6 -v 4.1 -t tcp
-nfs42_ipv6_04 nfs04.sh -6 -v 4.2 -t tcp
+nfs04.sh_v3_ipv4_udp nfs04.sh -v 3 -t udp
+nfs04.sh_v3_ipv4_tcp nfs04.sh -v 3 -t tcp
+nfs04.sh_v4_ipv4_tcp nfs04.sh -v 4 -t tcp
+nfs04.sh_v4.1_ipv4_tcp nfs04.sh -v 4.1 -t tcp
+nfs04.sh_v4.2_ipv4_tcp nfs04.sh -v 4.2 -t tcp
+nfs04.sh_v3_ipv6_udp nfs04.sh -6 -v 3 -t udp
+nfs04.sh_v3_ipv6_tcp nfs04.sh -6 -v 3 -t tcp
+nfs04.sh_v4_ipv6_tcp nfs04.sh -6 -v 4 -t tcp
+nfs04.sh_v4.1_ipv6_tcp nfs04.sh -6 -v 4.1 -t tcp
+nfs04.sh_v4.2_ipv6_tcp nfs04.sh -6 -v 4.2 -t tcp
 
-nfs3_05 nfs05.sh -v 3 -t udp
-nfs3t_05 nfs05.sh -v 3 -t tcp
-nfs4_05 nfs05.sh -v 4 -t tcp
-nfs41_05 nfs05.sh -v 4.1 -t tcp
-nfs42_05 nfs05.sh -v 4.2 -t tcp
-nfs3_ipv6_05 nfs05.sh -6 -v 3 -t udp
-nfs3t_ipv6_05 nfs05.sh -6 -v 3 -t tcp
-nfs4_ipv6_05 nfs05.sh -6 -v 4 -t tcp
-nfs41_ipv6_05 nfs05.sh -6 -v 4.1 -t tcp
-nfs42_ipv6_05 nfs05.sh -6 -v 4.2 -t tcp
+nfs05.sh_v3_ipv4_udp nfs05.sh -v 3 -t udp
+nfs05.sh_v3_ipv4_tcp nfs05.sh -v 3 -t tcp
+nfs05.sh_v4_ipv4_tcp nfs05.sh -v 4 -t tcp
+nfs05.sh_v4.1_ipv4_tcp nfs05.sh -v 4.1 -t tcp
+nfs05.sh_v4.2_ipv4_tcp nfs05.sh -v 4.2 -t tcp
+nfs05.sh_v3_ipv6_udp nfs05.sh -6 -v 3 -t udp
+nfs05.sh_v3_ipv6_tcp nfs05.sh -6 -v 3 -t tcp
+nfs05.sh_v4_ipv6_tcp nfs05.sh -6 -v 4 -t tcp
+nfs05.sh_v4.1_ipv6_tcp nfs05.sh -6 -v 4.1 -t tcp
+nfs05.sh_v4.2_ipv6_tcp nfs05.sh -6 -v 4.2 -t tcp
 
-nfs01_06  nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp"
-nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
-nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
+nfs06.sh_v3_v4_ipv4_tcp_udp  nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp"
+nfs06.sh_vall_ipv4tcp nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
+nfs06.sh_v4*_ipv6_tcp nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
 
-nfs3_07 nfs07.sh -v 3 -t udp
-nfs3t_07 nfs07.sh -v 3 -t tcp
-nfs4_07 nfs07.sh -v 4 -t tcp
-nfs41_07 nfs07.sh -v 4.1 -t tcp
-nfs42_07 nfs07.sh -v 4.2 -t tcp
-nfs3_ipv6_07 nfs07.sh -6 -v 3 -t udp
-nfs3t_ipv6_07 nfs07.sh -6 -v 3 -t tcp
-nfs4_ipv6_07 nfs07.sh -6 -v 4 -t tcp
-nfs41_ipv6_07 nfs07.sh -6 -v 4.1 -t tcp
-nfs42_ipv6_07 nfs07.sh -6 -v 4.2 -t tcp
+nfs07.sh_v3_ipv4_udp nfs07.sh -v 3 -t udp
+nfs07.sh_v3_ipv4_tcp nfs07.sh -v 3 -t tcp
+nfs07.sh_v4_ipv4_tcp nfs07.sh -v 4 -t tcp
+nfs07.sh_v4.1_ipv4_tcp nfs07.sh -v 4.1 -t tcp
+nfs07.sh_v4.2_ipv4_tcp nfs07.sh -v 4.2 -t tcp
+nfs07.sh_v3_ipv6_udp nfs07.sh -6 -v 3 -t udp
+nfs07.sh_v3_ipv6_tcp nfs07.sh -6 -v 3 -t tcp
+nfs07.sh_v4_ipv6_tcp nfs07.sh -6 -v 4 -t tcp
+nfs07.sh_v4.1_ipv6_tcp nfs07.sh -6 -v 4.1 -t tcp
+nfs07.sh_v4.2_ipv6_tcp nfs07.sh -6 -v 4.2 -t tcp
 
-nfs3_08 nfs08.sh -v 3 -t udp
-nfs3t_08 nfs08.sh -v 3 -t tcp
-nfs4_08 nfs08.sh -v 4 -t tcp
-nfs41_08 nfs08.sh -v 4.1 -t tcp
-nfs42_08 nfs08.sh -v 4.2 -t tcp
-nfs3_ipv6_08 nfs08.sh -6 -v 3 -t udp
-nfs3t_ipv6_08 nfs08.sh -6 -v 3 -t tcp
-nfs4_ipv6_08 nfs08.sh -6 -v 4 -t tcp
-nfs41_ipv6_08 nfs08.sh -6 -v 4.1 -t tcp
-nfs42_ipv6_08 nfs08.sh -6 -v 4.2 -t tcp
+nfs08.sh_v3_ipv4_udp nfs08.sh -v 3 -t udp
+nfs08.sh_v3_ipv4_tcp nfs08.sh -v 3 -t tcp
+nfs08.sh_v4_ipv4_tcp nfs08.sh -v 4 -t tcp
+nfs08.sh_v4.1_ipv4_tcp nfs08.sh -v 4.1 -t tcp
+nfs08.sh_v4.2_ipv4_tcp nfs08.sh -v 4.2 -t tcp
+nfs08.sh_v3_ipv6_udp nfs08.sh -6 -v 3 -t udp
+nfs08.sh_v3_ipv6_tcp nfs08.sh -6 -v 3 -t tcp
+nfs08.sh_v4_ipv6_tcp nfs08.sh -6 -v 4 -t tcp
+nfs08.sh_v4.1_ipv6_tcp nfs08.sh -6 -v 4.1 -t tcp
+nfs08.sh_v4.2_ipv6_tcp nfs08.sh -6 -v 4.2 -t tcp
 
-nfslock3_01 nfslock01.sh -v 3 -t udp
-nfslock3t_01 nfslock01.sh -v 3 -t tcp
-nfslock4_01 nfslock01.sh -v 4 -t tcp
-nfslock41_01 nfslock01.sh -v 4.1 -t tcp
-nfslock42_01 nfslock01.sh -v 4.2 -t tcp
-nfslock3_ipv6_01 nfslock01.sh -6 -v 3 -t udp
-nfslock3t_ipv6_01 nfslock01.sh -6 -v 3 -t tcp
-nfslock4_ipv6_01 nfslock01.sh -6 -v 4 -t tcp
-nfslock41_ipv6_01 nfslock01.sh -6 -v 4.1 -t tcp
-nfslock42_ipv6_01 nfslock01.sh -6 -v 4.2 -t tcp
+nfslock01.sh_v3_ipv4_udp nfslock01.sh -v 3 -t udp
+nfslock01.sh_v3_ipv4_tcp nfslock01.sh -v 3 -t tcp
+nfslock01.sh_v4_ipv4_tcp nfslock01.sh -v 4 -t tcp
+nfslock01.sh_v4.1_ipv4_tcp nfslock01.sh -v 4.1 -t tcp
+nfslock01.sh_v4.2_ipv4_tcp nfslock01.sh -v 4.2 -t tcp
+nfslock01.sh_v3_ipv6_udp nfslock01.sh -6 -v 3 -t udp
+nfslock01.sh_v3_ipv6_tcp nfslock01.sh -6 -v 3 -t tcp
+nfslock01.sh_v4_ipv6_tcp nfslock01.sh -6 -v 4 -t tcp
+nfslock01.sh_v4.1_ipv6_tcp nfslock01.sh -6 -v 4.1 -t tcp
+nfslock01.sh_v4.2_ipv6_tcp nfslock01.sh -6 -v 4.2 -t tcp
 
-nfsstat3_01 nfsstat01.sh
+nfsstat3.sh nfsstat01.sh
 
-nfsx3 fsx.sh -v 3 -t udp
-nfsx3t fsx.sh -v 3 -t tcp
-nfsx4 fsx.sh -v 4 -t tcp
-nfsx41 fsx.sh -v 4.1 -t tcp
-nfsx42 fsx.sh -v 4.2 -t tcp
-nfsx3_ipv6 fsx.sh -6 -v 3 -t udp
-nfsx3t_ipv6 fsx.sh -6 -v 3 -t tcp
-nfsx4_ipv6 fsx.sh -6 -v 4 -t tcp
-nfsx41_ipv6 fsx.sh -6 -v 4.1 -t tcp
-nfsx42_ipv6 fsx.sh -6 -v 4.2 -t tcp
+fsx.sh_v3_ipv4_udp fsx.sh -v 3 -t udp
+fsx.sh_v3_ipv4_tcp fsx.sh -v 3 -t tcp
+fsx.sh_v4_ipv4_tcp fsx.sh -v 4 -t tcp
+fsx.sh_v4.1_ipv4_tcp fsx.sh -v 4.1 -t tcp
+fsx.sh_v4.2_ipv4_tcp fsx.sh -v 4.2 -t tcp
+fsx.sh_v3_ipv6_udp fsx.sh -6 -v 3 -t udp
+fsx.sh_v3_ipv6_tcp fsx.sh -6 -v 3 -t tcp
+fsx.sh_v4_ipv6_tcp fsx.sh -6 -v 4 -t tcp
+fsx.sh_v4.1_ipv6_tcp fsx.sh -6 -v 4.1 -t tcp
+fsx.sh_v4.2_ipv6_tcp fsx.sh -6 -v 4.2 -t tcp
-- 
2.42.0


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [LTP] [PATCH 2/2] runtest/net.nfs: Rename test names
  2023-10-31 16:09 ` [LTP] [PATCH 2/2] runtest/net.nfs: Rename test names Petr Vorel
@ 2024-01-19 16:28   ` Cyril Hrubis
  2024-01-21 19:37     ` Petr Vorel
  2024-01-30 12:29   ` Martin Doucha
  1 sibling, 1 reply; 8+ messages in thread
From: Cyril Hrubis @ 2024-01-19 16:28 UTC (permalink / raw)
  To: Petr Vorel; +Cc: ltp

Hi!
> -nfs01_06  nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp"
> -nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
> -nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
> +nfs06.sh_v3_v4_ipv4_tcp_udp  nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp"
> +nfs06.sh_vall_ipv4tcp nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
> +nfs06.sh_v4*_ipv6_tcp nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"

Putting * into a test name is asking for a trouble, who knows where it
may expand...

Otherwise I'm all for having more descriptive names:

Reviewed-by: Cyril Hrubis <chrubis@suse.cz>

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [LTP] [PATCH 1/2] tst_test.sh: Print test command + params
  2023-10-31 16:09 [LTP] [PATCH 1/2] tst_test.sh: Print test command + params Petr Vorel
  2023-10-31 16:09 ` [LTP] [PATCH 2/2] runtest/net.nfs: Rename test names Petr Vorel
@ 2024-01-19 16:31 ` Cyril Hrubis
  2024-01-21 19:59   ` Petr Vorel
  1 sibling, 1 reply; 8+ messages in thread
From: Cyril Hrubis @ 2024-01-19 16:31 UTC (permalink / raw)
  To: Petr Vorel; +Cc: ltp

Hi!
> +	tst_res TINFO "Running: $(basename $0) $TST_ARGS"

If you think that it's important I suppose this is probably safe enough
for the release.

Reviewed-by: Cyril Hrubis <chrubis@suse.cz>

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [LTP] [PATCH 2/2] runtest/net.nfs: Rename test names
  2024-01-19 16:28   ` Cyril Hrubis
@ 2024-01-21 19:37     ` Petr Vorel
  0 siblings, 0 replies; 8+ messages in thread
From: Petr Vorel @ 2024-01-21 19:37 UTC (permalink / raw)
  To: Cyril Hrubis; +Cc: ltp

> Hi!
> > -nfs01_06  nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp"
> > -nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
> > -nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
> > +nfs06.sh_v3_v4_ipv4_tcp_udp  nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp"
> > +nfs06.sh_vall_ipv4tcp nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
> > +nfs06.sh_v4*_ipv6_tcp nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"

> Putting * into a test name is asking for a trouble, who knows where it
> may expand...

Hi Cyril,

would be nfs06.sh_v4+_ipv6_tcp better? or nfs06.sh_v4plus_ipv6_tcp
Or should I merge as nfs06.sh_v4_ipv6_tcp

Thanks a lot for having a look!

Kind regards,
Petr

> Otherwise I'm all for having more descriptive names:

> Reviewed-by: Cyril Hrubis <chrubis@suse.cz>

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [LTP] [PATCH 1/2] tst_test.sh: Print test command + params
  2024-01-19 16:31 ` [LTP] [PATCH 1/2] tst_test.sh: Print test command + params Cyril Hrubis
@ 2024-01-21 19:59   ` Petr Vorel
  0 siblings, 0 replies; 8+ messages in thread
From: Petr Vorel @ 2024-01-21 19:59 UTC (permalink / raw)
  To: Cyril Hrubis; +Cc: ltp

Hi Cyril,

> Hi!
> > +	tst_res TINFO "Running: $(basename $0) $TST_ARGS"

> If you think that it's important I suppose this is probably safe enough
> for the release.

Thanks, merged!

Kind regards,
Petr

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [LTP] [PATCH 2/2] runtest/net.nfs: Rename test names
  2023-10-31 16:09 ` [LTP] [PATCH 2/2] runtest/net.nfs: Rename test names Petr Vorel
  2024-01-19 16:28   ` Cyril Hrubis
@ 2024-01-30 12:29   ` Martin Doucha
  2024-01-30 13:19     ` Petr Vorel
  1 sibling, 1 reply; 8+ messages in thread
From: Martin Doucha @ 2024-01-30 12:29 UTC (permalink / raw)
  To: Petr Vorel, ltp

Hi,
I agree with the general idea but I think this version is a bit too 
verbose. How about something like this?

nfs01_v30_ip4u nfs01.sh -v 3 -t udp
nfs01_v30_ip4t nfs01.sh -v 3 -t tcp
nfs01_v40_ip4t nfs01.sh -v 4 -t tcp
nfs01_v41_ip4t nfs01.sh -v 4.1 -t tcp
nfs01_v42_ip4t nfs01.sh -v 4.2 -t tcp
nfs01_v30_ip6u nfs01.sh -6 -v 3 -t udp
nfs01_v30_ip6t nfs01.sh -6 -v 3 -t tcp
nfs01_v40_ip6t nfs01.sh -6 -v 4 -t tcp
nfs01_v41_ip6t nfs01.sh -6 -v 4.1 -t tcp
nfs01_v42_ip6t nfs01.sh -6 -v 4.2 -t tcp

nfs06_v30_v40_ip4  nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp"
nfs06_vall_ip4t nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t 
"tcp,tcp,tcp,tcp,tcp,tcp"
nfs06_v4x_ip6t nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t 
"tcp,tcp,tcp,tcp,tcp,tcp"


On 31. 10. 23 17:09, Petr Vorel wrote:
> Test names weren't obvious, rename to be more descriptive.
> 
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
>   runtest/net.nfs | 188 ++++++++++++++++++++++++------------------------
>   1 file changed, 94 insertions(+), 94 deletions(-)
> 
> diff --git a/runtest/net.nfs b/runtest/net.nfs
> index 15a960017..7556ed4fc 100644
> --- a/runtest/net.nfs
> +++ b/runtest/net.nfs
> @@ -2,107 +2,107 @@
>   #
>   # PLEASE READ THE README FILE network/README.md BEFORE RUNNING THESE.
>   #
> -nfs3_01 nfs01.sh -v 3 -t udp
> -nfs3t_01 nfs01.sh -v 3 -t tcp
> -nfs4_01 nfs01.sh -v 4 -t tcp
> -nfs41_01 nfs01.sh -v 4.1 -t tcp
> -nfs42_01 nfs01.sh -v 4.2 -t tcp
> -nfs3_ipv6_01 nfs01.sh -6 -v 3 -t udp
> -nfs3t_ipv6_01 nfs01.sh -6 -v 3 -t tcp
> -nfs4_ipv6_01 nfs01.sh -6 -v 4 -t tcp
> -nfs41_ipv6_01 nfs01.sh -6 -v 4.1 -t tcp
> -nfs42_ipv6_01 nfs01.sh -6 -v 4.2 -t tcp
> +nfs01.sh_v3_ipv4_udp nfs01.sh -v 3 -t udp
> +nfs01.sh_v3_ipv4_tcp nfs01.sh -v 3 -t tcp
> +nfs01.sh_v4_ipv4_tcp nfs01.sh -v 4 -t tcp
> +nfs01.sh_v4.1_ipv4_tcp nfs01.sh -v 4.1 -t tcp
> +nfs01.sh_v4.2_ipv4_tcp nfs01.sh -v 4.2 -t tcp
> +nfs01.sh_v3_ipv6_udp nfs01.sh -6 -v 3 -t udp
> +nfs01.sh_v3_ipv6_tcp nfs01.sh -6 -v 3 -t tcp
> +nfs01.sh_v4_ipv6_tcp nfs01.sh -6 -v 4 -t tcp
> +nfs01.sh_v4.1_ipv6_tcp nfs01.sh -6 -v 4.1 -t tcp
> +nfs01.sh_v4.2_ipv6_tcp nfs01.sh -6 -v 4.2 -t tcp
>   
> -nfs3_02 nfs02.sh -v 3 -t udp
> -nfs3t_02 nfs02.sh -v 3 -t tcp
> -nfs4_02 nfs02.sh -v 4 -t tcp
> -nfs41_02 nfs02.sh -v 4.1 -t tcp
> -nfs42_02 nfs02.sh -v 4.2 -t tcp
> -nfs3_ipv6_02 nfs02.sh -6 -v 3 -t udp
> -nfs3t_ipv6_02 nfs02.sh -6 -v 3 -t tcp
> -nfs4_ipv6_02 nfs02.sh -6 -v 4 -t tcp
> -nfs41_ipv6_02 nfs02.sh -6 -v 4.1 -t tcp
> -nfs42_ipv6_02 nfs02.sh -6 -v 4.2 -t tcp
> +nfs02.sh_v3_ipv4_udp nfs02.sh -v 3 -t udp
> +nfs02.sh_v3_ipv4_tcp nfs02.sh -v 3 -t tcp
> +nfs02.sh_v4_ipv4_tcp nfs02.sh -v 4 -t tcp
> +nfs02.sh_v4.1_ipv4_tcp nfs02.sh -v 4.1 -t tcp
> +nfs02.sh_v4.2_ipv4_tcp nfs02.sh -v 4.2 -t tcp
> +nfs02.sh_v3_ipv6_udp nfs02.sh -6 -v 3 -t udp
> +nfs02.sh_v3_ipv6_tcp nfs02.sh -6 -v 3 -t tcp
> +nfs02.sh_v4_ipv6_tcp nfs02.sh -6 -v 4 -t tcp
> +nfs02.sh_v4.1_ipv6_tcp nfs02.sh -6 -v 4.1 -t tcp
> +nfs02.sh_v4.2_ipv6_tcp nfs02.sh -6 -v 4.2 -t tcp
>   
> -nfs3_03 nfs03.sh -v 3 -t udp
> -nfs3t_03 nfs03.sh -v 3 -t tcp
> -nfs4_03 nfs03.sh -v 4 -t tcp
> -nfs41_03 nfs03.sh -v 4.1 -t tcp
> -nfs42_03 nfs03.sh -v 4.2 -t tcp
> -nfs3_ipv6_03 nfs03.sh -6 -v 3 -t udp
> -nfs3t_ipv6_03 nfs03.sh -6 -v 3 -t tcp
> -nfs4_ipv6_03 nfs03.sh -6 -v 4 -t tcp
> -nfs41_ipv6_03 nfs03.sh -6 -v 4.1 -t tcp
> -nfs42_ipv6_03 nfs03.sh -6 -v 4.2 -t tcp
> +nfs03.sh_v3_ipv4_udp nfs03.sh -v 3 -t udp
> +nfs03.sh_v3_ipv4_tcp nfs03.sh -v 3 -t tcp
> +nfs03.sh_v4_ipv4_tcp nfs03.sh -v 4 -t tcp
> +nfs03.sh_v4.1_ipv4_tcp nfs03.sh -v 4.1 -t tcp
> +nfs03.sh_v4.2_ipv4_tcp nfs03.sh -v 4.2 -t tcp
> +nfs03.sh_v3_ipv6_udp nfs03.sh -6 -v 3 -t udp
> +nfs03.sh_v3_ipv6_tcp nfs03.sh -6 -v 3 -t tcp
> +nfs03.sh_v4_ipv6_tcp nfs03.sh -6 -v 4 -t tcp
> +nfs03.sh_v4.1_ipv6_tcp nfs03.sh -6 -v 4.1 -t tcp
> +nfs03.sh_v4.2_ipv6_tcp nfs03.sh -6 -v 4.2 -t tcp
>   
> -nfs3_04 nfs04.sh -v 3 -t udp
> -nfs3t_04 nfs04.sh -v 3 -t tcp
> -nfs4_04 nfs04.sh -v 4 -t tcp
> -nfs41_04 nfs04.sh -v 4.1 -t tcp
> -nfs42_04 nfs04.sh -v 4.2 -t tcp
> -nfs3_ipv6_04 nfs04.sh -6 -v 3 -t udp
> -nfs3t_ipv6_04 nfs04.sh -6 -v 3 -t tcp
> -nfs4_ipv6_04 nfs04.sh -6 -v 4 -t tcp
> -nfs41_ipv6_04 nfs04.sh -6 -v 4.1 -t tcp
> -nfs42_ipv6_04 nfs04.sh -6 -v 4.2 -t tcp
> +nfs04.sh_v3_ipv4_udp nfs04.sh -v 3 -t udp
> +nfs04.sh_v3_ipv4_tcp nfs04.sh -v 3 -t tcp
> +nfs04.sh_v4_ipv4_tcp nfs04.sh -v 4 -t tcp
> +nfs04.sh_v4.1_ipv4_tcp nfs04.sh -v 4.1 -t tcp
> +nfs04.sh_v4.2_ipv4_tcp nfs04.sh -v 4.2 -t tcp
> +nfs04.sh_v3_ipv6_udp nfs04.sh -6 -v 3 -t udp
> +nfs04.sh_v3_ipv6_tcp nfs04.sh -6 -v 3 -t tcp
> +nfs04.sh_v4_ipv6_tcp nfs04.sh -6 -v 4 -t tcp
> +nfs04.sh_v4.1_ipv6_tcp nfs04.sh -6 -v 4.1 -t tcp
> +nfs04.sh_v4.2_ipv6_tcp nfs04.sh -6 -v 4.2 -t tcp
>   
> -nfs3_05 nfs05.sh -v 3 -t udp
> -nfs3t_05 nfs05.sh -v 3 -t tcp
> -nfs4_05 nfs05.sh -v 4 -t tcp
> -nfs41_05 nfs05.sh -v 4.1 -t tcp
> -nfs42_05 nfs05.sh -v 4.2 -t tcp
> -nfs3_ipv6_05 nfs05.sh -6 -v 3 -t udp
> -nfs3t_ipv6_05 nfs05.sh -6 -v 3 -t tcp
> -nfs4_ipv6_05 nfs05.sh -6 -v 4 -t tcp
> -nfs41_ipv6_05 nfs05.sh -6 -v 4.1 -t tcp
> -nfs42_ipv6_05 nfs05.sh -6 -v 4.2 -t tcp
> +nfs05.sh_v3_ipv4_udp nfs05.sh -v 3 -t udp
> +nfs05.sh_v3_ipv4_tcp nfs05.sh -v 3 -t tcp
> +nfs05.sh_v4_ipv4_tcp nfs05.sh -v 4 -t tcp
> +nfs05.sh_v4.1_ipv4_tcp nfs05.sh -v 4.1 -t tcp
> +nfs05.sh_v4.2_ipv4_tcp nfs05.sh -v 4.2 -t tcp
> +nfs05.sh_v3_ipv6_udp nfs05.sh -6 -v 3 -t udp
> +nfs05.sh_v3_ipv6_tcp nfs05.sh -6 -v 3 -t tcp
> +nfs05.sh_v4_ipv6_tcp nfs05.sh -6 -v 4 -t tcp
> +nfs05.sh_v4.1_ipv6_tcp nfs05.sh -6 -v 4.1 -t tcp
> +nfs05.sh_v4.2_ipv6_tcp nfs05.sh -6 -v 4.2 -t tcp
>   
> -nfs01_06  nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp"
> -nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
> -nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
> +nfs06.sh_v3_v4_ipv4_tcp_udp  nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp"
> +nfs06.sh_vall_ipv4tcp nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
> +nfs06.sh_v4*_ipv6_tcp nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp"
>   
> -nfs3_07 nfs07.sh -v 3 -t udp
> -nfs3t_07 nfs07.sh -v 3 -t tcp
> -nfs4_07 nfs07.sh -v 4 -t tcp
> -nfs41_07 nfs07.sh -v 4.1 -t tcp
> -nfs42_07 nfs07.sh -v 4.2 -t tcp
> -nfs3_ipv6_07 nfs07.sh -6 -v 3 -t udp
> -nfs3t_ipv6_07 nfs07.sh -6 -v 3 -t tcp
> -nfs4_ipv6_07 nfs07.sh -6 -v 4 -t tcp
> -nfs41_ipv6_07 nfs07.sh -6 -v 4.1 -t tcp
> -nfs42_ipv6_07 nfs07.sh -6 -v 4.2 -t tcp
> +nfs07.sh_v3_ipv4_udp nfs07.sh -v 3 -t udp
> +nfs07.sh_v3_ipv4_tcp nfs07.sh -v 3 -t tcp
> +nfs07.sh_v4_ipv4_tcp nfs07.sh -v 4 -t tcp
> +nfs07.sh_v4.1_ipv4_tcp nfs07.sh -v 4.1 -t tcp
> +nfs07.sh_v4.2_ipv4_tcp nfs07.sh -v 4.2 -t tcp
> +nfs07.sh_v3_ipv6_udp nfs07.sh -6 -v 3 -t udp
> +nfs07.sh_v3_ipv6_tcp nfs07.sh -6 -v 3 -t tcp
> +nfs07.sh_v4_ipv6_tcp nfs07.sh -6 -v 4 -t tcp
> +nfs07.sh_v4.1_ipv6_tcp nfs07.sh -6 -v 4.1 -t tcp
> +nfs07.sh_v4.2_ipv6_tcp nfs07.sh -6 -v 4.2 -t tcp
>   
> -nfs3_08 nfs08.sh -v 3 -t udp
> -nfs3t_08 nfs08.sh -v 3 -t tcp
> -nfs4_08 nfs08.sh -v 4 -t tcp
> -nfs41_08 nfs08.sh -v 4.1 -t tcp
> -nfs42_08 nfs08.sh -v 4.2 -t tcp
> -nfs3_ipv6_08 nfs08.sh -6 -v 3 -t udp
> -nfs3t_ipv6_08 nfs08.sh -6 -v 3 -t tcp
> -nfs4_ipv6_08 nfs08.sh -6 -v 4 -t tcp
> -nfs41_ipv6_08 nfs08.sh -6 -v 4.1 -t tcp
> -nfs42_ipv6_08 nfs08.sh -6 -v 4.2 -t tcp
> +nfs08.sh_v3_ipv4_udp nfs08.sh -v 3 -t udp
> +nfs08.sh_v3_ipv4_tcp nfs08.sh -v 3 -t tcp
> +nfs08.sh_v4_ipv4_tcp nfs08.sh -v 4 -t tcp
> +nfs08.sh_v4.1_ipv4_tcp nfs08.sh -v 4.1 -t tcp
> +nfs08.sh_v4.2_ipv4_tcp nfs08.sh -v 4.2 -t tcp
> +nfs08.sh_v3_ipv6_udp nfs08.sh -6 -v 3 -t udp
> +nfs08.sh_v3_ipv6_tcp nfs08.sh -6 -v 3 -t tcp
> +nfs08.sh_v4_ipv6_tcp nfs08.sh -6 -v 4 -t tcp
> +nfs08.sh_v4.1_ipv6_tcp nfs08.sh -6 -v 4.1 -t tcp
> +nfs08.sh_v4.2_ipv6_tcp nfs08.sh -6 -v 4.2 -t tcp
>   
> -nfslock3_01 nfslock01.sh -v 3 -t udp
> -nfslock3t_01 nfslock01.sh -v 3 -t tcp
> -nfslock4_01 nfslock01.sh -v 4 -t tcp
> -nfslock41_01 nfslock01.sh -v 4.1 -t tcp
> -nfslock42_01 nfslock01.sh -v 4.2 -t tcp
> -nfslock3_ipv6_01 nfslock01.sh -6 -v 3 -t udp
> -nfslock3t_ipv6_01 nfslock01.sh -6 -v 3 -t tcp
> -nfslock4_ipv6_01 nfslock01.sh -6 -v 4 -t tcp
> -nfslock41_ipv6_01 nfslock01.sh -6 -v 4.1 -t tcp
> -nfslock42_ipv6_01 nfslock01.sh -6 -v 4.2 -t tcp
> +nfslock01.sh_v3_ipv4_udp nfslock01.sh -v 3 -t udp
> +nfslock01.sh_v3_ipv4_tcp nfslock01.sh -v 3 -t tcp
> +nfslock01.sh_v4_ipv4_tcp nfslock01.sh -v 4 -t tcp
> +nfslock01.sh_v4.1_ipv4_tcp nfslock01.sh -v 4.1 -t tcp
> +nfslock01.sh_v4.2_ipv4_tcp nfslock01.sh -v 4.2 -t tcp
> +nfslock01.sh_v3_ipv6_udp nfslock01.sh -6 -v 3 -t udp
> +nfslock01.sh_v3_ipv6_tcp nfslock01.sh -6 -v 3 -t tcp
> +nfslock01.sh_v4_ipv6_tcp nfslock01.sh -6 -v 4 -t tcp
> +nfslock01.sh_v4.1_ipv6_tcp nfslock01.sh -6 -v 4.1 -t tcp
> +nfslock01.sh_v4.2_ipv6_tcp nfslock01.sh -6 -v 4.2 -t tcp
>   
> -nfsstat3_01 nfsstat01.sh
> +nfsstat3.sh nfsstat01.sh
>   
> -nfsx3 fsx.sh -v 3 -t udp
> -nfsx3t fsx.sh -v 3 -t tcp
> -nfsx4 fsx.sh -v 4 -t tcp
> -nfsx41 fsx.sh -v 4.1 -t tcp
> -nfsx42 fsx.sh -v 4.2 -t tcp
> -nfsx3_ipv6 fsx.sh -6 -v 3 -t udp
> -nfsx3t_ipv6 fsx.sh -6 -v 3 -t tcp
> -nfsx4_ipv6 fsx.sh -6 -v 4 -t tcp
> -nfsx41_ipv6 fsx.sh -6 -v 4.1 -t tcp
> -nfsx42_ipv6 fsx.sh -6 -v 4.2 -t tcp
> +fsx.sh_v3_ipv4_udp fsx.sh -v 3 -t udp
> +fsx.sh_v3_ipv4_tcp fsx.sh -v 3 -t tcp
> +fsx.sh_v4_ipv4_tcp fsx.sh -v 4 -t tcp
> +fsx.sh_v4.1_ipv4_tcp fsx.sh -v 4.1 -t tcp
> +fsx.sh_v4.2_ipv4_tcp fsx.sh -v 4.2 -t tcp
> +fsx.sh_v3_ipv6_udp fsx.sh -6 -v 3 -t udp
> +fsx.sh_v3_ipv6_tcp fsx.sh -6 -v 3 -t tcp
> +fsx.sh_v4_ipv6_tcp fsx.sh -6 -v 4 -t tcp
> +fsx.sh_v4.1_ipv6_tcp fsx.sh -6 -v 4.1 -t tcp
> +fsx.sh_v4.2_ipv6_tcp fsx.sh -6 -v 4.2 -t tcp

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


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [LTP] [PATCH 2/2] runtest/net.nfs: Rename test names
  2024-01-30 12:29   ` Martin Doucha
@ 2024-01-30 13:19     ` Petr Vorel
  0 siblings, 0 replies; 8+ messages in thread
From: Petr Vorel @ 2024-01-30 13:19 UTC (permalink / raw)
  To: Martin Doucha; +Cc: ltp

Hi Martin,

> Hi,
> I agree with the general idea but I think this version is a bit too verbose.
> How about something like this?

> nfs01_v30_ip4u nfs01.sh -v 3 -t udp
> nfs01_v30_ip4t nfs01.sh -v 3 -t tcp
> nfs01_v40_ip4t nfs01.sh -v 4 -t tcp
> nfs01_v41_ip4t nfs01.sh -v 4.1 -t tcp
> nfs01_v42_ip4t nfs01.sh -v 4.2 -t tcp
> nfs01_v30_ip6u nfs01.sh -6 -v 3 -t udp
> nfs01_v30_ip6t nfs01.sh -6 -v 3 -t tcp
> nfs01_v40_ip6t nfs01.sh -6 -v 4 -t tcp
> nfs01_v41_ip6t nfs01.sh -6 -v 4.1 -t tcp
> nfs01_v42_ip6t nfs01.sh -6 -v 4.2 -t tcp

> nfs06_v30_v40_ip4  nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp"
> nfs06_vall_ip4t nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t
> "tcp,tcp,tcp,tcp,tcp,tcp"
> nfs06_v4x_ip6t nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t
> "tcp,tcp,tcp,tcp,tcp,tcp"

Very good point, thank you!

We again get inconsistent with the rest of network runtest files, but I can live
with that).

I'll Cc you with next version.

Kind regards,
Petr

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2024-01-30 13:19 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-31 16:09 [LTP] [PATCH 1/2] tst_test.sh: Print test command + params Petr Vorel
2023-10-31 16:09 ` [LTP] [PATCH 2/2] runtest/net.nfs: Rename test names Petr Vorel
2024-01-19 16:28   ` Cyril Hrubis
2024-01-21 19:37     ` Petr Vorel
2024-01-30 12:29   ` Martin Doucha
2024-01-30 13:19     ` Petr Vorel
2024-01-19 16:31 ` [LTP] [PATCH 1/2] tst_test.sh: Print test command + params Cyril Hrubis
2024-01-21 19:59   ` Petr Vorel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox