mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Mahler <bmah...@apache.org>
Subject Review Request 70508: Fixed the flaky ExamplesTest.DynamicReservationFramework.
Date Fri, 19 Apr 2019 20:18:25 GMT

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

Review request for mesos, Chun-Hung Hsiao and Meng Zhu.


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


Repository: mesos


Description
-------

The test failed in MESOS-5804 due to the following race:

  1. Framework launches task T, moves from RESERVED to
     TASK_RUNNING state.
  2. Allocation cycle triggers and will send the unreserved
     resources to the framework.
  3. Before the offer gets to the framework, task T finishes and
     framework moves from TASK_RUNNING to RESERVED.
  4. In the RESERVED state, the framework expects the reservation
     in the offer. But, it's coming in a later offer, and the one
     that arrives is for the unreserved resources since it was
     generated while the task was still running.

Tne fix applied here for this specific race is to use a 2 week
filter rather than a 0 second filter. That would ensure that the
unreserved resources do not get re-offered to the framework on
their own. However, this fix does not work until MESOS-9616 is
resolved.


Diffs
-----

  src/examples/dynamic_reservation_framework.cpp f9c7dfe46a1e8dd1bc8eae45ed1b65b7a6d60dfc



Diff: https://reviews.apache.org/r/70508/diff/1/


Testing
-------

The test fails now due to MESOS-9616, will need to re-test
once that is resolved.


Thanks,

Benjamin Mahler


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