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 tracing #134

Open
sideninja opened this issue Nov 29, 2022 · 1 comment
Open

OpenTelemetry tracing #134

sideninja opened this issue Nov 29, 2022 · 1 comment
Assignees
Labels
feature A new user feature or a new package API

Comments

@sideninja
Copy link
Contributor

Integrate open telemetry tracing inside the GraphQL and using prometheus as the exporter. At this point we should just trace requests/responses no need to yet dive deeper in the codebase with traces as that will require refactoring of the functions to include passing of context (which we should do at later point anyway). Use Go opentelemtry library https://pkg.go.dev/github.com/rot1024/otelgqlgen
Some examples:
https://github.com/open-telemetry/opentelemetry-go/blob/main/example/prometheus/main.go

@sideninja sideninja moved this from 🏗 In progress to 🔖 Ready in 🌊 Flow 4D Nov 29, 2022
@DylanTinianov DylanTinianov moved this from 🔖 Ready to 🏗 In progress in 🌊 Flow 4D Nov 29, 2022
@DylanTinianov DylanTinianov moved this from 🏗 In progress to 👀 In review in 🌊 Flow 4D Dec 6, 2022
@DylanTinianov
Copy link
Contributor

DylanTinianov commented Dec 6, 2022

Needs SRE to enable Grafana Tracing Agent. Then we can simple update the telemetry config env var and re-deploy

@DylanTinianov DylanTinianov moved this from 👀 In review to ✅ Done in 🌊 Flow 4D Dec 8, 2022
@DylanTinianov DylanTinianov moved this from ✅ Done to 🏗 In progress in 🌊 Flow 4D Dec 12, 2022
@DylanTinianov DylanTinianov moved this from 🏗 In progress to ✅ Done in 🌊 Flow 4D Dec 12, 2022
@DylanTinianov DylanTinianov added the feature A new user feature or a new package API label May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new user feature or a new package API
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants