Re: Review Request 32744: PortMapping: change to not host namespace symlink handles in /var/run/netns.

2015-04-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32744/#review78707 --- Patch looks great! Reviews applied: [32744] All tests passed. - M

Re: Review Request 32744: PortMapping: change to not host namespace symlink handles in /var/run/netns.

2015-04-02 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32744/ --- (Updated April 2, 2015, 7:06 p.m.) Review request for mesos, Jie Yu and Cong Wa

Re: Review Request 32744: PortMapping: change to not host namespace symlink handles in /var/run/netns.

2015-04-02 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32744/#review78669 --- Ship it! This looks great! Just some nits! src/slave/containerize

Re: Review Request 32744: PortMapping: change to not host namespace symlink handles in /var/run/netns.

2015-04-01 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32744/#review78631 --- Patch looks great! Reviews applied: [32744] All tests passed. - M

Re: Review Request 32744: PortMapping: change to not host namespace symlink handles in /var/run/netns.

2015-04-01 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32744/ --- (Updated April 1, 2015, 10:36 p.m.) Review request for mesos, Jie Yu and Cong W

Re: Review Request 32744: PortMapping: change to not host namespace symlink handles in /var/run/netns.

2015-04-01 Thread Cong Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32744/#review78601 --- At least the comments above getContainerIdFromSymlink() definition n

Re: Review Request 32744: PortMapping: change to not host namespace symlink handles in /var/run/netns.

2015-04-01 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32744/ --- (Updated April 1, 2015, 9:19 p.m.) Review request for mesos, Jie Yu and Cong Wa

Review Request 32744: PortMapping: change to not host namespace symlink handles in /var/run/netns.

2015-04-01 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32744/ --- Review request for mesos, Jie Yu and Cong Wang. Bugs: mesos-2574 https://is