You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Sailaja Polavarapu <sp...@hortonworks.com> on 2015/10/09 20:52:46 UTC

Review Request 39185: RANGER-652: Support for Ldap connection check tool.

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

Review request for ranger, Alok Lal, Don Bosco Durai, Abhay Kulkarni, Madhan Neethiraj, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan Periasamy.


Bugs: RANGER-652
    https://issues.apache.org/jira/browse/RANGER-652


Repository: ranger


Description
-------

Patch contains the changes for ldap connection check tool. This tool is packaged as part of ranger usersyn module and can be used for discovering and/or verifying the usersync ldap/AD related properties as well as ldap/AD authentication related properties before configuring ranger.


Diffs
-----

  src/main/assembly/usersync.xml 8c2600e 
  ugsync/ldapconfigchecktool/ldapconfigcheck/conf/input.properties PRE-CREATION 
  ugsync/ldapconfigchecktool/ldapconfigcheck/pom.xml PRE-CREATION 
  ugsync/ldapconfigchecktool/ldapconfigcheck/scripts/run.sh PRE-CREATION 
  ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/AuthenticationCheck.java PRE-CREATION 
  ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/CommandLineOptions.java PRE-CREATION 
  ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/LdapConfig.java PRE-CREATION 
  ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/LdapConfigCheckMain.java PRE-CREATION 
  ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/UserSync.java PRE-CREATION 

Diff: https://reviews.apache.org/r/39185/diff/


Testing
-------

Tests performed:
1. Verified the packaging and running the tool with various supported command line options.
2. Tested the tool against openLdap server as well as Active directory setup.
3. Used the discovered properties from the tool and configured 2.3 ranger and validated the results.
4. Tested few negative cases on in valid credentials for authentication, wrong domain while discovering usersync properties, and providing invalid bind password as part of the user input.


Thanks,

Sailaja Polavarapu


Re: Review Request 39185: RANGER-652: Support for Ldap connection check tool.

Posted by Velmurugan Periasamy <vp...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39185/#review102172
-----------------------------------------------------------

Ship it!


Ship It!

- Velmurugan Periasamy


On Oct. 9, 2015, 6:52 p.m., Sailaja Polavarapu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39185/
> -----------------------------------------------------------
> 
> (Updated Oct. 9, 2015, 6:52 p.m.)
> 
> 
> Review request for ranger, Alok Lal, Don Bosco Durai, Abhay Kulkarni, Madhan Neethiraj, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-652
>     https://issues.apache.org/jira/browse/RANGER-652
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Patch contains the changes for ldap connection check tool. This tool is packaged as part of ranger usersyn module and can be used for discovering and/or verifying the usersync ldap/AD related properties as well as ldap/AD authentication related properties before configuring ranger.
> 
> 
> Diffs
> -----
> 
>   src/main/assembly/usersync.xml 8c2600e 
>   ugsync/ldapconfigchecktool/ldapconfigcheck/conf/input.properties PRE-CREATION 
>   ugsync/ldapconfigchecktool/ldapconfigcheck/pom.xml PRE-CREATION 
>   ugsync/ldapconfigchecktool/ldapconfigcheck/scripts/run.sh PRE-CREATION 
>   ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/AuthenticationCheck.java PRE-CREATION 
>   ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/CommandLineOptions.java PRE-CREATION 
>   ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/LdapConfig.java PRE-CREATION 
>   ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/LdapConfigCheckMain.java PRE-CREATION 
>   ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/UserSync.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/39185/diff/
> 
> 
> Testing
> -------
> 
> Tests performed:
> 1. Verified the packaging and running the tool with various supported command line options.
> 2. Tested the tool against openLdap server as well as Active directory setup.
> 3. Used the discovered properties from the tool and configured 2.3 ranger and validated the results.
> 4. Tested few negative cases on in valid credentials for authentication, wrong domain while discovering usersync properties, and providing invalid bind password as part of the user input.
> 
> 
> Thanks,
> 
> Sailaja Polavarapu
> 
>