Skip to content

retry config transactions on Unavailable errors - #468

Open
alexandernorth wants to merge 1 commit into
mainfrom
feature/retryable-error-improvements
Open

retry config transactions on Unavailable errors#468
alexandernorth wants to merge 1 commit into
mainfrom
feature/retryable-error-improvements

Conversation

@alexandernorth

Copy link
Copy Markdown
Contributor

Problem

A data-server "not connected" error (device unreachable) was classified as
non-recoverable, so the Config was marked FailedUnRecoverable and never
retried resulting in a config applied while the device was down being stuck in failed state,
even after the device recovered.

Change

  • Classify recoverability by gRPC code via isRecoverableTransactionError
  • targetconfig reconciler: use exponential backoff, bounded to 1s–60s, replacing the fixed 500ms retry

Pairs-with: sdcio/data-server#466

@alexandernorth
alexandernorth requested a review from a team as a code owner July 24, 2026 14:31
@alexandernorth alexandernorth added the safe to test Apply this label for PRs to kick off CI integration testing label Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test Apply this label for PRs to kick off CI integration testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants