ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tom Beerbower" <tbeerbo...@hortonworks.com>
Subject Review Request 29237: Rolling Upgrade: inconsistent statuses
Date Fri, 19 Dec 2014 04:19:25 GMT

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

Review request for Ambari, Jonathan Hurley and Nate Cole.


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


Repository: ambari


Description
-------

1. The currently executing upgrade group goes back and forth from IN_PROGRESS to PENDING state.

2. Completed upgrade groups should have COMPLETED state instead of PENDING.
3. When the upgrade is paused, overall progress shouldn't be at 100%. The percentage completion
should remain as is.


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java
549a20d 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StageResourceProvider.java
f2ea76d 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeGroupResourceProvider.java
46dcd1e 
  ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StageResourceProviderTest.java
c97de8a 

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


Testing
-------

Manual tests.

New unit tests added.

All pass.

Results :

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

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 27:02 min
[INFO] Finished at: 2014-12-18T22:59:53-05:00
[INFO] Final Memory: 34M/449M
[INFO] ------------------------------------------------------------------------


Thanks,

Tom Beerbower


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message