-
Notifications
You must be signed in to change notification settings - Fork 820
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
main/cli: add mordor classic testnet configuration #192
Conversation
Signed-off-by: soc1c <soc1c@users.noreply.github.com>
Signed-off-by: soc1c <soc1c@users.noreply.github.com>
Signed-off-by: soc1c <soc1c@users.noreply.github.com>
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
Gonna have to roll this back. It doesn't work and fails a field test. (a) Setting classic block to zero requires all blocks at zero to have the mainnet 1920000 hash. That will never happen.
Probably a bug in besu code, but until that bug is sussed out we can't support mordor. |
(a) Setting classic block to zero requires all blocks at zero to have the mainnet 1920000 hash. That will never happen. No clients will sync as long as this is in place (b) a full sync fails ``` 2019-11-18 10:46:51.301-07:00 | EthScheduler-Services-5 (importBlock) | WARN | MainnetBlockBodyValidator | Invalid block: state root mismatch (expected=0xb8df31366db4c4dd0076bc0ddaffdeceba6c882681108cc000db9bc5f85c300c, actual=0xf4344e62f031897ff61f9e02a465b3d13f983220f52f1e452690448dc366a2ee) ``` Probably a bug in besu code, but until that bug is sussed out we can't support mordor. Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
Yes this was not ready to be merged. I should have marked this as Draft. I was waiting for @edwardmack to add the Agharta checks. Happy to revisit this. If Mordor fails, Classic would also fail. I believe it's rather an issue with the spec than the besu code. |
(a) Setting classic block to zero requires all blocks at zero to have the mainnet 1920000 hash. That will never happen. No clients will sync as long as this is in place (b) a full sync fails ``` 2019-11-18 10:46:51.301-07:00 | EthScheduler-Services-5 (importBlock) | WARN | MainnetBlockBodyValidator | Invalid block: state root mismatch (expected=0xb8df31366db4c4dd0076bc0ddaffdeceba6c882681108cc000db9bc5f85c300c, actual=0xf4344e62f031897ff61f9e02a465b3d13f983220f52f1e452690448dc366a2ee) ``` Probably a bug in besu code, but until that bug is sussed out we can't support mordor. Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
Signed-off-by: soc1c <soc1c@users.noreply.github.com>
(a) Setting classic block to zero requires all blocks at zero to have the mainnet 1920000 hash. That will never happen. No clients will sync as long as this is in place (b) a full sync fails ``` 2019-11-18 10:46:51.301-07:00 | EthScheduler-Services-5 (importBlock) | WARN | MainnetBlockBodyValidator | Invalid block: state root mismatch (expected=0xb8df31366db4c4dd0076bc0ddaffdeceba6c882681108cc000db9bc5f85c300c, actual=0xf4344e62f031897ff61f9e02a465b3d13f983220f52f1e452690448dc366a2ee) ``` Probably a bug in besu code, but until that bug is sussed out we can't support mordor. Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
Signed-off-by: soc1c <soc1c@users.noreply.github.com> Signed-off-by: edwardmack <ed@edwardmack.com>
(a) Setting classic block to zero requires all blocks at zero to have the mainnet 1920000 hash. That will never happen. No clients will sync as long as this is in place (b) a full sync fails ``` 2019-11-18 10:46:51.301-07:00 | EthScheduler-Services-5 (importBlock) | WARN | MainnetBlockBodyValidator | Invalid block: state root mismatch (expected=0xb8df31366db4c4dd0076bc0ddaffdeceba6c882681108cc000db9bc5f85c300c, actual=0xf4344e62f031897ff61f9e02a465b3d13f983220f52f1e452690448dc366a2ee) ``` Probably a bug in besu code, but until that bug is sussed out we can't support mordor. Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com> Signed-off-by: edwardmack <ed@edwardmack.com>
PR description
Adds Mordor Classic Testnet support.
Fixed Issue(s)
eth-classic/mordor#3