ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dineshkumar Yadav <dineshkumar.ya...@outlook.com>
Subject Re: Review Request 72299: RANGER-2772 : Adding the functionality of merging the policy
Date Fri, 03 Apr 2020 06:12:53 GMT

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

(Updated April 3, 2020, 6:12 a.m.)


Review request for ranger, Ankita Sinha, Gautam Borad, Abhay Kulkarni, Mehul Parikh, Pradeep
Agrawal, Vishal Suvagia, and Velmurugan Periasamy.


Changes
-------

added ranger community for review


Repository: ranger


Description
-------

Adding the functionality of merging policy while creation of the policy.
The following conditions should match in order to merge the policy.
   1. There should be already existing policy.
   2. One should pass param mergeIfExists=true in create Policy API
   3. One should also pass param "serviceName" and "policyName". 
   4. You can pass "zoneName" if available.


Diffs
-----

  security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 8618f3279 


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


Testing
-------

1. Existing create or update policy.
2. Creating and updating policy using postman
3. Creating policy if already exist using below param
    a) mergeIfExists=true
    b) serviceName
    c) policyName
   expected behavior policy should get merged in existing policy.
4. Creating policy if already exist using below param
    a) updateIfExists=true
    b) serviceName
    c) policyName
   expected behavior policy should get updated with existing policy.
5. Creating policy if already exist using below param
    a) deleteIfExists=true
    b) serviceName
    c) policyName
   expected behavior existing policy should get deleted and new policy is created.


Thanks,

Dineshkumar Yadav


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message