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

Review request for Ambari, Alejandro Fernandez, Di Li, and Jayush Luniya.


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


Repository: ambari


Description
-------

Services should be able to provide their own pre-req checks by supplying a jar 
file.

This would allow custom services to supply their own jar files to handle 
pre-req checks rather than forcing third party developers to make changes to 
ambari-server code.


Diffs
-----

  ambari-server/pom.xml 5731c9d 
  
ambari-server/src/main/java/org/apache/ambari/server/checks/CheckDescription.java
 498481d 
  
ambari-server/src/main/java/org/apache/ambari/server/checks/UpgradeCheckRegistry.java
 4ed345c 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/PreUpgradeCheckResourceProvider.java
 7d7b618 
  
ambari-server/src/main/java/org/apache/ambari/server/stack/CommonServiceDirectory.java
 cdedbb4 
  
ambari-server/src/main/java/org/apache/ambari/server/stack/ServiceDirectory.java
 30663a3 
  ambari-server/src/main/java/org/apache/ambari/server/stack/ServiceModule.java 
a77a22f 
  
ambari-server/src/main/java/org/apache/ambari/server/stack/StackServiceDirectory.java
 8656896 
  ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java 
6fda8bc 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/PreUpgradeCheckResourceProviderTest.java
 PRE-CREATION 
  
ambari-server/src/test/java/org/apache/ambari/server/sample/checks/SampleServiceCheck.java
 PRE-CREATION 
  
ambari-server/src/test/java/org/apache/ambari/server/stack/ServiceModuleTest.java
 304fd5c 
  
ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerExtensionTest.java
 659ae12 

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


Testing
-------

mvn clean test 
-Dtest=PreUpgradeCheckResourceProviderTest,PreUpgradeCheckServiceTest,StackManagerExtensionTest,ServiceModuleTest,StackManagerCommonServicesTest,StackManagerTest,StackManagerMiscTest


Thanks,

Tim Thorpe

Reply via email to