Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

all: live chain-aware tracing - add tracers subscription #27694

Draft
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

joohhnnn
Copy link
Contributor

@joohhnnn joohhnnn commented Jul 11, 2023

Depends on #27629.
This PR makes it possible to subscribe to traces being produced during normal block processing. So you can do eth_subscribe('traces') and get notifications from the tracer that has been configured on node start-up to be run alongside the chain. Details of the notifications depend on each tracer. This PR only as a sample adds some notifications in the Printer.
Todo: find a solution that works for all tracers.

@Javaweb7672

This comment was marked as spam.

Copy link

@Javaweb7672 Javaweb7672 left a comment

Choose a reason for hiding this comment

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

Approval request

@holiman
Copy link
Contributor

holiman commented Mar 22, 2024

This is pretty heavily bitrotted (as can be expected since it was based on a fast-moving target).

However, #29189 was just merged, so if you want to revisit this, now would be a good time to rebase it on master.

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.

4 participants