> On May 6, 2016, 10:29 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/params.py, > > line 230 > > <https://reviews.apache.org/r/47060/diff/1/?file=1374560#file1374560line230> > > > > Not using default? It's 100% guaranteed to be there? > > Mugdha Varadkar wrote: > These wont fail, because of the lazy failing implemented. CC @Andrew > Onishuk who had asked in one of the earlier RR to use this feature instead of > default().
I'm not following - if `ranger.tagsync.keystore.filename` isn't in the dictionary, then this line will absolutely throw an error. - Jonathan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47060/#review132031 ----------------------------------------------------------- On May 9, 2016, 1: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, 1: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 > >