Re: [LTP] [COMMITTED][PATCH 1/1] nfs05_make_tree: Restore 5 min timeout

[Cyril Hrubis <chrubis@suse.cz>]

Hi!
> nfs05_make_tree is a long running test. The previous default timeout 5
> min was enough after reducing runtime in 05be61cf6.
> 
> But the new default 30 sec introduced in the concept of max runtime is
> definitely not enough. Restore the previous timeout 5 min.

Looking at the code this is actually a helper binary rather than a test
itself. I think that the problem here is that we mix the shell test
library and C library and the test is a mess of both with unexpected
consequencies.

I guess that it would probably implement main() in the nfs05_make_tree.c
instead as we do for the various tools in testcases/lib/*.c. That would
at least make sure that we will not break anything with changes intended
for tests and not for helper binaries as this one.

> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
>  testcases/network/nfs/nfs_stress/nfs05_make_tree.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/testcases/network/nfs/nfs_stress/nfs05_make_tree.c b/testcases/network/nfs/nfs_stress/nfs05_make_tree.c
> index fdc13bd5d..5456c1bf0 100644
> --- a/testcases/network/nfs/nfs_stress/nfs05_make_tree.c
> +++ b/testcases/network/nfs/nfs_stress/nfs05_make_tree.c
> @@ -215,4 +215,5 @@ static struct tst_test test = {
>  	.options = opts,
>  	.test_all = do_test,
>  	.setup = setup,
> +	.max_runtime = 300,
>  };
> -- 
> 2.36.1
> 
> 
> -- 
> Mailing list info: https://lists.linux.it/listinfo/ltp

-- 
Cyril Hrubis
chrubis@suse.cz

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