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

Revert "Update fastssz" #7100

Merged
merged 2 commits into from
Aug 24, 2020
Merged

Revert "Update fastssz" #7100

merged 2 commits into from
Aug 24, 2020

Conversation

terencechain
Copy link
Member

Reverts #6760

My proposers were failing to propose blocks:

[2020-08-24 11:41:22] ERROR validator: Failed to propose block error=rpc error: code = Internal desc = Could not process beacon block: could not process block: could not execute state transition: could not process block in slot 145706: could not process block header: signature did not verify pubKey=0xb8d5ccb51ad0

I walked through the recent comments and found this to be the potential offender. I reverted this for run time and verified proposing blocks has indeed worked without this PR. Reverting this while spending time to debug what is wrong

@terencechain terencechain requested a review from a team as a code owner August 24, 2020 19:12
@prylabs-bulldozer prylabs-bulldozer bot merged commit b954db9 into master Aug 24, 2020
terencechain added a commit that referenced this pull request Aug 25, 2020
@farazdagi farazdagi deleted the revert-6760-update-fssz branch August 25, 2020 15:47
@prestonvanloon prestonvanloon mentioned this pull request Aug 26, 2020
2 tasks
prylabs-bulldozer bot pushed a commit that referenced this pull request Aug 27, 2020
* Revert "Revert "Update fastssz" (#7100)"

This reverts commit b954db9.
* Preston's patch
* Merge branch 'master' of github.com:prysmaticlabs/prysm into revert-7100-revert-6760-update-fssz
* Update fssz, add regression test case
* more HTR with fssz
* fix some tests
* only one test left
* Make it so that HTR will work
* gofmt, imports
* gofmt, imports
* fix
* Merge branch 'master' of github.com:prysmaticlabs/prysm into revert-7100-revert-6760-update-fssz
* fix
* Merge branch 'master' into revert-7100-revert-6760-update-fssz
* Merge refs/heads/master into revert-7100-revert-6760-update-fssz
* gaz
* Merge branch 'revert-7100-revert-6760-update-fssz' of github.com:prysmaticlabs/prysm into revert-7100-revert-6760-update-fssz
* Merge refs/heads/master into revert-7100-revert-6760-update-fssz
* fix test
* Merge branch 'revert-7100-revert-6760-update-fssz' of github.com:prysmaticlabs/prysm into revert-7100-revert-6760-update-fssz
* Merge refs/heads/master into revert-7100-revert-6760-update-fssz
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.

2 participants