----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65245/#review195926 -----------------------------------------------------------
src/webui/master/static/agent.html Line 45 (original), 45 (patched) <https://reviews.apache.org/r/65245/#comment275323> Hmm. This is inconsistent with how we did the stream vs download for sandboxes. The stream is just a hyper link with the filename followed by a Download button. Can we do the same with these logs for consistency? A hyperlink "LOG" that opens up pailer (which is what it is today) and a "Download" button next to it with some space (like we did for sandboxes). - Vinod Kone On Jan. 20, 2018, 4:14 p.m., Armand Grillet wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65245/ > ----------------------------------------------------------- > > (Updated Jan. 20, 2018, 4:14 p.m.) > > > Review request for mesos, Benjamin Mahler and Vinod Kone. > > > Bugs: MESOS-8454 > https://issues.apache.org/jira/browse/MESOS-8454 > > > Repository: mesos > > > Description > ------- > > The home and agents Web UI are now more descriptive. The name of the > functions called when clicking these two links have also been updated. > > > Diffs > ----- > > src/webui/master/static/agent.html 908d01d1d28f2a4fc595b581fa16515c5be1410a > src/webui/master/static/home.html df3eef61eb9564dbdfc3d138ecfd18e85f747f11 > src/webui/master/static/js/controllers.js > 59665869dbea77c00740e42f2590473181dfe2fe > > > Diff: https://reviews.apache.org/r/65245/diff/1/ > > > Testing > ------- > > Checked manually that the feature was still working on the home and agents > endpoints. > > > Thanks, > > Armand Grillet > >