chore(ci): cleanup tests referencing integration-tests/ccips-tests/*#22161
chore(ci): cleanup tests referencing integration-tests/ccips-tests/*#22161
Conversation
product-security-plaid-production
Bot
requested review from
ChrisAmora and
kalverra
|
✅ No conflicts with other open PRs targeting |
There was a problem hiding this comment.
Pull request overview
This PR removes legacy E2E test matrix entries that referenced the deleted integration-tests/ccip-tests/... directory, to stop CI from attempting to execute tests whose sources no longer exist.
Changes:
- Deletes the entire “CCIP tests” block from the E2E runner matrix in
.github/e2e-tests.yml. - Eliminates all
integration-tests/ccip-tests/...path references from that matrix file.
| CCIP_V16_TEST_ENV: docker | ||
| test_go_project_path: integration-tests | ||
| # END: CCIPv1.6 tests |
There was a problem hiding this comment.
After removing the legacy integration-tests/ccip-tests/... entries, this matrix no longer contains any test entries with the triggers PR E2E CCIP Tests, Merge Queue E2E CCIP Tests, E2E CCIP Chaos Tests, or E2E CCIP Load Tests. However, existing workflows (e.g. .github/workflows/integration-tests.yml, ccip-chaos-tests.yml, ccip-load-tests.yml) still invoke run-e2e-tests.yml with these triggers. This likely results in the CCIP jobs running zero tests (or potentially failing, depending on how run-e2e-tests.yml handles an empty selection). Consider either (a) updating remaining CCIPv1.6 test entries to include the relevant CCIP triggers, or (b) updating/disabling the workflows to stop using triggers that no longer exist in this matrix.
|
fixed by #22118 |
Pull request was closed
Cleans up all test entries relating to the
integration-tests/ccip-testspath which no longer exists (deleted in 652a07d / #22039 ).These tests are running in merge queue and constantly failing now that the source for them is gone, so we should delete them.