----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53980/#review158525 -----------------------------------------------------------
One thing I'm not clear on is how this will affect an existing installation. Won't Ranger have a problem if you change the name of a repo on an already running cluster? ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java (line 362) <https://reviews.apache.org/r/53980/#comment229287> Doc. ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java (line 377) <https://reviews.apache.org/r/53980/#comment229286> Doc. ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java (line 384) <https://reviews.apache.org/r/53980/#comment229288> I don't think this can be null/empty - ever. ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py (lines 221 - 229) <https://reviews.apache.org/r/53980/#comment229289> This is a lot of logic repeated over and over - can this be placed in a common function and just re-used in order to get the name of the repo? Similar to what you did in the upgrade? - Jonathan Hurley On Dec. 7, 2016, 2:36 p.m., Mugdha Varadkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53980/ > ----------------------------------------------------------- > > (Updated Dec. 7, 2016, 2:36 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jonathan > Hurley, Sumit Mohanty, and Velmurugan Periasamy. > > > Bugs: AMBARI-18945 > https://issues.apache.org/jira/browse/AMBARI-18945 > > > Repository: ambari > > > Description > ------- > > Need to add a provision for creating default repository with static name or > user provided name. > > This will help user to provided service name under Ranger supported plugins > config tab and can take dump of latest snapshot of all policies and use it > while recreating another/multiple clusters with same service name. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java > ac97987 > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py > 32e1510 > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py > f47dc8f > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params_linux.py > 64f67b9 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py > 496102e > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1/package/scripts/params.py > 5635fe3 > > ambari-server/src/main/resources/common-services/KAFKA/0.9.0/configuration/ranger-kafka-security.xml > 91061d1 > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/params_linux.py > 4cefbda > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/configuration/ranger-kms-security.xml > 95e653c > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/params.py > c20bdde > > ambari-server/src/main/resources/common-services/STORM/0.10.0/configuration/ranger-storm-security.xml > 983702f > > ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py > 4859534 > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py > 4d42861 > > ambari-server/src/main/resources/stacks/HDP/2.3/services/HBASE/configuration/ranger-hbase-security.xml > 1deb9e5 > > ambari-server/src/main/resources/stacks/HDP/2.3/services/HDFS/configuration/ranger-hdfs-security.xml > cfd8a4f > > ambari-server/src/main/resources/stacks/HDP/2.3/services/HIVE/configuration/ranger-hive-security.xml > ce4074a > > ambari-server/src/main/resources/stacks/HDP/2.3/services/KNOX/configuration/ranger-knox-security.xml > 9bd1079 > > ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/configuration/ranger-yarn-security.xml > 5f69962 > ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py > 3c34fc6 > > ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/ranger-atlas-security.xml > ea0a026 > ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py > 57555ee > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog250Test.java > d4f2e8b > ambari-server/src/test/python/stacks/2.0.6/configs/nn_eu.json a6ed6ce > ambari-server/src/test/python/stacks/2.3/common/test_stack_advisor.py > 4a77086 > ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py > f4538d7 > ambari-server/src/test/python/stacks/2.5/configs/ranger-kms-default.json > 05cb78a > ambari-server/src/test/python/stacks/2.5/configs/ranger-kms-secured.json > 7054e8f > ambari-server/src/test/python/stacks/2.6/common/test_stack_advisor.py > 011a874 > > Diff: https://reviews.apache.org/r/53980/diff/ > > > Testing > ------- > > mvn -DskipPythonTests -Dtest=UpgradeCatalog250Test test > > ------------------------------------------------------- > T E S T S > ------------------------------------------------------- > OpenJDK 64-Bit Server VM warning: ignoring option MaxPermSize=512m; support > was removed in 8.0 > Running org.apache.ambari.server.upgrade.UpgradeCatalog250Test > Tests run: 13, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.548 sec - > in org.apache.ambari.server.upgrade.UpgradeCatalog250Test > > Results : > > Tests run: 13, Failures: 0, Errors: 0, Skipped: 0 > > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 30.347 s > [INFO] Finished at: 2016-11-22T11:51:23+00:00 > [INFO] Final Memory: 63M/1107M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Mugdha Varadkar > >