Re: Review Request 41491: Exposed docker/appc image manifest to mesos containerizer.

2016-01-19 Thread Gilbert Song
> On Jan. 17, 2016, 10:57 p.m., Qian Zhang wrote: > > src/slave/containerizer/mesos/provisioner/provisioner.hpp, line 60 > > > > > > We only expose Docker image manifest? But the summary of this RR says > > we will

Re: Review Request 41491: Exposed docker/appc image manifest to mesos containerizer.

2016-01-15 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41491/ --- (Updated Jan. 15, 2016, 11:04 a.m.) Review request for mesos, Artem

Re: Review Request 41491: Exposed docker/appc image manifest to mesos containerizer.

2016-01-15 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41491/#review114783 --- Patch looks great! Reviews applied: [41491] Passed command:

Re: Review Request 41491: Exposed docker/appc image manifest to mesos containerizer.

2016-01-15 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41491/#review114690 --- Patch looks great! Reviews applied: [41491] Passed command:

Re: Review Request 41491: Exposed docker/appc image manifest to mesos containerizer.

2016-01-14 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41491/ --- (Updated Jan. 14, 2016, 9:30 p.m.) Review request for mesos, Artem