Garcia/operator with telemetry #27
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… and error handling. Ensure script is run from repository root and provide detailed instructions for accessing Zipkin UI.
- Integrated OpenTelemetry SDK for browser tracing in the client application. - Added necessary dependencies for OpenTelemetry in package.json and package-lock.json. - Updated Dockerfile to include OTEL_COLLECTOR_URL as an environment variable. - Enhanced deployment scripts to set up port-forwarding for the OpenTelemetry collector. - Updated collector image version in values.yaml to 0.141.0. - Improved CORS configuration in the collector's configmap for local development.
- Modified expressions in the dashboard to improve metric aggregation for subgraphs and HTTP requests. - Updated titles and legend formats for clarity in visualizations. - Enhanced README with dashboard requirements, usage instructions, and known limitations regarding HTTP status codes in subgraph metrics.
faisalwaseem
reviewed
Jan 6, 2026
There was a problem hiding this comment.
Do we want to build our own dashboard or reference the APM project and link it here.
There was a problem hiding this comment.
As I commented earlier. I much rather look at updating the APM platform and pulling the dashboard from theere
There was a problem hiding this comment.
Just a general comment we could look at substituting these value with labels and deployments
faisalwaseem
approved these changes
Jan 6, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a script for setting up tracing and metrics telemetry via an OTEL collector that exports for prometheus to scrape from the collector with a grafana dashboard.
The client ui, router, coprocessor, and subgraphs should all be submitting trace information. The router, coprocessor, and subgraphs should be submitting metric information to be viewed on the grafana dashboard.
The dashboard itself is an early version of the grafana dashboard that I will submit as the updated template in the
apm-templatesrepo for grafana and is based on the Datadog dashboard that was announced at SummitHere is a visual of that dashboard