-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44269/#review122123
-----------------------------------------------------------




src/slave/containerizer/mesos/isolators/network/cni.cpp (line 45)
<https://reviews.apache.org/r/44269/#comment184027>

    Remove the period at the end. We don't have terminal sentences in ERROR 
strings.



src/slave/containerizer/mesos/isolators/network/cni.cpp (line 50)
<https://reviews.apache.org/r/44269/#comment184028>

    Remove period.



src/slave/containerizer/mesos/isolators/network/cni.cpp (line 61)
<https://reviews.apache.org/r/44269/#comment184029>

    Add a `+` at the end. While not necessary, makes it more readable.


- Avinash sridharan


On March 4, 2016, 5:07 p.m., Qian Zhang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/44269/
> -----------------------------------------------------------
> 
> (Updated March 4, 2016, 5:07 p.m.)
> 
> 
> Review request for mesos, Avinash sridharan, Gilbert Song, and Jie Yu.
> 
> 
> Bugs: MESOS-4759
>     https://issues.apache.org/jira/browse/MESOS-4759
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Added the framework of 'network/cni' isolator.
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt 0eabfad66f43adff12d08dbf34ca3db1b801c49e 
>   src/Makefile.am 5b54fe04734afc5faa1bba3b84f43a1b8efe8753 
>   src/slave/containerizer/mesos/isolators/network/cni.hpp PRE-CREATION 
>   src/slave/containerizer/mesos/isolators/network/cni.cpp PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/44269/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Qian Zhang
> 
>

Reply via email to