-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72325/
-----------------------------------------------------------
(Updated April 13, 2020, 12:51 p.m.)
Review request for ranger, Ankita Sinha, Gautam Borad, Kishor Gollapalliwar, Abhay Kulkarni,
Mehul Parikh, Pradeep Agrawal, Vishal Suvagia, and Velmurugan Periasamy.
Changes
-------
Removing policy condition functionality when policy send with mergeIfExist=true
Repository: ranger
Description
-------
Handling the scenario when non-existing policy send with mergeIfExist=true
Diffs (updated)
-----
security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 38b49820b
security-admin/src/main/java/org/apache/ranger/rest/ServiceRESTUtil.java d85028c95
Diff: https://reviews.apache.org/r/72325/diff/4/
Changes: https://reviews.apache.org/r/72325/diff/3-4/
Testing (updated)
-------
Test case:
Creating non-existing policy with below param
a) mergeIfExists=true
b) serviceName
c) policyName
Expected: New policy should be created.
Creating policy with below param and role which is already exist
a) mergeIfExists=true
b) serviceName
c) policyName
Expected: role should get merged.
Thanks,
Dineshkumar Yadav
|