Skip to content

Enable RetryInfo by default on ResourceExhausted errors - #7088

Merged
electron0zero merged 3 commits into
grafana:mainfrom
electron0zero:enable_retry_info_by_default
Apr 30, 2026
Merged

Enable RetryInfo by default on ResourceExhausted errors#7088
electron0zero merged 3 commits into
grafana:mainfrom
electron0zero:enable_retry_info_by_default

Conversation

@electron0zero

@electron0zero electron0zero commented Apr 29, 2026

Copy link
Copy Markdown
Member

What this PR does:

Enable RetryInfo by default on ResourceExhausted errors so client can retry on ResourceExhausted errors.

Related: #3019 and #5741

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]

Copilot AI 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.

Pull request overview

Enables OTLP RetryInfo hints by default for ResourceExhausted responses by changing the cluster default distributor.retry_after_on_resource_exhausted from 0 to 5s, and updates documentation/changelog accordingly.

Changes:

  • Default RetryAfterOnResourceExhausted to 5s at the distributor (cluster) level.
  • Update docs config reference + manifest to reflect the new default and how to disable it.
  • Add a changelog entry describing the breaking default change and the opt-out mechanisms.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
modules/overrides/config.go Updates default override comment for ingestion.retry_info_enabled.
modules/distributor/config.go Changes cluster default RetryAfterOnResourceExhausted from 0 to 5s.
docs/sources/tempo/configuration/manifest.md Updates generated manifest default for retry_after_on_resource_exhausted to 5s.
docs/sources/tempo/configuration/_index.md Updates config docs to reflect the new default and disable behavior.
CHANGELOG.md Adds a breaking-change entry for the new default behavior.
Comment thread modules/overrides/config.go Outdated
Comment thread CHANGELOG.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9e1443e5ae

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread modules/distributor/config.go
@electron0zero
electron0zero enabled auto-merge (squash) April 30, 2026 13:44
@electron0zero
electron0zero merged commit 14969c1 into grafana:main Apr 30, 2026
28 checks passed
@electron0zero
electron0zero deleted the enable_retry_info_by_default branch April 30, 2026 14:00
mattdurham pushed a commit to mattdurham/tempo that referenced this pull request Jun 18, 2026
* Enable RetryInfo by default on ResourceExhausted errors

* Add CHANGELOG.md

* fix review comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants