----------------------------------------------------------- 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 > >
