Skip OVN checks in minor update when OVN is disabled #1744
Conversation
Fixes a bug where the minor update sequence would hang when OVN is disabled. The controller was checking OVN conditions that don't exist when OVN is not enabled, causing the update process to stall. Changes: - Add Ovn.Enabled checks before OVN image and condition validation - Skip OVN controlplane and dataplane update checks when disabled - Add comprehensive functional test for minor updates with OVN disabled Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> Jira: OSPRH-23103
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dprince The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
when this lands on main we need to propose a backport to FR4 |
|
/test openstack-operator-build-deploy-kuttl |
1 similar comment
|
/test openstack-operator-build-deploy-kuttl |
|
/lgtm |
|
/cherry-pick 18.0-fr4 |
|
@lmiccini: once the present PR merges, I will cherry-pick it on top of DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
openshift-merge-bot
bot
merged commit c5c4018
into
openstack-k8s-operators:main
|
@lmiccini: new pull request created: #1749 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/lgtm also did a testpatch for non ovn minor update and went fine |
4 participants
Fixes a bug where the minor update sequence would hang when OVN is disabled. The controller was checking OVN conditions that don't exist when OVN is not enabled, causing the update process to stall.
Changes:
Co-Authored-By: Claude Sonnet 4.5 noreply@anthropic.com
Jira: OSPRH-23103