Re: Review Request 73992: RANGER-3767 : Add text message in HDFS and YARN policy pages to highlight the fallback ACL option.

[Dhaval Rajpara <dh...@gmail.com>]

> On May 25, 2022, 6:07 p.m., Kirby Zhou wrote:
> >

Currently, we added notice/hints for HDFS and YARN services. Once we have more info about other services we will definitely add that to the respective services.


- Dhaval


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


On May 18, 2022, 1:28 p.m., Dhaval Rajpara wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73992/
> -----------------------------------------------------------
> 
> (Updated May 18, 2022, 1:28 p.m.)
> 
> 
> Review request for ranger, Dhaval Shah, Dineshkumar Yadav, Harshal Chavan, Kishor Gollapalliwar, Mateen Mansoori, Mehul Parikh, Nitin Galave, Pradeep Agrawal, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3767
>     https://issues.apache.org/jira/browse/RANGER-3767
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> HDFS and YARN policy pages need to show a text message highlighting the fallback ACL option. By default, fallback is enabled for these components. Showing this message in policy page will allow customers to understand the behavior better and decide.
> 
> Message could be something like below.
> 
> "By default, fallback to [HDFS|Yarn] ACLs are enabled. If access cannot be determined by Ranger policies, authorization will fall back to [HDFS|Yarn] ACLs. If this behavior needs to be changed, modify [HDFS plugin config|Yarn plugin config]"
> 
> 
> Diffs
> -----
> 
>   security-admin/src/main/webapp/scripts/utils/XAEnums.js 6407b68f3 
>   security-admin/src/main/webapp/scripts/utils/XAUtils.js 37b35d2dd 
>   security-admin/src/main/webapp/scripts/views/policies/NRangerPolicyTableLayout.js d7ff26da1 
>   security-admin/src/main/webapp/scripts/views/policies/RangerPolicyTableLayout.js af7c70a80 
>   security-admin/src/main/webapp/templates/policies/RangerPolicyTableLayout_tmpl.html d1ca2a94c 
> 
> 
> Diff: https://reviews.apache.org/r/73992/diff/1/
> 
> 
> Testing
> -------
> 
> Tested that message display properly with HDFS and YARN base policy listing page with there config name.
> 
> 
> Thanks,
> 
> Dhaval Rajpara
> 
>