Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Synchronize tree log storage when a node first appears in container #1329

Closed
Tracked by #1322
fyrchik opened this issue Apr 25, 2022 · 1 comment · Fixed by #1929
Closed
Tracked by #1322

Synchronize tree log storage when a node first appears in container #1329

fyrchik opened this issue Apr 25, 2022 · 1 comment · Fixed by #1929
Assignees
@carpawell
Labels
neofs-storage Storage node application issues U0 Needs to be resolved immediately
Milestone
v0.34.0

Comments

@fyrchik
Copy link
Contributor

fyrchik commented Apr 25, 2022 •
edited
Loading

This is an easy synchronization case: node has nothing and needs to receive the most actual tree snapshot. Note that this can be done in parallel with accepting new replicated operations.
@fyrchik fyrchik mentioned this issue Apr 25, 2022
Open
23 tasks
@cthulhu-rider cthulhu-rider added neofs-storage Storage node application issues U3 Regular and removed triage labels May 4, 2022
@fyrchik fyrchik mentioned this issue May 12, 2022
Merged
@fyrchik
Copy link
Contributor Author

fyrchik commented May 24, 2022

Partially done in #1373 .

All that is left is to fetch list of containers and trees and execute Synchronize.

cthulhu-rider pushed a commit that referenced this issue May 25, 2022
@fyrchik @cthulhu-rider
[#1329] services/tree: Implement GetOpLog RPC
3f3fb90 
Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
cthulhu-rider pushed a commit that referenced this issue May 25, 2022
@fyrchik @cthulhu-rider
[#1329] pilorama: Allow to benchmark all tree backends
cafd38f 
Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
cthulhu-rider pushed a commit that referenced this issue May 25, 2022
@fyrchik @cthulhu-rider
[#1329] services/tree: Synchronize from the last stored height
e15252c 
Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
@carpawell carpawell mentioned this issue Oct 6, 2022
Merged
@carpawell carpawell mentioned this issue Oct 18, 2022
Merged
aprasolova pushed a commit to aprasolova/neofs-node that referenced this issue Oct 19, 2022
@fyrchik @aprasolova
[nspcc-dev#1329] services/tree: Implement GetOpLog RPC
99556b5 
Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
aprasolova pushed a commit to aprasolova/neofs-node that referenced this issue Oct 19, 2022
@fyrchik @aprasolova
[nspcc-dev#1329] pilorama: Allow to benchmark all tree backends
41239ee 
Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
aprasolova pushed a commit to aprasolova/neofs-node that referenced this issue Oct 19, 2022
@fyrchik @aprasolova
[nspcc-dev#1329] services/tree: Synchronize from the last stored height
c46efb4 
Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
aprasolova pushed a commit to aprasolova/neofs-node that referenced this issue Oct 19, 2022
@carpawell @aprasolova
[nspcc-dev#1329] tree: Do not sync trees that are from external conta…
6dab64a 
…iners

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
aprasolova pushed a commit to aprasolova/neofs-node that referenced this issue Oct 19, 2022
@carpawell @aprasolova
[nspcc-dev#1329] tree: Sync tree on startup
c23d720 
Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
aprasolova pushed a commit to aprasolova/neofs-node that referenced this issue Oct 19, 2022
@carpawell @aprasolova
[nspcc-dev#1329] node: Set undefined netmap status by default
a92ad3a 
It allows general usage of `controlNetmapStatus` getter: does not matter of
handler (`NewEpoch` handler of initial setter).

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
aprasolova pushed a commit to aprasolova/neofs-node that referenced this issue Oct 19, 2022
@carpawell @aprasolova
[nspcc-dev#1329] tree: Fix sync applying
6381504 
Add the node position in a container and the container size to the CID
descriptor that is passed to the `TreeApply`. Previously, `checkValid` does
not allow any log operarations.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
aprasolova pushed a commit to aprasolova/neofs-node that referenced this issue Oct 19, 2022
@carpawell @aprasolova
[nspcc-dev#1329] Update CHANGELOG
24ffe64 
Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@carpawell carpawell

Labels
neofs-storage Storage node application issues U0 Needs to be resolved immediately
Projects
None yet
Milestone
v0.34.0
Development

Successfully merging a pull request may close this issue.

Feat/Sync trees when a node first time appears in a container
4 participants
@fyrchik @carpawell @alexchetaev @cthulhu-rider