You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Girish Chaudhari <gi...@sungardas.com> on 2014/04/29 04:19:05 UTC

[Reminder]:- Re: Review Request 20007: CLOUDSTACK-2697 - Populated the clusterId with actual value instead of Null in the alert message logs

Can someone please have a look into this review request.

Thanks,
Girish

On Thu, Apr 3, 2014 at 10:19 PM, Girish Chaudhari <girish.chaudhari@sungard
.com> wrote:

>    This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20007/
>   Review request for cloudstack and prashant mishra.
> By Girish Chaudhari.
>  *Bugs: * CLOUDSTACK-2697<https://issues.apache.org/jira/browse/CLOUDSTACK-2697>
>  *Repository: * cloudstack-git
> Description
>
> CLOUDSTACK-2697 - cluster id in alert message is null {alertType:: 1 // dataCenterId:: 1 // podId:: 1 // clusterId:: null }
>
> The Cluster ID is getting populated as null in the logs for the alert messages. In the code base the ClusterId has been hardcoded as null for the log messages. I have modified to code to provide the actual cluster ID.
>
>   Testing
>
> Tested on the 4.3 platform.
>
>   Diffs
>
>    - server/src/com/cloud/alert/AlertManagerImpl.java (2a343d5)
>
> View Diff <https://reviews.apache.org/r/20007/diff/>
>