Skip to content

Emit OTEL telemetry for downstream rate limits#259

Merged
nicobistolfi merged 1 commit intomainfrom
vigilante/issue-256-emit-otel-error-events-when-any-downstream-service-it-depends-on-returns-a-rate-limiting-or-quota-related-failure
Mar 19, 2026
Merged

Emit OTEL telemetry for downstream rate limits#259
nicobistolfi merged 1 commit intomainfrom
vigilante/issue-256-emit-otel-error-events-when-any-downstream-service-it-depends-on-returns-a-rate-limiting-or-quota-related-failure

Conversation

@nicobistolfi
Copy link
Copy Markdown
Collaborator

@nicobistolfi nicobistolfi commented Mar 19, 2026

Summary

  • emit a dedicated downstream rate-limit/quota telemetry signal with bounded OTEL log attributes and analytics properties
  • trigger that signal from both runner-side provider failures and app-side daemon/service failures
  • add focused coverage for provider quota, GitHub/API throttling, and privacy-safe payloads

Validation

  • go test ./internal/telemetry ./internal/runner ./internal/app

Closes #256

@nicobistolfi nicobistolfi mentioned this pull request Mar 19, 2026
Closed
5 tasks
@nicobistolfi nicobistolfi merged commit 81f5433 into main Mar 19, 2026
1 check passed
@nicobistolfi nicobistolfi deleted the vigilante/issue-256-emit-otel-error-events-when-any-downstream-service-it-depends-on-returns-a-rate-limiting-or-quota-related-failure branch March 19, 2026 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Emit OTEL error events when any downstream service it depends on returns a rate-limiting or quota-related failure

1 participant

@nicobistolfi