mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jay Guo <>
Subject Re: Review Request 49015: Implemented GET_CONTAINERS Call in v1 Agent API.
Date Tue, 21 Jun 2016 09:55:30 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated June 21, 2016, 9:55 a.m.)

Review request for mesos, Anand Mazumdar, haosdent huang, and Vinod Kone.

Bugs: MESOS-5518

Repository: mesos


Implemented GET_CONTAINERS Call in v1 Agent API.

Diffs (updated)

  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 



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.


Jay Guo

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message