allow multiple subnets to connect to vpn

Registered by yong sheng gong

I think we should allow more than one subnet_id in one vpnservice object.

This bp will allow user to create vpn service with multiple subnets, insert subnet into vpn service or remove it from vpn service.

We will try to keep compatible with the existing one subnet API

Blueprint information

Status:
Complete
Approver:
Akihiro Motoki
Priority:
Low
Drafter:
Paul Michali
Direction:
Approved
Assignee:
Paul Michali
Definition:
Approved
Series goal:
Accepted for mitaka
Implementation:
Implemented
Milestone target:
milestone icon mitaka-1
Started by
Armando Migliaccio
Completed by
Akihiro Motoki

Whiteboard

Nov-23-2015(armax): the feature is indeed implemented, but there's a problem with discoverability of the API extension, and bug #1515670 is about that.

Nov-23-2015(armax): some bugs look still open. Are we sure we can claim this implemented?

armax(Oct-19-2015): If I am not mistaken, this has been taken care of in these patches:

https://review.openstack.org/#/q/topic:bug/1459423,n,z - Note (pc_m): This is the RFE, which at the time of approval, a spec/BP was not deemed neccesary for this change (by mestery)

- server side review https://review.openstack.org/#/c/230164/
- client change https://review.openstack.org/#/c/231133/
- functional test enhancements commit https://review.openstack.org/#/c/237090/
- API documentation for both endpoint groups (the foundation code) and multiple local subnets, https://review.openstack.org/#/c/231977 and https://review.openstack.org/#/c/233206, respectively.

(pc_m)There are some related tasks that will need to be addressed, once the main feature is upstreamed. Provided here for reference:

- API test for new endpoint group and multiple local subnet APIs (once API tests available in neutron-vpnaas repo).
- Horizon support for multiple local subnet feature.
- Check when changing subnet in use by endpoint group (https://bugs.launchpad.net/neutron/+bug/1503862)
- Akihiro would like to see endpoint group API placed in an extension shim to make the API discoverable. (https://bugs.launchpad.net/neutron/+bug/1515670)

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.