You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Mehul Parikh <me...@freestoneinfotech.com> on 2016/03/18 13:04:49 UTC
Review Request 45027: RANGER-887 : Changes needed to convert
log4j.xml to log4j.properties
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45027/
-----------------------------------------------------------
Review request for ranger, Alok Lal, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, Madhan Neethiraj, Ramesh Mani, and Selvamohan Neethiraj.
Bugs: RANGER-887
https://issues.apache.org/jira/browse/RANGER-887
Repository: ranger
Description
-------
**Problem Statement**
In order to allow log4j to be exposed to Ambari and configurable from Ambari, we need to convert log4j.xml file to log4j.properties file.
**Proposed Solution**
Making required changes in pom.xml (usersync) and web.xml(ranger-admin) to use log4j.properties and converting log4j.xml to log4j.properties.
Diffs
-----
security-admin/src/main/webapp/WEB-INF/log4j.properties PRE-CREATION
security-admin/src/main/webapp/WEB-INF/web.xml 0725633
tagsync/conf.dist/log4j.properties PRE-CREATION
unixauthservice/conf.dist/log4j.properties PRE-CREATION
unixauthservice/pom.xml edff076
unixauthservice/scripts/ranger-usersync-services.sh 4b3d4d4
Diff: https://reviews.apache.org/r/45027/diff/
Testing
-------
** Testing Done**
Verified manual install as well as ambari based installs of Ranger to use log4j.properties to log required activities.
Thanks,
Mehul Parikh
Re: Review Request 45027: RANGER-887 : Changes needed to convert
log4j.xml to log4j.properties
Posted by Velmurugan Periasamy <vp...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45027/#review129050
-----------------------------------------------------------
Ship it!
Ship It!
- Velmurugan Periasamy
On March 31, 2016, 7 a.m., Mehul Parikh wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45027/
> -----------------------------------------------------------
>
> (Updated March 31, 2016, 7 a.m.)
>
>
> Review request for ranger, Alok Lal, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, Madhan Neethiraj, Ramesh Mani, and Selvamohan Neethiraj.
>
>
> Bugs: RANGER-887
> https://issues.apache.org/jira/browse/RANGER-887
>
>
> Repository: ranger
>
>
> Description
> -------
>
> **Problem Statement**
> In order to allow log4j to be exposed to Ambari and configurable from Ambari, we need to convert log4j.xml file to log4j.properties file.
>
> **Proposed Solution**
> Making required changes in pom.xml (usersync) and web.xml(ranger-admin) to use log4j.properties and converting log4j.xml to log4j.properties.
>
>
> Diffs
> -----
>
> security-admin/src/main/webapp/WEB-INF/log4j.properties PRE-CREATION
> security-admin/src/main/webapp/WEB-INF/web.xml 0725633
> tagsync/conf.dist/log4j.properties PRE-CREATION
> tagsync/scripts/ranger-tagsync-services.sh 058ebf3
> tagsync/scripts/setup.py 6d28cd2
> unixauthservice/conf.dist/log4j.properties PRE-CREATION
> unixauthservice/pom.xml edff076
> unixauthservice/scripts/ranger-usersync-services.sh 622c986
>
> Diff: https://reviews.apache.org/r/45027/diff/
>
>
> Testing
> -------
>
> ** Testing Done**
>
> Verified manual install as well as ambari based installs of Ranger to use log4j.properties to log required activities.
>
>
> Thanks,
>
> Mehul Parikh
>
>
Re: Review Request 45027: RANGER-887 : Changes needed to convert
log4j.xml to log4j.properties
Posted by Mehul Parikh <me...@freestoneinfotech.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45027/
-----------------------------------------------------------
(Updated March 31, 2016, 7 a.m.)
Review request for ranger, Alok Lal, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, Madhan Neethiraj, Ramesh Mani, and Selvamohan Neethiraj.
Changes
-------
Added suggestion from Abhay to - change configs of TagSync matching to Ranger Usersync.
Bugs: RANGER-887
https://issues.apache.org/jira/browse/RANGER-887
Repository: ranger
Description
-------
**Problem Statement**
In order to allow log4j to be exposed to Ambari and configurable from Ambari, we need to convert log4j.xml file to log4j.properties file.
**Proposed Solution**
Making required changes in pom.xml (usersync) and web.xml(ranger-admin) to use log4j.properties and converting log4j.xml to log4j.properties.
Diffs (updated)
-----
security-admin/src/main/webapp/WEB-INF/log4j.properties PRE-CREATION
security-admin/src/main/webapp/WEB-INF/web.xml 0725633
tagsync/conf.dist/log4j.properties PRE-CREATION
tagsync/scripts/ranger-tagsync-services.sh 058ebf3
tagsync/scripts/setup.py 6d28cd2
unixauthservice/conf.dist/log4j.properties PRE-CREATION
unixauthservice/pom.xml edff076
unixauthservice/scripts/ranger-usersync-services.sh 622c986
Diff: https://reviews.apache.org/r/45027/diff/
Testing
-------
** Testing Done**
Verified manual install as well as ambari based installs of Ranger to use log4j.properties to log required activities.
Thanks,
Mehul Parikh