mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Neil Conway <>
Subject Re: Review Request 49377: Fixed allocator to update total resources in quota sorter.
Date Mon, 04 Jul 2016 08:53:43 GMT

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

(Updated July 4, 2016, 8:53 a.m.)

Review request for mesos, Alexander Rukletsov, Joris Van Remoortere, and Michael Park.


Remove comment per review.

Bugs: MESOS-5698

Repository: mesos


Each DRFSorter tracks the total resources in the cluster. This means
that each sorter must be updated when the resources in the cluster have
changed, e.g., due to the creation of a dynamic reservation or a
persistent volume. In the previous implementation, the quota role sorter
was not updated for non-quota roles when a reservation or persistent
volume was created by a framework. This resulted in inconsistency
between the total resources in the allocator and the quota role sorter.

This could cause several problems. First, removing a slave from the
cluster would leak resources in the quota role sorter. Second, certain
interleavings of slave removals and reserve/unreserve operations by
frameworks and via HTTP endpoints could lead to CHECK failures.

Diffs (updated)

  src/master/allocator/mesos/hierarchical.cpp eca949e07abb00423a2f35a56eedc5d4287d81f3 
  src/master/allocator/sorter/drf/sorter.cpp 967290d4d1100208900b4b724422c3218abc23cb 
  src/tests/persistent_volume_endpoints_tests.cpp 971a7b38d26b26c8543815e467f30cbb83ee412c



make check on OSX and recent Arch Linux.


Neil Conway

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