> On July 6, 2016, 11:21 a.m., Adam B wrote: > > src/sched/sched.cpp, line 483 > > <https://reviews.apache.org/r/49308/diff/3/?file=1437634#file1437634line483> > > > > Seems `failedAuthentications` is never 0 here (since you increment it > > just before), so you'll never delay by the `[0, b * 2^0]` amount you > > suggest in the docs. Should we make this `std::pow(2, > > failedAuthentications-1)` or update the doc? > > Same issue on the agent.
I just updated the doc. We also discussed adding an initial delay before authenticating, to prevent thundering herds. I added TODOs to that effect. - Adam ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49308/#review141041 ----------------------------------------------------------- On July 6, 2016, 6:58 a.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49308/ > ----------------------------------------------------------- > > (Updated July 6, 2016, 6:58 a.m.) > > > Review request for mesos, Adam B and Vinod Kone. > > > Bugs: MESOS-2043 > https://issues.apache.org/jira/browse/MESOS-2043 > > > Repository: mesos > > > Description > ------- > > The backoff follows to existing pattern for backoff used during agent > or scheduler registration where we backoff for some random time in an > interval of increasing length, capped by > `REGISTER_RETRY_INTERVAL_MAX`. > > > Diffs > ----- > > docs/configuration.md 8c8678c7e2251923298b90b7216a4e584faf6b26 > docs/endpoints/slave/state.json.md 0f82c1926404e79b281b2ea5f4d0ca21323aeded > docs/endpoints/slave/state.md b34459e8624f0b29e927ff79be7fc845ac88080b > src/sched/constants.hpp df8a1cc83ee3986400d633b2192b6da7fbe6b626 > src/sched/flags.hpp 989cebe40c6b4ecc7c4d47f8cf9d968cc795ad3f > src/sched/sched.cpp 9f561d73a2e591afdc3ba4adb35a11763dced402 > src/slave/constants.hpp 668fc47e72d6f1b904aef5d0750b990fe162c9a3 > src/slave/flags.hpp ff45876a44ed00fdea36986f052f10e8b8031925 > src/slave/flags.cpp 010e78347f72edd5e60628b8bdda8de8b5feed21 > src/slave/slave.hpp 484ba758b4c87935aabd2f76a0e654a3c6d09167 > src/slave/slave.cpp 36f63bc54bec88f7e7b11ed0cde8bc78314908b2 > > Diff: https://reviews.apache.org/r/49308/diff/ > > > Testing > ------- > > make check (OS X w/o optimizations). > > Ran agent-related `AuthenticationTest`s in repetition (300 times). > > > Thanks, > > Benjamin Bannier > >
