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

Bump geth to v1.11.6 #14

Merged
merged 744 commits into from
May 1, 2023
Merged

Bump geth to v1.11.6 #14

merged 744 commits into from
May 1, 2023

Conversation

ygaberman-px
Copy link

@ygaberman-px ygaberman-px commented Apr 28, 2023

This PR merges the v1.11.6 upgrade to our fork and adds the simulator changes back in.

Changes

  • Merge in new release
git merge --strategy-option theirs tags/v1.11.6
  • Fix merge conflicts
go vet ./...
  • Add in simulator commit:
git cherry-pick b28c0a71ffab5d288a56b509445ae921b2253191

Test

  • Ran go test ./... to verify all tests pass
  • Verify our tests pass

dbadoy and others added 30 commits January 16, 2023 11:21
les/fetcher : fix requestTimer leak
* core/txpool: check if initcode size is exceeded

* core/txpool: move check
Kiln was deprecated after the merge.
Fixes ethereum#26505 where the console crashed when a property getter
raised an exception during autocompletion. I also noticed while fixing this
issue that autocomplete wasn't working for objects/fields with numbers in
them (most importantly web3.<tab><tab>) which is also now fixed.
…6481)

* params: define cancun and osaka as timestamp based forks

* core, params: change osaka to prague

* params: fix
This change introduces a breaking change to miner.etherbase is configured.

Previously, users did not need to explicitly set the  etherbase address via flag, since 'first' local account was used as etherbase automatically. This change removes the  "default first account" feature.

In Proof-of-stake world, the fee recipient address is provided by CL, and not configured in Geth any more - meaning that miner.etherbase is mostly for legacy networks(pow, clique networks etc).
This fixes a regression where the flag did not accept values without
the 0x prefix anymore. What's worse, if an invalid value was passed,
the client would just log an INFO level message and continue.
This PR changes the API so that uint64 is used for fork timestamps.
It's a good choice because types.Header also uses uint64 for time.

Co-authored-by: Felix Lange <[email protected]>
This change implements withdrawals as specified in EIP-4895.

Co-authored-by: [email protected] <[email protected]>
Co-authored-by: marioevz <[email protected]>
Co-authored-by: Martin Holst Swende <[email protected]>
Co-authored-by: Felix Lange <[email protected]>
This fixes a regression in ethereum#26541 where we turned the miner address
being missing into a startup error. The address was not configured in
--dev mode.
This adds explicit checks for the presence of withdrawals in the engine API.

Co-authored-by: Felix Lange <[email protected]>
This change removes the Infura rinkeby bootnode as well as two deprecated ropsten bootnodes. 

Co-authored-by: Nicola Cocchiaro <[email protected]>
Co-authored-by: Martin Holst Swende <[email protected]>
Adds blocktest subcommand to the evm command, which is very similar to statetest, but instead of loading a StateTest static test it loads a BlockchainTest from a json file and runs it.

Co-authored-by: Martin Holst Swende <[email protected]>
Co-authored-by: lightclient <[email protected]>
joohhnnn and others added 25 commits April 4, 2023 02:59
)

* consensus/misc, params: add EIP-4844 blobfee conversions

* consensus/misc: pull in fakeExponential test cases

* consensus/misc: reuse bigints

* consensus/misc: nit renames, additional larger testcase

---------

Co-authored-by: Roberto Bayardo <[email protected]>
Co-authored-by: Martin Holst Swende <[email protected]>
This includes a semantic change to the `callTracer` as well as `flatCallTracer`.
The value of field `gas` in the **first** call frame will change as follows:

- It previously contained gas available after initial deductions (i.e. tx costs)
- It will now contain the full tx gasLimit value

Signed-off-by: jsvisa <[email protected]>
This PR removes the Debug field from vmconfig, making it so that if a tracer is set, debug=true is implied.

---------

Co-authored-by: 0xTylerHolmes <[email protected]>
Co-authored-by: Sina Mahmoodi <[email protected]>
…7025)

Makes the float-gauges lock-free

name                      old time/op  new time/op  delta
CounterFloat64Parallel-8  1.45µs ±10%  0.85µs ± 6%  -41.65%  (p=0.008 n=5+5)

---------

Co-authored-by: Exca-DK <[email protected]>
Co-authored-by: Martin Holst Swende <[email protected]>
Use the new atomic types in package eth/tracers

---------

Co-authored-by: Martin Holst Swende <[email protected]>
Co-authored-by: Sina Mahmoodi <[email protected]>
Prior to this change, it was possible that transactions are erroneously deemed as 'future' although they are in fact 'pending', causing them to be dropped due to 'future' not being allowed to replace 'pending'. 

This change fixes that, by doing a more in-depth inspection of the queue.
…ethereum#27038)

This PR unifies the error handling in miner. 

Whenever an error occur while applying a transaction, the transaction should be regarded as invalid and all following transactions from the same sender not executable because of the nonce restriction. The only exception is the `nonceTooLow` error which is handled separately.
build: upgrade -dlgo version to Go 1.20.3
New sepolia bootnodes managed by EF devops
cmd/devp2p: fix log of ignored recent nodes counter
Sets a meaningful name on test-files
The EIP150Hash was an idea where, after the fork, we hardcoded the forked hash as an extra defensive mechanism. It wasn't really used, since forks weren't contentious and for all the various testnets and private networks it's been a hassle to have around. 

This change removes that config field. 

---------

Signed-off-by: jsvisa <[email protected]>
Follow-up to ethereum#26697, makes the crawler less verbose on route53-based scenarios.

It also changes the loglevel from debug to info on Updates, which are typically the root, and can be interesting to see.
renames `--vmodule` to `--log.vmodule`, and prints a warning if the old form is used.
In this PR, all TryXXX(e.g. TryGet) APIs of trie are renamed to XXX(e.g. Get) with an error returned.

The original XXX(e.g. Get) APIs are renamed to MustXXX(e.g. MustGet) and does not return any error -- they print a log output. A future PR will change the behaviour to panic on errorrs.
@@ -63,11 +63,10 @@ type SimulatedBackend struct {
database ethdb.Database // In memory database to store our testing data
blockchain *core.BlockChain // Ethereum blockchain to handle the consensus

mu sync.Mutex
stuckTransactions types.Transactions // holds onto all txes that don't go into the pending block due to low gas
Copy link

Choose a reason for hiding this comment

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

this was the reason to have the fork. are we deciding to remove this and any tests that used it in pax?

Copy link
Author

Choose a reason for hiding this comment

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

will add this back in

Copy link
Author

Choose a reason for hiding this comment

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

fixed merge conflicts and added commit back in

@gitteri gitteri merged commit ecc2edb into paxosglobal:master May 1, 2023
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.