----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68120/ -----------------------------------------------------------
(Updated July 31, 2018, 7:26 p.m.) Review request for mesos, Armand Grillet and Gastón Kleiman. Changes ------- Updated the html to indicate that the log viewer is pointing to the leading master's log. Repository: mesos Description (updated) ------- This path pulls up the leader URL prefix generation to a top-level function in order to have it re-used across controllers. This also ensures that everything shown is consistently coming from the leading master, specifically, now the log viewer shows the leading master's log (and the UI messaging is updated in this patch accordingly). Having this logic in just one place makes it easier to modify it, e.g., to make it possible to use the UI via a reverse proxy. Diffs (updated) ----- src/webui/app/controllers.js 7c228a1629d7b1dcce56b432f042f02d3ec1583b src/webui/app/home.html ff965952d7a8d84c9dcf5555d5d6e7e045cd72bc Diff: https://reviews.apache.org/r/68120/diff/2/ Changes: https://reviews.apache.org/r/68120/diff/1-2/ Testing ------- manual testing Thanks, Benjamin Mahler
