Skip to content

[exporterhelper] Improve retry sender log message clarity#15023

Open
Vanshul97 wants to merge 2 commits intoopen-telemetry:mainfrom
Vanshul97:fix/retry-sender-error-message
Open

[exporterhelper] Improve retry sender log message clarity#15023
Vanshul97 wants to merge 2 commits intoopen-telemetry:mainfrom
Vanshul97:fix/retry-sender-error-message

Conversation

@Vanshul97
Copy link
Copy Markdown

Summary

Improves the retry sender log message to reduce user confusion about transient export failures.

Fixes #13943

Problem

The current log message:

INFO "Exporting failed. Will retry the request after interval." error="rpc error: code = DeadlineExceeded desc = context deadline exceeded" interval="9.64s"

This causes concern because:

  • "Exporting failed" sounds alarming — users cannot tell if data is being lost
  • No indication that the error is transient and will be automatically retried
  • No retry count to gauge how long the issue has persisted

Fix

Changed the log message to:

INFO "Transient error, will retry the request after interval." error="..." interval="9.64s" retry_num=3

Changes:

  1. Message: "Exporting failed. Will retry..." → "Transient error, will retry..." — immediately signals this is a recoverable, transient failure
  2. Added retry_num: Shows which retry attempt this is, helping operators gauge severity
  3. Moved retryNum++ before logging: So the logged retry number is 1-based (first retry = 1)
  4. Span event updated: Same improvements applied to the trace span event, plus retry_num attribute added

Test Plan

  • TestRetrySenderWithContextTimeout updated to expect new message
  • All retry sender tests pass with -race

@Vanshul97 Vanshul97 requested review from a team, bogdandrutu and dmitryax as code owners March 29, 2026 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[retry_sender] Message on retryable error is causing concern

1 participant