Skip to content

feat(observability): Add OpenTelemetry tracing for traffic decisions and lease operations - #16

Merged
yashdotdev13 merged 3 commits into
mainfrom
feat/6-opentelemetry-tracing
Oct 9, 2026
Merged

yashdotdev13 merged 3 commits into
mainfrom
feat/6-opentelemetry-tracing

Conversation

@yashdotdev13

Copy link
Copy Markdown
Owner

Summary

Closes #6

This PR introduces OpenTelemetry tracing for the distributed traffic-control workflow and Redis-backed lease operations. It also adds a local Jaeger setup for inspecting traces and documents the tracing configuration and verification steps.

Changes

  • Add observations around the traffic-decision workflow, including policy lookup, quota evaluation, lease allocation, and lease consumption.
  • Instrument Redis-backed lease acquisition, consumption, renewal, and release operations.
  • Inject the shared ObservationRegistry into the traffic decision engine and Redis lease coordinator.
  • Configure OTLP trace export and sampling through application properties and environment variables.
  • Add Jaeger to Docker Compose for local trace collection and visualization.
  • Update the README with an OpenTelemetry overview and link to the dedicated documentation.
  • Add docs/open-telemetry.md covering instrumentation, configuration, local setup, testing, and troubleshooting.

Validation

  • Verified that the Maven build completed successfully during Docker image construction.
  • Started the local Docker Compose stack with Redis, three gateway instances, and Jaeger.
  • Confirmed that all three gateway instances became healthy.
  • Sent a request to POST /api/v1/traffic/evaluate and received an ALLOWED response.
  • Confirmed that the distributed-traffic-control service and traces appeared in Jaeger.

Further inspection of the custom span hierarchy and the Redis lease-acquisition path is needed to confirm that every intended operation is represented correctly.

Documentation

See [docs/open-telemetry.md](https://chatgpt.com/g/g-p-6a79a4ad81088191b64779ba4aeebe75-open-source/c/docs/open-telemetry.md) for setup instructions, configuration details, and troubleshooting.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-09T19:15:46.054160Z 3c4164c PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@sonarqubecloud

sonarqubecloud Bot commented Oct 9, 2026

Copy link
Copy Markdown

@yashdotdev13
yashdotdev13 merged commit 73ee191 into main Oct 9, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add OpenTelemetry tracing for traffic decisions and lease operations

1 participant