-
Notifications
You must be signed in to change notification settings - Fork 153
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
Conversation
This allows using the freezer from multiple processes at once in read-only mode. Co-authored-by: Martin Holst Swende <[email protected]>
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 <[email protected]> Co-authored-by: Felix Lange <[email protected]>
Co-authored-by: Felix Lange <[email protected]>
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 <[email protected]>
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 <[email protected]>
Signed-off-by: jsvisa <[email protected]>
* 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.
Small trie prefetcher nits
* ethdb/pebble: upgrade pebble * ethdb/pebble, go.mod: update pebble to master (aa077af62593) --------- Co-authored-by: Péter Szilágyi <[email protected]>
This reverts commit b85c183
Date: 2024-02-04 07:59:44-07:00 Signed-off-by: meows <[email protected]>
…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 <[email protected]>
…/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
…4' into merge/foundation-release/1.13.5
@@ -76,7 +76,7 @@ var Defaults = Config{ | |||
DatasetsOnDisk: 2, | |||
DatasetsLockMmap: false, | |||
}, | |||
NetworkId: vars.DefaultNetworkID, | |||
NetworkId: 0, // enable auto configuration of networkID == chainID |
There was a problem hiding this comment.
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...) |
There was a problem hiding this comment.
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
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 |
…alChainConfigurator patch Date: 2024-02-20 12:25:34-07:00 Signed-off-by: meows <[email protected]>
Don't know why the "go generate check" is failing... can't reproduce locally. |
Gosh I just love green lights. Thanks @ziogaschr 💚 |
Syncing finished fine on all nodes @meowsbits. Let's merge it |
Rel #575, #563.
Merges https://github.com/ethereum/go-ethereum/releases/tag/v1.13.5.