----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35996/#review89762 -----------------------------------------------------------
Ship it! Ship It! - Yusaku Sako On June 29, 2015, 3:45 a.m., Jaimin Jetly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35996/ > ----------------------------------------------------------- > > (Updated June 29, 2015, 3:45 a.m.) > > > Review request for Ambari, Richard Zang, Srimanth Gunturi, and Yusaku Sako. > > > Bugs: AMBARI-12197 > https://issues.apache.org/jira/browse/AMBARI-12197 > > > Repository: ambari > > > Description > ------- > > While adding a new service in kerberized cluster, we are using kerberos > descriptor persisted at cluster resource endpoint. This kerberos descriptor > has kerberos definition for all services present in the stack definition when > kerberos wizard was executed. > > When a kerberos supporting uninstalled service and non-existing in a stack of > already kerberized cluster gets added later in the stack via HDP upgrade, Add > service wizard triggered subsequently after upgrade to install that service > will not show kerberos identities or configuration of that service for any > user customizations. > > As part of this patch, new services properties are fetched from stack > descriptor instead of cluster descriptor. Also all the kerberos realted > configuration (identities and properties) of a service in the stack > descriptor that are not present in the cluster descriptor are shown for > customization and applied to the cluster. > > > Diffs > ----- > > ambari-web/app/controllers/main/admin/kerberos/step4_controller.js 6746f0a > ambari-web/app/controllers/main/service/add_controller.js a33ccaf > ambari-web/app/mixins/wizard/addSecurityConfigs.js 7e436ae > > Diff: https://reviews.apache.org/r/35996/diff/ > > > Testing > ------- > > Tested e2e. > Verified that all unit test for ambari-web passess. > > > Thanks, > > Jaimin Jetly > >
