Re: Review Request 54846: Removed `docker exec` when perform health checks in docker executor.

2017-02-14 Thread Gastón Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54846/#review165493 --- Ship it! Ship It! - Gastón Kleiman On Dec. 18, 2016, 5:30

Re: Review Request 54846: Removed `docker exec` when perform health checks in docker executor.

2017-02-08 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54846/#review164759 --- Bad patch! Reviews applied: [54846] Failed command: python

Re: Review Request 54846: Removed `docker exec` when perform health checks in docker executor.

2017-02-08 Thread James DeFelice
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54846/#review164683 --- src/docker/executor.cpp (line 486)

Re: Review Request 54846: Removed `docker exec` when perform health checks in docker executor.

2016-12-18 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54846/ --- (Updated Dec. 18, 2016, 5:30 p.m.) Review request for mesos, Alexander

Review Request 54846: Removed `docker exec` when perform health checks in docker executor.

2016-12-18 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54846/ --- Review request for mesos, Alexander Rukletsov, Benjamin Mahler, Gastón Kleiman,