----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45302/#review125623 -----------------------------------------------------------
Ship it! Ship It! - Dmitro Lisnichenko On March 27, 2016, 4:03 p.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45302/ > ----------------------------------------------------------- > > (Updated March 27, 2016, 4:03 p.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 > >