cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Murali Reddy" <muralimmre...@gmail.com>
Subject Re: Review Request 18677: CLOUDSTACK-3272
Date Wed, 26 Mar 2014 06:36:33 GMT

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


Thanks for the patch. Could you please ensure patch can be applied to master, its fails to
apply.

- Murali Reddy


On March 24, 2014, 12:46 p.m., Sonal Ojha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/18677/
> -----------------------------------------------------------
> 
> (Updated March 24, 2014, 12:46 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
> 
> 
> Thanks,
> 
> Sonal Ojha
> 
>


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