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.

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

eth/protocols/snap: snap sync testing #22179

Merged
merged 22 commits into from
Jan 25, 2021
Merged
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
4a878d1
eth/protocols/snap: make timeout configurable
holiman Jan 18, 2021
9d8994c
eth/protocols/snap: snap sync testing
holiman May 27, 2020
935c001
eth/protocols/snap: test to trigger panic
holiman Jan 18, 2021
375588b
eth/protocols/snap: fix race condition on timeouts
holiman Jan 18, 2021
404b299
eth/protocols/snap: return error on cancelled sync
holiman Jan 18, 2021
f191848
squashme: updates + test causing panic + properly serve accounts in o…
holiman Jan 19, 2021
dc21744
eth/protocols/snap: revert failing storage response
holiman Jan 19, 2021
09d0610
eth/protocols/snap: revert on bad responses (storage, code)
holiman Jan 19, 2021
f963041
eth/protocols/snap: fix account handling stall
holiman Jan 19, 2021
cf8b96b
eth/protocols/snap: fix remaining revertal-issues
holiman Jan 19, 2021
6944299
eth/protocols/snap: timeouthandler for bytecode requests
holiman Jan 20, 2021
b86bd4b
eth/protocols/snap: debugging + fix log message
holiman Jan 20, 2021
f358473
eth/protocols/snap: fix misspelliings in docs
holiman Jan 20, 2021
ac7c967
eth/protocols/snap: fix race in bytecode handling
holiman Jan 20, 2021
a8e39d1
eth/protocols/snap: undo deduplication of storage roots
holiman Jan 20, 2021
be0ab3b
synctests: refactor + minify panic testcase
holiman Jan 20, 2021
324b8a6
eth/protocols/snap: minor polishes
karalabe Jan 24, 2021
88dc6ee
eth: minor polishes to make logs more useful
karalabe Jan 24, 2021
4aeb629
eth/protocols/snap: remove excessive logs from the test runs
karalabe Jan 24, 2021
a6a131f
eth/protocols/snap: stress tests with concurrency
karalabe Jan 24, 2021
00b8129
eth/protocols/snap: further fixes to test cancel channel handling
karalabe Jan 24, 2021
2aafa6a
eth/protocols/snap: extend test timeouts on CI
karalabe Jan 25, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
eth/protocols/snap: fix remaining revertal-issues
  • Loading branch information
holiman committed Jan 22, 2021
commit cf8b96b49892e4922f36a070912f6148c3c9d924
4 changes: 4 additions & 0 deletions eth/protocols/snap/sync.go
Original file line number Diff line number Diff line change
Expand Up @@ -2392,6 +2392,8 @@ func (s *Syncer) OnTrieNodes(peer PeerIF, id uint64, trienodes [][]byte) error {
}
// We've either ran out of hashes, or got unrequested data
logger.Warn("Unexpected healing trienodes", "count", len(trienodes)-i)
// Signal this request as failed, and ready for rescheduling
s.scheduleRevertTrienodeHealRequest(req)
return errors.New("unexpected healing trienode")
}
// Response validated, send it to the scheduler for filling
Expand Down Expand Up @@ -2484,6 +2486,8 @@ func (s *Syncer) onHealByteCodes(peer PeerIF, id uint64, bytecodes [][]byte) err
}
// We've either ran out of hashes, or got unrequested data
logger.Warn("Unexpected healing bytecodes", "count", len(bytecodes)-i)
// Signal this request as failed, and ready for rescheduling
s.scheduleRevertBytecodeHealRequest(req)
return errors.New("unexpected healing bytecode")
}
// Response validated, send it to the scheduler for filling
Expand Down