-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18677/
-----------------------------------------------------------
(Updated April 15, 2014, 12:37 p.m.)
Review request for cloudstack, Chiradeep Vittal and Murali Reddy.
Changes
-------
Removed the earlier not working patch files and added a new patch file
(0001-CLOUDSTACK-3272-Latest-changes-to-add-global-configu.patch) which works
on master with commit 55f53e13e. I could successfully apply the latest patch
attached on master and test the alert/action/resource/usage events published on
rabbitmq message bus. Earlier patch didnt work as it was built on the older
version and had some changes which werent required. Kindly take a look and
revert back.
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 (updated)
----------------
Working patch for master (latest git commit 55f53e13e)
https://reviews.apache.org/media/uploaded/files/2014/04/15/a6b65b2b-32ea-41d1-97e5-b9958d8b8841__0001-CLOUDSTACK-3272-Latest-changes-to-add-global-configu.patch
Thanks,
Sonal Ojha
