----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48991/#review139108 -----------------------------------------------------------
Fix it, then Ship it! src/slave/containerizer/mesos/isolators/network/cni/cni.cpp (lines 1493 - 1495) <https://reviews.apache.org/r/48991/#comment204255> Let's not use glog in a subprocess. Please do not log anything. - Jie Yu On June 21, 2016, 6:11 p.m., Gilbert Song wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48991/ > ----------------------------------------------------------- > > (Updated June 21, 2016, 6:11 p.m.) > > > Review request for mesos, Avinash sridharan, Guangya Liu, Artem Harutyunyan, > Jie Yu, and Qian Zhang. > > > Bugs: MESOS-5669 > https://issues.apache.org/jira/browse/MESOS-5669 > > > Repository: mesos > > > Description > ------- > > Added exception for etc hostname mount in cni isolator. > > > Diffs > ----- > > src/slave/containerizer/mesos/isolators/network/cni/cni.cpp > 9fc709e95ecff7e97e159ed05a4efcf4e887d5e0 > > Diff: https://reviews.apache.org/r/48991/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Gilbert Song > >
