Fix amphora failover flow in amphorav2 driver

Remove amphora_id from the failover flow parameters in amphorav2, it is
unused and it hides in a later task the amphora_id of the newly created
amphora, breaking a simple amphora failover.

Story 2008848
Task 42360

Change-Id: Idd9021c8a73d1be87f5df40d3f27c0fba9067e3c
(cherry picked from commit 7e2dab64a0)
This commit is contained in:
Gregory Thiemonge 2021-04-23 07:09:09 +02:00 committed by Carlos Goncalves
parent 46979a0d1b
commit 19504cef8d
3 changed files with 13 additions and 16 deletions

View File

@ -945,8 +945,7 @@ class ControllerWorker(object):
constants.LOADBALANCER: provider_lb_dict, constants.LOADBALANCER: provider_lb_dict,
constants.SERVER_GROUP_ID: server_group_id, constants.SERVER_GROUP_ID: server_group_id,
constants.LOADBALANCER_ID: lb_id, constants.LOADBALANCER_ID: lb_id,
constants.VIP: vip_dict, constants.VIP: vip_dict}
constants.AMPHORA_ID: amphora_id}
self.run_flow( self.run_flow(
flow_utils.get_failover_amphora_flow, flow_utils.get_failover_amphora_flow,

View File

@ -1270,8 +1270,7 @@ class TestControllerWorker(base.TestCase):
constants.LOADBALANCER: mock_provider_lb.to_dict(), constants.LOADBALANCER: mock_provider_lb.to_dict(),
constants.LOADBALANCER_ID: LB_ID, constants.LOADBALANCER_ID: LB_ID,
constants.SERVER_GROUP_ID: None, constants.SERVER_GROUP_ID: None,
constants.VIP: mock_lb.vip.to_dict(), constants.VIP: mock_lb.vip.to_dict()}
constants.AMPHORA_ID: AMP_ID}
cw = controller_worker.ControllerWorker() cw = controller_worker.ControllerWorker()
cw.services_controller.reset_mock() cw.services_controller.reset_mock()
@ -1326,8 +1325,7 @@ class TestControllerWorker(base.TestCase):
constants.LOADBALANCER: mock_provider_lb.to_dict(), constants.LOADBALANCER: mock_provider_lb.to_dict(),
constants.LOADBALANCER_ID: LB_ID, constants.LOADBALANCER_ID: LB_ID,
constants.SERVER_GROUP_ID: None, constants.SERVER_GROUP_ID: None,
constants.VIP: mock_lb.vip.to_dict(), constants.VIP: mock_lb.vip.to_dict()}
constants.AMPHORA_ID: AMP_ID}
cw = controller_worker.ControllerWorker() cw = controller_worker.ControllerWorker()
cw.services_controller.reset_mock() cw.services_controller.reset_mock()
@ -1382,8 +1380,7 @@ class TestControllerWorker(base.TestCase):
constants.LOADBALANCER: mock_provider_lb.to_dict(), constants.LOADBALANCER: mock_provider_lb.to_dict(),
constants.LOADBALANCER_ID: LB_ID, constants.LOADBALANCER_ID: LB_ID,
constants.SERVER_GROUP_ID: SERVER_GROUP_ID, constants.SERVER_GROUP_ID: SERVER_GROUP_ID,
constants.VIP: mock_lb.vip.to_dict(), constants.VIP: mock_lb.vip.to_dict()}
constants.AMPHORA_ID: AMP_ID}
cw = controller_worker.ControllerWorker() cw = controller_worker.ControllerWorker()
cw.services_controller.reset_mock() cw.services_controller.reset_mock()
@ -1435,8 +1432,7 @@ class TestControllerWorker(base.TestCase):
constants.LOADBALANCER: mock_provider_lb.to_dict(), constants.LOADBALANCER: mock_provider_lb.to_dict(),
constants.LOADBALANCER_ID: LB_ID, constants.LOADBALANCER_ID: LB_ID,
constants.SERVER_GROUP_ID: SERVER_GROUP_ID, constants.SERVER_GROUP_ID: SERVER_GROUP_ID,
constants.VIP: mock_lb.vip.to_dict(), constants.VIP: mock_lb.vip.to_dict()}
constants.AMPHORA_ID: AMP_ID}
cw = controller_worker.ControllerWorker() cw = controller_worker.ControllerWorker()
cw.services_controller.reset_mock() cw.services_controller.reset_mock()
@ -1492,8 +1488,7 @@ class TestControllerWorker(base.TestCase):
constants.LOADBALANCER: mock_provider_lb.to_dict(), constants.LOADBALANCER: mock_provider_lb.to_dict(),
constants.LOADBALANCER_ID: LB_ID, constants.LOADBALANCER_ID: LB_ID,
constants.SERVER_GROUP_ID: None, constants.SERVER_GROUP_ID: None,
constants.VIP: mock_lb.vip.to_dict(), constants.VIP: mock_lb.vip.to_dict()}
constants.AMPHORA_ID: AMP_ID}
mock_get_flavor_meta.return_value = {'taste': 'spicy'} mock_get_flavor_meta.return_value = {'taste': 'spicy'}
cw = controller_worker.ControllerWorker() cw = controller_worker.ControllerWorker()
@ -1549,8 +1544,7 @@ class TestControllerWorker(base.TestCase):
constants.LOADBALANCER: mock_provider_lb.to_dict(), constants.LOADBALANCER: mock_provider_lb.to_dict(),
constants.LOADBALANCER_ID: LB_ID, constants.LOADBALANCER_ID: LB_ID,
constants.SERVER_GROUP_ID: None, constants.SERVER_GROUP_ID: None,
constants.VIP: mock_lb.vip.to_dict(), constants.VIP: mock_lb.vip.to_dict()}
constants.AMPHORA_ID: AMP_ID}
mock_get_az_meta.return_value = {'planet': 'jupiter'} mock_get_az_meta.return_value = {'planet': 'jupiter'}
cw = controller_worker.ControllerWorker() cw = controller_worker.ControllerWorker()
@ -1666,8 +1660,7 @@ class TestControllerWorker(base.TestCase):
constants.LOADBALANCER: None, constants.LOADBALANCER: None,
constants.LOADBALANCER_ID: None, constants.LOADBALANCER_ID: None,
constants.SERVER_GROUP_ID: None, constants.SERVER_GROUP_ID: None,
constants.VIP: {}, constants.VIP: {}}
constants.AMPHORA_ID: AMP_ID}
cw = controller_worker.ControllerWorker() cw = controller_worker.ControllerWorker()
cw.services_controller.reset_mock() cw.services_controller.reset_mock()

View File

@ -0,0 +1,5 @@
---
fixes:
- |
Fix an issue with amphorav2 driver, a failover of an amphora created an
amphora with an ERROR status.