Re: Review Request 68158: Fixed the iptables deadlock in CNI port mapper plugin.

2018-08-06 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68158/ --- (Updated Aug. 6, 2018, 11:42 p.m.) Review request for mesos, Avinash

Re: Review Request 68158: Fixed the iptables deadlock in CNI port mapper plugin.

2018-08-06 Thread Jie Yu
> On Aug. 6, 2018, 8:40 p.m., Chun-Hung Hsiao wrote: > > src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp > > Lines 382 (patched) > > > > > > If `iptables` prints something

Re: Review Request 68158: Fixed the iptables deadlock in CNI port mapper plugin.

2018-08-06 Thread Chun-Hung Hsiao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68158/#review206904 --- Fix it, then Ship it!

Re: Review Request 68158: Fixed the iptables deadlock in CNI port mapper plugin.

2018-08-06 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68158/ --- (Updated Aug. 6, 2018, 8:30 p.m.) Review request for mesos, Avinash sridharan,

Re: Review Request 68158: Fixed the iptables deadlock in CNI port mapper plugin.

2018-08-06 Thread Chun-Hung Hsiao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68158/#review206903 ---

Re: Review Request 68158: Fixed the iptables deadlock in CNI port mapper plugin.

2018-08-06 Thread Chun-Hung Hsiao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68158/#review206892 --- Ship it! Ship It! - Chun-Hung Hsiao On Aug. 6, 2018, 4:52

Re: Review Request 68158: Fixed the iptables deadlock in CNI port mapper plugin.

2018-08-06 Thread Deepak Goel
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68158/#review206887 --- Ship it! Ship It! - Deepak Goel On Aug. 6, 2018, 4:52 p.m.,

Re: Review Request 68158: Fixed the iptables deadlock in CNI port mapper plugin.

2018-08-06 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68158/ --- (Updated Aug. 6, 2018, 4:52 p.m.) Review request for mesos, Avinash sridharan,

Re: Review Request 68158: Fixed the iptables deadlock in CNI port mapper plugin.

2018-08-04 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68158/ --- (Updated Aug. 5, 2018, 5:30 a.m.) Review request for mesos, Avinash sridharan,

Re: Review Request 68158: Fixed the iptables deadlock in CNI port mapper plugin.

2018-08-02 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68158/ --- (Updated Aug. 2, 2018, 7:17 p.m.) Review request for mesos, Avinash sridharan,

Re: Review Request 68158: Fixed the iptables deadlock in CNI port mapper plugin.

2018-08-02 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68158/#review206785 --- FAIL: Some of the unit tests failed. Please check the relevant

Review Request 68158: Fixed the iptables deadlock in CNI port mapper plugin.

2018-08-01 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68158/ --- Review request for mesos, Avinash sridharan, Chun-Hung Hsiao, and Greg Mann.