NO-JIRA: Promote stable Tech Preview e2e tests into the default e2e-operator suite#382
NO-JIRA: Promote stable Tech Preview e2e tests into the default e2e-operator suite#382bharath-b-rh wants to merge 1 commit intoopenshift:masterfrom
Conversation
Signed-off-by: Bharath B <bhb@redhat.com>
|
@bharath-b-rh: This pull request explicitly references no jira issue. DetailsIn response to this:
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. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (7)
WalkthroughThe PR updates test labeling across multiple e2e test files by removing TechPreview labels from certain tests and introducing Platform:None labels to additional test suites. The Makefile's E2E_GINKGO_LABEL_FILTER is updated to include Platform:None in the AWS platform condition, broadening test selection criteria. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Comment Tip Migrating from UI to YAML configuration.Use the |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bharath-b-rh The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/test e2e-operator-tech-preview |
|
@bharath-b-rh: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. |
Summary
Moves recently added Tech Preview e2e coverage into the default e2e-operator suite so it runs on the main CI path, after several tech-preview runs showed stable behavior.
Rationale
Those scenarios lived under the tech-preview job only while they were new. They’ve now been exercised repeatedly without observed flakiness, so promoting them matches GA-level confidence and avoids splitting “real” coverage behind an optional job.
What changed
Verification