----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31158/#review72961 -----------------------------------------------------------
Ship it! Ship It! - Robert Levas On Feb. 18, 2015, 9:42 a.m., Vitalyi Brodetskyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31158/ > ----------------------------------------------------------- > > (Updated Feb. 18, 2015, 9:42 a.m.) > > > Review request for Ambari, Dmitro Lisnichenko and Robert Levas. > > > Bugs: AMBARI-9693 > https://issues.apache.org/jira/browse/AMBARI-9693 > > > Repository: ambari > > > Description > ------- > > In Ambari 1.7.0, UI used to make appropriate config property changes when > kerberizing the cluster. > In 2.0.0, this is all driven by the Kerb API. > But during testing we've noticed that some properties that were previously > set were no longer set. For example, hbase.hbase.coprocessor.master.classes, > hbase.coprocessor.region.classes, etc. > See > https://github.com/apache/ambari/blob/branch-1.7.0/ambari-web/app/data/HDP2/secure_mapping.js#L554 > There could be a bunch of other parameters that need to be ported over from > the secure_mapping.js file in order for the cluster to function properly in a > secure manner. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelper.java > fa829a4 > ambari-server/src/main/java/org/apache/ambari/server/utils/StageUtils.java > f6d44d8 > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/kerberos.json > 9ddad69 > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/kerberos.json > af8f93b > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/kerberos.json > 42d0c1e > > ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java > 1c8af3f > > Diff: https://reviews.apache.org/r/31158/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Vitalyi Brodetskyi > >
