Re: Review Request 56809: Added test for nested container machine reboot case.

2017-02-22 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56809/#review166374 --- src/tests/containerizer/provisioner_docker_tests.cpp (lines 409

Re: Review Request 56809: Added test for nested container machine reboot case.

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

Re: Review Request 56809: Added test for nested container machine reboot case.

2017-02-21 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56809/ --- (Updated Feb. 21, 2017, 2:34 p.m.) Review request for mesos, Avinash

Re: Review Request 56809: Added test for nested container machine reboot case.

2017-02-21 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56809/ --- (Updated Feb. 21, 2017, 2 p.m.) Review request for mesos, Avinash sridharan,

Re: Review Request 56809: Added test for nested container machine reboot case.

2017-02-21 Thread Gilbert Song
> On Feb. 19, 2017, 8:32 p.m., Jie Yu wrote: > > src/tests/containerizer/nested_mesos_containerizer_tests.cpp, lines 998-1012 > > > > > > Instead of that, can we call 'provisioner->provision' and use > >

Re: Review Request 56809: Added test for nested container machine reboot case.

2017-02-19 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56809/#review166047 --- src/tests/containerizer/nested_mesos_containerizer_tests.cpp

Re: Review Request 56809: Added test for nested container machine reboot case.

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

Review Request 56809: Added test for nested container machine reboot case.

2017-02-17 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56809/ --- Review request for mesos, Avinash sridharan, Jie Yu, and Kevin Klues.