-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62778/
-----------------------------------------------------------
(Updated Oct. 5, 2017, 6:06 p.m.)
Review request for Ambari, Di Li, Jonathan Hurley, and Jayush Luniya.
Changes
-------
Removed commented out code
Bugs: AMBARI-22137
https://issues.apache.org/jira/browse/AMBARI-22137
Repository: ambari
Description
-------
Currently if you have extension version 0.1 linked to HDP 2.5, you can't link
extension version 0.2 to HDP 2.6.
Attempting to make that link will fail with an exception stating that the
services included in extension version 0.2 are already in HDP 2.6 (inherited
from HDP 2.5 via the 0.1 version of the extension).
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementHelper.java
2dd6f12
ambari-server/src/main/java/org/apache/ambari/server/stack/ExtensionHelper.java
86e532a
ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java
6dc2b93
ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerExtensionTest.java
20c8f40
ambari-server/src/test/resources/extensions/EXT/0.2/metainfo.xml 0d37b3e
ambari-server/src/test/resources/extensions/EXT/0.3/metainfo.xml d827314
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/metainfo.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/repos/repoinfo.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/services/HBASE/metainfo.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/services/HDFS/configuration/global.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/services/HDFS/configuration/hadoop-env.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/services/HDFS/configuration/hbase-site.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/services/HDFS/configuration/hdfs-log4j.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/services/HDFS/configuration/hdfs-site.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/services/HDFS/metainfo.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/services/HDFS/package/dummy-script.py
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/services/HIVE/metainfo.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/services/MAPREDUCE/metainfo.xml
PRE-CREATION
ambari-server/src/test/resources/stacks_with_extensions/HDP/0.4/services/ZOOKEEPER/metainfo.xml
PRE-CREATION
Diff: https://reviews.apache.org/r/62778/diff/2/
Changes: https://reviews.apache.org/r/62778/diff/1-2/
Testing
-------
Manual testing with different extension and stack versions.
mvn test -DskipPythonTests
-Dtest=StackManagerCommonServicesTest,StackManagerTest,StackManagerMiscTest,StackManagerExtensionTest
Thanks,
Tim Thorpe