Review Request 45302: Use repository version number to indicate repository to use when installing
--- 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
Re: Review Request 45302: Use repository version number to indicate repository to use when installing
--- 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 (updated) --- 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
Re: Review Request 45302: Use repository version number to indicate repository to use when installing
--- 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 > >
Re: Review Request 45302: Use repository version number to indicate repository to use when installing
--- 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 > >