Skip to content
This repository was archived by the owner on Aug 9, 2025. It is now read-only.

OpenTelemetry#26

Closed
drewda wants to merge 19 commits intomainfrom
otel
Closed

OpenTelemetry#26
drewda wants to merge 19 commits intomainfrom
otel

Conversation

@drewda
Copy link
Copy Markdown
Member

@drewda drewda commented Jul 9, 2025

@drewda drewda changed the title OpenTelemetry tracing in both HTTP Chi router and River jobs OpenTelemetry tracing in both HTTP Chi router, River jobs, and database driver Jul 9, 2025
@drewda drewda changed the title OpenTelemetry tracing in both HTTP Chi router, River jobs, and database driver OpenTelemetry tracing in both HTTP Chi router and River jobs Aug 4, 2025
@drewda drewda changed the title OpenTelemetry tracing in both HTTP Chi router and River jobs OpenTelemetry Aug 5, 2025
@drewda drewda marked this pull request as ready for review August 5, 2025 04:35
@drewda drewda requested a review from Copilot August 5, 2025 04:35
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds comprehensive OpenTelemetry (OTel) observability support to the project, implementing tracing for HTTP requests, GraphQL operations, database queries, and background jobs. The changes enable monitoring and debugging capabilities across the entire application stack.

  • Implements OpenTelemetry tracing middleware for HTTP, GraphQL, and River job processing
  • Adds database query tracing through wrapped SQL drivers
  • Provides configurable exporter support (console, OTLP) with environment variable configuration

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
metrics/otel/otel.go Core OpenTelemetry SDK initialization with configurable exporters and middleware creation
metrics/otel/graphql.go GraphQL-specific tracing extension for gqlgen with field-level instrumentation
jobs/river/river_jobs.go Integration of OpenTelemetry middleware into River job processing
go.mod Addition of OpenTelemetry dependencies and version updates
dbutil/db.go Database connection utilities with optional OpenTelemetry SQL tracing

Comment thread metrics/otel/otel.go Outdated
Comment thread metrics/otel/otel.go Outdated
Comment thread metrics/otel/otel.go Outdated
Comment thread metrics/otel/graphql.go
Comment thread jobs/river/river_jobs.go Outdated
Comment thread dbutil/db.go Outdated
@irees
Copy link
Copy Markdown
Contributor

irees commented Aug 6, 2025

Moved all code to tlv2 repo for now. Once the api and details have been stabilized it can be moved to transitland-mw.

@irees irees closed this Aug 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants