-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2770/
-----------------------------------------------------------
(Updated 2011-11-15 02:29:18.050994)
Review request for mesos, Benjamin Hindman and Andy Konwinski.
Changes
-------
Changed "required string public_hostname" to "required string webui_hostname"
in include/mesos/mesos.proto
Changed "JSON::Object model(const Slave& slave)" in src/master/http.cpp to use
"slave.info.webui_hostname" and swapped the order of the url and port
references
Changed line 273 in src/slave/slave.cpp to use "webui_hostname"
Changed line 278 in src/slave/slave.cpp to eliminate the unnecessary webuiPort
variable by passing the result of "conf.get<int>("webui_port", 8081)" directly
as a parameter to "info.set_webui_port()
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 (updated)
-----
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
