mirror of
https://github.com/nspcc-dev/neo-go.git
synced 2026-03-01 04:28:51 +00:00
Adopt possibly incomplete NeoFS SEARCH results in NeoFSBlockFetcher and upload-bin CLI command #1404
Labels
No labels
I1
I2
I3
I4
S1
S2
S3
S4
U0
U1
U2
U3
U3
U4
blocked
bug
bug
cli
compiler
config
config
consensus
dependencies
discussion
documentation
enhancement
epic
feature
go
good first issue
help wanted
neotest
network
oracle
performance
question
rpc
security
smartcontract
task
task
task
test
vm
wallet
windows
windows
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
nspcc-dev/neo-go#1404
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @AnnaShaleva on GitHub (Oct 24, 2024).
Current Behavior
Some blocks are uploaded to NeoFS, then restart of the script happens. The script starts to upload from 0 block, not from the latest incomplete batch:
The pattern repeats.
Expected Behavior
Reupload must happen starting from latest incomplete batch.
Possible Solution
Find the problem in
fetchLatestMissingBlockIndex, fix it.@AnnaShaleva commented on GitHub (Oct 24, 2024):
One more example:
@AnnaShaleva commented on GitHub (Oct 24, 2024):
But sometimes it works differently (logs are from the same mainnet service):
@AliceInHunterland commented on GitHub (Oct 24, 2024):
can be connected #3615
@AnnaShaleva commented on GitHub (Oct 24, 2024):
Well, of course it's a bug in fetchLatestMissingBlockIndex only if our batches are full and don't have gaps.
Check currently uploaded data for N3 mainet. See if there are gaps in batches.
@AnnaShaleva commented on GitHub (Oct 24, 2024):
Checked, depends on #3615 resolution.
@AnnaShaleva commented on GitHub (Oct 24, 2024):
To resolve this issue, we need to adopt the SEARCH completeness marker once https://github.com/nspcc-dev/neofs-node/issues/2721 implemented.
@AnnaShaleva commented on GitHub (Oct 24, 2024):
See also https://github.com/nspcc-dev/neofs-node/issues/2721#issuecomment-2435121374: for situations where all SNs from REP policy are dead, we need to shut down BlockFetcher, because it won't be able to receive even block OIDs and proceed with addition of blocks to the chain.
@AnnaShaleva commented on GitHub (Nov 11, 2024):
Within the scope of this issue we need to revert changes made by https://github.com/nspcc-dev/neo-go/issues/3670 and fall back from per-object SEARCH to SEARCH for the range of objects.