Skip to content

OCM: removed datatx protocol, now deprecated, and added accessTypes#5476

Merged
glpatcern merged 1 commit intomasterfrom
ocm-remove-datatx
Feb 2, 2026
Merged

OCM: removed datatx protocol, now deprecated, and added accessTypes#5476
glpatcern merged 1 commit intomasterfrom
ocm-remove-datatx

Conversation

@glpatcern
Copy link
Copy Markdown
Member

@glpatcern glpatcern commented Jan 16, 2026

Depends on cs3org/cs3apis#259

In this PR the Nextcloud storage driver tests have been disabled as we're not actively supporting Nextcloud as a storage provider.

@glpatcern glpatcern marked this pull request as draft January 16, 2026 16:36
@glpatcern glpatcern force-pushed the ocm-remove-datatx branch 5 times, most recently from 19aac29 to 2000e1c Compare January 23, 2026 17:39
@glpatcern glpatcern marked this pull request as ready for review January 28, 2026 18:06
Copy link
Copy Markdown
Member

@rawe0 rawe0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall I think! would approve when tests pass (once the cs3api changes are merged).

Comment thread pkg/share/manager/sql/model/model.go
Comment thread pkg/share/manager/sql/model/model.go
Comment thread internal/http/services/opencloudmesh/ocmd/protocols_test.go Outdated
Comment thread go.mod Outdated
@glpatcern glpatcern force-pushed the ocm-remove-datatx branch 5 times, most recently from 864384a to b73aba5 Compare January 31, 2026 17:42
@glpatcern glpatcern requested a review from rawe0 February 2, 2026 07:07
Copy link
Copy Markdown
Member

@rawe0 rawe0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now! approved.

@glpatcern glpatcern merged commit e250285 into master Feb 2, 2026
24 checks passed
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.

2 participants