> On July 3, 2017, 4:09 p.m., Jonathan Hurley wrote: > > 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.
With the new design Review Board shows "Fix It" on top of "Ship It" (the green background is slightly visible) instead of "Fix It, then Ship It". - Attila ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60595/#review179505 ----------------------------------------------------------- On July 3, 2017, 4:19 p.m., Attila Doroszlai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60595/ > ----------------------------------------------------------- > > (Updated July 3, 2017, 4:19 p.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/2/ > > > Testing > ------- > > Manual test on local cluster. > New unit test. > > > Thanks, > > Attila Doroszlai > >
