-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52289/
-----------------------------------------------------------

(Updated Sept. 30, 2016, 3:44 p.m.)


Review request for Ambari, Jonathan Hurley, Nate Cole, Robert Nettleton, and 
Sandor Magyari.


Changes
-------

While re-testing this, I found that the principals being created have the wrong 
permissions if services are installed in the Ambari server host.  This patch 
fixes that issue as well as fixes the writing of the krb5JAASLogin.conf file. 

Please re-review since the change is rather significant.


Bugs: AMBARI-18463
    https://issues.apache.org/jira/browse/AMBARI-18463


Repository: ambari


Description
-------

When installing a secure cluster using Blueprints, Ambari's 
{{/etc/ambari-server/conf/krb5JAASLogin.conf}} is not updated to reflect the 
details of the Ambari Kerberos identity.

This was introduced by the patch for AMBARI-18406.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java
 a3c6fd4 
  
ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/ConfigureAmbariIdentitiesServerAction.java
 10647e8 
  
ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/FinalizeKerberosServerAction.java
 7205d09 
  
ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/KerberosOperationHandler.java
 02cbb57 
  
ambari-server/src/main/java/org/apache/ambari/server/utils/ShellCommandUtil.java
 39981ef 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/kerberos.json 3aad080 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java
 7e6a056 
  
ambari-server/src/test/java/org/apache/ambari/server/serveraction/kerberos/ConfigureAmbariIdentitiesServerActionTest.java
 PRE-CREATION 

Diff: https://reviews.apache.org/r/52289/diff/


Testing
-------

Manualy tested with UI and BP installs

# Local test results:
```
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 1:06:17.245s
[INFO] Finished at: Mon Sep 26 19:52:31 EDT 2016
[INFO] Final Memory: 67M/699M
[INFO] ------------------------------------------------------------------------
```

# Jenkins test results: PENDING


Thanks,

Robert Levas

Reply via email to