-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31755/
-----------------------------------------------------------
(Updated March 5, 2015, 8:32 p.m.)
Review request for Ambari, Jonathan Hurley and Nate Cole.
Bugs: AMBARI-9939
https://issues.apache.org/jira/browse/AMBARI-9939
Repository: ambari
Description
-------
Performing an RU will run Service Check groups after several major phases. The
Service Check group needs to include all services that can run a service check.
However, certain services like Pig still have service check scripts even though
they only have client components.
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeContext.java
f8752a2
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ServiceCheckGrouping.java
e40706c
ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerTest.java
f9e81af
ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java
4a733b3
ambari-server/src/test/resources/stacks/HDP/2.1.1/services/PIG/metainfo.xml
f310b70
ambari-server/src/test/resources/stacks/HDP/2.1.1/services/TEZ/metainfo.xml
PRE-CREATION
Diff: https://reviews.apache.org/r/31755/diff/
Testing (updated)
-------
Performed an RU request and verified that the Service Check group included one
more service, namely Pig.
Waiting for unit test results.
Local unit tests passed,
OK
----------------------------------------------------------------------
Total run:609
Total errors:0
Total failures:0
OK
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 32:36.712s
AMBARI-9939. RU - Service Check group to include all services with a
service_check script (alejandro)
[INFO] Finished at: Thu Mar 05 12:21:27 PST 2015
[INFO] Final Memory: 67M/834M
[INFO] ------------------------------------------------------------------------
Thanks,
Alejandro Fernandez
