-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9587/
-----------------------------------------------------------
(Updated March 11, 2013, 7:46 a.m.)
Review request for cloudstack and Rajesh Battala.
Changes
-------
updated the patch as per Alex suggestions and squashed the commits in one
Description
-------
This patch implements the SNMP Alerts support in CloudStack. With this feature
one can configure log4j-cloud.xml to get the SNMP alerts. This feature uses the
snmp4j library with Apache License. This feature is implemented by extending
log4j Appeneder. Multiple SNMP Managers can be specified. To configure multiple
SNMP Managers one needs to modify following portion in log4j-cloud.xml
<appender name="SNMP"
class="org.apache.cloudstack.alert.snmp.SnmpTrapAppender">
<param name="Threshold" value="WARN"/>
<param name="SnmpManagerIpAddresses" value=""/>
<param name="SnmpManagerPorts" value=""/>
<param name="SnmpManagerCommunities" value=""/>
<layout
class="org.apache.cloudstack.alert.snmp.SnmpEnhancedPatternLayout">
<param name="PairDelimeter" value="//"/>
<param name="KeyValueDelimeter" value="::"/>
</layout>
</appender>
To specify multiple SnmpManagerIpAddresses one has to give as follows with
each SNMP Manager separated by ,
<appender name="SNMP"
class="org.apache.cloudstack.alert.snmp.SnmpTrapAppender">
<param name="Threshold" value="WARN"/>
<param name="SnmpManagerIpAddresses" value="10.1.1.1,10.1.1.2"/>
<param name="SnmpManagerPorts" value="162,162"/>
<param name="SnmpManagerCommunities" value="public,public"/>
<layout
class="org.apache.cloudstack.alert.snmp.SnmpEnhancedPatternLayout">
<param name="PairDelimeter" value="//"/>
<param name="KeyValueDelimeter" value="::"/>
</layout>
</appender>
This patch also resolves the bug
https://issues.apache.org/jira/browse/CLOUDSTACK-960
This addresses bug https://issues.apache.org/jira/browse/CLOUDSTACK-680.
Diffs (updated)
-----
client/pom.xml cda6ab8
client/tomcatconf/log4j-cloud.xml.in 0866693
core/src/com/cloud/alert/AlertManager.java a24e18c
plugins/alert-handlers/snmp-alerts/pom.xml PRE-CREATION
plugins/alert-handlers/snmp-alerts/src/org/apache/cloudstack/alert/snmp/CsSnmpConstants.java
PRE-CREATION
plugins/alert-handlers/snmp-alerts/src/org/apache/cloudstack/alert/snmp/SnmpEnhancedPatternLayout.java
PRE-CREATION
plugins/alert-handlers/snmp-alerts/src/org/apache/cloudstack/alert/snmp/SnmpHelper.java
PRE-CREATION
plugins/alert-handlers/snmp-alerts/src/org/apache/cloudstack/alert/snmp/SnmpTrapAppender.java
PRE-CREATION
plugins/alert-handlers/snmp-alerts/src/org/apache/cloudstack/alert/snmp/SnmpTrapInfo.java
PRE-CREATION
plugins/alert-handlers/snmp-alerts/test/org/apache/cloudstack/alert/snmp/SnmpEnhancedPatternLayoutTest.java
PRE-CREATION
plugins/alert-handlers/snmp-alerts/test/org/apache/cloudstack/alert/snmp/SnmpTrapAppenderTest.java
PRE-CREATION
plugins/pom.xml 88f617b
server/src/com/cloud/alert/AlertManagerImpl.java f8a8fd8
usage/src/com/cloud/usage/UsageAlertManagerImpl.java a0765b2
Diff: https://reviews.apache.org/r/9587/diff/
Testing
-------
9 unit tests have been written to test the functionality. Manually tested alert
generation for few alerts.
Thanks,
Anshul Gangwar
