In some of the tests, cleanups were missing after creating Octavia
resources.
This patch adds the missing cleanups.
Change-Id: I848e08295580709fa739df2d216dad0efd19a795
The API test for pool creation has a check to wait for the
operating status of a pool to transition to ONLINE if a
listener has been provided.
The scenario test for pool creation doesn't wait for any
operating status transition after a pool is created with
a listener, and instead always assumes it's OFFLINE.
They should both wait for a transition, since otherwise
drivers that transition to ONLINE quickly, like OVN, can
race and fail the scenario test.
Depends-on: https://review.opendev.org/c/openstack/octavia/+/763530
Change-Id: I9eb3c7ce8af879cd910b06234c99b53670907e49
Story: 2008009
Task: 40659
Wait for the loadbalancer to become ACTIVE after deleting an
healthmonitor or a listener, to ensure the next test uses a LB in a
non-transitional state.
This also fixes the cidrs tests that were using the wrong waiter
timeouts.
This patch also moves individual deletion of children resources in class
tear down to a more performant way by cascade deleting the load
balancer.
Story: 2008219
Task: 41008
Depends-On: https://review.opendev.org/#/c/757604/
Depends-On: https://review.opendev.org/#/c/757840/
Depends-On: https://review.opendev.org/#/c/757841/
Depends-On: https://review.opendev.org/#/c/757842/
Co-Authored-By: Carlos Goncalves <cgoncalves@redhat.com>
Change-Id: I6a4eed7269e4f502bd0fc8613cb4ec4da13890e7
This is a patch to restructrue the scenario tests to use the
new skip_if_not_implemented capability.
Change-Id: I49a7fb6650030f2a1115c6d42442062bd33415fd
Currently we are not applying the custom hacking checks we use in
Octavia to the octavia-tempest-plugin. This leads to style issues
sneaking into the plugin.
This patch copies the same checks from Octavia (for simpler
management) into octavia-tempest-plugin and resolves some of the
existing style violations.
Change-Id: Ic103d8bededfec77ec0034d7c167ac1e49954a5c
Enabled the same flake8 extensions as the Octavia tree,
fixing the bugs that they now caught.
Trivialfix
Change-Id: I0fc3f5e3a48dc9dc0286cf9b11847a77573ac411
When a test method completes it must assert its resources are in a
mutable state or following test methods in same test class can fail to
create, update and delete shared test class resources.
This problem has been recently seen in upstream CI [1]. The test method
was not waiting for the load balancer to come back up after a CUD
operation in its child resources, resulting in the next test method
execution failing to create a pool on the test class-level load
balancer.
[1] https://zuul.opendev.org/t/openstack/build/ \
5d8eb87364c84480b54c87fd6966dbf4/log/controller/logs/tempest_log.txt#7968
Story: 2007187
Task: 38313
Change-Id: I487b1fc7e3de8724b16b6f8eabad5e69f5009758
This patch adds map of supported algorithms by
provider drivers. For a first iteration lets select
the first from supported algorithms to be used as
a default and run the tests with it.
In addition this patch splits check_members_balanced()
into subfunctions related to the algorithm
that is validated.
Story: 2006264
Task: 35972
Change-Id: Id055763f35b487da539eddfe802c543a11246503
Not all provider drivers may support APP_COOKIE/HTTP_COOKIE,
so this patch modifies the test to reflect the same.
Change-Id: Ifea51c1f927947588ac2ed82f25fa35ee56c98f7
There is at least one provider driver that does not currently support
any other pool algorithm than round-robin. This patch adds a new option
that allows one to toggle enablement of pool algorithms (default is
True/enabled).
Change-Id: I11e1569f91fe66a6bc735a1abd7b1545ca1490a0
Currently not all Provider drivers support L7 in Octavia.
Until the flavors option is not enabled, it is proposed to skip
L7 in favor of L4( specifically TCP ) protocol tests for the provider
drivers.
Change-Id: I07b4f7038d71d684c23885c89257af2711a442a0
Neutron can be slow to cleanup ports from subnets/networks.
This patch adds retries when deleting subnets and networks in the
tempest teardown/clean up phase after tests.
Also, there were cases where addClassResourceCleanup was being used
inside test cases instead of addCleanup. This patch corrects those to
use addCleanup.
Story: 2004826
Task: 29000
Change-Id: Ia29541d1c89f3559a3ce22b1a27c6bcf079ce2cc