Skip to content

fix(storage): classify version mismatch as retryable#425

Merged
sbalabanov merged 1 commit into
mainfrom
fix/version-mismatch-retryable
Jul 22, 2026
Merged

fix(storage): classify version mismatch as retryable#425
sbalabanov merged 1 commit into
mainfrom
fix/version-mismatch-retryable

Conversation

@sbalabanov

Copy link
Copy Markdown
Contributor

Summary

  • classify SubmitQueue and Stovepipe storage.ErrVersionMismatch sentinels as retryable infrastructure errors at declaration
  • remove controller-level retryability wrappers for version mismatches
  • update tests and documentation to reflect declaration-level classification

Testing

  • make test
  • make lint
  • make check-tidy
  • make check-gazelle

@sbalabanov
sbalabanov requested review from a team and behinddwalls as code owners July 22, 2026 17:50
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Comment thread stovepipe/extension/storage/storage.go
@sbalabanov
sbalabanov added this pull request to the merge queue Jul 22, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 22, 2026
@sbalabanov
sbalabanov merged commit 1cde2ef into main Jul 22, 2026
14 of 15 checks passed
@sbalabanov
sbalabanov deployed to stack-rebase July 22, 2026 18:10 — with GitHub Actions Active
@behinddwalls
behinddwalls deleted the fix/version-mismatch-retryable branch July 22, 2026 18:10
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.

4 participants