----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28259/ -----------------------------------------------------------
(Updated Nov. 20, 2014, 12:22 p.m.) Review request for Ambari, Srimanth Gunturi and Yusaku Sako. Bugs: AMBARI-8388 https://issues.apache.org/jira/browse/AMBARI-8388 Repository: ambari Description ------- Nagios is no longer available as a service in trunk. Ambari Web needs to remove calls to get legacy_alerts, as this will no longer work once the server code removes the provider for Nagios alerts. As part of this ticket, remove legacy_alerts from being requested and make sure that the general UI does not break. Diffs (updated) ----- ambari-web/app/assets/test/tests.js c831716 ambari-web/app/controllers.js c4385ce ambari-web/app/controllers/global/update_controller.js bfb625c ambari-web/app/controllers/main/alerts_controller.js b28f711 ambari-web/app/controllers/main/host.js dc966db ambari-web/app/messages.js 9943582 ambari-web/app/routes/main.js 361312b ambari-web/app/templates/main/dashboard/alert_notification_popup.hbs 8a37e23 ambari-web/app/templates/main/host/alerts_popup.hbs 247753a ambari-web/app/templates/main/service/info/summary.hbs 8ccc64e ambari-web/app/utils/ajax/ajax.js d4a8402 ambari-web/app/views/main/dashboard/widgets.js 29b7063 ambari-web/app/views/main/service/info/summary.js 99c049b ambari-web/test/controllers/main/alerts_controller_test.js aee782a ambari-web/test/views/main/service/info/summary_test.js f64ee18 Diff: https://reviews.apache.org/r/28259/diff/ Testing ------- tested Thanks, Xi Wang