ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Krisztian Horvath" <krisztian.horv...@sequenceiq.com>
Subject Review Request 25185: Ambari Groovy client enhancements
Date Fri, 29 Aug 2014 09:01:46 GMT

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

Review request for Ambari and Yusaku Sako.


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


Repository: ambari


Description
-------

new:
addHost - adds a registered host to the cluster
removeHost - gracefully decommission and remove the host from the cluster (synchronous)
waitForRequestsToFinish - waits for any given request to finish, used to synchronize rest
calls
decommissionDataNode - decommissions the datanode on a given host
decommissionNodeManager - decommissions the nodemanager on a given host
decommission - can decommission any given components which supports decommission
deleteHostComponents - deletes the given components from a host
deleteHost - deletes the host from the cluster
installComponentsToHost - can install components from either blueprint or from the specified
component list
restartServiceComponents - restarts the given services components on all hosts
startComponentsOnHost - starts the provided components on a host
stopComponentsOnHost - stops the provided components on a host
validateBlueprint - can validate the blueprint for recommendAssignments
modifyConfiguration - can modify any services configuration (the whole config must be provided)
getHostNamesByState - retrieves the hosts by state
getPublicHostNames - returns the public host names which the provided host components are
installed to
getInternalHostNames - returns the internal host names which the provided host components
are installed to
getClusterHosts - returns only the hosts which are part of the cluster
resolveInternalHostName - can resolve a host's internal name to the public one
modified:
recommendAssignments - host group and host recommendation changed:
at least host group number of host is required
if there are more hosts than host groups in the blueprint
all host group get a single node and the leftover hosts are distributed
among all the slave_X host groups
getInstallProgress - renamed to getRequestProgress
startAllServices - return type changed
stopAllServices - return type changed
blueprints:
HDP single/multi node blueprints added to defaults
dependencies:
commons-io added


Diffs
-----

  ambari-client/groovy-client/pom.xml 9600cd7 
  ambari-client/groovy-client/src/main/groovy/org/apache/ambari/groovy/client/AmbariClient.groovy
85c52a4 
  ambari-client/groovy-client/src/main/groovy/org/apache/ambari/groovy/client/InvalidBlueprintException.groovy
PRE-CREATION 
  ambari-client/groovy-client/src/main/groovy/org/apache/ambari/groovy/client/InvalidHostGroupHostAssociation.groovy
PRE-CREATION 
  ambari-client/groovy-client/src/main/resources/blueprints/hdp-multinode-default PRE-CREATION

  ambari-client/groovy-client/src/main/resources/blueprints/hdp-singlenode-default PRE-CREATION

  ambari-client/groovy-client/src/main/resources/blueprints/lambda-architecture 3deec4c 
  ambari-client/groovy-client/src/main/resources/blueprints/multi-node-hdfs-yarn 27a602a 
  ambari-client/groovy-client/src/main/resources/blueprints/single-node-hdfs-yarn 46ca508

  ambari-client/groovy-client/src/test/groovy/org/apache/ambari/groovy/client/AmbariBlueprintsTest.groovy
b467c56 
  ambari-client/groovy-client/src/test/groovy/org/apache/ambari/groovy/client/AmbariHostsTest.groovy
80d2f51 
  ambari-client/groovy-client/src/test/groovy/org/apache/ambari/groovy/client/AmbariRecommendTest.groovy
PRE-CREATION 
  ambari-client/groovy-client/src/test/groovy/org/apache/ambari/groovy/client/TestResources.groovy
f9ee519 
  ambari-client/groovy-client/src/test/resources/blueprint-config.json PRE-CREATION 
  ambari-client/groovy-client/src/test/resources/hdp-multinode-default.json PRE-CREATION 
  ambari-client/groovy-client/src/test/resources/hdp-multinode-default2.json PRE-CREATION

  ambari-client/groovy-client/src/test/resources/multi-node-hdfs-yarn-config.json PRE-CREATION

  ambari-client/groovy-client/src/test/resources/multi-node-hdfs-yarn.json PRE-CREATION 
  ambari-shell/ambari-groovy-shell/src/main/java/org/apache/ambari/shell/commands/ClusterCommands.java
dafdb85 
  ambari-shell/ambari-groovy-shell/src/main/java/org/apache/ambari/shell/flash/InstallProgress.java
69164ea 
  ambari-shell/ambari-groovy-shell/src/test/java/org/apache/ambari/shell/commands/ClusterCommandsTest.java
777d05d 

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


Testing
-------

unit tests applied


Thanks,

Krisztian Horvath


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message