cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sonal Ojha" <sonal.o...@sungard.com>
Subject Re: Review Request 18677: CLOUDSTACK-3272
Date Wed, 14 May 2014 10:43:19 GMT


> On May 14, 2014, 8:41 a.m., 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


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18677/#review42951
-----------------------------------------------------------


On May 6, 2014, 6:53 a.m., Sonal Ojha wrote:
> 
> -----------------------------------------------------------
> 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
> -----
> 
>   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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message