----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49015/#review138955 -----------------------------------------------------------
include/mesos/agent/agent.proto (line 168) <https://reviews.apache.org/r/49015/#comment204148> s/of/about/ here and below. include/mesos/agent/agent.proto (line 169) <https://reviews.apache.org/r/49015/#comment204149> s/meatadata/metadata/ include/mesos/agent/agent.proto (line 170) <https://reviews.apache.org/r/49015/#comment204150> s/of containers/of the containers/ include/mesos/agent/agent.proto (line 177) <https://reviews.apache.org/r/49015/#comment204145> s/status/container_status/ here and below. include/mesos/agent/agent.proto (line 178) <https://reviews.apache.org/r/49015/#comment204146> s/stats/resource_statistics/ here and below. src/slave/http.cpp (line 660) <https://reviews.apache.org/r/49015/#comment204154> hmm. looks like this is not reusing any code from `_containers()`. can you extract the common code from `_containers` (e.g., authz) into `__containers()` and re-use that here. similar to what we did with `getTasks()`. - Vinod Kone On June 21, 2016, 9:55 a.m., Jay Guo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49015/ > ----------------------------------------------------------- > > (Updated June 21, 2016, 9:55 a.m.) > > > Review request for mesos, Anand Mazumdar, haosdent huang, and Vinod Kone. > > > Bugs: MESOS-5518 > https://issues.apache.org/jira/browse/MESOS-5518 > > > Repository: mesos > > > Description > ------- > > Implemented GET_CONTAINERS Call in v1 Agent API. > > > Diffs > ----- > > include/mesos/agent/agent.proto 1792d40140da3687faec4bbf446c97eb8ccab864 > include/mesos/v1/agent/agent.proto 7b5988fdb71deba907d76df39052579af4391be4 > src/slave/http.cpp b32afb6b776f2fecac850b054ada61aa90ac0052 > src/slave/slave.hpp 58ff2bfac6918d989ab36b67cf6ba2f3657c8356 > src/tests/api_tests.cpp 55df322f6a843e52d67bb09d3fe763739515602b > > Diff: https://reviews.apache.org/r/49015/diff/ > > > Testing > ------- > > The test is flaky and I would appreciate some suggestions. > > If you run individual test `make check > GTEST_FILTER="ContentType/AgentAPITest.GetContainers/0"`, it may _pass_ or > throw _segfault_ (and sometimes the test passes although teardown fails). > However, when running whole testset `make check > GTEST_FILTER="*AgentAPITest*"`, it is _very likely_ that GET_CONTAINERS > related tests _fail with segfaults_. > > I suspect that `Owned<agent::Response>` being captured is causing problem but > I'm running out of ideas at this point. > > > Thanks, > > Jay Guo > >
