-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55710/
-----------------------------------------------------------

(Updated Jan. 20, 2017, 1:29 p.m.)


Review request for mesos, Benjamin Bannier, Benjamin Mahler, and Guangya Liu.


Changes
-------

Added a test to check `capablities` is included in `/state` response.


Bugs: MESOS-6902
    https://issues.apache.org/jira/browse/MESOS-6902


Repository: mesos


Description
-------

We introduced Capabilities to SlaveInfo protobuf message in ec1a326.
It is automatically wrapped into response of GetState v1 API. This
patch added this field to /state v0 API for consistency.


Diffs (updated)
-----

  src/master/http.cpp a44621f39cb059e654a56f57f75b38947f3a4230 
  src/tests/master_tests.cpp da7094dbbafbb0ab1153a0a4a6fcabd63888d67a 

Diff: https://reviews.apache.org/r/55710/diff/


Testing (updated)
-------

make check GTEST_FILTER="MasterTest.StateEndpointAgentCapabilities"
[       OK ] MasterTest.StateEndpointAgentCapabilities (85 ms)
[----------] 1 test from MasterTest (94 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (122 ms total)

make check on Ubuntu 14.04


Thanks,

Jay Guo

Reply via email to