Skip to content

chore: add e2e_deploy_contract/deploy_method.test.ts to flaky test patterns#21356

Draft
AztecBot wants to merge 1 commit intomerge-train/spartanfrom
claudebox/fix-merge-train-spartan-ci
Draft

chore: add e2e_deploy_contract/deploy_method.test.ts to flaky test patterns#21356
AztecBot wants to merge 1 commit intomerge-train/spartanfrom
claudebox/fix-merge-train-spartan-ci

Conversation

@AztecBot
Copy link
Collaborator

@AztecBot AztecBot commented Mar 11, 2026

Summary

  • Adds e2e_deploy_contract/deploy_method.test.ts to .test_patterns.yml as a known flaky test
  • This test failed on merge-train/spartan CI (log) despite the only code change being adding AZTEC_SLOT_DURATION=36 to a spartan env file (PR chore: revert l2 slot time 72 -> 36 on scenario network #21291), confirming it's a flaky failure unrelated to any code change
  • The test contains timing-sensitive logic (deploying and calling a contract in the same block with priority fees) that can intermittently fail

Context

CI failure on commit 2270423d10 blocked merge-train/spartan PR #21248. The previous commit 4f980f1e0b passed CI cleanly.

ClaudeBox log: https://claudebox.work/s/216e5fc2243f1449?run=1

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 11, 2026
@AztecBot AztecBot force-pushed the claudebox/fix-merge-train-spartan-ci branch from 689d65d to 4278757 Compare March 11, 2026 13:38
@AztecBot AztecBot changed the title fix(p2p): stabilize flaky rate limit test in reqresp fix: remove obsolete KVArchiverDataStore constructor arg in test Mar 11, 2026
@AztecBot AztecBot force-pushed the claudebox/fix-merge-train-spartan-ci branch from 4278757 to c376ade Compare March 11, 2026 15:24
@AztecBot AztecBot changed the title fix: remove obsolete KVArchiverDataStore constructor arg in test chore: add e2e_deploy_contract/deploy_method.test.ts to flaky test patterns Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant