Skip to content

feat: [stable-8.0][OCISDEV-855] receivedsharecache: retry on transient storage errors in CAS loop - #709

Merged
mklos-kw merged 3 commits into
stable-8.0from
backport/8.0/feat-ocisdev-855-receivedsharecache-retry-transient
Aug 13, 2026
Merged

feat: [stable-8.0][OCISDEV-855] receivedsharecache: retry on transient storage errors in CAS loop#709
mklos-kw merged 3 commits into
stable-8.0from
backport/8.0/feat-ocisdev-855-receivedsharecache-retry-transient

Conversation

@mklos-kw

Copy link
Copy Markdown
Member

backports: #657

…rors in CAS loop (#657)

* fix: ocisdev-855, cs3/decomposefs transactional concurent upload

* fix: ocisdev-855, cs3/decomposefs optimistic-abort concurent upload

* fix: ocisdev-855, cs3/decomposefs optimistic-abort concurent upload

* feat: logs and cleanup

* feat: cleanup

* feat: review fixes

* feat: cleanup in tracing
@kw-security

kw-security commented Aug 12, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

…e Download in concurrent test

status.NewTooEarly was added after stable-8.0 branched; map IsTooEarly to
NewAborted instead (same retry semantics for the client).

DiskStorage.Upload on stable-8.0 uses os.WriteFile (not renameio), so
concurrent Download calls can read a partial file. Add a mutex to
barrierStorage that serializes Upload/Download pairs in the test.
@mzner
mzner self-requested a review August 13, 2026 07:48
@mklos-kw
mklos-kw merged commit 3dbd3c2 into stable-8.0 Aug 13, 2026
16 checks passed
@mklos-kw
mklos-kw deleted the backport/8.0/feat-ocisdev-855-receivedsharecache-retry-transient branch August 13, 2026 08:13
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