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

implement montblanc hard fork #132

Merged
merged 6 commits into from
Sep 27, 2024
Merged

Conversation

jed-wemade
Copy link

@jed-wemade jed-wemade commented Sep 24, 2024

After the block reached to mont blanc fork, mining worker skips committing work for the futher mont blanc blocks.
Instead, warning log appears constantly:

WARN [09-24|01:21:30.277] go-wemix skips mining due to mont blanc fork height=30 parent-hash=11cc16..c0ea5f
WARN [09-24|01:21:31.277] go-wemix skips mining due to mont blanc fork height=30 parent-hash=11cc16..c0ea5f

In addition, mont blanc block excluding the block generated at fork time is filtered out during header verification.

DEBUG[09-25|10:07:14.819] Propagated block verification failed     peer=72c3d5b3a6972776253f2e535dcb52199e6fbf58cce653bcb80eab965e97e2ce number=10,867,700 hash=17583b..92886c err="go-wemix does not support mont blanc fork"
DEBUG[09-25|10:07:14.819] Message handling failed in `eth`         id=72c3d5b3a6972776 conn=staticdial err=EOF
DEBUG[09-25|10:07:14.819] Removing Ethereum peer                   peer=72c3d5b3 snap=true
DEBUG[09-25|10:07:14.819] Message handling failed in `snap`        peer=72c3d5b3 err=EOF
DEBUG[09-25|10:07:14.819] Removing p2p peer                        peercount=1 id=72c3d5b3a6972776 duration=32.627s     req=false err="useless peer"

@jed-wemade jed-wemade added the enhancement New feature or request label Sep 24, 2024
@jed-wemade jed-wemade self-assigned this Sep 24, 2024
@jed-wemade jed-wemade linked an issue Sep 24, 2024 that may be closed by this pull request
Copy link
Collaborator

@egonspace egonspace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jed-wemade jed-wemade merged commit acc69f6 into dev Sep 27, 2024
4 checks passed
@egonspace
Copy link
Collaborator

This merge was reset because it had not been done by squash and merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

implement montblanc hard fork
2 participants