> On May 16, 2016, 4:19 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.1/upgrades/nonrolling-upgrade-2.3.xml, > > line 40 > > <https://reviews.apache.org/r/47421/diff/1/?file=1384744#file1384744line40> > > > > Do we need a safeguard so that any intermediate stacks outside of the > > range are inogred?
If you define a stack outside the range, I don't think it would hurt anything. The configurations aren't necessarily created if the upgrade pack doesn't use them. It just creates the "in-memory" definitions available to pick from when reading an upgrade pack. - Jonathan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47421/#review133427 ----------------------------------------------------------- On May 16, 2016, 2:50 p.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47421/ > ----------------------------------------------------------- > > (Updated May 16, 2016, 2:50 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, Jayush > Luniya, and Nate Cole. > > > Bugs: AMBARI-16687 > https://issues.apache.org/jira/browse/AMBARI-16687 > > > Repository: ambari > > > Description > ------- > > During an upgrade from HDP 2.x to 2.y, all of the configuration tasks are > being shown as skipped. This is due to AMBARI-15222 where the configuration > packs were being calculated incorrectly resulting in empty maps. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java > b069862 > > ambari-server/src/main/java/org/apache/ambari/server/stack/StackDefinitionDirectory.java > f2e00fe > > ambari-server/src/main/resources/stacks/HDP/2.1/upgrades/nonrolling-upgrade-2.3.xml > 564b5bd > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/config-upgrade.xml > 257893f > > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.4.xml > 1fa4d00 > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.4.xml > bf041de > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/config-upgrade.xml > 45e2f94 > > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.5.xml > 7873853 > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.5.xml > 1cd2ffa > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/config-upgrade.xml > f71ef1a > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/upgrade-2.5.xml > 9c6a02d > ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/config-upgrade.xml > a7dbba3 > > ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/nonrolling-upgrade-2.5.xml > 2e5c002 > ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/upgrade-2.5.xml > 1745de8 > > ambari-server/src/test/java/org/apache/ambari/server/orm/InMemoryDefaultTestModule.java > 771f830 > > ambari-server/src/test/java/org/apache/ambari/server/state/stack/ConfigUpgradeValidityTest.java > PRE-CREATION > > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/config-upgrade.xml > cb034d0 > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test.xml > 623b45c > > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test_partial.xml > 4932e92 > > Diff: https://reviews.apache.org/r/47421/diff/ > > > Testing > ------- > > Wrote a new unit test which validates all upgrade XML files which have > configuration tasks. > > > Thanks, > > Jonathan Hurley > >