mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Mahler <bmah...@apache.org>
Subject Re: Review Request 69981: Fixed a flaky test `MasterQuotaTest.RemoveSingleQuota`.
Date Thu, 14 Feb 2019 17:12:56 GMT

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


Fix it, then Ship it!





src/tests/master_quota_tests.cpp
Lines 524-525 (patched)
<https://reviews.apache.org/r/69981/#comment298724>

    Hm.. we're actually waiting for the 200 OK, is this needed because the http handler doesn't
wait for the allocator to finish processing the update?
    
    Probably we should spell it out here and below so it's clear


- Benjamin Mahler


On Feb. 14, 2019, 12:27 a.m., Meng Zhu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/69981/
> -----------------------------------------------------------
> 
> (Updated Feb. 14, 2019, 12:27 a.m.)
> 
> 
> Review request for mesos, Alexander Rukletsov and Benjamin Mahler.
> 
> 
> Bugs: MESOS-9143
>     https://issues.apache.org/jira/browse/MESOS-9143
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> The test is flaky due to a race between metrics update
> and metrics query.
> 
> This patch adds clock settle to ensure quota update and
> removal are fully processed (including metrics updates) before
> continuing with the metrics query.
> 
> 
> Diffs
> -----
> 
>   src/tests/master_quota_tests.cpp 354a9e988c3e4cdf94c99170d2abb7cb17f9e11a 
> 
> 
> Diff: https://reviews.apache.org/r/69981/diff/2/
> 
> 
> Testing
> -------
> 
> Test ran continuously without failure
> 
> 
> Thanks,
> 
> Meng Zhu
> 
>


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