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

[CCIP Fix] Updated the Finalizer to reduce the frequency of heavy query #16153

Draft
wants to merge 1 commit into
base: release/2.20.0-ccip1.5
Choose a base branch
from

Conversation

amit-momin
Copy link
Contributor

No description provided.

Copy link
Contributor

github-actions bot commented Jan 30, 2025

AER Report: CI Core

aer_workflow , commit , Scheduled Run Frequency , Detect Changes , Clean Go Tidy & Generate , GolangCI Lint (.) , Core Tests (go_core_tests) , Core Tests (go_core_tests_integration) , Core Tests (go_core_ccip_deployment_tests) , Core Tests (go_core_fuzz) , test-scripts , Core Tests (go_core_race_tests) , lint , SonarQube Scan

1. Test failed due to incorrect number of items in the result set: [go_core_tests]

Source of Error:
integration_v2_reverted_txns_test.go:103: 
	Error Trace:	/home/runner/work/chainlink/chainlink/core/services/vrf/v2/integration_v2_reverted_txns_test.go:103
	Error: 	"[{0x4e64b83a9267624a4f1db22d0f076674efeccf8f6f321054fc2ae5576657891b {[] 1 222492 [64 0 0 0 0 1 0 0 0 0 0 0 128 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 8 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 16 0 4 0 0 2 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 4 0 0 0 0 0 0 1 0 0 0 0 0 0 0 2 0 0 0 0 4 0 0 0 1 0 0 0 8 0 0 0 64 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 64 8 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 128 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 128 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 32 0 0 0 0 0 0 0 0 0 0 0 0 0 0 8 0 0 0 0 0 0 64 0 0 0 0 0 0 0 0 0 0 0 2 0 0 16 0 0 0 0 0 0 0 2 0 0 0 0 0 0 0 0 0 0] [0xc01f9e0790 0xc01f9e0840 0xc01f9e08f0 0xc01f9e0b00] 0x4e64b83a9267624a4f1db22d0f076674efeccf8f6f321054fc2ae5576657891b 0x0000000000000000000000000000000000000000 222492 0x3a86329fdcb2177ca28583a577d3208e01a0043e7716a2fbb5a1fe62fa51b6b5 51 0 []} 0xA6C9283c53f3F8257213d3eA8F71e9ccF385B24a 0x0000000000000000000000000000000000000000 0x 0 1 0x169a91f17e71156e4a850bb453f3a8b7d3c367ff455693af59bc96f66b8957b0 0xc268ddf89ec15df7d47caf1fb1c3a5b94bba0da00af4eb8d9f1363127082bfa4 8}]" should have 2 item(s), but has 1
	Test: 	TestVRFV2Integration_ForceFulfillmentRevertedTxn_Retry

Why: The test TestVRFV2Integration_ForceFulfillmentRevertedTxn_Retry expected 2 items in the result set but only found 1. This discrepancy indicates that the test logic or the underlying code did not produce the expected number of transactions.

Suggested fix: Review the test setup and the logic that generates the transactions. Ensure that the conditions for generating the expected number of transactions are met. Add debugging statements to verify the intermediate states and identify why the second transaction is missing.

2. Error processing head due to context cancellation: [go_core_tests]

Source of Error:
logger.go:146: 2025-01-30T23:09:22.061Z	ERROR	EVM.1337.Txm.Confirmer	txmgr/confirmer.go:239	Error processing head	{"version": "unset@unset", "err": "failed to find re-org'd or included transactions based on the mined transaction count 1: failed to load evm.tx_attempts: loadEthTxesAttempts failed to load evm.tx_attempts: timeout: context canceled\nunable to fetch mined transaction count for address 0xA6C9283c53f3F8257213d3eA8F71e9ccF385B24a: context canceled"}

Why: The error indicates that the process was unable to load transaction attempts due to a context cancellation. This typically happens when the operation takes too long and exceeds the allowed time limit, causing the context to be canceled.

Suggested fix: Increase the timeout duration for the context to ensure that the operation has enough time to complete. Additionally, investigate any potential performance bottlenecks in the transaction loading process that might be causing delays.

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant