ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alejandro Fernandez" <afernan...@hortonworks.com>
Subject Re: Review Request 30343: RU - FinalizeUpgradeAction needs to ignore hosts without a host_version record
Date Thu, 29 Jan 2015 00:10:35 GMT

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

(Updated Jan. 29, 2015, 12:10 a.m.)


Review request for Ambari, Jonathan Hurley, Nate Cole, and Yurii Shylov.


Changes
-------

Fixed more bugs after testing with a cluster that only had AMS on a host.


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


Repository: ambari


Description
-------

Install a cluster such that one host only has AMS component, so that it does not create a
host_version record for that host.
The Finalize Upgrade Action will fail because the host was never added to the set of hosts
with a correct version state.


Diffs (updated)
-----

  ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostComponentStateDAO.java
fd1ce9e2d7e7b55e6a48cfc3dc559d94ae59c10f 
  ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostComponentStateEntity.java
1e489504ae3096f85f73628caff2d3bd0d6d8f84 
  ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FinalizeUpgradeAction.java
56281a2fe8808c3454ad97223a04582aec18e7b2 
  ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java 0bfaa191270cc5589fc10bc9a6f8a5843cc77070

  ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
dbdef2bfc136404133ad23fb65ba5406003fdeee 
  ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostSummary.java
PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml 139dbdb10cf00d39f4c0cca8359741e3da4e73ff


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


Testing
-------

In-progress


Thanks,

Alejandro Fernandez


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