-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18677/
-----------------------------------------------------------
(Updated May 6, 2014, 6:53 a.m.)
Review request for cloudstack, Chiradeep Vittal, daan Hoogland, 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 (updated)
-----
engine/components-api/src/com/cloud/event/UsageEventUtils.java f1707bd
engine/components-api/src/com/cloud/network/NetworkStateListener.java 0a5e033
engine/orchestration/src/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
96dafe9
server/src/com/cloud/configuration/Config.java 89f47d4
server/src/com/cloud/event/ActionEventUtils.java 363bb93
server/src/com/cloud/event/AlertGenerator.java 5982eea
server/src/com/cloud/network/IpAddressManagerImpl.java 746221f
server/src/com/cloud/storage/StorageManagerImpl.java c27a457
server/src/com/cloud/storage/listener/SnapshotStateListener.java 45f0c5d
server/src/com/cloud/storage/listener/VolumeStateListener.java 1ab2755
server/src/com/cloud/test/DatabaseConfig.java 07e4348
server/src/com/cloud/vm/UserVmManagerImpl.java dbca21b
server/src/com/cloud/vm/UserVmStateListener.java 6631ca3
server/test/org/apache/cloudstack/affinity/AffinityApiUnitTest.java e7581d6
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
----------------
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
Works on commit 6c44c0661dae0e9de4ed53fad8951e72dbbd3335
https://reviews.apache.org/media/uploaded/files/2014/05/05/ea7c6ce6-7750-456a-96c0-41aa1aee9437__0002-CLOUDSTACK-3272-Latest-changes-to-add-global-configu.patch
Thanks,
Sonal Ojha
