Skip to content

Move runtime config reconcilation post service setup#578

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
karelyatin:OSPRH-27984-fix
Jun 19, 2026
Merged

Move runtime config reconcilation post service setup#578
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
karelyatin:OSPRH-27984-fix

Conversation

@karelyatin

@karelyatin karelyatin commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

We need to wait for ovn cluster to be ready and that relies on pods to be in ready state. And for multi replica ovn cluster the pods needs to communicate with each other using dns addresses and those are provided by the headless service.

So moving the runtime config task post setting up the services so cluster can be formed and pods move to the ready state. This issue was catched in the backup/restore exercise [2].

[1] #540
[2] https://github.com/openstack-k8s-operators/dev-docs/blob/main/backup-restore/user-guide.md

Related-Issue: #OSPRH-27985

Comment thread internal/controller/ovndbcluster_controller.go
We need to wait for ovn cluster to be ready and that
relies on pods to be in ready state. And for multi replica
ovn cluster the pods needs to communicate with each other
using dns addresses and those are provided by the headless
service.

So moving the runtime config task post setting up the
services so cluster can be formed and pods move to the ready state.
This issue was catched in the backup/restore exercise [2].

Also adapted getPodIPInNetwork to set networkattachment condition
and accordingly the envtest.

[1] openstack-k8s-operators#540
[2] https://github.com/openstack-k8s-operators/dev-docs/blob/main/backup-restore/user-guide.md

Related-Issue: #OSPRH-27985
Signed-off-by: Yatin Karel <ykarel@redhat.com>

@stuggi stuggi 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

@openshift-ci

openshift-ci Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: karelyatin, stuggi

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

@openshift-merge-bot openshift-merge-bot Bot merged commit 969d107 into openstack-k8s-operators:main Jun 19, 2026
6 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.

2 participants