Re: Review Request 52827: Added backend suffix to image layer rootfs path.

2016-10-15 Thread Qian Zhang
> On Oct. 14, 2016, 7:34 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/provisioner/docker/paths.cpp, lines 61-64 > > > > > > In fact, i think we should use aufs style whiteout by default because > > that's

Re: Review Request 52827: Added backend suffix to image layer rootfs path.

2016-10-14 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52827/#review152721 --- Fix it, then Ship it! Can you follow up with a test to test

Re: Review Request 52827: Added backend suffix to image layer rootfs path.

2016-10-14 Thread Qian Zhang
> On Oct. 14, 2016, 7:34 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/provisioner/docker/paths.cpp, lines 61-64 > > > > > > In fact, i think we should use aufs style whiteout by default because > > that's

Re: Review Request 52827: Added backend suffix to image layer rootfs path.

2016-10-14 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52827/ --- (Updated Oct. 14, 2016, 11:36 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 52827: Added backend suffix to image layer rootfs path.

2016-10-13 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52827/#review152585 --- src/slave/containerizer/mesos/provisioner/docker/paths.cpp

Re: Review Request 52827: Added backend suffix to image layer rootfs path.

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

Review Request 52827: Added backend suffix to image layer rootfs path.

2016-10-13 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52827/ --- Review request for mesos and Jie Yu. Bugs: MESOS-6360