Re: Review Request 36585: Exposed docker container IP via state.json

2015-07-21 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36585/ --- (Updated July 21, 2015, 1:02 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 36585: Exposed docker container IP via state.json

2015-07-21 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36585/#review92452 --- src/docker/executor.cpp (line 163)

Re: Review Request 36585: Exposed docker container IP via state.json

2015-07-21 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36585/#review92456 --- Patch looks great! Reviews applied: [36574, 36575, 36580, 36585]

Re: Review Request 36585: Exposed docker container IP via state.json

2015-07-21 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36585/#review92471 --- Ship it! src/tests/docker_containerizer_tests.cpp (line 729)