----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69972/ -----------------------------------------------------------
(Updated Feb. 18, 2019, 10:31 p.m.) Review request for mesos, Andrei Budnik and Gilbert Song. Changes ------- Minor changes. Bugs: MESOS-9507 https://issues.apache.org/jira/browse/MESOS-9507 Repository: mesos Description ------- There are two cases we need to handle: 1. The checkpointed docker volumes file does not exist. 2. The checkpointed docker volumes file is empty. For both of the two cases, in the recovery of `docker/volume` isolator, we should remove the orphan container's checkpoint directory and then skip the container. But for recoverable containers, we return an error since the above two cases should not happen for them. Diffs (updated) ----- src/slave/containerizer/mesos/isolators/docker/volume/isolator.hpp 2fd0493bc3e1749c12b81b9f2d281b86cb2c410f src/slave/containerizer/mesos/isolators/docker/volume/isolator.cpp a72fc84da6fb0f24d363dd4c635500510da675d8 Diff: https://reviews.apache.org/r/69972/diff/3/ Changes: https://reviews.apache.org/r/69972/diff/2-3/ Testing ------- Thanks, Qian Zhang