Kindly take a look at the new patch file and let me know your comments. Thanks, Sonal
On Wed, Mar 26, 2014 at 10:31 PM, Sonal Ojha <sonal.o...@sungard.com> wrote: > > > > On March 26, 2014, 6:36 a.m., Murali Reddy wrote: > > > Thanks for the patch. Could you please ensure patch can be applied to > master, its fails to apply. > > Please try the latest one attached, let me know if you see any issue. > > > - Sonal > > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18677/#review38582 > ----------------------------------------------------------- > > > On March 26, 2014, 5 p.m., Sonal Ojha wrote: > > > > ----------------------------------------------------------- > > This is an automatically generated e-mail. To reply, visit: > > https://reviews.apache.org/r/18677/ > > ----------------------------------------------------------- > > > > (Updated March 26, 2014, 5 p.m.) > > > > > > Review request for cloudstack, Chiradeep Vittal and Murali Reddy. > > > > > > Bugs: CLOUDSTACK-3272 > > https://issues.apache.org/jira/browse/CLOUDSTACK-3272 > > > > > > Repository: cloudstack-git > > > > > > Description > > ------- > > > > At present, there are multiple event types that are generated by CS, > action events,usage events, resource state change events and alerts. > Current problem is > > all the events gets published on the event bus when event bus is > enabled. Intent is to introduce global setting config parameters to specify > which category of events to be published or not be published on the event > bus. > > > > > > Diffs > > ----- > > > > server/src/com/cloud/configuration/Config.java 2d55f46 > > server/src/com/cloud/event/ActionEventUtils.java 00122de > > server/src/com/cloud/test/DatabaseConfig.java 9370218 > > > > Diff: https://reviews.apache.org/r/18677/diff/ > > > > > > Testing > > ------- > > > > Successfully tested publishing / not publishing of various events based > on the global parameter publish.*.events. > > > > > > File Attachments > > ---------------- > > > > 0001-CLOUDSTACK-3272-Changes-to-allow-publishing-of-event.patch > > > https://reviews.apache.org/media/uploaded/files/2014/03/24/4deb6544-5515-4a99-9f1c-f927c88d3014__0001-CLOUDSTACK-3272-Changes-to-allow-publishing-of-event.patch > > Patch File for master > > > https://reviews.apache.org/media/uploaded/files/2014/03/26/6e14c1e6-9d10-4689-88a5-320473dc2806__0001-CLOUDSTACK-3272-Changes-to-allow-publishing-of-event.patch > > Patch File for master > > > https://reviews.apache.org/media/uploaded/files/2014/03/26/e412646f-9ab3-4376-9c08-e2c8b5ab25cb__0001-CLOUDSTACK-3272-Changes-to-allow-publishing-of-event.patch > > > > > > Thanks, > > > > Sonal Ojha > > > > > > -- *Thanks and Regards,* *Sonal Ojha ▪ Sr.Engineer - Product Development **▪ **Sungard Availability Services, India* 2nd Floor, Wing 4, Cluster D, MIDC Kharadi Knowledge Park, Pune - 411 014 ▪ Office: 267-234-9014 ▪ Mobile: +91 9922412645 ▪ sonal.o...@sungardas.com ▪ www.sungardas. <http://www.sungardas.com/>*in* *[image: Logo]* <http://www.sungardas.com/> *[image: cid:image019.png@01CF48EC.6617C7F0]* <http://blog.sungardas.com/> *[image: cid:image020.png@01CF48EC.6617C7F0]* <http://www.youtube.com/user/SunGardAS> *[image: cid:image021.png@01CF48EC.6617C7F0]*<https://plus.google.com/u/0/102459878242108588663/posts> *[image: cid:image022.png@01CF48EC.6617C7F0]*<https://www.facebook.com/SunGardAS> *[image: cid:image023.png@01CF48EC.6617C7F0]*<http://www.linkedin.com/company/sungard-availability-services> *[image: cid:image024.png@01CF48EC.6617C7F0]*<https://twitter.com/SunGardAS> *CONFIDENTIALITY:* This e-mail (including any attachments) may contain confidential, proprietary and privileged information, and unauthorized disclosure or use is prohibited. If you received this e-mail in error, please notify the sender and delete this e-mail from your system.
