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

(Updated March 29, 2016, 12:32 p.m.)

Review request for mesos, Artem Harutyunyan and Vinod Kone.


Changed this to a proper fix.

Bugs: MESOS-4961

Repository: mesos

Description (updated)

This removes a race between the test and the global `ReapProcess`, which were 
originally both calling `waitpid` on the logger subprocesses.  The test now 
defers the reaping to the `ReapProcess` and just waits for all logger 
subprocesses to exit naturally via `os::exists(pid)`.

Since the `ReapProcess` reaps on an interval, a `Clock::advance` was inserted 
to skip the reap delay.

Diffs (updated)

  src/tests/container_logger_tests.cpp 71101c31cee6a400b89cf285cf0a105d2d1534a8 

Diff: https://reviews.apache.org/r/45051/diff/

Testing (updated)

make check

--gtest_repeat=1000 --gtest_break_on_failure


Joseph Wu

Reply via email to