mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <>
Subject Re: Review Request 63999: Added env var to set default flags for parallel test runner.
Date Mon, 27 Nov 2017 15:44:25 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Nov. 27, 2017, 4:44 p.m.)

Review request for mesos and Alexander Rukletsov.


Added some explanation of implementation details to comment.

Repository: mesos


This patch modifies the parallel test runner to examine the
environment variable 'MESOS_GTEST_RUNNER_FLAGS' for a default set of
flags to pass. Flags given explicitly on the command line always have
precedence over default flags from the environment variable.

This allows e.g., to set the default level of parallelism to not
overload systems with many CPUs during 'make check' (which could
ultimately lead to tests hitting timeouts).

Diffs (updated)

  support/ 408e661f4fc9d0060c0eb7a45f1dfd1396f4f6c6 




Verified manually that

    $ MESOS_GTEST_RUNNER_FLAGS=-j20 ../support/ ./3rdparty/stout/tests/stout-tests

executes 20 concurrent runs and that

    $ MESOS_GTEST_RUNNER_FLAGS=-j20 ../support/ -j6 ./3rdparty/stout/tests/stout-tests
executes 6 concurrent runs and that


Benjamin Bannier

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message