-
Notifications
You must be signed in to change notification settings - Fork 5
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
mainnet: change layout #206
Conversation
f172b67
to
d50ec55
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems mostly fine to me.
# Ports used for all 3 EL nodes as they are mutually exclusive. | ||
exec_layer_p2p_port: '{{ 30303 + 3*idx|int + 1 }}' | ||
exec_layer_rpc_port: '{{ 8545 + idx|int + 1 }}' | ||
exec_layer_authrpc_port: '{{ 8551 + idx|int + 1 }}' | ||
exec_layer_metrics_port: '{{ 6060 + idx|int + 1 }}' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would put this at the top, below swap but above Geth settings, since it's not just for Geth.
3bc5be4
to
7679d21
Compare
Change layout:
Cleanup checklist:
|
Issues:
|
❯ ./rpc.sh eth_blockNumber
bash: warning: setlocale: LC_ALL: cannot change locale (en_GB.UTF-8)
{
"jsonrpc": "2.0",
"id": 1,
"result": "0x0"
}
❯ sudo du -sh erigon-mainnet-*
14M erigon-mainnet-stable
12M erigon-mainnet-unstable |
I think we can merge the layout changes even if some nodes are not fully synced, and follow that up int he issue(which should be linked in this PR and commit). |
321b2ed
to
547fc64
Compare
547fc64
to
a0dcb9c
Compare
a0dcb9c
to
e6518f0
Compare
No description provided.