Re: Review Request 62778: AMBARI-22137 - Different stack versions should be able to link to different extension versions
--- 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
Re: Review Request 62778: AMBARI-22137 - Different stack versions should be able to link to different extension versions
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62778/#review187171 --- Ship it! Ship It! - Di Li On Oct. 4, 2017, 7:51 p.m., Tim Thorpe wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62778/ > --- > > (Updated Oct. 4, 2017, 7:51 p.m.) > > > Review request for Ambari, Di Li, Jonathan Hurley, and Jayush Luniya. > > > 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 > - > > > 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/1/ > > > Testing > --- > > Manual testing with different extension and stack versions. > > mvn test -DskipPythonTests > -Dtest=StackManagerCommonServicesTest,StackManagerTest,StackManagerMiscTest,StackManagerExtensionTest > > > Thanks, > > Tim Thorpe > >
Review Request 62778: AMBARI-22137 - Different stack versions should be able to link to different extension versions
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62778/ --- Review request for Ambari, Di Li, Jonathan Hurley, and Jayush Luniya. 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 - 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/1/ Testing --- Manual testing with different extension and stack versions. mvn test -DskipPythonTests -Dtest=StackManagerCommonServicesTest,StackManagerTest,StackManagerMiscTest,StackManagerExtensionTest Thanks, Tim Thorpe