----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65995/#review204917 -----------------------------------------------------------
Fix it, then Ship it! src/tests/storage_local_resource_provider_tests.cpp Lines 2561-2562 (patched) <https://reviews.apache.org/r/65995/#comment287714> If there is a reason to `settle` before we `advance`, we should add a comment, otherwise I would expect a sequence of first `advance`, then `settle`. We don't seem to be very disciplined to _always_ `settle` after and `advance` in this file, we could clean that up in a follow-up. src/tests/storage_local_resource_provider_tests.cpp Lines 2718-2719 (patched) <https://reviews.apache.org/r/65995/#comment287715> Ditto. - Benjamin Bannier On June 14, 2018, 2:02 a.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65995/ > ----------------------------------------------------------- > > (Updated June 14, 2018, 2:02 a.m.) > > > Review request for mesos, Benjamin Bannier, Gastón Kleiman, Greg Mann, and > Jie Yu. > > > Bugs: MESOS-8825 > https://issues.apache.org/jira/browse/MESOS-8825 > > > Repository: mesos > > > Description > ------- > > The two SLRP tests assume that SLRP will send out a RAW resource in its > first `UPDATE_STATE` message, and expect that the test framework would > receive an offer containing the RAW resource in its first offer. However > this behavior is not guaranteed and should not be relied on. This patch > makes the tests decline unwanted offers by default so they no longer > rely on SLRP's internal behavior. > > > Diffs > ----- > > src/tests/storage_local_resource_provider_tests.cpp > 1afe8a8e0413ef225e952cd9cd6376e5d82774e5 > > > Diff: https://reviews.apache.org/r/65995/diff/7/ > > > Testing > ------- > > sudo make check > Ran the two tests in repitition. > > > Thanks, > > Chun-Hung Hsiao > >