----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39688/#review104280 -----------------------------------------------------------
Ship it! Ship It! - Nate Cole On Oct. 27, 2015, 11:03 a.m., Robert Levas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39688/ > ----------------------------------------------------------- > > (Updated Oct. 27, 2015, 11:03 a.m.) > > > Review request for Ambari, Jonathan Hurley, Nate Cole, Robert Nettleton, and > Sandor Magyari. > > > Bugs: AMBARI-13551 > https://issues.apache.org/jira/browse/AMBARI-13551 > > > Repository: ambari > > > Description > ------- > > When adding components to a Kerberized cluster, the set of hosts to create > principals for should be limited to only the relevant set. This is because > if a component for an existing service is added to an existing host, > principals and keytab files for the existing components will get > unnecessarily updated. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java > 152016a > > ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelper.java > f87fb04 > > ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java > bf8c519 > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/AbstractPrepareKerberosServerAction.java > 479a054 > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/KerberosServerAction.java > 901a80f > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/PrepareDisableKerberosServerAction.java > 4315f78 > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/PrepareKerberosIdentitiesServerAction.java > 5f067ec > > ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java > d3f54e5 > > Diff: https://reviews.apache.org/r/39688/diff/ > > > Testing > ------- > > Manually tested enabling Keberos, and adding hosts and services to a > Kerberized cluster. > > # Jenkins test results: PENDING > > > Thanks, > > Robert Levas > >
