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

Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, and 
Jonathan Hurley.


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


Repository: ambari


Description
-------

Add a way to supply the repo_version to use when creating a cluster.  This will 
result in a new cluster_version record BEFORE any hosts have been added to the 
cluster.  This work effectively unblocks the UI to be able to supply a version 
when creating the cluster.  Including Blueprints.

What this patch does NOT do (other patches will address this):
- Refactor how RepositoryVersionState is computed for installs to work just 
like upgrades.
- Create a cluster using a Version Definition File (VDF) url, pull down the 
file, and create the repo_version record.  To use this feature, the 
repo_version must be created BEFORE supplying the version with the cluster.


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java
 32da5e8 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
 23d43aa 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/ClusterRequest.java
 5c7548c 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterResourceProvider.java
 f7d359c 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ProvisionClusterRequest.java
 a1740fb 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/predicate/ComparisonPredicate.java
 cc8cfdb 
  ambari-server/src/main/java/org/apache/ambari/server/state/Cluster.java 
ddd07f9 
  
ambari-server/src/main/java/org/apache/ambari/server/state/RepositoryVersionState.java
 119205a 
  
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
 8d6fec1 
  
ambari-server/src/main/java/org/apache/ambari/server/topology/AmbariContext.java
 87225ad 
  
ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java
 c317162 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerImplTest.java
 ec38f22 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterResourceProviderTest.java
 57cbebc 
  
ambari-server/src/test/java/org/apache/ambari/server/topology/AmbariContextTest.java
 1613d11 
  
ambari-server/src/test/java/org/apache/ambari/server/topology/ClusterInstallWithoutStartTest.java
 6e7c975 
  
ambari-server/src/test/java/org/apache/ambari/server/topology/TopologyManagerTest.java
 91f4993 

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


Testing
-------

Manual.  Automated pending.


Thanks,

Nate Cole

Reply via email to