----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52818/ -----------------------------------------------------------
(Updated Oct. 21, 2016, 11:01 a.m.) Review request for mesos and Anand Mazumdar. Changes ------- Addressed Anand's review comments. Summary (updated) ----------------- Ensured agent is recovered in SlaveEndpointTests. Repository: mesos Description (updated) ------- The tests in this case did either assume that e.g., requests for agent endpoints could always reliably be satisfied by a not yet fully recovered agent, or did not explicitly observe the recovery and instead relied on just settling the clock. Make sure agents are actually fully recovered, even if not strictly required right now. Also actually observe agent recovery calls like done elsewhere. Diffs (updated) ----- src/tests/slave_authorization_tests.cpp 6bd2aa96cb05aa087c0807cc3dd1830451efe106 Diff: https://reviews.apache.org/r/52818/diff/ Testing ------- `make check` (OS X, clang-3.9 w/o optimizations) Thanks, Benjamin Bannier