Re: Review Request 40838: Environment variable: Implemented `Env` specified in docker image returned from docker pull.

2015-12-19 Thread Gilbert Song
> On Dec. 18, 2015, 5:18 p.m., Timothy Chen wrote: > > src/docker/docker.cpp, line 407 > > > > > > Do we have a test for this? Yes, we do. I wrote a test case for it, but did not post that test. Becuase I am not

Re: Review Request 40838: Environment variable: Implemented `Env` specified in docker image returned from docker pull.

2015-12-19 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40838/ --- (Updated Dec. 19, 2015, 5:35 p.m.) Review request for mesos, Artem Harutyunyan

Re: Review Request 40838: Environment variable: Implemented `Env` specified in docker image returned from docker pull.

2015-12-18 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40838/#review111332 --- src/docker/docker.cpp (line 363)

Re: Review Request 40838: Environment variable: Implemented `Env` specified in docker image returned from docker pull.

2015-12-09 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40838/ --- (Updated Dec. 9, 2015, 10:41 a.m.) Review request for mesos, Artem Harutyunyan

Re: Review Request 40838: Environment variable: Implemented `Env` specified in docker image returned from docker pull.

2015-12-02 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40838/ --- (Updated Dec. 2, 2015, 2:09 p.m.) Review request for mesos, Artem Harutyunyan