-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[txmgr] Ensure that fee is increased by at least 10% #8747
[txmgr] Ensure that fee is increased by at least 10% #8747
Conversation
WalkthroughWalkthroughThe update pertains to the transaction manager component, involving a refinement in the gas price calculation logic. A new constant has been introduced to fine-tune the threshold value used in the calculation. Additionally, the test suite has been adjusted to align with the updated logic, changing the expected outcomes for gas price limit scenarios. Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on X ? TipsChat with CodeRabbit Bot (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, that's indeed the proper way to do it!
edec19f
Description
#8713 fixed a
txmgr
bug where if the fees were < 10, the fee would not be bumped at all.This PR ensures we conform to the spec a little more, guaranteeing that we bump the fee by at least 10%.
For example, the current implementation will bump
11
to12
, which is not technically 10% more (because we dofloor(fee * 1.1)
. This PR switches to useceil(fee * 1.1)
.Tests
Updated tests.
Additional context
go-ethereum currently rounds down internally, so this would not affect L1s running geth.