Bump chainlink-aptos to add node_balance metric for aptos#22340
Merged
pavel-raykov merged 3 commits intodevelopfrom May 7, 2026
Merged
Bump chainlink-aptos to add node_balance metric for aptos#22340pavel-raykov merged 3 commits intodevelopfrom
pavel-raykov merged 3 commits intodevelopfrom
Conversation
Bumps chainlink-aptos develop (77fc93b) which adds the standardised node_balance gauge metric for Aptos, replacing aptos_account_balance. Co-authored-by: Cursor <cursoragent@cursor.com>
…onflicts Co-authored-by: Cursor <cursoragent@cursor.com>
cawthorne
force-pushed
the
chore/bump-chainlink-aptos-node-balance
branch
from
c7c87f1 to
8ea9321
Compare
Contributor
|
I see you updated files related to
|
Contributor
|
✅ No conflicts with other open PRs targeting |
cawthorne
changed the title
product-security-plaid-production
Bot
requested review from
gustavogama-cll and
pavel-raykov
Contributor
There was a problem hiding this comment.
Pull request overview
Risk Rating: MEDIUM (dependency bumps affecting the Aptos plugin build/install path plus multiple test modules)
This PR updates the chainlink-aptos dependency (and corresponding plugin gitRef) to a newer pseudo-version to pick up the node_balance metric work, and bumps chainlink-framework/metrics accordingly. It also updates the generated module dependency graph to reflect chainlink-aptos’s dependency on chainlink-framework/metrics.
Changes:
- Bump
github.com/smartcontractkit/chainlink-aptostov0.0.0-20260507123701-77fc93b573bbacross root + deployment + integration/system test modules. - Bump
github.com/smartcontractkit/chainlink-framework/metricstov0.0.0-20260505202410-b350dca113b4where referenced indirectly. - Update
plugins.public.yamlandgo.mdto match the new plugin/ref and dependency graph.
Targeted areas for scrupulous human review:
- The referenced
chainlink-aptoscommit/tag contents (external to this PR): confirm it includes thenode_balancemetric and does not introduce breaking changes or unexpected transitive dependency changes. - Plugin installation flow (
loopinstall) using the updatedgitRefinplugins/plugins.public.yaml(ensure CI/build pipelines still resolve and build the module cleanly).
Reviewed changes
Copilot reviewed 9 out of 16 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| system-tests/tests/go.mod | Bump chainlink-aptos and chainlink-framework/metrics versions for system test module. |
| system-tests/tests/go.sum | Update sums for the bumped dependencies. |
| system-tests/lib/go.mod | Bump chainlink-aptos and chainlink-framework/metrics versions for system test library module. |
| system-tests/lib/go.sum | Update sums for the bumped dependencies. |
| plugins/plugins.public.yaml | Update Aptos plugin gitRef to the new pseudo-version. |
| integration-tests/load/go.mod | Bump chainlink-aptos and chainlink-framework/metrics versions for load tests module. |
| integration-tests/load/go.sum | Update sums for the bumped dependencies. |
| integration-tests/go.mod | Bump chainlink-aptos and chainlink-framework/metrics versions for integration tests module. |
| integration-tests/go.sum | Update sums for the bumped dependencies. |
| go.mod | Bump root module dependency on chainlink-aptos and indirect chainlink-framework/metrics. |
| go.sum | Update sums for the bumped dependencies. |
| go.md | Update mermaid dependency graph edge for chainlink-aptos to chainlink-framework/metrics. |
| deployment/go.mod | Bump chainlink-aptos and indirect chainlink-framework/metrics for deployment module. |
| deployment/go.sum | Update sums for the bumped dependencies. |
| core/scripts/go.mod | Bump indirect chainlink-aptos and indirect chainlink-framework/metrics for scripts module. |
| core/scripts/go.sum | Update sums for the bumped dependencies. |
|
ilija42
approved these changes
May 7, 2026
pavel-raykov
approved these changes
May 7, 2026
github-merge-queue
Bot
removed this pull request from the merge queue due to failed status checks
yashnevatia
pushed a commit
that referenced
this pull request
May 8, 2026
* chore: bump chainlink-aptos to emit node_balance metric (PLEX-2873) Bumps chainlink-aptos develop (77fc93b) which adds the standardised node_balance gauge metric for Aptos, replacing aptos_account_balance. Co-authored-by: Cursor <cursoragent@cursor.com> * gomodtidy --------- Co-authored-by: Cursor <cursoragent@cursor.com>
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.
https://smartcontract-it.atlassian.net/browse/PLEX-2873