-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review60562
-----------------------------------------------------------


Quick fly-by review


src/authentication/authenticator.hpp
<https://reviews.apache.org/r/27760/#comment101927>

    So, if I want to update credentials, I have to kill my Master (hence 
Authenticator) and create+initialize a new one? What would it take to 
live-update credentials?



src/authentication/cram_md5/authenticator.hpp
<https://reviews.apache.org/r/27760/#comment101928>

    s/CRAMMD5AuthenticatorProcess/CRAMMD5AuthenticatorSessionProcess/ ?



src/master/master.cpp
<https://reviews.apache.org/r/27760/#comment101929>

    When would this ever happen? If it's not possible, a CHECK might be more 
appropriate; whereas if it is possible, is an EXIT too strong?
    
    And if you're keeping the error message, I'd suggest the simpler 
"Authenticator 'foo' not initialized"



src/master/master.cpp
<https://reviews.apache.org/r/27760/#comment101930>

    s/for ever/forever/


- Adam B


On Nov. 9, 2014, 1:47 p.m., Till Toenshoff wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27760/
> -----------------------------------------------------------
> 
> (Updated Nov. 9, 2014, 1:47 p.m.)
> 
> 
> Review request for mesos, Adam B, Kapil Arya, Niklas Nielsen, and Vinod Kone.
> 
> 
> Bugs: MESOS-2050
>     https://issues.apache.org/jira/browse/MESOS-2050
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> The initial design and implementation of the authenticator module interface 
> caused issues and was not optimal for heavy lifting setup of external 
> dependencies. By introducing a two fold design, this has been decoupled from 
> the authentication message processing. The new design also gets us back on 
> track to the goal of makeing SASL a soft dependency of mesos.
> 
> 
> Diffs
> -----
> 
>   src/authentication/authenticator.hpp 460494a 
>   src/authentication/cram_md5/authenticator.hpp a23bf5d 
>   src/master/master.hpp 687f178 
>   src/master/master.cpp 5712b17 
>   src/tests/cram_md5_authentication_tests.cpp 1ed2312 
> 
> Diff: https://reviews.apache.org/r/27760/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Till Toenshoff
> 
>

Reply via email to