----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49610/ -----------------------------------------------------------
(Updated July 11, 2016, 8:47 a.m.) Review request for mesos and Alexander Rukletsov. Changes ------- Address review comments. Repository: mesos Description ------- This test checks that when setting aside unallocated resources to ensure that a quota guarantee can be met, we don't use resources that have been reserved for a different role. Diffs (updated) ----- src/tests/hierarchical_allocator_tests.cpp 0498cd5e54b0e4b87a767585a77699653aa52179 Diff: https://reviews.apache.org/r/49610/diff/ Testing ------- `./src/mesos-tests --gtest_filter="HierarchicalAllocatorTest.QuotaSetAsideReservedResources" --gtest_repeat=500 --gtest_break_on_failure` passes. When you apply a patch that breaks quota allocation behavior (https://gist.github.com/neilconway/97eed34e9b0b312b8511f0f4f57fe061), previously `make check` passes, but it fails with this new test added. Thanks, Neil Conway