----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61463/ -----------------------------------------------------------
(Updated Aug. 8, 2017, 5:43 p.m.) Review request for mesos, Gastón Kleiman, Gilbert Song, Jie Yu, Kevin Klues, and Vinod Kone. Changes ------- Addressed comments. Summary (updated) ----------------- Fixed a bug in the test `NamespacesIsolatorTest`. Repository: mesos Description (updated) ------- In this test, we used the command `stat -c %i` to get the current process's pid/ipc namespace inode, that is not correct since `/proc/self/ns/pid` and `/proc/self/ns/ipc` are symbol links, we should get the inode that the link references rather than the link itself. So we need to add a `-L` option in the `stat` command. Diffs (updated) ----- src/tests/containerizer/isolator_tests.cpp f3c541c92f8ecc5c12356363bbbf3561d3b75230 Diff: https://reviews.apache.org/r/61463/diff/2/ Changes: https://reviews.apache.org/r/61463/diff/1-2/ Testing ------- Thanks, Qian Zhang
