Prasanna, thanks for taking a look at the branch and for making the necessary changes to the marvin tests.
If there are no concerns I will merge the branch today. Thanks, Likitha >-----Original Message----- >From: Chip Childers [mailto:chip.child...@sungard.com] >Sent: Thursday, April 11, 2013 12:37 PM >To: dev@cloudstack.apache.org >Subject: Re: [MERGE] Dedicate Public IP Addresses > >On Tue, Apr 09, 2013 at 05:23:05PM +0000, Likitha Shetty wrote: >> Hi all, >> >> I would like to merge the feature Dedicate Public IP ranges to master. >> >> Jira ticket - https://issues.apache.org/jira/browse/CLOUDSTACK-704 >> Proposal discussion - >> http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-users/20 >> >1303.mbox/%3C64FB1554ABC9B44FAA773FBD6CB889C2010D9C03119B%40BA >NPMAILBO >> X01.citrite.net%3E FS - >> https://cwiki.apache.org/confluence/display/CLOUDSTACK/FS-+Dedicate+Pu >> blic+IP+Addresses+per+tenant >> Branch - dedicate_public_ip_range_2 >> Unit tests - Unit tests for the new API's can be found at >> cloudstack/server/test/com/cloud/configuration/ConfigurationManagerTes >> t.java Integration tests - Python tests can be found be found at >> test/integration/component/test_public_ip_range.py >> Rebased the branch with master - Commit id: >> d5d167cb97b95f5622c0e34fe4546642484016f6 >> RAT report - Attached >> Patch is available at - https://reviews.apache.org/r/10377/ >> >> Thanks, >> Likitha > >+1 - this is a great merge
