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: besu bonsai log disable #673

Merged
merged 1 commit into from
Jun 14, 2024
Merged

fix: besu bonsai log disable #673

merged 1 commit into from
Jun 14, 2024

Conversation

barnabasbusa
Copy link
Contributor

No description provided.

Signed-off-by: Barnabas Busa <busa.barnabas@gmail.com>
@barnabasbusa barnabasbusa enabled auto-merge (squash) June 14, 2024 07:18
@barnabasbusa barnabasbusa merged commit 955f19f into main Jun 14, 2024
57 checks passed
@barnabasbusa barnabasbusa deleted the barnabasbusa-patch-1 branch June 14, 2024 07:57
@skylenet
Copy link
Member

@barnabasbusa isn't this the default behaviour on Teku? I think this flag has to be true if you want to limit and prune tree logs: https://besu.hyperledger.org/public-networks/how-to/bonsai-limit-trie-logs#:~:text=To%20decrease%20the%20database%20size,GB%20each%20week%20on%20Mainnet.

I'm not sure what your intention is just based on the title of the PR.

@barnabasbusa
Copy link
Contributor Author

The point isn't to reduce the log size, the flag is set so that full sync would still work with bonsai post their latest update. And its besu not teku :D

@barnabasbusa
Copy link
Contributor Author

barnabasbusa pushed a commit that referenced this pull request Jul 3, 2024
🤖 I have created a release *beep* *boop*
---


##
[4.1.0](4.0.0...4.1.0)
(2024-07-03)


### Features

* add back k8s tests
([#699](#699))
([d621cf0](d621cf0))
* add checkpoint_enabled and checkpoint_url flags
([#689](#689))
([b8cd2b4](b8cd2b4))
* add eof support
([#682](#682))
([cb203ff](cb203ff))
* add mev relays to dora config
([#679](#679))
([293001a](293001a))
* Add static ports
([#677](#677))
([4f054d0](4f054d0))
* add ws_url to el_context
([#696](#696))
([26fea61](26fea61))
* introduce devnet_repo override
([#686](#686))
([9952361](9952361))
* use CDN URL for data snapshots used for shadow forks
([#676](#676))
([91dc68c](91dc68c))


### Bug Fixes

* besu bonsai log disable
([#673](#673))
([955f19f](955f19f))
* blockscout bad return
([#685](#685))
([e80870b](e80870b))
* bump peerdas images
([#678](#678))
([1acc201](1acc201))
* DNS-1035 label
([#697](#697))
([440fb31](440fb31))
* ephemery genesis loader
([#700](#700))
([0235063](0235063))
* ignore bootnodes if in shadowfork
([#660](#660))
([cda5dda](cda5dda))
* minimal eof
([#687](#687))
([26a7618](26a7618))
* release please manifest
([#675](#675))
([75ed7e1](75ed7e1))
* remove docker login
([#701](#701))
([ede5962](ede5962))
* return empty services
([#688](#688))
([6571a70](6571a70))
* return the correct network_id
([#705](#705))
([7c592f6](7c592f6))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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.

2 participants