----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35460/#review88499 -----------------------------------------------------------
Ship it! Ship It! - Yusaku Sako On June 15, 2015, 1:15 p.m., Robert Levas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35460/ > ----------------------------------------------------------- > > (Updated June 15, 2015, 1:15 p.m.) > > > Review request for Ambari, Aleksandr Kovalenko, Andrii Tkach, Jaimin Jetly, > and Yusaku Sako. > > > Bugs: AMBARI-11925 > https://issues.apache.org/jira/browse/AMBARI-11925 > > > Repository: ambari > > > Description > ------- > > #Steps to reproduce: > 1. Install Kerberos on the Cluster > 2. Go to Kerberos Configs > 3. Change any value in the configuration > 4. Click Save and Confirm > > #Expect Result: > The KDC Type Value should remain unchanged. > The user should be able to make further modifications to the configs after > this points. > > #Actual Result: > The Kerberos KDC Type Value is changed to the descriptive value and thus is > unexpected. Since this is a required field, but cannot be edited, the user > can no longer save any changes after this point > > > Diffs > ----- > > ambari-web/app/controllers/main/service/info/configs.js a21baa1 > ambari-web/app/mixins/common/configs/configs_saver.js d1891cd > ambari-web/app/models/configs/objects/service_config.js badd1dc > > Diff: https://reviews.apache.org/r/35460/diff/ > > > Testing > ------- > > Manually tested to see that the kdc-type value remained intact after updating > the Kerberos configuration > Local unit tests passed > > #Jenkins test results: PENDING > > > Thanks, > > Robert Levas > >
