----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53176/#review153869 -----------------------------------------------------------
Ship it! Ship It! - Velmurugan Periasamy On Oct. 25, 2016, 11:21 p.m., Sailaja Polavarapu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53176/ > ----------------------------------------------------------- > > (Updated Oct. 25, 2016, 11:21 p.m.) > > > Review request for ranger, Ankita Sinha, Gautam Borad, Abhay Kulkarni, Madhan > Neethiraj, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan Periasamy. > > > Bugs: RANGER-1191 > https://issues.apache.org/jira/browse/RANGER-1191 > > > Repository: ranger > > > Description > ------- > > Added code to save the state if ranger admin update fail or success. During > every sync interval for Unix or file based sync, in order to perform sync > process, added extra check to perform sync process if the previous update to > ranger admin has failed. > > > Diffs > ----- > > > ugsync/src/main/java/org/apache/ranger/unixusersync/process/FileSourceUserGroupBuilder.java > 298941a > > ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java > 296d94f > > ugsync/src/main/java/org/apache/ranger/unixusersync/process/UnixUserGroupBuilder.java > 88aa266 > ugsync/src/main/java/org/apache/ranger/usergroupsync/UserGroupSink.java > 6cce36f > > Diff: https://reviews.apache.org/r/53176/diff/ > > > Testing > ------- > > 1. Performed functional tests with unix and file based sync by stopping > Ranger admin and adding new users to the sync source. > 2. Performed regression tests for regular use cases. > 3. Performed functional test with ranger admin down during initial usersync > startup process. > > > Thanks, > > Sailaja Polavarapu > >
