----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47060/#review132998 -----------------------------------------------------------
@Mugdha, In your patch for AMBARI-13349 you had add logic in alert_ranger_admin_passwd_check.py in which based on HDP version we are triggering different behavior. I had missed it in that review. Can you also update that logic in this patch? security_enabled = False stack_is_hdp25_or_further = Script.is_stack_greater_or_equal("2.5") try: if security_enabled and stack_is_hdp25_or_further: result_code = 'UNKNOWN' label = 'This alert will get skipped for Ranger Admin on kerberos env' else: - Jayush Luniya On May 9, 2016, 5:58 a.m., Mugdha Varadkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47060/ > ----------------------------------------------------------- > > (Updated May 9, 2016, 5:58 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Gautam > Borad, Jonathan Hurley, Jayush Luniya, and Velmurugan Periasamy. > > > Bugs: AMBARI-16285 > https://issues.apache.org/jira/browse/AMBARI-16285 > > > Repository: ambari > > > Description > ------- > > Changes include: > 1) Renaming properties in ranger-tagsync-site.xml introduced in stack -2.5 > followed by theme change. > 2) Removed tagsync user password introduced in previous patch > 3) Add default password in jceks for rangertagsync and rangerusersync > 4) Derive tag source as ATLAS, if ATLAS is installed > 5) Added new properties under ranger-ugsync-site.xml related to Policy > manager user's credentials in Ranger usersync service > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/constants.py > abd2212 > > ambari-common/src/main/python/resource_management/libraries/functions/stack_features.py > daa97de > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/params.py > 59190fe > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_tagsync.py > 1b56044 > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_usersync.py > d013371 > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/setup_ranger_xml.py > ec0eea1 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_features.json > aabc37d > > ambari-server/src/main/resources/stacks/HDP/2.5/services/RANGER/configuration/ranger-tagsync-site.xml > 08f6235 > > ambari-server/src/main/resources/stacks/HDP/2.5/services/RANGER/configuration/ranger-ugsync-site.xml > 10ef1a8 > > ambari-server/src/main/resources/stacks/HDP/2.5/services/RANGER/configuration/tagsync-application-properties.xml > d369098 > > ambari-server/src/main/resources/stacks/HDP/2.5/services/RANGER/themes/theme_version_3.json > 19411d4 > ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py > 37e6ef6 > ambari-web/app/styles/widgets.less f385449 > > Diff: https://reviews.apache.org/r/47060/diff/ > > > Testing > ------- > > Tested Ranger Tagsync on centos 6. > > > Thanks, > > Mugdha Varadkar > >