When 2 amps were down, the failover flow created the first one and
needed to update both amp to configure VRRP, but as the 2nd was missing,
it was set to ERROR. Then the health-manager could not trigger a
failover becasue amphorae in ERROR are excluded from the automated
failover process.
This commit changes the tasks that must be run on both amphorae during a
failover of one amphora, it doesn't mark the secondary amphora in ERROR
if it is not reachable.
Closes-Bug: #2033734
Note: stable/2023.1 and older, the patch also includes modifications in octavia/controller/worker/v1/
Reviewed: https:/ /review. opendev. org/c/openstack /octavia/ +/898114 /opendev. org/openstack/ octavia/ commit/ 96b17f6108453c6 aa0e4776283493a c517fa43c5
Committed: https:/
Submitter: "Zuul (22348)"
Branch: stable/wallaby
commit 96b17f6108453c6 aa0e4776283493a c517fa43c5
Author: Gregory Thiemonge <email address hidden>
Date: Mon Sep 4 05:56:25 2023 -0400
Fix amphorae in ERROR during the failover
When 2 amps were down, the failover flow created the first one and
needed to update both amp to configure VRRP, but as the 2nd was missing,
it was set to ERROR. Then the health-manager could not trigger a
failover becasue amphorae in ERROR are excluded from the automated
failover process.
This commit changes the tasks that must be run on both amphorae during a
failover of one amphora, it doesn't mark the secondary amphora in ERROR
if it is not reachable.
Closes-Bug: #2033734
Note: stable/2023.1 and older, the patch also includes modifications in
octavia/ controller/ worker/ v1/
Conflicts:
octavia/ controller/ worker/ v2/tasks/ amphora_ driver_ tasks.py
Change-Id: I4bd027346c61b9 3b537ab53810c2e cb6160b6be2 74d870a763802a5 1586e0f63b) b8b71ad8fdb2d74 4f94625ec4) 9ec1ae3de511b3a 84b8324e04) 7930fa2f8ef6794 de5417c2c0) 345f6ac92abb27a e4282847d3) 2a15ddb4a4b8ac1 dd5b5a3a98)
(cherry picked from commit 248cf2893ea913a
(cherry picked from commit 5622431534d3bf9
(cherry picked from commit 10cfd8c7605e778
(cherry picked from commit d09065c4f6c08b0
(cherry picked from commit 488adb29b68074d
(cherry picked from commit 2a71565107657a0