----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28780/#review64139 -----------------------------------------------------------
ambari-web/app/assets/data/wizard/kerberos/kerberize_cluster.json <https://reviews.apache.org/r/28780/#comment106583> JSON files should not have license header (should be ignored by rat check exclude rule). ambari-web/app/templates/main/admin/kerberos/step5.hbs <https://reviews.apache.org/r/28780/#comment106585> Why "highAvailablity" here? - Yusaku Sako On Dec. 6, 2014, 1:22 a.m., Jaimin Jetly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28780/ > ----------------------------------------------------------- > > (Updated Dec. 6, 2014, 1:22 a.m.) > > > Review request for Ambari, Srimanth Gunturi and Yusaku Sako. > > > Bugs: AMBARI-8572 > https://issues.apache.org/jira/browse/AMBARI-8572 > > > Repository: ambari > > > Description > ------- > > Apart from implementing "Kerberize Cluster" page and "Test services" page, > this patch also removes KDC service component from Kerberos metainfo.xml. > > > Diffs > ----- > > > ambari-server/src/main/resources/stacks/HDP/2.2/services/KERBEROS/metainfo.xml > 8751892 > ambari-web/app/assets/data/wizard/kerberos/kerberize_cluster.json > PRE-CREATION > ambari-web/app/controllers/main/admin/kerberos/step2_controller.js f674f4a > ambari-web/app/controllers/main/admin/kerberos/step3_controller.js 3a4fdbd > ambari-web/app/controllers/main/admin/kerberos/step5_controller.js b15c8cf > ambari-web/app/controllers/main/admin/kerberos/step6_controller.js 800e0c8 > ambari-web/app/data/HDP2/site_properties.js 7971b6a > ambari-web/app/messages.js 7396499 > ambari-web/app/mixins/wizard/wizardProgressPageController.js 63a5fc7 > ambari-web/app/mixins/wizard/wizardProgressPageView.js bbb6420 > ambari-web/app/models/cluster_states.js 9571ea4 > ambari-web/app/routes/add_kerberos_routes.js 4afaedb > ambari-web/app/templates/main/admin/highAvailability/progress.hbs 5f95387 > ambari-web/app/templates/main/admin/kerberos/step5.hbs eadc752 > ambari-web/app/utils/ajax/ajax.js 3d9206e > ambari-web/app/views/main/admin/kerberos/step5_view.js bf46584 > ambari-web/app/views/main/admin/kerberos/wizard_view.js d82786a > > Diff: https://reviews.apache.org/r/28780/diff/ > > > Testing > ------- > > tested kerberos wizard on a live cluster > > > Thanks, > > Jaimin Jetly > >
