> On Jan. 23, 2017, 10:33 p.m., Michael Park wrote: > > Ship It! > > Michael Park wrote: > Do we have a patch to add `capabilities` to Agent `/state` endpoint as > well?
FYI I was looking into setting the MULTI_ROLE capability in the agent code, and I'd like to update what we committed to follow the approach we took for sending the agent's version. The version is communicated through the registration and re-registration messages as a separate field rather than being set inside `SlaveInfo`. This was because we don't handle an updated `SlaveInfo` currently (the agent considers `SlaveInfo` changes during restart to be incompatible, and the master does not update the `SlaveInfo` in the registry if it were to change). So, to avoid tackling these problems now, we can start with just having capabilities follow the same approach as what we did for the agent version. Make sense? Can you send patches for this? Note that it doesn't need to block any of your outstanding patches from going in AFAICT. - Benjamin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55710/#review162719 ----------------------------------------------------------- On Jan. 20, 2017, 5:29 a.m., Jay Guo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55710/ > ----------------------------------------------------------- > > (Updated Jan. 20, 2017, 5:29 a.m.) > > > Review request for mesos, Benjamin Bannier, Benjamin Mahler, and Guangya Liu. > > > 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 > ----- > > src/master/http.cpp a44621f39cb059e654a56f57f75b38947f3a4230 > src/tests/master_tests.cpp da7094dbbafbb0ab1153a0a4a6fcabd63888d67a > > Diff: https://reviews.apache.org/r/55710/diff/ > > > Testing > ------- > > 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 > >
