----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63999/ -----------------------------------------------------------
(Updated Nov. 27, 2017, 4:44 p.m.) Review request for mesos and Alexander Rukletsov. Changes ------- Added some explanation of implementation details to comment. Repository: mesos Description ------- 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/mesos-gtest-runner.py 408e661f4fc9d0060c0eb7a45f1dfd1396f4f6c6 Diff: https://reviews.apache.org/r/63999/diff/2/ Changes: https://reviews.apache.org/r/63999/diff/1-2/ Testing ------- Verified manually that $ MESOS_GTEST_RUNNER_FLAGS=-j20 ../support/mesos-gtest-runner.py ./3rdparty/stout/tests/stout-tests executes 20 concurrent runs and that $ MESOS_GTEST_RUNNER_FLAGS=-j20 ../support/mesos-gtest-runner.py -j6 ./3rdparty/stout/tests/stout-tests executes 6 concurrent runs and that Thanks, Benjamin Bannier