[CRE][Deployments] Add UpdateNodes changeset#21388
Conversation
product-security-plaid-production
bot
requested review from
graham-chainlink and
michael-cll
|
👋 bolekk, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
|
✅ No conflicts with other open PRs targeting |
|
![trunk-io[bot]](https://avatars.githubusercontent.com/in/120106?s=60&v=4){kind=small}
There was a problem hiding this comment.
🔴 Test Results: Unrelated Failure
Affected failures:
- Workflow Run: Run CCIP integration In Memory Tests For PR / smoke/ccip/ccip_messaging_test.go:Test_CCIPMessaging_Solana2EVM_LOOPP
- Workflow Run: Run CCIP integration In Memory Tests For PR / smoke/ccip/ccip_token_transfer_test.go:*_LOOPP
What Broke
These failures appear to be unrelated to the changes in this PR. The root cause is a timeout in the CCIP messaging and token transfer tests due to the absence or emptiness of expected commit reports. Although commit report events are being received, they consistently contain zero blessed and unblessed merkle roots, preventing the tests from finding the expected sequence number range or validating the commit.
Autofix Options
You can use our MCP server to get AI assistance with debugging and fixing these failures.
- Use MCP in your IDE to debug the issue. Try
Help me fix CI failures from xx4if5q1to get started.
Tip
Get Better Results: This CI job is not uploading test reports. Adding structured test reports enables more precise, test-level analysis with better root cause identification and more targeted fix recommendations.
👉🏻 Learn how to upload test results.
There was a problem hiding this comment.
🔴 Test Results: Missing Constants Compilation Error
Affected failures:
- Workflow Run: Integration Tests
What Broke
The CI failure is caused by a compilation error in the newly added test file update_nodes_changeset_test.go due to missing constant declarations.
Proposed Fixes
Add missing constant declarations (csaKey, signer1, signer2, p2pID1, p2pID2, encryptionPublicKey) to update_nodes_changeset_test.go to resolve compilation errors.
In update_nodes_changeset_test.go:24
+
+ // Local constants (same values used in existing tests)
+ const (
+ csaKey = "4240b57854dd1f21c10353ea458eecd8593624d0e0a7cca07c62a4b58df8c258"
+ signer1 = "5240b57854dd1f21c10353ea458eecd8593624d0e0a7cca07c62a4b58df8c251"
+ signer2 = "5240b57854dd1f21c10353ea458eecd8593624d0e0a7cca07c62a4b58df8c252"
+ p2pID1 = "p2p_12D3KooWM1111111111111111111111111111111111111111111"
+ p2pID2 = "p2p_12D3KooWM1111111111111111111111111111111111111111112"
+ encryptionPublicKey = "7240b57854dd1f21c10353ea458eecd8593624d0e0a7cca07c62a4b58df8c254"
+ )Autofix Options
You can apply the proposed fixes directly to your branch. Try the following:
- Comment
/trunk stack-fix fYx2x9lmto generate a stacked PR with the proposed fixes. - Use MCP in your IDE to fix the issue. Try
Help me fix CI failures from fYx2x9lmto get started.
3 participants
[CRE][Deployments] Add UpdateNodes changeset