----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40837/#review108572 -----------------------------------------------------------
Ship it! Ship It! - Velmurugan Periasamy On Dec. 1, 2015, 9:57 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, 9:57 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 > > 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 > >
