You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by bhavik patel <bh...@gmail.com> on 2018/08/01 05:36:30 UTC

Re: Review Request 68083: RANGER-2167 - Upgrade to Apache parent pom version 20

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



@Colm Can you please verify some basic functionality of ranger kms?

- bhavik patel


On July 27, 2018, 3:28 p.m., Colm O hEigeartaigh wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68083/
> -----------------------------------------------------------
> 
> (Updated July 27, 2018, 3:28 p.m.)
> 
> 
> Review request for ranger.
> 
> 
> Bugs: RANGER-2167
>     https://issues.apache.org/jira/browse/RANGER-2167
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> This task is to upgrade to the Apache parent pom version 20. This necessitates a PMD upgrade and quite a few fixes are required as a result. One notable change is to switch the KMS Keystore implementation to use ConcurrentHashMap instead of HashTable.
> 
> 
> Diffs
> -----
> 
>   agents-common/src/main/java/org/apache/ranger/plugin/model/RangerValiditySchedule.java 55f787d23 
>   agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerValidator.java ed5aa8d41 
>   agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerValidityScheduleValidator.java f34d124d6 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerResourceACLs.java 34098fab6 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java e16148d4b 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerValidityScheduleEvaluator.java b48ff3b68 
>   agents-common/src/test/java/org/apache/ranger/plugin/model/validation/TestRangerValidator.java f9b342832 
>   dev-support/ranger-pmd-ruleset.xml f03cda371 
>   kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStore.java cd5a8c2a8 
>   plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java 262447857 
>   plugin-kms/src/main/java/org/apache/ranger/authorization/kms/authorizer/RangerKmsAuthorizer.java c3d75a14d 
>   plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSClient.java af0ac71f0 
>   plugin-nifi/src/main/java/org/apache/ranger/services/nifi/client/NiFiConnectionMgr.java 739bef62c 
>   pom.xml 8d7ea131e 
>   ranger-tools/src/test/java/org/apache/ranger/policyengine/RangerPolicyEnginePerformanceTest.java 11af0a84d 
>   security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java e4449df2e 
>   security-admin/src/main/java/org/apache/ranger/security/web/filter/RangerKRBAuthenticationFilter.java b4a3f9355 
>   security-admin/src/main/java/org/apache/ranger/security/web/filter/RangerKrbFilter.java ca0d17eb6 
>   security-admin/src/main/java/org/apache/ranger/service/RangerPolicyService.java a3ff825dd 
>   security-admin/src/main/java/org/apache/ranger/service/RangerServiceService.java df3fdb57c 
>   security-admin/src/test/java/org/apache/ranger/biz/TestServiceDBStore.java b0b011022 
> 
> 
> Diff: https://reviews.apache.org/r/68083/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Colm O hEigeartaigh
> 
>


Re: Review Request 68083: RANGER-2167 - Upgrade to Apache parent pom version 20

Posted by Colm O hEigeartaigh <co...@apache.org>.

> On Aug. 1, 2018, 5:36 a.m., bhavik patel wrote:
> > @Colm Can you please verify some basic functionality of ranger kms?

Yes I've tested creating some encryption keys in HDFS and storing them in the Ranger KMS.


- Colm


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


On July 27, 2018, 3:28 p.m., Colm O hEigeartaigh wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68083/
> -----------------------------------------------------------
> 
> (Updated July 27, 2018, 3:28 p.m.)
> 
> 
> Review request for ranger.
> 
> 
> Bugs: RANGER-2167
>     https://issues.apache.org/jira/browse/RANGER-2167
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> This task is to upgrade to the Apache parent pom version 20. This necessitates a PMD upgrade and quite a few fixes are required as a result. One notable change is to switch the KMS Keystore implementation to use ConcurrentHashMap instead of HashTable.
> 
> 
> Diffs
> -----
> 
>   agents-common/src/main/java/org/apache/ranger/plugin/model/RangerValiditySchedule.java 55f787d23 
>   agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerValidator.java ed5aa8d41 
>   agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerValidityScheduleValidator.java f34d124d6 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerResourceACLs.java 34098fab6 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java e16148d4b 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerValidityScheduleEvaluator.java b48ff3b68 
>   agents-common/src/test/java/org/apache/ranger/plugin/model/validation/TestRangerValidator.java f9b342832 
>   dev-support/ranger-pmd-ruleset.xml f03cda371 
>   kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStore.java cd5a8c2a8 
>   plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java 262447857 
>   plugin-kms/src/main/java/org/apache/ranger/authorization/kms/authorizer/RangerKmsAuthorizer.java c3d75a14d 
>   plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSClient.java af0ac71f0 
>   plugin-nifi/src/main/java/org/apache/ranger/services/nifi/client/NiFiConnectionMgr.java 739bef62c 
>   pom.xml 8d7ea131e 
>   ranger-tools/src/test/java/org/apache/ranger/policyengine/RangerPolicyEnginePerformanceTest.java 11af0a84d 
>   security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java e4449df2e 
>   security-admin/src/main/java/org/apache/ranger/security/web/filter/RangerKRBAuthenticationFilter.java b4a3f9355 
>   security-admin/src/main/java/org/apache/ranger/security/web/filter/RangerKrbFilter.java ca0d17eb6 
>   security-admin/src/main/java/org/apache/ranger/service/RangerPolicyService.java a3ff825dd 
>   security-admin/src/main/java/org/apache/ranger/service/RangerServiceService.java df3fdb57c 
>   security-admin/src/test/java/org/apache/ranger/biz/TestServiceDBStore.java b0b011022 
> 
> 
> Diff: https://reviews.apache.org/r/68083/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Colm O hEigeartaigh
> 
>