You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Jan Schlicht <ja...@mesosphere.io> on 2017/12/07 15:52:18 UTC

Re: Review Request 64370: Provided resource provider infos in 'UpdateState' message.

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

(Updated Dec. 7, 2017, 4:52 p.m.)


Review request for mesos, Benjamin Bannier and Jie Yu.


Changes
-------

Addressed issues.


Summary (updated)
-----------------

Provided resource provider infos in 'UpdateState' message.


Repository: mesos


Description (updated)
-------

To support sending 'ResourceProviderInfo's of all known resource
providers to the master as part of 'UpdateSlaveMessage', these
information has been added to the 'UpdateState' message that is sent
from resource provider to agents. Also, an agent now keeps track of all
'ResourceProviderInfo's.


Diffs (updated)
-----

  src/resource_provider/manager.cpp 8d8b2f1396230d6edba590b8b7e9b4ca51366efe 
  src/resource_provider/message.hpp c94e9c7f0bb512cc579b5fe569831002a32256b0 
  src/slave/slave.hpp fc762fb1e8aeb57b0d7ad551d960ec4be06e8e33 
  src/slave/slave.cpp dd830d93f8a08720231e3d4a6421f7a058ba6093 
  src/tests/resource_provider_manager_tests.cpp a4c19ca769e66110d9aba0bae4792df9db3fed01 


Diff: https://reviews.apache.org/r/64370/diff/2/

Changes: https://reviews.apache.org/r/64370/diff/1-2/


Testing
-------

make check


Thanks,

Jan Schlicht