----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45014/#review124783 -----------------------------------------------------------
Patch looks great! Reviews applied: [45014] Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' COMPILER='gcc' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On March 18, 2016, 5:27 a.m., Jay Guo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45014/ > ----------------------------------------------------------- > > (Updated March 18, 2016, 5:27 a.m.) > > > Review request for mesos and Jie Yu. > > > Repository: mesos > > > Description > ------- > > Add /containers endpoint to return ResourceUsage. > > > Diffs > ----- > > src/slave/http.cpp 4eb1fafdfa72094511b0b2684a3c2705bd8c7c5e > src/slave/slave.hpp 7520cc356e2b1b7f5fff15f33071a46a7b05e762 > src/slave/slave.cpp 840534ff0687e82ed063c386e36bbabada230697 > src/tests/slave_tests.cpp ea1d776077bf638885db8421194aa4427c772169 > > Diff: https://reviews.apache.org/r/45014/diff/ > > > Testing > ------- > > make check > `curl agent_ip:port/containers` returns same json content as `curl > agent_ip:port/monitor/statistics.json` > > This is a draft patch of adding /containers endpoint to agent > [MESOS-4891](https://issues.apache.org/jira/browse/MESOS-4891) > > ContainerStatus will be added to response based on this patch. > > > Thanks, > > Jay Guo > >