Re: Review Request 71718: RANGER-2635 : Backport hadoop-kms changes in ranger-kms

2019-11-07 Thread Mehul Parikh

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


Ship it!




Ship It!

- Mehul Parikh


On Nov. 5, 2019, 2:33 p.m., Fatima Khan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/71718/
> ---
> 
> (Updated Nov. 5, 2019, 2:33 p.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Gautam Borad, Madhan Neethiraj, 
> Mehul Parikh, Nikhil P, Pradeep Agrawal, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-2635
> https://issues.apache.org/jira/browse/RANGER-2635
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> Backported hadoop-kms changes in ranger-kms to support hadoop 3.1
> 
> 
> Diffs
> -
> 
>   kms/dev-support/findbugsExcludeFile.xml bc92ed7 
>   kms/pom.xml 12c0002 
>   kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSACLs.java 
> ff2f6d9 
>   kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSAudit.java 
> 053d7e4 
>   
> kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSAuditLogger.java 
> PRE-CREATION 
>   
> kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSAuthenticationFilter.java
>  c818ce5 
>   
> kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSConfiguration.java
>  4bf2886 
>   kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSWebApp.java 
> ae24e5b 
>   
> kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/SimpleKMSAuditLogger.java
>  PRE-CREATION 
>   kms/src/main/resources/log4j-kmsaudit.properties cca6941 
>   kms/src/test/java/org/apache/hadoop/crypto/key/kms/server/TestKMSACLs.java 
> 3ff48d0 
>   kms/src/test/java/org/apache/hadoop/crypto/key/kms/server/TestKMSAudit.java 
> ced8610 
>   kms/src/test/resources/kms/kms-site.xml 5f2575a 
>   
> plugin-kms/src/test/java/org/apache/ranger/authorization/kms/authorizer/RangerKmsAuthorizerTest.java
>  647f431 
>   pom.xml 2140028 
> 
> 
> Diff: https://reviews.apache.org/r/71718/diff/1/
> 
> 
> Testing
> ---
> 
> Tested all the operations related to ranger-kms
> 
> 
> Thanks,
> 
> Fatima Khan
> 
>



Re: Review Request 71718: RANGER-2635 : Backport hadoop-kms changes in ranger-kms

2019-11-05 Thread Fatima Khan

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

(Updated Nov. 5, 2019, 2:33 p.m.)


Review request for ranger, Ankita Sinha, Gautam Borad, Madhan Neethiraj, Mehul 
Parikh, Nikhil P, Pradeep Agrawal, and Velmurugan Periasamy.


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


Repository: ranger


Description
---

Backported hadoop-kms changes in ranger-kms to support hadoop 3.1


Diffs
-

  kms/dev-support/findbugsExcludeFile.xml bc92ed7 
  kms/pom.xml 12c0002 
  kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSACLs.java 
ff2f6d9 
  kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSAudit.java 
053d7e4 
  kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSAuditLogger.java 
PRE-CREATION 
  
kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSAuthenticationFilter.java
 c818ce5 
  
kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSConfiguration.java 
4bf2886 
  kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSWebApp.java 
ae24e5b 
  
kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/SimpleKMSAuditLogger.java
 PRE-CREATION 
  kms/src/main/resources/log4j-kmsaudit.properties cca6941 
  kms/src/test/java/org/apache/hadoop/crypto/key/kms/server/TestKMSACLs.java 
3ff48d0 
  kms/src/test/java/org/apache/hadoop/crypto/key/kms/server/TestKMSAudit.java 
ced8610 
  kms/src/test/resources/kms/kms-site.xml 5f2575a 
  
plugin-kms/src/test/java/org/apache/ranger/authorization/kms/authorizer/RangerKmsAuthorizerTest.java
 647f431 
  pom.xml 2140028 


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


Testing
---

Tested all the operations related to ranger-kms


Thanks,

Fatima Khan