----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27486/#review59520 -----------------------------------------------------------
Ship it! Ship It! - Alexandr Antonenko On Ноя. 2, 2014, 12:32 п.п., Andrii Tkach wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27486/ > ----------------------------------------------------------- > > (Updated Ноя. 2, 2014, 12:32 п.п.) > > > Review request for Ambari, Aleksandr Kovalenko, Alexandr Antonenko, and Oleg > Nechiporenko. > > > Bugs: AMBARI-8105 > https://issues.apache.org/jira/browse/AMBARI-8105 > > > Repository: ambari > > > Description > ------- > > Check /api/v1/services/AMBARI/components/AMBARI_SERVER. > If os_type is "redhat5" or "sles11", do not show "New MySQL Database" option > in Customize Services page for the Install Wizard / Add Service Wizard. > In this scenario, the default selection should be "Existing MySQL database". > Note that in the Dashboard, if Hive is already configured with "New MySQL > Database", we still need to render it properly in Services > Hive. Configs, > even if the OS is redhat5 or sles11. This can happen if the user installed > the cluster prior to Ambari 1.7.0 and used "New MySQL Database". It's ok to > always show "New MySQL Database" on this page regardless of the OS. > Add "App.supports.alwaysEnableManagedMySQLForHive", which should be set to > false. This is to allow the use case in which the user is using SLES 11 or > RedHat 5 and wants to install Ambari-Managed, "New" MySQL Server. > > > Diffs > ----- > > ambari-web/app/assets/test/tests.js 775822b > ambari-web/app/config.js 2f49230 > ambari-web/app/controllers/installer.js f0a2146 > ambari-web/app/controllers/main.js 7a54ced > ambari-web/app/data/HDP2/site_properties.js f454b9b > ambari-web/app/models/service_config.js cb0e8f0 > ambari-web/app/utils/ajax/ajax.js b0a91aa > ambari-web/app/utils/config.js 61a4e13 > ambari-web/app/views/wizard/controls_view.js ae9f721 > ambari-web/test/controllers/installer_test.js 1a2dc7f > ambari-web/test/models/service_config_test.js a015d82 > ambari-web/test/utils/config_test.js 4c68e8b > > Diff: https://reviews.apache.org/r/27486/diff/ > > > Testing > ------- > > 4844 tests complete (7 seconds) > 60 tests pending > > > Thanks, > > Andrii Tkach > >
