Skip to content

fix: increase minFeePadding in e2e_bot bridge resume tests#20945

Closed
AztecBot wants to merge 1 commit into
nextfrom
claudebox/843b28c5543ca12c-1
Closed

fix: increase minFeePadding in e2e_bot bridge resume tests#20945
AztecBot wants to merge 1 commit into
nextfrom
claudebox/843b28c5543ca12c-1

Conversation

@AztecBot

Copy link
Copy Markdown
Collaborator

Summary

Fixes flaky e2e_bot test failure on merge-train/spartan caused by gas fee escalation beyond the 10x padding.

  • The does not reuse prior bridge claims if recipient address changes test was failing because gas fees rose ~13.4x between fee estimation and transaction processing
  • The previous minFeePadding: 9 (10x multiplier) added in chore: increase max fee bots use in tests #20867 was insufficient for this test, which builds many blocks due to bridge operations from earlier tests plus its own additional bridge claim
  • Increased minFeePadding from 9 to 99 (100x) for both bridge resume tests to handle worst-case fee volatility

Error: maxFeesPerGas.feePerL2Gas must be greater than or equal to gasFees.feePerL2Gas, but got maxFeesPerGas.feePerL2Gas=698400000 and gasFees.feePerL2Gas=932800000

Test plan

  • Change is purely numeric (9 → 99) in test config, no compilation risk
  • CI will verify the test passes with the increased padding

ClaudeBox log

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Feb 27, 2026
@PhilWindle PhilWindle marked this pull request as ready for review February 27, 2026 10:04
@PhilWindle PhilWindle enabled auto-merge (squash) February 27, 2026 10:05
@AztecBot AztecBot changed the base branch from merge-train/spartan to next March 3, 2026 21:02
@AztecBot AztecBot force-pushed the claudebox/843b28c5543ca12c-1 branch from 5e30055 to d290ee2 Compare March 3, 2026 21:04
@ludamad ludamad closed this Mar 3, 2026
auto-merge was automatically disabled March 3, 2026 21:08

Pull request was closed

@ludamad ludamad reopened this Mar 3, 2026
@AztecBot AztecBot closed this Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-to-v4 claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants