Re: Review Request 45621: Added tests for docker spec helpers.

2016-04-04 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45621/#review126973 --- Patch looks great! Reviews applied: [44672, 44673, 44674, 45621]

Re: Review Request 45621: Added tests for docker spec helpers.

2016-04-04 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45621/ --- (Updated April 4, 2016, 11:49 a.m.) Review request for mesos, Guangya Liu,

Re: Review Request 45621: Added tests for docker spec helpers.

2016-04-04 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45621/#review126839 --- Fix it, then Ship it!

Re: Review Request 45621: Added tests for docker spec helpers.

2016-04-02 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45621/#review126705 --- src/tests/containerizer/docker_spec_tests.cpp (line 158)

Review Request 45621: Added tests for docker spec helpers.

2016-04-02 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45621/ --- Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen. Bugs: