-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36564/
-----------------------------------------------------------
Review request for Ambari, John Speidel, Sumit Mohanty, and Sid Wagle.
Bugs: AMBARI-12429
https://issues.apache.org/jira/browse/AMBARI-12429
Repository: ambari
Description
-------
Created a cluster with multiple hosts, then stopped all of the services on 1
host (preferably one with only clients so it has nothing to stop). Then deleted
the host using the API.
E.g., curl -u admin:admin -H "X-Requested-By: ambari" -X DELETE
http://c6401.ambari.apache.org:8080/api/v1/hosts/c6404.ambari.apache.org
This led to Null Pointer Exceptions in ambari-server
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/Request.java
faebb20
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
de9ae52
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostResourceProvider.java
4c14426
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestOperationLevel.java
c7c0160
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java
fa49d7f
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/RequestOperationLevelDAO.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/RequestOperationLevelEntity.java
2c11e55
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java
665dd56
ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
9f25ad7
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/RequestOperationLevelTest.java
bd4ad90
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/RequestResourceProviderTest.java
ed8336e
Diff: https://reviews.apache.org/r/36564/diff/
Testing
-------
Still running system tests, and waiting for unit test results.
Thanks,
Alejandro Fernandez
