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

Update Defult MaxGasPrice #12

Closed
wants to merge 135 commits into from
Closed

Conversation

gr8h
Copy link
Collaborator

@gr8h gr8h commented May 1, 2024

Changes

Update MaxGasPrice constant to 50 GEWI

Types of changes

What types of changes does your code introduce?

  • Bugfix (a non-breaking change that fixes an issue)
  • New feature (a non-breaking change that adds functionality)
  • Breaking change (a change that causes existing functionality not to work as expected)
  • Optimization
  • Refactoring
  • Documentation update
  • Build-related changes
  • Other: Description

4rgon4ut and others added 30 commits December 6, 2023 12:31
…as` is null (NethermindEth#6320)

* fix

* fix for aura tests

* Apply Alexey suggestion

* revert statetest1
* Init

* minor cleanup

* Use regex

* fix helpText
* added chains joc json files

* added cfg file

* PR review changes

* few review fix

* removed default rpc as per review comments

* removed timeout as its default

* Add better configs, launch settings and syncSettings for joc

* Change ref from static to selected (helps in testing)

* Add missing port

* Bump action version

* Add github.ref

* Rollback branch

* Add better PR desc

---------

Co-authored-by: Kamil Chodoła <[email protected]>
* init

* tests

* format

* fix config tests

* cover more cases

* Add same logic to SyncProgressResolver with tests

* wip metadatadb usage to detect with what configuration was the sync started.

* add logic to ReceiptsSyncFeed

* fix ReceiptsSyncFeed

* apply fix

* remove useless using

* Fix bodies not finishing

* refactor and minor fixes

* fix test issue
* Use better processing timer for logging (NethermindEth#5843)

* Use different processing timer

* Use single queue

* Revert to simpler

* Don't use DarkGray (NethermindEth#5849)

* Bump to 1.20.0-rc

* Update DotNetty feed

(cherry picked from commit 3e330ad)

* Update DotNetty package

(cherry picked from commit 47a92ee)

* minor fixes

* Update version to 1.20.0

* Bump to 1.20.1

* Add workflow to compare rpcs

Fix path

Run on push temporarily

Fix file path

Fix inputs and missing vars

All fields not required

Inherit secrets when calling run-a-single-node-from-branch

Add default for cl_client

Use base_tag generated by run-a-single-node

Download artifacts from other post-merge-smoke-tests

Remove the old download-artifact

Change the name of token secret

Remove usage of external action

Allow to specify smoke tests branch to use

Remove additional option to fit dispatch limit

fix: wrong inputs string format

Export run id

Fix rpc url

Fix debug

Add runner's ip to `allowed_ips`

Maximum 10 inputs is allowed in workflow_dispatch

Add allowed_ips to workflow_dispatch inputs

Try to join two ips

Print `,` conditionally

Update public-ip

Debug

Use VPN

Add wireguard.conf

Fix location

Checkout the repository before installing wireguard

Update wireguard.conf to match nethermind repo secrets

Update environment variables to match secrets

Install flood before wireguard

Use include in matrix to specify both name and address

Revert "Use include in matrix to specify both name and address"

This reverts commit 71caa0c.

* Trigger workflow?

* Fix run-a-single-node-from-branch.yml

* Update branch name

* Wait for the node to sync

* Add a separate job to wait for node sync

* Fix missing checkout

* Use validator mode and set Pruning to None

* Increase timeout of wait to sync to 10 hours

* Change step timeout as well

* Wait 5.5h before running `wait_for_node_to_sync`
For github hosted runners time limit for a job is 6 hours.
It is less than a node needs to sync.
Thus we wait 5.5 hours before checking the sync status of node.

* Fix needs

* Wait another 5.5h

* Fix indentation

* Remove sleep2

* Remove debug info

* Change approach to multi nodes

* Fix desc

* Fix branch name for trigger

* Add logs

* Add hardcoded value

* Fix matrix

* Change matrix approach

* Fix json creation

* Fix path

* Select branch

* Absolut path

* revert path

* static path

* Remove ref

* bump checkout

* add logs

* Add fethc depth

* change debug log

* More logs

* Change logs

* Change back to old version of run0single-node

* revert changes

* Change job

* Add matrix to path

* Adjust ref

* fix allowed_ips

* missing needs

* Fix trigger job

* Use better processing timer for logging (NethermindEth#5843)

* Use different processing timer

* Use single queue

* Revert to simpler

* Don't use DarkGray (NethermindEth#5849)

* Bump to 1.20.0-rc

* Update DotNetty feed

(cherry picked from commit 3e330ad)

* Update DotNetty package

(cherry picked from commit 47a92ee)

* minor fixes

* Update version to 1.20.0

* Bump to 1.20.1

* Add workflow to compare rpcs

Fix path

Run on push temporarily

Fix file path

Fix inputs and missing vars

All fields not required

Inherit secrets when calling run-a-single-node-from-branch

Add default for cl_client

Use base_tag generated by run-a-single-node

Download artifacts from other post-merge-smoke-tests

Remove the old download-artifact

Change the name of token secret

Remove usage of external action

Allow to specify smoke tests branch to use

Remove additional option to fit dispatch limit

fix: wrong inputs string format

Export run id

Fix rpc url

Fix debug

Add runner's ip to `allowed_ips`

Maximum 10 inputs is allowed in workflow_dispatch

Add allowed_ips to workflow_dispatch inputs

Try to join two ips

Print `,` conditionally

Update public-ip

Debug

Use VPN

Add wireguard.conf

Fix location

Checkout the repository before installing wireguard

Update wireguard.conf to match nethermind repo secrets

Update environment variables to match secrets

Install flood before wireguard

Use include in matrix to specify both name and address

Revert "Use include in matrix to specify both name and address"

This reverts commit 71caa0c.

* Trigger workflow?

* Fix run-a-single-node-from-branch.yml

* Update branch name

* Wait for the node to sync

* Add a separate job to wait for node sync

* Fix missing checkout

* Use validator mode and set Pruning to None

* Increase timeout of wait to sync to 10 hours

* Change step timeout as well

* Wait 5.5h before running `wait_for_node_to_sync`
For github hosted runners time limit for a job is 6 hours.
It is less than a node needs to sync.
Thus we wait 5.5 hours before checking the sync status of node.

* Fix needs

* Wait another 5.5h

* Fix indentation

* Remove sleep2

* Remove debug info

* Change approach to multi nodes

* Fix branch name for trigger

* Fix desc

* Add logs

* Add hardcoded value

* Fix matrix

* Change matrix approach

* Fix json creation

* Fix path

* Select branch

* Absolut path

* revert path

* static path

* Remove ref

* bump checkout

* add logs

* Add fethc depth

* change debug log

* More logs

* Change logs

* Change back to old version of run0single-node

* revert changes

* Change job

* Add matrix to path

* Adjust ref

* fix allowed_ips

* missing needs

* Fix trigger job

* Revert bad rebase changes

* Fixed workflow for testing

* Fix needs

* Fix outputs

* add missing $

* Add more logs

* Change rpc_urls

* Change approach of node creation

* Comment out

* Fix needs

* Change a way of applying artifacts and rpc_urls

* Adjust needs

* Adjust

* Add missing variables

* Fix invalid file read

* Fix temp infura value

* Fix to much  '

* Fix artifact save

* Fix branches fetching

* Temp remove needs

* Switch to first job needs

* Fix list fill

* Invalid path

* Add verbose logging

* Adjust

* add missing repo

* Rollback run_id

* Fix needs

* Trigger

* Fix needs

* Change apprioach of downloading artifacts

* Add more params to artifact download

* Download all from current run

* Change the runid

* Add search

* Fix typo

* Change way of downlaoding artifacts

* Test a different run-id

* Change approach

* Add mising custom id

* Change approach to only 1 branch compare

* Fix

* Fix2

* add if commented

* Fetch only most recent run for selected ref

* Change name of job

* Fix a ref_name displayed

* Fix ref name

* Cleanup

* Fix typo

* Clean refs

* Change artifacts download path

* Change action temporarily

* Add path param

* Switch to david action type

* Trigger new nodes

* Rollback needs

* Fix if check for docker build creation

* Rolback to classic artifacts downoad

* Add better docker image check

* Add missing if condition

* Fix paths and add temp sepolia

* Change delimiter

* Adjust waiting for node

* Fix wrong output

* Mock and test

* Fix issue

* Fix issue2

* Fix issue

* Remove deps

* Fix invalid "

* Revert comments back

* Test on existing infra

* Fix bad replace

* Change aproach of running

* Fix failure

* Add a better order of artifacts on a rpc_list

* Adjust

* Fix

* Rollback to env

* Adjust flood script

* Fix rpc_urls creation

* Hardcode urls for test

* Comment out needs

* Add missing equality tests

* Change to kch flood fork

* Rollback to official flood

* Generate new nodes

* Add misiing extension

* adjist script and add logs

* Fix typo

* Test

* Change approach slightly

* One mroe adjustment

* Adjust to mainnet and add missing envs

* Bump timeout to something higher to fit tests

* Clean Up

* Add missing changes

---------

Co-authored-by: Ben Adams <[email protected]>
Co-authored-by: Kamil Chodoła <[email protected]>
Co-authored-by: Ruben Buniatyan <[email protected]>
Co-authored-by: Marcin Sobczak <[email protected]>
Co-authored-by: Piotr Piwoński <piwonskp>
Co-authored-by: Piotr Piwoński <[email protected]>
* Separate statedb initialization from blockchain initialization

* Minor cleanup

* World state factory

* Don't take db provider or trie store

* Use state reader instead of trie store for full state finder

* Unify and use property

* Boundary watcher

* Dont need this anymore

* Minor comment

* Reducing change

* Missed a setter

* Some tests

* Fix build

* Fix build

* Addressing comment

* Minor comment adjustment

* Fix startup block tree fixer missed
* Also check withdrawals before insertion

* refactor

* refactor

* unit tests

* unnecessary using
* System.Text.Json.Serialization

* System.Text.Deserialization

* Fixes

* lint

* Fixes

* Fix

* Fixes

* Fixes

* Lint

* Fix invalid Json

* u8

* Fixes

* Fixes

* Fix

* Don't go via exception for short reverts

* Fix

* Fix

* Multi-doc batch

* Lint

* Merge conflict

* Fixes

* Lint

* Fix tests

* Fix test

* Tidy up

* Turn of sync IO

* Async

* Don't double Advance

* Async wrapper stream use

* Fix build break

* Merge issues

* Compile issues

* File scoped namespaces

* Formatting

* Merge fixes

* Test Merge fixes

* lang ver

* Purge Newtonsoft

* Fix evm tests

* Fix parentBeaconBlockRootString serialization

* Change type for serialization

* Ignore syncing

* Capitalised bools are invalid json

* Capitalised bools are invalid json

* Fix merge conflict

* Fix data serialization

* Fix trace entry

* Merge conflicts

* Formatting

* Missed merge

* whitespace

* Fix EthereumTests Json

* Ethereum tests

* Fix hex prefix

* Fix difficultly test

* Add serializer options

* Fix abi tests

* Merge conflict

* Feedback

* revert to previous

* Remove Netwonsoft

* cosmetics

* Pipeline Json responses

* Merge fixes

* Whitespace

* fix test

* Send multiple messages

* Optional stream closing

* Case insensitive

* Update Nethermind.Serialization.Json.csproj

* Whitespace

* Update JavaScriptObjectConverter.cs

* Feedback

* Undo AI

* Put orders back

* Feedback

* Whitespace

* Use static constructor

* Remove irrelevant test parameter

* Add BigIntegerConverterTests back

* Use FrozenDictionary

* Deserialize collection to ArrayPoolList using GetArrayLength for initial capacity. Dispose it later.

Move WhiteSpace to utf8

* simplify

* Throw more explicit exception

* name cleanup

* Merge conflicts

* Merge conflict

* Add comments

* more comments

* Consolidate Disposable

* Use FrozenDictionary for RpcLookup

* Case sensitive

* Be consistent on parsing type

* Use correct type in test

* Optimizations

* Optimizations

* Optimize

* Optimize

* Vectorize FromUtf8 hex

* Optimization

* Optimize

* Parallel deserialize

* Revert "Parallel deserialize"

This reverts commit 94b08e3.

* Use using

* Fix check

* Parallel deserialize

* Increase parallelThreshold

* Use correct length

* Cache Reflection Parameters

* Fix tracing output

* Match geth output

* Turn AllowSynchronousIO back on

* Revert object type arrays

---------

Co-authored-by: lukasz.rozmej <[email protected]>
* Disable networking on OpStack configs

* Disable networking config for OP Stack testnets
* RLP refactor/cleanup (NethermindEth#6243)

Co-authored-by: Kamil Chodoła <[email protected]>

* Remove allowLeadingZeros

* Fix Keccak -> Hash256

* fix for optimism

---------

Co-authored-by: Kamil Chodoła <[email protected]>
Co-authored-by: Marcin Sobczak <[email protected]>
* Fix json rpc missing parameter check

* Add more check when enumerating

* Modify tests to serialize null on net_version
* Fix excessive timer allocation in rate limiter

* Looser match

* Rename to currentNextSlot
* typo fix

* typo fix

* typo fix

* typo fix
* Serialize Json direct to Http PipeWriter

* Whitespace
* Hive fixes

* invalid params

* Wrong place

* Only return for engine_newPayloadV3

* Make ExecutionPayloadV3.BlobGasUsed and ExcessBlobGas required

* small cleanup

* overload instead of override

* Revert "overload instead of override"

This reverts commit 27c3686.

* Revert "Revert "overload instead of override""

This reverts commit ab921f9.

* Revert "Revert "Revert "overload instead of override"""

This reverts commit 782200e.

---------

Co-authored-by: lukasz.rozmej <[email protected]>
…rent base fee (NethermindEth#6350)

* limit initial broadcast

* add BaseFeeThreshold to ChainHeadInfoProvider

* simplify broadcaster

* add simple test

* make BaseFeeThreshold configurable in TxPoolConfig

* remove from ChainHeadInfoProvider

* more tests

* cosmetic

* cosmetic

* fix posdao tests

* Update src/Nethermind/Nethermind.TxPool/ITxPoolConfig.cs

Co-authored-by: Ruben Buniatyan <[email protected]>

---------

Co-authored-by: Ruben Buniatyan <[email protected]>
smartprogrammer93 and others added 28 commits January 22, 2024 16:07
Co-authored-by: Ben Adams <[email protected]>
(cherry picked from commit ecd8a2a)
Co-authored-by: lukasz.rozmej <[email protected]>
Co-authored-by: Marc <[email protected]>

(cherry picked from commit 9b5a3f2)
* Resolved conflicts

* max body size for http json rpc (NethermindEth#36)

* Format

* Self-recovery of TxPool max capacity

* fix ProcessedTransactionsDbCleanerTests

* Update txPool worst value if removal unsuccessful (NethermindEth#6702)

(cherry picked from commit f262cb2)

* Clearer error message for missing starting block state (NethermindEth#6672)

* Update txPool worst value if removal unsuccessful (NethermindEth#6702)

* Add Cancun hard-fork settings for Gnosis (NethermindEth#6709)

* Cancun spec and config for mainnet (NethermindEth#6679)

* Add Paris release spec class for completeness (NethermindEth#6633)

* 0.0.0.0 to +

---------

Co-authored-by: smartprogrammer <[email protected]>
Co-authored-by: Marcin Sobczak <[email protected]>
Co-authored-by: Ben Adams <[email protected]>
Co-authored-by: Ruben Buniatyan <[email protected]>
Co-authored-by: Ahmad Bitar <[email protected]>
Co-authored-by: Kamil Chodoła <[email protected]>
Co-authored-by: Lukasz Rozmej <[email protected]>
* Add fuse support
Why?
Previously, transactions under 10 gwei could lower the average gas price over the last 50 blocks, potentially causing a backlog of stuck transactions. 
Solution:
This PR allow nodes to set a minimum gas price limit. As a result, eth_gasPrice calls will return either the average gas price from the last 50 blocks or this newly set minimum limit, whichever is higher.
Flag: `UseMinGasPriceInEstimates`
@gr8h gr8h closed this May 1, 2024
@gr8h gr8h deleted the feature/production-update-maxgasprice branch July 23, 2024 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.