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

Bsp-geth-DTM-RC:v1.6.0 #182

Merged
merged 389 commits into from
Jan 24, 2024
Merged

Bsp-geth-DTM-RC:v1.6.0 #182

merged 389 commits into from
Jan 24, 2024

Conversation

noslav
Copy link
Member

@noslav noslav commented Jan 24, 2024

No description provided.

hzysvilla and others added 30 commits September 26, 2023 11:58
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]>
* 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 <[email protected]>
* eth/catalyst: add validation error in new paylaod hash mismatch

* eth/catalyst/api: refactor api.invalid(..) to return nil latest valid hash if none provided
* cmd, eth: switch the dev synctarget to hash from block

* cmd/utils, eth/catalyst: terminate node wyen synctarget reached
Implements "EIP-7516: BLOBBASEFEE opcode" for cancun, as per spec: https://eips.ethereum.org/EIPS/eip-7516
This change adds a configurable limit to websocket message. 
---------

Co-authored-by: Martin Holst Swende <[email protected]>
This change updates `evm b11r` (blockbuilder) and `evm t8n` (transition) tools to contain cancun updates (e.g. new header fields)
---------

Co-authored-by: Mario Vega <[email protected]>
This change fixes the bug in a benchmark, where the input to the trie is reused in a way which is not correct. 

---------

Co-authored-by: Martin Holst Swende <[email protected]>
* cmd/devp2p, eth: drop eth/66

* eth/protocols/eth: yes sir, linter
* core, eth, miner: start propagating and consuming blob txs

* eth/protocols/eth: disable eth/67 if Cancun is enabled

* core/txpool, eth, miner: pass gas limit infos in lazy tx for mienr filtering

* core/txpool, miner: add lazy resolver for pending txs too

* core, eth: fix review noticed bugs

* eth, miner: minor polishes in the mining and announcing logs

* core/expool: unsubscribe the event scope
fjl and others added 25 commits December 30, 2023 21:16
slog.SetDefault has undesirable side effects. It also sets the default logger destination,
for example. So we should not call it by default in init.
cmd/evm: fixes the blob gas calculation if a transaction is invalid
This PR schedules the cancun fork for the goerli testnet as discussed on ACD.
Spec: ethereum/execution-specs#860

We schedule:
  goerli at 1705473120
When managing geth, it is sometimes desirable to do a partial wipe; deleting state but retaining freezer data. A partial wipe can be somewhat tricky to accomplish. 

This change implements the ability to perform partial wipe by making it possible to run geth removedb non-interactive, using command line options instead.
This is a rewrite of the 'simulated backend', an implementation of the ethclient interfaces
which is backed by a simulated blockchain. It was getting annoying to maintain the old
version of the simulated backend feature because there was a lot of code duplication with
the main client. 

The new version is built using parts that we already have: an in-memory geth node instance
running in developer mode provides the chain, while the Go API is provided by ethclient.
A backwards-compatibility wrapper is provided, but the simulated backend has also moved to
a more sensible import path: github.com/ethereum/go-ethereum/ethclient/simulated

---------

Co-authored-by: Felix Lange <[email protected]>
Co-authored-by: Gary Rong <[email protected]>
Signed-off-by: Pranay Valson <[email protected]>
Upgrade `bsp-geth` to go-ethereum v1.13.10 Sharjila B.

Signed-off-by: Pranay Valson <[email protected]>
Copy link

codecov bot commented Jan 24, 2024

Codecov Report

Attention: 1030 lines in your changes are missing coverage. Please review.

Comparison is base (a18a2bd) 55.51% compared to head (2d2607f) 57.23%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #182      +/-   ##
==========================================
+ Coverage   55.51%   57.23%   +1.72%     
==========================================
  Files         633      593      -40     
  Lines       85631    79133    -6498     
==========================================
- Hits        47534    45290    -2244     
+ Misses      34282    30095    -4187     
+ Partials     3815     3748      -67     
Files Coverage Δ
accounts/abi/bind/auth.go 0.00% <ø> (ø)
accounts/abi/bind/base.go 53.23% <100.00%> (+2.10%) ⬆️
accounts/abi/bind/bind.go 83.08% <ø> (ø)
accounts/abi/method.go 92.15% <100.00%> (ø)
accounts/abi/topics.go 95.69% <100.00%> (-0.05%) ⬇️
accounts/keystore/passphrase.go 59.78% <ø> (ø)
cmd/evm/compiler.go 0.00% <ø> (ø)
cmd/evm/disasm.go 0.00% <ø> (ø)
cmd/evm/main.go 70.83% <100.00%> (-12.10%) ⬇️
cmd/geth/logtestcmd_active.go 100.00% <100.00%> (ø)
... and 119 more

... and 208 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a18a2bd...2d2607f. Read the comment docs.

@noslav
Copy link
Member Author

noslav commented Jan 24, 2024

v1.6.0-bsp tested on cqt-bsp-refiner-v1 with bsp-agent v1.4.8

bsp-geth logs

Jan 24 20:11:45 cqt-bsp-refiner-v1 geth[2302839]: INFO [01-24|20:11:45.018] Starting peer-to-peer node               instance=Geth/v1.13.10-stable-2d2607fb/linux-amd64/go1.21.4
Jan 24 20:14:02 cqt-bsp-refiner-v1 geth[2302839]: INFO [01-24|20:14:02.445] Synchronisation completed, setting historical blocks synced to 1 latestnum=19,078,805 latesthash=6d6dce..5a7085 age=15s
Jan 24 20:14:02 cqt-bsp-refiner-v1 geth[2302839]: INFO [01-24|20:14:02.445] Fully Synced, BSP running in live sync mode "BSP Mode Config: "=1
Jan 24 20:14:13 cqt-bsp-refiner-v1 geth[2302839]: INFO [01-24|20:14:13.853] Forkchoice requested sync to new head    number=19,078,807 hash=c1cda1..6fc747 finalized=unknown
Jan 24 20:14:13 cqt-bsp-refiner-v1 geth[2302839]: INFO [01-24|20:14:13.904] Creating Block Specimen                  "Exported block"=19,078,806 hash=0x9142359456050d2a4b5cfd263011a813a174b4d1138e1a305d00646adb4cbe74

bsp-agent logs

Jan 24 20:15:06 cqt-bsp-refiner-v1 bspagent[2281113]: time="2024-01-24T20:15:06Z" level=info msg="Writing block-replica binary file to local directory: /scratch/bsp-agent/segments/1-19078780-replica-0x0e32ffa849e2158e509fbd5246014e2cae2adab1be44753421ca6a4197a4fc98" function=validatePath line=132
Jan 24 20:15:06 cqt-bsp-refiner-v1 bspagent[2281113]: time="2024-01-24T20:15:06Z" level=info msg="File written successfully to: /scratch/bsp-agent/segments/1-19078780-replica-0x0e32ffa849e2158e509fbd5246014e2cae2adab1be44753421ca6a4197a4fc98" function=WriteToBinFile line=35
Jan 24 20:15:09 cqt-bsp-refiner-v1 bspagent[2281113]: time="2024-01-24T20:15:09Z" level=info msg="client side cid is: bafybeic7ey7evw4bv33cuy5frrlmwzhujeuzhbnoxuliiada66k45i5pji, while uploaded is: bafybeic7ey7evw4bv33cuy5frrlmwzhujeuzhbnoxuliiada66k45i5pji" function=Store line=94
Jan 24 20:15:09 cqt-bsp-refiner-v1 bspagent[2281113]: time="2024-01-24T20:15:09Z" level=info msg="stream ids acked and trimmed: [1706127235033-1], for stream key: replication, with current length: 31" function=processStream line=163

@noslav noslav merged commit dafd3d8 into main Jan 24, 2024
5 checks passed
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.