----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53642/#review155566 -----------------------------------------------------------
Ship it! Ship It! - Aleksandr Kovalenko On Ноя. 10, 2016, 11:36 д.п., Andrii Tkach wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53642/ > ----------------------------------------------------------- > > (Updated Ноя. 10, 2016, 11:36 д.п.) > > > Review request for Ambari and Aleksandr Kovalenko. > > > Bugs: AMBARI-18844 > https://issues.apache.org/jira/browse/AMBARI-18844 > > > Repository: ambari > > > Description > ------- > > HOST_ORDERED upgrades which are initiated from the APIs currently don't show > up as an upgrade "In Progress" in the web client. Although this upgrade type > is not going to be exposed as a first-class upgrade type for Ambari 2.5, the > web client must still behave correctly when encountering a HOST_ORDERED > upgrade already running. > The following criteria should be met: > The web client displays a banner that there is an upgrade in progress (or > holding or waiting for input, etc) just like it would for other upgrade types. > Destructive behavior such as running commands, adding/remove hosts/services, > changing configs, etc, should be prevented. > > > Diffs > ----- > > ambari-web/app/config.js a2028ad > ambari-web/app/controllers/main/admin/stack_and_upgrade_controller.js > 3c69b84 > ambari-web/app/messages.js e707576 > ambari-web/app/styles/application.less f24f355 > ambari-web/app/styles/stack_versions.less 9d4190a > ambari-web/app/templates/application.hbs 669daa7 > ambari-web/app/templates/main/admin/stack_upgrade/upgrade_version_box.hbs > cb91bd7 > > ambari-web/app/templates/main/admin/stack_upgrade/upgrade_version_column.hbs > b7477ca > ambari-web/app/views/main/admin/stack_upgrade/upgrade_version_box_view.js > ebcd423 > ambari-web/test/controllers/main/admin/stack_and_upgrade_controller_test.js > 21def14 > > ambari-web/test/views/main/admin/stack_upgrade/upgrade_version_box_view_test.js > c64e5af > > Diff: https://reviews.apache.org/r/53642/diff/ > > > Testing > ------- > > 30497 tests complete (29 seconds) > 151 tests pending > > > Thanks, > > Andrii Tkach > >
