Yes, please check the code. It was reverted in 4.4.
On Wed, May 14, 2014 at 12:43 PM, Sonal Ojha <sonal.o...@sungard.com> wrote: > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18677/ > > On May 14th, 2014, 8:41 a.m. UTC, *daan Hoogland* wrote: > > Sonal, this one didn't make it to 4.4. Can you check it? It would really be > nice to have it in. > > I dont see these changes made into 4.4. > > > - Sonal > > On May 6th, 2014, 6:53 a.m. UTC, Sonal Ojha wrote: > Review request for cloudstack, Chiradeep Vittal, daan Hoogland, and > Murali Reddy. > By Sonal Ojha. > > *Updated May 6, 2014, 6:53 a.m.* > *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. > > Testing > > Successfully tested publishing / not publishing of various events based on > the global parameter publish.*.events. > > Diffs > > - 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) > > View Diff <https://reviews.apache.org/r/18677/diff/> > 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> > > -- Daan
