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

Eth1Data calculation error #1335

Open
rodrigo-o opened this issue Oct 18, 2024 · 0 comments
Open

Eth1Data calculation error #1335

rodrigo-o opened this issue Oct 18, 2024 · 0 comments
Labels
bug Something isn't working P0 This should be tackled NOW as in TODAY

Comments

@rodrigo-o
Copy link
Collaborator

After starting having long running Sepolia sesions being relatively stable we started to have long running local devnet session as well. Now we are able to find new issues related to validators, here is the first one:

** (MatchError) no match of right hand side value: {MapSet.new([%Types.Eth1Data{deposit_root:
...
6, 250, 91, 132, 49, 23, 22, 223, 209, 2, 41, 210, 137, 55, 249>>}}
    (lambda_ethereum_consensus 0.1.0) lib/lambda_ethereum_consensus/execution/execution_chain.ex:230: LambdaEthereumConsensus.Execution.ExecutionChain.get_first_valid_vote/5
    (lambda_ethereum_consensus 0.1.0) lib/lambda_ethereum_consensus/validator/block_builder.ex:310: LambdaEthereumConsensus.Validator.BlockBuilder.fetch_eth1_data/2
    (lambda_ethereum_consensus 0.1.0) lib/lambda_ethereum_consensus/validator/block_builder.ex:42: LambdaEthereumConsensus.Validator.BlockBuilder.build_block/2
    (lambda_ethereum_consensus 0.1.0) lib/lambda_ethereum_consensus/validator/validator.ex:371: LambdaEthereumConsensus.Validator.propose/3
    (elixir 1.16.2) lib/map.ex:916: Map.update!/3
    (lambda_ethereum_consensus 0.1.0) lib/lambda_ethereum_consensus/validator/validator_set.ex:228: LambdaEthereumConsensus.ValidatorSet.maybe_propose/4
    (lambda_ethereum_consensus 0.1.0) lib/libp2p_port.ex:789: LambdaEthereumConsensus.Libp2pPort.on_tick/2
    (lambda_ethereum_consensus 0.1.0) lib/libp2p_port.ex:512: LambdaEthereumConsensus.Libp2pPort.handle_info/2
Last message: :on_tick 

This is a bug in the calculation when multiple eth1data are present.

@rodrigo-o rodrigo-o added bug Something isn't working P0 This should be tackled NOW as in TODAY labels Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working P0 This should be tackled NOW as in TODAY
Projects
Status: No status
Development

No branches or pull requests

1 participant