You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Nate Cole <nc...@hortonworks.com> on 2014/12/17 14:35:22 UTC

Review Request 29154: Rolling Upgrade - Upgrade Pack to restart standby ResourceManager first, failover, and then upgrade the remaining RM

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29154/
-----------------------------------------------------------

Review request for Ambari, Alejandro Fernandez and Jonathan Hurley.


Bugs: AMBARI-8724
    https://issues.apache.org/jira/browse/AMBARI-8724


Repository: ambari


Description
-------

This was the manual review from yesterday (not Jonathan's changes, those will be future).  Alejandro, please look at this ASAP.


Runbook outlines that RM needs to first upgrade the standby, then failover to make the upgrade standby the active, and then upgrade the RM that is now the standby.


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/stack/MasterHostResolver.java 3baf187 
  ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java d7c09ea 
  ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ClusterGrouping.java 77e9922 
  ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ColocatedGrouping.java cc8b57b 
  ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml 285e876 
  ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java 6646b5e 

Diff: https://reviews.apache.org/r/29154/diff/


Testing
-------

Tests run: 2428, Failures: 0, Errors: 0, Skipped: 13

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 22:02.145s
[INFO] Finished at: Tue Dec 16 17:38:25 EST 2014
[INFO] Final Memory: 31M/331M
[INFO] ------------------------------------------------------------------------


Thanks,

Nate Cole


Re: Review Request 29154: Rolling Upgrade - Upgrade Pack to restart standby ResourceManager first, failover, and then upgrade the remaining RM

Posted by Jonathan Hurley <jh...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29154/#review65321
-----------------------------------------------------------

Ship it!


Ship It!

- Jonathan Hurley


On Dec. 17, 2014, 8:35 a.m., Nate Cole wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29154/
> -----------------------------------------------------------
> 
> (Updated Dec. 17, 2014, 8:35 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez and Jonathan Hurley.
> 
> 
> Bugs: AMBARI-8724
>     https://issues.apache.org/jira/browse/AMBARI-8724
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> This was the manual review from yesterday (not Jonathan's changes, those will be future).  Alejandro, please look at this ASAP.
> 
> 
> Runbook outlines that RM needs to first upgrade the standby, then failover to make the upgrade standby the active, and then upgrade the RM that is now the standby.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/stack/MasterHostResolver.java 3baf187 
>   ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java d7c09ea 
>   ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ClusterGrouping.java 77e9922 
>   ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ColocatedGrouping.java cc8b57b 
>   ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml 285e876 
>   ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java 6646b5e 
> 
> Diff: https://reviews.apache.org/r/29154/diff/
> 
> 
> Testing
> -------
> 
> Tests run: 2428, Failures: 0, Errors: 0, Skipped: 13
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 22:02.145s
> [INFO] Finished at: Tue Dec 16 17:38:25 EST 2014
> [INFO] Final Memory: 31M/331M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Nate Cole
> 
>