----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37662/#review96015 -----------------------------------------------------------
Ship it! Ship It! - Yusaku Sako On Aug. 20, 2015, 11:46 p.m., Jaimin Jetly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37662/ > ----------------------------------------------------------- > > (Updated Aug. 20, 2015, 11:46 p.m.) > > > Review request for Ambari, Richard Zang, Srimanth Gunturi, and Yusaku Sako. > > > Bugs: AMBARI-12840 > https://issues.apache.org/jira/browse/AMBARI-12840 > > > Repository: ambari > > > Description > ------- > > _STR_: > - Enable Kerberos Wizard > - Goto Download CSV Step and Click the back button > - AMS Identity and configurations gets set to HBase due to similar name even > though they belong to different config types. > - Hitting undo reverts to actual values from kerberos.json which are the > expected ones. > > The issue was that filename was not being considered while restoring cached > values for a config > > > Diffs > ----- > > ambari-web/app/controllers/wizard.js f67fd7b > ambari-web/app/mixins/wizard/addSecurityConfigs.js 0dc2f40 > ambari-web/app/utils/config.js dce2d50 > > Diff: https://reviews.apache.org/r/37662/diff/ > > > Testing > ------- > > Tested the patch on live cluster to resolve the issue. > Verified that all ambari-web unit tests passes successfully. > 6705 tests complete (11 seconds) > 94 tests pending > > > Thanks, > > Jaimin Jetly > >
