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

Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Nate Cole.


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


Repository: ambari


Description
-------

Hosts which are in maintenance mode are still included in the list of candidate 
hosts for service checks when checking a client-only service such as Tez or 
Slider.

- Install a cluster with at least 2 hosts which have a client-only service like 
Tez or Slider
- Place 1 of 2  client-only service hosts into maintenance mode 
- Upgrade

The orchestration will have a used the host in maintenance mode for service 
checks for the client-only service.


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/controller/ActionExecutionContext.java
 923fd92 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java
 1eb9ee5 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementController.java
 cbc5a95 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/MaintenanceStateHelper.java
 4c946dc 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
 0f9e65e 

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


Testing
-------

Live testing on an existing cluster which had the problem. Working on unit 
tests now.


Thanks,

Jonathan Hurley

Reply via email to