----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21274/#review42592 -----------------------------------------------------------
Ship it! Ship It! - Srimanth Gunturi On May 9, 2014, 6:16 p.m., Jaimin Jetly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21274/ > ----------------------------------------------------------- > > (Updated May 9, 2014, 6:16 p.m.) > > > Review request for Ambari, Srimanth Gunturi and Yusaku Sako. > > > Bugs: AMBARI-5716 > https://issues.apache.org/jira/browse/AMBARI-5716 > > > Repository: ambari > > > Description > ------- > > There is a possibility that "Start all Services" command will fail on Disable > Security wizard if App.Service DS model was not populated on the load of > "Disable security page" (timing issue). We need to make sure that the Service > model has populated before the "Disable Security page" is rendered. > > > Diffs > ----- > > ambari-web/app/controllers/main/admin/security/disable.js 436b416 > > ambari-web/app/controllers/main/admin/security/security_progress_controller.js > 2c80943 > ambari-web/app/routes/main.js 9e00962 > > Diff: https://reviews.apache.org/r/21274/diff/ > > > Testing > ------- > > Tested disable security with the patch applied on ambari branch-1.6.0 > > > Thanks, > > Jaimin Jetly > >
