openstack
/
octavia-tempest-plugin
Code Issues Proposed changes

Fix the pool client auth test 

There was an intermittent test failure in the
test_pool_reencryption_client_authentication test where members are detected
as unbalanced. This was due to the time it takes for the health monitor to
bring the member back into the pool.
This change adds a waiter for the members to go operating_status ONLINE before
checking for balanced member requests.

Story: 2010660
Task: 47710
Change-Id: Ibb1f351c77f3ed7a0c69c8295973f77ab2069dad
This commit is contained in:
Michael Johnson 2023-03-24 17:18:52 +00:00
parent 5b73479a4d
commit 554873fc00
1 changed files with 15 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
octavia_tempest_plugin/tests/barbican_scenario/v2/test_tls_barbican.py
View File

@ -1471,7 +1471,7 @@ class TLSWithBarbicanTest(test_base.LoadBalancerBaseTestWithCompute):
if self.lb_member_1_subnet:
member1_kwargs[const.SUBNET_ID] = self.lb_member_1_subnet[const.ID]
self.mem_member_client.create_member(**member1_kwargs)
member1 = self.mem_member_client.create_member(**member1_kwargs)
waiters.wait_for_status(
self.mem_lb_client.show_loadbalancer, self.lb_id,
const.PROVISIONING_STATUS, const.ACTIVE,
@ -1491,7 +1491,7 @@ class TLSWithBarbicanTest(test_base.LoadBalancerBaseTestWithCompute):
if self.lb_member_2_subnet:
member2_kwargs[const.SUBNET_ID] = self.lb_member_2_subnet[const.ID]
self.mem_member_client.create_member(**member2_kwargs)
member2 = self.mem_member_client.create_member(**member2_kwargs)
waiters.wait_for_status(
self.mem_lb_client.show_loadbalancer, self.lb_id,
const.PROVISIONING_STATUS, const.ACTIVE,
@ -1538,6 +1538,19 @@ class TLSWithBarbicanTest(test_base.LoadBalancerBaseTestWithCompute):
CONF.load_balancer.check_interval,
CONF.load_balancer.check_timeout)
# Make sure the health monitor has brought the members up after the
# the pool update.
waiters.wait_for_status(
self.mem_member_client.show_member, member1[const.ID],
const.OPERATING_STATUS, const.ONLINE,
CONF.load_balancer.check_interval,
CONF.load_balancer.check_timeout, error_ok=True, pool_id=pool_id)
waiters.wait_for_status(
self.mem_member_client.show_member, member2[const.ID],
const.OPERATING_STATUS, const.ONLINE,
CONF.load_balancer.check_interval,
CONF.load_balancer.check_timeout, error_ok=True, pool_id=pool_id)
self.check_members_balanced(self.lb_vip_address, protocol=const.HTTP,
protocol_port=85)