----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46016/#review129141 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/params.py (lines 209 - 211) <https://reviews.apache.org/r/46016/#comment192602> Instead of the if/else conditions, you can use the default("/foo/bar", defaultValue) construct (see above in this file) ambari-server/src/main/resources/stacks/HDP/2.5/services/RANGER_KMS/metainfo.xml (line 26) <https://reviews.apache.org/r/46016/#comment192604> I don't think we ship 0.6.0.2.5; does it need to get added to common services? Maybe Jayush who has more experience with common services can comment. - Jonathan Hurley On April 15, 2016, 1:34 a.m., Mugdha Varadkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46016/ > ----------------------------------------------------------- > > (Updated April 15, 2016, 1:34 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, Jonathan > Hurley, Jayush Luniya, and Velmurugan Periasamy. > > > Bugs: AMBARI-15752 > https://issues.apache.org/jira/browse/AMBARI-15752 > > > Repository: ambari > > > Description > ------- > > Add HSM support configuration properties for Ranger KMS in hdp-2.5 > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/constants.py > 86dbc48 > > ambari-common/src/main/python/resource_management/libraries/functions/stack_features.py > 7201c36 > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/kms.py > dd91acf > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/params.py > fc722da > > ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_features.json > 03b7e01 > > ambari-server/src/main/resources/stacks/HDP/2.5/services/RANGER_KMS/configuration/dbks-site.xml > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.5/services/RANGER_KMS/configuration/kms-env.xml > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.5/services/RANGER_KMS/metainfo.xml > 02fb6aa > > ambari-server/src/main/resources/stacks/HDP/2.5/services/RANGER_KMS/themes/theme_version_2.json > PRE-CREATION > > Diff: https://reviews.apache.org/r/46016/diff/ > > > Testing > ------- > > Installed Ranger KMS centos6. > > > Thanks, > > Mugdha Varadkar > >
