You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Madhan Neethiraj <ma...@apache.org> on 2022/03/09 17:31:54 UTC

Re: Review Request 73894: RANGER-3658: updated to run Ranger containers (admin, usersync, tagsync) as user=ranger, instead of user=root

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

(Updated March 9, 2022, 5:31 p.m.)


Review request for ranger, Ankita Sinha, Kishor Gollapalliwar, Abhay Kulkarni, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, Sailaja Polavarapu, and Velmurugan Periasamy.


Summary (updated)
-----------------

RANGER-3658: updated to run Ranger containers (admin, usersync, tagsync) as user=ranger, instead of user=root


Bugs: RANGER-3658
    https://issues.apache.org/jira/browse/RANGER-3658


Repository: ranger


Description (updated)
-------

- added USER directive in dockerfiles for admin, usersync, tagsync
- setup scripts for usersync, tagsync perform some actions (like create symlinks) that require root privileges. These tasks are now performed in dockerfile, so that setup scripts will skip these steps
- updated setup scripts to not bail out in case of errors in updating permissions/ownership (which happen when these scripts are run as user=ranger); instead proceed to rest of setup after printing appropriate warning


Diffs
-----

  dev-support/ranger-docker/Dockerfile.ranger 16492d45c 
  dev-support/ranger-docker/Dockerfile.ranger-tagsync 31f8446fd 
  dev-support/ranger-docker/Dockerfile.ranger-usersync b1738f8c2 
  dev-support/ranger-docker/scripts/ranger-tagsync.sh f7613946d 
  dev-support/ranger-docker/scripts/ranger-usersync.sh 48287d276 
  dev-support/ranger-docker/scripts/ranger.sh 46030b19b 
  tagsync/scripts/setup.py 1b88ae2d6 
  unixauthservice/scripts/setup.py 13d64412a 


Diff: https://reviews.apache.org/r/73894/diff/1/


Testing
-------

- verified that Ranger admin, usersync and tagsync containers startup and initialize successfully


Thanks,

Madhan Neethiraj

Re: Review Request 73894: RANGER-3658: updated to run Ranger containers (admin, usersync, tagsync) as user=ranger, instead of user=root

Posted by Ramesh Mani <rm...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73894/#review224148
-----------------------------------------------------------


Ship it!




Ship It!

- Ramesh Mani


On March 9, 2022, 5:31 p.m., Madhan Neethiraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73894/
> -----------------------------------------------------------
> 
> (Updated March 9, 2022, 5:31 p.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Kishor Gollapalliwar, Abhay Kulkarni, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, Sailaja Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3658
>     https://issues.apache.org/jira/browse/RANGER-3658
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> - added USER directive in dockerfiles for admin, usersync, tagsync
> - setup scripts for usersync, tagsync perform some actions (like create symlinks) that require root privileges. These tasks are now performed in dockerfile, so that setup scripts will skip these steps
> - updated setup scripts to not bail out in case of errors in updating permissions/ownership (which happen when these scripts are run as user=ranger); instead proceed to rest of setup after printing appropriate warning
> 
> 
> Diffs
> -----
> 
>   dev-support/ranger-docker/Dockerfile.ranger 16492d45c 
>   dev-support/ranger-docker/Dockerfile.ranger-tagsync 31f8446fd 
>   dev-support/ranger-docker/Dockerfile.ranger-usersync b1738f8c2 
>   dev-support/ranger-docker/scripts/ranger-tagsync.sh f7613946d 
>   dev-support/ranger-docker/scripts/ranger-usersync.sh 48287d276 
>   dev-support/ranger-docker/scripts/ranger.sh 46030b19b 
>   tagsync/scripts/setup.py 1b88ae2d6 
>   unixauthservice/scripts/setup.py 13d64412a 
> 
> 
> Diff: https://reviews.apache.org/r/73894/diff/1/
> 
> 
> Testing
> -------
> 
> - verified that Ranger admin, usersync and tagsync containers startup and initialize successfully
> 
> 
> Thanks,
> 
> Madhan Neethiraj
> 
>