Review Request 51095: Adding protobuf to represent errors returned by CNI plugins.

2016-08-15 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51095/ --- Review request for mesos, Jie Yu and Qian Zhang. Bugs: MESOS-6023 https://i

Re: Review Request 51095: Adding protobuf to represent errors returned by CNI plugins.

2016-08-25 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51095/#review146874 --- src/slave/containerizer/mesos/isolators/network/cni/spec.proto (l

Re: Review Request 51095: Adding protobuf to represent errors returned by CNI plugins.

2016-08-25 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51095/ --- (Updated Aug. 26, 2016, 12:26 a.m.) Review request for mesos, Jie Yu and Qian Z

Re: Review Request 51095: Adding protobuf to represent errors returned by CNI plugins.

2016-08-26 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51095/#review146925 --- src/slave/containerizer/mesos/isolators/network/cni/spec.cpp (lin

Re: Review Request 51095: Adding protobuf to represent errors returned by CNI plugins.

2016-08-26 Thread Avinash sridharan
> On Aug. 26, 2016, 7:24 a.m., Qian Zhang wrote: > > src/slave/containerizer/mesos/isolators/network/cni/spec.cpp, line 34 > > > > > > Why do we need to make this change? And it seems this change is not > > related t

Re: Review Request 51095: Adding protobuf to represent errors returned by CNI plugins.

2016-08-26 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51095/#review147001 --- Ship it! Ship It! - Jie Yu On Aug. 26, 2016, 12:26 a.m., Avi