Skip to content

Conversation

@SungJin1212
Copy link
Member

Add cortex_distributor_ingester_push_timeouts_total metric to track the number of push requests to ingesters that were canceled due to RemoteTimeout exceeded.
This metric helps in tuning -distributor.remote-timeout flag.

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@dosubot dosubot bot added component/distributor type/observability To help know what is going on inside Cortex labels Dec 11, 2025
@SungJin1212 SungJin1212 force-pushed the Add-distributor-ingester-push-timeouts-metric branch from ae25b40 to c673f45 Compare December 11, 2025 12:00
@SungJin1212 SungJin1212 force-pushed the Add-distributor-ingester-push-timeouts-metric branch from 822029d to 9279a01 Compare December 23, 2025 02:02
@SungJin1212 SungJin1212 force-pushed the Add-distributor-ingester-push-timeouts-metric branch from 9279a01 to 57d64b4 Compare December 30, 2025 05:37
Copy link
Contributor

@danielblando danielblando left a comment

Choose a reason for hiding this comment

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

LGTM

@SungJin1212 SungJin1212 force-pushed the Add-distributor-ingester-push-timeouts-metric branch from 57d64b4 to 0b0be00 Compare January 8, 2026 02:00
Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>
Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>
@SungJin1212 SungJin1212 force-pushed the Add-distributor-ingester-push-timeouts-metric branch from 0b0be00 to 40ea241 Compare January 8, 2026 06:57
Copy link
Member

@friedrichg friedrichg left a comment

Choose a reason for hiding this comment

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

I love it

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 9, 2026
@friedrichg friedrichg merged commit e0285cb into cortexproject:master Jan 9, 2026
50 of 52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/distributor lgtm This PR has been approved by a maintainer size/M type/observability To help know what is going on inside Cortex

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants