openstack/networking-bgpvpn - networking-bgpvpn - OpenDev: Free Software Needs Free Tools

Since bgpvpn might be run with the neutron l2 agent but without
bagpipe, we should only enable the bagpipe l2 agent extension
when devstack is told to use bagpipe.
Change-Id: I4e085f489ed99b72f09e03cffcd2814a1721dfe8
Closes-bug: 1559149

Update to:
- explain how to enable the OVS agent extension
- remove "enable_plugin bagpipe-bgp"
More precision and detail.
Change-Id: I69652dff0fe2788f1e6ce92e55ca50c3c8481ebe

Change-Id: I7ae4b1960443bd0bcec8efe2947e7498adea79f1

This test shows that only admin
tenants can create bgpvpns.
Change-Id: Ie09bb37f787227187c0492bf945f29ee6c082d1b

To support the latest version
of tempest we have to change to use
tempest.lib.
In addition the neutron clinet is
not a dependence for tempest anymore.
So we have to get rid of this
dependence as well.
Change-Id: I0c688a5bbc946ed0b488c6f95a80653e22a8d6e4

Change-Id: If6ca34b64f952271a7ca04935cf91747bb29cbae
Closes-Bug: #1557445 

This change introduces an OVS L2 agent extension that will now be used
instead of the previous agent in ovs_bagpipe_neutron_agent.py which
was inheriting from the Neutron OVS agent in a crude way, requiring
duplicating the main() code.
We also remove mention of the old bagpipe-openvswitch agent
in the documentation and and adapt the devstack plugin so that
old local.conf will still work.
Change-Id: Ibf27e2b028efffa1e49207d39bfc9b13464180ad
Closes-Bug: 1492021
Closes-Bug: 1531459

Adding rc files to allow to control the gate jobs parameters
without having to touch Jenkins job definitions in project-config.
Change-Id: I8d9580b480b1cdd3d185d51f13cd06be26310d17

Update test according to python-neutronclient change Ibe4b803f3bcac2d13a9fe5992b2f7b665801ef3f
that introduces pagination in list requests.
[1] 597800f503
Change-Id: I786c71cbea75af292cd4fa01db032d75067e4353

This change allows to specify tenants by name in BGPVPN
definition, and allows to specify BGPVPN by name in
BGPVPN-Net-Associations and BGPVPN-Router-Associations.
It also adds heat property constraints for these properties
and for a few others.
The heat documentation examples are updated as well.
Change-Id: I09afb557f0eb02193ef9be8506a0e8e98a98f52c
Closes-Bug: 1551851

Provide more examples and distinguish between admin
and tenant roles.
Also improve formatting and layout.
Change-Id: I1cef9d8043afd1cf59b4d845e5793fc9dd14e8cd

This patch adds the BGPVPN-ROUTER-ASSOCIATION resource to heat.
Change-Id: Id242864f782033cfa532fc23c9118bd3b63b6af8
Partially-Implements: blueprint heat-support-bgpvpn-association

This patch adds a heat plugin to provide BGPVPN and
BGPVPN-NET-ASSOCIATION resources to HEAT.
Partially-Implements: blueprint heat-support-bgpvpn-association
Change-Id: Ibebbb306abafc8306601fcf5484fc97ee753fba9

The bgpvpn devstack plugin currently configures tempest during
the post-config stage. But the tempest.conf file is overwritten
by tempest during the extra stage of devstack, hence erasing
modifications made by the bgpvpn plugin.
To fix this issue, the modification of the tempest.conf file
must be done during the extra stage of devstack.
This patch also corrects the indentation of the devstack plugin.
Change-Id: Ia2e76f1c9aea93e481694b0821d1e5c3bed4da11
Closes-bug: #1548364 

We're still forcing the use of the liberty release of
python-neutronclient.
This is pointless, and recent devstack breaks because of that.
Change-Id: I49ce9786fe0403559a85f7c10af725d661123e13

Change-Id: Ie085e2f2e1c03ccbf0f95405c6f6a53d73e1db38

This patch fixes the fact that some labels in block
diagrams were not readable.
Change-Id: Ic46ba596f60df52ab21ab43f917aa9fbe04c14a7

Follow the upstream changes in Neutron done in change
I6ad156a7dd686ab2a2349704a34385db1ee9681c.
Change-Id: I19c9bb83dc2519cdc2b0d8e0e11858e80d36cd4a

Update requirements (tools/tox_install.sh): our master
branch now needs to follow Neutron's master.
Update unit tests.
Bagpipe driver: follow changes in OVS agent main in Neutron master.
Change-Id: I310ea955329b6f523b417b6bf8c25f6d2db08eba

This adds ODL driver support for router association introduced
by the change id : Ia9db2b64bcf0cdcc25e4154eeb557073be9f2221
Change-Id: Ib11f8e84939232cd9bab7050e245458f7790ea6e
Closes-bug: 1523559

Until bagpipe supports IPv6, we need to restrict the DB lookup
to Ipv4 subnets.
Change-Id: I510a498cf260a09d12a515168eacd40967701d0b
Closes-bug: 1526705

Python 3.3/2.6 support would be dropped by
Infra team from mitaka,CI would no longer be testing it,
so projects should drop it also.
Change-Id: Icb7ce22ad95a508c1b04554a74ec3144685abc1e

OpenStack projects are no longer being tested under Python 2.6, so
remove the trove classifier implying that this project supports 2.6.
Change-Id: I931646c01bd60fdf7e3c31666fdac084357888ff

Router associations should only be allowed with bgpvpns that have type L3.
This commit adds a check and will raise a BadRequest exception when the
bgpvpn is not of type L3.
Change-Id: I8b9da298a4e74ceddb764d84d86decb4499febe2
Closes-Bug: 1525863

This change fixes the RT/RD validation code:
- use the same validation function for route_targets than
 the one used for import_rts and export_rts
- allow an empty value without allowing everything
 (_validate_rt_list_or_none was accepting anything non-empty)
- add unit tests testing the validation of the four RT/RD fields
Change-Id: I8769b0b135d9c9108475a4366c53266787228beb
Closes-Bug: 1524169

RTs consolidation specified in the spec is currently not implemented.
This consolidation will have to be done at the driver level,
not at the plugin level.
Change-Id: I3b3cbbf0846f8caadb4e7ce105bd33eac460dfe9
Closes-bug: 1478998

This change adds a test to check that the neutronclient
extension works as expected.
It will fail unless python-neutronclient includes the fix we
need for subresources support [1].
[1] https://review.openstack.org/#/c/235827
Change-Id: I01f192feadfce99d8eba8ba6701f0139ede985be

Add help text for each of the BGPVPN CLI commands.
Closes-bug: 1517978
Change-Id: I78ad91025a09830e5770d1f2d178487e9b87d3ee

We currently need to send 'network_id', 'mac_address', 'ip_address',
'gateway_ip' to the agent, even in the absence of a BGPVPN bound
to the port. If we don't, this information will lack on an
update_bgpvpn RPC.
Change-Id: I9f6e59cc49cbe1aad7291898c867508c3b2d2b2d
Closes-Bug: 1523398

This commit implements bp router-bgpvpn-association. Router associations
will work similar to network associations except in the bagpipe driver where
the router association will in fact get translated into multiple network
assocations.
Implements: blueprint router-bgpvpn-association
Change-Id: Ia9db2b64bcf0cdcc25e4154eeb557073be9f2221

When a request asks for creating a network association that
already exists, we have to catch the DB exception and raise an
exception that respond an error code 400.
Change-Id: Iafcaa6b016bc5f39f5ee1d64ef7ff627efa74183
Closes-Bug: 1512714

This change removes all the code related to the auto_aggregate
attribute. The intent is to revert this change when/if drivers
start to support this attribute.
This change also updates the API documentation to indicate
that the auto_aggregate attribute is not implemented yet.
Closes-Bug: 1522325
Change-Id: Ia4aec84f5f9022e98b0d417e265a9201cbc0d5af

This change updates the bagpipe driver so that
information is always retrieved from the db, which fixes the
cases where the port info is absent from the notification.
Closes-Bug: 1520629
Change-Id: I41a4e77b33d0b8790302b58ae625b41d28b2df9a

Update tools/tox_install.sh so that the right neutron
branch is also used when the zuul cloner is in place.
Change-Id: I6c87713f85d8153f53b89484116f2755d0098d40

Change-Id: I0cb5fefc5a620f101dd157e7ffd9898bf4b880c2

By using:
cookiecutter https://git.openstack.org/openstack/tempest-plugin-cookiecutter
A first testcase is added as well added:
test_create_bgpvpn
Change-Id: I71cc3564673bee5e4678dd08e365b13849d60f5a