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


Fix it, then Ship it!




Ship It!


src/slave/containerizer/mesos/isolators/network/cni/cni.cpp (line 1388)
<https://reviews.apache.org/r/46331/#comment192985>

    s/, and this/. This
    s/by command executor/by the command executor



src/slave/containerizer/mesos/isolators/network/cni/cni.cpp (line 1389)
<https://reviews.apache.org/r/46331/#comment192984>

    s/with host filesystem first and
    it will change to the container filesystem itself when launching task.
    
    /with rootfs of host filesystem and will later pivot to the rootfs of the 
container filesystem, when launching the task.


- Avinash sridharan


On April 19, 2016, 8:21 a.m., Qian Zhang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46331/
> -----------------------------------------------------------
> 
> (Updated April 19, 2016, 8:21 a.m.)
> 
> 
> Review request for mesos, Avinash sridharan and Jie Yu.
> 
> 
> Bugs: MESOS-5225
>     https://issues.apache.org/jira/browse/MESOS-5225
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Fixed the issue that command executor can not join CNI network.
> 
> 
> Diffs
> -----
> 
>   src/slave/containerizer/mesos/isolators/network/cni/cni.cpp 
> 6e3fe118fb63d1ce7d5b01a6ac32f8f3a1369cfb 
> 
> Diff: https://reviews.apache.org/r/46331/diff/
> 
> 
> Testing
> -------
> 
> 1. Start master
> sudo ./bin/mesos-master.sh --work_dir=/tmp
> 
> 2. Start agent
> sudo ./bin/mesos-slave.sh --master=192.168.122.171:5050 
> --containerizers=mesos --image_providers=docker 
> --isolation=filesystem/linux,docker/runtime,network/cni 
> --network_cni_config_dir=/opt/cni/net_configs 
> --network_cni_plugins_dir=/opt/cni/plugins
> 
> 3. Launch a command task with mesos-execute, and it will join a CNI network 
> net1.
> sudo src/mesos-execute --master=192.168.122.171:5050 --name=test 
> --docker_image=library/busybox --networks=net1 --command="sleep 10" 
> --shell=true
> I0418 21:34:57.248507 24604 scheduler.cpp:177] Version: 0.29.0
> Subscribed with ID 'c992158d-e625-4359-97fe-6320172fd957-0016'
> Submitted task 'test' to agent 'eeb0be14-77cb-462d-b088-657745453c83-S0'
> Received status update TASK_RUNNING for task 'test'
>   source: SOURCE_EXECUTOR
> Received status update TASK_FINISHED for task 'test'
>   message: 'Command exited with status 0'
>   source: SOURCE_EXECUTOR
> 
> 
> Thanks,
> 
> Qian Zhang
> 
>

Reply via email to