Skip to content

[cifmw_backup_restore] Move Neutron-OVN sync before EDPM deployment#4026

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
stuggi:br_update_ovn
Jul 9, 2026
Merged

[cifmw_backup_restore] Move Neutron-OVN sync before EDPM deployment#4026
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
stuggi:br_update_ovn

Conversation

@stuggi

@stuggi stuggi commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Run neutron-ovn-db-sync-util after the control plane is ready (Step 10) instead of after the EDPM deployment. This ensures OVN databases are populated before ovn-controller on dataplane nodes reconnects, preventing data plane outage from stale or empty databases.

Matches the reordering in dev-docs backup-restore user-guide.md.

Also fixes the selector for the node-agent pods.

@karelyatin

Copy link
Copy Markdown
Contributor

/lgtm
need to clear
ERROR: Commit message must start with:
(multiple)

Run neutron-ovn-db-sync-util after the control plane is ready (Step 10)
instead of after the EDPM deployment. This ensures OVN databases are
populated before ovn-controller on dataplane nodes reconnects, preventing
data plane outage from stale or empty databases.

Matches the reordering in dev-docs backup-restore user-guide.md.

Also fixes the selector for the node-agent pods.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Martin Schuppert <mschuppert@redhat.com>

@michburk michburk left a comment

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.

/lgtm

@abays abays left a comment

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.

/lgtm

@michburk

michburk commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

/approve

@openshift-ci

openshift-ci Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: michburk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@stuggi

stuggi commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

/retest

2 similar comments
@stuggi

stuggi commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

/retest

@stuggi

stuggi commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

/retest

@openshift-merge-bot openshift-merge-bot Bot merged commit 9505890 into openstack-k8s-operators:main Jul 9, 2026
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants