Skip to content

Fix kuttl test OLSConfig status condition expectations#65

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-lightspeed:mainfrom
lpiwowar:fix-kuttl-test-olsconfig-status
Jan 7, 2026
Merged

Fix kuttl test OLSConfig status condition expectations#65
openshift-merge-bot[bot] merged 1 commit into
openstack-lightspeed:mainfrom
lpiwowar:fix-kuttl-test-olsconfig-status

Conversation

@lpiwowar

@lpiwowar lpiwowar commented Jan 7, 2026

Copy link
Copy Markdown
Contributor

Update the test assertion to expect 'Available' instead of 'Reconciling' for OLSConfig status conditions. The OpenShift Lightspeed operator sets the reason to 'Available' when reconciliation completes successfully, which is the expected final state.

The 'Available' state was added with this PR [1] and thanks to the recent release of new version of the OLS operator we started to see the failure.

[1] openshift/lightspeed-operator#1190

Update the test assertion to expect 'Available' instead of 'Reconciling'
for OLSConfig status conditions. The OpenShift Lightspeed operator sets
the reason to 'Available' when reconciliation completes successfully,
which is the expected final state.

The 'Available' state was added with this PR [1] and thanks to the
recent release of new version of the OLS operator we started to see the
failure.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

[1] openshift/lightspeed-operator#1190

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@openshift-ci openshift-ci Bot requested review from Akrog and umago January 7, 2026 10:01
@openshift-ci

openshift-ci Bot commented Jan 7, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lpiwowar

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

@umago

umago commented Jan 7, 2026

Copy link
Copy Markdown
Contributor

/lgtm

Thanks for the fix!

@openshift-ci openshift-ci Bot added the lgtm label Jan 7, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit 253ee70 into openstack-lightspeed:main Jan 7, 2026
7 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