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

Upstream v2.49.0 #92

Closed
wants to merge 225 commits into from
Closed

Upstream v2.49.0 #92

wants to merge 225 commits into from

Conversation

ImTei
Copy link
Member

@ImTei ImTei commented Sep 22, 2023

Upstream sync PR

erigon

ledgerwatch/erigon v2.49.0 -> testinprod-io/op-erigon dd556a8

erigon-lib

ledgerwatch/erigon-lib dde477f -> testinprod-io/erigon-lib b7b8746

erigon-interfaces

ledgerwatch/interfaces 1fb30d8 -> testinprod-io/erigon-interfaces f2e7a04

mh0lt and others added 30 commits July 2, 2023 22:37
…ontech#7832)

The fixes here fix a couple of issues related to devnet start-up

1. macos threading and syscall error return where causing multi node
start to both not wait and fail
2. On windows creating DB's with the default 2 TB mapsize causes the os
to reserve about 4GB of committed memory per DB. This may not be used -
but is reserved by the OS - so a default bor node reserves around 10GB
of storage. Starting many nodes causes the OS page file to become
exhausted.

To fix this the consensus DB's now use the node's OpenDatabase function
rather than their own, which means that the consensus DB's take notice
of the config.MdbxDBSizeLimit.

This fix leaves one 4GB committed memory allocation in the TX pool which
needs its own MapSize setting.

---------

Co-authored-by: Alex Sharp <[email protected]>
some people are getting confused by this log. 

it is not really an error anyways.
This PR separates ENGINE from Ethbackend. It makes it so:

1) EthBackend not a god class
2) We can abstract away engine API so that we can make it CL-like and
enable Consensus-Execution driven design
3) Objective is Json-RPC -> Engine Consensus Module -> Execution module.
Allows for Caplin to be used with other ELs from aside of Erigon

---------

Co-authored-by: Giulio <[email protected]>
The DAO irregular state change was specific to Ethereum mainnet, while
sys calls are not used there.
Instead of getting the pending block with latest payload id, we just
store the latest block built and serve it in remote backend
…7877)

Fixes erigontech#7814, it can be
really confusing when `erigon` and `integration` process `~` in path in
different ways.

I can't fix it for `integration` the same way it is done for `erigon`,
because in main app it is done with `urfave/cli` package that is not
used in `integration`, so I've used other simplest way to do it.
Giulio2002 and others added 22 commits September 1, 2023 15:29
The FIX is necessary to permit to use API also in case block is in the
snapshot files
This is a non functional change which consolidates the various packages
under metrics into the top level package now that the dead code is
removed.

It is a precursor to the removal of Victoria metrics after which all
erigon metrics code will be contained in this single package.
Problem was errgroup subroutines finished before their logs could
internally trigger
…un prune in sync tx. partial progress loss fix (partial fix) (erigontech#8128)
This is a fix for 

```cache_total{target="acc_read"} counter it's invalid syntax of metric type. prometheus parsing error```

which does not remove the whole metadata tag
@ImTei ImTei requested a review from pcw109550 September 25, 2023 07:37
Copy link

github-actions bot commented Nov 5, 2023

This PR is stale because it has been open for 40 days with no activity.

@github-actions github-actions bot added the Stale label Nov 5, 2023
@pcw109550 pcw109550 removed the Stale label Nov 7, 2023
Copy link

This PR is stale because it has been open for 40 days with no activity.

@github-actions github-actions bot added the Stale label Dec 18, 2023
@ImTei ImTei closed this Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.