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

(Updated Feb. 27, 2017, 2:58 p.m.)


Review request for Ambari, Attila Doroszlai, Balázs Bence Sári, Robert Levas, 
and Sebastian Toader.


Changes
-------

inverted conditional


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


Repository: ambari


Description
-------

Spark2 and Spark1 shares the same principal name. This casuses a problem when 
adding Spark2 to a kerberized cluster with Spark1. Ambari doesn't publish the 
keytab to Spark2 because it thinks it already exists.

I extended the existing conditional to take the existence of the destination 
keytab file into consideration.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/CreateKeytabFilesServerAction.java
 b968383 

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


Testing
-------

- Created a cluster with Spark
- Enabled Kerberos
- Added Spark2
- Checked if Spark2 started successfully and the exitence of 
/etc/security/keytabs/spark2.headless.keytab


Existing tests ran without failure.


Thanks,

Attila Magyar

Reply via email to