mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <benjamin.bann...@mesosphere.io>
Subject Re: Review Request 65591: Explicitly tracked resource providers in master.
Date Thu, 15 Feb 2018 14:54:11 GMT

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

(Updated Feb. 15, 2018, 3:54 p.m.)


Review request for mesos, Jie Yu and Jan Schlicht.


Changes
-------

* addressed comments from Jan
* re-added reconciliation of operations on agent-default resources


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


Repository: mesos


Description
-------

This patch adds explicit tracking of resource providers to the master
process. While we already had explicitly send resource provider
information in e.g., `UpdateSlaveMessage`, we only stored that
information aggregated over the full agent in the master up to now.


Diffs (updated)
-----

  src/common/protobuf_utils.cpp b5c2997ada8362e42150fa3cfd762120e5ea715f 
  src/master/master.hpp c4d3c8080dc7899968b1030696172ed73d473bc1 
  src/master/master.cpp b06d7a6e2fbbb81b97eaf537d5b6745c73dc867d 


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

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


Testing
-------

`make check`


Thanks,

Benjamin Bannier


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