You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by bharat kumar <bh...@citrix.com> on 2013/06/28 11:14:20 UTC

Re: Review Request 12110: Cloudstack-3106 Delete all the ips except for the ip alias. Cloudstack-3119 Shared network removal doesn't cleanup corresponding IP ranges

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

(Updated June 28, 2013, 9:14 a.m.)


Review request for cloudstack and Abhinandan Prateek.


Summary (updated)
-----------------

Cloudstack-3106 Delete all the ips except for the ip alias. Cloudstack-3119 Shared network removal doesn't cleanup corresponding IP ranges


Bugs: Cloudstack-3106


Repository: cloudstack-git


Description (updated)
-------

Cloudstack-3106 Delete all the ips except for the ip alias.
https://issues.apache.org/jira/browse/CLOUDSTACK-3106

Cloudstack-3119 Shared network removal doesn't cleanup corresponding IP ranges
https://issues.apache.org/jira/browse/CLOUDSTACK-3119


Diffs
-----

  engine/schema/src/com/cloud/dc/VlanVO.java af6b5fc 
  engine/schema/src/com/cloud/network/dao/IPAddressDao.java fecd44a 
  engine/schema/src/com/cloud/network/dao/IPAddressDaoImpl.java 886011e 
  server/src/com/cloud/configuration/ConfigurationManagerImpl.java cb4f76b 
  server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 4b29eab 

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


Testing
-------

Tested on master.


Thanks,

bharat kumar