-
Notifications
You must be signed in to change notification settings - Fork 398
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
[chain] Add OTEL tracing functionality + Jaeger #2434
Comments
So excited to see observability to the next levels. |
OTEL and Jaeger appear overlapping to me. Basically we would use Jaeger only for viewing traces? |
@sw360cab AFAIK jaeger is only the visualization tool right now, instead of being the entire framework for tracing. The client for tracing is OTEL now. |
Part of #2618 |
Blocked by @gfanton working on pure tracing in Gno.land |
FYI: I'm integrating with Tempo instead of Jaeger, wdyt ? (it's more lightweight and simpler than Jaeger if we don't need strictly specific functionalities from Jaeger) @zivkovicmilos |
@zivkovicmilos I've setup the tracing system using otel/tempo and configure a grafana dashboard for tracing.
Could you please clarify a bit more about what should we trace for each of those flows ? thanks |
Due to the fact that they should be both (Jaeger and Grafana Tempo) aligned with OTEL, I don't have a preference. The way anyone will use the feature is not tied to any other technology. In this POC I would rather show how much flexible OTEL could be, so that Tempo and Jaeger could be interchangeable. |
@sw360cab I see. Thanks :) |
@sw360cab @zivkovicmilos I've integrated OTEL tracing for Consensus flow in #3815 Could you pls take a look to see if it's ok that way ? Thanks. |
Description
This task concerns setting up otel tracing functionality (off by default) for the
gnoland
chain binary.We should have tracing in:
We should set up a workflow that gathers tracing metrics on each
master
push, during a supernova run, or during the execution of the testing suite.Additionally, we should be able to utilize Jaeger (suggestions welcome) for viewing the tracing results.
cc @ajnavarro @mvertes
The text was updated successfully, but these errors were encountered: