----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64151/#review192860 -----------------------------------------------------------
include/mesos/agent/agent.proto Lines 445 (patched) <https://reviews.apache.org/r/64151/#comment271184> Let's introduce a nested message `GetResourceProviders.ResourceProvider` here so that we can add more information later (e.g., resource provider-specific resources, offer operations, ...). This message would contain just a `repeated ResourceProvider resource_providers = 1` for now. include/mesos/v1/agent/agent.proto Lines 445 (patched) <https://reviews.apache.org/r/64151/#comment271185> Ditto. - Benjamin Bannier On Dec. 5, 2017, 10:48 a.m., Jan Schlicht wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64151/ > ----------------------------------------------------------- > > (Updated Dec. 5, 2017, 10:48 a.m.) > > > Review request for mesos, Benjamin Bannier and Jie Yu. > > > Bugs: MESOS-8270 > https://issues.apache.org/jira/browse/MESOS-8270 > > > Repository: mesos > > > Description > ------- > > The 'GET_RESOURCE_PROVIDERS' call will list all subscribed local > resource providers of an agent. > > > Diffs > ----- > > include/mesos/agent/agent.proto 0f92f73ba0f7729f0ba7cd89a692ab3685125e8b > include/mesos/v1/agent/agent.proto 012ffef5f0dd7720fa95ae484c99479aaf256d7b > src/resource_provider/manager.hpp c2aeb15b0b8ebd167ddf9c42c9fc396d4c0a126b > src/resource_provider/manager.cpp 8d8b2f1396230d6edba590b8b7e9b4ca51366efe > src/slave/http.hpp a51831cdcebc1998ce6d4c3c19285e598a4ec9a3 > src/slave/http.cpp fd0e809d5dc7722e573b66621f75d791f0911dfa > src/slave/validation.cpp 32781fd8f124f71e61744804aec3fe4da59a5df2 > src/tests/api_tests.cpp 53c705ed4775846ba7eb42da57f962895f5669aa > src/tests/resource_provider_manager_tests.cpp > a4c19ca769e66110d9aba0bae4792df9db3fed01 > > > Diff: https://reviews.apache.org/r/64151/diff/5/ > > > Testing > ------- > > make check > > > Thanks, > > Jan Schlicht > >
