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 29969: Rollling Upgrade - Server bootstrap creates incorrect repo version
Date Fri, 16 Jan 2015 18:46:44 GMT

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



ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
<https://reviews.apache.org/r/29969/#comment112678>

    We need to be careful with the implications of this, because it forces the cluster_version
to be created when agents heartbeat with a component. I'll need to go through it more carefully,
so hold off for now.



ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
<https://reviews.apache.org/r/29969/#comment112683>

    We should not allow creating a new cluster_version in an installing state.


- Alejandro Fernandez


On Jan. 16, 2015, 6:26 p.m., Yurii Shylov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29969/
> -----------------------------------------------------------
> 
> (Updated Jan. 16, 2015, 6:26 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, and Nate Cole.
> 
> 
> Bugs: AMBARI-9183
>     https://issues.apache.org/jira/browse/AMBARI-9183
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Currently on clean install Ambari creates cluster entity and its repository version with
version taken from selected stack. I.e. for stack HDP-2.2, version 2.2 is written to database,
without any information about the actual minor version of that stack. Besides that we can't
assume that cluster has some version until it has at least one of components of that version.
> Repository version creation should be detached from cluster creation; instead of that
versions should be created after receiving the first actual version from service responses
during install.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/agent/HeartBeatHandler.java b5fda49

>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
6dabcbb 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
9b17b33 
>   ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
19a5f9f 
>   ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java
9ec8c36 
>   ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
31606ca 
>   ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterTest.java
57c0223 
> 
> Diff: https://reviews.apache.org/r/29969/diff/
> 
> 
> Testing
> -------
> 
> In progress
> 
> Manual testing was performed on 3-node cluster
> 
> 
> Thanks,
> 
> Yurii Shylov
> 
>


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