Skip to content

feat: electra MAX_BLOBS_PER_BLOCK_ELECTRA #1438

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 121 commits into from
Apr 25, 2025
Merged

feat: electra MAX_BLOBS_PER_BLOCK_ELECTRA #1438

merged 121 commits into from
Apr 25, 2025

Conversation

LeanSerra
Copy link
Contributor

@LeanSerra LeanSerra commented Apr 21, 2025 •
edited
Loading

Motivation

4 tests were failing because of Process block failed, error: Too many commitments

Description

  • Add new config MAX_BLOBS_PER_BLOCK_ELECTRA: 9
  • Enable ci to run with electra

Fixed Spec Tests

test/generated/mainnet/electra/sanity.exs:
previous: 97 tests, 2 failures, 1 skipped
now: 97 tests, 0 failures, 1 skipped

test/generated/minimal/electra/sanity.exs:
previous: 104 tests, 2 failures, 1 skipped
now: 104 tests, 0 failures, 1 skipped

tests fixed: 4

Spec Test Progress

11370 tests, 0 failures, 784 skipped

LeanSerra added 30 commits April 3, 2025 11:21
@LeanSerra
feat EpochProcessing.process_slashings electra changes
ed74379
@LeanSerra
feat compute_proposer_index changes
71b3674
@LeanSerra
feat compute_sync_committees Electra changes
9aa4202
@LeanSerra
fix fork_choice.ex tests were flaky because env var cleanup was not done
f09b71b 
in fails
@LeanSerra
feat eligible_for_activation_queue Electra changes
df45ee4
@LeanSerra
feat new compounding_withdrawal_credential? Electra function
8fb4d63
@LeanSerra
feat new has_compounding_withdrawal_credential Electra function
6e34108
@LeanSerra
feat new has_execution_withdrawal_credential Electra function
620e700
@LeanSerra
feat fully_withdrawable_validator? Electra changes
d904593
@LeanSerra
feat new get_max_effective_balance Electra function
5445583
@LeanSerra
feat partially_withdrawable_validator? Electra changes
5cbc5af
@LeanSerra
fix typo in struct field in has_compounding_withdrawal_credential
753badb
@LeanSerra
feat new get_committee_indices Electra function
29bf08a
@LeanSerra
feat get_attesting_indices Electra changes
17bf479
@LeanSerra
lint remove unnecessary parentheses in if
42b0bfb
@LeanSerra
feat slash_validator Electra changes
6df1b36
@LeanSerra
feat new get_balance_churn_limit Electra function
9e13ad5
@LeanSerra
feat new get_activation_exit_churn_limit Electra function
abc2585
@LeanSerra
feat compute_exit_epoch_and_churn Electra function
5b7e5dc
@LeanSerra
feat initiate_validator_exit Electra changes
4eddb4b
@LeanSerra
feat process_registry_updates Electra changes
44a00ec
@LeanSerra
refactor remove unused eject_validator(_,_,_, false) variant
9bc3a0b
@LeanSerra
fix compute_exit_epoch_and_update_churn wrong exit_balance calculation
ed8be8a
@LeanSerra
Merge branch 'electra-support' into electra_predicates
c084913
@LeanSerra
fix additional_epochs calculation in compute_exit_epoch_and_update_churn
6ec0216
@LeanSerra
fix wrong values for constants in minimal config
7da3d5a
@LeanSerra
Merge branch 'electra_predicates' into electra_validator_exit
70d40aa
@LeanSerra
feat process_epoch changes + deposits/consolidation scaffolding
e4e8607
@LeanSerra
feat new is_valid_deposit_signature Electra function
0e53f57
@LeanSerra
feat get_validator_from_deposit Electra changes
12ed2d3
@LeanSerra LeanSerra added elixir Pull requests that update Elixir code electra labels Apr 21, 2025
@LeanSerra LeanSerra self-assigned this Apr 21, 2025
Base automatically changed from electra_withdrawals to electra-support April 25, 2025 17:39
@LeanSerra LeanSerra marked this pull request as ready for review April 25, 2025 17:42
@LeanSerra LeanSerra requested a review from a team as a code owner April 25, 2025 17:42
rodrigo-o
rodrigo-o approved these changes Apr 25, 2025
View reviewed changes
Copy link
Collaborator

@rodrigo-o rodrigo-o left a comment

Choose a reason for hiding this comment

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

🚀

@LeanSerra LeanSerra merged commit e56e7d4 into electra-support Apr 25, 2025
13 checks passed
@LeanSerra LeanSerra deleted the electra_max_blobs branch April 25, 2025 20:19
rodrigo-o pushed a commit that referenced this pull request Apr 25, 2025
@LeanSerra @rodrigo-o
feat: electra MAX_BLOBS_PER_BLOCK_ELECTRA (#1438)
90c5aac
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rodrigo-o rodrigo-o rodrigo-o approved these changes

Assignees

@LeanSerra LeanSerra

Labels
electra elixir Pull requests that update Elixir code
Projects
Lambda Ethereum Consensus Node
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@LeanSerra @rodrigo-o