----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68120/#review206685 -----------------------------------------------------------
PASS: Mesos patch 68120 was successfully built and tested. Reviews applied: `['68120']` All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2014/mesos-review-68120 - Mesos Reviewbot Windows On July 31, 2018, 7:26 p.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > 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. > > > Repository: mesos > > > Description > ------- > > 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 > ----- > > src/webui/app/controllers.js 7c228a1629d7b1dcce56b432f042f02d3ec1583b > src/webui/app/home.html ff965952d7a8d84c9dcf5555d5d6e7e045cd72bc > > > Diff: https://reviews.apache.org/r/68120/diff/2/ > > > Testing > ------- > > manual testing > > > Thanks, > > Benjamin Mahler > >