----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9870/#review18502 -----------------------------------------------------------
Ship it! Branch: refs/heads/master Commit: 6a7156ad9af6ca173c0b685d69d5724d91d0a401 Parents: 2c176ab Author: Anshul Gangwar <anshul.gang...@citrix.com> Authored: Wed Mar 6 14:11:47 2013 +0530 Committer: Sateesh Chodapuneedi <sate...@apache.org> Committed: Fri Mar 29 09:53:05 2013 +0530 - Sateesh Chodapuneedi On March 29, 2013, 9:50 a.m., Anshul Gangwar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9870/ > ----------------------------------------------------------- > > (Updated March 29, 2013, 9:50 a.m.) > > > Review request for cloudstack, Rajesh Battala, Murali Reddy, and Sateesh > Chodapuneedi. > > > Description > ------- > > This patch implements the Syslog Enhancements in CloudStack. With this > feature one can configure log4j-cloud.xml to get the Syslog messages for > alerts. This feature is implemented by extending log4j Appeneder. Multiple > Syslog Hosts can be specified. To configure multiple Syslog Hosts one needs > to modify following portion in log4j-cloud.xml > <appender name="ALERTSYSLOG" > class="org.apache.cloudstack.syslog.AlertsSyslogAppender"> > <param name="Threshold" value="WARN"/> > <param name="SyslogHosts" value=""/> > <param name="Facility" value="LOCAL6"/> > <layout class="org.apache.log4j.PatternLayout"> > <param name="ConversionPattern" value="%-5p [%c{3}] (%t:%x) %m%n"/> > </layout> > </appender> > > To specify multiple Syslog Hosts one has to give as follows in above code > fragment with each Syslog Hosts separated by , > > <appender name="ALERTSYSLOG" > class="org.apache.cloudstack.syslog.AlertsSyslogAppender"> > <param name="Threshold" value="WARN"/> > <param name="SyslogHosts" value="10.1.1.1,10.1.1.2"/> > <param name="Facility" value="LOCAL6"/> > <layout class="org.apache.log4j.PatternLayout"> > <param name="ConversionPattern" value="%-5p [%c{3}] (%t:%x) %m%n"/> > </layout> > </appender> > > > This addresses bug https://issues.apache.org/jira/browse/CLOUDSTACK-712. > > > Diffs > ----- > > client/pom.xml 23892e1 > client/tomcatconf/log4j-cloud.xml.in ce4079f > plugins/alert-handlers/syslog-alerts/pom.xml PRE-CREATION > > plugins/alert-handlers/syslog-alerts/src/org/apache/cloudstack/syslog/AlertsSyslogAppender.java > PRE-CREATION > > plugins/alert-handlers/syslog-alerts/test/org/apache/cloudstack/syslog/AlertsSyslogAppenderTest.java > PRE-CREATION > plugins/pom.xml 39d9907 > > Diff: https://reviews.apache.org/r/9870/diff/ > > > Testing > ------- > > Units tests for functionality are written and manual testing for syslog > message generation for few alerts > > > Thanks, > > Anshul Gangwar > >
