aarifullin/frostfs-node
1
0
Fork 0
forked from TrueCloudLab/frostfs-node
Code Issues Pull requests Projects Releases Packages Wiki Activity

[#1329] tree: Do not sync trees that are from external containers

Browse source 
Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
This commit is contained in:
Pavel Karpy 2022-10-06 23:18:26 +03:00 committed by fyrchik
parent ea9a5690cb
commit f5f560d903
1 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
pkg/services/tree/sync.go
View file

@ -14,13 +14,21 @@ import (
"google.golang.org/grpc/credentials/insecure" "google.golang.org/grpc/credentials/insecure"
) )
// ErrNotInContainer is returned when operation could not be performed
// because the node is not included in the container.
var ErrNotInContainer = errors.New("node is not in container")
// Synchronize tries to synchronize log starting from the last stored height. // Synchronize tries to synchronize log starting from the last stored height.
func (s *Service) Synchronize(ctx context.Context, cid cid.ID, treeID string) error { func (s *Service) Synchronize(ctx context.Context, cid cid.ID, treeID string) error {
nodes, _, err := s.getContainerNodes(cid) nodes, pos, err := s.getContainerNodes(cid)
if err != nil { if err != nil {
return fmt.Errorf("can't get container nodes: %w", err) return fmt.Errorf("can't get container nodes: %w", err)
} }
if pos < 0 {
return ErrNotInContainer
}
lm, err := s.forest.TreeGetOpLog(cid, treeID, 0) lm, err := s.forest.TreeGetOpLog(cid, treeID, 0)
if err != nil && !errors.Is(err, pilorama.ErrTreeNotFound) { if err != nil && !errors.Is(err, pilorama.ErrTreeNotFound) {
return err return err