Skip to content

Add a serial e2e for accept-risks#1310

Draft
hongkailiu wants to merge 3 commits intoopenshift:mainfrom
hongkailiu:accept-risks-e2e-serial
Draft

Add a serial e2e for accept-risks#1310
hongkailiu wants to merge 3 commits intoopenshift:mainfrom
hongkailiu:accept-risks-e2e-serial

Conversation

@hongkailiu
Copy link
Member

@hongkailiu hongkailiu commented Jan 30, 2026

No description provided.

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 30, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 30, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@coderabbitai
Copy link

coderabbitai bot commented Jan 30, 2026 •
edited
Loading

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Excluded labels (none allowed) (1)
  • do-not-merge/work-in-progress

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 30, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hongkailiu

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 Jan 30, 2026
@hongkailiu
Copy link
Member Author

hongkailiu commented Feb 2, 2026

/test e2e-agnostic-ovn-techpreview-serial

@hongkailiu
Copy link
Member Author

hongkailiu commented Feb 2, 2026

/test all

@hongkailiu hongkailiu force-pushed the accept-risks-e2e-serial branch from 446b922 to 02c4876 Compare February 2, 2026 04:21
@hongkailiu
Copy link
Member Author

hongkailiu commented Feb 2, 2026

/test e2e-agnostic-ovn-techpreview-serial

1 similar comment
@hongkailiu
Copy link
Member Author

hongkailiu commented Feb 2, 2026

/test e2e-agnostic-ovn-techpreview-serial

@hongkailiu
Copy link
Member Author

hongkailiu commented Feb 2, 2026

/test images

@hongkailiu
Copy link
Member Author

hongkailiu commented Feb 2, 2026

/test all

@hongkailiu
Copy link
Member Author

hongkailiu commented Feb 2, 2026

/test images

@hongkailiu hongkailiu force-pushed the accept-risks-e2e-serial branch from 02c4876 to b84342a Compare February 3, 2026 00:00
@hongkailiu
Copy link
Member Author

hongkailiu commented Feb 3, 2026

/test e2e-agnostic-ovn-techpreview-serial

@hongkailiu
Define constants for external usages
bdd86a3 
`github.com/openshift/cluster-version-operator/pkg/external/constants.go`
seems the authentic place for the constants, but it is private to the
packages in the same root [1]. This change introduces shortcuts to the
constants that are used externally.

[1]. https://pkg.go.dev/cmd/go#hdr-Internal_Directories
@hongkailiu
to drop: Move "serial" to the top
f22af17
@hongkailiu hongkailiu force-pushed the accept-risks-e2e-serial branch from 05dddf5 to f22af17 Compare February 3, 2026 08:08
@hongkailiu
Copy link
Member Author

hongkailiu commented Feb 3, 2026

/test e2e-agnostic-ovn-techpreview-serial

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 3, 2026

@hongkailiu: 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/unit 02c4876 link true /test unit
ci/prow/verify-update 02c4876 link true /test verify-update
ci/prow/e2e-hypershift-conformance 02c4876 link true /test e2e-hypershift-conformance
ci/prow/lint 02c4876 link true /test lint
ci/prow/e2e-agnostic-ovn-upgrade-into-change 02c4876 link true /test e2e-agnostic-ovn-upgrade-into-change
ci/prow/okd-scos-images 02c4876 link true /test okd-scos-images
ci/prow/e2e-agnostic-operator 02c4876 link true /test e2e-agnostic-operator
ci/prow/e2e-agnostic-ovn 02c4876 link true /test e2e-agnostic-ovn
ci/prow/e2e-hypershift 02c4876 link true /test e2e-hypershift
ci/prow/e2e-aws-ovn-techpreview 02c4876 link true /test e2e-aws-ovn-techpreview
ci/prow/e2e-agnostic-ovn-upgrade-out-of-change 02c4876 link true /test e2e-agnostic-ovn-upgrade-out-of-change
ci/prow/e2e-agnostic-ovn-techpreview-serial f22af17 link true /test e2e-agnostic-ovn-techpreview-serial
ci/prow/images 02c4876 link true /test images

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.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@hongkailiu