Skip to content

port removing the legacy upload path - #949

Open
butonic wants to merge 2 commits into
port-746from
port-725
Open

butonic wants to merge 2 commits into
port-746from
port-725

Conversation

@butonic

@butonic butonic commented Oct 29, 2026

Copy link
Copy Markdown
Member

The drivers drop their session stores, InitiateUpload, Upload, ListUploadSessions and the tus data store methods. The coordinator owns the whole upload flow. The DisableVersioning aspect goes too.

Requires the OpenCloud side (wiring the coordinator, listing upload sessions through it) before the reva bump: opencloud-eu/opencloud#3839

The tip of this stack has the same changes as #922.

Review commit by commit: the original change keeps its author, the commit after it adapts the pkg decomposedfs fork and posix.

stacked on #948

Assisted by Claude Code (claude-opus-5-5): it ported and adapted the changes, split them into this stack and wrote this description.

@butonic
butonic added this pull request to stack #950 October 29, 2026 14:05
@butonic butonic added the Type:Maintenance E.g. technical debt, packaging, etc. label Oct 29, 2026
@butonic
butonic removed this pull request from stack #950 October 29, 2026 14:19
@butonic
butonic added this pull request to stack #952 October 29, 2026 14:19
@butonic
butonic force-pushed the port-725 branch 2 times, most recently from b328f2c to 49ba454 Compare October 30, 2026 10:16
f-firas and others added 2 commits October 30, 2026 11:37
The UploadSessionResolver interface stays: the coordinator implements
it for the tus handler. The posix driver and the other decomposedfs
copy are adapted in a follow-up commit.
The coordinator now owns the whole upload flow, so the pkg fork and
posix drop their own session store, the tus data store methods,
InitiateUpload, Upload, ListUploadSessions and ResolveUpload. The
DisableVersioning aspect goes too: only that session store read it.

The pkg fork's upload specs drive the uploads through a coordinator,
including the over-quota upload that finishes through tus with a 507.

Assisted-by: Claude Code:claude-opus-5-5
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type:Maintenance E.g. technical debt, packaging, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants