Revert "CNTRLPLANE-2656:Migrating oc cli tests to ote"#2191
Revert "CNTRLPLANE-2656:Migrating oc cli tests to ote"#2191openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
Conversation
|
@neisw: This pull request references CNTRLPLANE-2656 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 sub-task to target the "4.22.0" version, but no target version was set. 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. |
WalkthroughThis change removes a substantial portion of the e2e test suite infrastructure, including large test files with Ginkgo-based tests, comprehensive test utilities, fixture management helpers, and numerous YAML test data files. A minimal placeholder test file is added as a replacement. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~75 minutes ✨ Finishing touches
🧪 Generate unit tests (beta)
Tip 🧪 Unit Test Generation v2 is now available!We have significantly improved our unit test generation capabilities. To enable: Add this to your reviews:
finishing_touches:
unit_tests:
enabled: trueTry it out by using the Have feedback? Share your thoughts on our Discord thread! Comment |
|
/payload-job periodic-ci-openshift-release-master-nightly-4.22-e2e-metal-ipi-ovn-ipv6 |
|
@neisw: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/36420100-fd7d-11f0-8dab-7a75a17d39f2-0 |
|
/lgtm |
|
@ardaguclu: This PR has been marked as verified by 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. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ardaguclu, gangwgr, neisw 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 |
|
@neisw: 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. |
Reverts #2181
I'm seeing presubmit failures with these new tests
e2e-aws-ovn-microshift-serial
e2e-aws-ovn-microshift
e2e-metal-ipi-ovn-ipv6
Will use
-e2e-metal-ipi-ovn-ipv6to verify