----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25055/#review51539 -----------------------------------------------------------
src/master/master.cpp <https://reviews.apache.org/r/25055/#comment89947> The loopback address is guaranteed to be 127.x.x.x (any will work, 0.0.1 is standard) on any IPv4 machine and ::1 on any IPv6 machine. You probably want to extend this test to cover all of the IPv4 possibilities. src/scheduler/scheduler.cpp <https://reviews.apache.org/r/25055/#comment89948> worth a net::isLoopback() method? - Dominic Hamon On Aug. 25, 2014, 7:49 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25055/ > ----------------------------------------------------------- > > (Updated Aug. 25, 2014, 7:49 p.m.) > > > Review request for mesos and Ben Mahler. > > > Repository: mesos-git > > > Description > ------- > > Added logging when master/slaves/frameworks bind to loopback address. > > > Diffs > ----- > > src/master/master.cpp 18464baad0864ae890d91097035edf8c49b6f60c > src/sched/sched.cpp 447b96fd88111536fdd7704abd078cc16b5c7943 > src/scheduler/scheduler.cpp b7ce4de4b01a898bb6763dde070fe521e142ee6c > src/slave/slave.cpp 6253a5dcde543ff448ad2e710a74892620e3bb87 > > Diff: https://reviews.apache.org/r/25055/diff/ > > > Testing > ------- > > tested locally. > > > Thanks, > > Vinod Kone > >
