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