> On July 23, 2013, 2:44 a.m., Ross Allen wrote: > > Several controllers in the UI call '/monitor/usage.json' on slaves. Is that > > the same endpoint being removed here? If so, we need to update the UI to > > use the new endpoint.
Good catch, yes that's being removed so I'll have to update the webui in this change as well. There's a bit of added complexity as we no longer expose "cpu_usage" through the endpoint. This means the webui will have to compute the usage from the raw values. i.e. (new_cpu_time - old_cpu_time) / (new_timestamp - old_timestamp) - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12765/#review23674 ----------------------------------------------------------- On July 19, 2013, 4:51 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12765/ > ----------------------------------------------------------- > > (Updated July 19, 2013, 4:51 p.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > We kept the usage.json endpoint for temporary backwards compatibility. > > > Diffs > ----- > > src/slave/monitor.hpp 52568ad8ec566f7cf36c249c76d798d44eacb578 > src/slave/monitor.cpp 4f3c91fce94918de43d7ad0b598d4902a46be8b6 > > Diff: https://reviews.apache.org/r/12765/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Ben Mahler > >
