Skip to content

[exporter/loadbalancing] retry only failed backend data#47743

Open
jeevan6996 wants to merge 1 commit intoopen-telemetry:mainfrom
jeevan6996:fix-loadbalancer-retry-duplicates
Open

[exporter/loadbalancing] retry only failed backend data#47743
jeevan6996 wants to merge 1 commit intoopen-telemetry:mainfrom
jeevan6996:fix-loadbalancer-retry-duplicates

Conversation

@jeevan6996
Copy link
Copy Markdown

Summary

  • return consumererror partial-data errors from logs, traces, and metrics exporters when only a subset of backend exports fails
  • include only failed backend payloads in retryable error data instead of returning a generic aggregated error
  • add merge helpers for logs and metrics payloads to build retry subsets
  • add tests covering partial-data behavior for logs and traces failures, and mixed-success metrics fanout failures

Fixes #47600.

Testing

  • cd exporter/loadbalancingexporter && go test ./...

@jeevan6996 jeevan6996 requested a review from a team as a code owner April 18, 2026 11:00
@jeevan6996 jeevan6996 requested a review from axw April 18, 2026 11:00
@github-actions github-actions bot added the first-time contributor PRs made by new contributors label Apr 18, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Welcome, contributor! Thank you for your contribution to opentelemetry-collector-contrib.

Important reminders:

A maintainer will review your pull request soon. Thank you for helping make OpenTelemetry better!

Copy link
Copy Markdown
Contributor

@axw axw left a comment

Choose a reason for hiding this comment

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

@jeevan6996 thanks for opening a PR. There's already support in exporterhelper for retrying partial failures -- see https://github.com/open-telemetry/opentelemetry-collector/blob/22a5e61748b9daabc6781a22e7fa97d1c2c104e9/exporter/exporterhelper/internal/request/request.go#L34-L45

Rather than introducing code specific to the loadbalancing exporter, let's see if/how that can be used.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

loadbalancingexporter: failed endpoint retries re-route already-delivered records, causing duplicates on healthy endpoints

3 participants