Skip to content

[CLD-1924]: fix(deployment): remove ported code to cld-changesets#22188

Merged
gustavogama-cll merged 1 commit intodevelopfrom
ggoh/cld-1924/remove-ported-code
Apr 28, 2026
Merged

[CLD-1924]: fix(deployment): remove ported code to cld-changesets#22188
gustavogama-cll merged 1 commit intodevelopfrom
ggoh/cld-1924/remove-ported-code

Conversation

@graham-chainlink
Copy link
Copy Markdown
Collaborator

Remove solana/helpers.go and jobspec.go as they have been moved over to cld-changesets

JIRA: https://smartcontract-it.atlassian.net/browse/CLD-1924

Copilot AI review requested due to automatic review settings April 27, 2026 13:00
@graham-chainlink graham-chainlink requested review from a team as code owners April 27, 2026 13:00
@graham-chainlink graham-chainlink changed the title fix(deployment): remove ported code to cld-changesets [CLD-1924]: fix(deployment): remove ported code to cld-changesets Apr 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

👋 graham-chainlink, 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!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 27, 2026

CORA - Pending Reviewers

All codeowners have approved! ✅

Legend: ✅ Approved | ❌ Changes Requested | 💬 Commented | 🚫 Dismissed | ⏳ Pending | ❓ Unknown

For more details, see the full review summary.

@github-actions
Copy link
Copy Markdown
Contributor

I see you updated files related to core. Please run make gocs in the root directory to add a changeset as well as in the text include at least one of the following tags:

  • #added For any new functionality added.
  • #breaking_change For any functionality that requires manual action for the node to boot.
  • #bugfix For bug fixes.
  • #changed For any change to the existing functionality.
  • #db_update For any feature that introduces updates to database schema.
  • #deprecation_notice For any upcoming deprecation functionality.
  • #internal For changesets that need to be excluded from the final changelog.
  • #nops For any feature that is NOP facing and needs to be in the official Release Notes for the release.
  • #removed For any functionality/config that is removed.
  • #updated For any functionality that is updated.
  • #wip For any change that is not ready yet and external communication about it should be held off till it is feature complete.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 27, 2026

✅ No conflicts with other open PRs targeting develop

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Risk Rating: MEDIUM — Removes in-repo changeset implementations and switches callers to a newly added external module (cld-changesets), alongside several dependency bumps across multiple Go modules.

This PR removes Solana and Job Distributor changeset code that has been moved to cld-changesets, updates call sites to use the new module, and refreshes related dependency versions across the repo’s multiple Go modules.

Changes:

  • Remove deployment/common/changeset/solana/helpers.go and deployment/common/changeset/jobspec.go (ported to cld-changesets).
  • Update Solana funding changesets and CCIP jobspec integration tests to call cld-changesets equivalents.
  • Bump chainlink-deployments-framework, chainlink-testing-framework, pgproto3, and add cld-changesets to relevant go.mod/go.sum files.

Reviewed changes

Copilot reviewed 11 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
system-tests/tests/go.mod Bumps dependencies; adds cld-changesets indirectly.
system-tests/tests/go.sum Updates sums for bumped deps and new cld-changesets entry.
system-tests/lib/go.mod Bumps dependencies; adds cld-changesets indirectly.
system-tests/lib/go.sum Updates sums for bumped deps and new cld-changesets entry.
integration-tests/smoke/ccip/ccip_jobspec_test.go Switches JD changesets usage to cld-changesets/jd/changesets.
integration-tests/load/go.mod Bumps dependencies to align with the new changeset/module versions.
integration-tests/load/go.sum Updates sums for bumped deps.
integration-tests/go.mod Adds direct dependency on cld-changesets and bumps key deps.
integration-tests/go.sum Updates sums for bumped deps and new cld-changesets entry.
deployment/go.mod Adds direct dependency on cld-changesets and bumps framework/testing deps.
deployment/go.sum Updates sums for bumped deps and new cld-changesets entry.
deployment/common/changeset/solana/helpers.go Removes legacy Solana funding helpers now sourced from cld-changesets.
deployment/common/changeset/solana/fund_mcm_pdas.go Switches funding calls to cld-changesets Solana helpers.
deployment/common/changeset/jobspec.go Removes legacy JD revoke/delete changesets now sourced from cld-changesets.
deployment/common/changeset/example/solana_transfer_mcm.go Switches SOL transfer instruction creation to cld-changesets helper.
core/scripts/go.mod Bumps dependencies; adds cld-changesets indirectly.
core/scripts/go.sum Updates sums for bumped deps and new cld-changesets entry.

Comment thread deployment/common/changeset/solana/fund_mcm_pdas.go Outdated
Comment thread deployment/common/changeset/example/solana_transfer_mcm.go Outdated
Comment thread integration-tests/go.mod Outdated
@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented Apr 27, 2026

Static BadgeStatic BadgeStatic BadgeStatic Badge

Failed Test Failure Summary Logs
Test_CRE_V2_Suite_Bucket_A/[v2]_Proof_Of_Reserve_-_workflow-gateway-capabilities The test failed because a specific feed did not update within the expected timeout period. Logs ↗︎
Test_CRE_V2_Suite_Bucket_A The test 'Test_CRE_V2_Suite_Bucket_A' failed without providing specific error details. Logs ↗︎

View Full Report ↗︎Docs

@graham-chainlink graham-chainlink force-pushed the ggoh/cld-1924/remove-ported-code branch 3 times, most recently from 1354429 to 3b96e6a Compare April 27, 2026 15:06
ecPablo
ecPablo previously approved these changes Apr 27, 2026
@graham-chainlink graham-chainlink force-pushed the ggoh/cld-1924/remove-ported-code branch 2 times, most recently from a416a78 to 30abff2 Compare April 28, 2026 06:54
Remove solana/helpers.go and jobspec.go as they have been moved over to cld-changesets

JIRA: https://smartcontract-it.atlassian.net/browse/CLD-1924
@graham-chainlink graham-chainlink force-pushed the ggoh/cld-1924/remove-ported-code branch from 30abff2 to 7986f7c Compare April 28, 2026 12:19
@cl-sonarqube-production
Copy link
Copy Markdown

@graham-chainlink graham-chainlink added this pull request to the merge queue Apr 28, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 28, 2026
@gustavogama-cll gustavogama-cll added this pull request to the merge queue Apr 28, 2026
Merged via the queue into develop with commit 3b71613 Apr 28, 2026
263 of 267 checks passed
@gustavogama-cll gustavogama-cll deleted the ggoh/cld-1924/remove-ported-code branch April 28, 2026 16:21
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.

6 participants