-----------------------------------------------------------
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

Reply via email to