Merged
Conversation
Contributor
|
I see you updated files related to
|
Contributor
|
✅ No conflicts with other open PRs targeting |
product-security-plaid-production
Bot
requested review from
kalverra and
krehermann
cfal
approved these changes
Mar 18, 2026
|
Contributor
There was a problem hiding this comment.
Pull request overview
Risk Rating: MEDIUM (repo-wide dependency bumps across multiple Go modules)
This PR updates Aptos-related dependencies (Aptos Go SDK + Chainlink Aptos relayer) across the main module and test/deployment submodules, and also bumps several OpenTelemetry packages accordingly.
Changes:
- Bump
github.com/aptos-labs/aptos-go-sdktov1.12.1-0.20260318141106-21b6ef4ed363across modules. - Bump
github.com/smartcontractkit/chainlink-aptostov0.0.0-20260318173523-755cafb24200(includingplugins.public.yaml). - Bump
go.opentelemetry.io/otel(+metric,trace) tov1.42.0in multiple modules.
Targeted areas for scrupulous human review:
- The OpenTelemetry version alignment across modules (core vs SDK submodules) to ensure builds don’t break due to type/API skew.
- Any runtime-facing behavior changes introduced by the new
chainlink-aptosand Aptos SDK commits (especially if they alter transaction/signing behavior used by tests and deployment tooling).
Reviewer recommendations (from .github/CODEOWNERS):
- Root
go.mod/go.sum:@smartcontractkit/core,@smartcontractkit/foundations integration-tests/:@smartcontractkit/devex-tooling,@smartcontractkit/core,@smartcontractkit/foundationsdeployment/:@smartcontractkit/ccip-tooling,@smartcontractkit/ccip-offchain,@smartcontractkit/keystone,@smartcontractkit/operations-platform,@smartcontractkit/core
Reviewed changes
Copilot reviewed 8 out of 15 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| system-tests/tests/go.mod | Bumps aptos-go-sdk, chainlink-aptos, and OTel deps used by system tests. |
| system-tests/tests/go.sum | Updates checksums for the bumped deps. |
| system-tests/lib/go.mod | Bumps aptos-go-sdk, chainlink-aptos, and OTel deps for system test shared library. |
| system-tests/lib/go.sum | Updates checksums for the bumped deps. |
| plugins/plugins.public.yaml | Updates the Aptos plugin gitRef to the new chainlink-aptos pseudo-version. |
| integration-tests/load/go.mod | Bumps aptos-go-sdk, chainlink-aptos, and OTel deps for load tests. |
| integration-tests/load/go.sum | Updates checksums for the bumped deps. |
| integration-tests/go.mod | Bumps aptos-go-sdk, chainlink-aptos, and OTel deps for integration tests. |
| integration-tests/go.sum | Updates checksums for the bumped deps. |
| go.mod | Bumps aptos-go-sdk, chainlink-aptos, and OTel core packages in the main module. |
| go.sum | Updates checksums for the bumped deps. |
| deployment/go.mod | Bumps aptos-go-sdk, chainlink-aptos, and OTel deps used by deployment tooling. |
| deployment/go.sum | Updates checksums for the bumped deps. |
| core/scripts/go.mod | Bumps aptos-go-sdk, chainlink-aptos, and OTel deps used by core scripts module. |
| core/scripts/go.sum | Updates checksums for the bumped deps. |
| go.opentelemetry.io/otel/log v0.15.0 | ||
| go.opentelemetry.io/otel/metric v1.41.0 | ||
| go.opentelemetry.io/otel/metric v1.42.0 | ||
| go.opentelemetry.io/otel/sdk/metric v1.41.0 |
Comment on lines
701
to
703
| go.opentelemetry.io/otel/sdk v1.41.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/log v0.15.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/metric v1.41.0 // indirect |
Comment on lines
553
to
555
| go.opentelemetry.io/otel/sdk v1.41.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/log v0.15.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/metric v1.41.0 // indirect |
Comment on lines
612
to
614
| go.opentelemetry.io/otel/sdk v1.41.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/log v0.15.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/metric v1.41.0 // indirect |
Comment on lines
+610
to
+614
| go.opentelemetry.io/otel/metric v1.42.0 // indirect | ||
| go.opentelemetry.io/otel/sdk v1.41.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/log v0.15.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/metric v1.41.0 // indirect | ||
| go.opentelemetry.io/otel/trace v1.41.0 // indirect | ||
| go.opentelemetry.io/otel/trace v1.42.0 // indirect |
Comment on lines
502
to
504
| go.opentelemetry.io/otel/sdk v1.41.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/log v0.15.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/metric v1.41.0 // indirect |
Comment on lines
599
to
601
| go.opentelemetry.io/otel/sdk v1.41.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/log v0.15.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/metric v1.41.0 // indirect |
prashantkumar1982
approved these changes
Mar 18, 2026
jmank88
approved these changes
Mar 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.