[CCIP-2944] Merge CCIP E2E Tests and CI into Core Repo #14102
Conversation
* gitignore * Fix gitignore * Adds CCIP Tests Dir * Adds CCIP CI
…14070) * Fixes CI conflicts * Fixes config path * Separate CCIP smoke tests * Change repo for compatibility tests * Remove references to separate CCIP repo * Callout needed updates * Remove CCIP Upgrade Test --------- Co-authored-by: asoliman <[email protected]>
…ccip/merge-e2e-tests-and-ci
| workflow_run: | ||
| workflows: [ CCIP Load Test ] | ||
| types: [ completed ] | ||
| branches: [ ccip-develop ] |
There was a problem hiding this comment.
Nice catch, updated it elsewhere too.
…ccip/merge-e2e-tests-and-ci
| hostname: ${{ secrets.GRAFANA_INTERNAL_HOST }} | ||
| this-job-name: ETH Smoke Tests CCIP | ||
| matrix-aggregator-status: ${{ needs.eth-smoke-tests-matrix-ccip.result }} | ||
| continue-on-error: true |
There was a problem hiding this comment.
is there a reason for not making it part of eth-smoke-tests job? Is it so that we could skip them and exclude from required check?
There was a problem hiding this comment.
CCIP tests shouldn't be run on every single PR. We can gate them to run only on changes in the ccip directories. For now I'm leaving them running on each PR to stabilize this transition, but will separate them more clearly soon.
| else | ||
| echo "run_command=./smoke/${{ matrix.product.name }}_test.go" >> "$GITHUB_OUTPUT" | ||
| echo "run_command=./${dir}/${{ matrix.product.name }}_test.go" >> "$GITHUB_OUTPUT" |
There was a problem hiding this comment.
Yes, efforts there are tracked in TT-1441
| return func(c *chainlink.Config) { | ||
| c.EVM = evmConfigs | ||
| } | ||
| } |
There was a problem hiding this comment.
maybe we don't need this file and we could just use https://github.com/smartcontractkit/chainlink/blob/develop/integration-tests/types/config/node/core.go?
There was a problem hiding this comment.
Good callout, adding to TT-1441
| uses: smartcontractkit/chainlink-github-actions/chainlink-testing-framework/show-test-summary@75a9005952a9e905649cfb5a6971fd9429436acd # v2.3.25 | ||
|
|
||
| eth-smoke-tests-matrix-ccip: | ||
| if: ${{ !contains(join(github.event.pull_request.labels.*.name, ' '), 'skip-smoke-tests') }} |
There was a problem hiding this comment.
that's what we will use so that passing of this job is not required?
There was a problem hiding this comment.
Correct. We can require the CCIP tests separately.
…ccip/merge-e2e-tests-and-ci
|
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
This PR merges CCIP E2E tests and CI flows into the core repo. Some CI flows were removed as they have to do with releases on the new repo which haven't been solidified yet. There's more work to be done that is being tracked in TT-1441, but this should give us bare functionality for CCIP E2E tests in core.