----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62781/#review187156 -----------------------------------------------------------
FAIL: Failed to apply the dependent review: 62780. Failed command: `python.exe .\support\apply-reviews.py -n -r 62780` All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/62781 Relevant logs: - [apply-review-62780-stdout.log](http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/62781/logs/apply-review-62780-stdout.log): ``` error: patch failed: src/webui/master/static/index.html:7 error: src/webui/master/static/index.html: patch does not apply error: patch failed: src/webui/master/static/js/controllers.js:42 error: src/webui/master/static/js/controllers.js: patch does not apply error: patch failed: src/webui/master/static/pailer.html:24 error: src/webui/master/static/pailer.html: patch does not apply ``` - Mesos Reviewbot Windows 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 > >
