Skip to content

Conversation

@neisw
Copy link

@neisw neisw commented Jan 18, 2026

Reverts #1279

Tracked in TRT-2511

Per OpenShift policy, we are reverting this breaking change to get CI and/or nightly payloads flowing again.

Multiple payload failures due to microshift failures for cluster-version-operator should have correct runlevel and scc`

To unrevert this, revert this PR, and layer an additional separate commit on top that addresses the problem. Payload testing microshift may not be possible in your PR please review the failure and address the issue.

CC: @jiajliu @hongkailiu

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jan 18, 2026
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jan 18, 2026

@neisw: This pull request references OTA-1604 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Reverts #1279

Tracked in TRT-2511

Per OpenShift policy, we are reverting this breaking change to get CI and/or nightly payloads flowing again.

Multiple payload failures due to microshift failures for cluster-version-operator should have correct runlevel and scc`

To unrevert this, revert this PR, and layer an additional separate commit on top that addresses the problem. Payload testing microshift may not be possible in your PR please review the failure and address the issue.

CC: @jiajliu @hongkailiu

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 openshift-eng/jira-lifecycle-plugin repository.

@coderabbitai
Copy link

coderabbitai bot commented Jan 18, 2026

Walkthrough

The changes remove cluster-dependent test code and utilities from the CVO test suite. A test entry is deleted from the extension manifest, a large live-cluster test block is removed from cvo.go, and the entire util.go file containing Kubernetes and OpenShift client initialization functions is deleted.

Changes

Cohort / File(s) Summary
Test Manifest Configuration
.openshift-tests-extension/openshift_payload_cluster-version-operator.json
Removed one test entry ("cluster-version-operator should have correct runlevel and scc") from the payload; all other entries preserved
Live-Cluster Test Code
test/cvo/cvo.go
Removed BeforeEach setup block and live-cluster test that verified namespace run-level label and CVO pod SCC annotation; eliminated public constant cvoNamespace and Kubernetes-related imports; retained only non-cluster-dependent OC client test
Removed Utility Module
test/cvo/util.go
Entire file deleted; removed helper functions for HyperShift detection, conditional test skipping, and Kubernetes/OpenShift client initialization (IsHypershift, SkipIfHypershift, GetRestConfig, GetKubeClient, GetConfigClient)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

✨ Finishing touches
  • 📝 Generate docstrings

Comment @coderabbitai help to get the list of available commands and usage tips.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 18, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: neisw
Once this PR has been reviewed and has the lgtm label, please assign davidhurta 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

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 18, 2026

@neisw: The following test 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/e2e-agnostic-ovn 278da7b link true /test e2e-agnostic-ovn

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.

@neisw
Copy link
Author

neisw commented Jan 18, 2026

/retest-required

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants