mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Benjamin Bannier" <benjamin.bann...@mesosphere.io>
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:
https://reviews.apache.org/r/42247/
-----------------------------------------------------------

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


Review request for mesos, Jan Schlicht and Till Toenshoff.


Changes
-------

Rename result to launch to follow canonical naming scheme.


Bugs: MESOS-4329
    https://issues.apache.org/jira/browse/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
scheme.


Diffs (updated)
-----

  src/tests/slave_tests.cpp 7fe566770bbd802111885de061a53a3edf914840 

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


Testing
-------

make check (OS X 10.10.5)


Thanks,

Benjamin Bannier


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