----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30897/#review72035 -----------------------------------------------------------
Ship it! looks good - John Speidel On Feb. 11, 2015, 10:15 p.m., Robert Levas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30897/ > ----------------------------------------------------------- > > (Updated Feb. 11, 2015, 10:15 p.m.) > > > Review request for Ambari, John Speidel, Robert Nettleton, and Yusaku Sako. > > > Bugs: AMBARI-9580 > https://issues.apache.org/jira/browse/AMBARI-9580 > > > Repository: ambari > > > Description > ------- > > Currently, the `kdc_type` value is being set in the `krb5-conf` > configuration, it needs to be set in the `kerberos-env` since the data lies > outside the realm of the krb5.conf file. > > The solution includes altering how the Kerberos Wizard store the kcd_type > value and where the backend logic goes to retrieve it. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelper.java > ce319e6 > > ambari-server/src/main/resources/common-services/KERBEROS/1.10.3-10/configuration/kerberos-env.xml > 03b13d6 > > ambari-server/src/main/resources/common-services/KERBEROS/1.10.3-10/configuration/krb5-conf.xml > 38f6ab2 > > ambari-server/src/main/resources/stacks/HDP/2.2.GlusterFS/services/KERBEROS/configuration/krb5-conf.xml > 8e6b9ba > > ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java > ad6128c > ambari-server/src/test/python/stacks/2.2/KERBEROS/use_cases.py 0634a37 > > ambari-server/src/test/python/stacks/2.2/configs/journalnode-upgrade-hdfs-secure.json > 3511ae9 > ambari-server/src/test/python/stacks/2.2/configs/journalnode-upgrade.json > 3dc3d42 > ambari-web/app/assets/data/wizard/stack/hdp/version2.0.1/KERBEROS.json > e0f837b > ambari-web/app/data/HDP2/site_properties.js 2b2294c > > Diff: https://reviews.apache.org/r/30897/diff/ > > > Testing > ------- > > Manually tested in cluster > > #Jenkins test results: Pending > > > Thanks, > > Robert Levas > >
