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

OpenTelemetry with Tracing Integration for HttpProxy #237

Open
OshoOS opened this issue May 10, 2024 · 2 comments
Open

OpenTelemetry with Tracing Integration for HttpProxy #237

OshoOS opened this issue May 10, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@OshoOS
Copy link

OshoOS commented May 10, 2024

What is the problem your feature solves, or the need it fulfills?

It increases the observability through telemetry

Describe the solution you'd like

Integrating with Rust OpenTelemetry, and trace OpenTelemetry.

https://docs.rs/tracing-opentelemetry/latest/tracing_opentelemetry/

Describe alternatives you've considered

What other solutions, features, or workarounds have you considered that might also solve the issue?
What are the tradeoffs for these alternatives compared to what you're proposing?

Additional context

@OshoOS OshoOS changed the title OpenTelemetry with Tracing Integration with HttpProxy OpenTelemetry with Tracing Integration for HttpProxy May 10, 2024
@gdavidsson gdavidsson added the enhancement New feature or request label May 10, 2024
@github2023spring
Copy link

Currently, is there a way to measure the latency of an upstream request, and instrument it through metrics?

@drcaramelsyrup
Copy link
Contributor

It is possible to measure upstream latency by tracking timestamps and measuring across CTX fields depending on what you're interested in, e.g. saving Instant on connected_to_upstream and comparing to a timestamp in the upstream_response_filter for TTFB since connection establishment.

I think it would be nice to have more user guidance on how one might integrate OpenTelemetry/Jaeger tracing: #258. But it is possible to do today also by saving tracing context in the CTX.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants