-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[DO NOT MERGE] Gnosis dencun0 devnet configs #8465
Conversation
Please merge the latest |
Done |
"londonBlock": 0, | ||
"eip1559FeeCollectorTransition": 0, | ||
"eip1559FeeCollector": "0x1559000000000000000000000000000000000000", | ||
"terminalTotalDifficulty": 340282366920938463463374607431768211454, |
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.
terminalTotalDifficulty should be 0
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.
LGTM
"0xcC4e00A72d871D6c328BcFE9025AD93d0a26dF51": { | ||
"balance": "0xc9f2c9cd04674edea40000000" | ||
}, | ||
"0x1000000000000000000000000000000000000001": { |
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.
You need to add balances for a few precompiles:
"0x0000000000000000000000000000000000000001": {
"balance": "1"
},
"0x0000000000000000000000000000000000000002": {
"balance": "1"
},
"0x0000000000000000000000000000000000000003": {
"balance": "1"
},
"0x0000000000000000000000000000000000000004": {
"balance": "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.
Yes, I already did that and got the hash right.
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.
Please check in the commit so that we could share this branch with Gnosis.
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.
Done
Existing gno_dencun_0 nodes (supposed) don't comply with the latest specs on MAX_BLOB_GAS_PER_BLOCK in 4844 specs, so we may skip this devnet and shoot for devnet_1 |
No description provided.