Bonsai based reference test worldstate #5686

garyschulte · 2023-07-07T22:44:56Z

PR description

@shemnon pointed out that the reference tests support in besu does not use bonsai worldstate, nor does evmtool. This PR creates a bonsai-based worldstate and accumulator for ReferenceTestsWorldState.

Change Summary:

creates ReferenceTestWorldState interface, BonsaiReferenceTestWorldState and DefaultReferenceTestWorldState impl's
creates preimageProxy functionality for reference test worldstate
implements account storage streaming for bonsai
adds overload of streamFromKey that includes endKey
implements iterator based streamFromKey for BonsaiSnapshotWorldstate
removes archive from BonsaiWorldState
since bonsai checks calculated state root against blockheader stateroot:
- adds currentStateRoot to ReferenceTestEnv, adds stateRoot to evmtool t8n test cases
- rewrites constructed block headers from GeneralStateState reference tests with the expected stateroot from post

Discovered/Fixed Defects

fixes premature stream end with streamStorageFlatDatabase for unsorted storage
fixes account storage clearing issue

Fixed Issue(s)

#5688

github-actions · 2023-07-07T22:45:07Z

I thought about documentation and added the doc-change-required label to this PR if updates are required.
I thought about the changelog and included a changelog update if required.
If my PR includes database changes (e.g. KeyValueSegmentIdentifier) I have thought about compatibility and performed forwards and backwards compatibility tests

.../main/java/org/hyperledger/besu/ethereum/bonsai/storage/BonsaiWorldStateKeyValueStorage.java

matkt · 2023-08-31T09:58:09Z

.../main/java/org/hyperledger/besu/ethereum/bonsai/storage/BonsaiWorldStateKeyValueStorage.java

@@ -79,15 +92,25 @@ public class BonsaiWorldStateKeyValueStorage implements WorldStateStorage, AutoC

  protected final Subscribers<BonsaiStorageSubscriber> subscribers = Subscribers.create();

+  final BonsaiPreImageProxy preImageProxy;


why do we need this class ? it seems it's only needed for test. we don"t have another way to reduce the impact just for the test ?

I think we could create a subclass of BonsaiWorldStateKeyValueStorage that takes the preImageProxy, since we are only using it to recover hashes for reference test worldstate. Similarly we could move streamAccounts into the subclass.

seems to be good like that

.../main/java/org/hyperledger/besu/ethereum/bonsai/storage/BonsaiWorldStateKeyValueStorage.java

...re/src/main/java/org/hyperledger/besu/ethereum/bonsai/storage/flat/FlatDbReaderStrategy.java

...reum/core/src/main/java/org/hyperledger/besu/ethereum/bonsai/worldview/BonsaiWorldState.java

matkt · 2023-08-31T10:20:57Z

...n/java/org/hyperledger/besu/ethereum/bonsai/worldview/BonsaiWorldStateUpdateAccumulator.java

@@ -127,14 +132,18 @@ public EvmAccount createAccount(final Address address, final long nonce, final W
      bonsaiValue = new BonsaiValue<>(null, null);
      accountsToUpdate.put(address, bonsaiValue);
    } else if (bonsaiValue.getUpdated() != null) {
-      throw new IllegalStateException("Cannot create an account when one already exists");
+      if (bonsaiValue.getUpdated().isEmpty()) {


maybe

else if (bonsaiValue.getUpdated() != null && bonsaiValue.getUpdated().isEmpty()) { return new WrappedEvmAccount(track(new UpdateTrackingAccount<>(bonsaiValue.getUpdated()))); } else { throw new IllegalStateException("Cannot create an account when one already exists"); }

matkt · 2023-08-31T10:21:45Z

...n/java/org/hyperledger/besu/ethereum/bonsai/worldview/BonsaiWorldStateUpdateAccumulator.java

@@ -290,6 +299,19 @@ public void commit() {
              final BonsaiAccount updatedAccount;
              final BonsaiValue<BonsaiAccount> updatedAccountValue =
                  accountsToUpdate.get(updatedAddress);
+
+              final Map<StorageSlotKey, BonsaiValue<UInt256>> pendingStorageUpdates =


we need to fullsync some nodes to be sure we are not adding a consensus issue
(goerli, sepolia, and maybe etc at the minimum)

I snap synced goerli and mainnet nodes a few days ago and it has been going well. I will start a full sync on goerli and another on sepolia and let them run over the weekend

could you try to run a mainnet snapsync with the heal flat db flag ? I see that this PR is modifying a lot the stream of the flat db datas and it will be nice if can be sure that it's ok

ethereum/evmtool/src/test/resources/org/hyperledger/besu/evmtool/state-test/blockhash.json

matkt · 2023-09-11T09:09:59Z

Seems to be good. if we can just check a snapsync with the heal flat db flag it will be nice

garyschulte · 2023-09-11T18:54:20Z

Seems to be good. if we can just check a snapsync with the heal flat db flag it will be nice

I have been snapsyncing mainnet over the weekend, but I think it is experiencing the 99% error :/
The good news is that sepolia completed a full sync without issue. Goerli is at about 3 million blocks behind, but appears it is going to reach head without error. Just the mainnet snap sync with flatdb heal is outstanding.

maybe I can rebase off of your heal PR

matkt · 2023-09-12T07:54:35Z

Seems to be good. if we can just check a snapsync with the heal flat db flag it will be nice

I have been snapsyncing mainnet over the weekend, but I think it is experiencing the 99% error :/ The good news is that sepolia completed a full sync without issue. Goerli is at about 3 million blocks behind, but appears it is going to reach head without error. Just the mainnet snap sync with flatdb heal is outstanding.

maybe I can rebase off of your heal PR

I think it's a good idea. After the sync I think I will approve. Everything seems to be fine now

garyschulte · 2023-09-13T23:31:02Z

I think it's a good idea. After the sync I think I will approve. Everything seems to be fine now

The snap sync of mainnet was good. full sync of goerli and sepolia likewise. 👍

Signed-off-by: garyschulte <[email protected]>

--- Drop Kotti Network support (ETC) (hyperledger#5816) Signed-off-by: Diego López León <[email protected]> fix ForkId if there are no Forks and the starting timestamp is not 0 (hyperledger#5819) Signed-off-by: Stefan <[email protected]> enforce that BlobTransactions have at least one blob (hyperledger#5826) * enforce that BlobTransactions have at least one blob Signed-off-by: Stefan <[email protected]> Signed-off-by: Stefan Pingel <[email protected]> Do not create ignorable segments on `revert storage-variables` (hyperledger#5830) * fix the bug that creates the ignorable chain pruner segment, add rocks exception parsing to RocksDBColumnarKeyValueStorage subclasses * parse rocksdb error for unprintable column family id's Signed-off-by: garyschulte <[email protected]> add versioned hashes and number of blobs to toString() (hyperledger#5831) Signed-off-by: Stefan <[email protected]> add parent beacon block root to payload id calculation (hyperledger#5843) Signed-off-by: Stefan <[email protected]> bump version to 23.7.3-SNAPSHOT (hyperledger#5854) Signed-off-by: Daniel Lehrner <[email protected]> set the beacon root address to the correct value (hyperledger#5853) Signed-off-by: Stefan <[email protected]> docs(readme): fix broken link to installation of binaries page (hyperledger#5859) Fixes hyperledger#5858 Signed-off-by: Peter Somogyvari <[email protected]> Update RocksDB version from 8.0.0 to 8.3.2 (hyperledger#5832) Signed-off-by: Ameziane H <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> use non-deprecated authenticate methods (hyperledger#5852) Signed-off-by: Sally MacFarlane <[email protected]> move to Hyperledger shared runners for current github actions (hyperledger#5860) Signed-off-by: garyschulte <[email protected]> Add range tracing with worldstate (hyperledger#5844) Implement a method to trace a range of blocks and have access to the worldstate before and after the tracing Signed-off-by: Karim TAAM <[email protected]> Layered txpool by default and txpool options hoverhaul (hyperledger#5772) Signed-off-by: Fabio Di Fabio <[email protected]> Fix issue 5824 - Duplicate key errors in EthScheduler-Transactions (hyperledger#5857) Fix issue 5824 - Duplicate key errors in EthScheduler-Transactions Signed-off-by: Ameziane H <[email protected]> updated gradle verification metadata (hyperledger#5870) * removed old artefacts [skip ci] * works with compileTestJava * restored metadata needed for codeQL and trusted-artifacts block for javadoc/sources Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Sally MacFarlane <[email protected]> [4844] Add encodingContext to TransactionEncoder and TransactionDecoder (hyperledger#5820) * Add decode type to TransactionDecoder * Refactoring TransactionDecoder * Invert methods order * Use Transaction encoder instead of writeTo * Move enter and leave list to inner method as pr suggestion * Size calculation should use opaque bytes instead of rlp --------- Signed-off-by: Gabriel-Trintinalia <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Signed-off-by: Stefan <[email protected]> payload attributes: fix wrong warning and fail if beacon root is available before cancun (hyperledger#5872) Signed-off-by: Stefan <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Merge MutableAccount and EVMAccount (hyperledger#5863) Merge MutableAccount and EVMAccount functionalities by removing EVMAccount, all calls to getMutable, and the WrappedEVMAccount that was wrapping non-EVMAccounts in a mutable fashion. Instead, use a MutableAccount in all cases an EVMAccount would have been used. This also tends to reduce a level of layering in many places. Signed-off-by: Danno Ferrin <[email protected]> Add world context to transaction tracing API (hyperledger#5836) * Add world context to transaction tracing API Signed-off-by: Franklin Delehelle <[email protected]> * Update changelog with PR ID Signed-off-by: Franklin Delehelle <[email protected]> * Add the Transaction to traceEndTransaction Signed-off-by: Franklin Delehelle <[email protected]> * Rebase on main Signed-off-by: Franklin Delehelle <[email protected]> * Add receipt-linked information to the transaction tracer Signed-off-by: Franklin Delehelle <[email protected]> * added test Signed-off-by: Daniel Lehrner <[email protected]> --------- Signed-off-by: Franklin Delehelle <[email protected]> Signed-off-by: Daniel Lehrner <[email protected]> Co-authored-by: Daniel Lehrner <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Bonsai based reference test worldstate (hyperledger#5686) * create a bonsai based reference test worldstate -> getOrCreate in BonsaiWorldStateUpdateAccumulator - do not throw if we discover an empty account in a non-null BonsaiValue<Account> -> add curentStateRoot to t8n -> storageEntriesFrom and streamAccounts implemented in BonsaiWorldStateKeyValueStorage -> add endKey version of streamFromKey * bonsai fix for self-destruct and create2 at the same address and same block Signed-off-by: garyschulte <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Co-authored-by: Karim TAAM <[email protected]> Don't start BFT mining coordinators until initial sync has completed (hyperledger#5861) * Don't start BFT mining coordinators until initial sync has completed Signed-off-by: Matthew Whitehead <[email protected]> * Fix unit tests Signed-off-by: Matthew Whitehead <[email protected]> * Fix 'enable' logic Signed-off-by: Matthew Whitehead <[email protected]> --------- Signed-off-by: Matthew Whitehead <[email protected]> display only peers ready for requets on ethstats (hyperledger#5880) * display only ready for requets peers in ethstats Signed-off-by: Karim TAAM <[email protected]> * cast to int Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Karim TAAM <[email protected]> Signed-off-by: Sally MacFarlane <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Co-authored-by: Stefan Pingel <[email protected]> [MINOR] test RLP used for encode/decode blob tx should contain to field (hyperledger#5883) * validate to field on encode/decode for blob tx Signed-off-by: Sally MacFarlane <[email protected]> * revert decode/encode checks - tis done later in tx validation Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Sally MacFarlane <[email protected]> Fix: correctly convert percentage options in TOML configuration file (hyperledger#5886) Signed-off-by: Fabio Di Fabio <[email protected]> EIP7516 - Add BlobBaseFee opcode to Cancun EVM (hyperledger#5884) Signed-off-by: Gabriel-Trintinalia <[email protected]> Fix snapsync heal (hyperledger#5838) Signed-off-by: Karim TAAM <[email protected]> Upgrade besu-native (hyperledger#5893) Upgrade besu-native to 0.8.2 Signed-off-by: Danno Ferrin <[email protected]> Tune G1GC to reduce Besu memory footprint (hyperledger#5879) Signed-off-by: Fabio Di Fabio <[email protected]> Add updated storage to evmtool json trace (hyperledger#5892) Add the EIP-3155 "storage" option to the standard tracer, with the caveat only updated storage is logged. Signed-off-by: Danno Ferrin <[email protected]> Update holesky with fixed extraData, genesis time, shanghaiTime (hyperledger#5890) Signed-off-by: Simon Dudley <[email protected]> [CHANGELOG] removed duplicated line (hyperledger#5904) * removed duplicated line [skip ci] Signed-off-by: Sally MacFarlane <[email protected]> * fixed spelling on Holesky Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Sally MacFarlane <[email protected]>

* create a bonsai based reference test worldstate -> getOrCreate in BonsaiWorldStateUpdateAccumulator - do not throw if we discover an empty account in a non-null BonsaiValue<Account> -> add curentStateRoot to t8n -> storageEntriesFrom and streamAccounts implemented in BonsaiWorldStateKeyValueStorage -> add endKey version of streamFromKey * bonsai fix for self-destruct and create2 at the same address and same block Signed-off-by: garyschulte <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Co-authored-by: Karim TAAM <[email protected]>

* create a bonsai based reference test worldstate -> getOrCreate in BonsaiWorldStateUpdateAccumulator - do not throw if we discover an empty account in a non-null BonsaiValue<Account> -> add curentStateRoot to t8n -> storageEntriesFrom and streamAccounts implemented in BonsaiWorldStateKeyValueStorage -> add endKey version of streamFromKey * bonsai fix for self-destruct and create2 at the same address and same block Signed-off-by: garyschulte <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Co-authored-by: Karim TAAM <[email protected]> Signed-off-by: garyschulte <[email protected]>

* create a bonsai based reference test worldstate -> getOrCreate in BonsaiWorldStateUpdateAccumulator - do not throw if we discover an empty account in a non-null BonsaiValue<Account> -> add curentStateRoot to t8n -> storageEntriesFrom and streamAccounts implemented in BonsaiWorldStateKeyValueStorage -> add endKey version of streamFromKey * bonsai fix for self-destruct and create2 at the same address and same block Signed-off-by: garyschulte <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Co-authored-by: Karim TAAM <[email protected]>

burn-in candidate for 23.7.3 from main sha 6dc10a9..eef40bd https://github.com/hyperledger/besu/compare/6dc10a9..eef40bd --- * Drop Kotti Network support (ETC) (#5816) Signed-off-by: Diego López León <[email protected]> Signed-off-by: garyschulte <[email protected]> * fix ForkId if there are no Forks and the starting timestamp is not 0 (#5819) Signed-off-by: Stefan <[email protected]> Signed-off-by: garyschulte <[email protected]> * enforce that BlobTransactions have at least one blob (#5826) * enforce that BlobTransactions have at least one blob Signed-off-by: Stefan <[email protected]> Signed-off-by: Stefan Pingel <[email protected]> Signed-off-by: garyschulte <[email protected]> * Do not create ignorable segments on `revert storage-variables` (#5830) * fix the bug that creates the ignorable chain pruner segment, add rocks exception parsing to RocksDBColumnarKeyValueStorage subclasses * parse rocksdb error for unprintable column family id's Signed-off-by: garyschulte <[email protected]> * add versioned hashes and number of blobs to toString() (#5831) Signed-off-by: Stefan <[email protected]> Signed-off-by: garyschulte <[email protected]> * add parent beacon block root to payload id calculation (#5843) Signed-off-by: Stefan <[email protected]> Signed-off-by: garyschulte <[email protected]> * bump version to 23.7.3-SNAPSHOT (#5854) Signed-off-by: Daniel Lehrner <[email protected]> Signed-off-by: garyschulte <[email protected]> * set the beacon root address to the correct value (#5853) Signed-off-by: Stefan <[email protected]> Signed-off-by: garyschulte <[email protected]> * docs(readme): fix broken link to installation of binaries page (#5859) Fixes #5858 Signed-off-by: Peter Somogyvari <[email protected]> Signed-off-by: garyschulte <[email protected]> * Update RocksDB version from 8.0.0 to 8.3.2 (#5832) Signed-off-by: Ameziane H <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Signed-off-by: garyschulte <[email protected]> * use non-deprecated authenticate methods (#5852) Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: garyschulte <[email protected]> * move to Hyperledger shared runners for current github actions (#5860) Signed-off-by: garyschulte <[email protected]> * Add range tracing with worldstate (#5844) Implement a method to trace a range of blocks and have access to the worldstate before and after the tracing Signed-off-by: Karim TAAM <[email protected]> Signed-off-by: garyschulte <[email protected]> * Layered txpool by default and txpool options hoverhaul (#5772) Signed-off-by: Fabio Di Fabio <[email protected]> Signed-off-by: garyschulte <[email protected]> * Fix issue 5824 - Duplicate key errors in EthScheduler-Transactions (#5857) Fix issue 5824 - Duplicate key errors in EthScheduler-Transactions Signed-off-by: Ameziane H <[email protected]> Signed-off-by: garyschulte <[email protected]> * updated gradle verification metadata (#5870) * removed old artefacts [skip ci] * works with compileTestJava * restored metadata needed for codeQL and trusted-artifacts block for javadoc/sources Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: garyschulte <[email protected]> * [4844] Add encodingContext to TransactionEncoder and TransactionDecoder (#5820) * Add decode type to TransactionDecoder * Refactoring TransactionDecoder * Invert methods order * Use Transaction encoder instead of writeTo * Move enter and leave list to inner method as pr suggestion * Size calculation should use opaque bytes instead of rlp --------- Signed-off-by: Gabriel-Trintinalia <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Signed-off-by: garyschulte <[email protected]> * #5868: fix beacon root address and modulus for devnet 9 (#5871) Signed-off-by: Stefan <[email protected]> Signed-off-by: garyschulte <[email protected]> * payload attributes: fix wrong warning and fail if beacon root is available before cancun (#5872) Signed-off-by: Stefan <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Signed-off-by: garyschulte <[email protected]> * Merge MutableAccount and EVMAccount (#5863) Merge MutableAccount and EVMAccount functionalities by removing EVMAccount, all calls to getMutable, and the WrappedEVMAccount that was wrapping non-EVMAccounts in a mutable fashion. Instead, use a MutableAccount in all cases an EVMAccount would have been used. This also tends to reduce a level of layering in many places. Signed-off-by: Danno Ferrin <[email protected]> Signed-off-by: garyschulte <[email protected]> * Add world context to transaction tracing API (#5836) * Add world context to transaction tracing API Signed-off-by: Franklin Delehelle <[email protected]> * Update changelog with PR ID Signed-off-by: Franklin Delehelle <[email protected]> * Add the Transaction to traceEndTransaction Signed-off-by: Franklin Delehelle <[email protected]> * Rebase on main Signed-off-by: Franklin Delehelle <[email protected]> * Add receipt-linked information to the transaction tracer Signed-off-by: Franklin Delehelle <[email protected]> * added test Signed-off-by: Daniel Lehrner <[email protected]> --------- Signed-off-by: Franklin Delehelle <[email protected]> Signed-off-by: Daniel Lehrner <[email protected]> Co-authored-by: Daniel Lehrner <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Signed-off-by: garyschulte <[email protected]> * Bonsai based reference test worldstate (#5686) * create a bonsai based reference test worldstate -> getOrCreate in BonsaiWorldStateUpdateAccumulator - do not throw if we discover an empty account in a non-null BonsaiValue<Account> -> add curentStateRoot to t8n -> storageEntriesFrom and streamAccounts implemented in BonsaiWorldStateKeyValueStorage -> add endKey version of streamFromKey * bonsai fix for self-destruct and create2 at the same address and same block Signed-off-by: garyschulte <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Co-authored-by: Karim TAAM <[email protected]> Signed-off-by: garyschulte <[email protected]> * Don't start BFT mining coordinators until initial sync has completed (#5861) * Don't start BFT mining coordinators until initial sync has completed Signed-off-by: Matthew Whitehead <[email protected]> * Fix unit tests Signed-off-by: Matthew Whitehead <[email protected]> * Fix 'enable' logic Signed-off-by: Matthew Whitehead <[email protected]> --------- Signed-off-by: Matthew Whitehead <[email protected]> Signed-off-by: garyschulte <[email protected]> * display only peers ready for requets on ethstats (#5880) * display only ready for requets peers in ethstats Signed-off-by: Karim TAAM <[email protected]> * cast to int Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Karim TAAM <[email protected]> Signed-off-by: Sally MacFarlane <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Co-authored-by: Stefan Pingel <[email protected]> Signed-off-by: garyschulte <[email protected]> * [MINOR] test RLP used for encode/decode blob tx should contain to field (#5883) * validate to field on encode/decode for blob tx Signed-off-by: Sally MacFarlane <[email protected]> * revert decode/encode checks - tis done later in tx validation Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: garyschulte <[email protected]> * Fix: correctly convert percentage options in TOML configuration file (#5886) Signed-off-by: Fabio Di Fabio <[email protected]> Signed-off-by: garyschulte <[email protected]> * EIP7516 - Add BlobBaseFee opcode to Cancun EVM (#5884) Signed-off-by: Gabriel-Trintinalia <[email protected]> Signed-off-by: garyschulte <[email protected]> * Fix snapsync heal (#5838) Signed-off-by: Karim TAAM <[email protected]> Signed-off-by: garyschulte <[email protected]> * Upgrade besu-native (#5893) Upgrade besu-native to 0.8.2 Signed-off-by: Danno Ferrin <[email protected]> Signed-off-by: garyschulte <[email protected]> * Tune G1GC to reduce Besu memory footprint (#5879) Signed-off-by: Fabio Di Fabio <[email protected]> Signed-off-by: garyschulte <[email protected]> * Add updated storage to evmtool json trace (#5892) Add the EIP-3155 "storage" option to the standard tracer, with the caveat only updated storage is logged. Signed-off-by: Danno Ferrin <[email protected]> Signed-off-by: garyschulte <[email protected]> * Update holesky with fixed extraData, genesis time, shanghaiTime (#5890) Signed-off-by: Simon Dudley <[email protected]> Signed-off-by: garyschulte <[email protected]> * [CHANGELOG] removed duplicated line (#5904) * removed duplicated line [skip ci] Signed-off-by: Sally MacFarlane <[email protected]> * fixed spelling on Holesky Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: garyschulte <[email protected]> * add 23.7.2 release SHAs and bump to 23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-23.7.3-RC version Signed-off-by: garyschulte <[email protected]> --------- Signed-off-by: Diego López León <[email protected]> Signed-off-by: garyschulte <[email protected]> Signed-off-by: Stefan <[email protected]> Signed-off-by: Stefan Pingel <[email protected]> Signed-off-by: Daniel Lehrner <[email protected]> Signed-off-by: Peter Somogyvari <[email protected]> Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Signed-off-by: Fabio Di Fabio <[email protected]> Signed-off-by: Ameziane H <[email protected]> Signed-off-by: Gabriel-Trintinalia <[email protected]> Signed-off-by: Danno Ferrin <[email protected]> Signed-off-by: Franklin Delehelle <[email protected]> Signed-off-by: Matthew Whitehead <[email protected]> Signed-off-by: Simon Dudley <[email protected]> Co-authored-by: Diego López León <[email protected]> Co-authored-by: Stefan Pingel <[email protected]> Co-authored-by: Daniel Lehrner <[email protected]> Co-authored-by: Peter Somogyvari <[email protected]> Co-authored-by: ahamlat <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Co-authored-by: matkt <[email protected]> Co-authored-by: Fabio Di Fabio <[email protected]> Co-authored-by: Gabriel-Trintinalia <[email protected]> Co-authored-by: Danno Ferrin <[email protected]> Co-authored-by: delehef <[email protected]> Co-authored-by: Matt Whitehead <[email protected]> Co-authored-by: Simon Dudley <[email protected]>

* create a bonsai based reference test worldstate -> getOrCreate in BonsaiWorldStateUpdateAccumulator - do not throw if we discover an empty account in a non-null BonsaiValue<Account> -> add curentStateRoot to t8n -> storageEntriesFrom and streamAccounts implemented in BonsaiWorldStateKeyValueStorage -> add endKey version of streamFromKey * bonsai fix for self-destruct and create2 at the same address and same block Signed-off-by: garyschulte <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Co-authored-by: Karim TAAM <[email protected]>

…#6205) revert some modification that was made to pass tests #5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]>

…#6205) revert some modification that was made to pass tests #5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]> Signed-off-by: matkt <[email protected]>

* Sequenced pool synonym for legacy pool Signed-off-by: Matthew Whitehead <[email protected]> * Class rename Signed-off-by: Matthew Whitehead <[email protected]> * Spotless fixes Signed-off-by: Matthew Whitehead <[email protected]> * Add SEQUENCED to config overview test Signed-off-by: Matthew Whitehead <[email protected]> * Update CHANGELOG.md Co-authored-by: Sally MacFarlane <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> * add a fix to load correctly the storage trie in the Bonsai WorldState (#6205) revert some modification that was made to pass tests #5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]> --------- Signed-off-by: Matthew Whitehead <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Co-authored-by: matkt <[email protected]>

…hyperledger#6205) revert some modification that was made to pass tests hyperledger#5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]> Signed-off-by: matkt <[email protected]> Signed-off-by: jflo <[email protected]>

* Sequenced pool synonym for legacy pool Signed-off-by: Matthew Whitehead <[email protected]> * Class rename Signed-off-by: Matthew Whitehead <[email protected]> * Spotless fixes Signed-off-by: Matthew Whitehead <[email protected]> * Add SEQUENCED to config overview test Signed-off-by: Matthew Whitehead <[email protected]> * Update CHANGELOG.md Co-authored-by: Sally MacFarlane <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> * add a fix to load correctly the storage trie in the Bonsai WorldState (hyperledger#6205) revert some modification that was made to pass tests hyperledger#5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]> --------- Signed-off-by: Matthew Whitehead <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Co-authored-by: matkt <[email protected]> Signed-off-by: jflo <[email protected]>

* New cli options to limit rewards return by eth_feeHistory (#6202) * [#5851] Add error messages on authentication failures with username and password (#6212) * Add a constant for the 'password' * Add error messages on authentication failures with username and password Signed-off-by: Justin Florentine <[email protected]> Signed-off-by: jflo <[email protected]> Signed-off-by: Gabriel-Trintinalia <[email protected]> Signed-off-by: David Lutzardo <[email protected]> Signed-off-by: David Lutzardo <[email protected]> * Add test to check empty login and check response in body is not empty Signed-off-by: David Lutzardo <[email protected]> * Correct format (spotless) Signed-off-by: David Lutzardo <[email protected]> * Update ethereum/api/src/test/java/org/hyperledger/besu/ethereum/api/jsonrpc/JsonRpcHttpServiceLoginTest.java Co-authored-by: Fabio Di Fabio <[email protected]> Signed-off-by: David Lutzardo <[email protected]> * Update ethereum/api/src/test/java/org/hyperledger/besu/ethereum/api/jsonrpc/JsonRpcHttpServiceLoginTest.java Co-authored-by: Fabio Di Fabio <[email protected]> Signed-off-by: David Lutzardo <[email protected]> * Update JsonRpcHttpServiceLoginTest.java use containsIgnoringCase Signed-off-by: David Lutzardo <[email protected]> * Add a CHANGELOG entry for PR 6212 Signed-off-by: David Lutzardo <[email protected]> --------- Signed-off-by: David Lutzardo <[email protected]> Co-authored-by: Fabio Di Fabio <[email protected]> Signed-off-by: jflo <[email protected]> * Add RockDB Subcommand for printing usage per column family (#6185) * Add RockDB Subcommand for printing usage per column family Signed-off-by: Simon Dudley <[email protected]> * changed output to follow a MD table notation. Signed-off-by: Gabriel Fukushima <[email protected]> --------- Signed-off-by: Gabriel Fukushima <[email protected]> Co-authored-by: Gabriel Fukushima <[email protected]> Signed-off-by: jflo <[email protected]> * Deprecation warning if Forest pruning is enabled (#6230) Signed-off-by: Fabio Di Fabio <[email protected]> Signed-off-by: jflo <[email protected]> * Fix the annoying "Errors occurred while build effective model" during builds (#6241) Signed-off-by: Fabio Di Fabio <[email protected]> Signed-off-by: jflo <[email protected]> * Run ATs sequentially (#6244) Signed-off-by: Gabriel Fukushima <[email protected]> Signed-off-by: jflo <[email protected]> * fix: double calls to trace{Start,End}Transaction (#6247) Signed-off-by: delehef <[email protected]> Signed-off-by: jflo <[email protected]> * migrate to junit5 (#6234) Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: jflo <[email protected]> * fixes for problems discovered in main (#6248) Signed-off-by: garyschulte <[email protected]> Signed-off-by: jflo <[email protected]> * Pki - migrate to junit 5 (#6235) * migrate to junit5 Signed-off-by: Sally MacFarlane <[email protected]> * fix: double calls to trace{Start,End}Transaction (#6247) Signed-off-by: Franklin Delehelle <[email protected]> * migrate to junit5 (#6234) Signed-off-by: Sally MacFarlane <[email protected]> * fixes for problems discovered in main (#6248) Signed-off-by: garyschulte <[email protected]> * fixed test comparing size of collection Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: Franklin Delehelle <[email protected]> Signed-off-by: garyschulte <[email protected]> Co-authored-by: delehef <[email protected]> Co-authored-by: garyschulte <[email protected]> Signed-off-by: jflo <[email protected]> * junit 5 ftw (#6253) Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: jflo <[email protected]> * removed unnecessary use of static temp dir (#6251) * don't use static tempdir Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: jflo <[email protected]> * Remove parallelism usage from mainnet AT (#6252) * Remove parallelism usage from mainnet AT Signed-off-by: Gabriel Fukushima <[email protected]> * Increase parallelism usage from mainnet AT Signed-off-by: Gabriel Fukushima <[email protected]> * Add the split command back Signed-off-by: Gabriel Fukushima <[email protected]> --------- Signed-off-by: Gabriel Fukushima <[email protected]> Signed-off-by: jflo <[email protected]> * fix log params (#6254) Signed-off-by: Sally MacFarlane <[email protected]> Co-authored-by: Stefan Pingel <[email protected]> Signed-off-by: jflo <[email protected]> * add dependency on jar task (#6255) Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: jflo <[email protected]> * Fix and test that the BlockAwareOperationTracer methods are invoked the correct number of times (#6259) * Test that the BlockAwareOperationTracer are invoked the correct number of times * Remove redundant calls to traceEndBlock Signed-off-by: Fabio Di Fabio <[email protected]> Signed-off-by: jflo <[email protected]> * [RPC] Use apiConfiguration to limit gasPrice in eth_getGasPrice (#6243) Signed-off-by: Gabriel-Trintinalia <[email protected]> Signed-off-by: jflo <[email protected]> * log bootnodes and static nodes list at debug level (#6273) * log bootnodes and static nodes list at debug level Signed-off-by: Sally MacFarlane <[email protected]> * log if zero bootnodes Signed-off-by: Sally MacFarlane <[email protected]> * null safeguards Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: jflo <[email protected]> * junit 5 (#6256) Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: jflo <[email protected]> * Non bft group ats junit 5 (#6260) * migrate to junit 5 Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: jflo <[email protected]> * Txparse subcommand implementation (#6268) * txparse subcommand Signed-off-by: garyschulte <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Signed-off-by: jflo <[email protected]> * [MINOR] migrate remaining Crypto tests to junit 5 (#6280) * update crypto tests to junit5 * fixed temp file * removed vintage junit dep Signed-off-by: Sally MacFarlane <[email protected]> --------- Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: jflo <[email protected]> * add a fix to load correctly the storage trie in the Bonsai WorldState (#6205) revert some modification that was made to pass tests #5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]> Signed-off-by: matkt <[email protected]> Signed-off-by: jflo <[email protected]> * Sequenced pool synonym for legacy pool (#6274) * Sequenced pool synonym for legacy pool Signed-off-by: Matthew Whitehead <[email protected]> * Class rename Signed-off-by: Matthew Whitehead <[email protected]> * Spotless fixes Signed-off-by: Matthew Whitehead <[email protected]> * Add SEQUENCED to config overview test Signed-off-by: Matthew Whitehead <[email protected]> * Update CHANGELOG.md Co-authored-by: Sally MacFarlane <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> * add a fix to load correctly the storage trie in the Bonsai WorldState (#6205) revert some modification that was made to pass tests #5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]> --------- Signed-off-by: Matthew Whitehead <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Co-authored-by: matkt <[email protected]> Signed-off-by: jflo <[email protected]> * ETC mainnet 'Spiral' activation block (#6267) * Set ENR tree for DNS discovery for ETC mainnet network Signed-off-by: Diego López León <[email protected]> * Set activation block number for ECIP-1109 on ETC mainnet Signed-off-by: Diego López León <[email protected]> --------- Signed-off-by: Diego López León <[email protected]> Signed-off-by: jflo <[email protected]> * uprev to version 23.10.3-RC3 Signed-off-by: jflo <[email protected]> --------- Signed-off-by: Justin Florentine <[email protected]> Signed-off-by: jflo <[email protected]> Signed-off-by: Gabriel-Trintinalia <[email protected]> Signed-off-by: David Lutzardo <[email protected]> Signed-off-by: Gabriel Fukushima <[email protected]> Signed-off-by: Fabio Di Fabio <[email protected]> Signed-off-by: delehef <[email protected]> Signed-off-by: Sally MacFarlane <[email protected]> Signed-off-by: garyschulte <[email protected]> Signed-off-by: Franklin Delehelle <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Signed-off-by: matkt <[email protected]> Signed-off-by: Matthew Whitehead <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> Signed-off-by: Diego López León <[email protected]> Co-authored-by: Gabriel-Trintinalia <[email protected]> Co-authored-by: David Lutzardo <[email protected]> Co-authored-by: Fabio Di Fabio <[email protected]> Co-authored-by: Simon Dudley <[email protected]> Co-authored-by: Gabriel Fukushima <[email protected]> Co-authored-by: delehef <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Co-authored-by: garyschulte <[email protected]> Co-authored-by: Stefan Pingel <[email protected]> Co-authored-by: matkt <[email protected]> Co-authored-by: Matt Whitehead <[email protected]> Co-authored-by: Diego López León <[email protected]>

…hyperledger#6205) revert some modification that was made to pass tests hyperledger#5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]> Signed-off-by: matkt <[email protected]>

* Sequenced pool synonym for legacy pool Signed-off-by: Matthew Whitehead <[email protected]> * Class rename Signed-off-by: Matthew Whitehead <[email protected]> * Spotless fixes Signed-off-by: Matthew Whitehead <[email protected]> * Add SEQUENCED to config overview test Signed-off-by: Matthew Whitehead <[email protected]> * Update CHANGELOG.md Co-authored-by: Sally MacFarlane <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> * add a fix to load correctly the storage trie in the Bonsai WorldState (hyperledger#6205) revert some modification that was made to pass tests hyperledger#5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]> --------- Signed-off-by: Matthew Whitehead <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Co-authored-by: matkt <[email protected]>

…hyperledger#6205) revert some modification that was made to pass tests hyperledger#5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]> Signed-off-by: matkt <[email protected]> Signed-off-by: Gabriel Fukushima <[email protected]>

* Sequenced pool synonym for legacy pool Signed-off-by: Matthew Whitehead <[email protected]> * Class rename Signed-off-by: Matthew Whitehead <[email protected]> * Spotless fixes Signed-off-by: Matthew Whitehead <[email protected]> * Add SEQUENCED to config overview test Signed-off-by: Matthew Whitehead <[email protected]> * Update CHANGELOG.md Co-authored-by: Sally MacFarlane <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> * add a fix to load correctly the storage trie in the Bonsai WorldState (hyperledger#6205) revert some modification that was made to pass tests hyperledger#5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]> --------- Signed-off-by: Matthew Whitehead <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Co-authored-by: matkt <[email protected]> Signed-off-by: Gabriel Fukushima <[email protected]>

…hyperledger#6205) revert some modification that was made to pass tests hyperledger#5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]> Signed-off-by: matkt <[email protected]> Signed-off-by: jflo <[email protected]>

* Sequenced pool synonym for legacy pool Signed-off-by: Matthew Whitehead <[email protected]> * Class rename Signed-off-by: Matthew Whitehead <[email protected]> * Spotless fixes Signed-off-by: Matthew Whitehead <[email protected]> * Add SEQUENCED to config overview test Signed-off-by: Matthew Whitehead <[email protected]> * Update CHANGELOG.md Co-authored-by: Sally MacFarlane <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> * add a fix to load correctly the storage trie in the Bonsai WorldState (hyperledger#6205) revert some modification that was made to pass tests hyperledger#5686 and fix this tests by loading the storage with EMPTY_TRIE_HASH if we detect that it has been cleared before pushing the new slots after recreation. --------- Signed-off-by: Karim TAAM <[email protected]> --------- Signed-off-by: Matthew Whitehead <[email protected]> Signed-off-by: Matt Whitehead <[email protected]> Signed-off-by: Karim TAAM <[email protected]> Co-authored-by: Sally MacFarlane <[email protected]> Co-authored-by: matkt <[email protected]> Signed-off-by: jflo <[email protected]>

garyschulte force-pushed the feature/bonsai-reference-tests branch from 0f761e2 to a72b41a Compare July 10, 2023 16:32

non-fungible-nelson added the TeamChupa GH issues worked on by Chupacabara Team label Jul 11, 2023

garyschulte force-pushed the feature/bonsai-reference-tests branch 4 times, most recently from cab46aa to 3262a10 Compare August 4, 2023 23:23

github-advanced-security bot found potential problems Aug 4, 2023

View reviewed changes

.../main/java/org/hyperledger/besu/ethereum/bonsai/storage/BonsaiWorldStateKeyValueStorage.java Fixed Show fixed Hide fixed

garyschulte force-pushed the feature/bonsai-reference-tests branch from 3262a10 to af9932e Compare August 4, 2023 23:58

garyschulte force-pushed the feature/bonsai-reference-tests branch 2 times, most recently from b4c60e6 to 9c6d247 Compare August 25, 2023 21:10

garyschulte marked this pull request as ready for review August 29, 2023 16:45

garyschulte force-pushed the feature/bonsai-reference-tests branch from b33468e to 95c79c3 Compare August 29, 2023 17:48

garyschulte requested review from shemnon and matkt August 29, 2023 17:50

garyschulte force-pushed the feature/bonsai-reference-tests branch from 95c79c3 to 3ca67ad Compare August 30, 2023 23:50

matkt requested changes Aug 31, 2023

View reviewed changes

garyschulte force-pushed the feature/bonsai-reference-tests branch 4 times, most recently from b64b247 to 366b09e Compare September 6, 2023 15:29

garyschulte force-pushed the feature/bonsai-reference-tests branch from 366b09e to 5262279 Compare September 13, 2023 23:30

garyschulte force-pushed the feature/bonsai-reference-tests branch from 5262279 to 8fb41ee Compare September 14, 2023 00:06

matkt approved these changes Sep 14, 2023

View reviewed changes

garyschulte added 2 commits September 13, 2023 21:00

create a bonsai based reference test worldstate

f7d16f6

Signed-off-by: garyschulte <[email protected]>

fix some tests and initialization

4b6c7a6

Signed-off-by: garyschulte <[email protected]>

garyschulte added 2 commits September 13, 2023 21:00

filter inMemory key value storage rather than takeWhile to avoid sort

27d1e74

Signed-off-by: garyschulte <[email protected]>

rebase, javadoc and a bit of cleanup

a407318

Signed-off-by: garyschulte <[email protected]>

garyschulte force-pushed the feature/bonsai-reference-tests branch from 37bab7e to a407318 Compare September 14, 2023 04:00

garyschulte enabled auto-merge (squash) September 14, 2023 04:07

garyschulte merged commit 4b2ef68 into hyperledger:main Sep 14, 2023

garyschulte deleted the feature/bonsai-reference-tests branch September 14, 2023 04:31

siladu mentioned this pull request Sep 20, 2023

Burn in release of 23.7.3 from main (inclusive of sha eef40bd) #5905

Closed

garyschulte mentioned this pull request Sep 25, 2023

besu evmt8n daemon errors out on test data #5934

Closed

garyschulte mentioned this pull request Nov 18, 2023

Revert "fix tests collisions" #6183

Closed

matkt mentioned this pull request Nov 23, 2023

fix collision issue #6205

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Bonsai based reference test worldstate #5686

Bonsai based reference test worldstate #5686

garyschulte commented Jul 7, 2023 •

edited

Loading

github-actions bot commented Jul 7, 2023

matkt Aug 31, 2023

garyschulte Aug 31, 2023

matkt Sep 11, 2023

matkt Aug 31, 2023

matkt Aug 31, 2023

garyschulte Sep 8, 2023 •

edited

Loading

matkt Sep 11, 2023

matkt commented Sep 11, 2023

garyschulte commented Sep 11, 2023 •

edited

Loading

matkt commented Sep 12, 2023

garyschulte commented Sep 13, 2023

		@@ -79,15 +92,25 @@ public class BonsaiWorldStateKeyValueStorage implements WorldStateStorage, AutoC

		protected final Subscribers<BonsaiStorageSubscriber> subscribers = Subscribers.create();

		final BonsaiPreImageProxy preImageProxy;

Bonsai based reference test worldstate #5686

Bonsai based reference test worldstate #5686

Conversation

garyschulte commented Jul 7, 2023 • edited Loading

PR description

Change Summary:

Discovered/Fixed Defects

Fixed Issue(s)

github-actions bot commented Jul 7, 2023

matkt Aug 31, 2023

Choose a reason for hiding this comment

garyschulte Aug 31, 2023

Choose a reason for hiding this comment

matkt Sep 11, 2023

Choose a reason for hiding this comment

matkt Aug 31, 2023

Choose a reason for hiding this comment

matkt Aug 31, 2023

Choose a reason for hiding this comment

garyschulte Sep 8, 2023 • edited Loading

Choose a reason for hiding this comment

matkt Sep 11, 2023

Choose a reason for hiding this comment

matkt commented Sep 11, 2023

garyschulte commented Sep 11, 2023 • edited Loading

matkt commented Sep 12, 2023

garyschulte commented Sep 13, 2023

garyschulte commented Jul 7, 2023 •

edited

Loading

garyschulte Sep 8, 2023 •

edited

Loading

garyschulte commented Sep 11, 2023 •

edited

Loading