-----------------------------------------------------------
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
|