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




ambari-server/src/test/java/org/apache/ambari/server/sample/checks/SampleServiceCheck.java
 (line 1)
<https://reviews.apache.org/r/52223/#comment218575>

    Missing license header


- Robert Levas


On Sept. 23, 2016, 3:59 p.m., Tim Thorpe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52223/
> -----------------------------------------------------------
> 
> (Updated Sept. 23, 2016, 3:59 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Di Li, Jayush Luniya, and 
> Juanjo  Marron.
> 
> 
> 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