chore(console): remove e2e-cypress from 4.23/5.0+ - #85422
Conversation
Now that the cypress-based helm tests have been migrated to playwright, there are no more useful cypress tests (the remaining devconsole ones fail 100% of the time and already have playwright equivalents). Remove the e2e-cypress job as it no longer provides any value during pre-merge validation.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: ⛔ Files ignored due to path filters (5)
📒 Files selected for processing (5)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. WalkthroughThe five console CI configurations retain the Playwright browser path, remove the Cypress install override, and remove the optional ChangesConsole CI configuration cleanup
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The removed Cypress workflow is replaced by retained browser-based e2e coverage configuration in every affected branch. No actionable merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Comment |
openshift-ci
Bot
added
the
approved
|
[REHEARSALNOTIFIER]
A total of 33 jobs have been affected by this change. The above listing is non-exhaustive and limited to 25 jobs. A full list of affected jobs can be found here Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
/retest |
1 similar comment
|
/retest |
|
@logonoff: 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. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: logonoff, shahsahil264 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 |
|
The openshift/release PR has to go before the openshift/console PR, so that /pj-rehearse ack |
|
@logonoff: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
openshift-merge-bot
Bot
added
the
rehearsals-ack
2 participants
Now that the cypress-based helm tests have been migrated to playwright, there are no more useful cypress tests (the remaining devconsole ones fail 100% of the time and already have playwright equivalents).
Remove the e2e-cypress job as it no longer provides any value during pre-merge validation.
Replaces #82685
Summary by CodeRabbit
The OpenShift console CI configurations for releases 4.23, 5.0, 5.1, 5.2, and the main branch no longer run the
e2e-cypresspre-merge job.The changes remove Cypress credentials, dependencies, commands, resources, timeout, and workflow settings. They also remove the obsolete
CYPRESS_INSTALL_BINARY=0setting from the console test images.Playwright browser configuration remains in place.