-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52771/
-----------------------------------------------------------
(Updated Oct. 18, 2016, 12:02 a.m.)
Review request for mesos, Jie Yu and Qian Zhang.
Changes
-------
Addressed Jie's comments.
Bugs: MESOS-6282
https://issues.apache.org/jira/browse/MESOS-6282
Repository: mesos
Description
-------
Till now we were capturing only the `stdout` of the CNI plugin. However,
during errors it makes sense to add the output from `stderr` for the
CNI plugin as well, since as per the CNI spec the CNI plugin is
supposed to output all unstructured output (debugs for example) to
`stderr`. This greatly helps in debugging the exact reason for CNI
plugin failures.
Diffs (updated)
-----
src/slave/containerizer/mesos/isolators/network/cni/cni.hpp
70f30831819d7a0e6233fcb13a703dc6981324b6
src/slave/containerizer/mesos/isolators/network/cni/cni.cpp
7c35c3056326a8a135e4ad944ac741cda96cab99
Diff: https://reviews.apache.org/r/52771/diff/
Testing
-------
make and make check.
Thanks,
Avinash sridharan