You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by David Grizzanti <da...@sungard.com> on 2013/11/20 21:15:26 UTC
Review Request 15725: CLOUDSTACK-5022: NullPointerException when invalid
zone is passed into UsageEventUtils (master)
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15725/
-----------------------------------------------------------
Review request for cloudstack.
Bugs: CLOUDSTACK-5220
https://issues.apache.org/jira/browse/CLOUDSTACK-5220
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.
Diffs
-----
engine/components-api/src/com/cloud/event/UsageEventUtils.java a4be4dd
Diff: https://reviews.apache.org/r/15725/diff/
Testing
-------
built on master
Thanks,
David Grizzanti
Re: Review Request 15725: CLOUDSTACK-5022: NullPointerException when invalid
zone is passed into UsageEventUtils (master)
Posted by Murali Reddy <mu...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15725/#review29805
-----------------------------------------------------------
Ship it!
commit 22422b9ddd3ea1566f66d971307235569bffb003
- Murali Reddy
On Dec. 3, 2013, 4:50 p.m., David Grizzanti wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15725/
> -----------------------------------------------------------
>
> (Updated Dec. 3, 2013, 4:50 p.m.)
>
>
> Review request for cloudstack.
>
>
> Bugs: CLOUDSTACK-5220
> https://issues.apache.org/jira/browse/CLOUDSTACK-5220
>
>
> 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.
>
>
> Diffs
> -----
>
> engine/components-api/src/com/cloud/event/UsageEventUtils.java e71afb0
>
> Diff: https://reviews.apache.org/r/15725/diff/
>
>
> Testing
> -------
>
> built on master
>
>
> Thanks,
>
> David Grizzanti
>
>
Re: Review Request 15725: CLOUDSTACK-5022: NullPointerException when invalid
zone is passed into UsageEventUtils (master)
Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15725/#review29804
-----------------------------------------------------------
Commit 22422b9ddd3ea1566f66d971307235569bffb003 in branch refs/heads/master from David Grizzanti
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=22422b9 ]
CLOUDSTACK-5022: NullPointerException when invalid zone is passed into UsageEventUtils
- ASF Subversion and Git Services
On Dec. 3, 2013, 4:50 p.m., David Grizzanti wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15725/
> -----------------------------------------------------------
>
> (Updated Dec. 3, 2013, 4:50 p.m.)
>
>
> Review request for cloudstack.
>
>
> Bugs: CLOUDSTACK-5220
> https://issues.apache.org/jira/browse/CLOUDSTACK-5220
>
>
> 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.
>
>
> Diffs
> -----
>
> engine/components-api/src/com/cloud/event/UsageEventUtils.java e71afb0
>
> Diff: https://reviews.apache.org/r/15725/diff/
>
>
> Testing
> -------
>
> built on master
>
>
> Thanks,
>
> David Grizzanti
>
>
Re: Review Request 15725: CLOUDSTACK-5022: NullPointerException when invalid
zone is passed into UsageEventUtils (master)
Posted by David Grizzanti <da...@sungard.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15725/
-----------------------------------------------------------
(Updated Dec. 3, 2013, 4:50 p.m.)
Review request for cloudstack.
Changes
-------
Updating patch so it will work on master
Bugs: CLOUDSTACK-5220
https://issues.apache.org/jira/browse/CLOUDSTACK-5220
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.
Diffs (updated)
-----
engine/components-api/src/com/cloud/event/UsageEventUtils.java e71afb0
Diff: https://reviews.apache.org/r/15725/diff/
Testing
-------
built on master
Thanks,
David Grizzanti
Re: Review Request 15725: CLOUDSTACK-5022: NullPointerException when invalid
zone is passed into UsageEventUtils (master)
Posted by David Grizzanti <da...@sungard.com>.
> On Dec. 3, 2013, 11:46 a.m., Murali Reddy wrote:
> > patch does not apply on master. Please resubmit. Also do you want to push the changes to 4.3 as well?
Updated the diff so it will patch on master. This can be applied to 4.3 as well, though that would be the other patch I believe based on the state of that branch. Should i submit a separate patch for that branch?
- David
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15725/#review29656
-----------------------------------------------------------
On Dec. 3, 2013, 4:50 p.m., David Grizzanti wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15725/
> -----------------------------------------------------------
>
> (Updated Dec. 3, 2013, 4:50 p.m.)
>
>
> Review request for cloudstack.
>
>
> Bugs: CLOUDSTACK-5220
> https://issues.apache.org/jira/browse/CLOUDSTACK-5220
>
>
> 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.
>
>
> Diffs
> -----
>
> engine/components-api/src/com/cloud/event/UsageEventUtils.java e71afb0
>
> Diff: https://reviews.apache.org/r/15725/diff/
>
>
> Testing
> -------
>
> built on master
>
>
> Thanks,
>
> David Grizzanti
>
>
Re: Review Request 15725: CLOUDSTACK-5022: NullPointerException when invalid
zone is passed into UsageEventUtils (master)
Posted by Murali Reddy <mu...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15725/#review29656
-----------------------------------------------------------
patch does not apply on master. Please resubmit. Also do you want to push the changes to 4.3 as well?
- Murali Reddy
On Nov. 20, 2013, 8:15 p.m., David Grizzanti wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15725/
> -----------------------------------------------------------
>
> (Updated Nov. 20, 2013, 8:15 p.m.)
>
>
> Review request for cloudstack.
>
>
> Bugs: CLOUDSTACK-5220
> https://issues.apache.org/jira/browse/CLOUDSTACK-5220
>
>
> 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.
>
>
> Diffs
> -----
>
> engine/components-api/src/com/cloud/event/UsageEventUtils.java a4be4dd
>
> Diff: https://reviews.apache.org/r/15725/diff/
>
>
> Testing
> -------
>
> built on master
>
>
> Thanks,
>
> David Grizzanti
>
>
Re: Review Request 15725: CLOUDSTACK-5022: NullPointerException when invalid
zone is passed into UsageEventUtils (master)
Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15725/#review29654
-----------------------------------------------------------
Commit 03ba659ae7263ca4fdb7c1c0eaccbecdf45594e8 in branch refs/heads/4.2 from David Grizzanti
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=03ba659 ]
CLOUDSTACK-5022: NullPointerException when invalid zone is passed into UsageEventUtils
- ASF Subversion and Git Services
On Nov. 20, 2013, 8:15 p.m., David Grizzanti wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15725/
> -----------------------------------------------------------
>
> (Updated Nov. 20, 2013, 8:15 p.m.)
>
>
> Review request for cloudstack.
>
>
> Bugs: CLOUDSTACK-5220
> https://issues.apache.org/jira/browse/CLOUDSTACK-5220
>
>
> 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.
>
>
> Diffs
> -----
>
> engine/components-api/src/com/cloud/event/UsageEventUtils.java a4be4dd
>
> Diff: https://reviews.apache.org/r/15725/diff/
>
>
> Testing
> -------
>
> built on master
>
>
> Thanks,
>
> David Grizzanti
>
>