mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mesos Reviewbot Windows <revi...@mesos.apache.org>
Subject Re: Review Request 67513: Added a master flag to configure minimum allocatable resources.
Date Mon, 11 Jun 2018 22:57:41 GMT

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



PASS: Mesos patch 67513 was successfully built and tested.

Reviews applied: `['67510', '67516', '67513']`

All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/67513

- Mesos Reviewbot Windows


On June 11, 2018, 9:37 p.m., Meng Zhu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67513/
> -----------------------------------------------------------
> 
> (Updated June 11, 2018, 9:37 p.m.)
> 
> 
> Review request for mesos, Gastón Kleiman and Greg Mann.
> 
> 
> 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
> -----
> 
>   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 15dbbc1b3efe8e139a2c6b1a4a7087743dbe950c 
>   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/1/
> 
> 
> 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