Skip to content

Conversation

@Jakob-Naucke
Copy link
Contributor

Claimed more than one upon != 1 when it could also be zero. Fix.

@Jakob-Naucke Jakob-Naucke requested a review from alicefr January 7, 2026 15:30
@Jakob-Naucke
Copy link
Contributor Author

This caused some confusion in (even though it does not close) #130.

@openshift-ci
Copy link

openshift-ci bot commented Jan 7, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: alicefr, Jakob-Naucke

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

Claimed more than one upon != 1 when it could also be zero. Fix.

Signed-off-by: Jakob Naucke <[email protected]>
@openshift-ci openshift-ci bot removed the lgtm label Jan 7, 2026
@openshift-ci
Copy link

openshift-ci bot commented Jan 7, 2026

New changes are detected. LGTM label has been removed.

@openshift-ci
Copy link

openshift-ci bot commented Jan 7, 2026

@Jakob-Naucke: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/operator-lifecycle-verify 6da1153 link true /test operator-lifecycle-verify
ci/prow/infra-provision-verify 6da1153 link true /test infra-provision-verify

Full PR test history. Your PR dashboard.

Details

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. I understand the commands that are listed here.

@alicefr alicefr merged commit c0b8261 into trusted-execution-clusters:main Jan 12, 2026
8 of 10 checks passed
@Jakob-Naucke Jakob-Naucke deleted the cluster-count-msg branch January 12, 2026 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants