ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Hurley" <jhur...@hortonworks.com>
Subject Review Request 28592: Alerts: Upgrade Should Remove Nagios If Installed
Date Tue, 02 Dec 2014 01:37:54 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28592/
-----------------------------------------------------------

Review request for Ambari, Nate Cole and Sid Wagle.


Bugs: AMBARI-8502
    https://issues.apache.org/jira/browse/AMBARI-8502


Repository: ambari


Description
-------

On upgrading to Ambari 2.0.0, Nagios should be removed from all clusters defined in Ambari.


1) Install Ambari 1.7.0 with a cluster that has Nagios
2) Perform Upgrade to Ambari 2.0.0
3) Verify that Nagios is removed and that the new alerts framework has taken over correctly.


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertResourceProvider.java
57840d4 
  ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog200.java 004d213

  ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog150Test.java
d053d83 
  ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog170Test.java
d456101 
  ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog200Test.java
1545a03 
  ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalogHelper.java PRE-CREATION


Diff: https://reviews.apache.org/r/28592/diff/


Testing
-------

Installed a 1.7.0 cluster with Nagios and then performed the database upgrade. Started Ambari
after the upgrade and verified that Nagio has been removed from the host, and as a service
from the cluster.

New tests added to verify removal of Nagios during 2.0.0 DML update.


Thanks,

Jonathan Hurley


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message