-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30009/
-----------------------------------------------------------
(Updated Jan. 20, 2015, 9:47 p.m.)
Review request for Ambari, John Speidel, Robert Nettleton, and Tom Beerbower.
Changes
-------
Fixed bug that caused host check to hang when creating a new cluster.
Bugs: AMBARI-9149
https://issues.apache.org/jira/browse/AMBARI-9149
Repository: ambari
Description
-------
Intercept call to execute SERVICE_CHECK for KERBEROS service and (if necessary)
create and distribute test user keytab.
It will be hard coded that the test user information is taken from the
"smokeuser" identity in the relevant clusters Kerberos descriptor.
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
ae57d1f
ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelper.java
a425e95
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestStageContainer.java
12b7f71
ambari-server/src/main/resources/common-services/KERBEROS/1.10.3-10/kerberos.json
PRE-CREATION
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelperTest.java
5e933d2
ambari-server/src/test/java/org/apache/ambari/server/controller/BackgroundCustomCommandExecutionTest.java
2b00f40
ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java
0b58fc0
Diff: https://reviews.apache.org/r/30009/diff/
Testing
-------
Manually tested in test cluster
Updated unit tests
# Jenkins Test Results
Running
org.apache.ambari.server.controller.AmbariCustomCommandExecutionHelperTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 22.301 sec
Running org.apache.ambari.server.controller.KerberosHelperTest
Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.751 sec
Results for ambari-server tests
Tests run: 2551, Failures: 0, Errors: 0, Skipped: 15
Thanks,
Robert Levas
