OCPBUGS-106125: Remove all Cypress references from the codebase - #16883
shahsahil264 wants to merge 10 commits into
Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
@shahsahil264: This pull request references CONSOLE-5232 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 "5.0.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. |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
💤 Files with no reviewable changes (2)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. WalkthroughThe change replaces Cypress references with Playwright guidance and configuration, updates container-security test fixture paths, and removes the legacy builder image configuration, scripts, Dockerfile, and documentation. ChangesTest tooling transition and builder cleanup
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to No concrete merge-blocking issue remains from the available review evidence. 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Comment |
|
/hold |
556f4db to
07e0221
Compare
|
/pipeline required |
|
Scheduling tests matching the |
|
/pipeline required |
|
Scheduling tests matching the |
8ee55d9 to
a00e72c
Compare
|
/test e2e-gcp-console |
|
/jira refresh |
|
@shahsahil264: This pull request references Jira Issue OCPBUGS-123182, which is invalid:
Comment This pull request references Jira Issue OCPBUGS-121974, which is invalid:
Comment 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. |
|
@shahsahil264: This pull request references Jira Issue OCPBUGS-123182, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. This pull request references Jira Issue OCPBUGS-121974, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. 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. |
|
@shahsahil264: This pull request references Jira Issue OCPBUGS-106125, which is valid. 3 validation(s) were run on this bug
The bug has been updated to refer to the pull request using the external bug tracker. 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. |
|
Addressed the builder cleanup review. Removed:
The only remaining |
|
Addressed the ESLint review. Removed the Cypress ESLint overlay from |
ab239de to
a2c6a3e
Compare
|
/pipeline-required |
|
/test e2e-gcp-console |
|
/retest |
|
/test e2e-gcp-console |
|
/test e2e-gcp-console |
|
/test e2e-gcp-console |
|
@shahsahil264: The following tests failed, say
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. |
Analysis / Root cause:
The Cypress CI job is being removed after the migration to Playwright. This PR completes the repository-side cleanup required by OCPBUGS-106125 and CONSOLE-5232.
Solution description:
migrate-cypressskill and consolidated migration guidance into the existing Playwright E2E skill and migration context.PR #17156 separately removes the OLM and Dev Console migration artifacts. Together, the PRs satisfy the complete Cypress removal acceptance criteria.
Acceptance criteria validation:
migration-context.mdand Playwright skill documentation.Screenshots / screen recording:
N/A - infrastructure cleanup only.
Test setup:
No special setup required.
Related issues: