<div dir="ltr"><div><div><div><div><div><div><div><div><div><div><div><div><div><div>Hey Markus,<br><br></div></div></div>You are of course right about the timeouts, it was an oversight by myself. Fortunately,<br></div>the solution is easy. CMake tests have a TIMEOUT property, that takes precedence over<br></div>the timeout settings of the ctest run (which have no timeout if run without special options).<br></div>I propose the following:<br><br></div>We add a TIMEOUT argument to dune_add_test. If MPI_RANKS are specified, we make<br></div>specifying the TIMEOUT mandatory.<br><br></div>On a similar note, I would *LOVE* to set a global upperbound to how much time a test<br></div>may take, because many tests seem to be written with more than the minimal necessary<br></div>problem size or may need splitting.<br><br></div>Best,<br></div>Dominic<br><br></div>PS:<br>BTW, the snippet you are quoting is an old code that I was cherry-picking and later<br>porting to dune_add_test. But dune_add_test does basically the same.<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 2, 2015 at 8:51 PM, Markus Blatt <span dir="ltr"><<a href="mailto:markus@dr-blatt.de" target="_blank">markus@dr-blatt.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey,<br>
<span class=""><br>
On Sun, Nov 01, 2015 at 05:18:54PM +0100, Dominic Kempf wrote:<br>
> New commit, appeared at Sun Nov  1 17:18:54 2015 +0100<br>
> as part of the following ref changes:<br>
><br>
>     branch refs/heads/feature/yaspgrid-testing                           created as              7e47c04<br>
>     branch refs/merge-requests/3/head                                    created as              7e47c04<br>
><br>
</span><span class="">> +<br>
> +  # Create tests with mpi command and add them to the test target<br>
> +  foreach(_test ${TESTS_PARALLEL})<br>
> +    add_test(NAME ${_test}_parallel<br>
> +      COMMAND ${MPIEXEC} ${MPIEXEC_NUMPROC_FLAG} "2" ${_test})<br>
> +    add_dependencies(${_test_target} ${_test}_parallel)<br>
> +  endforeach(_test ${TESTS_PARALLEL})<br>
> +<br>
<br>
</span>Guys, this has to be improved. If you really want to do automatic<br>
parallel testing, then you need to find a way to stop deadlocked<br>
tests. Otherwise sooner or later your servers will not be usable<br>
anymore.<br>
<span class="HOEnZb"><font color="#888888"><br>
Markus<br>
<br>
--<br>
Join us at the DUNE User Meeting Sept. 28-29, 2015, Heidelberg, Germany<br>
<a href="http://users.dune-project.org/projects/dune-user-meeting-2015/wiki" rel="noreferrer" target="_blank">http://users.dune-project.org/projects/dune-user-meeting-2015/wiki</a><br>
<br>
Dr. Markus Blatt - HPC-Simulation-Software & Services <a href="http://www.dr-blatt.de" rel="noreferrer" target="_blank">http://www.dr-blatt.de</a><br>
Hans-Bunte-Str. 8-10, 69123 Heidelberg, Germany<br>
Tel.: <a href="tel:%2B49%20%280%29%20160%2097590858" value="+4916097590858">+49 (0) 160 97590858</a><br>
</font></span><br>_______________________________________________<br>
Dune-devel mailing list<br>
<a href="mailto:Dune-devel@dune-project.org">Dune-devel@dune-project.org</a><br>
<a href="http://lists.dune-project.org/mailman/listinfo/dune-devel" rel="noreferrer" target="_blank">http://lists.dune-project.org/mailman/listinfo/dune-devel</a><br>
<br></blockquote></div><br></div>