Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix 4303 #15394

Draft
wants to merge 29 commits into
base: develop
Choose a base branch
from
Draft

Fix 4303 #15394

Changes from 1 commit
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
6ba8f4a
CCIP-4303:Enabling in-memory test in integration-tests workflow
b-gopalswami Nov 22, 2024
73e4131
Fix naming for integration-tests.yml file to e2e-tests.yml
b-gopalswami Nov 22, 2024
f3d428d
Fix yml name
b-gopalswami Nov 22, 2024
935dcdf
fix environment name
b-gopalswami Nov 23, 2024
4134f25
Fix secrets
b-gopalswami Nov 23, 2024
c63be59
Merge branch 'develop' of github.com:smartcontractkit/chainlink into …
b-gopalswami Nov 25, 2024
dc28540
Update test secrets step
b-gopalswami Nov 25, 2024
939b128
Adding setup db step
b-gopalswami Nov 25, 2024
a94fc3d
fix step
b-gopalswami Nov 25, 2024
408b6f5
fix step
b-gopalswami Nov 25, 2024
3fa0084
Merge branch 'develop' of github.com:smartcontractkit/chainlink into …
b-gopalswami Nov 25, 2024
7c8f1b9
Merge branch 'develop' of github.com:smartcontractkit/chainlink into …
b-gopalswami Nov 25, 2024
56fc8d5
Rename the existing integration-tests to e2e-tests and adding new int…
b-gopalswami Nov 25, 2024
ba4e951
Merged base
b-gopalswami Nov 25, 2024
54dcf69
Remove images
b-gopalswami Nov 25, 2024
d61faed
Fix errors
b-gopalswami Nov 25, 2024
13a8b4a
Fix try
b-gopalswami Nov 25, 2024
0b0bb43
Remove cache
b-gopalswami Nov 25, 2024
8f408d2
Remove cache
b-gopalswami Nov 25, 2024
7211f50
Remove cache
b-gopalswami Nov 25, 2024
369a896
Remove cache
b-gopalswami Nov 25, 2024
7998e6e
Remove cache
b-gopalswami Nov 25, 2024
d233cff
Remove cache
b-gopalswami Nov 25, 2024
b452d17
Global pg instance
b-gopalswami Nov 26, 2024
cfb651d
Global pg instance
b-gopalswami Nov 26, 2024
b8a8afc
Merge branch 'develop' of github.com:smartcontractkit/chainlink into …
b-gopalswami Nov 26, 2024
a0f9f98
Final cleanup
b-gopalswami Nov 26, 2024
1abeae7
Merge branch 'develop' of github.com:smartcontractkit/chainlink into …
b-gopalswami Nov 26, 2024
e72adb3
Try another approach
b-gopalswami Nov 26, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove images
b-gopalswami committed Nov 25, 2024
commit 54dcf69b432dbd03cd5b64044d2505b90d840fcc
113 changes: 4 additions & 109 deletions .github/workflows/integration-tests.yml
Original file line number Diff line number Diff line change
@@ -28,49 +28,11 @@ concurrency:

env:
# for run-test variables and environment
ENV_JOB_IMAGE: ${{ secrets.QA_AWS_ACCOUNT_NUMBER }}.dkr.ecr.${{ secrets.QA_AWS_REGION }}.amazonaws.com/chainlink-tests:${{ inputs.evm-ref || github.sha }}
CHAINLINK_IMAGE: ${{ secrets.QA_AWS_ACCOUNT_NUMBER }}.dkr.ecr.${{ secrets.QA_AWS_REGION }}.amazonaws.com/chainlink
TEST_SUITE: smoke
TEST_ARGS: -test.timeout 12m
INTERNAL_DOCKER_REPO: ${{ secrets.QA_AWS_ACCOUNT_NUMBER }}.dkr.ecr.${{ secrets.QA_AWS_REGION }}.amazonaws.com
MOD_CACHE_VERSION: 2
COLLECTION_ID: chainlink-integration-tests

jobs:
enforce-ctf-version:
name: Enforce CTF Version
runs-on: ubuntu-latest
# We don't directly merge dependabot PRs, so let's not waste the resources
if: github.actor != 'dependabot[bot]'
steps:
- run: echo "${{github.event_name}}"
- name: Checkout the repo
uses: actions/checkout@v4.2.1
with:
repository: smartcontractkit/chainlink
ref: ${{ inputs.cl_ref }}
- name: Check Merge Group Condition
id: condition-check
run: |
echo "Checking event condition..."
SHOULD_ENFORCE="false"
if [[ "$GITHUB_EVENT_NAME" == "merge_group" ]]; then
echo "We are in a merge_group event, now check if we are on the develop branch"
target_branch=$(cat $GITHUB_EVENT_PATH | jq -r .merge_group.base_ref)
if [[ "$target_branch" == "refs/heads/develop" ]]; then
echo "We are on the develop branch, we should enforce ctf version"
SHOULD_ENFORCE="true"
fi
fi
echo "should we enforce ctf version = $SHOULD_ENFORCE"
echo "should-enforce=$SHOULD_ENFORCE" >> $GITHUB_OUTPUT
- name: Enforce CTF Version
if: steps.condition-check.outputs.should-enforce == 'true'
uses: smartcontractkit/.github/actions/ctf-check-mod-version@21b0189c5fdca0318617d259634b1a91e6d80262 # ctf-check-mod-version@0.0.0
with:
go-project-path: ./integration-tests
module-name: github.com/smartcontractkit/chainlink-testing-framework/lib
enforce-semantic-tag: "true"
changes:
environment: integration
name: Check Paths That Require Tests To Run
@@ -150,49 +112,6 @@ jobs:
args: --out-format colored-line-number,checkstyle:golangci-lint-report.xml
working-directory: ${{ matrix.project.path }}

build-chainlink:
environment: integration
permissions:
id-token: write
contents: read
strategy:
matrix:
image:
- name: ""
dockerfile: core/chainlink.Dockerfile
tag-suffix: ""
- name: (plugins)
dockerfile: plugins/chainlink.Dockerfile
tag-suffix: -plugins
name: Build Chainlink Image ${{ matrix.image.name }}
runs-on: ubuntu22.04-8cores-32GB
needs: [changes, enforce-ctf-version]
steps:
- name: Checkout the repo
uses: actions/checkout@v4.2.1
with:
repository: smartcontractkit/chainlink
ref: ${{ inputs.cl_ref || github.event.pull_request.head.sha || github.event.merge_group.head_sha }}
- name: Setup Github Token
if: ${{ inputs.evm-ref }}
id: get-gh-token
uses: smartcontractkit/.github/actions/setup-github-token@ef78fa97bf3c77de6563db1175422703e9e6674f # setup-github-token@0.2.1
with:
aws-role-arn: ${{ secrets.AWS_OIDC_GLOBAL_READ_ONLY_TOKEN_ISSUER_ROLE_ARN }}
aws-lambda-url: ${{ secrets.AWS_INFRA_RELENG_TOKEN_ISSUER_LAMBDA_URL }}
aws-region: ${{ secrets.AWS_REGION }}
set-git-config: "true"
- name: Build Chainlink Image
if: needs.changes.outputs.core_changes == 'true' || needs.changes.outputs.github_ci_changes == 'true' || github.event_name == 'workflow_dispatch'
uses: ./.github/actions/build-chainlink-image
with:
tag_suffix: ${{ matrix.image.tag-suffix }}
dockerfile: ${{ matrix.image.dockerfile }}
git_commit_sha: ${{ inputs.evm-ref || github.sha }}
AWS_REGION: ${{ secrets.QA_AWS_REGION }}
AWS_ROLE_TO_ASSUME: ${{ secrets.QA_AWS_ROLE_TO_ASSUME }}
dep_evm_sha: ${{ inputs.evm-ref }}

run-ccip-integration-tests-for-pr:
name: Run CCIP integration Tests For PR
permissions:
@@ -201,9 +120,9 @@ jobs:
pull-requests: write
id-token: write
contents: read
needs: [build-chainlink, changes]
needs: changes
if: github.event_name == 'pull_request' && ( needs.changes.outputs.core_changes == 'true' || needs.changes.outputs.github_ci_changes == 'true')
uses: smartcontractkit/.github/.github/workflows/run-integration-tests.yml@9dcd7f8e7e4ee66de19d7077d955751f33899d0f #ctf-run-tests@0.2.0
uses: smartcontractkit/.github/.github/workflows/run-integration-tests.yml@e214c63ad3022f3edd667da19f796fb9489850b2 #ctf-run-tests@0.2.0
with:
workflow_name: Run CCIP Integration Tests For PR
chainlink_version: ${{ inputs.evm-ref || github.sha }}
@@ -229,17 +148,13 @@ jobs:
pull-requests: write
id-token: write
contents: read
needs: [build-chainlink, changes]
needs: changes
if: github.event_name == 'merge_group' && ( needs.changes.outputs.core_changes == 'true' || needs.changes.outputs.github_ci_changes == 'true')
uses: smartcontractkit/.github/.github/workflows/run-integration-tests.yml@9dcd7f8e7e4ee66de19d7077d955751f33899d0f #ctf-run-tests@1.0.0
uses: smartcontractkit/.github/.github/workflows/run-integration-tests.yml@e214c63ad3022f3edd667da19f796fb9489850b2 #ctf-run-tests@1.0.0
with:
workflow_name: Run CCIP Integration Tests For Merge Queue
chainlink_version: ${{ inputs.evm-ref || github.sha }}
test_path: .github/integration-tests.yml
test_trigger: Merge Queue integration CCIP Tests
upload_cl_node_coverage_artifact: true
upload_cl_node_coverage_artifact_prefix: cl_node_coverage_data_
# Notify Test Tooling team in slack when merge queue tests fail
slack_notification_after_tests: on_failure
slack_notification_after_tests_channel_id: "#ccip-testing"
slack_notification_after_tests_name: Core integration Tests In Merge Queue
@@ -291,23 +206,3 @@ jobs:
environment: integration
ref: ${{ github.head_ref }} # See https://github.com/github/docs/issues/15319#issuecomment-1476705663

show-chainlink-node-coverage:
name: Show Chainlink Node Go Coverage
if: always()
needs: [run-ccip-integration-tests-for-pr, run-ccip-integration-tests-for-merge-queue]
runs-on: ubuntu-latest
steps:
- name: Checkout the repo
uses: actions/checkout@v4.2.1
with:
repository: smartcontractkit/chainlink
ref: ${{ inputs.cl_ref || github.event.pull_request.head.sha || github.event.merge_group.head_sha }}
- name: Download All Artifacts
uses: actions/download-artifact@v4.1.8
with:
path: cl_node_coverage_data
pattern: cl_node_coverage_data_*
merge-multiple: true
- name: Show Coverage
run: go run ./integration-tests/scripts/show_coverage.go "${{ github.workspace }}/cl_node_coverage_data/*/merged"