You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Murali Reddy <Mu...@citrix.com> on 2013/05/13 18:33:02 UTC
[MERGE] CLOUDSTACK-652 elastic IP
I would like to merge feature proposed as elastic IP [1] in to master. Feature has been developed in feature branch 'portablepublicip' [4]
In the original proposal [1] and FS [2] I was intending to introduces AWS like EIP semantics that provide a ability to move public IP across the availability zones. Since the real value of this feature is ability to move public IP across the zones, so tying it with just static NAT (which is EIP) is restricitve . So I have developed the feature in generic way so as to enable all network services not just static NAT and yet keep the public IP transferrable across the networks in a zone or across zones. I updated the FS which reflects simplified semantics at [3]. Also, to keep with generic semantics, I am proposing this pool of public IP's to be called 'portable ip's instead of 'EIP'.
Integration tests for this feature is available at [5]. Branch is synced to latest master and passes RAT check.
[1] http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-dev/201303.mbox/%3CCD6C2308.373DC%25murali.reddy@citrix.com%3E
[2] https://cwiki.apache.org/confluence/display/CLOUDSTACK/EIP+Enhancements
[3] https://cwiki.apache.org/confluence/display/CLOUDSTACK/portable+public+IP
[4] https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/portablepublicip
[5] https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob;f=test/integration/smoke/test_portable_publicip.py;h=101747d958e90dd017323b34bae910f0199fc3c5;hb=refs/heads/portablepublicip
Re: [MERGE] CLOUDSTACK-652 elastic IP
Posted by Chip Childers <ch...@sungard.com>.
On Mon, May 13, 2013 at 04:33:02PM +0000, Murali Reddy wrote:
> I would like to merge feature proposed as elastic IP [1] in to master. Feature has been developed in feature branch 'portablepublicip' [4]
>
> In the original proposal [1] and FS [2] I was intending to introduces AWS like EIP semantics that provide a ability to move public IP across the availability zones. Since the real value of this feature is ability to move public IP across the zones, so tying it with just static NAT (which is EIP) is restricitve . So I have developed the feature in generic way so as to enable all network services not just static NAT and yet keep the public IP transferrable across the networks in a zone or across zones. I updated the FS which reflects simplified semantics at [3]. Also, to keep with generic semantics, I am proposing this pool of public IP's to be called 'portable ip's instead of 'EIP'.
>
> Integration tests for this feature is available at [5]. Branch is synced to latest master and passes RAT check.
>
> [1] http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-dev/201303.mbox/%3CCD6C2308.373DC%25murali.reddy@citrix.com%3E
> [2] https://cwiki.apache.org/confluence/display/CLOUDSTACK/EIP+Enhancements
> [3] https://cwiki.apache.org/confluence/display/CLOUDSTACK/portable+public+IP
> [4] https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/portablepublicip
> [5] https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob;f=test/integration/smoke/test_portable_publicip.py;h=101747d958e90dd017323b34bae910f0199fc3c5;hb=refs/heads/portablepublicip
FWIW, this looks good to me. +1