----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27559/#review59700 -----------------------------------------------------------
ambari-web/app/controllers/main/admin/security/add/addSecurity_controller.js <https://reviews.apache.org/r/27559/#comment100995> We need an else clause here and it should return appropriate secure configs. - Yusaku Sako On Nov. 4, 2014, 1:03 a.m., Jaimin Jetly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27559/ > ----------------------------------------------------------- > > (Updated Nov. 4, 2014, 1:03 a.m.) > > > Review request for Ambari, Srimanth Gunturi and Yusaku Sako. > > > Bugs: AMBARI-8133 > https://issues.apache.org/jira/browse/AMBARI-8133 > > > Repository: ambari > > > Description > ------- > > Update categories for YARN configs on UI > > > Diffs > ----- > > > ambari-web/app/controllers/main/admin/security/add/addSecurity_controller.js > 6fd7bc2 > ambari-web/app/controllers/main/admin/security/add/step2.js dbe99dc > ambari-web/app/data/HDP2.2/site_properties.js 979d16a > ambari-web/app/data/HDP2.2/yarn_properties.js PRE-CREATION > ambari-web/app/data/HDP2/secure_configs.js 6ece4af > ambari-web/app/data/HDP2/secure_mapping.js 27af72f > ambari-web/app/data/HDP2/secure_properties.js bba5582 > ambari-web/app/data/HDP2/site_properties.js 242f02b > ambari-web/app/models/stack_service.js 0f817db > > ambari-web/test/controllers/main/admin/security/add/addSecurity_controller_test.js > bdbc146 > > Diff: https://reviews.apache.org/r/27559/diff/ > > > Testing > ------- > > tested e2e installer and security wizard. > > > Thanks, > > Jaimin Jetly > >
