ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Madhavi Amirneni <mami...@us.ibm.com>
Subject Re: Review Request 58921: Ranger-1552: Ranger usersync or ranger kms not able to communicate to ranger admin and no exception or error seen in the ranger user sync or kms logs.
Date Thu, 04 May 2017 20:38:39 GMT


> On May 4, 2017, 4:45 p.m., Colm O hEigeartaigh wrote:
> > Looks fine but please remove the trailing whitespace after the if statement.

Thank you for reviewing the fix. Updated the diff and addressed the comments. Please review.


- Madhavi


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


On May 4, 2017, 6 p.m., Madhavi Amirneni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58921/
> -----------------------------------------------------------
> 
> (Updated May 4, 2017, 6 p.m.)
> 
> 
> Review request for ranger, Gautam Borad, Madhan Neethiraj, and Sailaja Polavarapu.
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> The default communication between Ranger admin and Ranger usersync or Ranger kms is SSL
enabled. When there is an error during the hadoop keystore creation, there is no exception
thrown but there was a non-zeor return code returned. However, ranger did not check/handle
this error return code and continued, which has caused Ranger usersync to assume the communication
to be normal and continue.  
>  
> With this fix, when there is an error during the keystore creation, an error message
is logged and the process will exit.
> 
> 
> Diffs
> -----
> 
>   credentialbuilder/src/main/java/org/apache/ranger/credentialapi/buildks.java a7e433f

> 
> 
> Diff: https://reviews.apache.org/r/58921/diff/2/
> 
> 
> Testing
> -------
> 
> Manual unit testing & Automated tests.
> 
> 
> Thanks,
> 
> Madhavi Amirneni
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message