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


Ship it!




Ship It!

- Jonathan Hurley


On March 27, 2016, 9:03 a.m., Nate Cole wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45302/
> -----------------------------------------------------------
> 
> (Updated March 27, 2016, 9:03 a.m.)
> 
> 
> 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:
> 
> Tests run: 3992, Failures: 0, Errors: 0, Skipped: 33
> 
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Total time: 33:21.378s
> [INFO] Finished at: Thu Mar 24 15:59:06 EDT 2016
> [INFO] Final Memory: 33M/610M
> [INFO] 
> ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Nate Cole
> 
>

Reply via email to