mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Meng Zhu <m...@mesosphere.io>
Subject Re: Review Request 67513: Added a master flag to configure minimum allocatable resources.
Date Fri, 15 Jun 2018 22:03:00 GMT

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

(Updated June 15, 2018, 3:02 p.m.)


Review request for mesos, Gastón Kleiman and Greg Mann.


Changes
-------

Removed JSON input format.


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


Repository: mesos


Description
-------

This patch adds a new master flag `min_allocatable_resources`.
It specifies one or more resources quantities that define the
minimum allocatable resources for the allocator. The allocator
will only offer resources that are more than at least one of
the specified resources.


Diffs (updated)
-----

  include/mesos/allocator/allocator.hpp 647869286d46920b98521e219ce92c3260058c35 
  src/master/allocator/mesos/allocator.hpp c453c015b234deff7efd00269da25dcec8cbf1ae 
  src/master/allocator/mesos/hierarchical.hpp e9d1742bb35004735e3cb357286b4e5b17436a5c 
  src/master/allocator/mesos/hierarchical.cpp b558228290e5ae6dbcc1b8a6e1fe69db9fd5874c 
  src/master/flags.hpp 94b8ac2211180416190448b71ea9c81c6c0cd7fc 
  src/master/flags.cpp cc3317ee5d740cb1d58b51ae00ceeb8d55754b9d 
  src/master/master.cpp 5db5a8da85f02323a5654c93ac47ec4aa7e711d2 
  src/tests/allocator.hpp 341efa665ad0ce897e087fb8d73ec50fd041d559 
  src/tests/api_tests.cpp 85635a8662469c7dbe4e77bb8da6eb450ec8f01c 
  src/tests/hierarchical_allocator_tests.cpp c97b2ba0884a7ded867c2d80e4749de54c89b5e4 
  src/tests/master_allocator_tests.cpp e1aef8a9625a805e7ad2dfad37bfeedee82f160d 
  src/tests/master_quota_tests.cpp 94d85c5a4b70ef2bc4e8689b896fe99f3acfd4b9 
  src/tests/reservation_tests.cpp 7d121bf56b913c3217dec00c57f81663e9831351 
  src/tests/resource_offers_tests.cpp 54aafdb4258ad7713c5f1a59956e7f76f0e84d5b 
  src/tests/slave_recovery_tests.cpp 2a92acc193b4db5001ae5bca53e7333ba7203210 


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

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


Testing
-------

make check
Fixed all existing tests.
Dedicate test added in a subsequent patch.


Thanks,

Meng Zhu


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