In [1] we finally got rid of the unfinished lib/neutron module and kept
only lib/neutron-legacy. It's renamed to lib/neutron now and it's the
only neutron related module in Devstack.
So this patch removes left over todo comments about things to do when
migration to the new lib/neutron module will be finished.
[1] https://review.opendev.org/c/openstack/devstack/+/865014
Change-Id: Icd7030d36a2462263c9379e02c756d23f6ff6440
Call configure_placement_neutron to fill all the placement related
settings in neutron.conf if segments service plugin is enabled.
Change-Id: I9bfdd7f5eebcddbb5b39e089e6956742db866c63
In Ubuntu 22.04, the tcpdump binary is in /usr/bin instead of /usr/sbin,
adapt the path selection accordingly.
Needed-By: https://review.opendev.org/857031
Change-Id: Ida08e0ea98fe1b3dc8b4416101897dabc096fbdf
Commit [1] added by accident lines to an ovn local.conf sample file. In
this change, the file is restored to its previous state
TrivialFix
[1] https://review.opendev.org/c/openstack/neutron/+/828673
Change-Id: I9a43fa201291002fb1b1a9c76322581aa02f897b
This commit adds possibility to configure L3 ndp_proxy service
plugin and l3 extension with devstack plugin.
Change-Id: I41713b1cfbf55b0b0e752904c4aea06fe0a00e10
Partial-Bug: #1877301
This adds ability to configure local_ip service plugin
and local_ip OVS agent extension.
Needed to enable corresponding API/CLI and scenario tests.
Related-Bug: #1930200
Change-Id: Ia9269b7a73aef116cf8c5bc6ca0a1cca9deaa056
Without this port binding fails with below error:-
Network <nw> is type of vxlan but agent <host> or mechanism
driver only support ['gre', 'local', 'flat', 'vlan'].
Also fix permissions of /opt/stack/devstack in ml2 ovs testing
documentation and added these files to irrelevant-files to skip
running functional jobs as these files are not used in those jobs.
Related-Bug: #1934466
Change-Id: I3ca2ea19bf5e316e580669caab4c607447034a11
Those functions were part of the neutron devstack plugin but we
discussed it during last PTG [1] and decided to move to the Devstack
repo as plugins which are used by e.g. CI jobs which are defined outside
of the neutron repository.
Placement integration is used e.g. in the tempest-slow job which is
defined in tempest and used by many different OpenStack projects.
[1] https://etherpad.opendev.org/p/neutron-yoga-ptg#L142
Depends-On: https://review.opendev.org/c/openstack/devstack/+/815682
Change-Id: I72e2c4c55a3c40e2a3c491fe66c05b659b76e349
Since devstack had set OVN as the default backend for Neutron.
Then the minimum local.conf [1] for ML2 ovs will not work at
all. For some local testing of ML2 OVS, it is not right deployment
for users to test the ML2 OVS related cases.
This patch adds a sample local.conf for ml2 ovs to install a small
all in one environment for Neutron testing.
Sample tested OS:
1. CentOS Stream 8
2. CentOS Linux 8
[1] https://docs.openstack.org/devstack/latest/#create-a-local-conf
Closes-Bug: #1934466
Change-Id: Ie7bac1d2819c332a94a0ff308a300638c17f1b1f
devstack is not necessarily cloned to "$DEST/devstack"
("/opt/stack/devstack"), it can be run from any other directory.
$TOP_DIR [1] should be used when referring to the devstack working dir.
[1] https://opendev.org/openstack/devstack/src/commit/\
6af3cb9eb273c127c20bc07f65c9a5d7f8ba95cd/stack.sh#L73
Change-Id: Ibf4890985b6d07077d04f989eab833ae5b057200
This patch removes the devstack/lib/ovs module and update the scripts to
use the ovs_source module from DevStack instead.
Depends-On: https://review.opendev.org/c/openstack/devstack/+/791085
Change-Id: I65fe53ee753ac68340f7d4d928643d3d8e5c8694
Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com>
This is patchset 2 of 2 for OVN driver handling of security-group-logging.
It includes the core changes and tests for this feature.
This feature requires OVN 20.12 [0] or newer. Functional test will be
skipped for non-supported versions.
Related-Bug: 1468366
Closes-Bug: 1914757
[0]: 880dca99ea
Change-Id: Ic86fa70eb34c9b178267b80de1f8883a3ef03e98
Signed-off-by: Flavio Fernandes <flaviof@redhat.com>
Using OVN from packages was introduced in devstack via
https://review.opendev.org/c/openstack/devstack/+/763402
This change updates the sample files to make users aware of it,
especially when specifying the OVN branch to be used.
Change-Id: Ibfbdd712e49d1fb8522bcc39357be0f311ce3f5c
We have to pick OVS patches compatible with kernel 4.15.0-118
after switching to it.
Change-Id: I1ff47fc3398760d5b631bf89e6cfcc7ae339f0e3
Related-bug: #1883601
Closes-bug: #1896766
When the Octavia plugin code was moved to the Neutron
repo it was put under an 'if is_plugin_enabled octavia'
check. Depending on the way a node is built, sometimes
the function doesn't seem to be defined, so remove it
since it's ok to always define it.
Change-Id: Ib8c7bac4d0111345f30b7d160fe9b4a8f2132106
The DevStack module for OVN has now been moved to the DevStack repository,
this patch is deletes it from the Neutron repository.
Depends-On: https://review.opendev.org/#/c/748140/
Change-Id: I2c30b8130525380234d767a7cb46f9bcca2a0d6b
Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com>
This change updates sample OVN local.conf file in order
to enable neutron-segments service.
Related-Bug: #1871608
Closes-Bug: #1891243
Change-Id: Iac6227093b0e851716a76e38145508d6724c650a
This is a subset of the changes for implementing the floating IP
port forwarding feature in neutron, using OVN as the backend.
This changeset covers the core implementation for portforwarding/drivers/ovn,
mech_driver, ovn-router as well as a subset of tests.
Port forwarding support in ovn_db_sync is not included here to facilitate review.
That, as well as all other supporting changes, are under the ovn/port_forwarding topic:
https://review.opendev.org/#/q/topic:ovn/port_forwarding+(status:open+OR+status:merged)
Depends-On: https://review.opendev.org/#/c/726478/
Partially-implements: ovn/port_forwarding
Partial-Bug: #1877447
Change-Id: I019fe11ac1ddcf2304f3f144c62d52667fc11dce
This commit bumps the OVN version used in the gate from v20.03.0 to the
latest tagged version, v20.06.1.
Change-Id: Ie070f3c4d75272d12f295a2ea4f7fa6548a7b002
Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com>
This patch intends to make the transition of the OVN DevStack module
easier by allowing both copies of it to exist at the same time, the one
in Neutron repository and the one in the DevStack repository [0].
1) Make create_public_bridge idempotent
If we call this function more than once the second call and on will fail
with the following error:
2020-07-13 09:12:53.885633 | controller | ++
/opt/stack/neutron/devstack/lib/ovn_agent:create_public_bridge:263 :
sudo ip addr add 172.24.5.1/24 dev br-ex
2020-07-13 09:12:53.894247 | controller | RTNETLINK answers: File exists
During the transiton of moving the OVN DevStack from the Neutron
repository to the DevStack repository [0] this method is being invoked
twice in the ovn job in the DevStack gate because it will be cause as
part of the plugin being enabled and the normal code execution.
This patch makes the method idempotent by calling "addr replace" instead
of "addr add" to avoid the "RTNETLINK answers: File exists" error.
2) Move the setup of the tcpdump on br-ex to "extra" to it can start
after the create_public_bridge is called.
[0] https://review.opendev.org/#/c/734621/
Change-Id: I74b05ef245a343c7531fa45aeadb90060b2fd22e
Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com>
New cirros with recent fixes for metadata service
has been released. Lets update the image version on gate.
Also stop using different images for OVN. Lets use default
settings from devstack.
First we need to merge [2].
[1] e40bcd2964
[2] https://review.opendev.org/#/c/711492/
Change-Id: Idc614f9f25188bd1a1e1d5424274acf04ba99328
There were duplicated methods doing almost the same in terms
of OVS/OVN compilation.
This change:
* move of OVS related compilation code to devstack/lib/ovs
* delete of OVS related compilation code from devstack/lib/ovn_agent
* source unified functions in devstack/lib/ovn_agent from
devstack/lib/ovs
* Unify NEUTRON_PATH variable to NEUTRON_DIR
Closes-Bug: #1877377
Change-Id: Ia012a8e116a276a6674f86366c803e0e2d8ff704
The default path for the OVS tools is /usr/local/bin [0] but, by default
this location is not included in the $PATH for some distros (e.g
OpenSuse).
This patch includes the $OVS_BINDIR to the $PATH variable for deploying
DevStack.
[0]
0fc38297ba/python/ovs/dirs.py (L23)
Change-Id: If7d68212cadb8eb8a12fa3271023c9e86b2d411e
Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com>
This patch moves functions to enable br-ex-tcpdump and br-int-flows
services from the ovn devstack plugin to the main neutron devstack
plugin and enables it on all tempest and grenade jobs which are using
neutron-openvswitch-agent as a L2 backend.
Change-Id: Idfea869c08811914124d076da18cec17a12eee49
OVS tag v2.13.0 is not compatible with current used kernel on upstream
Bionic Ubuntu kernel 4.15.0. This patch sticks to commit hash to unblock
the gate. We can either change to newer 2.13 tag once released or better
stop compiling OVS and use one packaged by Ubuntu.
Run functional/fullstack tests with selected OVN and OVS versions
Previously for functional tests OVS version was hardcoded - v2.12.0,
and OVN was installed from OVS repository.
After we merged OVN driver to Neutron tree we run both
Neutron/OVS and Neutron/OVN functional tests in one job.
This patch adds possibility to specify from which tag/branch OVS and OVN
should be checkout.
Change-Id: I83688031951b97bfe64f3aaa761ad7afc1d5ea55
Closes-Bug: #1883601
Closes-Bug: #1878160
Previously, the function decided if the repository is after the split
based on the OVN branch name. This didn't work when a commit hash or other
branch name was used.
This patch changes the function to look at the tree and decide if the given
commit that split the OVN from OVS is present in the given OVS branch.
Change-Id: I89e1f450e4db4c9554e368d75155bec137074ba6
For now OVS doesn't support compilation of kernel module
for kernels newer than 5.5 [1].
Recently Fedora 31 upgraded the kernel to 5.6.8 and that is
why the periodic jobs started to fail [2].
By default we enabled OVS module compilation because Ubuntu
Bionic one doesn't have support for Openflow Meter actions.
In case flag OVN_BUILD_MODULES is set to True and kernel is
newer than 5.5 lets skip the module compilation with proper log.
Closes-Bug: #1877377
[1] https://zuul.openstack.org/builds?job_name=neutron-ovn-tempest-ovs-master-fedora
[2] 59e9944266
Change-Id: I63b2f198468c9e30af8284c8033fa31f2af2518a
If other openstack project uses Neutron with OVN
(as enable_plugin ... neutron) and doesn't specify
OVN_BRANCH and OVS_BRANCH variables in configuration
it will use master branch of those two repositories.
Master branch servers for OVN testing and aligning OVN
Neutron driver with new features.
For now we don't have voting OVN master CI job, to ensure that
everything is fine with it, so we should default OVN for other
projects to the latest stable version that we know works properly.
Change-Id: I6e36ccb8cf4846a1eb11619e7c339a0fc193c4f6
This commit adds possibility to configure fip port_forwarding
service plugin and l3 extension with devstack plugin for OVN.
Since OVN uses API workers, this change also introduces the
callbacks necessary in pf_plugin, so events related to port
forwarding are sent using neutron_lib callbacks registry.
Related-Bug: #1877447
Change-Id: I8124fac13bf4d802d232e8b3976e6a2cebc72106
OVN has been changed to use Openflow15 [1].
Instead of creating br-int and setting the openflow version
via ovn_agent script, it is better to delegate the bridge
creation to the ovn-controller. Thus, having ovn-controller
creating br-int addresses the potential version mismatch.
[1]: 6ec0b82038
Change-Id: I62e4e98556c71312a7cf85b6246ddbecbc59a039
Related-Bug: #1871239
Closes-Bug: #1877195
Sławek Kapłoński
yatinkarel