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 2015/08/12 11:53:44 UTC

Review Request 37390: RANGER-606: added support for deny policies

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

Review request for ranger, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, and Selvamohan Neethiraj.


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


Repository: ranger


Description
-------

- Ranger policy model updated to support 3 types of policies: ALLOW, DENY, DENY_OTHERS
- TODO: review Ranger Admin policy edit UI for selecting the policyType (currently via a Radio button)
- TODO: update to Ranger Admin UI to include the policyType in policy listing page
- TODO: update HDFS authorizer to support DENY and DENY_OTHERS policyTypes i.e. don't fall back to hadoop-acl


Diffs
-----

  agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPolicy.java e9f9ef9 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerAbstractPolicyEvaluator.java 178b9d8 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java c3a3680 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyItemEvaluator.java bf9b243 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerOptimizedPolicyEvaluator.java 8e726f4 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerPolicyEvaluator.java 57b9723 
  agents-common/src/test/resources/policyengine/test_policyengine_tag_hdfs.json f228449 
  agents-common/src/test/resources/policyengine/test_policyengine_tag_hive.json e6e137d 
  security-admin/src/main/webapp/scripts/models/RangerPolicy.js feacfec 
  security-admin/src/main/webapp/scripts/modules/globalize/message/en.js c8d187e 
  security-admin/src/main/webapp/scripts/utils/XAEnums.js f94e200 
  security-admin/src/main/webapp/scripts/views/policies/RangerPolicyForm.js 0f36791 

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


Testing
-------


Thanks,

Madhan Neethiraj