1208 Commits

Author SHA1 Message Date
Justin Florentine
f4dc48d94d Simplifying BesuCommand step 1 (#7682)
Simplifying BesuCommand
---------

Signed-off-by: Justin Florentine <justin+github@florentine.us>
2024-10-02 15:46:34 +00:00
Fabio Di Fabio
1640a29bbf Merge branch 'main' into zkbesu
# Conflicts:
#	.github/workflows/draft-release.yml
2024-10-02 10:08:21 +02:00
Gabriel-Trintinalia
49bf37cc31 Expose chain ID in the BlockchainService plugin API (#7702)
Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
2024-10-02 02:45:26 +00:00
Fabio Di Fabio
badb334812 Merge branch 'main' into zkbesu 2024-09-30 08:59:16 +02:00
Jason Frame
23ad2b58b1 Fix operator generator command excluding alloc field when generating config (#7684)
Signed-off-by: Jason Frame <jason.frame@consensys.net>
2024-09-30 10:05:24 +10:00
Fabio Di Fabio
98e09d11e5 Merge branch 'main' into zkbesu 2024-09-26 11:45:37 +02:00
amsmota
aed6bb0044 Implementing support for emptyBlockPeriodSeconds in QBFT (Issue #3810) (#6965)
Implemented support for emptyBlockPeriodSeconds in QBFT (Issue #3810)

Introduces experimental xemptyblockperiodseconds genesis config option for producing empty blocks at a specific interval independently of the value of the existing blockperiodseconds setting.

https://github.com/hyperledger/besu/issues/3810

---------

Signed-off-by: Antonio Mota <antonio.mota@citi.com>
Signed-off-by: amsmota <amsmota@gmail.com>
2024-09-24 11:35:08 +00:00
Gabriel-Trintinalia
e0518c6d94 Force besu to stop on plugin initialization errors (#7662)
Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
2024-09-24 10:50:19 +00:00
Matilda-Clerke
4f07e76a6c 7311: Add feature toggle for enabling use of the peertask system where available (#7633)
Signed-off-by: Matilda Clerke <matilda.clerke@consensys.net>
2024-09-24 13:04:03 +10:00
Fabio Di Fabio
b6d587f174 Merge branch 'main' into zkbesu 2024-09-23 17:11:35 +02:00
Rafael Matias
0d63955158 Docker: Only switch user if the current user is root (#7654)
* Update entrypoint script for Dockerfile to only switch user if its running as root

Signed-off-by: Rafael Matias <rafael@skyle.net>

* make root user check at the beginning

Signed-off-by: Rafael Matias <rafael@skyle.net>

---------

Signed-off-by: Rafael Matias <rafael@skyle.net>
2024-09-23 12:34:20 +00:00
Fabio Di Fabio
3e0e5cdc1f Fix reading tx-pool-min-score option from configuration file (#7623)
Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
2024-09-20 15:11:56 +00:00
Matt Whitehead
19d3ca84b2 Dev/test option for short BFT block periods (#7588)
* Dev mode for short BFT block periods

Signed-off-by: Matthew Whitehead <matthew1001@gmail.com>

* Refactoring

Signed-off-by: Matthew Whitehead <matthew1001@gmail.com>

* Fix comment

Signed-off-by: Matthew Whitehead <matthew1001@gmail.com>

* Refactor to make BFT block milliseconds an experimental QBFT config option

Signed-off-by: Matthew Whitehead <matthew1001@gmail.com>

* Update Json BFT config options

Signed-off-by: Matthew Whitehead <matthew1001@gmail.com>

---------

Signed-off-by: Matthew Whitehead <matthew1001@gmail.com>
2024-09-20 10:12:11 +01:00
Justin Florentine
7d3e376771 shift creation of plugin context to BesuCommand for now (#7625)
* shift creation of plugin context to BesuCommand for now
* mock component will provide a no-op metrics sys

---------

Signed-off-by: Justin Florentine <justin+github@florentine.us>
2024-09-18 14:31:03 +00:00
Fabio Di Fabio
0e21ab06c1 Merge branch 'main' into zkbesu 2024-09-18 16:02:04 +02:00
Bhanu Pulluri
0d9fa16a1f [#4292] Fix mounted data path directory permissions for besu user (#7575)
* Fix mounted data path directory permissions for besu user

Signed-off-by: Bhanu Pulluri <bhanu.pulluri@kaleido.io>

* Add besu CLI option to output dirs needing permission update

Signed-off-by: Bhanu Pulluri <bhanu.pulluri@kaleido.io>

* run spotless apply to handle PR test failure

Signed-off-by: Bhanu Pulluri <bhanu.pulluri@kaleido.io>

* Remove newly added --print-paths-and-exit option from config file test

This option doesn't have a corresponding config file entry as it's a
standalone option to be used with docker containers

Signed-off-by: Bhanu Pulluri <bhanu.pulluri@kaleido.io>

* Add optional user argument to --print-paths-and-exit and fix directory permissions

Signed-off-by: Bhanu Pulluri <bhanu.pulluri@kaleido.io>

* Correct build.gradle changes, remove a duplicate line and extra whitespaces

Signed-off-by: Bhanu Pulluri <bhanu.pulluri@kaleido.io>

* Fix checking for user in path's group membership

Signed-off-by: Bhanu Pulluri <bhanu.pulluri@kaleido.io>

* Add platform check to restrict --print-paths-and-exit option usage to Linux and Mac

Signed-off-by: Bhanu Pulluri <bhanu.pulluri@kaleido.io>

* Apply suggestions from code review

Co-authored-by: Fabio Di Fabio <fabio.difabio@consensys.net>
Signed-off-by: Bhanu Pulluri <59369753+pullurib@users.noreply.github.com>

---------

Signed-off-by: Bhanu Pulluri <bhanu.pulluri@kaleido.io>
Signed-off-by: Bhanu Pulluri <59369753+pullurib@users.noreply.github.com>
Co-authored-by: Bhanu Pulluri <bhanu.pulluri@kaleido.io>
Co-authored-by: Fabio Di Fabio <fabio.difabio@consensys.net>
2024-09-18 09:35:00 +00:00
Fabio Di Fabio
33085dd233 Fix flakiness ofTxPoolOptionsTest::txpoolForcePriceBumpToZeroWhenZeroBaseFeeMarket (#7610)
Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
2024-09-18 13:35:06 +10:00
Fabio Di Fabio
241c9b8c1e Merge branch 'main' into zkbesu 2024-09-16 17:24:15 +02:00
Sally MacFarlane
89dfa95860 [CHANGELOG] upcoming breaking change for k8s nat (#7534)
* upcoming breaking change for k8s nat
* log warning if deprecated NAT method used

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

---------

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
2024-09-16 13:26:14 +10:00
ITStarMan100
12caf7d83c Added GraphQL Service Information in responese result NetServices (#7580)
Signed-off-by: ITStarMan100 <danhopeman@outlook.com>
Co-authored-by: Justin Florentine <justin+github@florentine.us>
Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com>
2024-09-16 02:31:05 +00:00
Fabio Di Fabio
31c174b0c6 Revert "Re-enable TxPoolOptionsTest::txpoolForcePriceBumpToZeroWhenZeroBaseFeeMarket (#7565)" (#7613)
This reverts commit 07adb415ee.

Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
2024-09-13 06:43:08 -04:00
Fabio Di Fabio
7b249a6a81 Merge branch 'main' into zkbesu
# Conflicts:
#	.github/workflows/release.yml
2024-09-11 11:05:45 +02:00
Fabio Di Fabio
07adb415ee Re-enable TxPoolOptionsTest::txpoolForcePriceBumpToZeroWhenZeroBaseFeeMarket (#7565)
Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
2024-09-11 08:26:17 +10:00
Fabio Di Fabio
6f74319474 Merge branch 'main' into zkbesu 2024-09-10 13:58:39 +02:00
ITStarMan100
36454b40d5 Added help for setting up Stratum mining (#7584)
* Added help for setting up Stratum mining

Signed-off-by: ITStarMan100 <danhopeman@outlook.com>

* removed duplication from help message; fixed spotless

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

---------

Signed-off-by: ITStarMan100 <danhopeman@outlook.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com>
2024-09-10 01:34:45 +00:00
Fabio Di Fabio
75a540175d Merge branch 'main' into zkbesu
# Conflicts:
#	.github/workflows/acceptance-tests.yml
#	.github/workflows/pre-review.yml
#	build.gradle
2024-09-06 16:30:57 +02:00
daniellehrner
8eee569887 7702 for devenet-3 (#7444)
* wrapped WorldUpdater into `EVMWorldupdater` to remove the authority code injection from the implementation of the actual world updaters

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* add CHANGELOG entry

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* first draft for 7702 v2

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* change return value of DelegatedCodeGasCostHelper

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* fix upfront gas cost calculation, fix setting code multiple times in MutableDelegatedCodeAccount

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* fix gas refund for delegated code when account already exists, added gas cost deduction for code delegation resolution to ExtCodeSizeOperation

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* allow accounts with delegated code to send transactions

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* add refund for already existing account after nonce check

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* resolve delegated code only the first time to avoid delegation loops

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* handle invalid authorization signatures properly

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* refactored CodeDelegationProcessor to compute authorizer of a code delegation after the chain id check

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* fix canSetDelegatedCode method by checking code how it is in the trie and not the resolved code

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* optimize code hash calculation for empty code, fix check for empty code delegation list

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* check the all code delegation signatures hava a valid s value

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* rename encoder & decoder, handle invalid signature values in T8nExecutor

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* create the signatures for code delegation authorizations in T8nExecutor without checking if they are valid to test them later during the tx execution

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* check that recid is either 0 or 1

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* fixed acceptance tests, renamed the the remaining instances of set code to code delegation

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* fix delegate encoder & encoder unit tests

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* spotless

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* changed empty check for delegated accounts, fixed test

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* fix hasDelegatedCode method when code is null

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* run acceptance tests without deamon

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* disable CodeDelegationTransactionAcceptanceTest to check if it is causing the stuck ci pipeline

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* check if shouldTransferAllEthOfAuthorizerToSponsor is causing pipeline to stall

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* check if shouldCheckNonceAfterNonceIncreaseOfSender is causing pipeline to stall

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* check if closing the cluster after every test is causing pipeline to stall

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* spotless

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

---------

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>
Co-authored-by: Danno Ferrin <danno@numisight.com>
2024-09-06 12:04:41 +02:00
Fabio Di Fabio
f0d2a665d7 Remove tx from pool when its score is lower than a configured value (#7576)
* Remove tx from pool when its score is lower than a configured value

Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>

* Update besu/src/main/java/org/hyperledger/besu/cli/options/TransactionPoolOptions.java

Co-authored-by: Stefan Pingel <16143240+pinges@users.noreply.github.com>
Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>

* Check for below min score after the penalization

Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>

---------

Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
Co-authored-by: Stefan Pingel <16143240+pinges@users.noreply.github.com>
2024-09-05 15:08:20 +00:00
Justin Florentine
563afdb3ec Dagger plugin context (#7465)
* Eliminates dependency from Controller on BesuComponent

---------

Signed-off-by: Justin Florentine <justin+github@florentine.us>
Co-authored-by: garyschulte <garyschulte@gmail.com>
2024-09-03 22:40:57 +00:00
Justin Florentine
d5c10e2b4e Dagger controller tests (#7567)
Separate out flex group privacy test since it requires differently configured dependencies.
test specific, inline module/component definitions
reusable module for providing mockable enclaves, besu commands, and genesis
dagger support for transaction simulation
dagger support for plugin setup

---------

Signed-off-by: jflo <justin+github@florentine.us>
Signed-off-by: Justin Florentine <justin+github@florentine.us>
2024-09-03 12:43:51 -04:00
Fabio Di Fabio
af80dfbd86 Merge branch 'main' into zkbesu
# Conflicts:
#	crypto/algorithms/build.gradle
#	crypto/services/build.gradle
#	datatypes/build.gradle
#	ethereum/rlp/build.gradle
#	evm/build.gradle
#	plugin-api/build.gradle
#	util/build.gradle
2024-09-02 12:25:58 +02:00
Matilda-Clerke
da98fa5541 Reintroduce engine get client version v1 with commit in manifest (#7548)
* Include Commit-Hash in manifests
* Add commit method in BesuInfo
* Use 8 character hash in EngineGetClientVersionV1 and revert calculateVersion back to original spec

Signed-off-by: Matilda Clerke <matilda.clerke@consensys.net>
Co-authored-by: garyschulte <garyschulte@gmail.com>
2024-08-30 18:13:53 +00:00
Fabio Di Fabio
bff864b096 Merge branch 'main' into zkbesu 2024-08-30 17:23:25 +02:00
Matilda-Clerke
1ea1d4cede Revert engine getclientversionv1 changes (#7546)
* Revert "Check for commit hash before appending another in build.gradle caclulateVersion (#7537)"

This reverts commit a8bbcd5d8b.

Signed-off-by: Matilda Clerke <matilda.clerke@consensys.net>

* Revert engine_getClientVersionV1 changes

Signed-off-by: Matilda Clerke <matilda.clerke@consensys.net>

---------

Signed-off-by: Matilda Clerke <matilda.clerke@consensys.net>
2024-08-30 06:01:43 +00:00
Suyash Nayan
892846aeab Add qbft_getRequestTimeoutSeconds RPC Method (#7521)
* Add qbft_getRequestTimeoutSeconds RPC Method

Signed-off-by: 7suyash7 <suyashnyn1@gmail.com>
2024-08-30 15:08:34 +10:00
Rodion
03cdd45916 Drop messages that exceeds local message size limit (#7507)
Signed-off-by: Rodion Lim <rodion.lim@hotmail.com>
Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com>
2024-08-29 06:09:01 +00:00
Fabio Di Fabio
0e16693e1b Merge branch 'main' into zkbesu
# Conflicts:
#	build.gradle
2024-08-27 15:05:02 +02:00
Matilda-Clerke
e0aa4f6d4a add engine_getClientVersionV1 (#7512)
* add engine_getClientVersionV1

Signed-off-by: Matilda Clerke <matilda.clerke@consensys.net>
2024-08-27 04:49:12 +00:00
garyschulte
d87650b944 Fleet-mode safe behavior for fcU in SynchronizationService (#7517)
* fleet-mode safe behavior for fcU in SynchronizationService

Signed-off-by: garyschulte <garyschulte@gmail.com>

* spotless

Signed-off-by: garyschulte <garyschulte@gmail.com>

---------

Signed-off-by: garyschulte <garyschulte@gmail.com>
2024-08-23 16:35:18 -07:00
Fabio Di Fabio
80c7e4db48 Merge branch 'main' into zkbesu
# Conflicts:
#	acceptance-tests/dsl/src/main/java/org/hyperledger/besu/tests/acceptance/dsl/node/ThreadBesuNodeRunner.java
2024-08-21 10:19:57 +02:00
Sally MacFarlane
c555775c1f Refactor how genesis file overrides are applied (#7489)
* added tests that fail
* separate getConfigOptions() logic from applying overrides

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

---------

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
2024-08-20 23:50:07 +00:00
Fabio Di Fabio
fd077a7273 Revert "Dagger controller tests (#7341)" (#7497)
This reverts commit 38a025b870.

# Conflicts:
#	acceptance-tests/dsl/src/main/java/org/hyperledger/besu/tests/acceptance/dsl/node/ThreadBesuNodeRunner.java

Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
2024-08-20 23:24:46 +00:00
Fabio Di Fabio
74a56f5538 Revert "Dagger controller tests (#7341)"
This reverts commit 38a025b870.

# Conflicts:
#	acceptance-tests/dsl/src/main/java/org/hyperledger/besu/tests/acceptance/dsl/node/ThreadBesuNodeRunner.java
2024-08-20 16:51:16 +02:00
Fabio Di Fabio
22fab53366 Merge branch 'main' into zkbesu 2024-08-19 10:48:23 +02:00
Jason Frame
4acd7f120c Change default for receipt compaction to be enabled (#7450)
Signed-off-by: Jason Frame <jason.frame@consensys.net>
2024-08-19 05:09:38 +00:00
Gabriel-Trintinalia
68db5e8d55 Merge branch 'main' into zkbesu
Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
2024-08-16 16:44:07 +10:00
Gabriel-Trintinalia
d7041d4221 CLI option for disabling auto-registration of external plugins (#7470)
Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
2024-08-16 15:09:31 +10:00
Justin Florentine
ad19d4c7ff isolates MetricsOptionGroup for future reusability, minor renaming (#7464)
* isolates MetricsOptionGroup for future reusability, minor renaming


---------

Signed-off-by: Justin Florentine <justin+github@florentine.us>
2024-08-15 06:41:38 +10:00
Suyash Nayan
adf19afb1a Correct default --help values (#7454)
* Fix #7367: Correct default value display for --rpc-max-trace-filter-range along with a test
* Fix #7369: Show default value for --genesis-state-hash-cache-enabled

Signed-off-by: 7suyash7 <suyashnyn1@gmail.com>
2024-08-14 05:06:24 +00:00
Danno Ferrin
a55c331e21 Add test coverage for java precompiles (#7446)
For tests that have a native/java switch ensure that the java path gets
the same tests native paths do.

Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Danno Ferrin <danno@numisight.com>
2024-08-13 18:21:49 +00:00