-
Notifications
You must be signed in to change notification settings - Fork 3.6k
feat(infra-dev): add opentelemetry and grafana #12808
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
Conversation
…nd default database
# Conflicts: # packages/twenty-server/src/instrument.ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
Introduces OpenTelemetry and Grafana monitoring infrastructure with ClickHouse integration for telemetry data collection and visualization.
- Security Risk: Grafana ClickHouse datasource uses hardcoded credentials and disabled TLS verification in
grafana/provisioning/datasources/clickhouse-datasource.yaml
- Added shared Docker network (
twenty_network
) in Makefile to enable proper communication between monitoring services - OpenTelemetry collector configured for OTLP/HTTP protocol in
otel-collector/otel-collector-config.yaml
, but lacks proper authentication - Modified
metrics.service.ts
to implement OpenTelemetry-based metrics tracking with improved counter management
4 files reviewed, no comments
Edit PR Review Bot Settings | Greptile
📊 API Changes ReportREST API ChangesSummary🔄 Changed Operations (63)
|
🚀 Preview Environment Ready! Your preview environment is available at: http://bore.pub:62924 This environment will automatically shut down when the PR is closed or after 5 hours. |
No description provided.