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 Instrumentation #43

Merged
merged 19 commits into from
Nov 26, 2024
Merged

Conversation

ioaiaaii
Copy link
Owner

@ioaiaaii ioaiaaii commented Nov 20, 2024

Context

Time to observe the system. We'll need to instrument the code for network and os performance metrics, along with structural logging and tracing.

Suggested Solution

Instrumentation with OTEL.
Add most important performance metrics for SLOs

@ioaiaaii ioaiaaii added the preview Enable ArgoCD PR Trigger for Preview Env label Nov 20, 2024
Maintain all api config, from env or default vars
- Split pipelines into packages
- Instantiate main otel sdk setup
- Add basic net and golang metrics
- Add global slog for logger
- Migrate most important logs for now
- improve server start
- improve graceful shutdowns
- improve wait groups and context
- support otel
@ioaiaaii ioaiaaii force-pushed the feat/fontend-otel-instrumentation branch from 37868d7 to 6bf15e5 Compare November 22, 2024 16:18
@ioaiaaii ioaiaaii force-pushed the feat/fontend-otel-instrumentation branch from 78cdf88 to 5f9b754 Compare November 24, 2024 08:58
@ioaiaaii
Copy link
Owner Author

ioaiaaii commented Nov 26, 2024

  • Traces sent to OTEL Collector and exported to Tempo :
Screenshot 2024-11-26 at 7 20 05 PM
  • Metrics sent to OTEL Collector and exported to Prometheus
  • Logs sent to OTEL Collector and exported to debug currently, until Loki comes

@ioaiaaii ioaiaaii force-pushed the feat/fontend-otel-instrumentation branch from 800dda8 to d329574 Compare November 26, 2024 18:05
@ioaiaaii ioaiaaii merged commit c150bc3 into master Nov 26, 2024
5 checks passed
@ioaiaaii ioaiaaii deleted the feat/fontend-otel-instrumentation branch November 26, 2024 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preview Enable ArgoCD PR Trigger for Preview Env
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant