----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60789/#review180288 -----------------------------------------------------------
Ship it! Ship It! - pengjianhua On 七月 12, 2017, 2:42 a.m., pengjianhua wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60789/ > ----------------------------------------------------------- > > (Updated 七月 12, 2017, 2:42 a.m.) > > > Review request for ranger, Alok Lal, Ankita Sinha, Don Bosco Durai, Colm O > hEigeartaigh, Gautam Borad, Madhan Neethiraj, Ramesh Mani, Selvamohan > Neethiraj, Velmurugan Periasamy, and Qiang Zhang. > > > Bugs: RANGER-1690 > https://issues.apache.org/jira/browse/RANGER-1690 > > > Repository: ranger > > > Description > ------- > > The ranger.usersync.unix.minGroupId is not set default value. Like > ranger.usersync.unix.minUserId, it should be set default value. Following > variable equals to 0 when built the unix user list. > minimumGroupId=Integer.parseInt(config.getMinGroupId()); > Ranger usersync may get System Virtual User Group Name instead of real group > name. > if (numGroupId < minimumGroupId) > continue; > String groupName = groupId2groupNameMap.get(gid); > This result is that the Ranger usersync created error users. > http://www.archivum.info/[email protected]/2006-03/03888/Bug-333706-marked-as-done-(useradd-lacks-r-option).html > > > Diffs > ----- > > unixauthservice/conf.dist/ranger-ugsync-default.xml 5ed5b89 > unixauthservice/scripts/install.properties 13ae1e5 > unixauthservice/scripts/templates/installprop2xml.properties 1a9bf36 > unixauthservice/scripts/templates/ranger-ugsync-template.xml 0025dc8 > > > Diff: https://reviews.apache.org/r/60789/diff/1/ > > > Testing > ------- > > > Thanks, > > pengjianhua > >
