----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53901/#review156344 -----------------------------------------------------------
Ship it! Ship It! - Alejandro Fernandez On Nov. 18, 2016, 9:12 p.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53901/ > ----------------------------------------------------------- > > (Updated Nov. 18, 2016, 9:12 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Jayush > Luniya. > > > Bugs: AMBARI-18937 > https://issues.apache.org/jira/browse/AMBARI-18937 > > > Repository: ambari > > > Description > ------- > > Currently, a custom service can participate in RU/EU only if it adds an > Upgrade Pack for every target of a stack. If there is a case of the Upgrade > Pack ALWAYS being the same no matter the version, then only one file should > be sufficient. > > Also added that a group name must be unique in a XSD since the logic already > dictated that it must be true (and associated fixes). > > > Diffs > ----- > > ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java > 37f4167 > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/UpgradePack.java > edf5c89 > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.5.xml > 95c5f06 > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.6.xml > a01996a > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/upgrade-2.5.xml > f0c6131 > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/upgrade-2.6.xml > f520faf > ambari-server/src/main/resources/upgrade-pack.xsd 2871f05 > > ambari-server/src/test/java/org/apache/ambari/server/state/stack/UpgradePackTest.java > dde7ffa > > ambari-server/src/test/resources/stacks/HDP/2.2.0/services/GANGLIA/upgrades/HDP/rolling-upgrade.xml > PRE-CREATION > > Diff: https://reviews.apache.org/r/53901/diff/ > > > Testing > ------- > > Manual. Automated pending. > > > Thanks, > > Nate Cole > >
