openstack/octavia-tempest-plugin
Code Issues Proposed changes

Merge "Change pool create scenario test to wait for operating status"

Browse Source
This commit is contained in:
Zuul 2021-01-20 19:54:47 +00:00 committed by Gerrit Code Review
commit 2d19795b84
2 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
octavia_tempest_plugin/tests/api/v2/test_pool.py
View File

@ -443,6 +443,8 @@ class PoolAPITest(test_base.LoadBalancerBaseTest):
const.ONLINE, const.ONLINE,
CONF.load_balancer.build_interval, CONF.load_balancer.build_interval,
CONF.load_balancer.build_timeout) CONF.load_balancer.build_timeout)
else:
self.assertEqual(const.OFFLINE, pool[const.OPERATING_STATUS])
self.assertEqual(pool_name, pool[const.NAME]) self.assertEqual(pool_name, pool[const.NAME])
self.assertEqual(pool_description, pool[const.DESCRIPTION]) self.assertEqual(pool_description, pool[const.DESCRIPTION])

9
octavia_tempest_plugin/tests/scenario/v2/test_pool.py
View File

@ -434,7 +434,6 @@ class PoolScenarioTest(test_base.LoadBalancerBaseTest):
parser.parse(pool[const.CREATED_AT]) parser.parse(pool[const.CREATED_AT])
parser.parse(pool[const.UPDATED_AT]) parser.parse(pool[const.UPDATED_AT])
UUID(pool[const.ID]) UUID(pool[const.ID])
self.assertEqual(const.OFFLINE, pool[const.OPERATING_STATUS])
self.assertEqual(pool_protocol, pool[const.PROTOCOL]) self.assertEqual(pool_protocol, pool[const.PROTOCOL])
self.assertEqual(1, len(pool[const.LOADBALANCERS])) self.assertEqual(1, len(pool[const.LOADBALANCERS]))
self.assertEqual(self.lb_id, pool[const.LOADBALANCERS][0][const.ID]) self.assertEqual(self.lb_id, pool[const.LOADBALANCERS][0][const.ID])
@ -443,6 +442,7 @@ class PoolScenarioTest(test_base.LoadBalancerBaseTest):
self.assertEqual(listener_id, pool[const.LISTENERS][0][const.ID]) self.assertEqual(listener_id, pool[const.LISTENERS][0][const.ID])
else: else:
self.assertEmpty(pool[const.LISTENERS]) self.assertEmpty(pool[const.LISTENERS])
self.assertEqual(const.OFFLINE, pool[const.OPERATING_STATUS])
self.assertEqual(algorithm, pool[const.LB_ALGORITHM]) self.assertEqual(algorithm, pool[const.LB_ALGORITHM])
if session_persistence == const.SESSION_PERSISTENCE_APP_COOKIE: if session_persistence == const.SESSION_PERSISTENCE_APP_COOKIE:
@ -510,6 +510,13 @@ class PoolScenarioTest(test_base.LoadBalancerBaseTest):
self.assertEqual(new_description, pool[const.DESCRIPTION]) self.assertEqual(new_description, pool[const.DESCRIPTION])
self.assertTrue(pool[const.ADMIN_STATE_UP]) self.assertTrue(pool[const.ADMIN_STATE_UP])
self.assertEqual(algorithm, pool[const.LB_ALGORITHM]) self.assertEqual(algorithm, pool[const.LB_ALGORITHM])
if listener_protocol is not None:
pool = waiters.wait_for_status(
self.mem_pool_client.show_pool,
pool[const.ID], const.OPERATING_STATUS,
const.ONLINE,
CONF.load_balancer.build_interval,
CONF.load_balancer.build_timeout)
if session_persistence == const.SESSION_PERSISTENCE_APP_COOKIE: if session_persistence == const.SESSION_PERSISTENCE_APP_COOKIE:
self.assertIsNotNone(pool.get(const.SESSION_PERSISTENCE)) self.assertIsNotNone(pool.get(const.SESSION_PERSISTENCE))