You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by pengjianhua <pe...@zte.com.cn> on 2017/07/12 02:21:13 UTC

Review Request 60789: Ranger usersync created error users. Ranger usersync got System Virtual User Group Name instead of real group name when built the unix user list.

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

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. So 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/debian-bugs-closed@lists.debian.org/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


Re: Review Request 60789: Ranger usersync created error users. Ranger usersync got System Virtual User Group Name instead of real group name when built the unix user list.

Posted by pengjianhua <pe...@zte.com.cn>.
-----------------------------------------------------------
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/debian-bugs-closed@lists.debian.org/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
> 
>


Re: Review Request 60789: Ranger usersync created error users. Ranger usersync got System Virtual User Group Name instead of real group name when built the unix user list.

Posted by pengjianhua <pe...@zte.com.cn>.
-----------------------------------------------------------
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 (updated)
-------

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/debian-bugs-closed@lists.debian.org/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