----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73992/#review224465 -----------------------------------------------------------
Ship it! Ship It! - Mehul Parikh 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 > >