You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Ramesh Mani <rm...@hortonworks.com> on 2016/07/08 22:54:06 UTC

Review Request 49836: RANGER-1092 : Ranger YARN Plugin should not fails to download policy when UGI ticket exprires

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

Review request for ranger, Madhan Neethiraj and Velmurugan Periasamy.


Repository: ranger


Description
-------

RANGER-1092 : Ranger YARN Plugin should not fails to download policy when UGI ticket exprires


Diffs
-----

  agents-audit/src/main/java/org/apache/ranger/audit/provider/MiscUtil.java 7c48055 

Diff: https://reviews.apache.org/r/49836/diff/


Testing
-------

Tested by leaving the cluster for more than a day till UGI tgt expires and new get new ticket. Verified that YARN plugin successfully downloads the policy without error using the renewed ticket


Thanks,

Ramesh Mani


Re: Review Request 49836: RANGER-1092 : Ranger YARN Plugin should not fails to download policy when UGI ticket exprires

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




agents-audit/src/main/java/org/apache/ranger/audit/provider/MiscUtil.java (line 784)
<https://reviews.apache.org/r/49836/#comment206889>

    As per the documentation and looking into the code it relogin if TGT is expired or about to expiry. 
    http://grepcode.com/file/repo1.maven.org/maven2/org.apache.hadoop/hadoop-common/2.6.0/org/apache/hadoop/security/UserGroupInformation.java


- Ramesh Mani


On July 8, 2016, 10:54 p.m., Ramesh Mani wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49836/
> -----------------------------------------------------------
> 
> (Updated July 8, 2016, 10:54 p.m.)
> 
> 
> Review request for ranger, Madhan Neethiraj and Velmurugan Periasamy.
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> RANGER-1092 : Ranger YARN Plugin should not fails to download policy when UGI ticket exprires
> 
> 
> Diffs
> -----
> 
>   agents-audit/src/main/java/org/apache/ranger/audit/provider/MiscUtil.java 7c48055 
> 
> Diff: https://reviews.apache.org/r/49836/diff/
> 
> 
> Testing
> -------
> 
> Tested by leaving the cluster for more than a day till UGI tgt expires and new get new ticket. Verified that YARN plugin successfully downloads the policy without error using the renewed ticket
> 
> 
> Thanks,
> 
> Ramesh Mani
> 
>


Re: Review Request 49836: RANGER-1092 : Ranger YARN Plugin should not fails to download policy when UGI ticket exprires

Posted by Velmurugan Periasamy <vp...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49836/#review141365
-----------------------------------------------------------


Ship it!





agents-audit/src/main/java/org/apache/ranger/audit/provider/MiscUtil.java (line 784)
<https://reviews.apache.org/r/49836/#comment206888>

    Can you please confirm if this will not force unnecessary logins often?


- Velmurugan Periasamy


On July 8, 2016, 10:54 p.m., Ramesh Mani wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49836/
> -----------------------------------------------------------
> 
> (Updated July 8, 2016, 10:54 p.m.)
> 
> 
> Review request for ranger, Madhan Neethiraj and Velmurugan Periasamy.
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> RANGER-1092 : Ranger YARN Plugin should not fails to download policy when UGI ticket exprires
> 
> 
> Diffs
> -----
> 
>   agents-audit/src/main/java/org/apache/ranger/audit/provider/MiscUtil.java 7c48055 
> 
> Diff: https://reviews.apache.org/r/49836/diff/
> 
> 
> Testing
> -------
> 
> Tested by leaving the cluster for more than a day till UGI tgt expires and new get new ticket. Verified that YARN plugin successfully downloads the policy without error using the renewed ticket
> 
> 
> Thanks,
> 
> Ramesh Mani
> 
>