> On Feb. 27, 2018, 11:06 p.m., Benjamin Mahler wrote: > > src/webui/master/static/js/controllers.js > > Lines 562 (patched) > > <https://reviews.apache.org/r/65246/diff/5/?file=1966031#file1966031line569> > > > > Why did you need the additional query into the /flags here but not in > > agent? These fields should still be in $scope.state, no?
I should have added more information because it seems to be a bug in Mesos. Getting the `state` of a master returns the state of the leader master, even if the hostname used is not the one of the leader master. Parsing the `flags` endpoint correctly returns the flags of the master we want to reach. This problem did not appear when communicating with agents. > On Feb. 27, 2018, 11:06 p.m., Benjamin Mahler wrote: > > src/webui/master/static/js/controllers.js > > Lines 571-572 (patched) > > <https://reviews.apache.org/r/65246/diff/5/?file=1966031#file1966031line578> > > > > Why the `$('#alert').show()` here instead of the > > `$dialog.messageBox(...).open()` as was done before? I'm not familiar with > > the difference, should the UI use a consistent approach? Indeed, we used two different approaches in the past. `$('#alert').show();` is used many times to display alerts, while we were using a `$dialog` before this patch only in one case: if a user wanted to get unaccessible logs. The difference is that an alert is displayed when there is an error that was not expected (e.g. when reaching an endpoint) while a dialog was displayed when a user tried to do something he cannot do due to a configuration choice. I have removed the two uses of `$dialog` as they were not constructive, displaying buttons to access something that is not accessible and showing an alert being a strange user experience. `$dialog` is thus not used anymore following this patch, I have not removed the code supporting that feature as I think it makes sense for many use cases that are just not there yet. - Armand ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65246/#review198345 ----------------------------------------------------------- On Feb. 27, 2018, 11:40 p.m., Armand Grillet wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65246/ > ----------------------------------------------------------- > > (Updated Feb. 27, 2018, 11:40 p.m.) > > > Review request for mesos, Benjamin Mahler, Till Toenshoff, and Vinod Kone. > > > Bugs: MESOS-8454 > https://issues.apache.org/jira/browse/MESOS-8454 > > > Repository: mesos > > > Description > ------- > > Adds a new button to download the logs in addition to the ability to > view them. Does not display the buttons if the master or agent does not > have an external log file or a log directory. Displays the correct logs, > i.e. if you are on a non-leading master you will get the logs of that > master and not the leading master. > > Regarding the architectural choices, `HomeCtrl` is now similar to the > other controllers with an update function called when the state changes. > > > 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/65246/diff/6/ > > > Testing > ------- > > Tested using Google Chrome 64.0.3282.119. > > Created an High Availability Mode Mesos cluster locally: > ``` > $ zkServer start > $ bash mesos-master.sh --port='5061' --zk='zk://127.0.0.1:2181/mesos' > --work_dir='/tmp/master1' --quorum=1 > --webui_dir='/Users/Armand/Code/apache-mesos/src/webui' > --log_dir='/tmp/master1-log' > $ bash mesos-master.sh --port='5062' --zk='zk://127.0.0.1:2181/mesos' > --work_dir='/tmp/master2' --quorum=1 > --webui_dir='/Users/Armand/Code/apache-mesos/src/webui' > --log_dir='/tmp/master2-log' > $ bash mesos-agent.sh --port='5051' --master='zk://127.0.0.1:2181/mesos' > --work_dir='/tmp/agent1' --log_dir='/tmp/agent1-log' > ``` > Tested the download and streaming features on the home and agents Web UI > endpoints, from both `http://localhost:5061/` and `http://localhost:5062/`. > > New UI (masters): > ![New logs button master](https://i.imgur.com/Uz8aj1H.png) > > New UI (agents): > ![New logs button](https://i.imgur.com/tmGavCL.png) > > > Thanks, > > Armand Grillet > >