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

fix: tune Besu options to work with tx_spammer #612

Merged
merged 5 commits into from
May 15, 2024

Conversation

fab-10
Copy link
Contributor

@fab-10 fab-10 commented May 14, 2024

At startup Besu does not return a good gas price estimation (WIP to fix this) and the default number of accepted connection is not enough to sustain the tx_spammer, so this PR tune some options to allow to use the tx_spammer wth Besu.

@fab-10 fab-10 force-pushed the tune-besu-for-tx-spammer branch from 9837ce0 to 689d259 Compare May 14, 2024 08:51
@fab-10 fab-10 changed the title Tune Besu options to work with tx_spammer fix: tune Besu options to work with tx_spammer May 14, 2024
@fab-10
Copy link
Contributor Author

fab-10 commented May 14, 2024

@pk910 could you please review?

Co-authored-by: Barnabas Busa <barnabas.busa@ethereum.org>
@fab-10 fab-10 requested a review from barnabasbusa May 14, 2024 09:21
@barnabasbusa barnabasbusa enabled auto-merge (squash) May 14, 2024 12:31
@fab-10
Copy link
Contributor Author

fab-10 commented May 15, 2024

@barnabasbusa @pk910 could you please check why the CI / assertoor check is failing for this PR?

@pk910
Copy link
Member

pk910 commented May 15, 2024

Yea, our testing workflow is currently broken for branches that are not part of this repository.
Right now it tries to find branch tune-besu-for-tx-spammer within this repository, but it's part of your fork, so it can't find it.
We'll get that fixed :)

@pk910
Copy link
Member

pk910 commented May 15, 2024

PR workflow issue with assertoor got fixed with #616
Sorry for the delay ;)

@barnabasbusa barnabasbusa merged commit b395189 into ethpandaops:main May 15, 2024
12 checks passed
barnabasbusa pushed a commit that referenced this pull request Jun 10, 2024
🤖 I have created a release *beep* *boop*
---


##
[3.1.0](3.0.0...3.1.0)
(2024-06-07)


### Features

* add http url to el context
([#656](#656))
([4e69a4c](4e69a4c))
* add prefunded accounts to output
([#657](#657))
([bc06e2a](bc06e2a))
* add tracoor
([#651](#651))
([b100cb6](b100cb6))
* add vc_count to increase the number of validators per participant
([#633](#633))
([4272ff3](4272ff3))
* allow setting custom dora image & env variables
([#623](#623))
([08a65c3](08a65c3))
* **apache:** Serve all config files
([#606](#606))
([3f1f5e1](3f1f5e1))
* **config:** add peerdas vars
([#619](#619))
([22f1498](22f1498))
* expose network-params
([#659](#659))
([b0820dd](b0820dd))
* forky
([#625](#625))
([ded68bd](ded68bd))
* Support participants_matrix
([#620](#620))
([3a57467](3a57467))
* use `peer-das` image for dora when eip7594 is active
([#593](#593))
([1b4bd3d](1b4bd3d))


### Bug Fixes

* add additional prefund addresses
([#655](#655))
([6d2cdb6](6d2cdb6))
* add cl log level to builders
([#638](#638))
([ad46dbd](ad46dbd))
* Add EIP-7002 & EIP-2935 bytecode to ethereum-genesis-generator
([#597](#597))
([3d316ef](3d316ef))
* add http to teku endpoint
([#622](#622))
([085b6e1](085b6e1))
* add peer_das_epoch to egg
([#603](#603))
([91694df](91694df))
* add sha256 as an image label (if present)
([#637](#637))
([3dcf888](3dcf888))
* add static port config for apache
([#608](#608))
([b96e502](b96e502))
* **apache:** only set static port if wanted
([#610](#610))
([2c6b7b1](2c6b7b1))
* blockscout fix for json variant
([#662](#662))
([e79c510](e79c510))
* churn adjustments
([#614](#614))
([12ca872](12ca872))
* default config
([#632](#632))
([14be117](14be117))
* drop everythign after [@sha](https://github.com/sha) from image labels
([#636](#636))
([5d35463](5d35463))
* erigon v3 - new default image
([#629](#629))
([72cf150](72cf150))
* genesis generator bump
([#611](#611))
([5460f6f](5460f6f))
* nightly tests
([#595](#595))
([76c31e9](76c31e9))
* pectra example
([#605](#605))
([67e3da0](67e3da0))
* prysm vc key manager ports
([#639](#639))
([81c1ee7](81c1ee7))
* re-add images to labels
([#634](#634))
([71f6e28](71f6e28))
* README has invalid configs
([#631](#631))
([e33b971](e33b971))
* readme indentation
([#600](#600))
([583db1b](583db1b))
* registration flags when using beacon node only
([#618](#618))
([c12506b](c12506b))
* repair check workflow for external PRs
([#616](#616))
([a584682](a584682))
* seperate vc service names
([#654](#654))
([a5ffe14](a5ffe14))
* tune Besu options to work with tx_spammer
([#612](#612))
([b395189](b395189))
* update dora images
([#598](#598))
([dd28d61](dd28d61))
* update prysm image
([#599](#599))
([0a38114](0a38114))
* use `electra-support` image for assertoor when electra fork epoch is
set
([#607](#607))
([cdeab93](cdeab93))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants