----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41120/#review109524 -----------------------------------------------------------
Ship it! Ship It! - Robert Nettleton On Dec. 9, 2015, 6:30 a.m., Sid Wagle wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41120/ > ----------------------------------------------------------- > > (Updated Dec. 9, 2015, 6:30 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Mahadev Konar, Robert > Nettleton, Sumit Mohanty, and Srimanth Gunturi. > > > Bugs: AMBARI-14283 > https://issues.apache.org/jira/browse/AMBARI-14283 > > > Repository: ambari > > > Description > ------- > > - Blueprint deploy the cluster with install, configure and start all in one > shot. > - In a scenario where user wants to use Ambari to takeover a live cluster, it > would become desirable to allow blueprint to only perform INSTALL and > CONFIGURE actions. > - This patch allows specification of a _provision_action_ to allow > overriding this behavior. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterResourceProvider.java > 84c13b9 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ProvisionClusterRequest.java > 56c4b76 > > ambari-server/src/main/java/org/apache/ambari/server/topology/ClusterTopology.java > 0ac2e2a > > ambari-server/src/main/java/org/apache/ambari/server/topology/ClusterTopologyImpl.java > 687c2e1 > > ambari-server/src/main/java/org/apache/ambari/server/topology/HostRequest.java > 518cbaf > > ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java > 960915f > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterResourceProviderTest.java > 84de604 > > ambari-server/src/test/java/org/apache/ambari/server/topology/ClusterInstallWithoutStartTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/41120/diff/ > > > Testing > ------- > > Added unit tests. > Manual testing done. > > > Thanks, > > Sid Wagle > >
