----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60595/#review179505 -----------------------------------------------------------
Ship it! I don't know why it's not showing Fix It, Then Ship It - but that's what I picked. Either way, I'll Ship It directly and you can correct the above comments. - Jonathan Hurley On July 3, 2017, 6:32 a.m., Attila Doroszlai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60595/ > ----------------------------------------------------------- > > (Updated July 3, 2017, 6:32 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Sumit > Mohanty. > > > Bugs: AMBARI-21390 > https://issues.apache.org/jira/browse/AMBARI-21390 > > > Repository: ambari > > > Description > ------- > > Old and new stack root paths are hard-coded for now. I will try to make it > configurable later. > The patch also copies BigInsights 4.2 upgrade pack to 4.2.5 to be able to > upgrade from the latter version. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/ChangeStackRootDirectoryAction.java > PRE-CREATION > > ambari-server/src/main/resources/stacks/BigInsights/4.2.5/upgrades/config-upgrade.xml > PRE-CREATION > > ambari-server/src/main/resources/stacks/BigInsights/4.2.5/upgrades/nonrolling-upgrade-to-hdp-2.6.xml > PRE-CREATION > > ambari-server/src/main/resources/stacks/BigInsights/4.2/upgrades/nonrolling-upgrade-to-hdp-2.6.xml > e8a1e33545c17faaf5856b77b77c4621249bb4ac > > ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/ChangeStackRootDirectoryActionTest.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/60595/diff/1/ > > > Testing > ------- > > Manual test on local cluster. > New unit test. > > > Thanks, > > Attila Doroszlai > >
