From aa37078570f332d4aca0521f7c94e7063b778700 Mon Sep 17 00:00:00 2001 From: Pavel Karpy Date: Tue, 18 Oct 2022 17:21:16 +0300 Subject: [PATCH] [#1902] node: Sync all the trees on bootstrap Do not limit synchronization by "system" and "version" tree IDs. Signed-off-by: Pavel Karpy --- cmd/neofs-node/tree.go | 20 +++++++------------- 1 file changed, 7 insertions(+), 13 deletions(-) diff --git a/cmd/neofs-node/tree.go b/cmd/neofs-node/tree.go index e7c8a839..befa5fdc 100644 --- a/cmd/neofs-node/tree.go +++ b/cmd/neofs-node/tree.go @@ -76,20 +76,14 @@ func syncTrees(ctx context.Context, treeSvc *tree.Service, cnrCli *containerClie return } - // TODO: #1902 fetch all the trees via a new tree RPC - wellKnownTrees := [...]string{"version", "system"} - for _, id := range ids { - for i := range wellKnownTrees { - err = treeSvc.SynchronizeTree(ctx, id, wellKnownTrees[i]) - if err != nil && !errors.Is(err, tree.ErrNotInContainer) { - log.Warn( - "tree synchronization failed", - zap.Stringer("cid", id), - zap.String("tree_id", wellKnownTrees[i]), - zap.Error(err), - ) - } + err = treeSvc.SynchronizeAllTrees(ctx, id) + if err != nil && !errors.Is(err, tree.ErrNotInContainer) { + log.Warn( + "tree synchronization failed", + zap.Stringer("cid", id), + zap.Error(err), + ) } }