Skip to content

Comments

l2geth: Record rollup transaction metrics#2497

Merged
mergify[bot] merged 2 commits intoethereum-optimism:developfrom
Inphi:inphi/trace-causality
Apr 27, 2022
Merged

l2geth: Record rollup transaction metrics#2497
mergify[bot] merged 2 commits intoethereum-optimism:developfrom
Inphi:inphi/trace-causality

Conversation

@Inphi
Copy link
Contributor

@Inphi Inphi commented Apr 26, 2022

Adds metrics for transaction execution. These metrics were only collected when USING_OVM=false as a rollup node uses a different codepath during metrics collection.

The immediate usecase for this is to be able to estimate when indexers and vm tracers will no longer keep up with the increasing rate of transactions executed by replicas.

@changeset-bot
Copy link

changeset-bot bot commented Apr 26, 2022

🦋 Changeset detected

Latest commit: 209432f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/l2geth Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added 2-reviewers C-protocol-critical Category: Modifies protocol-critical code A-cannon Area: cannon labels Apr 26, 2022
@mergify mergify bot requested review from mslipper, tuxcanfly and tynes April 26, 2022 16:45
Add new metrics for transaction execution.
@Inphi Inphi force-pushed the inphi/trace-causality branch from 403116e to 476c42b Compare April 27, 2022 17:13
@mergify
Copy link
Contributor

mergify bot commented Apr 27, 2022

This PR has been added to the merge queue, and will be merged soon.

@mergify
Copy link
Contributor

mergify bot commented Apr 27, 2022

This PR is next in line to be merged, and will be merged as soon as checks pass.

@mergify mergify bot merged commit 81f09f1 into ethereum-optimism:develop Apr 27, 2022
@mslipper mslipper mentioned this pull request Apr 28, 2022
theochap pushed a commit that referenced this pull request Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-cannon Area: cannon C-protocol-critical Category: Modifies protocol-critical code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants