mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Qian Zhang <zhq527...@gmail.com>
Subject Re: Review Request 61237: Updated docker executor to return IPv6 address of a container.
Date Thu, 24 Aug 2017 09:07:21 GMT

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




src/docker/docker.cpp
Line 326 (original), 329 (patched)
<https://reviews.apache.org/r/61237/#comment259769>

    Not yours. s/fail back/fallback/


- Qian Zhang


On Aug. 24, 2017, 8:48 a.m., Avinash sridharan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61237/
> -----------------------------------------------------------
> 
> (Updated Aug. 24, 2017, 8:48 a.m.)
> 
> 
> Review request for mesos, Jie Yu and Qian Zhang.
> 
> 
> Bugs: MESOS-7807
>     https://issues.apache.org/jira/browse/MESOS-7807
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> A docker container can have a v4 and a v6 address. Currently the docker
> executor was returning only the IPv4 address. This changes allows the
> executor to return the IPv4 and IPv6 address of the container.
> 
> 
> Diffs
> -----
> 
>   src/docker/docker.hpp 95e60a7dbbd6ccc659f70ca3dca8d13433f3ea07 
>   src/docker/docker.cpp 192e170acce895f80df6c83e437da020489de468 
>   src/docker/executor.cpp 99a62244b7805d25b926d0293cb62574981d2878 
> 
> 
> Diff: https://reviews.apache.org/r/61237/diff/3/
> 
> 
> Testing
> -------
> 
> Start agent:
> sudo ./bin/mesos-agent.sh --ip=10.0.2.15 --master=10.0.2.15:5050 --work_dir=/tmp/mesos
--containerizers=mesos,docker
> 
> Start master:
> sudo ./bin/mesos-master.sh --ip=10.0.2.15 --work_dir=/tmp/mesos-master
> 
> Use mesos-execute to launch a docker container on an IPv6 network:
> sudo src/mesos-execute --master=10.0.2.15:5050 --task=file:///home/vagrant/dev/mesos_tasks/docker.json
2> log.txt
> 
> cat /home/vagrant/dev/mesos_tasks/docker.json
> {
>   "name": "test",
>   "task_id": {"value" : "test"},
>   "agent_id": {"value" : ""},
>   "resources": [
>     {
>       "name": "cpus",
>       "type": "SCALAR",
>       "scalar": {
>         "value": 0.1
>       }
>     },
>     {
>       "name": "mem",
>       "type": "SCALAR",
>       "scalar": {
>         "value": 32
>       }
>     }
>   ],
>   "command": {
>     "shell": false
>   },
>   "container": {
>     "type": "DOCKER",
>     "docker": {
>       "image": "nginx",
>       "network": "USER"
>     },
>     "network_infos": [
>         {
>           "name": "ipv6"
>         }
>     ]
>   }
> }
> 
> 
> Curl the Mesos state to make sure that the right IPv6 addresses are reflected for the
container:
>  curl http://10.0.2.15:5051/state | jq
> 
> "role": "*",
>               "statuses": [
>                 {
>                   "state": "TASK_RUNNING",
>                   "timestamp": 1501288232.79384,
>                   "container_status": {
>                     "container_id": {
>                       "value": "59319a5b-a28b-46bf-a20f-8472e6598abd"
>                     },
>                     "network_infos": [
>                       {
>                         "ip_addresses": [
>                           {
>                             "protocol": "IPv4",
>                             "ip_address": "172.19.0.2"
>                           },
>                           {
>                             "protocol": "IPv6",
>                             "ip_address": "fd00:0:0:1::2"
>                           }
>                         ],
>                         "name": "ipv6"
>                       }
>                     ]
>                   }
>                 }
>               ],
>               "container": {
>                 "type": "DOCKER",
>                 "docker": {
>                   "image": "nginx",
>                   "network": "USER",
>                   "privileged": false
>                 },
>                 "network_infos": [
>                   {
>                     "name": "ipv6"
>                   }
>                 ]
>               }
>             }
>           ],
> 
> 
> Thanks,
> 
> Avinash sridharan
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message