-
-
Notifications
You must be signed in to change notification settings - Fork 289
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
feat: add max activation churn limit and other deneb devnet-9 spec updates #5958
Conversation
886bc4c
to
7200d73
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.
lgtm
Performance Report✔️ no performance regression detected Full benchmark results
|
working on the failing spec tests + modifying spec tests for deneb forkchoice however some tests in new spec version are erroneously failing issues like change for e.g. is in validator 30 's expected exit epoch and withdrawal epoch ( {
"activation_eligibility_epoch": "0"
"activation_epoch": "0"
"effective_balance": "32000000000"
- "exit_epoch": "5"
+ "exit_epoch": "6"
"pubkey": "0xb29043a7273d0a2dbc2b747dcf6a5eccbd7ccb44b2d72e985537b117929bc3fd3a99001481327788ad040b4077c47c0d"
"slashed": true
- "withdrawable_epoch": "261"
+ "withdrawable_epoch": "262"
"withdrawal_credentials": "0x00a4c7cea67274a6e12bcb8b3108b84b5201dd10d4b3e8e4d712c21d2f2a8fdf"
}
Escalated the issue in telegram channel UPDATE ISSUE FIXED with minimal churn param update |
0b477b0
to
1d55b82
Compare
🎉 This PR is included in v1.12.0 🎉 |
Motivation
Description
Add max activation churn limit. Should wait for spec tests release next Monday.