Skip to content

fix: increase minFeePadding in e2e_bot bridge resume tests#20945

Open
AztecBot wants to merge 1 commit intomerge-train/spartanfrom
claudebox/843b28c5543ca12c-1
Open

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

Conversation

@AztecBot
Copy link
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

… flaky gas fee failures

Gas fees can escalate >10x during rapid block building in the test environment.
The previous 10x padding (minFeePadding: 9) was insufficient — the failing test
saw fees rise ~13.4x between estimation and processing. Increased to 100x
(minFeePadding: 99) for both bridge resume tests, with the second test being
especially susceptible because changing salt triggers additional bridge operations.
@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
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.

2 participants