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.

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

Gas Limit reported lower than the Gas Used of a block #558

Closed
infsaulo opened this issue Sep 19, 2024 · 1 comment · Fixed by #564
Closed

Gas Limit reported lower than the Gas Used of a block #558

infsaulo opened this issue Sep 19, 2024 · 1 comment · Fixed by #564
Assignees
@m-Peter
Labels
Bug Something isn't working EVM

Comments

@infsaulo
Copy link

Problem

For block at height 755106, the gasLimit value is lower than the gasUsed. Wondering, based on EIP-1559 if the gasLimit should be bumped up to the max cap of 30M, as gasUsed target more than 15M of the limit.

Steps to Reproduce

curl https://mainnet.evm.nodes.onflow.org -X POST -H "Content-Type: application/json" -d '{"jsonrpc":"2.0","method":"eth_getBlockByNumber","params":["0xB85A2", false],"id":1}' | jq

Context

Running a block parser and found under the validation step gasLimit < gasUsed
@infsaulo infsaulo changed the title Gas Used reported higher than the Gas Limit of a block Gas Limit reported lower than the Gas Used of a block Sep 19, 2024
@m-Peter m-Peter self-assigned this Sep 19, 2024
@m-Peter m-Peter added Bug Something isn't working EVM and removed Feedback labels Sep 19, 2024
@m-Peter
Copy link
Collaborator

m-Peter commented Sep 19, 2024

@infsaulo Nice catch 💯 We need to update that constant for the block gas limit. And I guess it might go over 30M as well.

@m-Peter m-Peter mentioned this issue Sep 20, 2024
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@m-Peter m-Peter

Labels
Bug Something isn't working EVM
Projects
🌊 Flow 4D
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update constant for block gas limit onflow/flow-evm-gateway
2 participants
@infsaulo @m-Peter