Skip to content

Conversation

@stuggi
Copy link
Contributor

@stuggi stuggi commented Jan 9, 2026

Add validation to ensure the subscription's installedCSV matches currentCSV after the installplan is applied. This confirms the operator upgrade has completed successfully before proceeding with the rest of the update workflow. Uses label selector to find the subscription rather than hardcoding the name.

Add missing wait for OpenStackControlPlane to reach ready state after operator updates in OLM mode. Include a 60-second delay before the wait to allow the control plane's ready condition to transition from its initial state, preventing premature completion while the control plane is still reconciling the operator changes.
This step is important to validate that the current deployed control plane can successfully reach ready state with the new set of operators, but using the old service containers. Otherwise we start alreay updating to the new service images.

Jira: OSPRH-23688

Add validation to ensure the subscription's installedCSV matches
currentCSV after the installplan is applied. This confirms the operator
upgrade has completed successfully before proceeding with the rest
of the update workflow. Uses label selector to find the subscription
rather than hardcoding the name.

Add missing wait for OpenStackControlPlane to reach ready state after
operator updates in OLM mode. Include a 60-second delay before the
wait to allow the control plane's ready condition to transition from
its initial state, preventing premature completion while the control
plane is still reconciling the operator changes.
This step is important to validate that the current deployed control
plane can successfully reach ready state with the new set of operators,
but using the old service containers. Otherwise we start alreay updating
to the new service images.

Jira: OSPRH-23688

Co-Authored-By: Claude <[email protected]>
Signed-off-by: Martin Schuppert <[email protected]>
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 9, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign jistr for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

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.

1 participant