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

Tracking: Open Telemetry for metrics and/or tracing #258

Open
markmandel opened this issue May 14, 2021 · 1 comment
Open

Tracking: Open Telemetry for metrics and/or tracing #258

markmandel opened this issue May 14, 2021 · 1 comment
Labels
area/operations Installation, updating, metrics etc good first issue Good for newcomers help wanted Extra attention is needed kind/cleanup Refactoring code, fixing up documentation, etc priority/medium Issues that we want to resolve, but don't require immediate resolution.

Comments

@markmandel
Copy link
Contributor

Right now we use the prometheus for metric capture and exposure.

OpenTelemetry is a project to watch as an OSS standard for both metrics and telemetry data, with the ability to send that data to a variety of backends.

They recently went 1.0 on the tracing specification, but have yet to reach a stable specification for metrics.

Some resources to dig into further:
OpenTelemetry Metrics Roadmap, Mar 2021
OpenTelemetry Projects, with latest timelines

Highlight a few timeline points from the above article that may be most relevant (I'm sure they will be subject to change):

  • 31st May, 2021 -Release an “Experimental” metrics API/SDK specification which we can recommend to language client owners to implement a metrics preview release - At this point we may want to review it and/or provide feedback if we see any blocking issues.
  • 30 Sept, 2021 - Metrics API/SDK specification reaches “Feature-freeze” - this seems like a good point to start exploration of cutover from Prometheus to OpenTelemetry, and ensure if the benefits outweigh the costs.
  • 30 Nov, 2021 - Metrics API/SDK specification reaches “Stable”. Together with the stable version of the specification, we should expect release candidates from multiple language clients, similar to what we had for tracing.
@markmandel markmandel added area/operations Installation, updating, metrics etc kind/cleanup Refactoring code, fixing up documentation, etc labels May 14, 2021
@XAMPPRocky XAMPPRocky added the priority/low Issues that don't need to be addressed in the near term. label Jul 5, 2021
@XAMPPRocky XAMPPRocky added priority/medium Issues that we want to resolve, but don't require immediate resolution. good first issue Good for newcomers help wanted Extra attention is needed and removed priority/low Issues that don't need to be addressed in the near term. labels Dec 5, 2022
@XAMPPRocky
Copy link
Collaborator

Recent profiling has shown that the current prometheus metrics are actually a bit expensive right now (~24ms), so if we moved to openmetrics we could see a significant reduction in the the time.

https://github.com/prometheus/client_rust

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/operations Installation, updating, metrics etc good first issue Good for newcomers help wanted Extra attention is needed kind/cleanup Refactoring code, fixing up documentation, etc priority/medium Issues that we want to resolve, but don't require immediate resolution.
Projects
None yet
Development

No branches or pull requests

2 participants