-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22222/
-----------------------------------------------------------
(Updated June 20, 2014, 6:08 p.m.)
Review request for mesos, Benjamin Hindman, Dominic Hamon, and Till Toenshoff.
Changes
-------
Changes after Ben's comments
Bugs: MESOS-1391
https://issues.apache.org/jira/browse/MESOS-1391
Repository: mesos-git
Description
-------
After Ben's comments, regroup of all types of authentication in one common
definition, this is a draft for this Issue, here are some questions:
- Do I replace credentials flag completely or if this slight duplication will
suffice for now ?
- If I don't replace it, for now as it is in this patch, the flag is useless, I
should give it to the sasl/authenticator but I would really appreciate some
comments for that part.
Diffs (updated)
-----
include/mesos/mesos.proto 2f6be05
src/Makefile.am b1b7d2d
src/credentials/credentials.hpp 98b9088
src/master/flags.hpp 47bb0dc
src/master/master.hpp b56e9f4
src/master/master.cpp dcf28ad
src/sasl/authenticator.hpp 365db5f
src/slave/flags.hpp 3b8ba08
src/slave/slave.cpp ed3483f
src/tests/authentication_tests.cpp 5cf2da4
src/tests/credentials_tests.cpp PRE-CREATION
src/tests/mesos.cpp 1037420
Diff: https://reviews.apache.org/r/22222/diff/
Testing
-------
Thanks,
Isabel Jimenez
