Merge "Spare amps have no role"

This commit is contained in:
Zuul 2018-06-14 20:35:17 +00:00 committed by Gerrit Code Review
commit bc3eaeb609
1 changed files with 3 additions and 1 deletions

View File

@ -134,12 +134,14 @@ class AmphoraScenarioTest(test_base.LoadBalancerBaseTest):
UUID(amp1[const.ID])
UUID(amp1[const.COMPUTE_ID])
UUID(amp1[const.VRRP_PORT_ID])
self.assertIn(amp1[const.ROLE], const.AMPHORA_ROLES)
self.assertIn(amp1[const.STATUS], const.AMPHORA_STATUSES)
# We might have gotten unassigned/spare amps?
if amp1[const.STATUS] == const.STATUS_ALLOCATED:
UUID(amp1[const.HA_PORT_ID])
UUID(amp1[const.LOADBALANCER_ID])
self.assertIn(amp1[const.ROLE], const.AMPHORA_ROLES)
else:
self.assertIsNone(amp1[const.ROLE])
# Test that all of the fields from the amp list match those from a show
for field in const.SHOW_AMPHORA_RESPONSE_FIELDS: