----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29838/#review67782 -----------------------------------------------------------
Ship it! Ship It! - Yusaku Sako On Jan. 13, 2015, 2:09 a.m., Jaimin Jetly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29838/ > ----------------------------------------------------------- > > (Updated Jan. 13, 2015, 2:09 a.m.) > > > Review request for Ambari, Srimanth Gunturi, Xi Wang, and Yusaku Sako. > > > Bugs: AMBARI-9099 > https://issues.apache.org/jira/browse/AMBARI-9099 > > > Repository: ambari > > > Description > ------- > > Provide UI to collect the LDAP URL and Principal container DN for Active > Directory integration. > > Collected information should be stored in the {{kerberos-env}} configuration > using the following properties: > > 1. ldap_url - The URL to the Active Directory LDAP Interface > 2. container_dn - The distinguished name (DN) of the container used store > service principals > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/KERBEROS/1.10.3-10/configuration/kerberos-env.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/KERBEROS/1.10.3-10/metainfo.xml > 64f42c1 > ambari-web/app/controllers/main/admin/kerberos/step2_controller.js 0eeee2c > ambari-web/app/data/HDP2/site_properties.js a02036e > > Diff: https://reviews.apache.org/r/29838/diff/ > > > Testing > ------- > > tested e2e that the new configuration kerberos-env is created and applied to > the cluster with the two mentioned properties in the description > > **ambari-web unit test result with the patch:** > > 5215 tests complete (7 seconds) > 60 tests pending > > > Thanks, > > Jaimin Jetly > >
