> On 2011-11-14 20:26:15, Benjamin Hindman wrote: > > src/slave/slave.cpp, lines 278-280 > > <https://reviews.apache.org/r/2770/diff/1/?file=56823#file56823line278> > > > > Feel free to just stick all of conf.get(...) into the set_webui_port > > parameter.
Done. - Sumedh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2770/#review3220 ----------------------------------------------------------- On 2011-11-15 02:29:18, Sumedh Sawant wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2770/ > ----------------------------------------------------------- > > (Updated 2011-11-15 02:29:18) > > > Review request for mesos, Benjamin Hindman and Andy Konwinski. > > > Summary > ------- > > Added webUI field to the SlaveInfo message so that a master can correctly > display hyperlinks to slaves in its webUI. > > > This addresses bug MESOS-33. > https://issues.apache.org/jira/browse/MESOS-33 > > > Diffs > ----- > > include/mesos/mesos.proto 33248ec > src/master/http.cpp 47caf48 > src/slave/slave.cpp de5716c > src/webui/master/index.tpl 16e3446 > > Diff: https://reviews.apache.org/r/2770/diff > > > Testing > ------- > > > Thanks, > > Sumedh > >
