[Bro-Dev] Problems parallelizing btests

[Daniel Thayer]

On 03/02/2012 11:45 AM, Robin Sommer wrote:
>
> On Wed, Feb 29, 2012 at 17:54 -0800, I wrote:
>
>> btest:topic/robin/parallel has a version of btest that can run tests
>> in parallel.
>
> This branch could use some testing as well, btw. I've also
> restructured things internally a bit. README isn't updated yet but the
> new options are:
>
>    -j THREADS, --jobs=THREADS
>                          number of threads to run tests in simultaniously; 0
>                          disables threading
>    -g GROUP, --group=GROUP
>                          execute only test of given group, or '-' for those
>                          without any group
>    -r, --rerun           Execute commands for tests that failed last time
>
>
> (For the Bro tests, one currently needs to remove the
> BRO_PROFILER_FILE variable from btest.cfg to make it work.)
>
>
> Robin
>

It seems that using the "-f" option (without "-b" or "-v") now
prevents the status message for each test from being output.
The following patch should fix this bug:


--- a/btest
+++ b/btest
@@ -927,11 +927,9 @@ if Options.diagfile:

  if Options.verbose:
      output_handlers += [Verbose(Options, )]
-
-if Options.brief:
+elif Options.brief:
      output_handlers += [Brief(Options, )]
-
-if not output_handlers:
+else:
      output_handlers += [Standard(Options, )]

  output_handler = Forwarder(Options, output_handlers)