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

Merge/foundation release/1.13.5 #593

Merged
merged 483 commits into from
Mar 11, 2024
Merged

Conversation

meowsbits
Copy link
Member

karalabe and others added 30 commits September 22, 2023 11:07
This allows using the freezer from multiple processes at once
in read-only mode.

Co-authored-by: Martin Holst Swende <martin@swende.se>
When MatcherSession encounters an error, it attempts to close the session.
Closing waits for all goroutines to finish, including the 'distributor'. However, the
distributor will not exit until all requests have returned.

This patch fixes the issue by delivering the (empty) result to the distributor
before calling Close().
This fixes an issue where the --bootnodes flag was overridden by the config file.

---------

Co-authored-by: NathanBSC <Nathan.l@nodereal.io>
Co-authored-by: Felix Lange <fjl@twurst.com>
Co-authored-by: Felix Lange <fjl@twurst.com>
Adding a space beween function opOrigin() and opcCaller() in instruciton.go.
Adding a space beween function opkeccak256()  and opAddress() in instruciton.go.
This PR will allow a previously underpriced transaction back in after a timeout
of 5 minutes. This will block most transaction spam but allow for transactions to
be re-broadcasted on networks with less transaction flow.

---------

Co-authored-by: Felix Lange <fjl@twurst.com>
So apparently in the spec the base block parameter of eth_call is optional.
I agree that "latest" is a sane default for this that most people would use.
* eth/downloader: remove rollback mechanism in downloader

* eth/downloader: remove the tests
core/txpool:fix typos
fix(core/txpool): fix typos
* core, accounts, eth, trie: handle genesis state missing

* core, eth, trie: polish

* core: manage txpool subscription in mainpool

* eth/backend: fix test

* cmd, eth: fix test

* core/rawdb, trie/triedb/pathdb: address comments

* eth, trie: address comments

* eth: inline the function

* eth: use synced flag

* core/txpool: revert changes in txpool

* core, eth, trie: rename functions
This change contains the final (?) address for 4788 beacon root contract. The update to the EIP is being tracked here: ethereum/EIPs#7672

---------

Co-authored-by: Martin Holst Swende <martin@swende.se>
* fix: typo

* feat: revert symbol name
…elDB (#28224)

ethdb, internal/ethapi: support exposing Pebble stats too, besinde LevelDB
* fix(core/txpool): fix typos

* core/asm: fix typos

* core/bloombits: fix typos

* core/rawdb: fix typos
Same way that the gasUsed in header is updated when a tx 
is added we should update blob gas used instead of requiring caller
to set it manually.
* ethdb/pebble: upgrade pebble

* ethdb/pebble, go.mod: update pebble to master (aa077af62593)

---------

Co-authored-by: Péter Szilágyi <peterke@gmail.com>
whilei and others added 2 commits February 4, 2024 07:56
ziogaschr and others added 7 commits February 8, 2024 16:50
…ethash,core,core/txpool/blobpool,core/txpool,core/types,core/vm,eth/catalyst,eth/protocols/eth,miner,params/types/coregeth,params/types/ctypes,params/types/genesisT,params/types/goethereum: add conditions per block number for cancun EIPs
…atherForks

forkid_test.go:563:40: not enough arguments in call to gatherForks
    have (ctypes.ChainConfigurator)
    want (ctypes.ChainConfigurator, uint64) (typecheck)
Signed-off-by: Diego López León <dieguitoll@gmail.com>
…/clique,consensus/ethash,core,core/txpool/blobpool,core/txpool,core/types,core/vm,eth/catalyst,eth/protocols/eth,miner,params/types/coregeth,params/types/ctypes,params/types/genesisT,params/types/goethereum: Merge branch 'merge/foundation-release/1.13.4' into merge/foundation-release/1.13.5

# Conflicts:
#	core/chain_makers.go
@@ -76,7 +76,7 @@ var Defaults = Config{
DatasetsOnDisk: 2,
DatasetsLockMmap: false,
},
NetworkId: vars.DefaultNetworkID,
NetworkId: 0, // enable auto configuration of networkID == chainID
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a note to be extra cautious in this change.

Classic config has this hardcoded to 1, which is correct. 👍

@@ -284,7 +284,12 @@ func newHandler(config *handlerConfig) (*handler, error) {
}
return 0, nil
}
return h.chain.InsertChain(blocks)
n, err := h.chain.InsertChain(blocks)
// PTAL(meowsbits) This chunk was removed upstream. (No acceptTxs.Store...)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice that you've added this comment here so as we don't miss it on subsequent merges.

Conflicts:
      eth/ethconfig/config.go
      eth/ethconfig/gen_config.go
      params/version.go
@meowsbits
Copy link
Member Author

This branch is a superset of the 13.x branches:PRs below, with these stacked PRs originally intended only to piece-out the diffs for review's sake.

Those PRs have been reviewed and ✔️'d.

This branch has been merged with the latest versions of both those branches (as noops, but just to be safe), so I'm going to close those PRs in favor of this one. Then I'll change the base (/target) branch of this PR to master.

@meowsbits meowsbits changed the base branch from merge/foundation-release/1.13.4 to master February 20, 2024 19:17
@meowsbits meowsbits changed the title Merge/foundation release/1.13.5 to 1.13.4 Merge/foundation release/1.13.5 Feb 20, 2024
…alChainConfigurator patch

Date: 2024-02-20 12:25:34-07:00
Signed-off-by: meows <b5c6@protonmail.com>
@meowsbits
Copy link
Member Author

Don't know why the "go generate check" is failing... can't reproduce locally.

@meowsbits
Copy link
Member Author

Gosh I just love green lights. Thanks @ziogaschr 💚

@ziogaschr
Copy link
Member

Syncing finished fine on all nodes @meowsbits. Let's merge it

@meowsbits meowsbits merged commit 0e5f9f6 into master Mar 11, 2024
7 checks passed
@meowsbits meowsbits deleted the merge/foundation-release/1.13.5 branch March 11, 2024 13:10
diega added a commit to diega/core-geth that referenced this pull request Jun 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.