-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51953/
-----------------------------------------------------------
Review request for mesos and Vinod Kone.
Bugs: MESOS-5951
https://issues.apache.org/jira/browse/MESOS-5951
Repository: mesos
Description
-------
Registrar operations to admit and remove slaves now behave the same way,
regardless of the value of the "strict" flag.
This commit changes the behavior of these operations by enabling the
previous "strict" behavior. For example, removing a slave that is not
registered results in an `Error`, rather than silently ignoring the
problem. However, the master should never attempt to remove a slave that
is not registered, so this is a useful invariant to check. Similarly,
attempting to admit a slave that is already admitted will now result in
an `Error`; this should only occur when there is a slave ID collision,
so again it seems appropriate to produce an error rather than ignoring
the problem.
Diffs
-----
src/master/master.hpp 4992ab0a0bb5babbf6a4fa3e6eff3577590fc879
src/master/master.cpp b88472f6350d3f71e057bab34822423da5427151
src/tests/registrar_tests.cpp b04fc92c66449f1094212a534899e16d5f6f01bd
Diff: https://reviews.apache.org/r/51953/diff/
Testing
-------
`make check` on OSX and Linux.
Thanks,
Neil Conway
