Re: Review Request 56284: Added ProvisionerDockerLocalStoreTest.MissingLayer test.

2017-02-08 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56284/#review164789 --- Ship it! Ship It! - Jie Yu On Feb. 8, 2017, 8:30 p.m., Ilya

Re: Review Request 56284: Added ProvisionerDockerLocalStoreTest.MissingLayer test.

2017-02-08 Thread Ilya Pronin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56284/ --- (Updated Feb. 8, 2017, 8:30 p.m.) Review request for mesos, Jie Yu and Timothy

Re: Review Request 56284: Added ProvisionerDockerLocalStoreTest.MissingLayer test.

2017-02-07 Thread Ilya Pronin
> On Feb. 7, 2017, 6:57 p.m., Gilbert Song wrote: > > src/tests/containerizer/provisioner_docker_tests.cpp, lines 241-242 > > > > > > Maybe you should fix the comment. Yes, the comment should be changed. > On

Re: Review Request 56284: Added ProvisionerDockerLocalStoreTest.MissingLayer test.

2017-02-07 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56284/#review164539 --- src/tests/containerizer/provisioner_docker_tests.cpp (lines 241

Re: Review Request 56284: Added ProvisionerDockerLocalStoreTest.MissingLayer test.

2017-02-06 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56284/#review164449 --- Patch looks great! Reviews applied: [56174, 56284] Passed

Re: Review Request 56284: Added ProvisionerDockerLocalStoreTest.MissingLayer test.

2017-02-06 Thread Ilya Pronin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56284/ --- (Updated Feb. 6, 2017, 8:40 p.m.) Review request for mesos, Jie Yu and Timothy