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




src/tests/containerizer/mesos_containerizer_tests.cpp (line 1065)
<https://reviews.apache.org/r/47095/#comment196387>

    s/SandboxEnv/ContainerSandboxEnv ?



src/tests/containerizer/mesos_containerizer_tests.cpp (line 1083)
<https://reviews.apache.org/r/47095/#comment196386>

    const string command



src/tests/containerizer/mesos_containerizer_tests.cpp (line 1110)
<https://reviews.apache.org/r/47095/#comment196378>

    4 spaces



src/tests/containerizer/provisioner_docker_tests.cpp (line 561)
<https://reviews.apache.org/r/47095/#comment196388>

    s/ROOT_INTERNET_CURL_MesosSandboxEnv/ROOT_INTERNET_CURL_ContainerSandboxEnv



src/tests/containerizer/provisioner_docker_tests.cpp (lines 580 - 582)
<https://reviews.apache.org/r/47095/#comment196384>

    MockScheduler sched;
    
    MesosSchedulerDriver driver(
        &sched,
        frameworkInfo,
        master.get()->pid,
        DEFAULT_CREDENTIAL);



src/tests/containerizer/provisioner_docker_tests.cpp (lines 630 - 638)
<https://reviews.apache.org/r/47095/#comment196380>

    4 spaces



src/tests/containerizer/provisioner_docker_tests.cpp (line 640)
<https://reviews.apache.org/r/47095/#comment196369>

    s/MESOS_SANDBOX/`MESOS_SANDBOX`


- Guangya Liu


On May 9, 2016, 3:51 a.m., Shuai Lin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47095/
> -----------------------------------------------------------
> 
> (Updated May 9, 2016, 3:51 a.m.)
> 
> 
> Review request for mesos, Guangya Liu and Jie Yu.
> 
> 
> Bugs: MESOS-5312
>     https://issues.apache.org/jira/browse/MESOS-5312
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Added tests for MESOS_SANDBOX env for unified containerizer.
> 
> 
> Diffs
> -----
> 
>   src/tests/containerizer/mesos_containerizer_tests.cpp 
> 09742ff21513dc2570684d384b257868dd57a9ce 
>   src/tests/containerizer/provisioner_docker_tests.cpp 
> 69505c922617273effc2eb52b0c567a3c01bf640 
> 
> Diff: https://reviews.apache.org/r/47095/diff/
> 
> 
> Testing
> -------
> 
> "make check" on ubuntu 14.04 64bit with gcc 4.8.4
> 
> 
> Thanks,
> 
> Shuai Lin
> 
>

Reply via email to