openstack/octavia-tempest-plugin
Code Issues Proposed changes

Fix request interval in _wait_for_lb_functional

Browse Source 
The request_interval parameter is passed to the _wait_for_lb_functional
function but it wasn't used in the function.
Now the function sleeps for request_interval sec for each iteration of
the loop.
It mitigates some random failures with the TCP listener +
least_connection lb_algorithm tests that require a specific delay
between the requests [0]

[0] https://opendev.org/openstack/octavia-tempest-plugin/src/branch/master/octavia_tempest_plugin/tests/scenario/v2/test_traffic_ops.py#L854

Change-Id: Idf0a9fb71178abea732748f92f01eed5a55a3228
This commit is contained in:
Gregory Thiemonge 2023-04-06 09:50:38 +02:00
parent cbda95e727
commit d698a18781
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
octavia_tempest_plugin/tests/validators.py
View File

@ -412,7 +412,7 @@ class ValidatorsMixin(test.BaseTestCase):
return return
except Exception: except Exception:
LOG.warning('Server is not passing initial traffic. Waiting.') LOG.warning('Server is not passing initial traffic. Waiting.')
time.sleep(1) time.sleep(request_interval)
LOG.debug('Loadbalancer wait for load balancer response totals: %s', LOG.debug('Loadbalancer wait for load balancer response totals: %s',
response_counts) response_counts)