Since [1] OVN/OVS source deploy jobs running with
OVN_BRANCH=main fails to compile ovn as this now
requires newer ovs commits from branch-3.2.
[1] https://github.com/ovn-org/ovn/commit/558da0cd
Change-Id: Ia546671f0d7be3e893eb2c7de67c82287bc53f52
With [1] OVN main branch compilation fails, until
main branch is fixed to work with ovs master, let's
pin OVS_BRANCH to working commit.
[1] https://github.com/openvswitch/ovs/commit/07cf5810de
Related-Bug: #2015728
Change-Id: Icdd1affc944de6c1e00da9539e13a8d698cfc0e6
According to [1], the needed version for OVN master is
5a686267d36c5c4229ec801a9616ceb60740fbe3. So pin to this
for OVN master build from sources jobs.
[1] http:github.com/ovn-org/ovn/commit/2e4f393650ccf298f26787583c13a88197ba8348
Change-Id: If504cd0cc73f9e2340668cff12a2cd63030653bc
This is an automatically generated patch to ensure unit testing
is in place for all the of the tested runtimes for antelope. Also,
updating the template name to generic one.
See also the PTI in governance [1].
[1]: https://governance.openstack.org/tc/reference/project-testing-interface.html
Change-Id: I2a622bd3ae05bcb4681e1e50a521d223d51b34d9
According to [1], the needed version for OVN master is
fe171e4f109f001f07b867756a261d898f0d2cfc, but after
running git submodule status the output indicates
6f24c2bc769afde0a390ce344de1a7d9c592e5a6. So pin to this
for OVN master build from sources jobs.
[ovn]$ git submodule status
-6f24c2bc769afde0a390ce344de1a7d9c592e5a6 ovs
[1] 9f25b2cedf
Closes-Bug: #1987308
Change-Id: I1af8e5fe53447811fbcc388d31e5c7eecb8ff2e4
OVN Octavia Provider is registering opts a soon modules (driver,
agent or helper) are imported, so in a test scope terms, when some
tests run the setUp they are broken by a DuplicateOptionError
because they are based on TestOVNFunctionalBase from Neutron
where same options are loaded to same oslo_config group.
This fix proposes some refactoring in a similar way to [1] and [2]
in order to avoid the registration of opts a soon a module is
imported, instead we switch to one charge per required class
whenever possible.
Additionally in order to fix gates and according to [3], we pin
version of OVS to d94cd0d3eec33e4290d7ca81918f5ac61444886e hash
[1] https://review.opendev.org/c/openstack/neutron/+/837392
[2] https://review.opendev.org/c/openstack/neutron/+/839783
[3] f93206ce40
Related-Bug: #1972278
Related-Bug: #1967472
Change-Id: I2f36af767a0a0a4c19488b6998a414b8672114f5
Create some additional tests to increase coverage to 92%.
Co-authored-by: Brian Haley <haleyb.dev@gmail.com>
Co-authored-by: Fernando Royo <froyo@redhat.com>
Depends-On: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/836144
Change-Id: I3bd686db234f472c693c3ac1d23e58be58a5acb6
According to [1], the needed version for OVN v21.06.0 is
a4b04276ab5934d087669ff2d191a23931335c87
[1]4b0a02635f
And according to [2], the need version of job using OVN
master is 498cedc483f3239c839c55b4d9f2261b61fb6ace
[2] 576491f507
Related-Bug: #1967472
Change-Id: I63cfc82b329687204e08c2061c6ac8eb7582ab77
* The "devstack_localrc" key was missing, without it local.conf won't be
populated.
* Added OVN_BUILD_FROM_SOURCE to True to make sure OVN is compiled
from code and not installed from packages.
* Updated the version of OVN and OVS in the -released job to a newer
version.
Closes-Bug: #1964339
Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com>
Change-Id: I05ed8fb053c78bd16a52b5d82a3ab51faf856d78
Enable OVS and OVN compilation from source in functional testing
CI jobs.
This patch uses the new variable defined in [1]:
"Q_BUILD_OVS_FROM_GIT".
[1]https://review.opendev.org/c/openstack/devstack/+/799621
Closes-Bug: #1959028
Change-Id: Ie42255d2f06563b1890eb70f87e0213b731eb581
This patch switches using f-strings instead of regular ones in places
where pep8 was complaining about with error C0209.
It also updates some of the lower-constraints to align them with
Neutron's lower-constraints and to make ci job happy.
It also changes OVN_BRANCH from "master" to "main" as that was renamed
in the OVN repo some time ago.
Change-Id: I8c805504e8a9e06dcdd4b1ac850d54400ef05b40
TCP and UDP Connect health monitors are now supported by the
provider driver, when the underlying OVN version supports
them.
The health monitor uses the OVN distributed DHCP port as the
source IP for messages, one must exist or the create or update
operation will fail. The list of member ports to monitor is
updated whenever one is created or deleted.
Added support for SBDB notifications for Service Monitor
events generated when the OVN health monitor code detects
issues with connectivity. This will trigger updates to the
Octavia objects as members go on/offline so that the Octavia
API reflects the current running state of the load balancer.
Added a lot of new unit and functional tests to cover all the
new code, and enabled the tempest health monitor feature so
those tests are run as well.
Change-Id: I776a7a6b036ad72ce7eaa1ab39f448b344974be6
These two tests are failing a significant amount of the time,
disable while we investigate the reason so we can continue to
merge other code.
Related-bug: #1934915
Change-Id: Ie503fa211f9d6ec0e4ac62ee39aa9e781e0ff1b0
Since Kuryr Kubernetes is a consumer of the OVN provider
driver, let's add a relevant job to co-gate on, non-voting
for now.
Change-Id: I437a2dca911668216110177df2f96fd1c1b0f58e
Tempest replaced the below rolevar for run-tempest role
- tempest_test_blacklist is replaced by tempest_test_exclude_list
- tempest_black_regex is replaced by tempest_exclude_regex
old name are still supported for compatiblity but we recommend
to switch to new one.
Depends-On: https://review.opendev.org/c/openstack/tempest/+/774835
Change-Id: Iaa398e20575729c82c14a0a00d3992b88ccdfce8
The Octavia API supports 'SCTP' as a protocol when creating
listeners and pools, so add it to OVN_NATIVE_LB_PROTOCOLS.
Core OVN already has support for SCTP.
Required a bump in octavia-lib version to pick-up constant.
Change-Id: Ica57a5c77af61bffba812b35608bfc2a90011ba8
Make sure we collect all OVN logs to help with
debugging. Copied from neutron repo job definition.
Change-Id: I84ce28a5e5dd4d5d6fffac41098e82faece478f5
Depending on the image used for testing, we might have one
with an older version of OVN that does not have selection
fields in load balancers, which will lead to failures since
connections will not be balanced correctly. Change to build
from source so we test with both 20.06 and the tip of master.
Change-Id: If7ca57e19bc0f2dac4eb0ec8859b53fa5fd7ebdb
OVN Octavia provider driver relies now on new features
comming from OVN master. That means that would be great
to use master, at least in the check queue.
Change-Id: Ifd909f03c00b403837c513afdde085442cf9ccd8
This is a combination of a number of fixes required to
fix the broken gate.
Fix lower-constraints job for focal, bumping requirements
and fixing resultant failuers as necessary.
The devstack module for OVN has now been moved to the
devstack repository, so include it from there so the
functional jobs can run.
Set not_implemented_is_error to False.
The Octavia tempest plugin tests are being changed to
better skip tests for not implemented features. Setting
this to False will treat NotImplemented failures as
skips and not errors.
Fix tempest blacklist regex.
Raise error on listener create with 'allowed_cidrs' present.
Change-Id: I70adcfad3f4d489d94da171670c23dba51d6f2a1
Without this config the tempest API test
octavia_tempest_plugin.tests.api.v2.test_provider.ProviderAPITest.test_provider_list
fails.
Change-Id: I18ccbe1a6f0896ff51440a245acd979321fce3bb
This change includes a 3 different patches that we need
to squash in order to pass the gate.
1) New versions of isort broke pylint. This patch fixes it at 4.3.21.
2) The functional job fails because of bugfixes in neutron
devstack lib. We need to update our functional jobs also.
3) Add functional release and master job that builds OVN
In order to test the latest code in the core OVN repository,
add a job that builds it from source from the master branch.
Define also second job that will run code with latest
OVN/OVS release.
Closes-Bug: #1887363
Change-Id: Ic013e5a0605c28453d3ee1b64031022f6f75f8f6
We still have scenario tests failing and it needs further work
in Octavia tempest plugin [1] and Core-OVN [2].
However, scenario tests are not the only ones that we expect to pass, so
for now ignore those test cases:
octavia_tempest_plugin.tests.scenario.v2.test_traffic_ops.TrafficOperationsScenarioTest.test_basic_traffic
octavia_tempest_plugin.tests.scenario.v2.test_traffic_ops.TrafficOperationsScenarioTest.test_basic_udp_traffic
and set this job to voting.
[1] https://review.opendev.org/#/c/714004/
[2] https://bugs.launchpad.net/neutron/+bug/1871239
Change-Id: I388438133e9028e7805aba017334d94a11aeda5f
This patch imports OVN LB functional tests.
Unfortunately neutron hasn't been released yet
so for test requirements we need to install it
from git, from commit that imports functional
base class to neutron [1].
[1] https://review.opendev.org/#/c/701733/
Change-Id: Ia65492db1688f3a3fce22a4432ae8445f67c37b3