Re: Review Request 52771: Captured the `stderr` during execution of CNI plugin.

2016-10-17 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52771/#review153004 --- Fix it, then Ship it!

Re: Review Request 52771: Captured the `stderr` during execution of CNI plugin.

2016-10-17 Thread Avinash sridharan
--- 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

Re: Review Request 52771: Captured the `stderr` during execution of CNI plugin.

2016-10-17 Thread Avinash sridharan
> On Oct. 17, 2016, 11:30 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/network/cni/cni.cpp, lines 1507-1508 > > > > > > Looks like CNI spec does not specify if stdout will be empty or not. > >

Re: Review Request 52771: Captured the `stderr` during execution of CNI plugin.

2016-10-17 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52771/#review152988 --- src/slave/containerizer/mesos/isolators/network/cni/cni.cpp