Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-03-01 Thread Robert Levas


> On March 1, 2017, 2:04 p.m., Alejandro Fernandez wrote:
> > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/kerberos.json
> > Line 18 (original), 18 (patched)
> > 
> >
> > I thought Spark2 was supposed to have its own keytab.

Apparently Spark and Spark2 work better if they share the same Kerberos 
identity. Since Ambari does not handle the same principal name in multiple 
keytab files, both services need to use the same keytab file - which is 
preferred since Ambari has to do less work.


- Robert


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


On March 1, 2017, 6:18 a.m., Attila Magyar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56980/
> ---
> 
> (Updated March 1, 2017, 6:18 a.m.)
> 
> 
> Review request for Ambari, Attila Doroszlai, Balázs Bence Sári, Robert Levas, 
> and Sebastian Toader.
> 
> 
> 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.
> 
> 
> Diffs
> -
> 
>   ambari-server/src/main/resources/common-services/SPARK2/2.0.0/kerberos.json 
> cf7eeba 
>   
> ambari-server/src/main/resources/stacks/HDP/2.6/services/SPARK2/kerberos.json 
> 01ee87e 
> 
> 
> Diff: https://reviews.apache.org/r/56980/diff/4/
> 
> 
> Testing
> ---
> 
> - Created a cluster with Spark
> - Enabled Kerberos
> - Added Spark2
> - Checked if Spark2 started successfully 
> - removed spark
> - restarted spark2 successfully, checked the exitance of the spark keytab
> 
> Existing tests ran without failure.
> 
> 
> Thanks,
> 
> Attila Magyar
> 
>



Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-03-01 Thread Robert Levas

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


Ship it!




Ship It!

- Robert Levas


On March 1, 2017, 6:18 a.m., Attila Magyar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56980/
> ---
> 
> (Updated March 1, 2017, 6:18 a.m.)
> 
> 
> Review request for Ambari, Attila Doroszlai, Balázs Bence Sári, Robert Levas, 
> and Sebastian Toader.
> 
> 
> 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.
> 
> 
> Diffs
> -
> 
>   ambari-server/src/main/resources/common-services/SPARK2/2.0.0/kerberos.json 
> cf7eeba 
>   
> ambari-server/src/main/resources/stacks/HDP/2.6/services/SPARK2/kerberos.json 
> 01ee87e 
> 
> 
> Diff: https://reviews.apache.org/r/56980/diff/4/
> 
> 
> Testing
> ---
> 
> - Created a cluster with Spark
> - Enabled Kerberos
> - Added Spark2
> - Checked if Spark2 started successfully 
> - removed spark
> - restarted spark2 successfully, checked the exitance of the spark keytab
> 
> Existing tests ran without failure.
> 
> 
> Thanks,
> 
> Attila Magyar
> 
>



Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-03-01 Thread Sebastian Toader

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


Ship it!




Ship It!

- Sebastian Toader


On March 1, 2017, 12:18 p.m., Attila Magyar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56980/
> ---
> 
> (Updated March 1, 2017, 12:18 p.m.)
> 
> 
> Review request for Ambari, Attila Doroszlai, Balázs Bence Sári, Robert Levas, 
> and Sebastian Toader.
> 
> 
> 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.
> 
> 
> Diffs
> -
> 
>   ambari-server/src/main/resources/common-services/SPARK2/2.0.0/kerberos.json 
> cf7eeba 
>   
> ambari-server/src/main/resources/stacks/HDP/2.6/services/SPARK2/kerberos.json 
> 01ee87e 
> 
> 
> Diff: https://reviews.apache.org/r/56980/diff/4/
> 
> 
> Testing
> ---
> 
> - Created a cluster with Spark
> - Enabled Kerberos
> - Added Spark2
> - Checked if Spark2 started successfully 
> - removed spark
> - restarted spark2 successfully, checked the exitance of the spark keytab
> 
> Existing tests ran without failure.
> 
> 
> Thanks,
> 
> Attila Magyar
> 
>



Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-03-01 Thread Attila Magyar

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

(Updated March 1, 2017, 11:18 a.m.)


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


Changes
---

dropping the previous approach, let's use the same keytab file in both cases


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


Repository: ambari


Description (updated)
---

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.


Diffs (updated)
-

  ambari-server/src/main/resources/common-services/SPARK2/2.0.0/kerberos.json 
cf7eeba 
  ambari-server/src/main/resources/stacks/HDP/2.6/services/SPARK2/kerberos.json 
01ee87e 


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

Changes: https://reviews.apache.org/r/56980/diff/3-4/


Testing (updated)
---

- Created a cluster with Spark
- Enabled Kerberos
- Added Spark2
- Checked if Spark2 started successfully 
- removed spark
- restarted spark2 successfully, checked the exitance of the spark keytab

Existing tests ran without failure.


Thanks,

Attila Magyar



Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-02-28 Thread Eugene Chekanskiy

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


Ship it!




Ship It!

- Eugene Chekanskiy


On Лют. 27, 2017, 2:58 після полудня, Attila Magyar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56980/
> ---
> 
> (Updated Лют. 27, 2017, 2:58 після полудня)
> 
> 
> Review request for Ambari, Attila Doroszlai, Balázs Bence Sári, Robert Levas, 
> and Sebastian Toader.
> 
> 
> 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
> -
> 
>   
> 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
> 
>



Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-02-27 Thread Balázs Bence Sári


> On Feb. 27, 2017, 2:39 p.m., Balázs Bence Sári wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/CreateKeytabFilesServerAction.java,
> >  line 232
> > 
> >
> > What does the comment mean?
> 
> Attila Magyar wrote:
> We don't cache keytab files that belong to servie principals. The error 
> should be added only if the principal is a non service principal (cache 
> should have been existed) but there is no cached keytab file

Now that the condition has been inverted it is in line with the comment.


- Balázs Bence


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


On Feb. 27, 2017, 2:58 p.m., Attila Magyar wrote:
> 
> ---
> 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.
> 
> 
> 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
> -
> 
>   
> 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
> 
>



Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-02-27 Thread Attila Magyar


> On Feb. 27, 2017, 2:39 p.m., Balázs Bence Sári wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/CreateKeytabFilesServerAction.java,
> >  line 232
> > 
> >
> > What does the comment mean?

We don't cache keytab files that belong to servie principals. The error should 
be added only if the principal is a non service principal (cache should have 
been existed) but there is no cached keytab file


- Attila


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


On Feb. 27, 2017, 2:58 p.m., Attila Magyar wrote:
> 
> ---
> 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.
> 
> 
> 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
> -
> 
>   
> 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
> 
>



Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-02-27 Thread Attila Magyar

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



Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-02-27 Thread Balázs Bence Sári

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




ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/CreateKeytabFilesServerAction.java
 (line 232)


What does the comment mean?


- Balázs Bence Sári


On Feb. 27, 2017, 2:14 p.m., Attila Magyar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56980/
> ---
> 
> (Updated Feb. 27, 2017, 2:14 p.m.)
> 
> 
> Review request for Ambari, Attila Doroszlai, Balázs Bence Sári, Robert Levas, 
> and Sebastian Toader.
> 
> 
> 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
> -
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/CreateKeytabFilesServerAction.java
>  5d6ccc0 
> 
> 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
> 
>



Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-02-27 Thread Attila Magyar

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

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


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


Changes
---

service principals are not cached, ignore if cachedKeytabPath is null


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

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



Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-02-23 Thread Sebastian Toader

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


Ship it!




Ship It!

- Sebastian Toader


On Feb. 23, 2017, 1:11 p.m., Attila Magyar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56980/
> ---
> 
> (Updated Feb. 23, 2017, 1:11 p.m.)
> 
> 
> Review request for Ambari, Attila Doroszlai, Balázs Bence Sári, Robert Levas, 
> and Sebastian Toader.
> 
> 
> 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
> -
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/CreateKeytabFilesServerAction.java
>  5d6ccc0 
> 
> 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
> 
>



Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-02-23 Thread Robert Levas

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


Ship it!




Ship It!

- Robert Levas


On Feb. 23, 2017, 7:11 a.m., Attila Magyar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56980/
> ---
> 
> (Updated Feb. 23, 2017, 7:11 a.m.)
> 
> 
> Review request for Ambari, Attila Doroszlai, Balázs Bence Sári, Robert Levas, 
> and Sebastian Toader.
> 
> 
> 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
> -
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/CreateKeytabFilesServerAction.java
>  5d6ccc0 
> 
> 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
> 
>



Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-02-23 Thread Balázs Bence Sári

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


Ship it!




Ship It!

- Balázs Bence Sári


On Feb. 23, 2017, 12:11 p.m., Attila Magyar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56980/
> ---
> 
> (Updated Feb. 23, 2017, 12:11 p.m.)
> 
> 
> Review request for Ambari, Attila Doroszlai, Balázs Bence Sári, Robert Levas, 
> and Sebastian Toader.
> 
> 
> 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
> -
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/CreateKeytabFilesServerAction.java
>  5d6ccc0 
> 
> 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
> 
>



Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-02-23 Thread Attila Magyar

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

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


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
-

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

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