-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55683/
-----------------------------------------------------------
(Updated Jan. 19, 2017, 1:11 a.m.)
Review request for mesos, Benjamin Bannier, Joris Van Remoortere, Michael Park,
and Jiang Yan Xu.
Changes
-------
Rebase and address review feedback.
Bugs: MESOS-6946
https://issues.apache.org/jira/browse/MESOS-6946
Repository: mesos
Description (updated)
-------
Introduce the WSUCCEEDED() API to test that a forked process exited
successfully. Use it in all the places that were performing this test
bespokely, and update error logs to user WSTRINGIFY() if appropriate.
This also fixes called that were checking the exit status but not the
signaled status.
Diffs (updated)
-----
src/common/status_utils.hpp 26b94d49973dd61a6138e6a64d5ec0bdec6dea7e
src/docker/executor.cpp 9b5c469e2d0f33e228ec746711e6bc6ed352cbc7
src/hdfs/hdfs.cpp bebba8bb3c16f40293d21b5e00c353135a0f8f26
src/launcher/default_executor.cpp 57e4799e750f8f5352a9fec58af40efe432ea865
src/launcher/executor.cpp e035a4ee4438a3342f68a5548a5fd8d57a315cfa
src/launcher/fetcher.cpp 5f9a38d3ae8fb9b4b21203f96d53c5bef0909eba
src/linux/ns.hpp da43266ba67f76c9ca856584a540794062097691
src/slave/containerizer/docker.cpp 7a8a7271b54af0b4dcdae7a2aa8a90a8a7d05fd6
src/slave/containerizer/mesos/io/switchboard.cpp
2c4eed8f2dc7f91b9fd08f91f1b08484fca9a9af
src/slave/containerizer/mesos/launch.cpp
e482ab8bdfc358f695b87cda72ca59fb64cd8c4d
src/tests/script.cpp ef6b22aaeb056ed489f2c519e1a02644f2653329
Diff: https://reviews.apache.org/r/55683/diff/
Testing
-------
make check && sudo make check (Fedora 25)
Thanks,
James Peach
