You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Gautam Borad <gb...@gmail.com> on 2015/09/29 18:40:27 UTC

Review Request 38850: RANGER-274 : Ranger UI fixes related to tags (DGI)

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

Review request for ranger, Alok Lal, Don Bosco Durai, Abhay Kulkarni, Madhan Neethiraj, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan Periasamy.


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


Repository: ranger


Description
-------

Patch for solving few Ranger UI issues : 
1] Getting Page Not Found (404) page after deleting service on Ranger UI.
2] Add policyItem level validation in policy form for newly added items for example : allowException, denyItem, DenyException.
3] Show Group/User name on report and policy listing page for newly added policyItems.


Diffs
-----

  security-admin/src/main/webapp/scripts/utils/XAUtils.js 83eeff857526e099838ead1b88139dcafcfc17de 
  security-admin/src/main/webapp/scripts/views/policies/RangerPolicyCreate.js a5f0bcd494ebaaf27236f5bbeced34b05367fc6c 
  security-admin/src/main/webapp/scripts/views/policies/RangerPolicyForm.js 55c36518752a5372ea9800e193882acb45a47274 
  security-admin/src/main/webapp/scripts/views/reports/PlugableServiceDiffDetail.js ceb8898bd6651d682fc28bb0c351eb4006b14944 
  security-admin/src/main/webapp/scripts/views/service/ServiceCreate.js 7488cc5e1376704336dc66cd04abc17e0e45c980 
  security-admin/src/main/webapp/scripts/views/users/UserTableLayout.js 2ade8681b3dd2f79f892215cb9d0a7de8f4695ca 
  security-admin/src/main/webapp/styles/xa.css d17f9472845724c666cb8e0ae37c0e575328dc2c 

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


Testing
-------

Tested CRUD for Services, Policies on a instance having Ranger. Also checked for validations in Policy Edit form.


Thanks,

Gautam Borad


Re: Review Request 38850: RANGER-274 : Ranger UI fixes related to tags (DGI)

Posted by Madhan Neethiraj <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38850/#review100973
-----------------------------------------------------------

Ship it!


Ship It!

- Madhan Neethiraj


On Sept. 29, 2015, 4:40 p.m., Gautam Borad wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38850/
> -----------------------------------------------------------
> 
> (Updated Sept. 29, 2015, 4:40 p.m.)
> 
> 
> Review request for ranger, Alok Lal, Don Bosco Durai, Abhay Kulkarni, Madhan Neethiraj, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-274
>     https://issues.apache.org/jira/browse/RANGER-274
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Patch for solving few Ranger UI issues : 
> 1] Getting Page Not Found (404) page after deleting service on Ranger UI.
> 2] Add policyItem level validation in policy form for newly added items for example : allowException, denyItem, DenyException.
> 3] Show Group/User name on report and policy listing page for newly added policyItems.
> 
> 
> Diffs
> -----
> 
>   security-admin/src/main/webapp/scripts/utils/XAUtils.js 83eeff857526e099838ead1b88139dcafcfc17de 
>   security-admin/src/main/webapp/scripts/views/policies/RangerPolicyCreate.js a5f0bcd494ebaaf27236f5bbeced34b05367fc6c 
>   security-admin/src/main/webapp/scripts/views/policies/RangerPolicyForm.js 55c36518752a5372ea9800e193882acb45a47274 
>   security-admin/src/main/webapp/scripts/views/reports/PlugableServiceDiffDetail.js ceb8898bd6651d682fc28bb0c351eb4006b14944 
>   security-admin/src/main/webapp/scripts/views/service/ServiceCreate.js 7488cc5e1376704336dc66cd04abc17e0e45c980 
>   security-admin/src/main/webapp/scripts/views/users/UserTableLayout.js 2ade8681b3dd2f79f892215cb9d0a7de8f4695ca 
>   security-admin/src/main/webapp/styles/xa.css d17f9472845724c666cb8e0ae37c0e575328dc2c 
> 
> Diff: https://reviews.apache.org/r/38850/diff/
> 
> 
> Testing
> -------
> 
> Tested CRUD for Services, Policies on a instance having Ranger. Also checked for validations in Policy Edit form.
> 
> 
> Thanks,
> 
> Gautam Borad
> 
>