----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62781/#review187160 -----------------------------------------------------------
Patch looks great! Reviews applied: [62780, 62781] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On Oct. 5, 2017, 12:35 a.m., Gaston Kleiman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62781/ > ----------------------------------------------------------- > > (Updated Oct. 5, 2017, 12:35 a.m.) > > > Review request for mesos, Benjamin Mahler and James Peach. > > > Repository: mesos > > > Description > ------- > > This path extracts the generation of URLs of agent endpoints to a single > function. > > 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/master/static/browse.html > c586b9050e14697ca89fabfe636c2d18a4ecf6f5 > src/webui/master/static/js/controllers.js > cb1f093ea21864eadd0ae32f3c33a7ff88b6fc27 > src/webui/master/static/js/services.js > b99ad685e9614ef77586efb5fa52345fc173e51e > > > Diff: https://reviews.apache.org/r/62781/diff/2/ > > > Testing > ------- > > Verified that the UI still works when running a master and agent on a remote > server. > > I tried: > > * Clicking all the tabs. > * Viewing the master/agent logs. > * Browsing a task sandbox and reading a file using the pailer. > > > Thanks, > > Gaston Kleiman > >
