mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Mahler <>
Subject Review Request 55870: Update the allocator to handle frameworks with multiple roles.
Date Wed, 25 Jan 2017 21:55:12 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for mesos, Benjamin Bannier, Jay Guo, Guangya Liu, and Michael Park.

Bugs: MESOS-6635

Repository: mesos


The allocator now sets `Resource.AllocationInfo` when performing
an allocation, and the interface is updated to expose allocations
for each role within a framework.

The allocator also assumes that `Resource.AllocationInfo` is set
for inbound resources that are allocated (e.g. when adding an agent,
when adding a framework, when recovering resources). Note however,
that the necessary changes to the master and agent to enforce this
will be done via separate patches.


  include/mesos/allocator/allocator.hpp 558594983beb6ff49c1fddf875ba29c1983b1288 
  src/master/allocator/mesos/allocator.hpp 8e0f37a5eedd4d71d765991c0039b49c96f0ca53 
  src/master/allocator/mesos/hierarchical.hpp 9b66c23f26b37c02ed850c06c4518ea99078b02d 
  src/master/allocator/mesos/hierarchical.cpp c2211be7458755aeb91ef078e4bfe92ac474044a 
  src/master/master.hpp 8e8a9037af11cf95961b6498540a0fd486ed091b 
  src/master/master.cpp 73159328ce3fd838e02eba0e6a30cf69efc319ba 
  src/tests/allocator.hpp 1f9261d1e7afa027bebc07915ba3e871a58e6558 



Adjustments to existing tests are split out into a subsequent patch.

New tests for frameworks having multiple roles will be added as a follow up.


Benjamin Mahler

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