Skip to content

Conversation

@martinothamar
Copy link
Contributor

@martinothamar martinothamar commented Feb 4, 2026

Description

OTel SDK to gateway

Verification

  • Related issues are connected (if applicable)
  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)

Summary by CodeRabbit

  • Chores
    • Enabled OpenTelemetry observability infrastructure for distributed tracing, metrics collection, and structured logging.
    • Integrated automatic instrumentation for ASP.NET Core and HTTP request tracking.
    • Configured telemetry exporter endpoint and service identification for observability platform integration.
    • Added exception recording to trace instrumentation for improved diagnostics.

@github-actions github-actions bot added skip-releasenotes Issues that do not make sense to list in our release notes skip-second-approval Pull requests that only need one approval from a reviewer. labels Feb 4, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 4, 2026

Warning

Rate limit exceeded

@martinothamar has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 14 minutes and 48 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

This pull request integrates OpenTelemetry observability support into StudioGateway API by adding NuGet package dependencies, creating an extension method for configuring tracing, metrics, and logging instrumentation, and setting up OTLP exporter configuration through environment variables.

Changes

Cohort / File(s) Summary
Package Management
src/Runtime/StudioGateway/Directory.Packages.props, src/Runtime/StudioGateway/src/StudioGateway.Api/StudioGateway.Api.csproj
Declares and references five OpenTelemetry NuGet packages (v1.15.0) for OTLP export, hosting integration, and instrumentation of ASP.NET Core, HTTP, and runtime components.
Configuration & Infrastructure
src/Runtime/StudioGateway/infra/kustomize/base/deployment.yaml
Adds two environment variables to configure OpenTelemetry: service name ("gateway") and OTLP exporter endpoint (otel-router in runtime-obs namespace).
Application Code
src/Runtime/StudioGateway/src/StudioGateway.Api/Hosting/OpenTelemetryExtensions.cs, src/Runtime/StudioGateway/src/StudioGateway.Api/Program.cs
Introduces new extension method for configuring OpenTelemetry instrumentation across tracing, metrics, and logging with exception recording enabled; integrates extension into application startup.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Observability hops forth today,
With telemetry traces lighting the way,
Metrics and logs now dance in the night,
OTLP exports glowing so bright!
The gateway watches with eyes open wide,
Nothing shall hide from our observant pride!

🚥 Pre-merge checks | ✅ 1 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is minimal but matches the template structure; however, the 'Description' section lacks sufficient detail about what was changed and why. Expand the description section to explain what OpenTelemetry components were added, their purpose, and any configuration details for better clarity.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: gateway add otel sdk' directly relates to the main change, which adds OpenTelemetry SDK packages and configuration to the gateway service.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/gateway-otel-sdk

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@martinothamar martinothamar force-pushed the chore/gateway-otel-sdk branch 2 times, most recently from 41a66ac to de0db6e Compare February 4, 2026 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-releasenotes Issues that do not make sense to list in our release notes skip-second-approval Pull requests that only need one approval from a reviewer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant