Skip to content

Commit 5ead8e6

Browse files
Merge pull request #1372 from gthiemonge/fix_octavia_failover
octavia-adoption: perform failover only on amphora LBs
2 parents 75b848b + 9597ca9 commit 5ead8e6

2 files changed

Lines changed: 6 additions & 3 deletions

File tree

docs_user/modules/proc_performing-post-adoption-cleanup-of-load-balancers.adoc

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,15 @@ $ alias openstack="oc exec -t openstackclient -- openstack"
2121
$ openstack loadbalancer create --vip-subnet-id public-subnet --name lb-post-adoption --wait
2222
----
2323

24-
. Trigger a failover for all existing load balancers to upgrade the amphorae virtual machines to use the new image and to establish connectivity with the new control plane:
24+
. Trigger a failover for all existing amphora load balancers to upgrade the amphorae virtual machines to use the new image and to establish connectivity with the new control plane:
2525
+
2626
----
27-
$ openstack loadbalancer list -f value -c id | \
27+
$ openstack loadbalancer list -f value -c id --provider amphora | \
2828
xargs -r -n1 -P4 ${BASH_ALIASES[openstack]} loadbalancer failover --wait
2929
----
30+
+
31+
[NOTE]
32+
Failover does not apply to OVN load balancers. As native components of the OVN infrastructure, they do not require actions during upgrades and updates.
3033

3134
. Delete old flavors that were migrated to the new control plane:
3235
+

tests/roles/dataplane_adoption/tasks/octavia_post.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@
4343
ansible.builtin.shell: |
4444
{{ shell_header }}
4545
{{ octavia_header }}
46-
${BASH_ALIASES[openstack]} loadbalancer list -f value -c id --not-tag new | \
46+
${BASH_ALIASES[openstack]} loadbalancer list -f value -c id --not-tag new --provider amphora | \
4747
xargs -r -n1 -P4 ${BASH_ALIASES[openstack]} loadbalancer failover --wait
4848
4949
- name: check for existing load balancer

0 commit comments

Comments
 (0)