mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Benjamin Bannier" <>
Subject Re: Review Request 42247: Made sure the container launcher terminated before we leave the test.
Date Mon, 18 Jan 2016 11:14:32 GMT

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

(Updated Jan. 18, 2016, 12:14 p.m.)

Review request for mesos, Jan Schlicht and Till Toenshoff.


Rename result to launch to follow canonical naming scheme.

Bugs: MESOS-4329

Repository: mesos

Description (updated)

Waiting for the launch result is not enough as it is unrelated to
whether the launched container has terminated. If we do not wait for
that the global teardown might find it still running and fail the tests
(this can happen if we e.g., execute this test in isolation).

Also along the name of the launch result to following the canonical

Diffs (updated)

  src/tests/slave_tests.cpp 7fe566770bbd802111885de061a53a3edf914840 



make check (OS X 10.10.5)


Benjamin Bannier

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