Skip to content

[cifmw_backup_restore] Add Octavia OVN sync after Neutron sync #4045

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
stuggi wants to merge 1 commit into
base: main
Choose a base branch
from
+32 −2
Open

[cifmw_backup_restore] Add Octavia OVN sync after Neutron sync #4045

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 32 additions & 2 deletions roles/cifmw_backup_restore/tasks/restore.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -597,7 +597,37 @@
changed_when: true

# ========================================
# Step 11: Restore DataPlane (Order 60)
# Step 11: Sync Octavia to OVN (optional)
# ========================================
# Only when Octavia with OVN provider is deployed. Repopulates OVN NB DB
# with load balancer entries from Octavia's database.
# Ref: https://docs.openstack.org/ovn-octavia-provider/latest/admin/driver.html#octavia-db-to-ovn-database-population
- name: Check if Octavia is enabled
ansible.builtin.shell: |
oc get openstackcontrolplane {{ _ctlplane_name.stdout }} \
-n {{ cifmw_backup_restore_namespace }} \
-o jsonpath='{.spec.octavia.enabled}'
register: _octavia_enabled
changed_when: false

- name: Sync Octavia load balancers to OVN
ansible.builtin.shell: |
oc exec -n {{ cifmw_backup_restore_namespace }} \
-c octavia-api deploy/octavia-api -- \
octavia-ovn-db-sync-util
register: _octavia_ovn_sync
when: _octavia_enabled.stdout == 'true'
changed_when: true

- name: Report Octavia OVN sync result
ansible.builtin.debug:
msg: >-
Octavia OVN sync:
{{ 'completed' if _octavia_enabled.stdout == 'true' else 'skipped (Octavia not enabled)' }}.
when: _octavia_enabled.stdout == 'true' or not (_octavia_enabled.stdout == 'true')

@michburk michburk Jul 9, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when condition is always true, could be removed


# ========================================
# Step 12: Restore DataPlane (Order 60)
# ========================================
- name: Render dataplane restore
ansible.builtin.template:
Expand All @@ -620,7 +650,7 @@
_step_name: "Step 11 (DataPlane restore)"

@michburk michburk Jul 9, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like the number in this _step_name is out of sync now,
Step 11 -> Step 12


# ========================================
# Step 12: EDPM Deployment
# Step 13: EDPM Deployment
# ========================================
- name: Get DataPlaneNodeSet names
ansible.builtin.shell: |
Expand Down
Loading