Review Request 27856: The NET.IPRELEASE events are not added to usage_event on IP range deletion from Physical Networks

2014-11-10 Thread Damodar Reddy Talakanti

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

Review request for cloudstack, Jayapal Reddy and Kishan Kavala.


Bugs: CLOUDSTACK-7877
https://issues.apache.org/jira/browse/CLOUDSTACK-7877


Repository: cloudstack-git


Description
---

Once you create a IP range and tie to an account and try to delete the range 
before allocating any IP it will not stop metering usage even after deletion of 
the range.


Diffs
-

  server/src/com/cloud/configuration/ConfigurationManagerImpl.java 1f71c0f 

Diff: https://reviews.apache.org/r/27856/diff/


Testing
---

Tested on Xen against the above use case and some other use cases.


Thanks,

Damodar Reddy Talakanti



Re: Review Request 27856: The NET.IPRELEASE events are not added to usage_event on IP range deletion from Physical Networks

2014-11-14 Thread Jayapal Reddy

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

Ship it!


Ship It!

- Jayapal Reddy


On Nov. 11, 2014, 6:15 a.m., Damodar Reddy Talakanti wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27856/
> ---
> 
> (Updated Nov. 11, 2014, 6:15 a.m.)
> 
> 
> Review request for cloudstack, Jayapal Reddy and Kishan Kavala.
> 
> 
> Bugs: CLOUDSTACK-7877
> https://issues.apache.org/jira/browse/CLOUDSTACK-7877
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Once you create a IP range and tie to an account and try to delete the range 
> before allocating any IP it will not stop metering usage even after deletion 
> of the range.
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/configuration/ConfigurationManagerImpl.java 1f71c0f 
> 
> Diff: https://reviews.apache.org/r/27856/diff/
> 
> 
> Testing
> ---
> 
> Tested on Xen against the above use case and some other use cases.
> 
> 
> Thanks,
> 
> Damodar Reddy Talakanti
> 
>