octavia-adoption: perform failover only on amphora LBs#1372
Conversation
During our tests, tobiko creates an ovn load balancer in the source cloud. ovn load balancers don't support failovers so they need to be skipped in the "perform failover of existing load balancers" task. Update the docs accordingly.
|
This PR is stale because it has been for over 15 days with no activity. |
|
@gthiemonge Are you planing to land the change? It is marked stale and you have a suggestion from Katie after review to be addressed. |
Yeah, I'm still working on it, I'm testing it in the CI |
Ack, thank you for confirming |
Co-authored-by: Katie Gilligan <kgilliga@redhat.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
| ---- | ||
| + | ||
| [NOTE] | ||
| OVN load balancers do not support failover because they do not need to be updated during upgrades. |
There was a problem hiding this comment.
I'm a bit confused by this sentence. During an upgrade, is it optional to update OVN load balancers? And what is the distinction between "update" and "upgrade" in this context?
4 participants
During our tests, tobiko creates an ovn load balancer in the source cloud.
ovn load balancers don't support failovers so they need to be skipped in the "perform failover of existing load balancers" task. Update the docs accordingly.