You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Murali Reddy <mu...@gmail.com> on 2013/12/03 12:45:24 UTC

Re: Review Request 15722: CLOUDSTACK-5022: NullPointerException when invalid zone is passed into UsageEventUtils (4.2.0)

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

Ship it!


commit 03ba659ae7263ca4fdb7c1c0eaccbecdf45594e8

- Murali Reddy


On Nov. 20, 2013, 7:54 p.m., David Grizzanti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15722/
> -----------------------------------------------------------
> 
> (Updated Nov. 20, 2013, 7:54 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-5022
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5022
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> CLOUDSTACK-5022: NullPointerException when invalid zone is passed into UsageEventUtils
> 
> Updated UsageEventUtils code to not blow up when an invalid zone ID is passed in. Now will still generate usage event but leaving out zone UUID.
> 
> UsageEventUtils has been moved on master, so a separate review will be submitted for that fix.
> 
> 
> Diffs
> -----
> 
>   server/src/com/cloud/event/UsageEventUtils.java 2e42229 
> 
> Diff: https://reviews.apache.org/r/15722/diff/
> 
> 
> Testing
> -------
> 
> Tested on 4.2.0
> 
> 
> Thanks,
> 
> David Grizzanti
> 
>