-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42369/
-----------------------------------------------------------
Review request for Ambari, Robert Nettleton, Sandor Magyari, and Srimanth
Gunturi.
Bugs: AMBARI-14698
https://issues.apache.org/jira/browse/AMBARI-14698
Repository: ambari
Description
-------
Kerberos automation logic should use the stack advisor when determining
configuration updates. This will ensure that property updates are valid given
the cluster's configuration and whether the cluster was created using the API,
UI, or BluePrints.
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/StackAdvisorBlueprintProcessor.java
337ad06
ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelper.java
b6686b4
ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java
6f0fa27
ambari-server/src/main/java/org/apache/ambari/server/serveraction/ServerActionExecutor.java
4c241d1
ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/AbstractPrepareKerberosServerAction.java
359e651
ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/PrepareDisableKerberosServerAction.java
8ab04ff
ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/PrepareEnableKerberosServerAction.java
84248dc
ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/PrepareKerberosIdentitiesServerAction.java
fba3eea
ambari-server/src/main/java/org/apache/ambari/server/topology/ClusterConfigurationRequest.java
1e89272
ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py
ca7d2dd
ambari-server/src/test/java/org/apache/ambari/server/api/services/stackadvisor/StackAdvisorBlueprintProcessorTest.java
d5531a6
ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java
8861549
ambari-server/src/test/java/org/apache/ambari/server/topology/ClusterConfigurationRequestTest.java
d1886b9
Diff: https://reviews.apache.org/r/42369/diff/
Testing
-------
Manualled tested:
- creating a cluster via the UI then enabling Kerberos and disabling Kerberos
- creating a cluster via Blueprints then enabling Kerberos and disabling
Kerberos
- creating a Kerberized cluster via Blueprints then disabling Kerberos
# Local test results: PASSED
# Jenkins test results: PENDING
Thanks,
Robert Levas