Skip to content

TRT-2549: Revert "Add a serial e2e for accept-risks"#1319

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
neisw:revert-1310-accept-risks-e2e-serial
Feb 16, 2026
Merged

TRT-2549: Revert "Add a serial e2e for accept-risks"#1319
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
neisw:revert-1310-accept-risks-e2e-serial

Conversation

@neisw
Copy link
Contributor

@neisw neisw commented Feb 16, 2026

Reverts #1310

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

Multiple payload failures due to console-operator panic

To unrevert this, revert this PR, and layer an additional separate commit on top that addresses the problem. Before merging the unrevert, please run these jobs on the PR and check the result of these jobs to confirm the fix has corrected the problem:

/payload-job periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-ovn-serial-1of2

CC: @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 Feb 16, 2026
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Feb 16, 2026

@neisw: This pull request references OTA-1546 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 story to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Reverts #1310

Want to test recent serial test failures

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 Feb 16, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

This PR removes exported constants from the external and internal packages that provided defaults for Cluster Version Operator configuration, replacing them with hardcoded values in affected files. It also simplifies conformance suite qualifier logic by removing checks for "Serial" labels, deletes an integration test file, removes test utility functions, and updates a test manifest entry.

Changes

Cohort / File(s) Summary
Manifest & Conformance Configuration
.openshift-tests-extension/openshift_payload_cluster-version-operator.json, cmd/cluster-version-operator-tests/main.go
Removed a test payload entry; simplified parallel and serial conformance suite qualifiers by eliminating "Serial" label checks.
Constants Package Removals
pkg/external/constants.go, pkg/internal/constants.go
Deleted exported constants (DefaultCVONamespace, DefaultClusterVersionName, DefaultDeploymentName, DefaultContainerName) that defined CVO defaults.
Hardcoded Constant Replacements
lib/resourcebuilder/apps.go, pkg/cvo/sync_worker.go, pkg/start/start.go, test/cvo/cvo.go
Replaced external/internal constant references with hardcoded strings ("openshift-cluster-version", "version", "cluster-version-operator"); added new local constants in pkg/start and test/cvo.
Test File Deletions
test/cvo/accept_risks.go, test/util/util.go
Removed full integration test file exercising AcceptRisks flow; deleted public utility functions (IsTechPreviewNoUpgrade, SkipIfNotTechPreviewNoUpgrade) and FauxinnatiAPIURL constant.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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

@neisw
Copy link
Contributor Author

neisw commented Feb 16, 2026

/payload-job periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-ovn-serial-1of2

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 16, 2026

@neisw: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-ovn-serial-1of2

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/60bd01a0-0ad4-11f1-8f07-aebdfbabbd0d-0

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Feb 16, 2026

@neisw: This pull request references OTA-1546 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 story to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Reverts #1310

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

Multiple payload failures due to console-operator panic

To unrevert this, revert this PR, and layer an additional separate commit on top that addresses the problem. Before merging the unrevert, please run these jobs on the PR and check the result of these jobs to confirm the fix has corrected the problem:

/payload-job periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-ovn-serial-1of2

CC: @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.

@neisw neisw changed the title Revert "OTA-1546: Add a serial e2e for accept-risks" TRT-2549: Revert "Add a serial e2e for accept-risks" Feb 16, 2026
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Feb 16, 2026

@neisw: This pull request references TRT-2549 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 ticket to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Reverts #1310

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

Multiple payload failures due to console-operator panic

To unrevert this, revert this PR, and layer an additional separate commit on top that addresses the problem. Before merging the unrevert, please run these jobs on the PR and check the result of these jobs to confirm the fix has corrected the problem:

/payload-job periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-ovn-serial-1of2

CC: @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.

@neisw
Copy link
Contributor Author

neisw commented Feb 16, 2026

/override ci/prow/e2e-agnostic-ovn-upgrade-out-of-change
/override ci/prow/e2e-agnostic-ovn-techpreview-serial
/override ci/prow/e2e-agnostic-operator

deprovision, timeout and watch pod failures but all tests passed.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 16, 2026

@neisw: Overrode contexts on behalf of neisw: ci/prow/e2e-agnostic-operator, ci/prow/e2e-agnostic-ovn-techpreview-serial, ci/prow/e2e-agnostic-ovn-upgrade-out-of-change

Details

In response to this:

/override ci/prow/e2e-agnostic-ovn-upgrade-out-of-change
/override ci/prow/e2e-agnostic-ovn-techpreview-serial
/override ci/prow/e2e-agnostic-operator

deprovision, timeout and watch pod failures but all tests passed.

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.

@neisw
Copy link
Contributor Author

neisw commented Feb 16, 2026

/test e2e-hypershift-conformance

@neisw
Copy link
Contributor Author

neisw commented Feb 16, 2026

/override ci/prow/e2e-hypershift-conformance

unrelated flakes

@neisw
Copy link
Contributor Author

neisw commented Feb 16, 2026

/verified by payload-job

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 16, 2026

@neisw: Overrode contexts on behalf of neisw: ci/prow/e2e-hypershift-conformance

Details

In response to this:

/override ci/prow/e2e-hypershift-conformance

unrelated flakes

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-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Feb 16, 2026
@openshift-ci-robot
Copy link
Contributor

@neisw: This PR has been marked as verified by payload-job.

Details

In response to this:

/verified by payload-job

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.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 16, 2026

@neisw: all tests passed!

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.

@xueqzhan
Copy link

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Feb 16, 2026
@hongkailiu
Copy link
Member

/lgtm

@hongkailiu
Copy link
Member

/approve

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 16, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hongkailiu, neisw, xueqzhan

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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 16, 2026
@openshift-merge-bot openshift-merge-bot bot merged commit 27d70c3 into openshift:main Feb 16, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants