Skip to content

cluster: base storage implementation refactoring#1281

Open
sssciel wants to merge 1 commit intov3from
g.rybakov/tntp-7385-add-gs-data-types-cluster
Open

cluster: base storage implementation refactoring#1281
sssciel wants to merge 1 commit intov3from
g.rybakov/tntp-7385-add-gs-data-types-cluster

Conversation

@sssciel
Copy link
Copy Markdown
Member

@sssciel sssciel commented Apr 29, 2026

lib/cluster has its own implementations for connecting to configuration
repositories. Work has begun on the transition
to a single go-storage package.

Closes TNTP-7385

@sssciel sssciel marked this pull request as draft April 29, 2026 11:24
@sssciel sssciel force-pushed the g.rybakov/tntp-7385-add-gs-data-types-cluster branch 3 times, most recently from 9cd7cf0 to d653189 Compare April 29, 2026 19:18
@sssciel sssciel marked this pull request as ready for review April 29, 2026 19:18
@sssciel sssciel requested review from bigbes and elhimov April 29, 2026 19:21
@sssciel sssciel added the full-ci Enables full ci tests label Apr 29, 2026
@sssciel sssciel force-pushed the g.rybakov/tntp-7385-add-gs-data-types-cluster branch 2 times, most recently from 60785ee to 96a0dd5 Compare April 30, 2026 10:53
Comment thread lib/cluster/errors.go Outdated
errDataMissing = errors.New("data does not exist")
errWrongRevision = errors.New("wrong revision")

errManyResponses = errors.New("too many responses")
Copy link
Copy Markdown
Contributor

@patapenka-alexey patapenka-alexey Apr 30, 2026

Choose a reason for hiding this comment

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

Looks unused in the repository.

ap@ap:work/git/tt (g.rybakov/tntp-7385-add-gs-data-types-cluster) $ grep -rne 'errManyResponses' . --include=*.go
./lib/cluster/errors.go:35:	errManyResponses = errors.New("too many responses")
ap@ap:work/git/tt (g.rybakov/tntp-7385-add-gs-data-types-cluster) $ 

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Thanks, done

Comment thread lib/cluster/storage.go
Comment on lines +277 to +278

func NewStorage(storage gstorage.Storage, prefix string, timeout time.Duration, key string, storageType string) RawStorage {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There should be a comment, I think.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Thanks, done

lib/cluster has its own implementations for connecting to configuration
repositories. Work has begun on the transition
to a single go-storage package.

Closes TNTP-7385
@sssciel sssciel force-pushed the g.rybakov/tntp-7385-add-gs-data-types-cluster branch from 96a0dd5 to b192606 Compare April 30, 2026 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full-ci Enables full ci tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants