----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45321/#review125740 -----------------------------------------------------------
Ship it! ambari-server/src/main/java/org/apache/ambari/server/orm/entities/UpgradeEntity.java (lines 203 - 205) <https://reviews.apache.org/r/45321/#comment188591> I see this in trunk already as isSuspended() ? - Nate Cole On March 24, 2016, 6:28 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45321/ > ----------------------------------------------------------- > > (Updated March 24, 2016, 6:28 p.m.) > > > Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, Jonathan > Hurley, and Nate Cole. > > > Bugs: AMBARI-15569 > https://issues.apache.org/jira/browse/AMBARI-15569 > > > Repository: ambari > > > Description > ------- > > *STR:* > 1. Install Ambari 2.2.0 > 2. Upgrade stack from HDP-2.2.4 to HDP-2.3.2 > 3. At finalize step, choose downgrade option > 4. Introduce failures in some clients such as Zookeeper and Yarn. > > In this case, the status of some tasks are marked as ABORTED. > This causes the overall status of the Downgrade to be ABORTED, and hence the > UI shows an incorrect message and button. > Technically, the Downgrade is still progressing, yet the UI shows "Downgrade > paused" and a button to "Resume Downgrade" > > This means that the calculations of the stages is incorrect. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CalculatedStatus.java > 1d8df9b > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java > d43daca > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/UpgradeEntity.java > fd866a1 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/CalculatedStatusTest.java > 8d80fa6 > > Diff: https://reviews.apache.org/r/45321/diff/ > > > Testing > ------- > > Verified on live cluster and ran unit tests in CalculateStatusTest.java > > > Thanks, > > Alejandro Fernandez > >
