Re: Review Request 52435: Fixed children list issue and log in containerizer destroy.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52435/ --- (Updated Sept. 30, 2016, 11:20 a.m.) Review request for mesos, Artem Harutyunyan, Jie Yu, and Vinod Kone. Changes --- Added 'return' for default case. Repository: mesos Description --- This patch fixed the following issues: 1. Current the children list does not erase the nested container when the nested container is destroyed. We need to maintain this information. 2. Removed redundant 'Destroying nested container' log. 3. Added log in containerizer destory for container state change. Diffs (updated) - src/slave/containerizer/mesos/containerizer.hpp c542902714ccf2a951bf73fef62902f553d7ea1d src/slave/containerizer/mesos/containerizer.cpp 522d2c37229b07b66a0824c3e246c32f8d803b10 Diff: https://reviews.apache.org/r/52435/diff/ Testing --- make check Thanks, Gilbert Song
Review Request 52435: Fixed children list issue and log in containerizer destroy.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52435/ --- Review request for mesos, Artem Harutyunyan, Jie Yu, and Vinod Kone. Repository: mesos Description --- This patch fixed the following issues: 1. Current the children list does not erase the nested container when the nested container is destroyed. We need to maintain this information. 2. Removed redundant 'Destroying nested container' log. 3. Added log in containerizer destory for container state change. Diffs - src/slave/containerizer/mesos/containerizer.hpp c542902714ccf2a951bf73fef62902f553d7ea1d src/slave/containerizer/mesos/containerizer.cpp 522d2c37229b07b66a0824c3e246c32f8d803b10 Diff: https://reviews.apache.org/r/52435/diff/ Testing --- make check Thanks, Gilbert Song
