-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35237/
-----------------------------------------------------------
(Updated June 9, 2015, 7:06 a.m.)
Review request for Ambari, Arsen Babych, Emil Anca, Robert Nettleton, and Tom
Beerbower.
Bugs: AMBARI-11795
https://issues.apache.org/jira/browse/AMBARI-11795
Repository: ambari
Description
-------
Service configs are not updated with enabling Kerberos unless the service has
identities to process. This is an issue in the case a client component requires
a configuration change when Kerberos is enabled and the Service has no Kerberos
identities.
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java
8a5d4fd
ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/AbstractPrepareKerberosServerAction.java
b3b6b48
ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/PrepareDisableKerberosServerAction.java
68fcca0
ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/PrepareEnableKerberosServerAction.java
2295eeb
ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/PrepareKerberosIdentitiesServerAction.java
e39d868
ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java
f8ba840
Diff: https://reviews.apache.org/r/35237/diff/
Testing (updated)
-------
Manually tested
Updated unit tests
Local unit tests ran
#Jenkins test results:
Running org.apache.ambari.server.controller.KerberosHelperTest
Tests run: 43, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.416 sec
Full ambari-server test suite
Tests run: 3064, Failures: 0, Errors: 0, Skipped: 28
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 01:33 h
[INFO] Finished at: 2015-06-09T04:06:55+00:00
[INFO] Final Memory: 47M/498M
[INFO] ------------------------------------------------------------------------
Thanks,
Robert Levas
