----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65351/#review196357 -----------------------------------------------------------
PASS: Mesos patch 65351 was successfully built and tested. Reviews applied: `['65351']` All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/65351 - Mesos Reviewbot Windows On Jan. 26, 2018, 3:16 p.m., Armand Grillet wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65351/ > ----------------------------------------------------------- > > (Updated Jan. 26, 2018, 3:16 p.m.) > > > Review request for mesos and Benjamin Mahler. > > > Bugs: MESOS-8495 > https://issues.apache.org/jira/browse/MESOS-8495 > > > Repository: mesos > > > Description > ------- > > Use the alphabetical order of the name of the roles by > default to display their hierarchy straightforwardly. > > > Diffs > ----- > > src/webui/master/static/js/app.js 463c563cd121e881f7d137123534a047c5e420d2 > src/webui/master/static/roles.html 7578852e4e2ad29a0f07e16dcff85af6a2255b02 > > > Diff: https://reviews.apache.org/r/65351/diff/1/ > > > Testing > ------- > > ``` > $ bash mesos-master.sh --port='5061' --work_dir='/tmp/master1' > --webui_dir='/Users/Armand/Code/apache-mesos/src/webui' > --log_dir='/tmp/master1-log' --weights="example=42,example/a=2" > --registry=in_memory > ``` > UI before patch when viewing the roles tab: > ![Old UI](https://i.imgur.com/dflxj11.png) > UI by default after patch: > ![New UI](https://i.imgur.com/PRyAwNs.png) > > > Thanks, > > Armand Grillet > >