maticnetwork / bor

Official repository for the Polygon Blockchain

https://polygon.technology/

GNU Lesser General Public License v3.0

986 stars 476 forks source link

core, eth, internal, miner, params: enforce 30gwei for gas related configs in bor #1233

Closed marcello33 closed 2 months ago

marcello33 commented 2 months ago

Description

This PR enforces the params txpool.pricelimit, miner.gasprice and gpo.ignoreprice to be set to 30 gwei.

Changes

[ ] Bugfix (non-breaking change that solves an issue)
[ ] Hotfix (change that solves an urgent issue, and requires immediate attention)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)
[ ] Changes only for a subset of nodes

Checklist

[x] I have added at least 2 reviewer or the whole pos-v1 team
[x] I have added sufficient documentation in code
[x] I will be resolving comments - if any - by pushing each fix in a separate commit and linking the commit hash in the comment reply
[ ] Created a task in Jira and informed the team for implementation in Erigon client (if applicable)
[ ] Includes RPC methods changes, and the Notion documentation has been updated

Cross repository changes

Related erigon PR: https://github.com/ledgerwatch/erigon/pull/

Testing

[x] I have added unit tests
[ ] I have added tests to CI
[x] I have tested this code manually on local environment
[ ] I have tested this code manually on remote devnet using express-cli
[ ] I have tested this code manually on mumbai/amoy
[ ] I have created new e2e tests into express-cli