Skip to content

[execution] Add Move VM telemetry reporting to ExecutionMetrics#26178

Merged
tzakian merged 1 commit intomainfrom
tzakian/execution-metrics-vm-metrics
Apr 15, 2026
Merged

[execution] Add Move VM telemetry reporting to ExecutionMetrics#26178
tzakian merged 1 commit intomainfrom
tzakian/execution-metrics-vm-metrics

Conversation

@tzakian
Copy link
Copy Markdown
Contributor

@tzakian tzakian commented Apr 9, 2026

Description

Make the Move VM runtime telemetry module public and add MoveVMTelemetry (boundary type) and MoveVMTelemetryMetrics (Prometheus gauges) to ExecutionMetrics. The latest execution adapter calls try_update after each transaction. This lazily reads the VM telemetry report every 30 seconds and sets the Prometheus gauges covering cache state, timing, and stack sizes.

Test plan

CI


Release notes

Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.

For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

  • Protocol:
  • Nodes (Validators and Full nodes):
  • gRPC:
  • JSON-RPC:
  • GraphQL:
  • CLI:
  • Rust SDK:
  • Indexing Framework:

@tzakian tzakian requested review from a team, cgswords and dariorussi April 9, 2026 16:33
@tzakian tzakian temporarily deployed to sui-typescript-aws-kms-test-env April 9, 2026 16:33 — with GitHub Actions Inactive
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sui-docs Ready Ready Preview, Comment Apr 10, 2026 5:13pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
multisig-toolkit Ignored Ignored Preview Apr 10, 2026 5:13pm
sui-kiosk Ignored Ignored Preview Apr 10, 2026 5:13pm

Request Review

@tzakian tzakian force-pushed the tzakian/execution-metrics-vm-metrics branch from d0e6b23 to d57d983 Compare April 9, 2026 16:45
@tzakian tzakian temporarily deployed to sui-typescript-aws-kms-test-env April 9, 2026 16:45 — with GitHub Actions Inactive
Base automatically changed from tzakian/execution-metrics-refactor to main April 10, 2026 17:09
Make the Move VM runtime telemetry module public and add
MoveVMTelemetry (boundary type) and MoveVMTelemetryMetrics
(Prometheus gauges) to ExecutionMetrics. The latest execution
adapter calls try_update after each transaction. This lazily
reads the VM telemetry report every 30 seconds and sets the
Prometheus gauges covering cache state, timing, and stack sizes.
@tzakian tzakian force-pushed the tzakian/execution-metrics-vm-metrics branch from d57d983 to db85c1e Compare April 10, 2026 17:12
@tzakian tzakian temporarily deployed to sui-typescript-aws-kms-test-env April 10, 2026 17:12 — with GitHub Actions Inactive
Copy link
Copy Markdown
Contributor

@cgswords cgswords left a comment

Choose a reason for hiding this comment

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

LGTM but I think we should get someone from an ops team to look at it, too.

@tzakian tzakian merged commit 1beda9a into main Apr 15, 2026
62 checks passed
@tzakian tzakian deleted the tzakian/execution-metrics-vm-metrics branch April 15, 2026 15:39
jordanjennings-mysten pushed a commit that referenced this pull request Apr 16, 2026
## Description

Make the Move VM runtime telemetry module public and add MoveVMTelemetry
(boundary type) and MoveVMTelemetryMetrics (Prometheus gauges) to
ExecutionMetrics. The latest execution adapter calls try_update after
each transaction. This lazily reads the VM telemetry report every 30
seconds and sets the Prometheus gauges covering cache state, timing, and
stack sizes.

## Test plan 

CI

---

## Release notes

Check each box that your changes affect. If none of the boxes relate to
your changes, release notes aren't required.

For each box you select, include information after the relevant heading
that describes the impact of your changes that a user might notice and
any actions they must take to implement updates.

- [ ] Protocol: 
- [ ] Nodes (Validators and Full nodes): 
- [ ] gRPC:
- [ ] JSON-RPC: 
- [ ] GraphQL: 
- [ ] CLI: 
- [ ] Rust SDK:
- [ ] Indexing Framework:
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.

2 participants