Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

fix/telemetrygateway: reduce context cancellation error reports#63551

Merged
bobheadxi merged 2 commits into
mainfrom
fix-telemetrygateway-contextcancelled
Jun 28, 2024
Merged

fix/telemetrygateway: reduce context cancellation error reports#63551
bobheadxi merged 2 commits into
mainfrom
fix-telemetrygateway-contextcancelled

Conversation

@bobheadxi

Copy link
Copy Markdown
Member

To cover very slow requests we have RPC metrics, and I've added some logging on some points where a very slow processing time could be problematic.

Test plan

CI

@bobheadxi bobheadxi requested review from a team and dadlerj June 28, 2024 17:37
@cla-bot cla-bot Bot added the cla-signed label Jun 28, 2024
@bobheadxi bobheadxi requested a review from akalia25 June 28, 2024 17:37
Comment thread cmd/telemetry-gateway/internal/server/publish_events.go Outdated
@bobheadxi bobheadxi requested a review from akalia25 June 28, 2024 17:46

@akalia25 akalia25 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! Thank you!

@bobheadxi bobheadxi merged commit 8e04c65 into main Jun 28, 2024
@bobheadxi bobheadxi deleted the fix-telemetrygateway-contextcancelled branch June 28, 2024 19:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants