---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65139/#review195366
---
Ship it!
Ship It!
- Vinod Kone
On Jan. 13, 2018, 12:15 a.m., Gilbert Song wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65139/
> ---
>
> (Updated Jan. 13, 2018, 12:15 a.m.)
>
>
> Review request for mesos, Andrei Budnik, Alexander Rukletsov, Benjamin
> Mahler, Jie Yu, and Vinod Kone.
>
>
> Bugs: MESOS-8391
> https://issues.apache.org/jira/browse/MESOS-8391
>
>
> Repository: mesos
>
>
> Description
> ---
>
> This reverts commit 95decd404438abd422794524e01d72a889821566.
>
> There are two reasons to revert this commit:
> 1. After the agent recovers, if the nested containers that are
> launched beforehand are killed, they will no longer be updated
> with new status, because the `WAIT_NESTED_CONTAINER` call from
> the default executor will end up with a future forever. Please
> see MESOS-8391 for details.
> 2. The original commit makes the composing containerizer wait()
> and destroy() rely on the same future of a ContainerTermination
> promise. This would get into the bug that composing containerizer
> destroy() may fail due to the wait() future got discarded.
> Need to protect it by using `undiscardable()`. Please see
> MESOS-7926 for details.
>
>
> Diffs
> -
>
> src/slave/containerizer/composing.cpp
> 9ace70d9fbd78182715c5ef13fcaf7ad45f76f97
>
>
> Diff: https://reviews.apache.org/r/65139/diff/1/
>
>
> Testing
> ---
>
> make check
>
>
> Thanks,
>
> Gilbert Song
>
>