----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60047/#review177733 -----------------------------------------------------------
Ship it! Ship It! - Sebastian Toader On June 13, 2017, 2:42 p.m., Attila Magyar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60047/ > ----------------------------------------------------------- > > (Updated June 13, 2017, 2:42 p.m.) > > > Review request for Ambari, Balázs Bence Sári, Laszlo Puskas, Robert Levas, > and Sebastian Toader. > > > Bugs: AMBARI-21236 > https://issues.apache.org/jira/browse/AMBARI-21236 > > > Repository: ambari > > > Description > ------- > > Kerberos principal placeholders like > sasl:${principals/YARN/RESOURCEMANAGER/resource_manager_rm|principalPrimary()}:rwcda > are not getting replaced if the cluster is created with blueprints > (kerberos=enabled). > > I moved the logic that populates the replacemap with kerberos principals to a > different place. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java > 802c93d > > > Diff: https://reviews.apache.org/r/60047/diff/1/ > > > Testing > ------- > > - created a cluster with blueprint (kerberos=enabled) > - checked yarn-site.xml and zookeeper ACLs > > existing tests: Tests run: 4982, Failures: 0, Errors: 0, Skipped: 39 > > > Thanks, > > Attila Magyar > >
