Re: Review Request 63035: Made `getMountNamespaceTarget` more reliable by ignoring exited childs.

2017-10-19 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63035/#review188690 --- Ship it! Ship It! - Jie Yu On Oct. 17, 2017, 1:45 p.m.,

Re: Review Request 63035: Made `getMountNamespaceTarget` more reliable by ignoring exited childs.

2017-10-18 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63035/#review188503 --- PASS: Mesos patch 63035 was successfully built and tested.

Re: Review Request 63035: Made `getMountNamespaceTarget` more reliable by ignoring exited childs.

2017-10-18 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63035/#review188497 --- Patch looks great! Reviews applied: [63074, 63035] Passed

Re: Review Request 63035: Made `getMountNamespaceTarget` more reliable by ignoring exited childs.

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

Re: Review Request 63035: Made `getMountNamespaceTarget` more reliable by ignoring exited childs.

2017-10-17 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63035/#review188323 --- PASS: Mesos patch 63035 was successfully built and tested.

Re: Review Request 63035: Made `getMountNamespaceTarget` more reliable by ignoring exited childs.

2017-10-17 Thread Andrei Budnik
> On Oct. 16, 2017, 5:05 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/utils.cpp > > Line 75 (original), 75 (patched) > > > > > > I think we can make `ns::getns` return a `Result`. Returnning > > `None()`

Re: Review Request 63035: Made `getMountNamespaceTarget` more reliable by ignoring exited childs.

2017-10-17 Thread Andrei Budnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63035/ --- (Updated Oct. 17, 2017, 1:45 p.m.) Review request for mesos, Alexander

Re: Review Request 63035: Made `getMountNamespaceTarget` more reliable by ignoring exited childs.

2017-10-16 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63035/#review188193 --- PASS: Mesos patch 63035 was successfully built and tested.

Re: Review Request 63035: Made `getMountNamespaceTarget` more reliable by ignoring exited childs.

2017-10-16 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63035/#review188182 --- Patch looks great! Reviews applied: [63035] Passed command:

Re: Review Request 63035: Made `getMountNamespaceTarget` more reliable by ignoring exited childs.

2017-10-16 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63035/#review188154 --- src/slave/containerizer/mesos/utils.cpp Line 75 (original), 75

Review Request 63035: Made `getMountNamespaceTarget` more reliable by ignoring exited childs.

2017-10-16 Thread Andrei Budnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63035/ --- Review request for mesos, Alexander Rukletsov, Jie Yu, and Kevin Klues. Bugs: