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

(Updated Jan. 9, 2016, 9:02 p.m.)


Review request for Ambari, Robert Nettleton, Sumit Mohanty, and Sid Wagle.


Changes
-------

re-upload patch


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


Repository: ambari


Description
-------

1. If the cluster is provisioned using Blueprint and if there are no tasks 
created verify the status of the topology logical request (that will exist only 
for Blueprints) to determine if the request was prior completed or it's waiting 
for hosts to register and be assigned to host groups. 
2. Upon host deletion delete the topology host requests associated with the 
deleted host.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostResourceProvider.java
 f0d9fde 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java
 1a71afa 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostGroupComponentEntity.java
 046bbd8 
  
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java
 ca09ab8 
  
ambari-server/src/main/java/org/apache/ambari/server/topology/LogicalRequest.java
 ad7c8fd 
  
ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java
 80d19d2 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/HostResourceProviderTest.java
 03ef06d 
  
ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterImplTest.java
 178adc5 

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


Testing
-------

Manual testing on the 3 node cluster created using Blueprint (before agents 
registered with ambari server)

Unit tests:
Results :

Tests run: 3741, Failures: 0, Errors: 0, Skipped: 29


Thanks,

Sebastian Toader

Reply via email to