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

Full BSC support with validator mode #3233

Merged
merged 88 commits into from
Jan 14, 2022
Merged

Full BSC support with validator mode #3233

merged 88 commits into from
Jan 14, 2022

Conversation

dmitry123
Copy link
Contributor

This PR includes BSC support for Erigon blockchain node implementation:

  • Migrated Parlia consensus engine
  • Added genesis block generation and peer discovery settings for bsc, chapel and rialto
  • Fixed bug with modifying mutable balance object in inter block state
  • Full support of all BSC forks
  • Validator mode for Parlia consensus
  • BSC and Parlia under flags
  • Fully compatible with ETH mode

This is a copy of this PR but from this remote's branch: #3144

dmitry123 and others added 30 commits November 23, 2021 22:29
[*] fixed bug with loading snapshot
[+] enabled gas bailout
[+] added fix to prevent syncing more than 1000 headers (for testing only)
[*] fixed bug with crashing sender recover sometimes
[+] migrated lightclient contracts from bsc
[*] fixed parlia consensus config param
[*] fixed syscalls from coinbase
[*] more logging
[*] set header checkpoint to 10 million blocks (for testing forks)
…em transaction execution

[-] removed some extra logs and comments
@const-tmp
Copy link

const-tmp commented Jan 13, 2022

Trying to sync Chapel testnet and sometimes getting this error in logs:

[EROR] [01-13|13:20:50.331] Staged Sync error="runtime error: index out of range [46] with length 46, trace: [stageloop.go:120 panic.go:1047 panic.go:90 header_data_struct.go:117 heap.go:93 heap.go:86 header_algos.go:593 stage_headers.go:423 stage_headers.go:121 default_stages.go:20 sync.go:318 sync.go:235 stageloop.go:154 stageloop.go:58 asm_amd64.s:1581]"

Build info git_branch=bsc git_tag=v2022.01.01 git_commit=78bdc3b70202fdfc62637bbde8a081fb1bfbd734

@AlexeyAkhunov
Copy link
Contributor

Trying to sync Chapel testnet and sometimes getting this error in logs:

[EROR] [01-13|13:20:50.331] Staged Sync error="runtime error: index out of range [46] with length 46, trace: [stageloop.go:120 panic.go:1047 panic.go:90 header_data_struct.go:117 heap.go:93 heap.go:86 header_algos.go:593 stage_headers.go:423 stage_headers.go:121 default_stages.go:20 sync.go:318 sync.go:235 stageloop.go:154 stageloop.go:58 asm_amd64.s:1581]"

Build info git_branch=bsc git_tag=v2022.01.01 git_commit=78bdc3b70202fdfc62637bbde8a081fb1bfbd734

Thanks for reporting, I will add fix for that too

@AskAlexSharov
Copy link
Collaborator

Similar error here: #3258

@dmitry123
Copy link
Contributor Author

@AskAlexSharov Why you've removed map size for chaindata database?

900aef2#diff-375d57e386f20eaa5f09f02bb9d28bfc48ac3dca18d0325f59492208219e5618L525

@AlexeyAkhunov
Copy link
Contributor

@AskAlexSharov Why you've removed map size for chaindata database?

900aef2#diff-375d57e386f20eaa5f09f02bb9d28bfc48ac3dca18d0325f59492208219e5618L525

We need to think about/discuss it separately. As we are having an issue with 4Tb size anyway. I am not against this change, but I would like to bring it in separately, because it may affect not just BSC

@AlexeyAkhunov AlexeyAkhunov merged commit a49d409 into devel Jan 14, 2022
@AlexeyAkhunov AlexeyAkhunov deleted the bsc branch January 14, 2022 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants