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

2013-06-28 Thread bharat kumar
--- 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

Re: Review Request 12110: Cloudstack-3106 Delete all the ips except for the ip alias.

2013-06-28 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12110/ --- (Updated June 28, 2013, 9:13 a.m.) Review request for cloudstack and Abhinandan

Re: Review Request 12110: Cloudstack-3106 Delete all the ips except for the ip alias.

2013-06-27 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12110/#review22494 --- 1. Move txn code to method removeFromDB. 2. Rename removeFromDB to m

Re: Review Request 12110: Cloudstack-3106 Delete all the ips except for the ip alias.

2013-06-27 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12110/ --- (Updated June 27, 2013, 10:57 a.m.) Review request for cloudstack and Abhinanda

Re: Review Request 12110: Cloudstack-3106 Delete all the ips except for the ip alias.

2013-06-27 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12110/ --- (Updated June 27, 2013, 10:56 a.m.) Review request for cloudstack and Abhinanda

Re: Review Request 12110: Cloudstack-3106 Delete all the ips except for the ip alias.

2013-06-27 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12110/#review22467 --- Ship it! Ship It! - Abhinandan Prateek On June 26, 2013, 4 p.m.,

Review Request 12110: Cloudstack-3106 Delete all the ips except for the ip alias.

2013-06-26 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12110/ --- Review request for cloudstack and Abhinandan Prateek. Bugs: Cloudstack-3106 R