----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50336/ -----------------------------------------------------------
(Updated July 22, 2016, 4:09 p.m.) Review request for Ambari, Alejandro Fernandez, Oliver Szabo, and Robert Levas. Changes ------- Applied review note. Bugs: AMBARI-17853 https://issues.apache.org/jira/browse/AMBARI-17853 Repository: ambari Description ------- Problem: While upgrading HDP 2.4 to HDP 2.5 a cluster having namenode HA unwanted configurations get added to the cluster configurations. Resolution: Affected upgrade scripts have benn updated to perform hdfs namenode HA related configuration adjustments. (Similar to Earlier upgrade configurations) Diffs (updated) ----- ambari-agent/src/main/python/ambari_agent/ActionQueue.py 1e7b1b6 ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionScheduler.java 205ef9f ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ExecutionCommandWrapper.java ef12c3a ambari-server/src/main/java/org/apache/ambari/server/agent/ExecutionCommand.java 28de8ed ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommand.java 6f612af ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java 399f26c ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariActionExecutionHelper.java 9c2db1c ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 2174a64 ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java 066acab ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java af24a69 ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java c390f44 ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java 216d850 ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java 8dcd163 ambari-server/src/main/java/org/apache/ambari/server/topology/ConfigurationFactory.java f4dc879 ambari-server/src/main/resources/common-services/RANGER/0.6.0/configuration/ranger-admin-site.xml 477df7a ambari-server/src/main/resources/common-services/RANGER/0.6.0/kerberos.json 94c681d ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/livy_server.py 1e859a8 ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/params.py 61f73fb ambari-server/src/main/resources/stacks/HDP/2.2/services/stack_advisor.py b77980a ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/config-upgrade.xml bd1bef2 ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/nonrolling-upgrade-2.5.xml 065a933 ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/upgrade-2.5.xml d152f34 ambari-server/src/main/resources/stacks/stack_advisor.py 437fe4f ambari-server/src/test/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommandTest.java 25d634b ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelperTest.java 7a5f377 ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerImplTest.java 37a2e63 ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProviderTest.java 9603226 ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ProvisionClusterRequestTest.java 0610d10 ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderHDP22Test.java 115b518 ambari-server/src/test/java/org/apache/ambari/server/topology/ConfigurationFactoryTest.java 1c2b177 ambari-server/src/test/python/stacks/2.2/common/test_stack_advisor.py 5fe534a ambari-server/src/test/python/stacks/2.5/SPARK/test_spark_livy.py PRE-CREATION ambari-server/src/test/python/stacks/2.5/configs/default.json 148616b ambari-web/app/controllers/wizard/step7_controller.js 62a48d7 ambari-web/app/mixins/common/configs/enhanced_configs.js 4436cbb ambari-web/app/routes/main.js ebf9cb3 ambari-web/test/controllers/wizard/step7_test.js f54802e Diff: https://reviews.apache.org/r/50336/diff/ Testing ------- Manual testing underway. Thanks, Laszlo Puskas