ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sailaja Polavarapu" <spolavar...@hortonworks.com>
Subject Re: Review Request 40837: Ranger-755, Ranger-756, Ranger-757: Ldap tool fails with different command line options.
Date Tue, 01 Dec 2015 21:47:14 GMT


> On Dec. 1, 2015, 9:25 p.m., Madhan Neethiraj wrote:
> > ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/CommandLineOptions.java,
line 75
> > <https://reviews.apache.org/r/40837/diff/1/?file=1149797#file1149797line75>
> >
> >     This change would result in "r" option (retrieve) be ignored when "d" option
(discoverProperties) is specified. Is this intentional?

That is true. We can eiter discover the properties or retrieve the properties but not both
at the same time.


- Sailaja


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


On Dec. 1, 2015, 8:59 p.m., Sailaja Polavarapu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40837/
> -----------------------------------------------------------
> 
> (Updated Dec. 1, 2015, 8:59 p.m.)
> 
> 
> Review request for ranger, Alok Lal, Abhay Kulkarni, Madhan Neethiraj, Ramesh Mani, and
Velmurugan Periasamy.
> 
> 
> Bugs: Ranger-755, Ranger-756 and Ranger-757
>     https://issues.apache.org/jira/browse/Ranger-755
>     https://issues.apache.org/jira/browse/Ranger-756
>     https://issues.apache.org/jira/browse/Ranger-757
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> "getopts" function in shell script reads the command line options one character at a
time. Hence -noauth is treated as -n, -o, -a, etc... which is the root cause for Ranger-755.
> For Ranger-756 & Ranger-757: According to the original design of ldap tool, verification
of authentication is intended to be performed only while discovering all the usersync properties.
Some of the conditions to disable auth verification were not handled properly.
> 
> 
> Diffs
> -----
> 
>   ugsync/ldapconfigchecktool/ldapconfigcheck/scripts/run.sh 787e216 
>   ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/CommandLineOptions.java
790330f 
> 
> Diff: https://reviews.apache.org/r/40837/diff/
> 
> 
> Testing
> -------
> 
> 1. Tested various combinations on the command line arguments and verified the desired
results.
> 2. Also tested with both dev & QE setup to run different tests for regressions.
> 
> 
> Thanks,
> 
> Sailaja Polavarapu
> 
>


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