----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55625/#review161981 -----------------------------------------------------------
Patch looks great! Reviews applied: [55625] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos ReviewBot On Jan. 17, 2017, 8:42 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55625/ > ----------------------------------------------------------- > > (Updated Jan. 17, 2017, 8:42 p.m.) > > > Review request for mesos, Alexander Rukletsov and Benjamin Mahler. > > > Bugs: MESOS-6432 > https://issues.apache.org/jira/browse/MESOS-6432 > > > Repository: mesos > > > Description > ------- > > In the certain coarse-grained allocation scheme it is possible to game > quota since the allocator will perform allocations until the quotas > for all resource kinds are satisfied. This can lead to allocations for > certain resource kinds far exceeding the set quota. > > This patch changes the notion of what constitutes a "satisfied quota". > Where before a quota could only be satisfied if for all resource kinds > allocations had been me up to the set quota, we now consider a quota > satisfied as soon as at least one resource kinds is allocated up to > the set quota. We here take only "common" resource kinds into account > where a common resource is a resource present on every agent node. The > notion of common resource kinds is needed to avoid strong bias from > rare resources in the introduced quota allocation scheme. > > Ultimately this change can only be temporary and should be replaced > with a more complete fix, e.g., chunked allocations, see MESOS-3765. > > > Diffs > ----- > > src/master/allocator/mesos/hierarchical.cpp > c2211be7458755aeb91ef078e4bfe92ac474044a > src/tests/hierarchical_allocator_tests.cpp > 1edd0ecc8a93cd41532e1cf3641f67c780ab23a5 > > Diff: https://reviews.apache.org/r/55625/diff/ > > > Testing > ------- > > Tested of various Linux configurations in internal CI. > > > Thanks, > > Benjamin Bannier > >
